mirror of
https://github.com/garrytan/gstack.git
synced 2026-07-12 02:36:34 +02:00
fix(preamble): reorder AskUserQuestion Format above model overlay + rewrite Opus 4.7 pacing directive
Root cause of plan-review regression (v1.6.4.0): model overlays rendered
ABOVE the pacing rule in every SKILL.md, so Opus 4.7 read "Batch your
questions" first and absorbed it as the ambient default. The overlay's
claimed subordination ("skill wins on pacing, always") didn't stick —
literal-interpretation mode reads physical order, not claimed hierarchy.
Part 1 of 4 (plan: ~/.claude/plans/system-instruction-you-are-working-polymorphic-twilight.md):
scripts/resolvers/preamble.ts
- Move generateAskUserFormat above generateModelOverlay in section array
- Comment explains why — prevents future refactors from silently reverting
model-overlays/opus-4-7.md
- Replace "Batch your questions" block with "Pace questions to the skill"
- New wording makes one-question-per-turn the default when the skill
contains STOP directives; batching becomes the explicit exception
Regenerated 30 SKILL.md files via bun run gen:skill-docs.
Verified:
- With --model opus-4-7: Format renders at line 359, Model-Specific
Patch at 373, "Pace questions" at 419 (Format comes first, overlay
second, pacing directive intact).
- bun test passes.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
@@ -28,12 +28,13 @@ extended thinking for genuinely hard subproblems: architectural tradeoffs, subtl
|
||||
security implications, design decisions with competing constraints. Over-thinking
|
||||
simple steps wastes tokens and time.
|
||||
|
||||
**Batch your questions.** If you need to clarify multiple things before proceeding,
|
||||
ask all of them in a single AskUserQuestion turn. Do not drip-feed one question per
|
||||
turn. Three questions in one message beats three back-and-forth exchanges. Exception:
|
||||
skill workflows that explicitly require one-question-at-a-time pacing (e.g., plan
|
||||
review skills with "STOP. AskUserQuestion once per issue. Do NOT batch.") override this
|
||||
nudge. The skill wins on pacing, always.
|
||||
**Pace questions to the skill.** If the current skill's text contains
|
||||
`STOP. AskUserQuestion` anywhere, pace one question per turn — emit the question as
|
||||
a tool_use, stop, wait for the user's response, then continue. Do not batch. A
|
||||
finding with an "obvious fix" is still a finding and still needs user approval
|
||||
before it lands in the plan. Only batch clarifying questions upfront when (a) the
|
||||
skill has no `STOP. AskUserQuestion` directive AND (b) you need multiple unrelated
|
||||
clarifications before you can begin. When in doubt, ask one question per turn.
|
||||
|
||||
**Literal interpretation awareness.** Opus 4.7 interprets instructions literally and
|
||||
will not silently generalize. When the user says "fix the tests," fix all failing tests
|
||||
|
||||
Reference in New Issue
Block a user