review-pr

Read-only first-pass PR review, partitioned into 3 sequential stages. Gather PR context and the full diff (inline) → review the whole diff in cohesion as a single isolated-context subagent (the critical-judgement step) → post the review comment, apply the deterministic security and lane:* labels, apply the reviewed label last, write the local report (inline). Never checks out code, never fixes issues, never pushes commits — that's double-check's job.

Purpose

The monolithic review-pr ran the dedup gate, context gathering, diff read, analysis, confidence-scoring, convention check, comment post, label, and report write in one session. The critical-judgement step (analyze + confidence-score the diff) shared a context window already polluted by raw gh JSON, the full diff, CI output, and existing comments. This proc isolates that judgement into one clean-context subagent that sees only the curated PR context and diff.

The ICM win here is clean-context isolation of the review verdict — not parallelism. A PR is reviewed in COHESION: the entire diff, all dimensions (correctness, conventions, Closes-vs-Refs) together in ONE review stage. There is NO per-file fan-out and NO parallel subagents.