review-toolkit

|

What it does

Comprehensive PR review pipeline: a pre-review readiness gate (stops on merge conflicts or red code-related CI with a fix-it note, no verdict), branch-level analysis with merge-base attribution (the verdict is decided only by defects the diff introduced, worsened, or interacts with — purely pre-existing defects are reported in full in a separate outside-findings section and routed to a filed issue, parked branch, or memory entry instead of blocking), sequential Claude+Codex dual-model review with evidence-based verification, and opt-in GitHub PR review publishing under a five-frame substance framework. A value/design gate weighs whether the change should exist at all — value versus permanent maintenance cost, right layer, root cause in-repo versus upstream — and can block a PR even when the code is flawless. Remedies stay proportional: changed logic needs tests in the existing test surface, fixed prose is its own deliverable, and a new checker or guard script is never a required remedy. Findings carry git-computed origin labels (feature vs churn) that diagnose diverging review loops without ever touching the verdict. Reviews land on an LGTM or NOT LGTM verdict, compatible with non-blocking recommendations; the only no-verdict path is the readiness gate's fix-CI / fix-conflicts note.