agent-review-pipeline

agent-review-pipeline

Autonomous dual-engine code review pipeline for Claude Code. Asymmetric dispatch — Codex runs ARP's dual-framing (correctness + adversarial) while Gemini CLI runs its own /ce:review compound-engineering pipeline. 3 parallel perspectives per iteration. Dedups findings by confidence, auto-fixes inline, escalates unresolvable findings. Verification delegated to your CI.

What It Does

1. Asymmetric Dual-Engine Review — 3 parallel dispatches per iteration: Codex × correctness, Codex × adversarial, Gemini × /ce:review. Codex gets ARP's framing discipline; Gemini uses its own multi-persona compound-engineering pipeline (P0-P3 tiering) because it already has one. No redundancy.
2. Confidence-Weighted Consensus — Findings fingerprinted by file:line:severity:normalize(issue):sha1(fix_code[:200]). Multi-source agreement boosts confidence by +0.15 per extra source. Findings below 0.60 confidence are dropped.
3. Bounded Auto-Fix Loop — Applies fix_code inline, re-runs until PASS or maxIterations (1-10, default 3). Unlimited loops are intentionally not supported.
4. Loop-Thrash Kill Switch — If a fingerprint reappears after its fix was applied, the fix didn't work. The finding is escalated to human review instead of looping forever.
5. Safe Defaults — autoCommit and postPrComment default to false. --dry-run previews findings without editing. Dependency precheck fails fast if Codex / Gemini CLI missing.
6. Agreement Telemetry — Per-run .arp_session_log.json records Codex↔Gemini agreement rate. Tune dual-engine cost/value over time.

How It Works

flowchart TD
    Start([/arp PR]) --> Pre

    subgraph Stage0["Stage 0: Pre-flight"]
        Pre[deps · flock · gh auth · PR resolve<br/>+ PR conversation context fetch]
    end

    subgraph Stage1["Stage 1: Review (dispatch → merge → fix → loop)"]
        Disp{3 parallel dispatches}
        Disp --> C1["Codex × Correctness<br/>Agent: codex-rescue<br/>read-only contract"]
        Disp --> C2["Codex × Adversarial<br/>Agent: codex-rescue<br/>read-only contract"]
        Disp --> G["Gemini × /ce:review<br/>Bash: gemini -p<br/>3-layer read-only"]
        C1 --> S1[snapshot_git diff]
        C2 --> S2[snapshot_git diff]
        G  --> S3[snapshot_git diff]
        S1 --> M
        S2 --> M
        S3 --> M
        M[Merge + Fingerprint<br/>file:line:severity:issue:fix-hash<br/>+0.15 per source · drop conf below 0.60]
        M --> L{any findings?}
        L -- yes --> K{fingerprint<br/>seen before?}
        K -- yes --> E[ESCALATE<br/>human review]
        K -- no --> F[Apply fix_code via Edit tool]
        F --> I{iter < max?}
        I -- yes --> Disp
        I -- no --> FE{failOnError?}
        FE -- true --> Ab([abort exit non-zero])
        FE -- false --> E
    end

    subgraph Stage2["Stage 2: Deliver"]
        D[summary · agreement rate · parse-error counts]
        D --> AC{autoCommit?}
        AC -- true --> GC[git commit]
        AC -- false --> PC{postPrComment?}
        GC --> PC
        PC -- true --> SCR[Scrub secrets<br/>API keys · JWT · creds · bearer]
        SCR --> GH[gh pr comment]
        GH --> End([end])
        PC -- false --> End
    end

    Pre --> Disp
    L -- no --> D
    E --> D

Why asymmetric (the "3 dispatches from 2 engines" question):

Codex                              Gemini
  │                                  │
  ├── correctness framing  (ARP)     │
  ├── adversarial framing (ARP)     ─┴── /ce:review
  │                                       (multi-persona pipeline,
  │                                        P0–P3 tiering, internal)
  ▼                                   ▼
 2 dispatches                     1 dispatch
  └────────── 3 perspectives merged by fingerprint ──────────┘

Codex has no built-in multi-persona reviewer, so ARP supplies its dual-framing discipline directly. Gemini already ships /ce:review (the compound-engineering pipeline) — running ARP-side dual-framing on top would just duplicate work, so ARP delegates to it.

How It Differs from Single-Engine Agents

Single-Engine Agent	ARP
One LLM, single framing	Codex × 2 framings + Gemini × /ce:review → 3 parallel perspectives
Suggests, leaves fix to human	Auto-fixes inline, bounded loop (cap 10)
Loops forever on unfixable bugs	Fingerprint kill switch escalates stuck findings
Commits without review	Opt-in commit and PR comment
Blind cost	Agreement telemetry for both → single-engine tuning

Installation

/plugin marketplace add onchainyaotoshi/agent-review-pipeline
/plugin install agent-review-pipeline@agent-review-pipeline
/reload-plugins

Usage

Review the current branch, asymmetric 3-dispatch by default:

/arp

Review a specific PR:

/arp 42