@ikunin/sprintpilot 2.3.2 → 2.3.4

package/_Sprintpilot/manifest.yaml

@@ -1,6 +1,6 @@
 addon:
   name: sprintpilot
-  version: 2.3.2
+  version: 2.3.4
   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/scripts/resolve-dag.js

@@ -260,12 +260,28 @@ function buildEdges(strategies, nodes, depsDoc, { includeCrossEpic = false } = {
       out.push([a, b]);
     }
   };
+  // Track whether explicit produced any edges in this build — if it did,
+  // ordering is suppressed for the same nodeset to avoid the back-edge
+  // cycle bug: when an explicit dep points "backwards" against story
+  // order (e.g. 6-1 depends on 6-3 — explicit emits 6-3 → 6-1), the
+  // linear ordering chain (6-1 → 6-2 → 6-3) closes a fake cycle that
+  // the user never declared. Explicit deps express the user's intent
+  // in full; ordering is only a fallback when none are provided.
+  let explicitProducedEdges = false;
   for (const strat of strategies) {
     if (strat === 'explicit') {
-      pushEdges(edgesFromExplicit(depsDoc, nodes));
-      if (includeCrossEpic) pushEdges(edgesFromCrossEpic(depsDoc, nodes));
+      const explicit = edgesFromExplicit(depsDoc, nodes);
+      pushEdges(explicit);
+      if (explicit.length > 0) explicitProducedEdges = true;
+      if (includeCrossEpic) {
+        const cross = edgesFromCrossEpic(depsDoc, nodes);
+        pushEdges(cross);
+        if (cross.length > 0) explicitProducedEdges = true;
+      }
     } else if (strat === 'ordering') {
-      pushEdges(edgesFromOrdering(nodes));
+      if (!explicitProducedEdges) {
+        pushEdges(edgesFromOrdering(nodes));
+      }
     } else if (strat === 'files') {
       // files strategy opt-in, not implemented yet — a future
       // sprintpilot-infer-dependencies skill populates the explicit sidecar.

package/lib/commands/install.js

@@ -1410,6 +1410,17 @@ async function runInstall(options = {}) {
     }
   }
 
+  // v2.3.3 — Sprintpilot bundles js-yaml into _Sprintpilot/node_modules/
+  // at install time (see step 6b below). Guard against users committing it.
+  const nodeModulesResult = await addIgnoreEntry(
+    ignore.path,
+    '_Sprintpilot/node_modules/',
+    { dryRun },
+  );
+  if (nodeModulesResult.added && !dryRun) {
+    console.log(`Added '_Sprintpilot/node_modules/' to ${path.basename(ignore.path)}`);
+  }
+
   // 5. Install skills per tool
   let totalInstalled = 0;
   const allSkills = await listSkills();
@@ -1567,6 +1578,32 @@ async function runInstall(options = {}) {
     }
     console.log('Runtime resources installed to _Sprintpilot/');
 
+    // 6b. v2.3.3 — bundle js-yaml into _Sprintpilot/node_modules/.
+    //     Two runtime scripts (sprint-plan.js, infer-dependencies.js)
+    //     require('js-yaml') for full YAML parse/dump support that the
+    //     in-tree yaml-lite intentionally doesn't cover. Without this,
+    //     `/sprintpilot-plan-sprint` and dependency inference crash with
+    //     MODULE_NOT_FOUND on every invocation in the consumer project.
+    //
+    //     We resolve js-yaml from Sprintpilot's own node_modules (where
+    //     npm placed it when the user ran `npx @ikunin/sprintpilot`) and
+    //     copy the whole package into <projectRoot>/_Sprintpilot/node_modules/js-yaml/.
+    //     Node's require walk finds it from _Sprintpilot/scripts/ at runtime.
+    try {
+      const jsYamlPkgJson = require.resolve('js-yaml/package.json');
+      const jsYamlSrc = path.dirname(jsYamlPkgJson);
+      const jsYamlDest = path.join(targetAddonDir, 'node_modules', 'js-yaml');
+      await fs.remove(jsYamlDest);
+      await fs.copy(jsYamlSrc, jsYamlDest, { overwrite: true });
+      console.log('  Bundled js-yaml → _Sprintpilot/node_modules/js-yaml/');
+    } catch (err) {
+      console.warn(
+        pc.yellow(
+          `  WARN: failed to bundle js-yaml (${err.message || err}). sprint-plan and dependency inference will fail at runtime.`,
+        ),
+      );
+    }
+
     // 6a. Re-apply v1 module-config snapshot (if any) — MUST happen after
     //     step 6 because step 6 wrote pristine bundled configs that would
     //     otherwise clobber the user's values. On failure, persist the

package/package.json

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