--- name: check-phase description: Use when you are acting as Guardian, Skeptic, Sage, or Trickster archetype in the Check phase. Defines shared review rules and output format. --- # Check Phase Multiple reviewers examine the Maker's implementation in parallel. Each agent definition has its specific protocol — this skill defines the shared rules. ## Shared Rules 1. **Read the proposal first.** Review against the intended design, not invented requirements. 2. **Read the actual code.** Use `git diff` on the Maker's branch. Don't review descriptions alone. 3. **Each finding needs:** Location (file:line), severity, description, suggested fix. 4. **Severity:** - **CRITICAL** — Must fix. Blocks approval. - **WARNING** — Should fix. Doesn't block alone. - **INFO** — Nice to have. Never blocks. 5. **Clear verdict:** `APPROVED` or `REJECTED` with rationale. ## Consolidated Output After all reviewers finish, compile: ```markdown ## Check Phase Results — Cycle N ### Guardian: APPROVED - WARNING: Missing rate limit (src/auth/handler.ts:52) ### Skeptic: APPROVED - INFO: Consider caching validated tokens ### Sage: APPROVED - WARNING: Test names could be more descriptive ### Trickster: REJECTED - CRITICAL: Empty string bypasses validation (src/auth/handler.ts:48) Reproduction: POST /auth with `{"token": ""}` Expected: 400 | Actual: 500 ### Verdict: REJECTED — 1 critical finding → Feed back to Plan phase for cycle N+1 ```