@ikunin/sprintpilot 2.2.30 → 2.2.31

package/_Sprintpilot/bin/autopilot.js

@@ -343,6 +343,26 @@ function readSprintStatuses(projectRoot) {
 //   - sprint-status doesn't exist or fails to parse
 //   - no stories match the epic
 //   - all matching stories are already done
+// Terminal statuses for sprint-status entries — stories in these states
+// are NOT counted as "remaining" for epic-done routing. BMad's official
+// vocabulary only has `done`, but users frequently need to mark stories
+// out-of-scope without lying that they shipped:
+//   - skipped / wont_do / cancelled / deferred — explicit user intent
+//   - abandoned — alternate spelling seen in the wild
+// Pre-2.2.31 only `done` counted, so any deferred 4-N story trapped the
+// orchestrator on next-story routing instead of letting the user close
+// out the epic with a retrospective.
+const TERMINAL_STATUSES = new Set([
+  'done',
+  'skipped',
+  'wont_do',
+  "won't_do",
+  'cancelled',
+  'canceled',
+  'deferred',
+  'abandoned',
+]);
+
 function resolveStoriesForEpic(projectRoot, epicId) {
   if (!epicId) return [];
   const stories = readSprintStatuses(projectRoot);
@@ -354,7 +374,7 @@ function resolveStoriesForEpic(projectRoot, epicId) {
     const derivedEpic = deriveEpicFromStoryKey(key);
     if (derivedEpic !== epicId && derivedEpic !== `epic-${epicId}`) continue;
     const status = String(stories[key].status || '').trim().toLowerCase();
-    if (status === 'done') continue;
+    if (TERMINAL_STATUSES.has(status)) continue;
     out.push(key);
   }
   return out;

package/_Sprintpilot/lib/orchestrator/user-command-applier.js

@@ -169,6 +169,35 @@ function applyOne(state, profile, cmd) {
       });
       break;
 
+    case 'trigger_retrospective':
+      // Force-route to RETROSPECTIVE regardless of remaining_stories_in_epic.
+      // Used when the user explicitly wants to close out the current epic
+      // with deferred stories still showing as non-terminal in sprint-status.
+      // Story-bound fields cleared so the retro skill reads from current_epic.
+      newState = {
+        ...state,
+        phase: STATES.RETROSPECTIVE,
+        story_key: null,
+        story_file_path: null,
+        ac_summary: null,
+        prior_diagnosis: null,
+        patch_findings: null,
+        tests_to_rerun: null,
+        retry_count_this_phase: 0,
+        verify_reject_count: 0,
+        consecutive_test_failures: 0,
+        // current_epic intentionally preserved — retro skill needs it.
+      };
+      effects.push({
+        kind: 'state_transition',
+        from: state.phase,
+        to: STATES.RETROSPECTIVE,
+        reason: 'user_trigger_retrospective',
+        epic: state.current_epic || null,
+        details: cmd.reason || null,
+      });
+      break;
+
     default:
       effects.push({ kind: 'state_transition', reason: 'unknown_user_command', cmd });
   }

package/_Sprintpilot/lib/orchestrator/user-commands.js

@@ -19,6 +19,13 @@
 //     the stored alternative as the next action and clears the pending
 //     entry. Validation rejects this kind when no alternative is pending
 //     in state — see user-command-applier.js for the runtime check.
+//   trigger_retrospective { reason?: string }
+//     Force-routes the orchestrator into RETROSPECTIVE for the current
+//     epic regardless of `remaining_stories_in_epic`. Used when the user
+//     explicitly wants to close out an epic with deferred stories still
+//     in the queue (BMad has no formal `skipped`/`deferred` status for
+//     stories, so the orchestrator otherwise counts them as remaining
+//     and routes to next-story instead of retro).
 //
 // Validation returns { ok: true, command } | { ok: false, errors: string[] }.
 
@@ -34,6 +41,7 @@ const COMMAND_KINDS = [
   'change_profile',
   'pause',
   'accept_alternative',
+  'trigger_retrospective',
 ];
 
 const STORY_KEY_RE = /^[A-Za-z0-9._-]{1,64}$/;
@@ -73,7 +81,8 @@ function validateOne(cmd) {
     case 'abort_sprint':
     case 'force_continue':
     case 'pause':
-    case 'accept_alternative': {
+    case 'accept_alternative':
+    case 'trigger_retrospective': {
       if ('reason' in cmd && cmd.reason !== undefined && typeof cmd.reason !== 'string')
         errors.push(`${cmd.kind}.reason must be string when present`);
       break;

package/_Sprintpilot/manifest.yaml

@@ -1,6 +1,6 @@
 addon:
   name: sprintpilot
-  version: 2.2.30
+  version: 2.2.31
   description: Sprintpilot — autopilot and multi-agent addon for BMad Method (git workflow, parallel agents, autonomous story execution)
   bmad_compatibility: ">=6.2.0"
   modules:

package/_Sprintpilot/skills/sprint-autopilot-on/workflow.orchestrator.md

@@ -83,7 +83,7 @@ Wrap everything in `{ "status": "...", ... }` and pass to
 | `failure`             | `reason`, `diagnosis` (first-class — fed back into next retry), `recoverable: boolean`           |
 | `blocked`             | `blocker_kind` (one of the 5 TRUE BLOCKERS or recoverable kinds), `details`, `user_input_needed`, `consecutive_count?` |
 | `propose_alternative` | `reason`, `alternative` (full Action object), `urgency_hint?` (raises impact only). Low impact → auto-accepted; medium / high → orchestrator stores the alternative in `state.pending_alternative` and emits `user_prompt`. The user accepts via `user_input` `{ kind: 'accept_alternative' }` or rejects via `force_continue` (both clear `pending_alternative`). |
-| `user_input`          | `commands: UserCommand[]` (validated server-side; see user-commands.js). Kinds: `skip_story`, `abort_sprint`, `force_continue`, `override_decision`, `change_profile`, `pause` (cleanly halts THIS session; next `/sprint-autopilot-on` resumes), `accept_alternative` (dispatches the stored `pending_alternative`). **NEVER send `pause` on your own initiative** — see "Pause is human-only" below. |
+| `user_input`          | `commands: UserCommand[]` (validated server-side; see user-commands.js). Kinds: `skip_story`, `abort_sprint`, `force_continue`, `override_decision`, `change_profile`, `pause` (cleanly halts THIS session; next `/sprint-autopilot-on` resumes), `accept_alternative` (dispatches the stored `pending_alternative`), `trigger_retrospective` (v2.2.31+ — force-routes to RETROSPECTIVE for the current epic regardless of `remaining_stories_in_epic`; use when the user explicitly says "close out epic N with retro" while non-terminal stories remain). **NEVER send `pause` on your own initiative** — see "Pause is human-only" below. |
 | `verify_override`     | `evidence: { decision_log_ref?, explanation, expected_paths? }` — used when verify.js is wrong   |
 
 ## Pause is human-only

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ikunin/sprintpilot",
-  "version": "2.2.30",
+  "version": "2.2.31",
   "description": "Sprintpilot — autopilot and multi-agent addon for BMad Method v6: git workflow, parallel agents, autonomous story execution",
   "license": "Apache-2.0",
   "repository": {