code-reviewer
Installation
SKILL.md
Code Reviewer
Two responsibilities: requesting review (dispatching a reviewer subagent) and receiving review (acting on feedback with technical rigor). Review early, review often. Verify before implementing feedback.
Part 1: Requesting Review
Mandatory after: each task in SDD, each planned execution wave, each major feature, before merge. Optional: when stuck, before refactoring, after fixing complex bugs.
Procedure
- Get SHAs:
BASE_SHA=$(git rev-parse HEAD~1)andHEAD_SHA=$(git rev-parse HEAD) - Dispatch
general-purposesubagent using template atskills/code-reviewer/code-reviewer-prompt.md - Fill placeholders:
[DESCRIPTION](what was built),[PLAN_OR_REQUIREMENTS](contract/spec reference),[BASE_SHA],[HEAD_SHA],[WAVE_ID], and a distinct[REVIEW_REPORT_FILE]. - Require the reviewer to write a non-empty persisted review report at
.superpowers/sdd/reviews/<wave-id>.md, then record that exact in-overlay path in the wave receipt withssf execution review <change-dir> --wave <wave-id> --base <base-sha> --head <head-sha> --report .superpowers/sdd/reviews/<wave-id>.md --verdict <pass|fail>. The execution plan initializes this directory; paths outside it are rejected for audit safety. - Act on feedback: Critical/Important findings require a
failreceipt, focused repair, re-review, and replacementpassreceipt before a dependent wave or closing can proceed. Note Minor for later, push back with reasoning if reviewer is wrong. - At
adjudication-required, wait for a human to runssf execution adjudicate <change-dir> --wave <id> --decision allow-review --confirm --reason <text>before another review. It authorizes one review and never substitutes forpass.