fix: supervise Orca issue pickup - #16
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: QUIET Plan: Advanced Run ID: 📒 Files selected for processing (3)
Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 5 remain after this review. 📝 WalkthroughWalkthroughThe Orca guidance now describes supervised worker launches and worktree linking for issue work. Unattended-run instructions cover coordinator activity from issue pickup through worker settlement, including messaging, authority limits, and recovery when launch outcomes are uncertain. Validation scenarios cover active-worker pickup limits, worker questions, unknown start outcomes, unsupervised handoffs, and final worker outcomes. Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~12 minutes Merge Risk: ⚪ Minimal · up to No established defect prevents merging. Confirm that the deployed Orca session can remain available through worker settlement before enabling scheduled pickup. Architecture SummaryArchitecture risk: 🔵 Low · up to The change affects 2 systems. Changed systems: Architecture concerns Review detailsSystems and components
Before / after behavior
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Comment |
Idle pickup launched standalone workers and ended immediately, leaving no coordinator to receive questions or report progress. Require a supervised Run, Task and Dispatch, keep the coordinator inbox active through settlement, and route multi-repository work through one conversation. Preserve standalone launch for explicit ownership handoffs.
Document issue linking, launch recovery, missing-information handling and terminal cleanup, and add corresponding skill-validation scenarios.
Validation:
just checkpassed all 32 tests;git diff --checkpassed. A live Orca smoke test verified worker ask, coordinator reply, worker completion with the received token, and terminal release. The scheduled issue-pickup flow itself has not been exercised end to end.