From 86e2bbdabf25522f0992c692074705ef7f6fbab7 Mon Sep 17 00:00:00 2001 From: garrytan Date: Thu, 8 Oct 2026 06:20:39 +0000 Subject: [PATCH] fix(plan reviews): skip the /office-hours prerequisite offer in spawned and headless sessions (#1958) The BENEFITS_FROM offer had no recommended option, so a spawned run had nothing defined to auto-pick and a headless run could only block. Skip it there and continue with the standard review. Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com> --- scripts/resolvers/spec-review.ts | 2 ++ 1 file changed, 2 insertions(+) diff --git a/scripts/resolvers/spec-review.ts b/scripts/resolvers/spec-review.ts index 4f9832a1d..54303646e 100644 --- a/scripts/resolvers/spec-review.ts +++ b/scripts/resolvers/spec-review.ts @@ -249,6 +249,8 @@ export function generateBenefitsFrom(ctx: TemplateContext): string { When the design doc check above prints "No design doc found," offer the prerequisite skill before proceeding. +Skip the offer and proceed with the standard review when the preamble echoed \`SESSION_KIND\` \`spawned\` or \`headless\`. + ${ctx.skillName === 'plan-eng-review' ? 'Build the next full decision brief from these facts and options, using the preamble transport, numbering and format:' : 'Say to the user via AskUserQuestion:'} > "No design doc found for this branch. ${skillList} produces a structured problem