review
Review — adversarial code review, both directions
Review is the penultimate gate of the rsc SDD chain: constitution → specify → clarify → plan → tasks → analyze → implement → verify → **review** → ship. By the time a diff reaches review, the verify phase has already proven that lint, types, tests and the stack verify.sh are green. Review answers a different question: is the green diff actually correct, well-scoped, and faithful to the spec — or did it pass for the wrong reasons?
This skill owns two roles that share one discipline:
- Giving a review — read a diff adversarially, find the real defects, rank them by impact, ship the verdict as evidence not opinion.
- Receiving a review — take incoming feedback (human or machine), verify each point against the code before acting, fix the real findings, and push back — with proof — on the wrong ones.
The discipline is the same in both directions: every finding and every rebuttal carries evidence — no agreement without verification, no objection without a defect. A reviewer who waves through a bug to be agreeable, and an author who edits working code because a comment sounded authoritative, are making the same mistake. Everything below is in service of that one rule.
Route out when the ask isn't this gate: a deep OWASP / threat-model pass belongs to ../secure-coding/SKILL.md (review folds its findings in, but the dedicated pass is owned there); a "is this idiomatic React/FastAPI/Go/SQL" question belongs to the stack skill (../nextjs/SKILL.md, ../fastapi/SKILL.md, ../go/SKILL.md, ../postgresdb/SKILL.md, ../flutter/SKILL.md) — pull the idiom there, then judge against it here.
Read these first
02-DOCS/wiki/sdd/specs/<slug>.mdand02-DOCS/wiki/sdd/plans/<slug>.md— what the diff was supposed to do. A review with no spec is a review of vibes.02-DOCS/wiki/sdd/constitution.md— the project's non-negotiables (stack canon, quality bars, conventions). Constitution violations are findings even when the code "works".02-DOCS/wiki/harness/user-profile.md— the accompaniment dial (see "Narration dial" below).