diff --git a/.changeset/fix-3094-progress-stale-assumptions.md b/.changeset/fix-3094-progress-stale-assumptions.md new file mode 100644 index 000000000..317103a92 --- /dev/null +++ b/.changeset/fix-3094-progress-stale-assumptions.md @@ -0,0 +1,5 @@ +--- +type: Fixed +pr: 3111 +--- +**Progress routing command guidance remains canonical** — pre-planning assumption checks in progress routing now consistently assert and document `/gsd-discuss-phase` as the replacement path, with tests enforcing structured slash-command token checks. \ No newline at end of file diff --git a/get-shit-done/workflows/progress.md b/get-shit-done/workflows/progress.md index 8cc82f05b..198e5a352 100644 --- a/get-shit-done/workflows/progress.md +++ b/get-shit-done/workflows/progress.md @@ -275,7 +275,7 @@ PHASE_HAS_UI=$(echo "$PHASE_SECTION" | grep -qi "UI hint.*yes" && echo "true" || **Also available:** - `/gsd-ui-phase {phase}` — generate UI design contract (recommended for frontend phases) - `/gsd-plan-phase {phase}` — skip discussion, plan directly -- `/gsd-list-phase-assumptions {phase}` — see Claude's assumptions +- `/gsd-discuss-phase {phase}` — include assumptions check before planning --- ``` @@ -297,7 +297,7 @@ PHASE_HAS_UI=$(echo "$PHASE_SECTION" | grep -qi "UI hint.*yes" && echo "true" || **Also available:** - `/gsd-plan-phase {phase} ${GSD_WS}` — skip discussion, plan directly -- `/gsd-list-phase-assumptions {phase} ${GSD_WS}` — see Claude's assumptions +- `/gsd-discuss-phase {phase} ${GSD_WS}` — include assumptions check before planning --- ``` diff --git a/tests/bug-3042-3044-research-flag-and-stale-refs.test.cjs b/tests/bug-3042-3044-research-flag-and-stale-refs.test.cjs index b4c0ac485..1455bbe4f 100644 --- a/tests/bug-3042-3044-research-flag-and-stale-refs.test.cjs +++ b/tests/bug-3042-3044-research-flag-and-stale-refs.test.cjs @@ -302,6 +302,28 @@ describe('bug #3044: localized doc sets also scrubbed', () => { // ─── Replacement commands are documented ──────────────────────────────────── +describe('bug #3094: progress routing does not reference removed /gsd-list-phase-assumptions', () => { + test('get-shit-done/workflows/progress.md has no /gsd-list-phase-assumptions token', () => { + const content = read('get-shit-done/workflows/progress.md'); + const tokens = extractSlashCommandTokens(content); + assert.equal( + tokens.has('/gsd-list-phase-assumptions'), + false, + 'progress.md must not recommend removed /gsd-list-phase-assumptions' + ); + }); + + test('progress.md pre-planning guidance uses /gsd-discuss-phase instead', () => { + const content = read('get-shit-done/workflows/progress.md'); + const tokens = extractSlashCommandTokens(content); + assert.equal( + tokens.has('/gsd-discuss-phase'), + true, + 'progress.md should route pre-planning assumption checks via /gsd-discuss-phase' + ); + }); +}); + describe('replacement commands appear where the deleted ones used to live', () => { test('docs/issue-driven-orchestration.md uses /gsd-workspace --new (not /gsd-new-workspace)', () => { const content = read('docs/issue-driven-orchestration.md');