fx-review

What problem does it solve? Code review loops stall when reviewers report out-of-scope findings, fix one instance of a repeated pattern, or act on low-value nitpicks, producing endless review cycles. This Skill defines one canonical review procedure — Scope Brief, triage, materiality, convergence, and reporting — that every reviewer adapter in the catalog follows, so reviews terminate instead of drifting. ## Core Features & Use Cases - Scope Brief enforcement: Requires every review to carry a Scope Brief so deliberately excluded work is never reported as missing, while security, privacy, and correctness defects always stay in scope. - Ordered triage and dispositions: Applies scope, contract, and materiality filters in order, then assigns exactly one disposition — blocking, immaterial, or deferred — with defined actions for each. - Class-level fixing and convergence: Fixes findings as swept classes proven by search, re-runs until no blocking finding remains unresolved, and reports per-pass trends and tier counts. - Use Case: When reviewing a branch with Codex, Copilot, or CodeRabbit, load this procedure first so the adapter skill only handles tool invocation while triage, convergence, and reporting stay consistent across all reviewers. ## Quick Start Review the current branch changes using the fx-review procedure with the Scope Brief from the active workflow.