[object Object]

← back to Cli Printing Press

fix(skills): drop context: fork from output-review so AskUserQuestion works natively (#417)

ac98c00ac588d8a5e9d0b7c22c59afc0773d29d3 · 2026-04-29 22:52:50 -0700 · Trevin Chow

Same fix as #410, applied to the next sub-skill in the migration.
Output-review ran in a forked context so its reviewer-agent diagnostic
chatter wouldn't pollute the parent generation flow. The cost is the
same deferred-tool unreliability AskUserQuestion has in forks: its
schema is invisible to the fork's starting tool list, and prompt sites
inconsistently fall back to plain-text instead of running ToolSearch
first. The plain-text fallback strands the user — their reply lands in
the parent session, never reaches the fork, and the workflow stalls.

Take the noise pollution; gain reliable user prompting. Output-review
runs in the parent context now; AskUserQuestion is available natively
at every prompt site. Updates the parent skill prose at
skills/printing-press/SKILL.md to match.

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Files touched

Diff

commit ac98c00ac588d8a5e9d0b7c22c59afc0773d29d3
Author: Trevin Chow <trevin@trevinchow.com>
Date:   Wed Apr 29 22:52:50 2026 -0700

    fix(skills): drop context: fork from output-review so AskUserQuestion works natively (#417)
    
    Same fix as #410, applied to the next sub-skill in the migration.
    Output-review ran in a forked context so its reviewer-agent diagnostic
    chatter wouldn't pollute the parent generation flow. The cost is the
    same deferred-tool unreliability AskUserQuestion has in forks: its
    schema is invisible to the fork's starting tool list, and prompt sites
    inconsistently fall back to plain-text instead of running ToolSearch
    first. The plain-text fallback strands the user — their reply lands in
    the parent session, never reaches the fork, and the workflow stalls.
    
    Take the noise pollution; gain reliable user prompting. Output-review
    runs in the parent context now; AskUserQuestion is available natively
    at every prompt site. Updates the parent skill prose at
    skills/printing-press/SKILL.md to match.
    
    Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 skills/printing-press-output-review/SKILL.md | 1 -
 skills/printing-press/SKILL.md               | 2 +-
 2 files changed, 1 insertion(+), 2 deletions(-)

diff --git a/skills/printing-press-output-review/SKILL.md b/skills/printing-press-output-review/SKILL.md
index 73007c09..5daa4ea1 100644
--- a/skills/printing-press-output-review/SKILL.md
+++ b/skills/printing-press-output-review/SKILL.md
@@ -7,7 +7,6 @@ description: >
   Skill tool by main printing-press SKILL.md (Phase 4.85) and printing-press-polish
   SKILL.md during the diagnostic loop. Not for direct user invocation — its
   actionable wrappers are /printing-press and /printing-press-polish.
-context: fork
 user-invocable: false
 allowed-tools:
   - Bash
diff --git a/skills/printing-press/SKILL.md b/skills/printing-press/SKILL.md
index e85ffd7b..7201c561 100644
--- a/skills/printing-press/SKILL.md
+++ b/skills/printing-press/SKILL.md
@@ -1971,7 +1971,7 @@ Skill(
 )
 ```
 
-The sub-skill carries `context: fork` so the reviewer agent's diagnostic chatter stays isolated from this generation flow. It returns a `---OUTPUT-REVIEW-RESULT---` block with `status: PASS|WARN|SKIP` and a list of findings.
+The sub-skill returns a `---OUTPUT-REVIEW-RESULT---` block with `status: PASS|WARN|SKIP` and a list of findings.
 
 **Wave B rollout policy:** all findings surface as **warnings**, not blockers. Shipcheck does not fail on Phase 4.85 findings. Log the findings to `manuscripts/<api>/<run>/proofs/phase-4.85-findings.md` and surface them to the user. The user decides case by case whether to fix before shipping. Wave B calibrates false-positive rates before Wave C flips errors to blocking.

← 4e1db1e7 fix(cli): repair v3 release — version regex, module path, CI  ·  back to Cli Printing Press  ·  chore(main): release 3.0.1 (#416) aa80c9d9 →