valent-pipeline 0.19.29 → 0.19.30

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "valent-pipeline",
-  "version": "0.19.29",
+  "version": "0.19.30",
   "description": "v3 multi-agent AI pipeline for software development lifecycle",
   "type": "module",
   "bin": {

package/src/lib/sprint.js

@@ -154,14 +154,21 @@ export function packSprint(stories, velocity, opts = {}) {
   const groomed = stories.filter((s) => (!scope || scope.has(s.id)) && isExecReady(s) && bugsResolvable(s));
   const byId = new Map(groomed.map((s) => [s.id, s]));
 
-  // Out-of-batch prerequisite check (review 2026-06-10 #20): a depends_on that is not packable
-  // this sprint clears only if it already landed (or is absent from the file — pre-backlog work);
-  // a pending/stranded/blocked prerequisite withholds the dependent.
-  const outOfBatchDepsSatisfied = (story) => depsOf(story).every((depId) => {
-    if (byId.has(depId)) return true; // packable this sprint — sequenced before the story below
-    const known = byIdAll.get(depId);
-    return !known || satisfied.has(known.status);
-  });
+  // Out-of-batch prerequisite check (review 2026-06-10 #20), now TRANSITIVE (HARNESS-GAPS 2026-06-18
+  // Gap 3): a depends_on clears only if it already landed (or is absent — pre-backlog work), OR is
+  // packable this sprint AND its OWN dependency chain likewise clears. Checking only DIRECT deps let
+  // an in-sprint dep whose prerequisite is unshipped slip in as a velocity-filler, building against
+  // unbuilt code — `resolveEligibleStories` already requires the WHOLE chain to be live; the packer
+  // must agree or the two disagree about what runs.
+  const outOfBatchDepsSatisfied = (story, seen = new Set()) => {
+    if (seen.has(story.id)) return true; // cycle guard — a dependency cycle is a backlog error, not the packer's to resolve
+    seen.add(story.id);
+    return depsOf(story).every((depId) => {
+      if (byId.has(depId)) return outOfBatchDepsSatisfied(byId.get(depId), seen); // in-sprint dep — its chain must hold too
+      const known = byIdAll.get(depId);
+      return !known || satisfied.has(known.status);
+    });
+  };
 
   const byPriority = [...groomed].sort(
     (a, b) => (a.priority ?? Infinity) - (b.priority ?? Infinity),