reviewlisted

# Review Scope is **the diff**, not the modules it touches. Do not read whole files where a hunk answers the question. ## 0 — Focus An argument, when given, names the passes to run — `correctness`, `contract`, `consistency`, `perf` — separated by commas or spaces (`/feature-workflow:review contract,perf`). Anything unrecognized, or no argument at all, runs all four. A focused run is a **shorter report over the same hunks**, never a wider read. Scope stays the diff, section 1 still runs, and the passes you skipped go unmentioned rather than being reported as clean. `security` is deliberately not a pass here: Claude Code ships `/security-review` over the same diff, and a half-version of it inside a conventions review would be worse than either. ## 1 — Scope ```bash git status --short git diff $(git merge-base HEAD "$(git symbolic-ref --short refs/remotes/origin/HEAD 2>/dev/null | sed 's|origin/||' || echo main)")..HEAD ``` Include unstaged and untracked files. If a `.planning/*/PLAN.md` covers this work, read it and its `contract.md`; otherwise review against the project's conventions alone. ## 2 — Passes **Correctness** — does it do what the plan says? Every acceptance criterion gets a verdict backed by `file:line`, not a summary. **Contract** — every type and payload against `.planning/*/contract.md`: required vs optional, enums, nullability. A field that is not in the contract is a bug, not a nicety. **Consistency** — each changed file against the sibling module