Skip to content

C4: review engine swap — codex exec review → argus-review (loop + execplan exit-gate) #31

Description

@SSFSKIM

Problem & intent

Replace codex exec review with argus-review at every point doperpowers names a review METHODOLOGY: the reviewing-prs loop engine (skills/reviewing-prs/scripts/review-engine.sh + the worker protocol around it) and the execplan final-branch exit-gate (skills/execplan/SKILL.md), plus documentation prose. Child C4 of docs/doperpowers/specs/2026-07-26-claude-review-stack-roadmap.md.

Constraints

  • Consumes X1 (bench + bar) and X2 (headless mechanism proven by C1.G3); names the minimum argus-review version it requires.
  • Non-review codex uses (triage worker, codex-spawn substrate) are OUT of scope — standing exclusion.
  • Bench decision rule (roadmap Risks): if the side-by-side bench fails the X1 bar, ONE retry with the auto-routed default effort raised; still failing → park needs-human with the numbers (iterate-C2 / ship-anyway / abandon is the human's call).

Success criteria

  • G1: a dispatched PR review on the live loop runs argus as its engine, and the X1 side-by-side bench (argus vs codex engine) passes the bar.
  • G2: execplan's exit-gate and reviewing-prs docs name argus first; codex at most as fallback.

Open questions

  • Engine-contract shape for the argus invocation (compact findings file, bounded wait, event stream?) — child's grill, seeded by C1.G3's probe findings.

Decision log

  • Covers the execplan exit-gate, not just the loop (roadmap Decision Log — leaving it on codex would contradict the unit's purpose).

Metadata

Metadata

Assignees

No one assigned

    Labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions