agent-reviewlisted

# Agent Review The review step of `/implement`'s repo path. Runs on **committed** work (never unstaged changes), inside the draft PR. Dispatches a review subagent — fed the diff plus review criteria — and returns **structured findings** for the recipe to act on. Dependency-free by default: a fresh install gets working review with nothing beyond dossier-mcp. The richer `pr-review-toolkit` specialist panel is an opt-in **override** of this skill (see Project overrides), not a built-in — so `/implement` never *requires* the toolkit. ## Input - **diff** — the committed change to review, as a ref range (default `origin/main...HEAD`) or a PR number. - **criteria** (optional) — focus areas; defaults below. - **context** (optional) — the `plan-file` path and the ticket / acceptance criteria, passed to the reviewer so it can check the change against intent. ## Default review criteria (the floor) - **Correctness** — does the code do what it claims; do its own commands/claims hold up when checked; logic errors, edge cases, off-by-ones. - **AC coverage** — does the change actually satisfy the ticket's acceptance criteria. - **Silent failures / error handling** — swallowed errors, missing guards, fallback that hides real failures. - **Internal consistency** — self-contradiction; drift from the change's own stated contract. - **Convention consistency** — does it match sibling code/patterns in the repo. A project override can replace or extend these. If the AC / plan context isn't pr