chore(tinyagents): align v2.1 and reconcile migration plan#5130
Conversation
📝 WalkthroughWalkthroughThis change consolidates TinyAgents migration planning, updates drift and historical status documentation, redirects host references, adjusts a ChangesTinyAgents migration alignment
Estimated code review effort: 2 (Simple) | ~15 minutes Possibly related PRs
Suggested labels: Suggested reviewers: Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Comment |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: d4df050fe5
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
…ration-2026-07-22 # Conflicts: # vendor/tinyflows
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Resolved conflicts in: - app/src-tauri/Cargo.lock: windows-sys version bump (0.59.0 → 0.60.2) - docs/tinycortex-memory-migration-plan.md: deletion-ledger wording - docs/tinycortex-migration-spec.md: migration status description - vendor/tinycortex: submodule tracking (→ 55bf066)
There was a problem hiding this comment.
Actionable comments posted: 3
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs/tinyagents-full-migration-plan/C4-journal-progress-parity-plan.md`:
- Around line 6-7: Align the migration plan’s deletion ownership and parity
terminology: update the S6 prerequisite statement and the S5/S6 ownership
assignments so `ProviderDelta` and `progress_tracing` have consistent deletion
ownership, then revise the §5 gate to use “journal projection parity” instead of
“V3 projection parity.”
In `@docs/tinyagents-migration-plan-2026-07-22.md`:
- Line 366: Remove the trailing space before the closing backtick in the inline
grep command under the **Exit** criterion, leaving the command content otherwise
unchanged.
In `@src/openhuman/inference/README.md`:
- Around line 16-17: Update the provider schema reference in the README text to
use the complete repository path src/openhuman/config/schema/cloud_providers.rs,
while leaving the credentials storage reference unchanged.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: b12e7f13-85d9-4a5b-9a90-cc8d5cb947e1
📒 Files selected for processing (14)
app/src-tauri/Cargo.tomldocs/tinyagents-drift-ledger.mddocs/tinyagents-full-migration-plan/99-deletion-ledger.mddocs/tinyagents-full-migration-plan/C4-journal-progress-parity-plan.mddocs/tinyagents-inference-migration-plan.mddocs/tinyagents-migration-plan-2026-07-22.mddocs/tinyagents-phase3-router-registry-design.mddocs/tinyagents-port-plan.mdsrc/openhuman/agent/harness/subagent_runner/mod.rssrc/openhuman/agent_orchestration/mod.rssrc/openhuman/inference/README.mdsrc/openhuman/tinyagents/subagent_graph.rssrc/openhuman/tools/impl/system/node_exec.rsvendor/tinyflows
💤 Files with no reviewable changes (1)
- app/src-tauri/Cargo.toml
| continuation notes, and it is the gated prerequisite for **S6 below** — | ||
| deleting the `ProviderDelta` bridge and `progress_tracing`. |
There was a problem hiding this comment.
🎯 Functional Correctness | 🟠 Major | ⚡ Quick win
Align the S5/S6 deletion ownership and parity gate.
Line 6-7 says S6 deletes both ProviderDelta and progress_tracing, but Lines 173-178 assign progress_tracing to S5 and ProviderDelta to S6. Also, the updated “journal projection parity” wording at Lines 174-175 leaves the §5 gate at Lines 182-183 referring to “V3 projection parity.” Update these cross-references before this document governs deletion work.
Also applies to: 174-178
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@docs/tinyagents-full-migration-plan/C4-journal-progress-parity-plan.md`
around lines 6 - 7, Align the migration plan’s deletion ownership and parity
terminology: update the S6 prerequisite statement and the S5/S6 ownership
assignments so `ProviderDelta` and `progress_tracing` have consistent deletion
ownership, then revise the §5 gate to use “journal projection parity” instead of
“V3 projection parity.”
| `agent/harness/session/turn_tests.rs` / `subagent_runner/ops_tests.rs` to | ||
| the tinyagents path before deletion (failing-before/passing-after). | ||
|
|
||
| **Exit:** one turn engine; `grep -rn "OPENHUMAN_AGENT_GRAPH_" ` returns only |
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
Remove the space inside the code span.
The trailing space in `grep -rn "OPENHUMAN_AGENT_GRAPH_" ` violates MD038 and makes the inline command malformed.
🧰 Tools
🪛 markdownlint-cli2 (0.23.0)
[warning] 366-366: Spaces inside code span elements
(MD038, no-space-in-code)
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@docs/tinyagents-migration-plan-2026-07-22.md` at line 366, Remove the
trailing space before the closing backtick in the inline grep command under the
**Exit** criterion, leaving the command content otherwise unchanged.
Source: Linters/SAST tools
| The current matrix is `config/schema/cloud_providers.rs`; credentials are | ||
| stored under `provider:<slug>` in the auth-profile store. |
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
Use the complete repository path for the provider schema.
The authoritative file is src/openhuman/config/schema/cloud_providers.rs; config/schema/cloud_providers.rs omits the src/openhuman/ prefix and sends readers to an invalid or ambiguous location.
Suggested correction
- The current matrix is `config/schema/cloud_providers.rs`; credentials are
+ The current matrix is `src/openhuman/config/schema/cloud_providers.rs`; credentials are📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| The current matrix is `config/schema/cloud_providers.rs`; credentials are | |
| stored under `provider:<slug>` in the auth-profile store. | |
| The current matrix is `src/openhuman/config/schema/cloud_providers.rs`; credentials are | |
| stored under `provider:<slug>` in the auth-profile store. |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@src/openhuman/inference/README.md` around lines 16 - 17, Update the provider
schema reference in the README text to use the complete repository path
src/openhuman/config/schema/cloud_providers.rs, while leaving the credentials
storage reference unchanged.
1 participant
Summary
upstream/mainfrom compiling.Problem
finish_flow_run_rowsignature left one resume failure path uncompilable.Solution
flow_idthrough the compatibility-rejection finalization path.Submission Checklist
Impact
BarrierRelieffor later migration work.Related
docs/tinyagents-migration-plan-2026-07-22.md.AI Authored PR Metadata (required for Codex/Linear PRs)
Linear Issue
Commit & Branch
feat/tinyagents-migration-2026-07-22d4df050feValidation Run
cargo test --all-features(1,200 unit tests plus integration/doc tests); TinyFlowscargo check --locked.GGML_NATIVE=OFF cargo check; slim--no-default-features --features tokenjuice-treesittercheck.Validation Blocked
command:GGML_NATIVE=OFF cargo check --manifest-path app/src-tauri/Cargo.tomlerror:local CMake 3.19.1; CEF requires CMake 3.21 or newer.impact:no code diagnostic was emitted before CEF setup stopped; GitHub Actions supplies the authoritative app-world check.Behavior Changes
Parity Contract
Duplicate / Superseded PR Handling
Summary by CodeRabbit
Documentation
Bug Fixes
Chores