Phase 4: Review¶
Comprehensive code review using 6 parallel specialized reviewers.
Command: /4-review [--scope path/to/review]
When to use¶
After completing implementation in Phase 3. The review runs in a forked context
(context: fork) so it forms an independent assessment without being influenced
by the build conversation.
What happens¶
-
Test gate verification -- Reads
.test-gate.mdfrom the project root. If the file is missing, hasstatus: FAIL, or was generated on a different branch, the review is blocked. Run the test-runner in full suite mode first.Gate State Result status: PASS, branch matchesReview proceeds status: FAILBlocked — fix failing tests File missing Blocked — run full test suite Branch mismatch Blocked — stale gate, re-run on current branch -
Checkpoint and focus -- Reads the build checkpoint for focus hints, but forms its own independent conclusions.
- Six parallel reviewers dispatch simultaneously:
| Reviewer | Focus |
|---|---|
| security-sentinel | OWASP vulnerabilities, XSS, injection, auth |
| kieran-typescript-reviewer | TypeScript quality, strict typing |
| julik-frontend-races-reviewer | Race conditions, async/DOM state |
| code-simplicity-reviewer | YAGNI, complexity, dead code |
| performance-oracle | N+1 queries, memory leaks, bundle size |
| pattern-recognition-specialist | Design patterns, anti-patterns |
- Findings classification -- Issues are rated Critical, High, Medium, or Low.
- Quality gate -- PASS requires all of:
.test-gate.mdstatus is PASS- Zero Critical findings
- No more than 2 High findings
- Spec compliance (when SpecKit active) -- Validates implementation against
the active
specs/[N]-feature/spec.mdrequirements (see.design-state.yamlfor the active spec) and.specify/memory/constitution.mdprinciples. - Documentation check (when
docs/user-manual/exists) -- Verifies feature documentation pages exist and are non-empty. Reported as Medium severity.
See Testing Procedures for details on running tests and the gate file format. See Documentation Guide for the documentation workflow.
Output¶
- Review report with findings by category and severity
- Quality gate decision (PASS or FAIL)
- Spec compliance coverage percentage (when SpecKit active)
- Specific fix guidance for any failed items
Next step¶
- If PASS: Phase 5: Deploy
- If FAIL: Return to Phase 3: Build to fix issues, then re-review.