feat(worktrees): add exact agent routing and classified navigation - #431
Conversation
Route implicit state from Git managers to the selected worktree, preserve frozen CLI/MCP surfaces, and align IssueOps architecture links with main (#448).
styler-ai
changed the title
|
@codex review |
![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: c325958545
ℹ️ 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".
Exercise the joint classified worktree agent flow explicitly in RC verification and every hosted CI platform lane.
|
@codex review exact head 22dd9f8. Please perform a fresh whole-PR review of architecture and ownership, the complete Rust and SQLite pattern fit, OpenSpec fulfillment, positive/negative/failure/concurrency/compatibility/platform coverage, intended-scale performance, and the v0.4.5-rc1 boundaries. Do not rely on earlier-head review conclusions. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 22dd9f82cf
ℹ️ 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".
|
@codex review exact head 867ad9d. Please review the complete PR at this exact head for correctness, regressions, architecture and ownership, the applicable Rust and SQLite pattern fit, full OpenSpec task fulfillment, positive/negative/failure/concurrency/compatibility coverage, cross-platform behavior, intended-scale performance, agent-first workflow boundaries, and RC1 readiness. Keep any partially implemented or partially proven task incomplete. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 867ad9db71
ℹ️ 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".
|
@codex review exact head 348a3e4. Please perform a fresh whole-PR review for correctness, regressions, architecture and ownership, the applicable Rust and SQLite pattern fit, complete OpenSpec fulfillment, positive/negative/failure/concurrency/compatibility coverage, cross-platform behavior, intended-scale performance, agent-first boundaries, and v0.4.5-rc1 readiness. This head removes the failed-initial-sync fallback registration; keep any partially proven task incomplete. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 348a3e4f39
ℹ️ 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".
|
@codex review exact head 14be3dd. Please perform a fresh whole-PR review for correctness, regressions, architecture and ownership, the applicable Rust and SQLite pattern fit, complete OpenSpec fulfillment, positive/negative/failure/concurrency/compatibility coverage, cross-platform behavior, intended-scale performance, agent-first boundaries, and v0.4.5-rc1 readiness. This head makes final aggregate synchronization reject unobserved unbound-to-bound catalog transitions; keep any partially proven task incomplete. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 14be3dddac
ℹ️ 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".
|
@codex review exact head 8c437bf. Please perform a fresh whole-PR review for correctness, regressions, architecture and ownership, the complete applicable Rust and SQLite pattern fit, OpenSpec fulfillment, positive/negative/failure/concurrency/compatibility coverage, all four hosted platforms, intended-scale performance, agent-first workflow boundaries, and v0.4.5-rc1 readiness. This head makes failed local snapshot export non-committing and keeps the complete repository aggregate read under validated catalog writer exclusion while leaving benchmark enrichment outside that lock. Treat all prior-head reviews as historical and keep partially implemented or partially proven tasks incomplete. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 8c437bf1b1
ℹ️ 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".
|
@codex review exact head def3318. Please perform a fresh whole-PR review for correctness, regressions, architecture and ownership, the complete applicable Rust and SQLite pattern fit, OpenSpec fulfillment, positive/negative/failure/concurrency/compatibility coverage, all four hosted platforms, intended-scale performance, agent-first workflow boundaries, and v0.4.5-rc1 readiness. This head makes every deferred first binding atomically accept the local aggregate snapshot or remain unbound, with database and MCP rollback regressions and synchronized lifecycle documentation. Treat prior-head reviews as historical and keep partially implemented or partially proven tasks incomplete. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: def3318612
ℹ️ 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".
|
@codex review exact head |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: d3720e6b8e
ℹ️ 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".
|
@codex review Please review exact head |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 436b8325e5
ℹ️ 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".
|
Codex Review: Didn't find any major issues. Keep them coming! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
|
@codex review exact head ab104af. This is a checklist-only synchronization after the complete source head passed fresh review, the full local gate, and the named holistic E2E on Linux, Windows, macOS Intel, and macOS ARM. Please verify the final OpenSpec/issue completion state and report any remaining actionable issue before merge. |
|
Codex Review: Didn't find any major issues. Keep them coming! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
1 participant
Summary
atlas_worktree_list,atlas_worktree_add, andatlas_worktree_remove, plus shortworktreealias routing across normal tools while preserving mutually exclusive legacyproject_pathroutingatlas_init(worktree=...)hydrate an absent target from a compatible complete control atlas through a private SQLite candidate, assign target identity, clear non-transferable state, reconcile current branch/dirty bytes, and fall back visibly to ordinary initBoundaries
Validation
cargo run --locked -p projectatlas-lints --bin cargo-projectatlas-lints -- strict-stringscargo fmt --all --checkcargo check --workspace --all-targets --all-features --lockedcargo clippy --workspace --all-targets --all-features --locked -- -D warningscargo deny --locked --all-features check -D warningscargo test --workspace --all-features --lockedcargo test --locked -p projectatlas-cli --test e2e holistic_agent_worktree_flow_keeps_local_atlases_isolated_across_cli_watch_and_mcp -- --exact --include-ignored --nocapturecargo test --doc --workspace --all-features --lockedRUSTDOCFLAGS="-D warnings" cargo doc --workspace --no-deps --all-features --lockedlint --report-untracked --purpose-level lowCloses #430
Closes #440