From a54dda383701e2a68eb0b21ae6e79c3796d4619c Mon Sep 17 00:00:00 2001 From: Tom Boucher Date: Mon, 4 May 2026 22:47:16 -0400 Subject: [PATCH] fix(progress): remove stale list-phase-assumptions routing --- get-shit-done/workflows/progress.md | 4 ++-- ...3044-research-flag-and-stale-refs.test.cjs | 20 +++++++++++++++++++ 2 files changed, 22 insertions(+), 2 deletions(-) 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..36f7b93f6 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,26 @@ 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'); + assert.ok( + /\/gsd-discuss-phase/.test(content), + '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');