[codex] Fix Nexus session handoff routing#459
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Fixes the Nexus-backed session handoff path so Codex work can be routed to Claude reviewers and reflected accurately in session state and the TUI.
What Changed
grove session startwhen Nexus is configured.GROVE_ZONE_IDordefault.Root Cause
The headless session orchestrator mirrored the session to Nexus, but still polled local SQLite for contributions. Agent MCP servers wrote session-scoped contributions to Nexus, so downstream reviewers did not reliably receive handoffs. The session count and completion mirrors also had separate gaps, which left the TUI showing
0cor Nexus sessions still marked active after local completion.Validation
bun --config=/dev/null test src/core/session-orchestrator.test.ts src/cli/commands/session.test.ts --timeout 30000bun --config=/dev/null test src/core/acp-runtime.spawn.test.ts --timeout 30000bun run typecheckbun run check(passes with existing warnings)timeout 180s bun run buildgit diff --checknexus-hello-final.txt, Claude reviewed the actual coder workspace, emitted[DONE], and both local and Nexus session stores reportedcompleted,achieved,contributionCount: 3. TUI capture showed the completed row as3c · achieved.