@nathapp/nax 0.67.5 → 0.67.7

package/dist/nax.js

@@ -52096,7 +52096,12 @@ async function runRectification(ctx, state, phaseCosts, phaseOutputs) {
       storyId: ctx.storyId
     });
   }
-  const exhaustedReasons = new Set(["max-attempts-total", "max-attempts-per-strategy", "bail-when"]);
+  const exhaustedReasons = new Set([
+    "max-attempts-total",
+    "max-attempts-per-strategy",
+    "bail-when",
+    "no-strategy"
+  ]);
   if (exhaustedReasons.has(cycleResult.exitReason) && cycleResult.finalFindings.length > 0) {
     return { rectificationExhausted: true, unfixedFindings: cycleResult.finalFindings };
   }
@@ -52634,6 +52639,9 @@ async function closeAllRunSessions(sessionManager, agentGetFn, opts) {
 }
 
 // src/execution/post-run.ts
+function shouldRollbackTddFailure(tddMode, failureCategory) {
+  return tddMode?.rollbackEnabled === true && failureCategory === "isolation-violation";
+}
 function extractPauseReason(phaseOutputs) {
   for (const output of Object.values(phaseOutputs)) {
     if (output !== null && typeof output === "object") {
@@ -52816,8 +52824,11 @@ async function decideStageAction(ctx, planResult, inspection, opts) {
   const logger = getLogger();
   const isTdd = opts.tddMode !== null;
   const isLiteMode = opts.tddMode?.isLite ?? false;
-  const shouldRollback = opts.tddMode?.rollbackEnabled === true;
+  const shouldRollback = shouldRollbackTddFailure(opts.tddMode, inspection.failureCategory);
   const { agentResult, selfVerificationFailed, pauseReason, failureCategory, needsHumanReview, combinedOutput } = inspection;
+  if (isTdd && !planResult.success) {
+    ctx.tddFailureCategory = failureCategory;
+  }
   if (planResult.rectificationExhausted && planResult.unfixedFindings && planResult.unfixedFindings.length > 0) {
     const sources = new Set(planResult.unfixedFindings.map((f) => f.source));
     const allMechanical = [...sources].every((s) => s === "lint" || s === "typecheck");
@@ -52827,6 +52838,16 @@ async function decideStageAction(ctx, planResult, inspection, opts) {
       });
       return { action: "continue" };
     }
+    if (!(isTdd && shouldRollback)) {
+      const findingSources = [...sources].filter((source) => typeof source === "string");
+      logger.error("execution", "Rectification exhausted with unfixed findings", {
+        storyId: ctx.story.id,
+        findingsCount: planResult.unfixedFindings.length,
+        findingSources
+      });
+      await cleanupSessionOnFailure(ctx);
+      return { action: "escalate", reason: "Rectification exhausted with unfixed findings" };
+    }
   }
   if (selfVerificationFailed) {
     logger.warn("execution", "Self-verification reported explicit failure", {
@@ -52862,7 +52883,6 @@ Reason: ${pauseReason}`,
     return { action: "pause", reason: pauseReason };
   }
   if (isTdd && !planResult.success) {
-    ctx.tddFailureCategory = failureCategory;
     if (shouldRollback && opts.initialRef) {
       try {
         await _postRunDeps.rollbackToRef(ctx.workdir, opts.initialRef);
@@ -56733,7 +56753,7 @@ var package_default;
 var init_package = __esm(() => {
   package_default = {
     name: "@nathapp/nax",
-    version: "0.67.5",
+    version: "0.67.7",
     description: "AI Coding Agent Orchestrator \u2014 loops until done",
     type: "module",
     bin: {
@@ -56828,8 +56848,8 @@ var init_version = __esm(() => {
   NAX_VERSION = package_default.version;
   NAX_COMMIT = (() => {
     try {
-      if (/^[0-9a-f]{6,10}$/.test("bd8ffe1a"))
-        return "bd8ffe1a";
+      if (/^[0-9a-f]{6,10}$/.test("1c26c38d"))
+        return "1c26c38d";
     } catch {}
     try {
       const result = Bun.spawnSync(["git", "rev-parse", "--short", "HEAD"], {

package/package.json

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