From 6d050d543060a89780c90ab7c416a276ccd3872d Mon Sep 17 00:00:00 2001 From: Garry Tan Date: Fri, 28 Aug 2026 01:47:09 +0000 Subject: [PATCH] feat(preamble): bounded-closer output rule for tier-2+ skills MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit After completing work, skills report in a few short lines — what changed, what was skipped, what to watch — and cut any explanation that outgrows the change. Explicit exemptions protect every mandated output: decision briefs, completion-status blocks, user-requested explanations, and report-shaped skills' report formats (the report IS the work in /qa-only, /plan-*-review, /retro, /document-generate). Rationale is signal-to-noise, not tokens: ponytail's own benchmark shows terse prose alone doesn't cut cost (caveman arm: -20% LOC, +7% tokens), and independent replications found its 'skipped on purpose' essays ate the code savings. Includes a good/bad closer example pair per the model-overlay guidance that a positive example beats a 'don't be verbose' instruction. Co-Authored-By: Claude Fable 5 --- autoplan/SKILL.md | 5 +++++ canary/SKILL.md | 5 +++++ codex/SKILL.md | 5 +++++ context-restore/SKILL.md | 5 +++++ context-save/SKILL.md | 5 +++++ cso/SKILL.md | 5 +++++ design-consultation/SKILL.md | 5 +++++ design-html/SKILL.md | 5 +++++ design-review/SKILL.md | 5 +++++ design-shotgun/SKILL.md | 5 +++++ devex-review/SKILL.md | 5 +++++ document-generate/SKILL.md | 5 +++++ document-release/SKILL.md | 5 +++++ health/SKILL.md | 5 +++++ investigate/SKILL.md | 5 +++++ ios-clean/SKILL.md | 5 +++++ ios-design-review/SKILL.md | 5 +++++ ios-fix/SKILL.md | 5 +++++ ios-qa/SKILL.md | 5 +++++ ios-sync/SKILL.md | 5 +++++ land-and-deploy/SKILL.md | 5 +++++ landing-report/SKILL.md | 5 +++++ learn/SKILL.md | 5 +++++ office-hours/SKILL.md | 5 +++++ pair-agent/SKILL.md | 5 +++++ plan-ceo-review/SKILL.md | 5 +++++ plan-design-review/SKILL.md | 5 +++++ plan-devex-review/SKILL.md | 5 +++++ plan-eng-review/SKILL.md | 5 +++++ plan-tune/SKILL.md | 5 +++++ qa-only/SKILL.md | 5 +++++ qa/SKILL.md | 5 +++++ retro/SKILL.md | 5 +++++ review/SKILL.md | 5 +++++ scripts/resolvers/preamble/generate-voice-directive.ts | 7 ++++++- setup-deploy/SKILL.md | 5 +++++ setup-gbrain/SKILL.md | 5 +++++ ship/SKILL.md | 5 +++++ skillify/SKILL.md | 5 +++++ spec/SKILL.md | 5 +++++ sync-gbrain/SKILL.md | 5 +++++ 41 files changed, 206 insertions(+), 1 deletion(-) diff --git a/autoplan/SKILL.md b/autoplan/SKILL.md index 0f43b55fe..b23da543c 100644 --- a/autoplan/SKILL.md +++ b/autoplan/SKILL.md @@ -228,6 +228,11 @@ GStack voice: Garry-shaped product and engineering judgment, compressed for runt Good: "auth.ts:47 returns undefined when the session cookie expires. Users hit a white screen. Fix: add a null check and redirect to /login. Two lines." Bad: "I've identified a potential issue in the authentication flow that may cause problems under certain conditions." +**Bounded closer.** After completing work, report in at most a few short lines: what changed, what was skipped, what to watch. No feature tours, no unrequested design notes. If the explanation outgrows the change, cut the explanation. Exempt: AskUserQuestion decision briefs, completion-status blocks, anything the user explicitly asked to be explained, and a skill's mandated report format — the report IS the work in report-shaped skills (/qa-only, /plan-*-review, /retro, /document-generate); this rule governs unrequested prose around the deliverable, never the deliverable. + +Good closer: "Renamed the flag in 3 files, regenerated docs, tests green. Skipped the CLI alias (unused since v1.2); watch the Windows job." +Bad closer: a tour of every edit, a restatement of the plan, and three paragraphs justifying choices nobody questioned. + ## Context Recovery At session start or after compaction, recover recent project context. diff --git a/canary/SKILL.md b/canary/SKILL.md index 471a844be..00c96e2d0 100644 --- a/canary/SKILL.md +++ b/canary/SKILL.md @@ -221,6 +221,11 @@ GStack voice: Garry-shaped product and engineering judgment, compressed for runt Good: "auth.ts:47 returns undefined when the session cookie expires. Users hit a white screen. Fix: add a null check and redirect to /login. Two lines." Bad: "I've identified a potential issue in the authentication flow that may cause problems under certain conditions." +**Bounded closer.** After completing work, report in at most a few short lines: what changed, what was skipped, what to watch. No feature tours, no unrequested design notes. If the explanation outgrows the change, cut the explanation. Exempt: AskUserQuestion decision briefs, completion-status blocks, anything the user explicitly asked to be explained, and a skill's mandated report format — the report IS the work in report-shaped skills (/qa-only, /plan-*-review, /retro, /document-generate); this rule governs unrequested prose around the deliverable, never the deliverable. + +Good closer: "Renamed the flag in 3 files, regenerated docs, tests green. Skipped the CLI alias (unused since v1.2); watch the Windows job." +Bad closer: a tour of every edit, a restatement of the plan, and three paragraphs justifying choices nobody questioned. + ## Context Recovery At session start or after compaction, recover recent project context. diff --git a/codex/SKILL.md b/codex/SKILL.md index 1e7148300..c5b6ed879 100644 --- a/codex/SKILL.md +++ b/codex/SKILL.md @@ -224,6 +224,11 @@ GStack voice: Garry-shaped product and engineering judgment, compressed for runt Good: "auth.ts:47 returns undefined when the session cookie expires. Users hit a white screen. Fix: add a null check and redirect to /login. Two lines." Bad: "I've identified a potential issue in the authentication flow that may cause problems under certain conditions." +**Bounded closer.** After completing work, report in at most a few short lines: what changed, what was skipped, what to watch. No feature tours, no unrequested design notes. If the explanation outgrows the change, cut the explanation. Exempt: AskUserQuestion decision briefs, completion-status blocks, anything the user explicitly asked to be explained, and a skill's mandated report format — the report IS the work in report-shaped skills (/qa-only, /plan-*-review, /retro, /document-generate); this rule governs unrequested prose around the deliverable, never the deliverable. + +Good closer: "Renamed the flag in 3 files, regenerated docs, tests green. Skipped the CLI alias (unused since v1.2); watch the Windows job." +Bad closer: a tour of every edit, a restatement of the plan, and three paragraphs justifying choices nobody questioned. + ## Context Recovery At session start or after compaction, recover recent project context. diff --git a/context-restore/SKILL.md b/context-restore/SKILL.md index 7e2afa5f3..6ac1e7b49 100644 --- a/context-restore/SKILL.md +++ b/context-restore/SKILL.md @@ -225,6 +225,11 @@ GStack voice: Garry-shaped product and engineering judgment, compressed for runt Good: "auth.ts:47 returns undefined when the session cookie expires. Users hit a white screen. Fix: add a null check and redirect to /login. Two lines." Bad: "I've identified a potential issue in the authentication flow that may cause problems under certain conditions." +**Bounded closer.** After completing work, report in at most a few short lines: what changed, what was skipped, what to watch. No feature tours, no unrequested design notes. If the explanation outgrows the change, cut the explanation. Exempt: AskUserQuestion decision briefs, completion-status blocks, anything the user explicitly asked to be explained, and a skill's mandated report format — the report IS the work in report-shaped skills (/qa-only, /plan-*-review, /retro, /document-generate); this rule governs unrequested prose around the deliverable, never the deliverable. + +Good closer: "Renamed the flag in 3 files, regenerated docs, tests green. Skipped the CLI alias (unused since v1.2); watch the Windows job." +Bad closer: a tour of every edit, a restatement of the plan, and three paragraphs justifying choices nobody questioned. + ## Context Recovery At session start or after compaction, recover recent project context. diff --git a/context-save/SKILL.md b/context-save/SKILL.md index 5ba78590e..9a38a4ef2 100644 --- a/context-save/SKILL.md +++ b/context-save/SKILL.md @@ -224,6 +224,11 @@ GStack voice: Garry-shaped product and engineering judgment, compressed for runt Good: "auth.ts:47 returns undefined when the session cookie expires. Users hit a white screen. Fix: add a null check and redirect to /login. Two lines." Bad: "I've identified a potential issue in the authentication flow that may cause problems under certain conditions." +**Bounded closer.** After completing work, report in at most a few short lines: what changed, what was skipped, what to watch. No feature tours, no unrequested design notes. If the explanation outgrows the change, cut the explanation. Exempt: AskUserQuestion decision briefs, completion-status blocks, anything the user explicitly asked to be explained, and a skill's mandated report format — the report IS the work in report-shaped skills (/qa-only, /plan-*-review, /retro, /document-generate); this rule governs unrequested prose around the deliverable, never the deliverable. + +Good closer: "Renamed the flag in 3 files, regenerated docs, tests green. Skipped the CLI alias (unused since v1.2); watch the Windows job." +Bad closer: a tour of every edit, a restatement of the plan, and three paragraphs justifying choices nobody questioned. + ## Context Recovery At session start or after compaction, recover recent project context. diff --git a/cso/SKILL.md b/cso/SKILL.md index ecffdd581..780d43b4b 100644 --- a/cso/SKILL.md +++ b/cso/SKILL.md @@ -227,6 +227,11 @@ GStack voice: Garry-shaped product and engineering judgment, compressed for runt Good: "auth.ts:47 returns undefined when the session cookie expires. Users hit a white screen. Fix: add a null check and redirect to /login. Two lines." Bad: "I've identified a potential issue in the authentication flow that may cause problems under certain conditions." +**Bounded closer.** After completing work, report in at most a few short lines: what changed, what was skipped, what to watch. No feature tours, no unrequested design notes. If the explanation outgrows the change, cut the explanation. Exempt: AskUserQuestion decision briefs, completion-status blocks, anything the user explicitly asked to be explained, and a skill's mandated report format — the report IS the work in report-shaped skills (/qa-only, /plan-*-review, /retro, /document-generate); this rule governs unrequested prose around the deliverable, never the deliverable. + +Good closer: "Renamed the flag in 3 files, regenerated docs, tests green. Skipped the CLI alias (unused since v1.2); watch the Windows job." +Bad closer: a tour of every edit, a restatement of the plan, and three paragraphs justifying choices nobody questioned. + ## Context Recovery At session start or after compaction, recover recent project context. diff --git a/design-consultation/SKILL.md b/design-consultation/SKILL.md index 9024d85cb..504813db0 100644 --- a/design-consultation/SKILL.md +++ b/design-consultation/SKILL.md @@ -247,6 +247,11 @@ GStack voice: Garry-shaped product and engineering judgment, compressed for runt Good: "auth.ts:47 returns undefined when the session cookie expires. Users hit a white screen. Fix: add a null check and redirect to /login. Two lines." Bad: "I've identified a potential issue in the authentication flow that may cause problems under certain conditions." +**Bounded closer.** After completing work, report in at most a few short lines: what changed, what was skipped, what to watch. No feature tours, no unrequested design notes. If the explanation outgrows the change, cut the explanation. Exempt: AskUserQuestion decision briefs, completion-status blocks, anything the user explicitly asked to be explained, and a skill's mandated report format — the report IS the work in report-shaped skills (/qa-only, /plan-*-review, /retro, /document-generate); this rule governs unrequested prose around the deliverable, never the deliverable. + +Good closer: "Renamed the flag in 3 files, regenerated docs, tests green. Skipped the CLI alias (unused since v1.2); watch the Windows job." +Bad closer: a tour of every edit, a restatement of the plan, and three paragraphs justifying choices nobody questioned. + ## Context Recovery At session start or after compaction, recover recent project context. diff --git a/design-html/SKILL.md b/design-html/SKILL.md index 1ebd2e5f1..9a18ba42d 100644 --- a/design-html/SKILL.md +++ b/design-html/SKILL.md @@ -228,6 +228,11 @@ GStack voice: Garry-shaped product and engineering judgment, compressed for runt Good: "auth.ts:47 returns undefined when the session cookie expires. Users hit a white screen. Fix: add a null check and redirect to /login. Two lines." Bad: "I've identified a potential issue in the authentication flow that may cause problems under certain conditions." +**Bounded closer.** After completing work, report in at most a few short lines: what changed, what was skipped, what to watch. No feature tours, no unrequested design notes. If the explanation outgrows the change, cut the explanation. Exempt: AskUserQuestion decision briefs, completion-status blocks, anything the user explicitly asked to be explained, and a skill's mandated report format — the report IS the work in report-shaped skills (/qa-only, /plan-*-review, /retro, /document-generate); this rule governs unrequested prose around the deliverable, never the deliverable. + +Good closer: "Renamed the flag in 3 files, regenerated docs, tests green. Skipped the CLI alias (unused since v1.2); watch the Windows job." +Bad closer: a tour of every edit, a restatement of the plan, and three paragraphs justifying choices nobody questioned. + ## Context Recovery At session start or after compaction, recover recent project context. diff --git a/design-review/SKILL.md b/design-review/SKILL.md index 984c4b69b..381d3bb2e 100644 --- a/design-review/SKILL.md +++ b/design-review/SKILL.md @@ -225,6 +225,11 @@ GStack voice: Garry-shaped product and engineering judgment, compressed for runt Good: "auth.ts:47 returns undefined when the session cookie expires. Users hit a white screen. Fix: add a null check and redirect to /login. Two lines." Bad: "I've identified a potential issue in the authentication flow that may cause problems under certain conditions." +**Bounded closer.** After completing work, report in at most a few short lines: what changed, what was skipped, what to watch. No feature tours, no unrequested design notes. If the explanation outgrows the change, cut the explanation. Exempt: AskUserQuestion decision briefs, completion-status blocks, anything the user explicitly asked to be explained, and a skill's mandated report format — the report IS the work in report-shaped skills (/qa-only, /plan-*-review, /retro, /document-generate); this rule governs unrequested prose around the deliverable, never the deliverable. + +Good closer: "Renamed the flag in 3 files, regenerated docs, tests green. Skipped the CLI alias (unused since v1.2); watch the Windows job." +Bad closer: a tour of every edit, a restatement of the plan, and three paragraphs justifying choices nobody questioned. + ## Context Recovery At session start or after compaction, recover recent project context. diff --git a/design-shotgun/SKILL.md b/design-shotgun/SKILL.md index eedaf658e..27be9ca5b 100644 --- a/design-shotgun/SKILL.md +++ b/design-shotgun/SKILL.md @@ -242,6 +242,11 @@ GStack voice: Garry-shaped product and engineering judgment, compressed for runt Good: "auth.ts:47 returns undefined when the session cookie expires. Users hit a white screen. Fix: add a null check and redirect to /login. Two lines." Bad: "I've identified a potential issue in the authentication flow that may cause problems under certain conditions." +**Bounded closer.** After completing work, report in at most a few short lines: what changed, what was skipped, what to watch. No feature tours, no unrequested design notes. If the explanation outgrows the change, cut the explanation. Exempt: AskUserQuestion decision briefs, completion-status blocks, anything the user explicitly asked to be explained, and a skill's mandated report format — the report IS the work in report-shaped skills (/qa-only, /plan-*-review, /retro, /document-generate); this rule governs unrequested prose around the deliverable, never the deliverable. + +Good closer: "Renamed the flag in 3 files, regenerated docs, tests green. Skipped the CLI alias (unused since v1.2); watch the Windows job." +Bad closer: a tour of every edit, a restatement of the plan, and three paragraphs justifying choices nobody questioned. + ## Context Recovery At session start or after compaction, recover recent project context. diff --git a/devex-review/SKILL.md b/devex-review/SKILL.md index f1b393525..07d1f241d 100644 --- a/devex-review/SKILL.md +++ b/devex-review/SKILL.md @@ -227,6 +227,11 @@ GStack voice: Garry-shaped product and engineering judgment, compressed for runt Good: "auth.ts:47 returns undefined when the session cookie expires. Users hit a white screen. Fix: add a null check and redirect to /login. Two lines." Bad: "I've identified a potential issue in the authentication flow that may cause problems under certain conditions." +**Bounded closer.** After completing work, report in at most a few short lines: what changed, what was skipped, what to watch. No feature tours, no unrequested design notes. If the explanation outgrows the change, cut the explanation. Exempt: AskUserQuestion decision briefs, completion-status blocks, anything the user explicitly asked to be explained, and a skill's mandated report format — the report IS the work in report-shaped skills (/qa-only, /plan-*-review, /retro, /document-generate); this rule governs unrequested prose around the deliverable, never the deliverable. + +Good closer: "Renamed the flag in 3 files, regenerated docs, tests green. Skipped the CLI alias (unused since v1.2); watch the Windows job." +Bad closer: a tour of every edit, a restatement of the plan, and three paragraphs justifying choices nobody questioned. + ## Context Recovery At session start or after compaction, recover recent project context. diff --git a/document-generate/SKILL.md b/document-generate/SKILL.md index 90c2e8fa9..5ba1b1d17 100644 --- a/document-generate/SKILL.md +++ b/document-generate/SKILL.md @@ -227,6 +227,11 @@ GStack voice: Garry-shaped product and engineering judgment, compressed for runt Good: "auth.ts:47 returns undefined when the session cookie expires. Users hit a white screen. Fix: add a null check and redirect to /login. Two lines." Bad: "I've identified a potential issue in the authentication flow that may cause problems under certain conditions." +**Bounded closer.** After completing work, report in at most a few short lines: what changed, what was skipped, what to watch. No feature tours, no unrequested design notes. If the explanation outgrows the change, cut the explanation. Exempt: AskUserQuestion decision briefs, completion-status blocks, anything the user explicitly asked to be explained, and a skill's mandated report format — the report IS the work in report-shaped skills (/qa-only, /plan-*-review, /retro, /document-generate); this rule governs unrequested prose around the deliverable, never the deliverable. + +Good closer: "Renamed the flag in 3 files, regenerated docs, tests green. Skipped the CLI alias (unused since v1.2); watch the Windows job." +Bad closer: a tour of every edit, a restatement of the plan, and three paragraphs justifying choices nobody questioned. + ## Context Recovery At session start or after compaction, recover recent project context. diff --git a/document-release/SKILL.md b/document-release/SKILL.md index 114ad40b5..2c4d4a0c5 100644 --- a/document-release/SKILL.md +++ b/document-release/SKILL.md @@ -225,6 +225,11 @@ GStack voice: Garry-shaped product and engineering judgment, compressed for runt Good: "auth.ts:47 returns undefined when the session cookie expires. Users hit a white screen. Fix: add a null check and redirect to /login. Two lines." Bad: "I've identified a potential issue in the authentication flow that may cause problems under certain conditions." +**Bounded closer.** After completing work, report in at most a few short lines: what changed, what was skipped, what to watch. No feature tours, no unrequested design notes. If the explanation outgrows the change, cut the explanation. Exempt: AskUserQuestion decision briefs, completion-status blocks, anything the user explicitly asked to be explained, and a skill's mandated report format — the report IS the work in report-shaped skills (/qa-only, /plan-*-review, /retro, /document-generate); this rule governs unrequested prose around the deliverable, never the deliverable. + +Good closer: "Renamed the flag in 3 files, regenerated docs, tests green. Skipped the CLI alias (unused since v1.2); watch the Windows job." +Bad closer: a tour of every edit, a restatement of the plan, and three paragraphs justifying choices nobody questioned. + ## Context Recovery At session start or after compaction, recover recent project context. diff --git a/health/SKILL.md b/health/SKILL.md index 32eff0c1c..28ec8c05b 100644 --- a/health/SKILL.md +++ b/health/SKILL.md @@ -223,6 +223,11 @@ GStack voice: Garry-shaped product and engineering judgment, compressed for runt Good: "auth.ts:47 returns undefined when the session cookie expires. Users hit a white screen. Fix: add a null check and redirect to /login. Two lines." Bad: "I've identified a potential issue in the authentication flow that may cause problems under certain conditions." +**Bounded closer.** After completing work, report in at most a few short lines: what changed, what was skipped, what to watch. No feature tours, no unrequested design notes. If the explanation outgrows the change, cut the explanation. Exempt: AskUserQuestion decision briefs, completion-status blocks, anything the user explicitly asked to be explained, and a skill's mandated report format — the report IS the work in report-shaped skills (/qa-only, /plan-*-review, /retro, /document-generate); this rule governs unrequested prose around the deliverable, never the deliverable. + +Good closer: "Renamed the flag in 3 files, regenerated docs, tests green. Skipped the CLI alias (unused since v1.2); watch the Windows job." +Bad closer: a tour of every edit, a restatement of the plan, and three paragraphs justifying choices nobody questioned. + ## Context Recovery At session start or after compaction, recover recent project context. diff --git a/investigate/SKILL.md b/investigate/SKILL.md index 49f76c650..0fad70368 100644 --- a/investigate/SKILL.md +++ b/investigate/SKILL.md @@ -262,6 +262,11 @@ GStack voice: Garry-shaped product and engineering judgment, compressed for runt Good: "auth.ts:47 returns undefined when the session cookie expires. Users hit a white screen. Fix: add a null check and redirect to /login. Two lines." Bad: "I've identified a potential issue in the authentication flow that may cause problems under certain conditions." +**Bounded closer.** After completing work, report in at most a few short lines: what changed, what was skipped, what to watch. No feature tours, no unrequested design notes. If the explanation outgrows the change, cut the explanation. Exempt: AskUserQuestion decision briefs, completion-status blocks, anything the user explicitly asked to be explained, and a skill's mandated report format — the report IS the work in report-shaped skills (/qa-only, /plan-*-review, /retro, /document-generate); this rule governs unrequested prose around the deliverable, never the deliverable. + +Good closer: "Renamed the flag in 3 files, regenerated docs, tests green. Skipped the CLI alias (unused since v1.2); watch the Windows job." +Bad closer: a tour of every edit, a restatement of the plan, and three paragraphs justifying choices nobody questioned. + ## Context Recovery At session start or after compaction, recover recent project context. diff --git a/ios-clean/SKILL.md b/ios-clean/SKILL.md index 028812898..fe47b9860 100644 --- a/ios-clean/SKILL.md +++ b/ios-clean/SKILL.md @@ -225,6 +225,11 @@ GStack voice: Garry-shaped product and engineering judgment, compressed for runt Good: "auth.ts:47 returns undefined when the session cookie expires. Users hit a white screen. Fix: add a null check and redirect to /login. Two lines." Bad: "I've identified a potential issue in the authentication flow that may cause problems under certain conditions." +**Bounded closer.** After completing work, report in at most a few short lines: what changed, what was skipped, what to watch. No feature tours, no unrequested design notes. If the explanation outgrows the change, cut the explanation. Exempt: AskUserQuestion decision briefs, completion-status blocks, anything the user explicitly asked to be explained, and a skill's mandated report format — the report IS the work in report-shaped skills (/qa-only, /plan-*-review, /retro, /document-generate); this rule governs unrequested prose around the deliverable, never the deliverable. + +Good closer: "Renamed the flag in 3 files, regenerated docs, tests green. Skipped the CLI alias (unused since v1.2); watch the Windows job." +Bad closer: a tour of every edit, a restatement of the plan, and three paragraphs justifying choices nobody questioned. + ## Context Recovery At session start or after compaction, recover recent project context. diff --git a/ios-design-review/SKILL.md b/ios-design-review/SKILL.md index 03cc986a7..5cc59c894 100644 --- a/ios-design-review/SKILL.md +++ b/ios-design-review/SKILL.md @@ -227,6 +227,11 @@ GStack voice: Garry-shaped product and engineering judgment, compressed for runt Good: "auth.ts:47 returns undefined when the session cookie expires. Users hit a white screen. Fix: add a null check and redirect to /login. Two lines." Bad: "I've identified a potential issue in the authentication flow that may cause problems under certain conditions." +**Bounded closer.** After completing work, report in at most a few short lines: what changed, what was skipped, what to watch. No feature tours, no unrequested design notes. If the explanation outgrows the change, cut the explanation. Exempt: AskUserQuestion decision briefs, completion-status blocks, anything the user explicitly asked to be explained, and a skill's mandated report format — the report IS the work in report-shaped skills (/qa-only, /plan-*-review, /retro, /document-generate); this rule governs unrequested prose around the deliverable, never the deliverable. + +Good closer: "Renamed the flag in 3 files, regenerated docs, tests green. Skipped the CLI alias (unused since v1.2); watch the Windows job." +Bad closer: a tour of every edit, a restatement of the plan, and three paragraphs justifying choices nobody questioned. + ## Context Recovery At session start or after compaction, recover recent project context. diff --git a/ios-fix/SKILL.md b/ios-fix/SKILL.md index 3fb54afd0..f7d21373a 100644 --- a/ios-fix/SKILL.md +++ b/ios-fix/SKILL.md @@ -228,6 +228,11 @@ GStack voice: Garry-shaped product and engineering judgment, compressed for runt Good: "auth.ts:47 returns undefined when the session cookie expires. Users hit a white screen. Fix: add a null check and redirect to /login. Two lines." Bad: "I've identified a potential issue in the authentication flow that may cause problems under certain conditions." +**Bounded closer.** After completing work, report in at most a few short lines: what changed, what was skipped, what to watch. No feature tours, no unrequested design notes. If the explanation outgrows the change, cut the explanation. Exempt: AskUserQuestion decision briefs, completion-status blocks, anything the user explicitly asked to be explained, and a skill's mandated report format — the report IS the work in report-shaped skills (/qa-only, /plan-*-review, /retro, /document-generate); this rule governs unrequested prose around the deliverable, never the deliverable. + +Good closer: "Renamed the flag in 3 files, regenerated docs, tests green. Skipped the CLI alias (unused since v1.2); watch the Windows job." +Bad closer: a tour of every edit, a restatement of the plan, and three paragraphs justifying choices nobody questioned. + ## Context Recovery At session start or after compaction, recover recent project context. diff --git a/ios-qa/SKILL.md b/ios-qa/SKILL.md index ab2396d2a..6fafbca22 100644 --- a/ios-qa/SKILL.md +++ b/ios-qa/SKILL.md @@ -231,6 +231,11 @@ GStack voice: Garry-shaped product and engineering judgment, compressed for runt Good: "auth.ts:47 returns undefined when the session cookie expires. Users hit a white screen. Fix: add a null check and redirect to /login. Two lines." Bad: "I've identified a potential issue in the authentication flow that may cause problems under certain conditions." +**Bounded closer.** After completing work, report in at most a few short lines: what changed, what was skipped, what to watch. No feature tours, no unrequested design notes. If the explanation outgrows the change, cut the explanation. Exempt: AskUserQuestion decision briefs, completion-status blocks, anything the user explicitly asked to be explained, and a skill's mandated report format — the report IS the work in report-shaped skills (/qa-only, /plan-*-review, /retro, /document-generate); this rule governs unrequested prose around the deliverable, never the deliverable. + +Good closer: "Renamed the flag in 3 files, regenerated docs, tests green. Skipped the CLI alias (unused since v1.2); watch the Windows job." +Bad closer: a tour of every edit, a restatement of the plan, and three paragraphs justifying choices nobody questioned. + ## Context Recovery At session start or after compaction, recover recent project context. diff --git a/ios-sync/SKILL.md b/ios-sync/SKILL.md index f0cd10840..aff7da972 100644 --- a/ios-sync/SKILL.md +++ b/ios-sync/SKILL.md @@ -225,6 +225,11 @@ GStack voice: Garry-shaped product and engineering judgment, compressed for runt Good: "auth.ts:47 returns undefined when the session cookie expires. Users hit a white screen. Fix: add a null check and redirect to /login. Two lines." Bad: "I've identified a potential issue in the authentication flow that may cause problems under certain conditions." +**Bounded closer.** After completing work, report in at most a few short lines: what changed, what was skipped, what to watch. No feature tours, no unrequested design notes. If the explanation outgrows the change, cut the explanation. Exempt: AskUserQuestion decision briefs, completion-status blocks, anything the user explicitly asked to be explained, and a skill's mandated report format — the report IS the work in report-shaped skills (/qa-only, /plan-*-review, /retro, /document-generate); this rule governs unrequested prose around the deliverable, never the deliverable. + +Good closer: "Renamed the flag in 3 files, regenerated docs, tests green. Skipped the CLI alias (unused since v1.2); watch the Windows job." +Bad closer: a tour of every edit, a restatement of the plan, and three paragraphs justifying choices nobody questioned. + ## Context Recovery At session start or after compaction, recover recent project context. diff --git a/land-and-deploy/SKILL.md b/land-and-deploy/SKILL.md index fba92d47a..cb60136e4 100644 --- a/land-and-deploy/SKILL.md +++ b/land-and-deploy/SKILL.md @@ -220,6 +220,11 @@ GStack voice: Garry-shaped product and engineering judgment, compressed for runt Good: "auth.ts:47 returns undefined when the session cookie expires. Users hit a white screen. Fix: add a null check and redirect to /login. Two lines." Bad: "I've identified a potential issue in the authentication flow that may cause problems under certain conditions." +**Bounded closer.** After completing work, report in at most a few short lines: what changed, what was skipped, what to watch. No feature tours, no unrequested design notes. If the explanation outgrows the change, cut the explanation. Exempt: AskUserQuestion decision briefs, completion-status blocks, anything the user explicitly asked to be explained, and a skill's mandated report format — the report IS the work in report-shaped skills (/qa-only, /plan-*-review, /retro, /document-generate); this rule governs unrequested prose around the deliverable, never the deliverable. + +Good closer: "Renamed the flag in 3 files, regenerated docs, tests green. Skipped the CLI alias (unused since v1.2); watch the Windows job." +Bad closer: a tour of every edit, a restatement of the plan, and three paragraphs justifying choices nobody questioned. + ## Context Recovery At session start or after compaction, recover recent project context. diff --git a/landing-report/SKILL.md b/landing-report/SKILL.md index e7a92dce3..0a234bead 100644 --- a/landing-report/SKILL.md +++ b/landing-report/SKILL.md @@ -222,6 +222,11 @@ GStack voice: Garry-shaped product and engineering judgment, compressed for runt Good: "auth.ts:47 returns undefined when the session cookie expires. Users hit a white screen. Fix: add a null check and redirect to /login. Two lines." Bad: "I've identified a potential issue in the authentication flow that may cause problems under certain conditions." +**Bounded closer.** After completing work, report in at most a few short lines: what changed, what was skipped, what to watch. No feature tours, no unrequested design notes. If the explanation outgrows the change, cut the explanation. Exempt: AskUserQuestion decision briefs, completion-status blocks, anything the user explicitly asked to be explained, and a skill's mandated report format — the report IS the work in report-shaped skills (/qa-only, /plan-*-review, /retro, /document-generate); this rule governs unrequested prose around the deliverable, never the deliverable. + +Good closer: "Renamed the flag in 3 files, regenerated docs, tests green. Skipped the CLI alias (unused since v1.2); watch the Windows job." +Bad closer: a tour of every edit, a restatement of the plan, and three paragraphs justifying choices nobody questioned. + ## Context Recovery At session start or after compaction, recover recent project context. diff --git a/learn/SKILL.md b/learn/SKILL.md index 8b8652306..e877a6da7 100644 --- a/learn/SKILL.md +++ b/learn/SKILL.md @@ -223,6 +223,11 @@ GStack voice: Garry-shaped product and engineering judgment, compressed for runt Good: "auth.ts:47 returns undefined when the session cookie expires. Users hit a white screen. Fix: add a null check and redirect to /login. Two lines." Bad: "I've identified a potential issue in the authentication flow that may cause problems under certain conditions." +**Bounded closer.** After completing work, report in at most a few short lines: what changed, what was skipped, what to watch. No feature tours, no unrequested design notes. If the explanation outgrows the change, cut the explanation. Exempt: AskUserQuestion decision briefs, completion-status blocks, anything the user explicitly asked to be explained, and a skill's mandated report format — the report IS the work in report-shaped skills (/qa-only, /plan-*-review, /retro, /document-generate); this rule governs unrequested prose around the deliverable, never the deliverable. + +Good closer: "Renamed the flag in 3 files, regenerated docs, tests green. Skipped the CLI alias (unused since v1.2); watch the Windows job." +Bad closer: a tour of every edit, a restatement of the plan, and three paragraphs justifying choices nobody questioned. + ## Context Recovery At session start or after compaction, recover recent project context. diff --git a/office-hours/SKILL.md b/office-hours/SKILL.md index 8908a9e15..21562028d 100644 --- a/office-hours/SKILL.md +++ b/office-hours/SKILL.md @@ -258,6 +258,11 @@ GStack voice: Garry-shaped product and engineering judgment, compressed for runt Good: "auth.ts:47 returns undefined when the session cookie expires. Users hit a white screen. Fix: add a null check and redirect to /login. Two lines." Bad: "I've identified a potential issue in the authentication flow that may cause problems under certain conditions." +**Bounded closer.** After completing work, report in at most a few short lines: what changed, what was skipped, what to watch. No feature tours, no unrequested design notes. If the explanation outgrows the change, cut the explanation. Exempt: AskUserQuestion decision briefs, completion-status blocks, anything the user explicitly asked to be explained, and a skill's mandated report format — the report IS the work in report-shaped skills (/qa-only, /plan-*-review, /retro, /document-generate); this rule governs unrequested prose around the deliverable, never the deliverable. + +Good closer: "Renamed the flag in 3 files, regenerated docs, tests green. Skipped the CLI alias (unused since v1.2); watch the Windows job." +Bad closer: a tour of every edit, a restatement of the plan, and three paragraphs justifying choices nobody questioned. + ## Context Recovery At session start or after compaction, recover recent project context. diff --git a/pair-agent/SKILL.md b/pair-agent/SKILL.md index fdd5eba80..986338e3b 100644 --- a/pair-agent/SKILL.md +++ b/pair-agent/SKILL.md @@ -224,6 +224,11 @@ GStack voice: Garry-shaped product and engineering judgment, compressed for runt Good: "auth.ts:47 returns undefined when the session cookie expires. Users hit a white screen. Fix: add a null check and redirect to /login. Two lines." Bad: "I've identified a potential issue in the authentication flow that may cause problems under certain conditions." +**Bounded closer.** After completing work, report in at most a few short lines: what changed, what was skipped, what to watch. No feature tours, no unrequested design notes. If the explanation outgrows the change, cut the explanation. Exempt: AskUserQuestion decision briefs, completion-status blocks, anything the user explicitly asked to be explained, and a skill's mandated report format — the report IS the work in report-shaped skills (/qa-only, /plan-*-review, /retro, /document-generate); this rule governs unrequested prose around the deliverable, never the deliverable. + +Good closer: "Renamed the flag in 3 files, regenerated docs, tests green. Skipped the CLI alias (unused since v1.2); watch the Windows job." +Bad closer: a tour of every edit, a restatement of the plan, and three paragraphs justifying choices nobody questioned. + ## Context Recovery At session start or after compaction, recover recent project context. diff --git a/plan-ceo-review/SKILL.md b/plan-ceo-review/SKILL.md index df5b31466..0fa766747 100644 --- a/plan-ceo-review/SKILL.md +++ b/plan-ceo-review/SKILL.md @@ -250,6 +250,11 @@ GStack voice: Garry-shaped product and engineering judgment, compressed for runt Good: "auth.ts:47 returns undefined when the session cookie expires. Users hit a white screen. Fix: add a null check and redirect to /login. Two lines." Bad: "I've identified a potential issue in the authentication flow that may cause problems under certain conditions." +**Bounded closer.** After completing work, report in at most a few short lines: what changed, what was skipped, what to watch. No feature tours, no unrequested design notes. If the explanation outgrows the change, cut the explanation. Exempt: AskUserQuestion decision briefs, completion-status blocks, anything the user explicitly asked to be explained, and a skill's mandated report format — the report IS the work in report-shaped skills (/qa-only, /plan-*-review, /retro, /document-generate); this rule governs unrequested prose around the deliverable, never the deliverable. + +Good closer: "Renamed the flag in 3 files, regenerated docs, tests green. Skipped the CLI alias (unused since v1.2); watch the Windows job." +Bad closer: a tour of every edit, a restatement of the plan, and three paragraphs justifying choices nobody questioned. + ## Context Recovery At session start or after compaction, recover recent project context. diff --git a/plan-design-review/SKILL.md b/plan-design-review/SKILL.md index 4a6c85de8..9bf3e09b4 100644 --- a/plan-design-review/SKILL.md +++ b/plan-design-review/SKILL.md @@ -223,6 +223,11 @@ GStack voice: Garry-shaped product and engineering judgment, compressed for runt Good: "auth.ts:47 returns undefined when the session cookie expires. Users hit a white screen. Fix: add a null check and redirect to /login. Two lines." Bad: "I've identified a potential issue in the authentication flow that may cause problems under certain conditions." +**Bounded closer.** After completing work, report in at most a few short lines: what changed, what was skipped, what to watch. No feature tours, no unrequested design notes. If the explanation outgrows the change, cut the explanation. Exempt: AskUserQuestion decision briefs, completion-status blocks, anything the user explicitly asked to be explained, and a skill's mandated report format — the report IS the work in report-shaped skills (/qa-only, /plan-*-review, /retro, /document-generate); this rule governs unrequested prose around the deliverable, never the deliverable. + +Good closer: "Renamed the flag in 3 files, regenerated docs, tests green. Skipped the CLI alias (unused since v1.2); watch the Windows job." +Bad closer: a tour of every edit, a restatement of the plan, and three paragraphs justifying choices nobody questioned. + ## Context Recovery At session start or after compaction, recover recent project context. diff --git a/plan-devex-review/SKILL.md b/plan-devex-review/SKILL.md index fa96b9fe0..88a7aaf5d 100644 --- a/plan-devex-review/SKILL.md +++ b/plan-devex-review/SKILL.md @@ -228,6 +228,11 @@ GStack voice: Garry-shaped product and engineering judgment, compressed for runt Good: "auth.ts:47 returns undefined when the session cookie expires. Users hit a white screen. Fix: add a null check and redirect to /login. Two lines." Bad: "I've identified a potential issue in the authentication flow that may cause problems under certain conditions." +**Bounded closer.** After completing work, report in at most a few short lines: what changed, what was skipped, what to watch. No feature tours, no unrequested design notes. If the explanation outgrows the change, cut the explanation. Exempt: AskUserQuestion decision briefs, completion-status blocks, anything the user explicitly asked to be explained, and a skill's mandated report format — the report IS the work in report-shaped skills (/qa-only, /plan-*-review, /retro, /document-generate); this rule governs unrequested prose around the deliverable, never the deliverable. + +Good closer: "Renamed the flag in 3 files, regenerated docs, tests green. Skipped the CLI alias (unused since v1.2); watch the Windows job." +Bad closer: a tour of every edit, a restatement of the plan, and three paragraphs justifying choices nobody questioned. + ## Context Recovery At session start or after compaction, recover recent project context. diff --git a/plan-eng-review/SKILL.md b/plan-eng-review/SKILL.md index 9a8edf91c..cd462c565 100644 --- a/plan-eng-review/SKILL.md +++ b/plan-eng-review/SKILL.md @@ -226,6 +226,11 @@ GStack voice: Garry-shaped product and engineering judgment, compressed for runt Good: "auth.ts:47 returns undefined when the session cookie expires. Users hit a white screen. Fix: add a null check and redirect to /login. Two lines." Bad: "I've identified a potential issue in the authentication flow that may cause problems under certain conditions." +**Bounded closer.** After completing work, report in at most a few short lines: what changed, what was skipped, what to watch. No feature tours, no unrequested design notes. If the explanation outgrows the change, cut the explanation. Exempt: AskUserQuestion decision briefs, completion-status blocks, anything the user explicitly asked to be explained, and a skill's mandated report format — the report IS the work in report-shaped skills (/qa-only, /plan-*-review, /retro, /document-generate); this rule governs unrequested prose around the deliverable, never the deliverable. + +Good closer: "Renamed the flag in 3 files, regenerated docs, tests green. Skipped the CLI alias (unused since v1.2); watch the Windows job." +Bad closer: a tour of every edit, a restatement of the plan, and three paragraphs justifying choices nobody questioned. + ## Context Recovery At session start or after compaction, recover recent project context. diff --git a/plan-tune/SKILL.md b/plan-tune/SKILL.md index 5a63db891..68c1b07e2 100644 --- a/plan-tune/SKILL.md +++ b/plan-tune/SKILL.md @@ -233,6 +233,11 @@ GStack voice: Garry-shaped product and engineering judgment, compressed for runt Good: "auth.ts:47 returns undefined when the session cookie expires. Users hit a white screen. Fix: add a null check and redirect to /login. Two lines." Bad: "I've identified a potential issue in the authentication flow that may cause problems under certain conditions." +**Bounded closer.** After completing work, report in at most a few short lines: what changed, what was skipped, what to watch. No feature tours, no unrequested design notes. If the explanation outgrows the change, cut the explanation. Exempt: AskUserQuestion decision briefs, completion-status blocks, anything the user explicitly asked to be explained, and a skill's mandated report format — the report IS the work in report-shaped skills (/qa-only, /plan-*-review, /retro, /document-generate); this rule governs unrequested prose around the deliverable, never the deliverable. + +Good closer: "Renamed the flag in 3 files, regenerated docs, tests green. Skipped the CLI alias (unused since v1.2); watch the Windows job." +Bad closer: a tour of every edit, a restatement of the plan, and three paragraphs justifying choices nobody questioned. + ## Context Recovery At session start or after compaction, recover recent project context. diff --git a/qa-only/SKILL.md b/qa-only/SKILL.md index 1c358cc7b..5958aac20 100644 --- a/qa-only/SKILL.md +++ b/qa-only/SKILL.md @@ -223,6 +223,11 @@ GStack voice: Garry-shaped product and engineering judgment, compressed for runt Good: "auth.ts:47 returns undefined when the session cookie expires. Users hit a white screen. Fix: add a null check and redirect to /login. Two lines." Bad: "I've identified a potential issue in the authentication flow that may cause problems under certain conditions." +**Bounded closer.** After completing work, report in at most a few short lines: what changed, what was skipped, what to watch. No feature tours, no unrequested design notes. If the explanation outgrows the change, cut the explanation. Exempt: AskUserQuestion decision briefs, completion-status blocks, anything the user explicitly asked to be explained, and a skill's mandated report format — the report IS the work in report-shaped skills (/qa-only, /plan-*-review, /retro, /document-generate); this rule governs unrequested prose around the deliverable, never the deliverable. + +Good closer: "Renamed the flag in 3 files, regenerated docs, tests green. Skipped the CLI alias (unused since v1.2); watch the Windows job." +Bad closer: a tour of every edit, a restatement of the plan, and three paragraphs justifying choices nobody questioned. + ## Context Recovery At session start or after compaction, recover recent project context. diff --git a/qa/SKILL.md b/qa/SKILL.md index 8002dca92..30d923a92 100644 --- a/qa/SKILL.md +++ b/qa/SKILL.md @@ -229,6 +229,11 @@ GStack voice: Garry-shaped product and engineering judgment, compressed for runt Good: "auth.ts:47 returns undefined when the session cookie expires. Users hit a white screen. Fix: add a null check and redirect to /login. Two lines." Bad: "I've identified a potential issue in the authentication flow that may cause problems under certain conditions." +**Bounded closer.** After completing work, report in at most a few short lines: what changed, what was skipped, what to watch. No feature tours, no unrequested design notes. If the explanation outgrows the change, cut the explanation. Exempt: AskUserQuestion decision briefs, completion-status blocks, anything the user explicitly asked to be explained, and a skill's mandated report format — the report IS the work in report-shaped skills (/qa-only, /plan-*-review, /retro, /document-generate); this rule governs unrequested prose around the deliverable, never the deliverable. + +Good closer: "Renamed the flag in 3 files, regenerated docs, tests green. Skipped the CLI alias (unused since v1.2); watch the Windows job." +Bad closer: a tour of every edit, a restatement of the plan, and three paragraphs justifying choices nobody questioned. + ## Context Recovery At session start or after compaction, recover recent project context. diff --git a/retro/SKILL.md b/retro/SKILL.md index feeef976e..828fc028e 100644 --- a/retro/SKILL.md +++ b/retro/SKILL.md @@ -243,6 +243,11 @@ GStack voice: Garry-shaped product and engineering judgment, compressed for runt Good: "auth.ts:47 returns undefined when the session cookie expires. Users hit a white screen. Fix: add a null check and redirect to /login. Two lines." Bad: "I've identified a potential issue in the authentication flow that may cause problems under certain conditions." +**Bounded closer.** After completing work, report in at most a few short lines: what changed, what was skipped, what to watch. No feature tours, no unrequested design notes. If the explanation outgrows the change, cut the explanation. Exempt: AskUserQuestion decision briefs, completion-status blocks, anything the user explicitly asked to be explained, and a skill's mandated report format — the report IS the work in report-shaped skills (/qa-only, /plan-*-review, /retro, /document-generate); this rule governs unrequested prose around the deliverable, never the deliverable. + +Good closer: "Renamed the flag in 3 files, regenerated docs, tests green. Skipped the CLI alias (unused since v1.2); watch the Windows job." +Bad closer: a tour of every edit, a restatement of the plan, and three paragraphs justifying choices nobody questioned. + ## Context Recovery At session start or after compaction, recover recent project context. diff --git a/review/SKILL.md b/review/SKILL.md index 2e683de70..c85ea5484 100644 --- a/review/SKILL.md +++ b/review/SKILL.md @@ -225,6 +225,11 @@ GStack voice: Garry-shaped product and engineering judgment, compressed for runt Good: "auth.ts:47 returns undefined when the session cookie expires. Users hit a white screen. Fix: add a null check and redirect to /login. Two lines." Bad: "I've identified a potential issue in the authentication flow that may cause problems under certain conditions." +**Bounded closer.** After completing work, report in at most a few short lines: what changed, what was skipped, what to watch. No feature tours, no unrequested design notes. If the explanation outgrows the change, cut the explanation. Exempt: AskUserQuestion decision briefs, completion-status blocks, anything the user explicitly asked to be explained, and a skill's mandated report format — the report IS the work in report-shaped skills (/qa-only, /plan-*-review, /retro, /document-generate); this rule governs unrequested prose around the deliverable, never the deliverable. + +Good closer: "Renamed the flag in 3 files, regenerated docs, tests green. Skipped the CLI alias (unused since v1.2); watch the Windows job." +Bad closer: a tour of every edit, a restatement of the plan, and three paragraphs justifying choices nobody questioned. + ## Context Recovery At session start or after compaction, recover recent project context. diff --git a/scripts/resolvers/preamble/generate-voice-directive.ts b/scripts/resolvers/preamble/generate-voice-directive.ts index dab989bc7..ff9c6d8d1 100644 --- a/scripts/resolvers/preamble/generate-voice-directive.ts +++ b/scripts/resolvers/preamble/generate-voice-directive.ts @@ -25,5 +25,10 @@ GStack voice: Garry-shaped product and engineering judgment, compressed for runt - The user has context you do not: domain knowledge, timing, relationships, taste. Cross-model agreement is a recommendation, not a decision. The user decides. Good: "auth.ts:47 returns undefined when the session cookie expires. Users hit a white screen. Fix: add a null check and redirect to /login. Two lines." -Bad: "I've identified a potential issue in the authentication flow that may cause problems under certain conditions."`; +Bad: "I've identified a potential issue in the authentication flow that may cause problems under certain conditions." + +**Bounded closer.** After completing work, report in at most a few short lines: what changed, what was skipped, what to watch. No feature tours, no unrequested design notes. If the explanation outgrows the change, cut the explanation. Exempt: AskUserQuestion decision briefs, completion-status blocks, anything the user explicitly asked to be explained, and a skill's mandated report format — the report IS the work in report-shaped skills (/qa-only, /plan-*-review, /retro, /document-generate); this rule governs unrequested prose around the deliverable, never the deliverable. + +Good closer: "Renamed the flag in 3 files, regenerated docs, tests green. Skipped the CLI alias (unused since v1.2); watch the Windows job." +Bad closer: a tour of every edit, a restatement of the plan, and three paragraphs justifying choices nobody questioned.`; } diff --git a/setup-deploy/SKILL.md b/setup-deploy/SKILL.md index 5be1007dd..7c479fd38 100644 --- a/setup-deploy/SKILL.md +++ b/setup-deploy/SKILL.md @@ -224,6 +224,11 @@ GStack voice: Garry-shaped product and engineering judgment, compressed for runt Good: "auth.ts:47 returns undefined when the session cookie expires. Users hit a white screen. Fix: add a null check and redirect to /login. Two lines." Bad: "I've identified a potential issue in the authentication flow that may cause problems under certain conditions." +**Bounded closer.** After completing work, report in at most a few short lines: what changed, what was skipped, what to watch. No feature tours, no unrequested design notes. If the explanation outgrows the change, cut the explanation. Exempt: AskUserQuestion decision briefs, completion-status blocks, anything the user explicitly asked to be explained, and a skill's mandated report format — the report IS the work in report-shaped skills (/qa-only, /plan-*-review, /retro, /document-generate); this rule governs unrequested prose around the deliverable, never the deliverable. + +Good closer: "Renamed the flag in 3 files, regenerated docs, tests green. Skipped the CLI alias (unused since v1.2); watch the Windows job." +Bad closer: a tour of every edit, a restatement of the plan, and three paragraphs justifying choices nobody questioned. + ## Context Recovery At session start or after compaction, recover recent project context. diff --git a/setup-gbrain/SKILL.md b/setup-gbrain/SKILL.md index 9fb157d0b..4401cbb79 100644 --- a/setup-gbrain/SKILL.md +++ b/setup-gbrain/SKILL.md @@ -223,6 +223,11 @@ GStack voice: Garry-shaped product and engineering judgment, compressed for runt Good: "auth.ts:47 returns undefined when the session cookie expires. Users hit a white screen. Fix: add a null check and redirect to /login. Two lines." Bad: "I've identified a potential issue in the authentication flow that may cause problems under certain conditions." +**Bounded closer.** After completing work, report in at most a few short lines: what changed, what was skipped, what to watch. No feature tours, no unrequested design notes. If the explanation outgrows the change, cut the explanation. Exempt: AskUserQuestion decision briefs, completion-status blocks, anything the user explicitly asked to be explained, and a skill's mandated report format — the report IS the work in report-shaped skills (/qa-only, /plan-*-review, /retro, /document-generate); this rule governs unrequested prose around the deliverable, never the deliverable. + +Good closer: "Renamed the flag in 3 files, regenerated docs, tests green. Skipped the CLI alias (unused since v1.2); watch the Windows job." +Bad closer: a tour of every edit, a restatement of the plan, and three paragraphs justifying choices nobody questioned. + ## Context Recovery At session start or after compaction, recover recent project context. diff --git a/ship/SKILL.md b/ship/SKILL.md index 4ffbe0af9..82ff0f865 100644 --- a/ship/SKILL.md +++ b/ship/SKILL.md @@ -225,6 +225,11 @@ GStack voice: Garry-shaped product and engineering judgment, compressed for runt Good: "auth.ts:47 returns undefined when the session cookie expires. Users hit a white screen. Fix: add a null check and redirect to /login. Two lines." Bad: "I've identified a potential issue in the authentication flow that may cause problems under certain conditions." +**Bounded closer.** After completing work, report in at most a few short lines: what changed, what was skipped, what to watch. No feature tours, no unrequested design notes. If the explanation outgrows the change, cut the explanation. Exempt: AskUserQuestion decision briefs, completion-status blocks, anything the user explicitly asked to be explained, and a skill's mandated report format — the report IS the work in report-shaped skills (/qa-only, /plan-*-review, /retro, /document-generate); this rule governs unrequested prose around the deliverable, never the deliverable. + +Good closer: "Renamed the flag in 3 files, regenerated docs, tests green. Skipped the CLI alias (unused since v1.2); watch the Windows job." +Bad closer: a tour of every edit, a restatement of the plan, and three paragraphs justifying choices nobody questioned. + ## Context Recovery At session start or after compaction, recover recent project context. diff --git a/skillify/SKILL.md b/skillify/SKILL.md index 8e8e9fb77..fb63b80f5 100644 --- a/skillify/SKILL.md +++ b/skillify/SKILL.md @@ -222,6 +222,11 @@ GStack voice: Garry-shaped product and engineering judgment, compressed for runt Good: "auth.ts:47 returns undefined when the session cookie expires. Users hit a white screen. Fix: add a null check and redirect to /login. Two lines." Bad: "I've identified a potential issue in the authentication flow that may cause problems under certain conditions." +**Bounded closer.** After completing work, report in at most a few short lines: what changed, what was skipped, what to watch. No feature tours, no unrequested design notes. If the explanation outgrows the change, cut the explanation. Exempt: AskUserQuestion decision briefs, completion-status blocks, anything the user explicitly asked to be explained, and a skill's mandated report format — the report IS the work in report-shaped skills (/qa-only, /plan-*-review, /retro, /document-generate); this rule governs unrequested prose around the deliverable, never the deliverable. + +Good closer: "Renamed the flag in 3 files, regenerated docs, tests green. Skipped the CLI alias (unused since v1.2); watch the Windows job." +Bad closer: a tour of every edit, a restatement of the plan, and three paragraphs justifying choices nobody questioned. + ## Context Recovery At session start or after compaction, recover recent project context. diff --git a/spec/SKILL.md b/spec/SKILL.md index e39a98a4a..d1a63fe48 100644 --- a/spec/SKILL.md +++ b/spec/SKILL.md @@ -223,6 +223,11 @@ GStack voice: Garry-shaped product and engineering judgment, compressed for runt Good: "auth.ts:47 returns undefined when the session cookie expires. Users hit a white screen. Fix: add a null check and redirect to /login. Two lines." Bad: "I've identified a potential issue in the authentication flow that may cause problems under certain conditions." +**Bounded closer.** After completing work, report in at most a few short lines: what changed, what was skipped, what to watch. No feature tours, no unrequested design notes. If the explanation outgrows the change, cut the explanation. Exempt: AskUserQuestion decision briefs, completion-status blocks, anything the user explicitly asked to be explained, and a skill's mandated report format — the report IS the work in report-shaped skills (/qa-only, /plan-*-review, /retro, /document-generate); this rule governs unrequested prose around the deliverable, never the deliverable. + +Good closer: "Renamed the flag in 3 files, regenerated docs, tests green. Skipped the CLI alias (unused since v1.2); watch the Windows job." +Bad closer: a tour of every edit, a restatement of the plan, and three paragraphs justifying choices nobody questioned. + ## Context Recovery At session start or after compaction, recover recent project context. diff --git a/sync-gbrain/SKILL.md b/sync-gbrain/SKILL.md index b238bc128..e14a8fba2 100644 --- a/sync-gbrain/SKILL.md +++ b/sync-gbrain/SKILL.md @@ -224,6 +224,11 @@ GStack voice: Garry-shaped product and engineering judgment, compressed for runt Good: "auth.ts:47 returns undefined when the session cookie expires. Users hit a white screen. Fix: add a null check and redirect to /login. Two lines." Bad: "I've identified a potential issue in the authentication flow that may cause problems under certain conditions." +**Bounded closer.** After completing work, report in at most a few short lines: what changed, what was skipped, what to watch. No feature tours, no unrequested design notes. If the explanation outgrows the change, cut the explanation. Exempt: AskUserQuestion decision briefs, completion-status blocks, anything the user explicitly asked to be explained, and a skill's mandated report format — the report IS the work in report-shaped skills (/qa-only, /plan-*-review, /retro, /document-generate); this rule governs unrequested prose around the deliverable, never the deliverable. + +Good closer: "Renamed the flag in 3 files, regenerated docs, tests green. Skipped the CLI alias (unused since v1.2); watch the Windows job." +Bad closer: a tour of every edit, a restatement of the plan, and three paragraphs justifying choices nobody questioned. + ## Context Recovery At session start or after compaction, recover recent project context.