From 4dc2b4d6946ef785eb5dd97b2a10948f94167e18 Mon Sep 17 00:00:00 2001 From: Paul Bakaus Date: Thu, 23 Jul 2026 09:50:14 -0700 Subject: [PATCH] Finish reviewer: the skill invocation authorizes its subagents A live session on a harness whose guidance gates subagent use on user request resolved the conflict silently against the skill: it never spawned the reviewer, stretched the no-subagents fallback to cover permission hesitancy, and self-reviewed with all the context that made its choices feel correct. Three tightenings: invoking the skill IS the user request that authorizes its shipped subagents; the fallback is for harnesses lacking the capability, not for hesitancy; a substituted review gets disclosed in one line at finish, never silently. Co-Authored-By: Claude Fable 5 --- skill/reference/new-work.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/skill/reference/new-work.md b/skill/reference/new-work.md index 09caa3b25..0b28d6a54 100644 --- a/skill/reference/new-work.md +++ b/skill/reference/new-work.md @@ -104,4 +104,4 @@ Preserve semantics, accessibility, performance, responsiveness, project conventi Inspect desktop and mobile, critique the render against the user's request, the direction contract, and DESIGN.md, fix material gaps, and re-inspect. On a Persuade surface, verify the mode did its job: a first-time visitor should know what this is, why it matters, and what to do within seconds, in the form's own vocabulary. After a first implementation of a new world, update DESIGN.md with the exact tokens and behaviors that survived the build. -Spawn the shipped finish reviewer, `impeccable-finish-reviewer` (`impeccable_finish_reviewer` in codex), and give it the original request, confirmed answers, the artifact path, its direction contract, DESIGN.md, existing hook findings, and the QUALITY BAR card and approved comp paths. This review never runs inside the build thread; a harness with no subagents runs it as its own fresh pass, after stepping fully out of the build context. The reviewer's first check is persistence: on a new or replacement world, PRODUCT.md and DESIGN.md exist and DESIGN.md matches the built world; a missing file fails the review before any craft point is scored. Its second check is the ceiling: against the chosen world's QUALITY BAR card and the approved comp, name the native devices the build left unused, frame, depth, lettering, ornament density; floor rigor is very good at disguising an unreached card. Then ask for a short list of material fixes, promise by promise against the contract, apply them, and finish. Do not run a second detector. +Spawn the shipped finish reviewer, `impeccable-finish-reviewer` (`impeccable_finish_reviewer` in codex), and give it the original request, confirmed answers, the artifact path, its direction contract, DESIGN.md, existing hook findings, and the QUALITY BAR card and approved comp paths. The user's invocation of this skill is the request that authorizes its shipped subagents; a harness rule gating subagent use on user request is satisfied by it, so hesitancy is not grounds to skip the spawn. This review never runs inside the build thread. Only a harness whose tool surface has no subagent capability at all substitutes a fresh in-thread pass after stepping fully out of the build context, and a substituted review is disclosed in one line at finish, never silently. The reviewer's first check is persistence: on a new or replacement world, PRODUCT.md and DESIGN.md exist and DESIGN.md matches the built world; a missing file fails the review before any craft point is scored. Its second check is the ceiling: against the chosen world's QUALITY BAR card and the approved comp, name the native devices the build left unused, frame, depth, lettering, ornament density; floor rigor is very good at disguising an unreached card. Then ask for a short list of material fixes, promise by promise against the contract, apply them, and finish. Do not run a second detector.