@nathapp/nax 0.68.4 → 0.68.5

package/dist/nax.js

@@ -16837,6 +16837,7 @@ var init_schemas_execution = __esm(() => {
     fullSuiteTimeoutSeconds: exports_external.number().int().min(10).max(600).default(120),
     maxFailureSummaryChars: exports_external.number().int().min(500).max(1e4).default(2000),
     abortOnIncreasingFailures: exports_external.boolean().default(true),
+    consecutiveIncreasesToBail: exports_external.number().int().min(1).max(10).default(2),
     escalateOnExhaustion: exports_external.boolean().optional().default(true),
     rethinkAtAttempt: exports_external.number().int().min(1).default(2),
     urgencyAtAttempt: exports_external.number().int().min(1).default(3)
@@ -17321,6 +17322,7 @@ var init_schemas3 = __esm(() => {
         fullSuiteTimeoutSeconds: 300,
         maxFailureSummaryChars: 2000,
         abortOnIncreasingFailures: true,
+        consecutiveIncreasesToBail: 2,
         escalateOnExhaustion: true,
         rethinkAtAttempt: 2,
         urgencyAtAttempt: 3
@@ -19086,6 +19088,12 @@ function resolveTestStrategy(raw) {
     return "three-session-tdd-lite";
   return "test-after";
 }
+function isThreeSessionStrategy(strategy) {
+  return strategy !== undefined && THREE_SESSION_STRATEGIES.has(strategy);
+}
+function isSingleSessionTestOwningStrategy(strategy) {
+  return strategy !== undefined && SINGLE_SESSION_TEST_OWNING_STRATEGIES.has(strategy);
+}
 function getAcQualityRules(profile) {
   const langSection = profile?.language ? LANGUAGE_PATTERNS[profile.language] : undefined;
   const typeSection = profile?.type ? TYPE_PATTERNS[profile.type] : undefined;
@@ -19098,7 +19106,7 @@ function getAcQualityRules(profile) {
 
 ${extras}`;
 }
-var VALID_TEST_STRATEGIES, COMPLEXITY_GUIDE = `## Complexity Classification Guide
+var VALID_TEST_STRATEGIES, THREE_SESSION_STRATEGIES, SINGLE_SESSION_TEST_OWNING_STRATEGIES, COMPLEXITY_GUIDE = `## Complexity Classification Guide
 
 Classify each story's complexity based on scope and risk \u2014 NOT acceptance criteria count.
 A story with 10 simple "add field" ACs is simpler than one with 3 ACs involving concurrent
@@ -19239,6 +19247,11 @@ var init_test_strategy = __esm(() => {
     "three-session-tdd",
     "three-session-tdd-lite"
   ];
+  THREE_SESSION_STRATEGIES = new Set([
+    "three-session-tdd",
+    "three-session-tdd-lite"
+  ]);
+  SINGLE_SESSION_TEST_OWNING_STRATEGIES = new Set(["tdd-simple", "test-after"]);
   LANGUAGE_PATTERNS = {
     go: `### Go-Specific AC Patterns
 
@@ -36046,7 +36059,7 @@ var init_autofix_test_writer = __esm(() => {
     kind: "run",
     name: "autofix-test-writer",
     stage: "rectification",
-    session: { role: "test-writer", lifetime: "fresh" },
+    session: { role: "test-writer", lifetime: "warm" },
     config: autofixConfigSelector,
     build(input, _ctx) {
       const prompt = RectifierPromptBuilder.testWriterRectification(input.failedChecks, input.story, {
@@ -36838,18 +36851,36 @@ var init__session_output = __esm(() => {
   EMPTY = { success: false, filesChanged: [], output: "", parsed: false };
 });
 
+// src/operations/execution-gates.ts
+function shouldRunReview(config2) {
+  return config2.review?.enabled === true;
+}
+function shouldRunRectification(config2) {
+  return config2.execution?.rectification?.enabled === true;
+}
+function shouldKeepSessionOpen(config2, role) {
+  return SESSION_CONTINUITY_ROLES.has(role) && (shouldRunReview(config2) || shouldRunRectification(config2));
+}
+var SESSION_CONTINUITY_ROLES;
+var init_execution_gates = __esm(() => {
+  init_config();
+  SESSION_CONTINUITY_ROLES = new Set(["implementer", "test-writer"]);
+});
+
 // src/operations/write-test.ts
 var testWriterOp;
 var init_write_test = __esm(() => {
   init_config();
   init_isolation();
   init__session_output();
+  init_execution_gates();
   testWriterOp = {
     kind: "run",
     name: "test-writer",
     stage: "run",
-    session: { role: "test-writer", lifetime: "fresh" },
+    session: { role: "test-writer", lifetime: "warm" },
     config: tddConfigSelector,
+    keepOpen: (_input, ctx) => shouldKeepSessionOpen(ctx.config, "test-writer"),
     build(input, _ctx) {
       if (input.promptMarkdown?.trim()) {
         return {
@@ -36896,20 +36927,6 @@ var init_write_test = __esm(() => {
   };
 });
 
-// src/operations/execution-gates.ts
-function shouldRunReview(config2) {
-  return config2.review?.enabled === true;
-}
-function shouldRunRectification(config2) {
-  return config2.execution?.rectification?.enabled === true;
-}
-function shouldKeepSessionOpen(config2, role) {
-  return role === "implementer" && (shouldRunReview(config2) || shouldRunRectification(config2));
-}
-var init_execution_gates = __esm(() => {
-  init_config();
-});
-
 // src/operations/implement.ts
 var implementerOp;
 var init_implement = __esm(() => {
@@ -38416,10 +38433,11 @@ var init__finding_to_check = __esm(() => {
 });
 
 // src/operations/autofix-implementer-strategy.ts
-function makeAutofixImplementerStrategy(story, config2, sink) {
+function makeAutofixImplementerStrategy(story, config2, sink, opts = {}) {
+  const claimsAdversarial = opts.includeAdversarialReview === true;
   return {
     name: "autofix-implementer",
-    appliesTo: (f) => f.fixTarget === "source" && IMPLEMENTER_SOURCES.has(f.source),
+    appliesTo: (f) => f.fixTarget === "source" && IMPLEMENTER_SOURCES.has(f.source) || claimsAdversarial && f.source === "adversarial-review",
     fixOp: implementerRectifyOp,
     buildInput: (findings, _prior, _cycleCtx) => ({
       failedChecks: findingsToFailedChecks(findings),
@@ -41078,18 +41096,18 @@ ${exceptions.join(`
 `)}`;
 }
 function implementerOwnsTests(story) {
-  return SINGLE_SESSION_TEST_OWNING_STRATEGIES.has(story.routing?.testStrategy ?? "");
+  return isSingleSessionTestOwningStrategy(story.routing?.testStrategy);
 }
 function testEditHeadline(story, prohibition) {
   return implementerOwnsTests(story) ? SINGLE_SESSION_PERMIT_HEADLINE : prohibition;
 }
 function exceptionCountWord(story) {
-  return THREE_SESSION_STRATEGIES.has(story.routing?.testStrategy ?? "") ? "four" : "three";
+  return isThreeSessionStrategy(story.routing?.testStrategy) ? "four" : "three";
 }
 function escapeHatchFor(story) {
   if (implementerOwnsTests(story))
     return SINGLE_SESSION_TEST_EDIT_POLICY;
-  const isTdd = THREE_SESSION_STRATEGIES.has(story.routing?.testStrategy ?? "");
+  const isTdd = isThreeSessionStrategy(story.routing?.testStrategy);
   return buildEscapeHatch({ includeMockHandoff: isTdd });
 }
 function noTestIsolationBlock(story) {
@@ -41294,7 +41312,7 @@ REASON: <one paragraph: which mock is wrong vs which dispatch the new code uses,
 Rules:
 - Do NOT make any edits yourself; the test-writer will fulfill.
 - Do NOT also emit \`UNRESOLVED:\` in the same turn \u2014 this declaration IS the handoff.
-- FILES must list real test files. Each path must exist and be a test file.`, THREE_SESSION_STRATEGIES, SINGLE_SESSION_TEST_OWNING_STRATEGIES, SINGLE_SESSION_PERMIT_HEADLINE = "You authored these tests in the same session as the implementation, so you MAY edit test files \u2014 but ONLY to resolve a genuine contradiction between a test and this story's acceptance criteria (or between two acceptance criteria). NEVER weaken, delete, loosen, or skip a test merely to make it pass. See the test-edit guidance appended below.", SINGLE_SESSION_TEST_EDIT_POLICY = `
+- FILES must list real test files. Each path must exist and be a test file.`, SINGLE_SESSION_PERMIT_HEADLINE = "You authored these tests in the same session as the implementation, so you MAY edit test files \u2014 but ONLY to resolve a genuine contradiction between a test and this story's acceptance criteria (or between two acceptance criteria). NEVER weaken, delete, loosen, or skip a test merely to make it pass. See the test-edit guidance appended below.", SINGLE_SESSION_TEST_EDIT_POLICY = `
 
 ## Test-edit guidance (single-session implementer)
 
@@ -41313,10 +41331,9 @@ If two findings or two acceptance criteria contradict each other and you cannot
 both even after adjusting tests, do not guess. Emit:
 UNRESOLVED: <which findings/ACs conflicted and why they cannot both be satisfied>`, CONTRADICTION_ESCAPE_HATCH, MAX_STRUCTURED_FINDINGS = 10, RAW_WITH_FINDINGS_LIMIT = 1000, RAW_FALLBACK_LIMIT = 4000;
 var init_rectifier_builder_helpers = __esm(() => {
+  init_config();
   init_review();
   init_sections2();
-  THREE_SESSION_STRATEGIES = new Set(["three-session-tdd", "three-session-tdd-lite"]);
-  SINGLE_SESSION_TEST_OWNING_STRATEGIES = new Set(["tdd-simple", "test-after"]);
   CONTRADICTION_ESCAPE_HATCH = buildEscapeHatch({ includeMockHandoff: false });
 });
 
@@ -53430,6 +53447,11 @@ function phasesToRevalidate(strategiesRun, allPhases) {
   }
   return allPhases.filter((p) => needed.has(p.kind));
 }
+function orderGateLast(phases) {
+  const rest = phases.filter((p) => p.kind !== "full-suite-gate");
+  const gates = phases.filter((p) => p.kind === "full-suite-gate");
+  return [...rest, ...gates];
+}
 function toReviewDecisionPayload(opName, output) {
   if (output === null || output === undefined || typeof output !== "object")
     return null;
@@ -53644,18 +53666,24 @@ async function runPhase(ctx, slot, phaseCosts, phaseOutputs, isThreeSession = fa
     scope.close();
   }
 }
-function withIncreasingFailuresBail(strategies, enabled) {
+function withIncreasingFailuresBail(strategies, enabled, consecutiveIncreases) {
   if (!enabled)
     return strategies;
+  const threshold = Math.max(1, consecutiveIncreases);
   return strategies.map((strategy) => ({
     ...strategy,
     bailWhen: (iterations) => {
       const userReason = strategy.bailWhen?.(iterations) ?? null;
       if (userReason !== null)
         return userReason;
-      const last = iterations[iterations.length - 1];
-      if (last && last.findingsAfter.length > last.findingsBefore.length) {
-        return `failure count increased: ${last.findingsBefore.length} -> ${last.findingsAfter.length}`;
+      if (iterations.length < threshold)
+        return null;
+      const trailing = iterations.slice(-threshold);
+      const allRegressed = trailing.every((it) => it.findingsAfter.length > it.findingsBefore.length);
+      if (allRegressed) {
+        const first = trailing[0];
+        const last = trailing[trailing.length - 1];
+        return `failure count increased for ${threshold} consecutive iteration(s): ${first.findingsBefore.length} -> ${last.findingsAfter.length}`;
       }
       return null;
     }
@@ -53685,13 +53713,14 @@ async function runRectification(ctx, state, phaseCosts, phaseOutputs) {
   const cycle = {
     findings: [...initialFindings],
     iterations: [],
-    strategies: withIncreasingFailuresBail(rectification.strategies, rectification.abortOnIncreasingFailures),
+    strategies: withIncreasingFailuresBail(rectification.strategies, rectification.abortOnIncreasingFailures, rectification.consecutiveIncreasesToBail ?? 1),
     config: { maxAttemptsTotal: rectification.maxAttempts, validatorRetries: 1 },
     validate: async (_validateCtx, opts) => {
       if (ctx.runtime.signal?.aborted)
         return { findings: [], shortCircuited: false };
       const lite = (opts?.mode ?? "full") === "lite";
-      const phases = phasesToRevalidate(opts?.strategiesRun, validationPhases);
+      const selected = phasesToRevalidate(opts?.strategiesRun, validationPhases);
+      const phases = lite ? orderGateLast(selected) : selected;
       getSafeLogger()?.debug("story-orchestrator", "rectification validate scope", {
         storyId: ctx.storyId,
         mode: opts?.mode ?? "full",
@@ -53701,9 +53730,6 @@ async function runRectification(ctx, state, phaseCosts, phaseOutputs) {
       const findings = [];
       let shortCircuited = false;
       for (const phase of phases) {
-        if (lite && phase.kind === "full-suite-gate") {
-          continue;
-        }
         await runPhase(ctx, phase.slot, phaseCosts, phaseOutputs);
         if (shouldSkipPhaseForRectification(phase, state, phaseOutputs))
           continue;
@@ -54019,9 +54045,6 @@ var init_story_orchestrator = __esm(() => {
 
 // src/execution/build-plan-for-strategy.ts
 import { join as join46 } from "path";
-function isThreeSessionStrategy(strategy) {
-  return THREE_SESSION_STRATEGIES2.has(strategy);
-}
 function requiresInitialRefCapture(strategy) {
   return isThreeSessionStrategy(strategy);
 }
@@ -54080,8 +54103,12 @@ async function buildPlanForStrategy(ctx, story, config2, testStrategy, inputs) {
       strategies.push(makeFullSuiteRectifyStrategy(story, config2));
     }
     if (config2.quality.autofix?.enabled !== false) {
-      strategies.push(makeAutofixImplementerStrategy(story, config2, sink));
-      strategies.push(makeAutofixTestWriterStrategy(story, config2, sink));
+      strategies.push(makeAutofixImplementerStrategy(story, config2, sink, {
+        includeAdversarialReview: !isThreeSession
+      }));
+      if (isThreeSession) {
+        strategies.push(makeAutofixTestWriterStrategy(story, config2, sink));
+      }
     }
     const postValidate = async (findings, _validateCtx) => {
       if (sink.testEdits.length === 0 && sink.mockHandoffs.length === 0)
@@ -54107,14 +54134,13 @@ async function buildPlanForStrategy(ctx, story, config2, testStrategy, inputs) {
   }
   return builder.build(ctx, { isThreeSession });
 }
-var THREE_SESSION_STRATEGIES2;
 var init_build_plan_for_strategy = __esm(() => {
+  init_config();
   init_operations();
   init_execution_gates();
   init_full_suite_rectify();
   init_test_runners();
   init_story_orchestrator();
-  THREE_SESSION_STRATEGIES2 = new Set(["three-session-tdd", "three-session-tdd-lite"]);
 });
 
 // src/execution/plan-inputs.ts
@@ -54314,7 +54340,8 @@ async function assemblePlanInputsFromCtx(ctx) {
   const rectificationInput = ctx.config.execution?.rectification?.enabled === true ? {
     maxAttempts: ctx.config.execution.rectification.maxAttemptsTotal,
     strategies: [],
-    abortOnIncreasingFailures: ctx.config.execution.rectification.abortOnIncreasingFailures
+    abortOnIncreasingFailures: ctx.config.execution.rectification.abortOnIncreasingFailures,
+    consecutiveIncreasesToBail: ctx.config.execution.rectification.consecutiveIncreasesToBail
   } : undefined;
   return {
     story,
@@ -54333,12 +54360,12 @@ async function assemblePlanInputsFromCtx(ctx) {
   };
 }
 var init_plan_inputs = __esm(() => {
+  init_config();
   init_context();
   init_errors();
   init_prompts();
   init_review();
   init_resolver();
-  init_build_plan_for_strategy();
 });
 
 // src/pipeline/stages/execution-helpers.ts
@@ -54865,6 +54892,7 @@ var init_post_run = __esm(() => {
 // src/pipeline/stages/execution.ts
 var RUNTIME_CRASH_CODES, executionStage, _executionDeps;
 var init_execution = __esm(() => {
+  init_config();
   init_agents();
   init_errors();
   init_build_plan_for_strategy();
@@ -58383,7 +58411,7 @@ var package_default;
 var init_package = __esm(() => {
   package_default = {
     name: "@nathapp/nax",
-    version: "0.68.4",
+    version: "0.68.5",
     description: "AI Coding Agent Orchestrator \u2014 loops until done",
     type: "module",
     bin: {
@@ -58478,8 +58506,8 @@ var init_version = __esm(() => {
   NAX_VERSION = package_default.version;
   NAX_COMMIT = (() => {
     try {
-      if (/^[0-9a-f]{6,10}$/.test("197c6530"))
-        return "197c6530";
+      if (/^[0-9a-f]{6,10}$/.test("1056817e"))
+        return "1056817e";
     } catch {}
     try {
       const result = Bun.spawnSync(["git", "rev-parse", "--short", "HEAD"], {
@@ -96937,6 +96965,7 @@ var FIELD_DESCRIPTIONS = {
   "execution.rectification.fullSuiteTimeoutSeconds": "Timeout for full test suite run in seconds",
   "execution.rectification.maxFailureSummaryChars": "Max characters in failure summary",
   "execution.rectification.abortOnIncreasingFailures": "Abort if failure count increases",
+  "execution.rectification.consecutiveIncreasesToBail": "Consecutive regressing iterations required before abortOnIncreasingFailures bails (default: 2; 1 = legacy behaviour)",
   "execution.rectification.escalateOnExhaustion": "Enable model tier escalation when attempts are exhausted with remaining failures",
   "execution.rectification.rethinkAtAttempt": "Attempt number at which 'rethink your approach' language is injected into the prompt (default: 2)",
   "execution.rectification.urgencyAtAttempt": "Attempt number at which 'final chance before escalation' urgency is added (default: 3)",

package/package.json

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