@nathapp/nax 0.67.0 → 0.67.2

package/dist/nax.js

@@ -35108,6 +35108,24 @@ async function getChangedFiles(workdir, fromRef = "HEAD") {
   return output.trim().split(`
 `).filter(Boolean);
 }
+async function getAddedLinesPerFile(workdir, fromRef = "HEAD") {
+  const proc = _isolationDeps.spawn(["git", "diff", "--numstat", fromRef], {
+    cwd: workdir,
+    stdout: "pipe",
+    stderr: "pipe"
+  });
+  const output = await Bun.readableStreamToText(proc.stdout);
+  await proc.exited;
+  const result = new Map;
+  for (const line of output.trim().split(`
+`).filter(Boolean)) {
+    const [addedStr, _deletedStr, path4] = line.split("\t");
+    const added = Number.parseInt(addedStr ?? "", 10);
+    if (path4 && Number.isFinite(added))
+      result.set(path4, added);
+  }
+  return result;
+}
 function matchesAllowedPath(filePath, allowedPaths) {
   return allowedPaths.some((pattern) => {
     const regexPattern = pattern.replace(/\*\*/g, ".*").replace(/\*/g, "[^/]*").replace(/\//g, "\\/");
@@ -35115,17 +35133,25 @@ function matchesAllowedPath(filePath, allowedPaths) {
     return regex.test(filePath);
   });
 }
-async function verifyTestWriterIsolation(workdir, beforeRef, allowedPaths = ["src/index.ts", "src/**/index.ts"], testFilePatterns = DEFAULT_TEST_FILE_PATTERNS) {
+async function verifyTestWriterIsolation(workdir, beforeRef, allowedPaths = ["src/index.ts", "src/**/index.ts"], testFilePatterns = DEFAULT_TEST_FILE_PATTERNS, mode = "strict") {
   const changed = await getChangedFiles(workdir, beforeRef);
   const sourceFiles = changed.filter((f) => isSourceFile(f) && !isTestFileByPatterns(f, testFilePatterns));
+  const addedLines = mode === "lite" && sourceFiles.length > 0 ? await getAddedLinesPerFile(workdir, beforeRef) : null;
   const softViolations = [];
   const violations = [];
   for (const file3 of sourceFiles) {
     if (matchesAllowedPath(file3, allowedPaths)) {
       softViolations.push(file3);
-    } else {
-      violations.push(file3);
+      continue;
+    }
+    if (addedLines) {
+      const added = addedLines.get(file3) ?? Number.POSITIVE_INFINITY;
+      if (added <= LITE_STUB_ADDED_LINES_CEILING) {
+        softViolations.push(file3);
+        continue;
+      }
     }
+    violations.push(file3);
   }
   return {
     passed: violations.length === 0,
@@ -35151,7 +35177,7 @@ async function verifyImplementerIsolation(workdir, beforeRef, testFilePatterns =
     description: "Implementer should not modify test files"
   };
 }
-var _isolationDeps, SRC_PATTERNS;
+var _isolationDeps, SRC_PATTERNS, LITE_STUB_ADDED_LINES_CEILING = 20;
 var init_isolation = __esm(() => {
   init_test_runners();
   init_bun_deps();
@@ -35235,7 +35261,7 @@ var init_write_test = __esm(() => {
         return parsed;
       const allowedPaths = ctx.config.tdd?.testWriterAllowedPaths ?? ["src/index.ts", "src/**/index.ts"];
       const testFilePatterns = typeof ctx.packageView.config.execution?.smartTestRunner === "object" && ctx.packageView.config.execution.smartTestRunner !== null ? ctx.packageView.config.execution.smartTestRunner.testFilePatterns : undefined;
-      const isolation = await verifyTestWriterIsolation(ctx.packageView.packageDir, input.beforeRef, allowedPaths, testFilePatterns);
+      const isolation = await verifyTestWriterIsolation(ctx.packageView.packageDir, input.beforeRef, allowedPaths, testFilePatterns, input.lite ? "lite" : "strict");
       return { ...parsed, isolation };
     }
   };
@@ -37189,10 +37215,32 @@ async function runRectificationLoop(opts) {
       }
     }
   });
-  const initialFailure = {
-    testOutput,
-    testSummary
-  };
+  let initialFailure = { testOutput, testSummary };
+  if (rectificationConfig.abortOnIncreasingFailures) {
+    const preCheck = await _rectificationDeps.runVerification({
+      workdir,
+      expectedFiles: getExpectedFiles(story),
+      command: testCommand,
+      timeoutSeconds,
+      forceExit: config2.quality.forceExit,
+      detectOpenHandles: config2.quality.detectOpenHandles,
+      detectOpenHandlesRetries: config2.quality.detectOpenHandlesRetries,
+      timeoutRetryCount: 0,
+      gracePeriodMs: config2.quality.gracePeriodMs,
+      drainTimeoutMs: config2.quality.drainTimeoutMs,
+      shell: config2.quality.shell,
+      stripEnvVars: config2.quality.stripEnvVars
+    });
+    if (preCheck.output) {
+      const preCheckSummary = parseTestOutput(preCheck.output);
+      initialFailure = { testOutput: preCheck.output, testSummary: preCheckSummary };
+    } else {
+      logger?.warn("rectification", "pre-check returned no output \u2014 abort baseline may be scope-mismatched", {
+        storyId: story.id,
+        preCheckStatus: preCheck.status
+      });
+    }
+  }
   const outcome = await runRetryLoop({
     stage: "rectification",
     storyId: story.id,
@@ -54346,10 +54394,13 @@ class ExecutionPlan {
     const phaseOutputs = {};
     const startedAt = Date.now();
     const logger = getSafeLogger();
-    const shortCircuitExempt = this.state.rectification ? new Set([
+    const verifierPresent = this.state.verifier !== undefined;
+    const rectificationExempt = this.state.rectification ? [
       ...this.state.fullSuiteGate ? [this.state.fullSuiteGate.slot.op.name] : [],
       ...this.state.verifier ? [this.state.verifier.slot.op.name] : []
-    ]) : new Set;
+    ] : [];
+    const verifierExempt = verifierPresent && this.state.fullSuiteGate ? [this.state.fullSuiteGate.slot.op.name] : [];
+    const shortCircuitExempt = new Set([...rectificationExempt, ...verifierExempt]);
     for (const phase of collectOrderedPhases(this.state)) {
       try {
         await runPhase(this.ctx, phase.slot, phaseCosts, phaseOutputs, this.isThreeSession);
@@ -54577,7 +54628,8 @@ async function assemblePlanInputsFromCtx(ctx) {
     story,
     promptMarkdown: testWriterPrompt,
     featureContextMarkdown: ctx.featureContextMarkdown,
-    constitution: ctx.constitution?.content
+    constitution: ctx.constitution?.content,
+    lite: isLite
   } : undefined;
   const greenfieldGateInput = _isTdd && _isFreshRun && resolvedTestPatterns ? { story, workdir: ctx.workdir, resolvedTestPatterns } : undefined;
   const implementerInput = {
@@ -54778,6 +54830,13 @@ function deriveTddFailureCategory(phaseOutputs) {
     }
     return "tests-failing";
   }
+  const verifierPassed = verifierOutput?.success === true;
+  if (!verifierPassed) {
+    const gateOutput = phaseOutputs[fullSuiteGateOp.name];
+    if (gateOutput && (gateOutput.success === false || gateOutput.passed === false)) {
+      return "tests-failing";
+    }
+  }
   const implOutput = phaseOutputs[implementerOp.name];
   if (implOutput?.success === false) {
     return "session-failure";
@@ -59160,7 +59219,7 @@ var package_default;
 var init_package = __esm(() => {
   package_default = {
     name: "@nathapp/nax",
-    version: "0.67.0",
+    version: "0.67.2",
     description: "AI Coding Agent Orchestrator \u2014 loops until done",
     type: "module",
     bin: {
@@ -59255,8 +59314,8 @@ var init_version = __esm(() => {
   NAX_VERSION = package_default.version;
   NAX_COMMIT = (() => {
     try {
-      if (/^[0-9a-f]{6,10}$/.test("ffa2f392"))
-        return "ffa2f392";
+      if (/^[0-9a-f]{6,10}$/.test("d2b13ea6"))
+        return "d2b13ea6";
     } catch {}
     try {
       const result = Bun.spawnSync(["git", "rev-parse", "--short", "HEAD"], {

package/package.json

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