@nathapp/nax 0.67.2 → 0.67.3

package/dist/nax.js

@@ -51805,12 +51805,12 @@ async function assertionSiteDiffCheck(workdir, beforeRef, files) {
   }
   return { violated: false };
 }
-async function runIsolationGuard(workdir, beforeRef, config2, packageDir) {
+async function runIsolationGuard(workdir, beforeRef, config2, packageDir, mode = "strict") {
   if (config2.quality.autofix?.enforceTestWriterIsolation === false) {
     return { violated: false, skipped: true };
   }
   const resolved = await resolveTestFilePatterns(config2, workdir, packageDir);
-  const result = await _guardDeps.verifyTestWriterIsolation(workdir, beforeRef, config2.tdd?.testWriterAllowedPaths, resolved.globs);
+  const result = await _guardDeps.verifyTestWriterIsolation(workdir, beforeRef, config2.tdd?.testWriterAllowedPaths, resolved.globs, mode);
   if (!result.passed) {
     return { violated: true, files: result.violations ?? [] };
   }
@@ -52155,7 +52155,7 @@ async function runAgentRectificationV2(ctx, _lintFixCmd, _formatFixCmd, _effecti
         });
         return { unresolved: `assertion_weakening:${assertionResult.file}:${assertionResult.line}` };
       }
-      const isolationResult = await _autofixCycleGuardDeps.runIsolationGuard(ctx.workdir, beforeRef, ctx.config, ctx.story.workdir || undefined);
+      const isolationResult = await _autofixCycleGuardDeps.runIsolationGuard(ctx.workdir, beforeRef, ctx.config, ctx.story.workdir || undefined, ctx.routing?.testStrategy === "three-session-tdd-lite" ? "lite" : "strict");
       if (isolationResult.violated) {
         await _autofixCycleGuardDeps.revertDiff(ctx.workdir, isolationResult.files);
         logger.info("autofix-cycle", "test-writer isolation guard violated \u2014 reverted", {
@@ -54211,6 +54211,12 @@ function collectOrderedPhases(state) {
     return [];
   });
 }
+function phaseExplicitlyPassed(output) {
+  if (output === null || output === undefined || typeof output !== "object")
+    return false;
+  const r = output;
+  return r.success === true || r.passed === true;
+}
 function phasePassed(opName, output) {
   if (output === null || output === undefined) {
     getSafeLogger()?.warn("story-orchestrator", "Phase produced no output \u2014 treating as pass", {
@@ -54419,7 +54425,17 @@ class ExecutionPlan {
       }
     }
     await runRectification(this.ctx, this.state, phaseCosts, phaseOutputs);
-    const success2 = Object.entries(phaseOutputs).every(([name, output]) => phasePassed(name, output));
+    const verifierName = this.state.verifier?.slot.op.name;
+    const gateName = this.state.fullSuiteGate?.slot.op.name;
+    const verifierPassedSsot = verifierName !== undefined && phaseExplicitlyPassed(phaseOutputs[verifierName]);
+    if (verifierPassedSsot && gateName !== undefined && !phasePassed(gateName, phaseOutputs[gateName])) {
+      logger?.warn("story-orchestrator", "Full-suite gate failed but verifier judged story OK \u2014 treating gate failures as unrelated regressions", { storyId: this.ctx.storyId, packageDir: this.ctx.packageDir });
+    }
+    const success2 = Object.entries(phaseOutputs).every(([name, output]) => {
+      if (verifierPassedSsot && name === gateName)
+        return true;
+      return phasePassed(name, output);
+    });
     const totalCostUsd = Object.values(phaseCosts).reduce((sum, cost) => sum + cost, 0);
     return {
       success: success2,
@@ -54935,6 +54951,24 @@ async function applyPostRunInspection(ctx, planResult, opts) {
   }
   const pauseReason = extractPauseReason(planResult.phaseOutputs);
   const failureCategory = isTdd && !planResult.success ? deriveTddFailureCategory(planResult.phaseOutputs) : undefined;
+  if (isTdd && !planResult.success && !failureCategory) {
+    const phaseSignals = {};
+    for (const [name, output] of Object.entries(planResult.phaseOutputs)) {
+      if (output && typeof output === "object") {
+        const r = output;
+        const signal = {};
+        if (typeof r.success === "boolean")
+          signal.success = r.success;
+        if (typeof r.passed === "boolean")
+          signal.passed = r.passed;
+        phaseSignals[name] = signal;
+      }
+    }
+    logger.warn("execution", "TDD plan failed but no failure category derived \u2014 defaulting to pause", {
+      storyId: ctx.story.id,
+      phaseSignals
+    });
+  }
   const tddIsolations = {};
   for (const opName of ["test-writer", "implementer", "verifier"]) {
     const phaseOut = planResult.phaseOutputs[opName];
@@ -59219,7 +59253,7 @@ var package_default;
 var init_package = __esm(() => {
   package_default = {
     name: "@nathapp/nax",
-    version: "0.67.2",
+    version: "0.67.3",
     description: "AI Coding Agent Orchestrator \u2014 loops until done",
     type: "module",
     bin: {
@@ -59314,8 +59348,8 @@ var init_version = __esm(() => {
   NAX_VERSION = package_default.version;
   NAX_COMMIT = (() => {
     try {
-      if (/^[0-9a-f]{6,10}$/.test("d2b13ea6"))
-        return "d2b13ea6";
+      if (/^[0-9a-f]{6,10}$/.test("bcfe96ad"))
+        return "bcfe96ad";
     } catch {}
     try {
       const result = Bun.spawnSync(["git", "rev-parse", "--short", "HEAD"], {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nathapp/nax",
-  "version": "0.67.2",
+  "version": "0.67.3",
   "description": "AI Coding Agent Orchestrator — loops until done",
   "type": "module",
   "bin": {