Codex PR Review

Ownership

Orchestrator:

• Own watcher runs, PR-body status interpretation, fixer delegation, worktree discipline, and final reporting.
• Keep PR review/fix work in a dedicated git worktree. Reuse a caller-supplied worktree, such as from handle-tickets, after verifying PR branch and current head; otherwise create or verify one before edits/checks.
• Track rounds by head SHA, files, themes, subsystems, accepted/rejected findings, fixes pushed, and checks to detect repeated patterns.
• Send the fixer packet: PR URL/number, branches, worktree, current headRefOid, the current review-cycle freshness boundary when known, watcher-fresh feedbackItems and activeCodexThreads, parsed priorities, watcher snapshot counts, P0/P1 safety-pass results when needed, Review Discipline scope baseline, related prior Codex comments for repeated patterns, prior code-review or review-fix outcomes, checks, and push policy. Include enough spec context to classify findings; do not ask for a fresh full code-review unless a finding specifically requires it.
• Verify the fixer handoff with gh pr view, remote head SHA, local status when sharing a worktree, and verdict/reaction outcomes. Do not analyze, fix, commit, or push delegated comment changes.
• If no fixer sub-agent can be spawned, handle active comments in the parent using Review Discipline and Severity Handling, run checks, and report the fallback.

Fixer sub-agent:

• Complete the supplied work directly without delegating. Refresh PR state, read the full body for every supplied item/thread, apply Review Discipline and Severity Handling, fix valid in-scope blockers, run checks, commit, and push.
• When assigned a repeated-pattern packet, investigate the shared root cause before patching another individual symptom.
• Return PR URL, start/end SHAs, verdicts, reactions, no-fix rationales, root-cause notes when relevant, files, checks, commits, push result, the UTC timestamp captured immediately after confirming any pushed remote head, and blockers.