@ikunin/sprintpilot 2.3.3 → 2.3.5

package/_Sprintpilot/manifest.yaml

@@ -1,6 +1,6 @@
 addon:
   name: sprintpilot
-  version: 2.3.3
+  version: 2.3.5
   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.
@@ -492,11 +508,16 @@ function renderMermaid(dag, plan) {
   const epicIssueIds = issueIdByEpicId(plan);
   const { intra, cross } = bucketEdges(dag.edges);
   const lines = [];
+  // The diagram-type declaration MUST be the first non-blank line.
+  // Strict markdown→mermaid renderers (Claude Code's chat renderer
+  // included) sniff for `flowchart`/`graph`/etc. as the first content
+  // line; leading `%%` comments cause them to silently skip rendering.
+  // Comments after the type declaration are fine.
+  lines.push('flowchart LR');
   lines.push(`%% plan-id: ${plan?.plan_id ?? 'unknown'}`);
   lines.push(`%% generated: ${plan?.generated ?? new Date().toISOString()}`);
   lines.push('%% Sprint plan DAG — node fill encodes plan_status; cross-epic edges are dashed.');
   lines.push('%% Story labels are prefixed with their issue_id when set in plan.stories.');
-  lines.push('flowchart LR');
 
   // Group nodes by epic (if any). When sprint-wide, emit subgraphs.
   const epicGroups = new Map();

package/package.json

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