review

Installation
SKILL.md

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

  1. 02-DOCS/wiki/sdd/specs/<slug>.md and 02-DOCS/wiki/sdd/plans/<slug>.md — what the diff was supposed to do. A review with no spec is a review of vibes.
  2. 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".
  3. 02-DOCS/wiki/harness/user-profile.md — the accompaniment dial (see "Narration dial" below).
Installs
3
GitHub Stars
116
First Seen
Aug 6, 2026
review — ericrisco/rsc-harness