@ikunin/sprintpilot 1.0.3 → 1.0.5

package/_Sprintpilot/skills/sprintpilot-update/workflow.md

@@ -34,8 +34,9 @@
 
 6. **Install the update.** On confirmation, run:
    ```bash
-   npx @ikunin/sprintpilot@latest@{latest} install --yes
+   npx @ikunin/sprintpilot@{latest} install --yes
    ```
+   (`{latest}` is the version resolved in step 2 — e.g., `npx @ikunin/sprintpilot@1.0.4 install --yes`.)
    Stream the output so the user can see progress.
 
 7. **Verify.** Read `_Sprintpilot/manifest.yaml` again and confirm the version updated:

package/_Sprintpilot/templates/epic-retrospective.md

@@ -0,0 +1,24 @@
+# Epic {{epic_id}} — {{epic_title}} — Retrospective
+
+**Completed:** {current_date}
+**Stories done:** {{n_done}}/{{n_total}}
+
+## Stories
+{{#each stories}}
+- **{{story-key}}** — {{title}}
+  - Tests: {{test_pass_count}}
+  - Patches applied: {{patch_count}}
+{{/each}}
+
+## Key decisions
+{{#each decisions}}
+- [{{impact}}] {{category}}: {{decision}} — {{rationale}}
+{{/each}}
+
+## Risks carried forward
+{{#each open_risks}}
+- {{risk}}
+{{/each}}
+
+## Notes
+Generated inline by Sprintpilot autopilot per `autopilot.retrospective_mode: auto`.

package/_Sprintpilot/templates/sprint-report.txt

@@ -0,0 +1,60 @@
+╔═══════════════════════════════════════════════════════════════╗
+║                   BMAD AUTOPILOT — REPORT                     ║
+╚═══════════════════════════════════════════════════════════════╝
+
+SUMMARY
+  Stories completed : {{done_count}}/{{total_stories}}
+  Epics completed   : {{done_epics}}/{{total_epics}}
+  Total tests       : {{N}}/{{N}} passed
+{{#if git_enabled}}
+  Platform          : {{platform}}
+{{/if}}
+
+STORIES
+{{#each epic}}
+  Epic {{epic_number}}: {{epic_title}}
+  {{#each stories}}
+  ✓ {{story-key}}  — {{test_count}} tests{{#if pr_url}}  PR: {{pr_url}}{{/if}}
+  {{/each}}
+{{/each}}
+{{#if remaining_stories}}
+  Not started:
+  {{#each remaining_stories}}
+  · {{story-key}}
+  {{/each}}
+{{/if}}
+
+DECISIONS REQUIRING REVIEW (high/medium impact)
+{{#each medium_high_decisions}}
+  #{{id}}  [{{impact}}] {{story}} / {{phase}}
+      {{decision}}
+      → {{rationale}}
+{{/each}}
+{{#if no_medium_high_decisions}}
+  None — all decisions were low-impact.
+{{/if}}
+
+  Full log: {decision_log_file}
+
+REVIEW FINDINGS APPLIED
+  Patches applied    : {{total_patches}}
+  Findings dismissed : {{total_dismissed}}
+  Review rounds      : {{total_review_rounds}}
+
+CODE REVIEW SUMMARY (per story)
+{{#each completed_stories}}
+  {{story-key}} : {{patches_applied}} patches applied, {{findings_dismissed}} dismissed
+{{/each}}
+
+WHAT TO DO NEXT
+  1. Review decisions marked medium/high above
+{{#if has_pr_urls}}
+  2. Merge open PRs: {{pr_urls_list}}
+{{/if}}
+{{#if launch_cmd}}
+  {{next_number}}. Run the app: {{launch_cmd}}
+{{/if}}
+  {{next_number}}. Manual smoke test checklist:
+{{#each completed_stories}}
+     · [{{story-key}}] {{smoke_test_suggestion}}
+{{/each}}

package/lib/commands/install.js

@@ -64,6 +64,9 @@ const { V1_ADDON_DIR_NAME, V1_SKILL_NAMES, detectV1Installation } = require('../
 
 const ADDON_DIR = path.resolve(__dirname, '..', '..', '_Sprintpilot');
 const PROJECT_ADDON_DIR_NAME = '_Sprintpilot';
+const DEFAULT_SESSION_STORY_LIMIT = 3;
+const DEFAULT_RETROSPECTIVE_MODE = 'auto';
+const RETROSPECTIVE_MODES = ['auto', 'stop', 'skip'];
 const RUNTIME_RESOURCES = [
   'Sprintpilot.md',
   'manifest.yaml',
@@ -603,6 +606,211 @@ async function evictV1Installation(projectRoot, { dryRun, migrateV1, yes }) {
   return { migrated: true, moduleConfigSnapshot: snapshot };
 }
 
+// Parse the user's existing autopilot config (if any) so interactive prompts
+// can default to the current values AND so a v1 migration's patcher run
+// preserves user-edited values rather than overwriting them with bundled
+// defaults.
+//
+// Checks both locations, in order of precedence:
+//   1. `_Sprintpilot/modules/autopilot/config.yaml`  — normal upgrade
+//   2. `_bmad-addons/modules/autopilot/config.yaml`  — v1 legacy (bmad-
+//      autopilot-addon), picked up BEFORE evictV1Installation moves it
+//
+// Regex-based so we don't add a YAML parser dep for two scalar fields.
+// Unrecognized / unreadable files fall back to bundled defaults.
+async function readExistingAutopilotConfig(projectRoot, v1Snapshot) {
+  const out = { sessionStoryLimit: null, retrospectiveMode: null };
+  let raw = null;
+
+  // Precedence order:
+  //   1. `_Sprintpilot/modules/autopilot/config.yaml`  — normal upgrade
+  //   2. `_bmad-addons/modules/autopilot/config.yaml`  — v1 legacy still
+  //      on disk (install was invoked before evictV1Installation ran)
+  //   3. v1 in-memory snapshot                         — v1 legacy already
+  //      evicted; evictV1Installation captured the buffer before removing
+  //      the directory, so we extract the autopilot/config.yaml bytes
+  //      from there. Without this the patcher would overwrite the user's
+  //      v1-preserved values with bundled defaults.
+  const candidates = [
+    path.join(projectRoot, PROJECT_ADDON_DIR_NAME, 'modules', 'autopilot', 'config.yaml'),
+    path.join(projectRoot, V1_ADDON_DIR_NAME, 'modules', 'autopilot', 'config.yaml'),
+  ];
+  for (const file of candidates) {
+    if (!(await fs.pathExists(file))) continue;
+    try {
+      raw = await fs.readFile(file, 'utf8');
+      break;
+    } catch {
+      /* try next candidate */
+    }
+  }
+
+  if (raw == null && v1Snapshot && Array.isArray(v1Snapshot.autopilot)) {
+    const entry = v1Snapshot.autopilot.find((f) => f.relPath === 'config.yaml');
+    if (entry && Buffer.isBuffer(entry.buffer)) {
+      try {
+        raw = entry.buffer.toString('utf8');
+      } catch {
+        /* unreadable — fall back to defaults */
+      }
+    }
+  }
+
+  if (raw == null) return out;
+
+  // Both patterns tolerate a trailing `# comment` tail so users can annotate
+  // their config without breaking upgrade detection (e.g.
+  // `retrospective_mode: stop  # we want manual retros`). `[ \t]` rather
+  // than `\s` inside each line so matches never cross a newline.
+  const commentTail = /[ \t]*(?:#.*)?$/.source;
+
+  // `session_story_limit: 3`  — unquoted integer, optional trailing comment
+  const limitMatch = raw.match(
+    new RegExp(`^[ \\t]*session_story_limit:[ \\t]*(\\d+)${commentTail}`, 'm'),
+  );
+  if (limitMatch) {
+    const n = Number.parseInt(limitMatch[1], 10);
+    if (Number.isFinite(n) && n >= 0) out.sessionStoryLimit = n;
+  }
+  // `retrospective_mode: auto` — unquoted or single/double-quoted string,
+  // optional trailing comment
+  const modeMatch = raw.match(
+    new RegExp(`^[ \\t]*retrospective_mode:[ \\t]*["']?([a-zA-Z_-]+)["']?${commentTail}`, 'm'),
+  );
+  if (modeMatch && RETROSPECTIVE_MODES.includes(modeMatch[1])) {
+    out.retrospectiveMode = modeMatch[1];
+  }
+  return out;
+}
+
+// Rewrite the `session_story_limit:` and `retrospective_mode:` scalar lines
+// in the freshly-copied autopilot/config.yaml with the user's chosen values.
+// Done as a line-level string replacement instead of a token-substitution
+// because workflow.md uses the `{{session_story_limit}}` / `{{retrospective_mode}}`
+// variable-reference syntax — threading these keys through `renderString`
+// would match the inner `{key}` inside `{{key}}` and corrupt the workflow
+// variables to literal `{value}` strings.
+//
+// Handles three shapes for each key:
+//   1. line present with a value    → in-place replace (preserves trailing comment)
+//   2. line present with no value   → in-place fill
+//   3. line missing                 → append to the `autopilot:` block
+// (3) is the path that catches v1 (bmad-autopilot-addon) migrations whose
+// legacy config predates `retrospective_mode` — without it, the user's
+// prompted choice would be silently dropped.
+function applyScalar(source, key, value) {
+  // Structure of a YAML scalar line we support:
+  //   [indent][key]:[space(s)][value][space(s) + # comment]?
+  //
+  // Groups, non-greedy on value so the trailing-comment capture wins:
+  //   indent  — leading whitespace (preserved verbatim)
+  //   value   — `\S[^#\n]*?` — starts non-ws, stops as early as possible so
+  //             the comment tail can match. Optional, so this also matches
+  //             "key:\n" (no value at all) — P1-C.
+  //   tail    — `\s*#.*` — the whitespace-before-# is INSIDE the tail capture
+  //             so the original spacing ("value  # note") round-trips
+  //             instead of collapsing to a single space.
+  // Using [ \t] instead of \s everywhere INSIDE the line so the match
+  // cannot cross a newline — `\s*` would greedily consume the trailing
+  // `\n` after "key:" on an empty-value line, breaking the rewrite.
+  const replaceRe = new RegExp(`^([ \\t]*)${key}:[ \\t]*(\\S[^#\\n]*?)?([ \\t]*#.*)?$`, 'm');
+  if (replaceRe.test(source)) {
+    return source.replace(replaceRe, (_m, indent, _oldValue, tail) => {
+      return `${indent}${key}: ${value}${tail || ''}`;
+    });
+  }
+
+  // Key is absent. Append under the `autopilot:` block at the file's end.
+  // The bundled shipping config always starts with `autopilot:` at column
+  // 0 and indents children two spaces — match that style. If the file has
+  // no `autopilot:` header at all (hand-edited to a different shape), bail
+  // rather than guess.
+  if (!/^autopilot:\s*$/m.test(source)) return source;
+  const trimmed = source.endsWith('\n') ? source : `${source}\n`;
+  return `${trimmed}  ${key}: ${value}\n`;
+}
+
+async function patchAutopilotConfig(projectRoot, { sessionStoryLimit, retrospectiveMode }) {
+  const file = path.join(
+    projectRoot,
+    PROJECT_ADDON_DIR_NAME,
+    'modules',
+    'autopilot',
+    'config.yaml',
+  );
+  if (!(await fs.pathExists(file))) return;
+  const original = await fs.readFile(file, 'utf8');
+  let updated = applyScalar(original, 'session_story_limit', sessionStoryLimit);
+  updated = applyScalar(updated, 'retrospective_mode', retrospectiveMode);
+  if (updated !== original) {
+    await writeAtomic(file, updated);
+  }
+}
+
+async function resolveAutopilotSettings({ projectRoot, yes, dryRun, v1Snapshot }) {
+  const existing = await readExistingAutopilotConfig(projectRoot, v1Snapshot);
+  const defaultLimit = existing.sessionStoryLimit ?? DEFAULT_SESSION_STORY_LIMIT;
+  const defaultMode = existing.retrospectiveMode ?? DEFAULT_RETROSPECTIVE_MODE;
+
+  if (yes) {
+    if (existing.sessionStoryLimit != null || existing.retrospectiveMode != null) {
+      console.log(
+        pc.dim(
+          `Preserving autopilot config: session_story_limit=${defaultLimit}, retrospective_mode=${defaultMode}`,
+        ),
+      );
+    }
+    return { sessionStoryLimit: defaultLimit, retrospectiveMode: defaultMode };
+  }
+
+  if (dryRun) {
+    console.log(
+      pc.dim(
+        `[DRY RUN] Would prompt for autopilot config (current: session_story_limit=${defaultLimit}, retrospective_mode=${defaultMode})`,
+      ),
+    );
+    return { sessionStoryLimit: defaultLimit, retrospectiveMode: defaultMode };
+  }
+
+  const limitRaw = await prompts.text({
+    message: 'Autopilot: stories to fully implement per session (0 = unlimited)',
+    initialValue: String(defaultLimit),
+    validate(value) {
+      if (value == null || value === '') return undefined;
+      const n = Number.parseInt(value, 10);
+      if (!Number.isFinite(n) || String(n) !== String(value).trim() || n < 0) {
+        return 'Enter a non-negative integer (0 = unlimited)';
+      }
+      return undefined;
+    },
+  });
+  const sessionStoryLimit =
+    limitRaw == null || String(limitRaw).trim() === ''
+      ? defaultLimit
+      : Number.parseInt(String(limitRaw).trim(), 10);
+
+  const retrospectiveMode = await prompts.select({
+    message: 'Autopilot: retrospective handling at epic completion',
+    options: [
+      {
+        value: 'auto',
+        label: 'Auto — autopilot generates retrospective inline and continues (recommended)',
+      },
+      {
+        value: 'stop',
+        label: 'Stop — pause autopilot so you can run /bmad-retrospective manually',
+      },
+      {
+        value: 'skip',
+        label: 'Skip — do not generate a retrospective (NOT RECOMMENDED)',
+      },
+    ],
+    initialValue: defaultMode,
+  });
+
+  return { sessionStoryLimit, retrospectiveMode };
+}
+
 async function runInteractiveToolPicker(detected) {
   const options = ALL_TOOLS.map((tool) => ({
     value: tool,
@@ -665,12 +873,24 @@ async function runInstall(options = {}) {
 
   // 2. Resolve output_folder
   const outputFolder = await readOutputFolder(projectRoot);
-  const ctx = buildContext({ outputFolder });
   if (outputFolder !== '_bmad-output') {
     console.log(pc.dim(`Using output_folder: ${outputFolder}`));
     console.log('');
   }
 
+  // 2a. Autopilot configuration (prompt or preserve existing values).
+  // These values are patched into modules/autopilot/config.yaml AFTER the
+  // runtime copy — they're NOT threaded through `renderString`, because
+  // workflow.md's `{{session_story_limit}}` / `{{retrospective_mode}}`
+  // variable references would collide with single-brace token matching.
+  const { sessionStoryLimit, retrospectiveMode } = await resolveAutopilotSettings({
+    projectRoot,
+    yes,
+    dryRun,
+    v1Snapshot: v1ConfigSnapshot,
+  });
+  const ctx = buildContext({ outputFolder });
+
   // 3. Detect + select tools
   const detected = await detectInstalledTools(projectRoot);
 
@@ -888,6 +1108,12 @@ async function runInstall(options = {}) {
     // .sprintpilot-v1-snapshot*.json were added up-front in
     // evictV1Installation (step 0) so they're in place even if the
     // module-snapshot branch never runs.
+
+    // 6b. Apply the resolved autopilot settings. Runs AFTER step 6 (which
+    //     wrote the bundled default config) AND after the v1 snapshot
+    //     reapply (which might have restored an older config.yaml without
+    //     `retrospective_mode`). The user's prompted values always win.
+    await patchAutopilotConfig(projectRoot, { sessionStoryLimit, retrospectiveMode });
   }
 
   // 7. Verify git check-ignore
@@ -936,8 +1162,15 @@ async function runInstall(options = {}) {
   console.log('    multi_agent.max_parallel_analysis  5      Codebase analysis agents');
   console.log('');
   console.log('  _Sprintpilot/modules/autopilot/config.yaml');
+  // padEnd so the description column stays aligned across both rows,
+  // independent of the chosen values' widths (e.g. 2-digit limit, 4-char mode).
+  const apKey = (k) => k.padEnd(31, ' ');
+  const apVal = (v) => String(v).padEnd(6, ' ');
+  console.log(
+    `    ${apKey('autopilot.session_story_limit')}${apVal(sessionStoryLimit)} Stories to fully implement per run (0 = unlimited)`,
+  );
   console.log(
-    '    autopilot.session_story_limit    3      Stories to fully implement per run (0 = unlimited)',
+    `    ${apKey('autopilot.retrospective_mode')}${apVal(retrospectiveMode)} Epic-end retrospective: auto (inline) | stop (pause) | skip (not recommended)`,
   );
   console.log('');
   console.log('Multi-agent skills — run parallel subagents for faster analysis:');
@@ -963,4 +1196,15 @@ async function runInstall(options = {}) {
   console.log('Apache 2.0 License — Igor Kunin — https://github.com/ikunin/sprintpilot');
 }
 
-module.exports = { runInstall };
+module.exports = {
+  runInstall,
+  // Exported for unit tests only — do not depend on this surface elsewhere.
+  _internals: {
+    readExistingAutopilotConfig,
+    patchAutopilotConfig,
+    applyScalar,
+    RETROSPECTIVE_MODES,
+    DEFAULT_SESSION_STORY_LIMIT,
+    DEFAULT_RETROSPECTIVE_MODE,
+  },
+};

package/lib/prompts.js

@@ -27,6 +27,26 @@ async function confirm(opts) {
   return result;
 }
 
+async function text(opts) {
+  const clack = await loadClack();
+  const result = await clack.text(opts);
+  if (clack.isCancel(result)) {
+    clack.cancel('Cancelled');
+    process.exit(0);
+  }
+  return result;
+}
+
+async function select(opts) {
+  const clack = await loadClack();
+  const result = await clack.select(opts);
+  if (clack.isCancel(result)) {
+    clack.cancel('Cancelled');
+    process.exit(0);
+  }
+  return result;
+}
+
 async function intro(message) {
   const clack = await loadClack();
   clack.intro(message);
@@ -76,5 +96,7 @@ module.exports = {
   note,
   multiselect,
   confirm,
+  text,
+  select,
   log,
 };

package/lib/substitute.js

@@ -19,7 +19,7 @@ function renderString(text, ctx) {
   return out;
 }
 
-function buildContext({ outputFolder }) {
+function buildContext({ outputFolder } = {}) {
   const out = outputFolder || '_bmad-output';
   return {
     output_folder: out,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ikunin/sprintpilot",
-  "version": "1.0.3",
+  "version": "1.0.5",
   "description": "Sprintpilot — autopilot and multi-agent addon for BMad Method v6: git workflow, parallel agents, autonomous story execution",
   "license": "Apache-2.0",
   "repository": {
@@ -33,6 +33,9 @@
   "engines": {
     "node": ">=18.0.0"
   },
+  "scripts": {
+    "prepare": "node scripts/setup-git-hooks.mjs"
+  },
   "dependencies": {
     "@clack/core": "^1.0.0",
     "@clack/prompts": "^1.0.0",