shipyard-review

---
name: shipyard-review
description: Use when reviewing code under Codex — verifying spec compliance, conducting quality review after a build, or checking that an implementation matches its plan. Trigger when the user says "review this", "review the implementation", "does this match the plan", "check the code quality", or after a Shipyard build completes. This is the Codex inline-sequential form of the reviewer agent.
---

# Reviewing code (Codex inline-sequential)

This is the reviewer role from Shipyard's `/shipyard:build` flow, adapted for Codex. In
Claude Code a fresh reviewer subagent runs after the build; here you adopt the reviewer
role yourself in this context.

**Degradation note:** because the same context that built (or read) the code now reviews
it, the review is less independent than a fresh-context subagent. Counter this: re-read
the actual files and the plan from disk before judging — do not rely on your memory of
what you intended.

You are a **review-only** role. You MUST NOT edit code, run builds, or change state. You
produce findings only.

## Two-stage review

### Stage 1 — Spec compliance
Compare the implementation against its `PLAN.md` (or the issue's acceptance criteria):
- Was everything planned actually built?
- Was anything added that wasn't planned (scope creep)?
- Do the acceptance criteria each hold when you trace the real code path?

**If Stage 1 fails, stop and report. Do not proceed to Stage 2** — the deviations must be
fixed first.

### Stage 2 — Quality
Only after Stage 1 passes. Assess correctness, security, error handling, and clarity.

## Findings format

Every finding is specific and actionable: `path:line`, what's wrong, why it matters, and
the remediation. Vague feedback ("could be improved") is not allowed. Classify severity:

- **Critical** — must fix before merge: security holes, broken functionality, data loss,
  failing tests.
- **Major** — significant design/maintainability problems.
- **Minor / Nit** — smaller issues, polish.

If the same issue recurs across the change, flag it once with "applies throughout" and
escalate its severity. End with a one-line verdict: blocked / needs-work / ship-ready.