@ikunin/sprintpilot 2.2.20 → 2.2.22

package/_Sprintpilot/lib/orchestrator/verify.js

@@ -353,22 +353,33 @@ function verifyDevRed(state, out, ctx) {
 
 function verifyDevGreen(state, out, ctx) {
   const issues = [];
+  let runnerTestsRun = null;
   if (ctx.runner) {
     const result = ctx.runner({ phase: 'green', files: out.test_files || [] });
     if (!result || typeof result.exit_code !== 'number') {
       issues.push('runner did not report exit_code');
     } else if (result.exit_code !== 0) {
       issues.push(`tests still failing on GREEN: exit ${result.exit_code}`);
-    } else if (typeof result.tests_run === 'number' && typeof out.tests_run === 'number') {
-      if (result.tests_run !== out.tests_run) {
-        issues.push(
-          `LLM reported ${out.tests_run} tests run but runner reported ${result.tests_run}`,
-        );
+    } else {
+      if (typeof result.tests_run === 'number') runnerTestsRun = result.tests_run;
+      if (typeof result.tests_run === 'number' && typeof out.tests_run === 'number') {
+        if (result.tests_run !== out.tests_run) {
+          issues.push(
+            `LLM reported ${out.tests_run} tests run but runner reported ${result.tests_run}`,
+          );
+        }
       }
     }
   }
+  // If the LLM omitted tests_run but the runner reported a positive
+  // count, accept the runner's number (same pattern as test_files
+  // auto-detect). A non-runner setup still requires the LLM to report.
   if (typeof out.tests_run !== 'number' || out.tests_run <= 0) {
-    issues.push('tests_run must be a positive number (per AGENTS.md test-result format)');
+    if (typeof runnerTestsRun === 'number' && runnerTestsRun > 0) {
+      // Recovered — don't push the "must be a positive number" issue.
+    } else {
+      issues.push('tests_run must be a positive number (per AGENTS.md test-result format)');
+    }
   }
   return { ok: issues.length === 0, issues };
 }
@@ -449,6 +460,7 @@ function verifyPatchApply(state, out, _ctx) {
 
 function verifyPatchRetest(state, out, ctx) {
   const issues = [];
+  let runnerTestsRun = null;
   if (ctx.runner) {
     const result = ctx.runner({
       phase: 'rereview',
@@ -458,10 +470,16 @@ function verifyPatchRetest(state, out, ctx) {
       issues.push('runner did not report exit_code');
     } else if (result.exit_code !== 0) {
       issues.push(`tests failed after patch: exit ${result.exit_code}`);
+    } else if (typeof result.tests_run === 'number') {
+      runnerTestsRun = result.tests_run;
     }
   }
+  // Same auto-recovery as verifyDevGreen: accept the runner's count when
+  // the LLM omits tests_run.
   if (typeof out.tests_run !== 'number' || out.tests_run <= 0) {
-    issues.push('tests_run must be a positive number');
+    if (!(typeof runnerTestsRun === 'number' && runnerTestsRun > 0)) {
+      issues.push('tests_run must be a positive number');
+    }
   }
   return { ok: issues.length === 0, issues };
 }
@@ -548,10 +566,26 @@ function verifyRetrospective(state, _out, ctx) {
 
 function verifyNanoQuickDev(state, out, ctx) {
   const issues = [];
+  // tests_run: accept the runner's count if the LLM omitted it (same
+  // pattern as verifyDevGreen). Nano flow can also have ctx.runner.
+  let runnerTestsRun = null;
+  if (ctx.runner) {
+    const result = ctx.runner({ phase: 'quick', files: out.test_files || [] });
+    if (result && typeof result.tests_run === 'number') runnerTestsRun = result.tests_run;
+  }
   if (typeof out.tests_run !== 'number' || out.tests_run <= 0) {
-    issues.push('tests_run must be a positive number');
+    if (!(typeof runnerTestsRun === 'number' && runnerTestsRun > 0)) {
+      issues.push('tests_run must be a positive number');
+    }
   }
   if (typeof out.tests_failed !== 'number') {
+    // tests_failed missing: if LLM signaled success AND runner exit_code
+    // was 0 OR all tests passed (out.tests_run > 0 with no failure
+    // indicator), default to 0. Otherwise reject.
+    // Conservative: only auto-fill 0 when we have positive evidence of
+    // no failures (LLM count present and matches success status).
+    // Keeping the strict check by default — tests_failed is a key
+    // signal the verifier uses for nano escalation.
     issues.push('tests_failed required (number; 0 for clean)');
   }
   if (!out.commit_sha) issues.push('commit_sha required');

package/_Sprintpilot/manifest.yaml

@@ -1,6 +1,6 @@
 addon:
   name: sprintpilot
-  version: 2.2.20
+  version: 2.2.22
   description: Sprintpilot — autopilot and multi-agent addon for BMad Method (git workflow, parallel agents, autonomous story execution)
   bmad_compatibility: ">=6.2.0"
   modules:

package/package.json

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