Merge pull request #3113 from gsd-build/fix/3083-resume-project-md-route-to-workflow-emit

fix: remove /clear then from resume route templates

.changeset/pr-3113-release-note.md

@@ -0,0 +1,5 @@
+---
+type: Fixed
+pr: 3113
+---
+Fixes for issue #3113 were applied to keep command/workflow behavior and SDK parity aligned with current documented usage.

get-shit-done/workflows/resume-project.md

@@ -225,9 +225,11 @@ Wait for user selection.
 </step>
 
 <step name="route_to_workflow">
-Based on user selection, route to appropriate workflow:
+Based on user selection, route to appropriate workflow.
 
-- **Execute plan** → Show command for user to run after clearing:
+Resume-specific exception: do **not** emit `/clear then:` here. Resume is already a session-entry flow, so the next command should be shown directly.
+
+- **Execute plan** → Show direct next command:
   ```
   ---
 
@@ -235,13 +237,11 @@ Based on user selection, route to appropriate workflow:
 
   **{phase}-{plan}: [Plan Name]** — [objective from PLAN.md]
 
-  `/clear` then:
-
   `/gsd-execute-phase {phase} ${GSD_WS}`
 
   ---
   ```
-- **Plan phase** → Show command for user to run after clearing:
+- **Plan phase** → Show direct next command:
   ```
   ---
 
@@ -249,8 +249,6 @@ Based on user selection, route to appropriate workflow:
 
   **Phase [N]: [Name]** — [Goal from ROADMAP.md]
 
-  `/clear` then:
-
   `/gsd-plan-phase [phase-number] ${GSD_WS}`
 
   ---

tests/bug-3083-resume-route-clear.test.cjs

@@ -0,0 +1,28 @@
+'use strict';
+
+const { test, describe } = require('node:test');
+const assert = require('node:assert/strict');
+const fs = require('node:fs');
+const path = require('node:path');
+
+describe('bug #3083: resume-project next-step routing should not include /clear then:', () => {
+  test('route_to_workflow block omits /clear then: in resume templates', () => {
+    const workflowPath = path.join(__dirname, '..', 'get-shit-done', 'workflows', 'resume-project.md');
+    const content = fs.readFileSync(workflowPath, 'utf-8');
+    const routeStart = content.indexOf('<step name="route_to_workflow">');
+    const routeEnd = content.indexOf('</step>', routeStart);
+    const routeBlock = content.slice(routeStart, routeEnd);
+
+    assert.equal(routeBlock.includes('/clear` then:'), false, 'resume route templates must not include `/clear` then:');
+  });
+
+  test('route_to_workflow block includes exception note explaining resume behavior', () => {
+    const workflowPath = path.join(__dirname, '..', 'get-shit-done', 'workflows', 'resume-project.md');
+    const content = fs.readFileSync(workflowPath, 'utf-8');
+    const routeStart = content.indexOf('<step name="route_to_workflow">');
+    const routeEnd = content.indexOf('</step>', routeStart);
+    const routeBlock = content.slice(routeStart, routeEnd);
+
+    assert.match(routeBlock, /resume.*exception|exception.*resume/i);
+  });
+});