@ikunin/sprintpilot 2.2.31 → 2.3.0

package/lib/commands/install.js

@@ -748,7 +748,15 @@ async function evictV1Installation(projectRoot, { dryRun, migrateV1, yes }) {
 // 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 };
+  const out = {
+    sessionStoryLimit: null,
+    retrospectiveMode: null,
+    // v2.3.0 additions. null means "not set in user config" → use the bundled
+    // default. autoInferDependencies is read only to surface a deprecation
+    // notice on upgrade — we never write it back.
+    autoPlanOnStart: null,
+    autoInferDependencies: null,
+  };
   let raw = null;
 
   // Precedence order:
@@ -809,6 +817,22 @@ async function readExistingAutopilotConfig(projectRoot, v1Snapshot) {
   if (modeMatch && RETROSPECTIVE_MODES.includes(modeMatch[1])) {
     out.retrospectiveMode = modeMatch[1];
   }
+  // v2.3.0 — `auto_plan_on_start: true|false`. Bool; bundled default is false.
+  const planMatch = raw.match(
+    new RegExp(`^[ \\t]*auto_plan_on_start:[ \\t]*(true|false)${commentTail}`, 'm'),
+  );
+  if (planMatch) {
+    out.autoPlanOnStart = planMatch[1] === 'true';
+  }
+  // Legacy `auto_infer_dependencies: true|false` — read so the installer can
+  // surface a deprecation notice when the user is upgrading from v2.2.x with
+  // the flag set to true (it's now a no-op). Never written back.
+  const inferMatch = raw.match(
+    new RegExp(`^[ \\t]*auto_infer_dependencies:[ \\t]*(true|false)${commentTail}`, 'm'),
+  );
+  if (inferMatch) {
+    out.autoInferDependencies = inferMatch[1] === 'true';
+  }
   return out;
 }
 
@@ -859,7 +883,10 @@ function applyScalar(source, key, value) {
   return `${trimmed}  ${key}: ${value}\n`;
 }
 
-async function patchAutopilotConfig(projectRoot, { sessionStoryLimit, retrospectiveMode }) {
+async function patchAutopilotConfig(
+  projectRoot,
+  { sessionStoryLimit, retrospectiveMode, autoPlanOnStart },
+) {
   const file = path.join(
     projectRoot,
     PROJECT_ADDON_DIR_NAME,
@@ -871,6 +898,12 @@ async function patchAutopilotConfig(projectRoot, { sessionStoryLimit, retrospect
   const original = await fs.readFile(file, 'utf8');
   let updated = applyScalar(original, 'session_story_limit', sessionStoryLimit);
   updated = applyScalar(updated, 'retrospective_mode', retrospectiveMode);
+  // v2.3.0 — auto_plan_on_start is a boolean. applyScalar handles literal
+  // values (true/false) the same way as numbers; we just need to pass the
+  // unquoted lowercase string for booleans.
+  if (autoPlanOnStart !== undefined && autoPlanOnStart !== null) {
+    updated = applyScalar(updated, 'auto_plan_on_start', autoPlanOnStart ? 'true' : 'false');
+  }
   if (updated !== original) {
     await writeAtomic(file, updated);
   }
@@ -915,6 +948,46 @@ async function readExistingComplexityProfile(projectRoot, v1Snapshot) {
   return m[1];
 }
 
+// v2.3.0 — post-install hygiene check. Cross-reference the project's
+// _Sprintpilot/manifest.yaml `installed_skills` against what actually
+// landed under `_Sprintpilot/skills/<name>/SKILL.md`. Catches the
+// classic "added skill to manifest but forgot to ship the files" bug
+// at install time rather than at first invocation.
+//
+// Returns { missing: string[] } — empty array means everything is
+// wired correctly. The caller chooses how to surface mismatches
+// (warning vs fail). We never fail the install on a mismatch; it's
+// hygiene, not correctness — the skill just won't appear under the
+// host tool's /-command if it's missing on disk.
+async function verifySkillManifest(projectRoot) {
+  const manifestPath = path.join(projectRoot, PROJECT_ADDON_DIR_NAME, 'manifest.yaml');
+  if (!(await fs.pathExists(manifestPath))) {
+    return { missing: [] };
+  }
+  let raw;
+  try {
+    raw = await fs.readFile(manifestPath, 'utf8');
+  } catch {
+    return { missing: [] };
+  }
+  // Parse the YAML list under `installed_skills:` via regex — bullet
+  // lines starting with `-` at consistent indent. Cheap; no YAML dep.
+  const skillNames = [];
+  const installedMatch = raw.match(/^[ \t]*installed_skills:\s*\n((?:[ \t]+- [^\n]+\n?)+)/m);
+  if (installedMatch) {
+    for (const line of installedMatch[1].split(/\n/)) {
+      const m = line.match(/^[ \t]+-\s+([A-Za-z0-9._-]+)/);
+      if (m) skillNames.push(m[1]);
+    }
+  }
+  const missing = [];
+  for (const name of skillNames) {
+    const skillFile = path.join(projectRoot, PROJECT_ADDON_DIR_NAME, 'skills', name, 'SKILL.md');
+    if (!(await fs.pathExists(skillFile))) missing.push(name);
+  }
+  return { missing };
+}
+
 async function patchComplexityProfile(projectRoot, profile) {
   const file = path.join(
     projectRoot,
@@ -1008,25 +1081,39 @@ 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;
+  // v2.3.0 — opt-in default false; preserve existing user choice on upgrade.
+  const defaultAutoPlan = existing.autoPlanOnStart ?? false;
 
   if (yes) {
-    if (existing.sessionStoryLimit != null || existing.retrospectiveMode != null) {
+    if (
+      existing.sessionStoryLimit != null ||
+      existing.retrospectiveMode != null ||
+      existing.autoPlanOnStart != null
+    ) {
       console.log(
         pc.dim(
-          `Preserving autopilot config: session_story_limit=${defaultLimit}, retrospective_mode=${defaultMode}`,
+          `Preserving autopilot config: session_story_limit=${defaultLimit}, retrospective_mode=${defaultMode}, auto_plan_on_start=${defaultAutoPlan}`,
         ),
       );
     }
-    return { sessionStoryLimit: defaultLimit, retrospectiveMode: defaultMode };
+    return {
+      sessionStoryLimit: defaultLimit,
+      retrospectiveMode: defaultMode,
+      autoPlanOnStart: defaultAutoPlan,
+    };
   }
 
   if (dryRun) {
     console.log(
       pc.dim(
-        `[DRY RUN] Would prompt for autopilot config (current: session_story_limit=${defaultLimit}, retrospective_mode=${defaultMode})`,
+        `[DRY RUN] Would prompt for autopilot config (current: session_story_limit=${defaultLimit}, retrospective_mode=${defaultMode}, auto_plan_on_start=${defaultAutoPlan})`,
       ),
     );
-    return { sessionStoryLimit: defaultLimit, retrospectiveMode: defaultMode };
+    return {
+      sessionStoryLimit: defaultLimit,
+      retrospectiveMode: defaultMode,
+      autoPlanOnStart: defaultAutoPlan,
+    };
   }
 
   const limitRaw = await prompts.text({
@@ -1065,7 +1152,17 @@ async function resolveAutopilotSettings({ projectRoot, yes, dryRun, v1Snapshot }
     initialValue: defaultMode,
   });
 
-  return { sessionStoryLimit, retrospectiveMode };
+  // v2.3.0 — single yes/no prompt for the new plan workflow. Default false:
+  // `autopilot start` runs in sprint-status order until the user explicitly
+  // invokes /sprintpilot-plan-sprint, which is always available regardless.
+  // Set this true to auto-trigger the planning skill on greenfield projects.
+  const autoPlanOnStart = await prompts.confirm({
+    message:
+      'Auto-build a sprint plan on first `autopilot start`? (v2.3.0; runs /sprintpilot-plan-sprint to infer dependencies. You can always invoke the skill manually regardless of this setting.)',
+    initialValue: defaultAutoPlan,
+  });
+
+  return { sessionStoryLimit, retrospectiveMode, autoPlanOnStart };
 }
 
 async function runInteractiveToolPicker(detected) {
@@ -1194,7 +1291,7 @@ async function runInstall(options = {}) {
   // 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({
+  const { sessionStoryLimit, retrospectiveMode, autoPlanOnStart } = await resolveAutopilotSettings({
     projectRoot,
     yes,
     dryRun,
@@ -1439,7 +1536,11 @@ async function runInstall(options = {}) {
     //     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 });
+    await patchAutopilotConfig(projectRoot, {
+      sessionStoryLimit,
+      retrospectiveMode,
+      autoPlanOnStart,
+    });
 
     // 6c. Persist the complexity_profile. Separate from patchAutopilotConfig
     //     so the existing upgrade test coverage (readExistingAutopilotConfig /
@@ -1501,6 +1602,56 @@ async function runInstall(options = {}) {
   console.log(
     `Total skills installed: ${totalInstalled} (${skillCount} skills x ${selectedTools.length} tools)`,
   );
+
+  // v2.3.0 — post-install hygiene: warn if any skill in manifest.yaml
+  // doesn't have a SKILL.md on disk. Non-blocking; surfaces packaging
+  // bugs without failing the install.
+  try {
+    const verify = await verifySkillManifest(projectRoot);
+    if (verify.missing.length > 0) {
+      console.log('');
+      console.log(
+        pc.yellow(`  WARN: manifest references skills missing from disk: ${verify.missing.join(', ')}`),
+      );
+      console.log(pc.yellow('  These won\'t appear under your host tool\'s / menu until the SKILL.md files are present.'));
+    }
+  } catch {
+    // Self-check failure is non-fatal — never block install on hygiene.
+  }
+
+  // v2.3.0 upgrade notes — surfaced only when the relevant signals are
+  // actually present. Greenfield installs see nothing; upgraders from
+  // v2.2.x see migration + deprecation notices.
+  const v23Notes = [];
+  const legacyDepsPath = path.join(projectRoot, '_Sprintpilot', 'sprints', 'dependencies.yaml');
+  if (await fs.pathExists(legacyDepsPath)) {
+    v23Notes.push(
+      'Legacy file detected: _Sprintpilot/sprints/dependencies.yaml',
+      '  Auto-migrated to sprint-plan.yaml on the first `autopilot start`.',
+      '  Run now: node _Sprintpilot/scripts/infer-dependencies.js migrate',
+    );
+  }
+  // Re-read so we can show the deprecation notice without threading state
+  // through every helper. Cheap (one regex scan); only happens once per install.
+  try {
+    const existingForNotes = await readExistingAutopilotConfig(projectRoot, v1ConfigSnapshot);
+    if (existingForNotes.autoInferDependencies === true) {
+      if (v23Notes.length > 0) v23Notes.push('');
+      v23Notes.push(
+        'Deprecated: autopilot.auto_infer_dependencies = true in your config.',
+        '  This flag is a no-op in v2.3.0 — superseded by auto_plan_on_start (default false).',
+        '  Safe to remove from config.yaml; the new /sprintpilot-plan-sprint workflow',
+        '  handles inference manually or on opt-in auto-trigger.',
+      );
+    }
+  } catch {
+    // Config re-read failure is non-fatal — skip the deprecation notice.
+  }
+  if (v23Notes.length > 0) {
+    console.log('');
+    console.log(pc.cyan('v2.3.0 upgrade notes:'));
+    for (const line of v23Notes) console.log('  ' + line);
+  }
   console.log('');
   console.log('Skills:');
   for (const skill of allSkills) console.log(`  - ${skill}`);
@@ -1513,6 +1664,13 @@ async function runInstall(options = {}) {
   console.log('  /sprint-autopilot-off  Disengage and show status');
   console.log('  /bmad-help             Orientation and next-step guidance (from BMad Method)');
   console.log('');
+  console.log('First steps for a new sprint:');
+  console.log('  1. BMad sprint planning:        /bmad-sprint-planning');
+  console.log('  2. (optional) Sprint plan:      /sprintpilot-plan-sprint');
+  console.log('  3. (optional) Inspect DAG:      /sprintpilot-dependency-graph mermaid');
+  console.log('  4. Start autopilot:             /sprint-autopilot-on');
+  console.log('  5. Check live progress:         /sprintpilot-sprint-progress');
+  console.log('');
   console.log('Configuration (edit these files to customize behavior):');
   console.log('');
   console.log('  _Sprintpilot/modules/git/config.yaml');
@@ -1542,6 +1700,18 @@ async function runInstall(options = {}) {
   console.log(
     `    ${apKey('autopilot.retrospective_mode')}${apVal(retrospectiveMode)} Epic-end retrospective: auto (inline) | stop (pause) | skip (not recommended)`,
   );
+  console.log(
+    `    ${apKey('autopilot.auto_plan_on_start')}${apVal(String(autoPlanOnStart))} Auto-build sprint plan on first start (v2.3.0; default off)`,
+  );
+  console.log('');
+  console.log('Sprint planning + progress (v2.3.0):');
+  console.log('  /sprintpilot-plan-sprint       Build dependency-aware sprint plan');
+  console.log('  /sprintpilot-sprint-progress   Concise health-check of autopilot execution');
+  console.log('  /sprintpilot-dependency-graph  Render DAG (mermaid / graphviz / text / layers / json)');
+  console.log('');
+  console.log('CLI utilities:');
+  console.log('  autopilot progress             Live status (--json / --story <key>)');
+  console.log('  autopilot start --no-auto-plan Skip auto-planning for one session');
   console.log('');
   console.log('Multi-agent skills — run parallel subagents for faster analysis:');
   console.log('  /sprintpilot-code-review       Parallel 3-layer adversarial review');
@@ -1551,6 +1721,11 @@ async function runInstall(options = {}) {
   console.log('  /sprintpilot-migrate           Legacy migration planning');
   console.log('  /sprintpilot-research          Parallel web research');
   console.log('  /sprintpilot-party-mode        Multi-persona agent discussions');
+  console.log('');
+  console.log('Documentation:');
+  console.log('  Sprint planning walkthrough:   docs/USAGE.md');
+  console.log('  Configuration reference:       docs/CONFIGURATION.md');
+  console.log('  Architecture deep-dive:        docs/ARCHITECTURE.md');
 
   const latestVersion = await latestVersionPromise;
   if (latestVersion && addonVersion && compareVersions(addonVersion, latestVersion) === 'behind') {
@@ -1586,5 +1761,6 @@ module.exports = {
     KEY_RENAMES,
     snapshotUserOwnedFiles,
     applyUserOwnedFiles,
+    verifySkillManifest,
   },
 };

package/package.json

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