UNRELEASED DEVELOPMENT SNAPSHOT · READ-ONLY EXCERPT

skills/review/SKILL.md

This is the cited technical source at local revision 275a354, inspected on 22 September 2026. It differs from the public release. This reference publishes only the cited sections, not an installable release or the development branch.

Line numbers refer to the complete recorded file. Uncited lines are omitted. Full-file SHA-256: 1f8c86b664e927d0e98d3be1e10bac97469930c62e8f6f553e0c7e4575ffd9c9.

1---
2name: review
3layer: foundation
4description: Use after BUILD, before SHIP, to adversarially review what was built — checks spec compliance, code quality, the active pack's compliance gates, and cross-artifact consistency. A P1 finding blocks SHIP until resolved.
5color: cyan
6tools: Read, Bash, Grep, Glob
7voice: internal
8cli_support: [claude-code, codex, copilot]
9necessity: STRONGLY_RECOMMENDED
10gap_if_skipped: "Unreviewed code reaches SHIP with no signal; spec-compliance and compliance gates never fire yet a ship-ready verdict is asserted."
11---
12
13You are the REVIEW skill — Phase 6 of the Lintel cycle.
14
15## What this skill does
16
… omitted uncited lines …
180- `codex exec` with read-only sandbox
181- Prompt: "Independent reviewer. 5 representative diffs. Spot what internal review may have missed. Output: P1/P2/P3 findings."
182- Run with 5-min timeout
183- Surface output verbatim under "OUTSIDE VOICE (Codex):" header
184- Cross-synthesize with internal findings
185
186If unavailable: skip silently.
187
188### Step 7 — Write artifacts
189
190**`review-report.md`** (`.claude/runtime/state/review-report-<datetime>.md`):
191```markdown
192# Review report: <wedge title>
193
194**Phase:** REVIEW
195**Date:** <date>
196**Stage 1 (spec compliance):** PASS | FAIL (N iterations)
197**Stage 2 (code quality):** PASS | FAIL (N iterations)
198**Stage 3 (compliance gates):** PASS | FAIL (per gate breakdown)
199
200## Stage 1 — Spec compliance findings
201[per-task deviations and fixes applied]
202
203## Stage 2 — Code quality findings
204### P1 (block ship)
… omitted uncited lines …
222[Codex output verbatim]
223
224## Verdict
225- Ship-ready: <yes/no/yes-with-caveats>
226- Remaining concerns: <list>
227- Loop-back recommended: <to BUILD if P1 unaddressed; to PLAN if scope-gap; to DEFINE if requirements unclear>
228```
229
230**`compliance-report.md`** (if the active pack defines compliance gates):
231- Per-gate breakdown for audit trail
232- Path: `.claude/runtime/state/compliance-report-<datetime>.md`
233
234### Step 8 — 00-state.md append
235
236Mechanical since v5.0 (ADR-0008) — one command, not a YAML obligation. `next=` is SHIP, or BUILD on loop-back, or DEFINE on scope gap; per-stage detail lives in review-report.md:
237
238```bash
239_sl="${LINTEL_SOURCE_ROOT:-${LINTEL_REPO_ROOT:-$(git rev-parse --show-toplevel 2>/dev/null)}}/lib/state.sh"
240[ -f "$_sl" ] || _sl="$HOME/.lintel/lib/state.sh"; source "$_sl"   # installed by install.sh in consumer repos
241state_append REVIEW <DONE|DONE_WITH_CONCERNS|BLOCKED> next=<SHIP|BUILD|DEFINE> review_report_path=<path> p1_findings=<count> p2_findings=<count> p3_findings=<count> ship_ready=<yes|no|yes-with-caveats>
242```
243
244## Status protocol
245
246- **DONE** — all stages PASS, P1 findings addressed, ship-ready
… omitted uncited lines …
271- optional swarm coordination plus all lane reports/reviews and integration attribution
272- plan.md (PLAN phase)
273- design doc (DEFINE phase)
274- build-log.md (BUILD phase)
275- CORE-PRINCIPLES.md
276- the active pack's compliance gates (`resolve_pack_field compliance.hooks`; none by default)
277- the active pack's voice corpus (`resolve_pack_field voice.corpus`; none by default — if voice gate)
278- `.claude/memory/lessons.md` (via `/li:lessons-surface`, keyword-scoped, non-blocking)
279
280**Writes:**
281- `.claude/runtime/state/review-report-<datetime>.md`
282- `.claude/runtime/state/compliance-report-<datetime>.md` (if the active pack defines compliance gates)
283- `.claude/runtime/state/00-state.md` (REVIEW entry)
284- `.claude/runtime/audit/review-metrics.jsonl`
285
286**Triggers:**
287- SHIP if PASS
288- BUILD loop-back if P1 unfixed
289- DEFINE loop-back if scope gap revealed
290
291## Recommended agents (concentrated review pool)
292
293**Engineering:**
294- CodeReviewer — primary for Stage 1 + 2
295- SanityChecker — cross-component architectural sanity
296- DebugForensics — if reviews reveal bug
297- RegressionDetective — if regression suspected
298