From be8a6269c4f41ce8872f6f477a11899e46e44218 Mon Sep 17 00:00:00 2001 From: Jesse Vincent Date: Tue, 9 Jun 2026 16:53:30 -0700 Subject: [PATCH] =?UTF-8?q?Spec=20reviewer:=20judge=20from=20the=20diff,?= =?UTF-8?q?=20grounded=20skepticism,=20=E2=9A=A0=EF=B8=8F=20verdict=20chan?= =?UTF-8?q?nel?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../spec-reviewer-prompt.md | 15 ++++++++++++--- 1 file changed, 12 insertions(+), 3 deletions(-) diff --git a/skills/subagent-driven-development/spec-reviewer-prompt.md b/skills/subagent-driven-development/spec-reviewer-prompt.md index 1cc84a76..ab4a781b 100644 --- a/skills/subagent-driven-development/spec-reviewer-prompt.md +++ b/skills/subagent-driven-development/spec-reviewer-prompt.md @@ -30,14 +30,20 @@ Subagent (general-purpose): Only read files in this diff. Do not crawl the broader codebase. + Spec compliance is judged by reading the diff against the requirements. + The implementer already ran the tests and reported TDD evidence — do not + re-run them. If a requirement cannot be verified from this diff alone + (it lives in unchanged code or spans tasks), report it as a ⚠️ item + instead of broadening your search. + ## Read-Only Review - Your review is read-only on this checkout. Do not mutate the working tree, the index, HEAD, or branch state in any way. Use tools like `git show`, `git diff`, and `git log` to inspect history. If you need a working copy of a different revision, check it out into a separate temporary directory (e.g. `git worktree add /tmp/review-[SHA] [SHA]`) — never move HEAD on this checkout. + Your review is read-only on this checkout. Do not mutate the working tree, the index, HEAD, or branch state in any way. Use tools like `git show`, `git diff`, and `git log` to inspect history. ## CRITICAL: Do Not Trust the Report - The implementer finished suspiciously quickly. Their report may be incomplete, - inaccurate, or optimistic. You MUST verify everything independently. + Treat the implementer's report as unverified claims about the code. It may + be incomplete, inaccurate, or optimistic. Verify the claims against the diff. **DO NOT:** - Take their word for what they implemented @@ -74,4 +80,7 @@ Subagent (general-purpose): Report: - ✅ Spec compliant (if everything matches after code inspection) - ❌ Issues found: [list specifically what's missing or extra, with file:line references] + - ⚠️ Cannot verify from diff: [requirements you could not verify from the + diff alone, and what the controller should check — report alongside the + ✅/❌ verdict for everything you could verify] ```