@nathapp/nax 0.81.0-canary.1 → 0.81.0

package/dist/nax.js

@@ -17148,8 +17148,7 @@ var init_schemas_infra = __esm(() => {
     agents: AgentRoutingConfigSchema.default({ enabled: true, strategy: "off", profiles: [] })
   });
   OptimizerConfigSchema = exports_external.object({
-    enabled: exports_external.boolean(),
-    strategy: exports_external.enum(["rule-based", "llm", "noop"]).optional()
+    enabled: exports_external.boolean()
   });
   PluginConfigEntrySchema = exports_external.object({
     module: exports_external.string().min(1, "plugin.module must be non-empty"),
@@ -18840,6 +18839,21 @@ function _applyRemovedRoutingKeysShim(conf, warn = defaultConfigWarn) {
   }
   return newRouting === routing ? conf : { ...conf, routing: newRouting };
 }
+function _applyRemovedOptimizerKeysShim(conf, warn = defaultConfigWarn) {
+  const optimizer = conf.optimizer;
+  if (!optimizer || typeof optimizer !== "object")
+    return conf;
+  const REMOVED_OPTIMIZER_KEYS = ["strategy", "strategies"];
+  let newOptimizer = optimizer;
+  for (const key of REMOVED_OPTIMIZER_KEYS) {
+    if (key in newOptimizer) {
+      warn(`optimizer.${key} was removed along with the rule-based optimizer and has no effect. Remove it from your config; set optimizer.enabled and provide a plugin optimizer instead.`);
+      const { [key]: _removed, ...rest } = newOptimizer;
+      newOptimizer = rest;
+    }
+  }
+  return newOptimizer === optimizer ? conf : { ...conf, optimizer: newOptimizer };
+}
 function _applyRemovedWorktreeInheritShim(conf, warn = defaultConfigWarn) {
   const execution = conf.execution;
   const worktreeDependencies = execution?.worktreeDependencies;
@@ -18968,7 +18982,17 @@ function _applyFinishAutoFlowShim(conf, warn = defaultConfigWarn) {
 function applyConfigCompatShims(conf, logger, dedupe) {
   const log = dedupe.wrapLogger(logger);
   const warn = dedupe.warn;
-  return _applyFinishAutoFlowShim(_applyRemovedWorktreeInheritShim(_applyLegacyReviewExecutionShim(_applyRemovedRoutingKeysShim(applyRoutingRetryDeprecationWarning(applyBatchModeCompat(applyRemovedStrategyCompat(migrateLegacyReviewModelKey(migrateLegacyTestPattern(conf, log), log), warn), warn), warn), warn), warn), warn), warn);
+  let out = migrateLegacyTestPattern(conf, log);
+  out = migrateLegacyReviewModelKey(out, log);
+  out = applyRemovedStrategyCompat(out, warn);
+  out = applyBatchModeCompat(out, warn);
+  out = applyRoutingRetryDeprecationWarning(out, warn);
+  out = _applyRemovedRoutingKeysShim(out, warn);
+  out = _applyLegacyReviewExecutionShim(out, warn);
+  out = _applyRemovedWorktreeInheritShim(out, warn);
+  out = _applyFinishAutoFlowShim(out, warn);
+  out = _applyRemovedOptimizerKeysShim(out, warn);
+  return out;
 }
 var SECURITY_SENSITIVE_KEY_PATHS, REMOVED_FINISH_KEYS;
 var init_compat_shims = __esm(() => {
@@ -29969,186 +29993,20 @@ class NoopOptimizer {
 }
 var init_noop_optimizer = () => {};
 
-// src/optimizer/rule-based.optimizer.ts
-class RuleBasedOptimizer {
-  name = "rule-based";
-  async optimize(input) {
-    const originalTokens = estimateTokens2(input.prompt);
-    const appliedRules = [];
-    let optimized = input.prompt;
-    const config2 = {
-      ...DEFAULT_CONFIG2,
-      ...input.config.optimizer?.strategies?.["rule-based"]
-    };
-    if (config2.stripWhitespace) {
-      const before = optimized;
-      optimized = this.stripWhitespace(optimized);
-      if (optimized !== before) {
-        appliedRules.push("stripWhitespace");
-      }
-    }
-    if (config2.compactCriteria) {
-      const before = optimized;
-      optimized = this.compactCriteria(optimized);
-      if (optimized !== before) {
-        appliedRules.push("compactCriteria");
-      }
-    }
-    if (config2.deduplicateContext && input.contextMarkdown) {
-      const before = optimized;
-      optimized = this.deduplicateContext(optimized, input.contextMarkdown);
-      if (optimized !== before) {
-        appliedRules.push("deduplicateContext");
-      }
-    }
-    if (config2.maxPromptTokens) {
-      const currentTokens = estimateTokens2(optimized);
-      if (currentTokens > config2.maxPromptTokens) {
-        optimized = this.trimToMaxTokens(optimized, config2.maxPromptTokens);
-        appliedRules.push("maxPromptTokens");
-      }
-    }
-    const optimizedTokens = estimateTokens2(optimized);
-    const savings = originalTokens > 0 ? (originalTokens - optimizedTokens) / originalTokens : 0;
-    return {
-      prompt: optimized,
-      originalTokens,
-      optimizedTokens,
-      savings,
-      appliedRules
-    };
-  }
-  stripWhitespace(prompt) {
-    return prompt.split(`
-`).map((line) => line.trimEnd()).join(`
-`).replace(/\n{3,}/g, `
-
-`);
-  }
-  compactCriteria(prompt) {
-    return prompt.replace(/The user should be able to /gi, "").replace(/The system must /gi, "").replace(/The system should /gi, "").replace(/When the /gi, "").replace(/When a /gi, "").replace(/it should validate all fields/gi, "validate all fields").replace(/display an error message/gi, "show error").replace(/error message/gi, "error");
-  }
-  deduplicateContext(prompt, contextMarkdown) {
-    const contextSectionMatch = prompt.match(/# Context\n([\s\S]*?)(?=\n#|$)/i);
-    if (!contextSectionMatch) {
-      return prompt;
-    }
-    const contextSection = contextSectionMatch[1];
-    const contextLines = contextSection.split(`
-`);
-    const dedupedLines = contextLines.filter((line) => {
-      const trimmed = line.trim();
-      if (!trimmed)
-        return true;
-      return !contextMarkdown.includes(trimmed);
-    });
-    if (dedupedLines.length < contextLines.length) {
-      const newContextSection = dedupedLines.join(`
-`);
-      return prompt.replace(contextSectionMatch[0], `# Context
-${newContextSection}`);
-    }
-    return prompt;
-  }
-  trimToMaxTokens(prompt, maxTokens) {
-    const currentTokens = estimateTokens2(prompt);
-    if (currentTokens <= maxTokens) {
-      return prompt;
-    }
-    const sections = this.extractSections(prompt);
-    const targetChars = maxTokens * 4;
-    const trimmedMessage = `
-... (context trimmed)`;
-    let result = "";
-    let remainingChars = targetChars;
-    if (sections.task) {
-      result += sections.task;
-      remainingChars -= sections.task.length;
-    }
-    if (sections.acceptanceCriteria) {
-      result += sections.acceptanceCriteria;
-      remainingChars -= sections.acceptanceCriteria.length;
-    }
-    const trimmable = sections.context ?? sections.other;
-    if (trimmable && remainingChars > 0) {
-      const reserveForMessage = trimmable.length > remainingChars ? trimmedMessage.length : 0;
-      const maxContextChars = Math.max(0, remainingChars - reserveForMessage);
-      const trimmedContext = trimmable.substring(0, maxContextChars);
-      result += trimmedContext;
-      if (trimmedContext.length < trimmable.length) {
-        result += trimmedMessage;
-      }
-    }
-    return result.trim() ? result : prompt;
-  }
-  extractSections(prompt) {
-    const sections = {};
-    const taskMatch = prompt.match(/# Task\n([\s\S]*?)(?=\n#|$)/i);
-    if (taskMatch) {
-      sections.task = taskMatch[0];
-    }
-    const contextMatch = prompt.match(/# Context\n([\s\S]*?)(?=\n#|$)/i);
-    if (contextMatch) {
-      sections.context = contextMatch[0];
-    }
-    const acMatch = prompt.match(/# Acceptance Criteria\n([\s\S]*?)(?=\n#|$)/i);
-    if (acMatch) {
-      sections.acceptanceCriteria = acMatch[0];
-    }
-    let other = prompt;
-    if (sections.task) {
-      other = other.replace(sections.task, "");
-    }
-    if (sections.context) {
-      other = other.replace(sections.context, "");
-    }
-    if (sections.acceptanceCriteria) {
-      other = other.replace(sections.acceptanceCriteria, "");
-    }
-    if (other.trim()) {
-      sections.other = other;
-    }
-    return sections;
-  }
-}
-var DEFAULT_CONFIG2;
-var init_rule_based_optimizer = __esm(() => {
-  DEFAULT_CONFIG2 = {
-    stripWhitespace: true,
-    compactCriteria: true,
-    deduplicateContext: true,
-    maxPromptTokens: 8000
-  };
-});
-
 // src/optimizer/index.ts
 function resolveOptimizer(config2, pluginRegistry) {
   if (!config2.optimizer?.enabled) {
     return new NoopOptimizer;
   }
-  if (pluginRegistry) {
-    const pluginOptimizers = pluginRegistry.getOptimizers();
-    if (pluginOptimizers.length > 0) {
-      return pluginOptimizers[0];
-    }
-  }
-  const strategy = config2.optimizer.strategy ?? "noop";
-  switch (strategy) {
-    case "rule-based":
-      return new RuleBasedOptimizer;
-    case "noop":
-      return new NoopOptimizer;
-    default:
-      getSafeLogger()?.warn("optimizer", `Unknown optimizer strategy '${strategy}', using noop`);
-      return new NoopOptimizer;
+  const pluginOptimizers = pluginRegistry?.getOptimizers() ?? [];
+  if (pluginOptimizers.length > 0) {
+    return pluginOptimizers[0];
   }
+  return new NoopOptimizer;
 }
 var init_optimizer = __esm(() => {
   init_noop_optimizer();
-  init_rule_based_optimizer();
-  init_logger2();
   init_noop_optimizer();
-  init_rule_based_optimizer();
 });
 
 // src/context/fragments/store.ts
@@ -33625,24 +33483,23 @@ async function saveRunMetrics(outputDir, runMetrics) {
       cacheCreationInputTokens: totalCacheCreationInputTokens
     })
   } : runMetrics;
-  const allMetrics = await withPathFileLock(metricsPath, async () => {
+  await withPathFileLock(metricsPath, async () => {
     const existing = await loadJsonFile(metricsPath, "metrics");
-    const base = Array.isArray(existing) ? existing : [];
-    base.push(finalMetrics);
-    return base;
+    const allMetrics = Array.isArray(existing) ? existing : [];
+    allMetrics.push(finalMetrics);
+    const isTruncating = allMetrics.length > MAX_RETAINED_RUNS;
+    const cappedMetrics = isTruncating ? allMetrics.slice(allMetrics.length - MAX_RETAINED_RUNS) : allMetrics;
+    if (isTruncating && !hasWarnedAboutRunTruncation) {
+      hasWarnedAboutRunTruncation = true;
+      const droppedCount = allMetrics.length - MAX_RETAINED_RUNS;
+      getLogger().warn("metrics", "Run-history cap reached \u2014 oldest run-entries dropped from metrics.json; aggregate metrics now cover only the retained window", {
+        droppedCount,
+        maxRetainedRuns: MAX_RETAINED_RUNS,
+        metricsPath
+      });
+    }
+    await saveJsonFile(metricsPath, cappedMetrics, "metrics");
   });
-  const isTruncating = allMetrics.length > MAX_RETAINED_RUNS;
-  const cappedMetrics = isTruncating ? allMetrics.slice(allMetrics.length - MAX_RETAINED_RUNS) : allMetrics;
-  if (isTruncating && !hasWarnedAboutRunTruncation) {
-    hasWarnedAboutRunTruncation = true;
-    const droppedCount = allMetrics.length - MAX_RETAINED_RUNS;
-    getLogger().warn("metrics", "Run-history cap reached \u2014 oldest run-entries dropped from metrics.json; aggregate metrics now cover only the retained window", {
-      droppedCount,
-      maxRetainedRuns: MAX_RETAINED_RUNS,
-      metricsPath
-    });
-  }
-  await saveJsonFile(metricsPath, cappedMetrics, "metrics");
 }
 async function loadRunMetrics(outputDir) {
   const metricsPath = metricsPathFor(outputDir);
@@ -48370,7 +48227,35 @@ async function exists(workdir, rel) {
     return false;
   }
 }
-async function auditGaps(report, workdir) {
+function isNoise(rel) {
+  const segments = rel.split("/");
+  if (segments.includes(".nax"))
+    return true;
+  const name = segments[segments.length - 1];
+  if (name.endsWith(".lock") || name === "package-lock.json" || name === "pnpm-lock.yaml")
+    return true;
+  if (/(^|\/)(dist|build|\.next|\.turbo|__pycache__)(\/|$)/.test(rel))
+    return true;
+  if (/\.generated\./.test(name))
+    return true;
+  return false;
+}
+function walkPathToken(line) {
+  const token = line.trim().split(/\s+/)[0];
+  return token.length > 0 ? token : null;
+}
+async function changedFiles(workdir, range) {
+  try {
+    const { stdout, exitCode } = await gitWithTimeout(["diff", "--name-only", `${range.base}...${range.head}`], workdir);
+    if (exitCode !== 0)
+      return [];
+    return stdout.split(`
+`).map((s) => s.trim()).filter((s) => s.length > 0 && !isNoise(s));
+  } catch {
+    return [];
+  }
+}
+async function auditGaps(report, workdir, range, phase = "spec") {
   const gaps = [];
   const touchpoints = report.touchpoints ?? [];
   if (!report.sawTouchpointsSection || touchpoints.length === 0) {
@@ -48378,13 +48263,29 @@ async function auditGaps(report, workdir) {
   } else if (!touchpoints.some((t) => t.path === "none")) {
     const checked = touchpoints.slice(0, MAX_CHECKED);
     const found = await Promise.all(checked.map((t) => exists(workdir, t.path)));
-    if (!found.some(Boolean)) {
+    const foundCount = found.filter(Boolean).length;
+    if (foundCount * 2 <= checked.length) {
       gaps.push(`touchpoint path does not exist in the repo (checked: ${checked.map((t) => t.path).join(", ")}) \u2014 list files you actually opened`);
     }
   }
   if (!report.sawWalkSection || (report.walk ?? []).length === 0) {
     gaps.push("no `## WALK` section: the per-AC (spec) or per-function (quality) enumeration is required");
   }
+  if (phase === "quality" && range) {
+    const required2 = await changedFiles(workdir, range);
+    if (required2.length > 0) {
+      const walked = new Set;
+      for (const line of report.walk ?? []) {
+        const token = walkPathToken(line);
+        if (token && required2.includes(token))
+          walked.add(token);
+      }
+      const unwalked = required2.filter((file3) => !walked.has(file3));
+      if (unwalked.length > 0) {
+        gaps.push(`unwalked changed files not named in \`## WALK\` (${unwalked.length}): ${unwalked.join(", ")} \u2014 walk every changed file`);
+      }
+    }
+  }
   return gaps;
 }
 async function validateDispositions(workdir, dispositions) {
@@ -48396,7 +48297,9 @@ async function validateDispositions(workdir, dispositions) {
   }));
 }
 var MAX_CHECKED = 20;
-var init_audit_gaps = () => {};
+var init_audit_gaps = __esm(() => {
+  init_git();
+});
 
 // src/finish/review/parse.ts
 function parseTouchpoint(text) {
@@ -48429,13 +48332,15 @@ function parseReviewReport(text) {
   let section = "findings";
   let current = null;
   let lastField = null;
+  const normalized = text.replace(GLUED_HEADING, `$1
+$2`);
   const flush = () => {
     if (current)
       report.findings.push(current);
     current = null;
     lastField = null;
   };
-  for (const line of text.split(`
+  for (const line of normalized.split(`
 `)) {
     const heading = HEADING.exec(line);
     if (heading) {
@@ -48514,9 +48419,10 @@ function parseDispositions(text) {
   }
   return out;
 }
-var HEADING, BLOCK, FIELD, NO_FINDINGS, BULLET, DISPOSITION, EVIDENCE;
+var HEADING, GLUED_HEADING, BLOCK, FIELD, NO_FINDINGS, BULLET, DISPOSITION, EVIDENCE;
 var init_parse5 = __esm(() => {
   HEADING = /^\s*#{1,6}\s*(TOUCHPOINTS|WALK|FINDINGS|DISPOSITIONS)\s*:?\s*$/i;
+  GLUED_HEADING = /([^\s#])(#{1,6}[ \t]*(?:TOUCHPOINTS|WALK|FINDINGS|DISPOSITIONS)[ \t]*:?)(?=[ \t]*(?:\r?\n|$))/gi;
   BLOCK = /^\s*\[(CRITICAL|HIGH|MEDIUM|LOW)\]\s+(.+?)\s*$/;
   FIELD = /^\s*(Problem|Fix|Judgment)\s*:\s*(.*)$/i;
   NO_FINDINGS = /^\s*no findings\.?\s*$/i;
@@ -48678,12 +48584,20 @@ test code \u2014 and judge each changed function on its own merits.
 
 ## Forcing function \u2014 enumerate before you conclude
 
-Before reporting, walk **every function/method the diff adds or changes** and
-write yourself a one-line verdict for each: *earns its place* or *concern: \u2026*.
-This enumeration is a thinking tool \u2014 it does not go in the final report, only
-the resulting findings do. Skipping it is how real maintainability issues get
-missed: an agent that pattern-matches a few obvious smells and stops will always
-under-report. Look at each changed function deliberately.
+Before reporting, walk **every function/method the diff adds or changes \u2014 in
+test files as well as production ones** \u2014 and write yourself a one-line verdict
+for each: *earns its place* or *concern: \u2026*. Skipping it is how real
+maintainability issues get missed: an agent that pattern-matches a few obvious
+smells and stops will always under-report. Look at each changed function
+deliberately.
+
+This per-function walk is **private scratch work** \u2014 it stays in your reasoning,
+you do not transcribe it into the reply. What you emit, in the \`## WALK\` section
+the reply contract requires, is **one line per changed file**, carrying that
+file's verdict (\`path \u2014 earns its place|concern: <one clause>\`). The
+per-function walk is what finds the defects; the per-file WALK is the
+checkable evidence that you walked the diff. A reply without the \`## WALK\`
+section is treated as an incomplete review and sent back.
 
 ## What to look for
 
@@ -48874,7 +48788,7 @@ reads to judge an integration-shaped defect.)
 
 // src/finish/review/prompt.ts
 function outputContract(phase) {
-  const walk = phase === "spec" ? "one line per AC in the spec: `AC-3 Covered|Partial|Missing \u2014 <one clause>`" : "one line per function or method the diff adds or changes: `path.ts:name \u2014 earns its place|concern: <one clause>`";
+  const walk = phase === "spec" ? "one line per AC in the spec: `AC-3 Covered|Partial|Missing \u2014 <one clause>`" : "one line per file the diff adds or changes: `path.ts \u2014 earns its place|concern: <one clause>`";
   return `# Reply contract \u2014 your reply must be these three sections, in this order
 
 ## TOUCHPOINTS
@@ -48908,10 +48822,11 @@ function buildReviewPrompt(phase, args) {
 `)
   ] : [];
   if (!args.since) {
+    const specNotice = phase === "spec" && args.specPath ? [`The spec/requirements source is: ${args.specPath}. Read it in full.`] : [];
     return [
       ...gapNotice,
       `You are the ${phase.toUpperCase()} reviewer for a completed feature.`,
-      `The spec/requirements source is: ${args.specPath}. Read it in full.`,
+      ...specNotice,
       `Fetch and review the diff: \`git diff ${args.base}...HEAD\` (also \`--name-only\` for the file list).`,
       WORKER_PROTOCOL_MECHANICS,
       dims,
@@ -48921,8 +48836,7 @@ function buildReviewPrompt(phase, args) {
 
 `);
   }
-  return [
-    ...gapNotice,
+  const reReviewBody = [
     `You are the ${phase.toUpperCase()} reviewer for a completed feature, continuing a review you already started.`,
     `On your previous pass over \`git diff ${args.base}...HEAD\` you raised the findings below, and they have since been fixed and committed. Everything else in that diff you already judged acceptable \u2014 do not re-derive a verdict on it.`,
     `Your findings from the previous pass:
@@ -48932,14 +48846,19 @@ ${JSON.stringify(args.priorFindings ?? [], null, 2)}`,
       "1. **Resolved?** Does the fix actually resolve each finding above? A finding that was papered over (assertion weakened, test deleted, check disabled) is NOT resolved \u2014 re-raise it.",
       "2. **Broken?** Did the fix introduce a new problem, in the changed lines or in the unchanged code they now call into?",
       "",
-      `Read whatever files you need \u2014 the spec is at ${args.specPath} and the whole repo is available. Scope means *what you judge*, not *what you may read*.`
+      ...phase === "spec" && args.specPath ? [
+        `Read whatever files you need \u2014 the spec is at ${args.specPath} and the whole repo is available. Scope means *what you judge*, not *what you may read*.`
+      ] : [
+        "Read whatever files you need \u2014 the whole repo is available. Scope means *what you judge*, not *what you may read*."
+      ]
     ].join(`
 `),
     WORKER_PROTOCOL_MECHANICS,
     dims,
     CLASSIFIER,
     outputContract(phase)
-  ].join(`
+  ];
+  return [...gapNotice, ...reReviewBody].join(`
 
 `);
 }
@@ -49087,7 +49006,10 @@ var init_finish_review = __esm(() => {
       exhaustedFallback: () => EMPTY_REVIEW_REPORT
     }),
     async verify(parsed, input, _verifyCtx) {
-      return { ...parsed, gaps: await auditGaps(parsed, input.workdir) };
+      return {
+        ...parsed,
+        gaps: await auditGaps(parsed, input.workdir, { base: input.base, head: "HEAD" }, input.phase)
+      };
     }
   };
 });
@@ -50719,7 +50641,7 @@ var package_default;
 var init_package = __esm(() => {
   package_default = {
     name: "@nathapp/nax",
-    version: "0.81.0-canary.1",
+    version: "0.81.0",
     description: "AI Coding Agent Orchestrator \u2014 loops until done",
     type: "module",
     bin: {
@@ -50761,6 +50683,8 @@ var init_package = __esm(() => {
       "test:e2e": "timeout -k 5s 180s bun test test/e2e/ --timeout=60000",
       "test:coverage": "bun run scripts/check-coverage.ts",
       "test:coverage:report": "bun run scripts/check-coverage.ts --report",
+      "test:coverage:update": "bun run scripts/check-coverage.ts --update-baseline",
+      "test:coverage:list": "bun run scripts/check-coverage.ts --list",
       "report:test-overlap": "bun run scripts/report-test-overlap.ts",
       "report:dead-tests": "bun run scripts/report-dead-tests.ts",
       "check:test-mocks": "bun scripts/check-inline-test-mocks.ts --strict",
@@ -50834,8 +50758,8 @@ var init_version = __esm(() => {
   NAX_VERSION = package_default.version;
   NAX_COMMIT = (() => {
     try {
-      if (/^[0-9a-f]{6,10}$/.test("ac91a9a7"))
-        return "ac91a9a7";
+      if (/^[0-9a-f]{6,10}$/.test("a2cc47be"))
+        return "a2cc47be";
     } catch {}
     try {
       const result = Bun.spawnSync(["git", "rev-parse", "--short", "HEAD"], {
@@ -53582,10 +53506,10 @@ function formatDuration2(ms) {
   return `${minutes}m ${seconds}s`;
 }
 function timeoutRetry(input) {
-  const { prompt, changedFiles, elapsedMs, attempt } = input;
+  const { prompt, changedFiles: changedFiles2, elapsedMs, attempt } = input;
   const duration3 = formatDuration2(elapsedMs);
   const attemptNumber = attempt + 1;
-  if (changedFiles.length === 0) {
+  if (changedFiles2.length === 0) {
     return `The previous attempt hit a timeout after ${elapsedMs}ms (${duration3}) with no file changes on disk.
 This is attempt ${attemptNumber} of the same story \u2014 the previous attempt left nothing behind, so the approach was wrong.
 Change your approach: pick a narrower scope, fewer file edits, or a different angle on the acceptance criteria.
@@ -53594,7 +53518,7 @@ Change your approach: pick a narrower scope, fewer file edits, or a different an
 
 ${prompt}`;
   }
-  const fileList = changedFiles.map((p) => `- ${p}`).join(`
+  const fileList = changedFiles2.map((p) => `- ${p}`).join(`
 `);
   return `The previous attempt hit a timeout after ${elapsedMs}ms (${duration3}), but left these files on disk:
 
@@ -54606,11 +54530,11 @@ function buildHopCallback(ctx, sessionId, _initialOptions) {
     }
     if (hopKind.kind === "timeout-retry") {
       const preAttemptGitRef = preAttemptGitRefPromise ? await preAttemptGitRefPromise : undefined;
-      const changedFiles = preAttemptGitRef ? await _buildHopCallbackDeps.captureWorkingTreeChanges(workdir, preAttemptGitRef) : [];
+      const changedFiles2 = preAttemptGitRef ? await _buildHopCallbackDeps.captureWorkingTreeChanges(workdir, preAttemptGitRef) : [];
       const elapsedMs = elapsedSincePriorHop;
       prompt = _buildHopCallbackDeps.timeoutRetry({
         prompt: resolvedRunOptions.prompt,
-        changedFiles,
+        changedFiles: changedFiles2,
         elapsedMs,
         attempt: hopKind.attempt
       });
@@ -62252,7 +62176,7 @@ async function checkGitignoreCoversNax(workdir) {
     "**/.nax-acceptance*",
     "**/_nax_acceptance_test.py",
     "**/_nax_suggested_test.py",
-    `**/${PROJECT_FEATURES_DIR}/*/`
+    `${PROJECT_FEATURES_DIR}/*/fragments/`
   ];
   const missing = patterns.filter((pattern) => !content.includes(pattern));
   const passed = missing.length === 0;
@@ -65650,8 +65574,8 @@ var init_completion = __esm(() => {
         }
         if (fragmentsEnabled) {
           try {
-            const changedFiles = [...await _completionDeps.getDiffFilePaths(ctx.workdir, ctx.storyGitRef)];
-            const body = _completionDeps.renderFragmentBody(ctx.story.id, ctx.story.title, ctx.story.acceptanceCriteria, changedFiles);
+            const changedFiles2 = [...await _completionDeps.getDiffFilePaths(ctx.workdir, ctx.storyGitRef)];
+            const body = _completionDeps.renderFragmentBody(ctx.story.id, ctx.story.title, ctx.story.acceptanceCriteria, changedFiles2);
             const maxTokens = ctx.config.context.v2.fragments.maxTokens;
             await _completionDeps.writeFragment(ctx.projectDir, featureId, ctx.story.id, body, maxTokens);
           } catch (err) {
@@ -66046,7 +65970,7 @@ async function acquireLock(workdir) {
         }
         const tombstonePath = `${lockPath}.stale.${process.pid}.${Date.now()}`;
         try {
-          await rename2(lockPath, tombstonePath);
+          await _lockDeps.rename(lockPath, tombstonePath);
         } catch (renameError) {
           if (renameError.code === "ENOENT") {
             return false;
@@ -66111,8 +66035,12 @@ async function releaseLock(workdir) {
     }
   }
 }
+var _lockDeps;
 var init_lock = __esm(() => {
   init_logger2();
+  _lockDeps = {
+    rename: rename2
+  };
 });
 
 // src/execution/helpers.ts
@@ -70664,11 +70592,11 @@ async function regenerateAcceptanceTest(testPath, acceptanceContext) {
       const repoRoot = acceptanceContext.projectDir ?? workdir;
       const packageDir = acceptanceContext.story.workdir && acceptanceContext.projectDir ? path15.join(acceptanceContext.projectDir, acceptanceContext.story.workdir) : undefined;
       const ignoreMatchers = acceptanceContext.naxIgnoreIndex?.getMatchers(packageDir) ?? await resolveNaxIgnorePatterns(repoRoot, packageDir);
-      const changedFiles = filterNaxInternalPaths(changedFilesRaw, ignoreMatchers);
+      const changedFiles2 = filterNaxInternalPaths(changedFilesRaw, ignoreMatchers);
       const MAX_BYTES = 51200;
       let totalBytes = 0;
       const entries = [];
-      for (const file3 of changedFiles) {
+      for (const file3 of changedFiles2) {
         if (totalBytes >= MAX_BYTES)
           break;
         const filePath = path15.join(workdir, file3);
@@ -75017,12 +74945,12 @@ async function runDeferredReview(workdir, _reviewConfig, plugins, runStartRef, n
   }
   const changedFilesRaw = await getChangedFilesForDeferred(workdir, runStartRef);
   const ignoreMatchers = naxIgnoreIndex?.getMatchers() ?? await resolveNaxIgnorePatterns(workdir);
-  const changedFiles = filterNaxInternalPaths(changedFilesRaw, ignoreMatchers);
+  const changedFiles2 = filterNaxInternalPaths(changedFilesRaw, ignoreMatchers);
   const reviewerResults = [];
   let anyFailed = false;
   for (const reviewer of reviewers) {
     try {
-      const result = await reviewer.check(workdir, changedFiles);
+      const result = await reviewer.check(workdir, changedFiles2);
       reviewerResults.push({
         name: reviewer.name,
         passed: result.passed,
@@ -75043,7 +74971,7 @@ async function runDeferredReview(workdir, _reviewConfig, plugins, runStartRef, n
       anyFailed = true;
     }
   }
-  return { runStartRef, changedFiles, reviewerResults, anyFailed };
+  return { runStartRef, changedFiles: changedFiles2, reviewerResults, anyFailed };
 }
 var _deferredReviewDeps;
 var init_deferred_review = __esm(() => {
@@ -75682,7 +75610,7 @@ ${missing.join(`
 `);
   return { created: false, added: [...missing] };
 }
-var NAX_GITIGNORE_ENTRIES, NAX_NAXIGNORE_ENTRIES, NAX_NAXIGNORE_HEADER = `# nax - paths excluded from context, review
+var FEATURE_RUN_ARTIFACTS, NAX_GITIGNORE_ENTRIES, NAX_NAXIGNORE_ENTRIES, NAX_NAXIGNORE_HEADER = `# nax - paths excluded from context, review
 # and verification scanning.
 # gitignore syntax. Also honoured per-package.
 
@@ -75695,24 +75623,32 @@ var NAX_GITIGNORE_ENTRIES, NAX_NAXIGNORE_ENTRIES, NAX_NAXIGNORE_HEADER = `# nax
 `;
 var init_gitignore = __esm(() => {
   init_config();
+  FEATURE_RUN_ARTIFACTS = [
+    "runs/",
+    "plan/",
+    "sessions/",
+    "stories/",
+    "fragments/",
+    "interactions/",
+    "semantic-verdicts/",
+    "status.json",
+    "checkpoint.jsonl",
+    "progress.txt",
+    "acp-sessions.json",
+    "acceptance-refined.json",
+    "*.bak"
+  ].map((artifact) => `**/${PROJECT_FEATURES_DIR}/*/${artifact}`);
   NAX_GITIGNORE_ENTRIES = [
     ".nax-verifier-verdict.json",
     "nax.lock",
     ".nax/**/runs/",
     ".nax/metrics.json",
-    `${PROJECT_FEATURES_DIR}/*/status.json`,
-    `${PROJECT_FEATURES_DIR}/*/plan/`,
-    `${PROJECT_FEATURES_DIR}/*/fragments/`,
-    `${PROJECT_FEATURES_DIR}/*/acp-sessions.json`,
-    `${PROJECT_FEATURES_DIR}/*/interactions/`,
-    `${PROJECT_FEATURES_DIR}/*/progress.txt`,
-    `${PROJECT_FEATURES_DIR}/*/acceptance-refined.json`,
+    ...FEATURE_RUN_ARTIFACTS,
     ".nax-pids",
     ".nax-wt/",
     "**/.nax-acceptance*",
     "**/_nax_acceptance_test.py",
     "**/_nax_suggested_test.py",
-    `**/${PROJECT_FEATURES_DIR}/*/`,
     ".nax/prompt-audit/",
     ".nax/finish-audit/",
     ".nax/mutation-journal/"
@@ -83694,6 +83630,7 @@ __export(exports_execution, {
   _postRunDeps: () => _postRunDeps,
   _pidRegistryDeps: () => _pidRegistryDeps,
   _newPackageSetupDeps: () => _newPackageSetupDeps,
+  _lockDeps: () => _lockDeps,
   StoryOrchestratorBuilder: () => StoryOrchestratorBuilder,
   StatusWriter: () => StatusWriter,
   STRICT_VERDICT_PHASE_NAMES: () => STRICT_VERDICT_PHASE_NAMES,
@@ -85115,8 +85052,7 @@ var init_config_descriptions = __esm(() => {
     "context.autoDetect.maxFiles": "Max files to auto-detect",
     "context.autoDetect.traceImports": "Trace imports to find related files",
     optimizer: "Prompt optimizer configuration",
-    "optimizer.enabled": "Enable prompt optimizer",
-    "optimizer.strategy": "Optimization strategy: rule-based | llm | noop",
+    "optimizer.enabled": "Enable prompt optimizer (pass-through unless a plugin provides one)",
     plugins: "Plugin configurations",
     hooks: "Hooks configuration",
     "hooks.skipGlobal": "Skip loading global hooks",
@@ -117545,6 +117481,7 @@ import { join as join106 } from "path";
 // src/commands/logs-formatter.ts
 init_source();
 init_formatter();
+init_bun_deps();
 import { readdirSync as readdirSync8 } from "fs";
 import { join as join105 } from "path";
 
@@ -117731,13 +117668,57 @@ async function displayLogs(filePath, options) {
     }
   }
 }
-async function followLogs(filePath, options) {
+async function defaultReadRange(filePath, start) {
+  return Bun.file(filePath).slice(start).text();
+}
+async function defaultSize(filePath) {
+  return (await Bun.file(filePath).stat()).size;
+}
+var DEFAULT_FOLLOW_LOGS_DEPS = {
+  emit: (line) => console.log(line),
+  sleep: (ms, signal) => cancellableDelay(ms, signal),
+  readRange: defaultReadRange,
+  size: defaultSize
+};
+async function followLogs(filePath, options, opts) {
+  const deps = { ...DEFAULT_FOLLOW_LOGS_DEPS, ...opts?._deps };
+  const signal = opts?.signal;
   const mode = options.json ? "json" : "normal";
-  const file3 = Bun.file(filePath);
-  const content = await file3.text();
-  const lines = content.trim().split(`
+  if (signal?.aborted)
+    return "cancelled";
+  let lastOffset = await consumeRange(filePath, 0, deps, options, mode);
+  while (true) {
+    if (signal?.aborted)
+      return "cancelled";
+    try {
+      await deps.sleep(500, signal);
+    } catch (err) {
+      if (signal?.aborted)
+        return "cancelled";
+      throw err;
+    }
+    let currentSize;
+    try {
+      currentSize = await deps.size(filePath);
+    } catch {
+      return "cancelled";
+    }
+    if (currentSize > lastOffset) {
+      lastOffset = await consumeRange(filePath, lastOffset, deps, options, mode);
+    } else if (currentSize < lastOffset) {
+      lastOffset = currentSize;
+    }
+  }
+}
+async function consumeRange(filePath, start, deps, options, mode) {
+  const chunk = await deps.readRange(filePath, start);
+  if (!chunk)
+    return start;
+  const lastNewline = chunk.lastIndexOf(`
 `);
-  for (const line of lines) {
+  const complete = lastNewline === -1 ? "" : chunk.slice(0, lastNewline + 1);
+  for (const line of complete.split(`
+`)) {
     if (!line.trim())
       continue;
     try {
@@ -117747,36 +117728,11 @@ async function followLogs(filePath, options) {
       }
       const formatted = formatLogEntry(entry, { mode, useColor: true });
       if (formatted.shouldDisplay && formatted.output) {
-        console.log(formatted.output);
+        deps.emit(formatted.output);
       }
     } catch {}
   }
-  let lastSize = (await Bun.file(filePath).stat()).size;
-  while (true) {
-    await Bun.sleep(500);
-    const currentSize = (await Bun.file(filePath).stat()).size;
-    if (currentSize > lastSize) {
-      const newFile = Bun.file(filePath);
-      const newContent = await newFile.text();
-      const newLines = newContent.slice(lastSize).trim().split(`
-`);
-      for (const line of newLines) {
-        if (!line.trim())
-          continue;
-        try {
-          const entry = JSON.parse(line);
-          if (!shouldDisplayEntry(entry, options)) {
-            continue;
-          }
-          const formatted = formatLogEntry(entry, { mode, useColor: true });
-          if (formatted.shouldDisplay && formatted.output) {
-            console.log(formatted.output);
-          }
-        } catch {}
-      }
-      lastSize = currentSize;
-    }
-  }
+  return start + Buffer.byteLength(complete, "utf8");
 }
 function shouldDisplayEntry(entry, options) {
   if (options.story && entry.storyId !== options.story) {
@@ -117800,7 +117756,7 @@ async function logsCommand(options) {
       return;
     }
     if (options.follow) {
-      await followLogs(runFile2, options);
+      await followLogs(runFile2, options, { signal: options.signal });
     } else {
       await displayLogs(runFile2, options);
     }
@@ -117823,7 +117779,7 @@ async function logsCommand(options) {
     throw new Error("No runs found for this feature");
   }
   if (options.follow) {
-    await followLogs(runFile, options);
+    await followLogs(runFile, options, { signal: options.signal });
     return;
   }
   await displayLogs(runFile, options);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nathapp/nax",
-  "version": "0.81.0-canary.1",
+  "version": "0.81.0",
   "description": "AI Coding Agent Orchestrator — loops until done",
   "type": "module",
   "bin": {
@@ -42,6 +42,8 @@
     "test:e2e": "timeout -k 5s 180s bun test test/e2e/ --timeout=60000",
     "test:coverage": "bun run scripts/check-coverage.ts",
     "test:coverage:report": "bun run scripts/check-coverage.ts --report",
+    "test:coverage:update": "bun run scripts/check-coverage.ts --update-baseline",
+    "test:coverage:list": "bun run scripts/check-coverage.ts --list",
     "report:test-overlap": "bun run scripts/report-test-overlap.ts",
     "report:dead-tests": "bun run scripts/report-dead-tests.ts",
     "check:test-mocks": "bun scripts/check-inline-test-mocks.ts --strict",