From 781f46d025dc56754e226d3c3440a7afc799b5f9 Mon Sep 17 00:00:00 2001 From: Garry Tan Date: Fri, 28 Aug 2026 02:00:50 +0000 Subject: [PATCH] =?UTF-8?q?feat(preamble):=20AskUserQuestion=20repetition?= =?UTF-8?q?=20cut=20=E2=80=94=20gated,=20passed=20NOT-WORSE=20A/B?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Removes the duplicate statements v1.71's compaction left in the AskUserQuestion Format section: the completeness rule restated in the prose triad, the auto-decide marker syntax stated twice, the Conductor-flakiness explanation stated twice, and the self-check's full triad restatement. Every verbosity floor and all 14 format pins stay (Layer 0 green). The gate this decision rested on ran before landing (new periodic skill-e2e-auq-repetition-cut-ab.test.ts, pre-cut ref 3263fffe vs this render, same harness as auq-verbose-vs-carved-ab): POST 7/7 format elements, substance 5 — identical to PRE. No degradation; the load-bearing-repetition hypothesis did not hold for these duplicates. Net: -236 bytes per tier-2+ skill (~9.7KB corpus). Golden ship baselines re-captured for the deliberate change. Co-Authored-By: Claude Fable 5 --- autoplan/SKILL.md | 10 +- canary/SKILL.md | 10 +- codex/SKILL.md | 10 +- context-restore/SKILL.md | 10 +- context-save/SKILL.md | 10 +- cso/SKILL.md | 10 +- design-consultation/SKILL.md | 10 +- design-html/SKILL.md | 10 +- design-review/SKILL.md | 10 +- design-shotgun/SKILL.md | 10 +- devex-review/SKILL.md | 10 +- document-generate/SKILL.md | 10 +- document-release/SKILL.md | 10 +- health/SKILL.md | 10 +- investigate/SKILL.md | 10 +- ios-clean/SKILL.md | 10 +- ios-design-review/SKILL.md | 10 +- ios-fix/SKILL.md | 10 +- ios-qa/SKILL.md | 10 +- ios-sync/SKILL.md | 10 +- land-and-deploy/SKILL.md | 10 +- landing-report/SKILL.md | 10 +- learn/SKILL.md | 10 +- office-hours/SKILL.md | 10 +- pair-agent/SKILL.md | 10 +- plan-ceo-review/SKILL.md | 10 +- plan-design-review/SKILL.md | 10 +- plan-devex-review/SKILL.md | 10 +- plan-eng-review/SKILL.md | 10 +- plan-tune/SKILL.md | 10 +- qa-only/SKILL.md | 10 +- qa/SKILL.md | 10 +- retro/SKILL.md | 10 +- review/SKILL.md | 10 +- .../preamble/generate-ask-user-format.ts | 10 +- setup-deploy/SKILL.md | 10 +- setup-gbrain/SKILL.md | 10 +- ship/SKILL.md | 10 +- skillify/SKILL.md | 10 +- spec/SKILL.md | 10 +- sync-gbrain/SKILL.md | 10 +- test/fixtures/golden/claude-ship-SKILL.md | 10 +- test/fixtures/golden/codex-ship-SKILL.md | 10 +- test/fixtures/golden/factory-ship-SKILL.md | 10 +- test/skill-e2e-auq-repetition-cut-ab.test.ts | 110 ++++++++++++++++++ 45 files changed, 330 insertions(+), 220 deletions(-) create mode 100644 test/skill-e2e-auq-repetition-cut-ab.test.ts diff --git a/autoplan/SKILL.md b/autoplan/SKILL.md index 6696664ff..3dbedcd0f 100644 --- a/autoplan/SKILL.md +++ b/autoplan/SKILL.md @@ -79,7 +79,7 @@ If `SKILL_PREFIX` is `"true"`, suggest/invoke `/gstack-*` names. Disk paths stay Branch on the skill-start STATUS lines, in this order: -1. **`CONDUCTOR_SESSION: true` echoed** → do NOT call AskUserQuestion at all (neither native nor any `mcp__*__AskUserQuestion` variant): render EVERY decision brief as the **prose form** below and STOP. Proactive, not a failure reaction — Conductor disables native AUQ and its MCP variant is flaky (`[Tool result missing due to internal error]`). **Auto-decide preferences still apply first:** a surfaced `[plan-tune auto-decide]