@nathapp/nax 0.61.2-canary.1 → 0.61.2

package/dist/nax.js

@@ -2634,17 +2634,6 @@ function formatDuration(durationMs) {
 function formatCost(cost) {
   return `$${cost.toFixed(4)}`;
 }
-function getStageEmoji(stage) {
-  if (stage.includes("routing"))
-    return EMOJI.routing;
-  if (stage.includes("execution") || stage.includes("agent"))
-    return EMOJI.execution;
-  if (stage.includes("review"))
-    return EMOJI.review;
-  if (stage.includes("tdd"))
-    return EMOJI.tdd;
-  return EMOJI.info;
-}
 function shouldDisplay(entry, mode) {
   if (mode === "json")
     return true;
@@ -2685,7 +2674,7 @@ function formatLogEntry(entry, options) {
   }
   return formatDefault(entry, colorize, timestamp, mode);
 }
-function formatRunStart(entry, c, timestamp, mode) {
+function formatRunStart(entry, c, timestamp, _mode) {
   const data = entry.data;
   const lines = [];
   lines.push("");
@@ -2704,7 +2693,7 @@ function formatRunStart(entry, c, timestamp, mode) {
     shouldDisplay: true
   };
 }
-function formatStoryStart(entry, c, timestamp, mode) {
+function formatStoryStart(entry, c, _timestamp, mode) {
   const data = entry.data;
   const storyId = String(data.storyId || entry.storyId || "unknown");
   const title = String(data.storyTitle || data.title || "Untitled story");
@@ -2734,7 +2723,7 @@ function formatStoryStart(entry, c, timestamp, mode) {
     shouldDisplay: true
   };
 }
-function formatStoryComplete(entry, c, timestamp, mode) {
+function formatStoryComplete(entry, c, _timestamp, mode) {
   const data = entry.data;
   const storyId = String(data.storyId || entry.storyId || "unknown");
   const success = data.success ?? true;
@@ -2766,7 +2755,7 @@ function formatStoryComplete(entry, c, timestamp, mode) {
     shouldDisplay: true
   };
 }
-function formatTDDSession(entry, c, timestamp, mode) {
+function formatTDDSession(entry, c, _timestamp, mode) {
   if (mode === "quiet") {
     return { output: "", shouldDisplay: false };
   }
@@ -2781,7 +2770,6 @@ function formatTDDSession(entry, c, timestamp, mode) {
 function formatDefault(entry, c, timestamp, mode) {
   const levelEmoji = entry.level === "error" ? EMOJI.failure : entry.level === "warn" ? EMOJI.warning : EMOJI.info;
   const levelColor = entry.level === "error" ? c.red : entry.level === "warn" ? c.yellow : c.gray;
-  const stageEmoji = getStageEmoji(entry.stage);
   const parts = [c.gray(`[${timestamp}]`), levelColor(`${levelEmoji} ${entry.stage}`)];
   if (entry.storyId) {
     parts.push(c.dim(`[${entry.storyId}]`));
@@ -3626,7 +3614,14 @@ var init_env = __esm(() => {
 
 // src/agents/acp/parser.ts
 function createParseState() {
-  return { text: "", tokenUsage: undefined, exactCostUsd: undefined, stopReason: undefined, error: undefined };
+  return {
+    text: "",
+    tokenUsage: undefined,
+    exactCostUsd: undefined,
+    stopReason: undefined,
+    error: undefined,
+    retryable: false
+  };
 }
 function parseAcpxJsonLine(line, state) {
   try {
@@ -3665,6 +3660,8 @@ function parseAcpxJsonLine(line, state) {
           const suffix = [data.acpxCode, data.detailCode].filter(Boolean).join("/");
           if (suffix)
             errorMsg = `${errorMsg} [${suffix}]`;
+          if (!state.error && data.retryable === true)
+            state.retryable = true;
         }
         if (!state.error)
           state.error = errorMsg;
@@ -3703,7 +3700,8 @@ function finalizeParseState(state) {
     tokenUsage: state.tokenUsage,
     exactCostUsd: state.exactCostUsd,
     stopReason: state.stopReason,
-    error: state.error
+    error: state.error,
+    retryable: state.retryable
   };
 }
 
@@ -3828,7 +3826,8 @@ class SpawnAcpSession {
         });
         return {
           messages: [{ role: "assistant", content: errorContent }],
-          stopReason: "error"
+          stopReason: "error",
+          retryable: parsedOnError.retryable
         };
       }
       try {
@@ -3904,7 +3903,6 @@ class SpawnAcpSession {
 }
 
 class SpawnAcpClient {
-  agentName;
   model;
   cwd;
   timeoutSeconds;
@@ -3918,7 +3916,6 @@ class SpawnAcpClient {
     if (!lastToken || lastToken.startsWith("-")) {
       throw new Error(`[acp-adapter] Could not parse agentName from cmdStr: "${cmdStr}"`);
     }
-    this.agentName = lastToken;
     this.cwd = cwd || process.cwd();
     this.timeoutSeconds = timeoutSeconds || 1800;
     this.env = buildAllowedEnv();
@@ -4132,17 +4129,6 @@ function estimateCostByDuration(modelTier, durationMs) {
     confidence: "fallback"
   };
 }
-function formatCostWithConfidence(estimate) {
-  const formattedCost = `$${estimate.cost.toFixed(2)}`;
-  switch (estimate.confidence) {
-    case "exact":
-      return formattedCost;
-    case "estimated":
-      return `~${formattedCost}`;
-    case "fallback":
-      return `~${formattedCost} (duration-based)`;
-  }
-}
 function estimateCostFromTokenUsage(usage, model) {
   const pricing = MODEL_PRICING[model];
   if (!pricing) {
@@ -18177,6 +18163,8 @@ var init_schemas3 = __esm(() => {
     iterationDelayMs: exports_external.number().int().nonnegative(),
     costLimit: exports_external.number().positive({ message: "costLimit must be > 0" }),
     sessionTimeoutSeconds: exports_external.number().int().positive({ message: "sessionTimeoutSeconds must be > 0" }).default(3600),
+    sessionErrorMaxRetries: exports_external.number().int().min(0).max(5).default(1),
+    sessionErrorRetryableMaxRetries: exports_external.number().int().min(0).max(10).default(3),
     verificationTimeoutSeconds: exports_external.number().int().min(1).max(3600).default(300),
     maxStoriesPerFeature: exports_external.number().int().positive(),
     rectification: RectificationConfigSchema,
@@ -19095,7 +19083,9 @@ class AcpAgentAdapter {
     });
     let currentAgent = this.resolveCurrentAgent(config2);
     const rateLimitedRetryAfter = new Map;
-    let sessionErrorRetried = false;
+    let sessionErrorRetries = 0;
+    const SESSION_ERROR_MAX_RETRIES = config2?.execution?.sessionErrorMaxRetries ?? 1;
+    const SESSION_ERROR_RETRYABLE_MAX_RETRIES = config2?.execution?.sessionErrorRetryableMaxRetries ?? 3;
     let legacyAttempt = 0;
     let retryCount = 0;
     while (true) {
@@ -19106,11 +19096,15 @@ class AcpAgentAdapter {
             exitCode: result.exitCode,
             ...result.output ? { output: result.output.slice(0, 500) } : {}
           });
-          if (result.sessionError && _acpAdapterDeps.shouldRetrySessionError && !sessionErrorRetried) {
-            sessionErrorRetried = true;
+          const maxSessionRetries = result.sessionErrorRetryable ? SESSION_ERROR_RETRYABLE_MAX_RETRIES : SESSION_ERROR_MAX_RETRIES;
+          if (result.sessionError && _acpAdapterDeps.shouldRetrySessionError && sessionErrorRetries < maxSessionRetries) {
+            sessionErrorRetries += 1;
             getSafeLogger()?.warn("acp-adapter", "Session error \u2014 retrying with fresh session", {
               storyId: options.storyId,
-              featureName: options.featureName
+              featureName: options.featureName,
+              retryable: result.sessionErrorRetryable,
+              attempt: sessionErrorRetries,
+              maxAttempts: maxSessionRetries
             });
             if (options.featureName && options.storyId) {
               await clearAcpSession(options.workdir, options.featureName, options.storyId, options.sessionRole);
@@ -19329,6 +19323,7 @@ class AcpAgentAdapter {
     }
     const success2 = lastResponse?.stopReason === "end_turn";
     const isSessionError = lastResponse?.stopReason === "error";
+    const isSessionErrorRetryable = isSessionError && lastResponse?.retryable === true;
     const output = extractOutput(lastResponse);
     const estimatedCost = totalExactCostUsd ?? (totalTokenUsage.input_tokens > 0 || totalTokenUsage.output_tokens > 0 ? estimateCostFromTokenUsage(totalTokenUsage, options.modelDef.model) : 0);
     const tokenUsage = totalTokenUsage.input_tokens > 0 || totalTokenUsage.output_tokens > 0 ? {
@@ -19347,6 +19342,7 @@ class AcpAgentAdapter {
       output: output.slice(-MAX_AGENT_OUTPUT_CHARS),
       rateLimited: false,
       sessionError: isSessionError,
+      sessionErrorRetryable: isSessionErrorRetryable,
       durationMs,
       estimatedCost,
       tokenUsage
@@ -23020,7 +23016,7 @@ class AutoInteractionPlugin {
     };
   }
   async destroy() {}
-  async send(request) {}
+  async send(_request) {}
   async receive(_requestId, _timeout = 60000) {
     throw new Error("Auto plugin requires full request context (not just requestId)");
   }
@@ -24518,7 +24514,7 @@ async function checkHomeEnvValid() {
     message: passed ? `HOME env is valid: ${home}` : home === "" ? "HOME env is not set \u2014 agent may write files to unexpected locations" : `HOME env is not an absolute path ("${home}") \u2014 may cause literal "~" directories in repo`
   };
 }
-async function checkLanguageTools(profile, workdir) {
+async function checkLanguageTools(profile, _workdir) {
   if (!profile || !profile.language) {
     return {
       name: "language-tools-available",
@@ -25247,7 +25243,7 @@ function countStories(prd) {
     decomposed: prd.userStories.filter((s) => s.status === "decomposed").length
   };
 }
-function markStoryPassed(prd, storyId, statusWriter) {
+function markStoryPassed(prd, storyId, _statusWriter) {
   const story = prd.userStories.find((s) => s.id === storyId);
   if (story) {
     story.passes = true;
@@ -26180,7 +26176,7 @@ function parseAcceptanceCriteria(specContent) {
   }
   return criteria;
 }
-function buildAcceptanceTestPrompt(criteria, featureName, codebaseContext, testPathConfig, language) {
+function buildAcceptanceTestPrompt(criteria, featureName, _codebaseContext, testPathConfig, language) {
   const criteriaList = criteria.map((ac) => `${ac.id}: ${ac.text}`).join(`
 `);
   const resolvedTestPath = resolveAcceptanceTestFile2(language, testPathConfig);
@@ -26329,7 +26325,7 @@ ${tests || "  // No acceptance criteria found"}
 });
 `;
 }
-function generateGoSkeletonTests(featureName, criteria) {
+function generateGoSkeletonTests(_featureName, criteria) {
   const sanitize = (text) => text.replace(/[^a-zA-Z0-9 ]/g, "").split(" ").map((w) => w.charAt(0).toUpperCase() + w.slice(1)).join("");
   const tests = criteria.map((ac) => {
     const funcName = `Test${sanitize(ac.text) || ac.id.replace("-", "")}`;
@@ -27514,40 +27510,8 @@ var init_claude = __esm(() => {
 function validateAgentForTier(agent, tier) {
   return agent.capabilities.supportedTiers.includes(tier);
 }
-function validateAgentFeature(agent, feature) {
-  return agent.capabilities.features.has(feature);
-}
-function describeAgentCapabilities(agent) {
-  const tiers = agent.capabilities.supportedTiers.join(",");
-  const features = Array.from(agent.capabilities.features).join(",");
-  const maxTokens = agent.capabilities.maxContextTokens;
-  return `${agent.name}: tiers=[${tiers}], maxTokens=${maxTokens}, features=[${features}]`;
-}
 
 // src/agents/index.ts
-var exports_agents = {};
-__export(exports_agents, {
-  validateAgentForTier: () => validateAgentForTier,
-  validateAgentFeature: () => validateAgentFeature,
-  parseTokenUsage: () => parseTokenUsage,
-  getInstalledAgents: () => getInstalledAgents,
-  getAllAgentNames: () => getAllAgentNames,
-  getAgentVersions: () => getAgentVersions,
-  getAgentVersion: () => getAgentVersion,
-  getAgent: () => getAgent,
-  formatCostWithConfidence: () => formatCostWithConfidence,
-  estimateCostFromTokenUsage: () => estimateCostFromTokenUsage,
-  estimateCostFromOutput: () => estimateCostFromOutput,
-  estimateCostByDuration: () => estimateCostByDuration,
-  estimateCost: () => estimateCost,
-  describeAgentCapabilities: () => describeAgentCapabilities,
-  checkAgentHealth: () => checkAgentHealth,
-  MODEL_PRICING: () => MODEL_PRICING,
-  CompleteError: () => CompleteError,
-  ClaudeCodeAdapter: () => ClaudeCodeAdapter,
-  COST_RATES: () => COST_RATES,
-  AllAgentsUnavailableError: () => AllAgentsUnavailableError
-});
 var init_agents = __esm(() => {
   init_types2();
   init_claude();
@@ -28114,7 +28078,7 @@ ${stat}
   return `${statPreamble}${truncated}
 ... (truncated at ${DIFF_CAP_BYTES} bytes, showing ${visibleFiles}/${totalFiles} files)`;
 }
-function buildPrompt(story, semanticConfig, diff, stat) {
+function buildPrompt(story, semanticConfig, diff, _stat) {
   const acList = story.acceptanceCriteria.map((ac, i) => `${i + 1}. ${ac}`).join(`
 `);
   const customRulesSection = semanticConfig.rules.length > 0 ? `
@@ -30002,9 +29966,7 @@ function deriveTestPatterns(contextFiles) {
 async function detectTestDir(workdir) {
   for (const dir of COMMON_TEST_DIRS) {
     const fullPath = path7.join(workdir, dir);
-    const file3 = Bun.file(path7.join(fullPath, "."));
     try {
-      const dirStat = await Bun.file(fullPath).exists();
       const proc = Bun.spawn(["test", "-d", fullPath], { stdout: "pipe", stderr: "pipe" });
       const exitCode = await proc.exited;
       if (exitCode === 0)
@@ -30805,7 +30767,7 @@ function globToRegex(pattern) {
   const regexStr = filePattern.replace(/\./g, "\\.").replace(/\*/g, "[^/]*").replace(/\{([^}]+)\}/g, (_, group) => `(${group.replace(/,/g, "|")})`).replace(/\\\.\\\*/g, "\\.[^/]*");
   return new RegExp(`${regexStr}$`);
 }
-async function isGreenfieldStory(story, workdir, testPattern = "**/*.{test,spec}.{ts,js,tsx,jsx}") {
+async function isGreenfieldStory(_story, workdir, testPattern = "**/*.{test,spec}.{ts,js,tsx,jsx}") {
   try {
     const regex = globToRegex(testPattern);
     const testFiles = await scanForTestFiles(workdir, regex);
@@ -33423,7 +33385,7 @@ var init_optimizer2 = __esm(() => {
   init_optimizer();
   optimizerStage = {
     name: "optimizer",
-    enabled: (ctx) => {
+    enabled: (_ctx) => {
       return true;
     },
     async execute(ctx) {
@@ -34773,7 +34735,7 @@ var init_classify = __esm(() => {
 });
 
 // src/routing/strategies/llm-prompts.ts
-function buildRoutingPrompt(story, config2) {
+function buildRoutingPrompt(story, _config) {
   const { title, description, acceptanceCriteria, tags } = story;
   const criteria = acceptanceCriteria.map((c, i) => `${i + 1}. ${c}`).join(`
 `);
@@ -34807,7 +34769,7 @@ Respond with:
 {"complexity":"simple|medium|complex|expert","modelTier":"fast|balanced|powerful","reasoning":"<one line>"}`;
   return wrapJsonPrompt(core2);
 }
-function buildBatchRoutingPrompt(stories, config2) {
+function buildBatchRoutingPrompt(stories, _config) {
   const storyBlocks = stories.map((story, idx) => {
     const criteria = story.acceptanceCriteria.map((c, i) => `   ${i + 1}. ${c}`).join(`
 `);
@@ -36676,7 +36638,7 @@ var package_default;
 var init_package = __esm(() => {
   package_default = {
     name: "@nathapp/nax",
-    version: "0.61.2-canary.1",
+    version: "0.61.2",
     description: "AI Coding Agent Orchestrator \u2014 loops until done",
     type: "module",
     bin: {
@@ -36756,8 +36718,8 @@ var init_version = __esm(() => {
   NAX_VERSION = package_default.version;
   NAX_COMMIT = (() => {
     try {
-      if (/^[0-9a-f]{6,10}$/.test("7e54444a"))
-        return "7e54444a";
+      if (/^[0-9a-f]{6,10}$/.test("16490524"))
+        return "16490524";
     } catch {}
     try {
       const result = Bun.spawnSync(["git", "rev-parse", "--short", "HEAD"], {
@@ -37104,7 +37066,7 @@ function parseImportStatements(content) {
   }
   return imports;
 }
-function resolveImportPaths(imports, workdir) {
+function resolveImportPaths(imports, _workdir) {
   const resolved = [];
   for (const imp of imports) {
     if (imp.startsWith(".")) {
@@ -38357,7 +38319,7 @@ function wireHooks(bus, hooks, workdir, feature) {
     return fn().catch((err) => logger?.warn("hooks-subscriber", `Hook "${name}" failed`, { error: String(err) })).catch(() => {});
   };
   const unsubs = [];
-  unsubs.push(bus.on("run:started", (ev) => {
+  unsubs.push(bus.on("run:started", (_ev) => {
     return safe("on-start", () => fireHook(hooks, "on-start", hookCtx(feature, { status: "running" }), workdir));
   }));
   unsubs.push(bus.on("story:started", (ev) => {
@@ -38378,7 +38340,7 @@ function wireHooks(bus, hooks, workdir, feature) {
   unsubs.push(bus.on("run:completed", (ev) => {
     return safe("on-complete", () => fireHook(hooks, "on-complete", hookCtx(feature, { status: "complete", cost: ev.totalCost ?? 0 }), workdir));
   }));
-  unsubs.push(bus.on("run:resumed", (ev) => {
+  unsubs.push(bus.on("run:resumed", (_ev) => {
     return safe("on-resume", () => fireHook(hooks, "on-resume", hookCtx(feature, { status: "running" }), workdir));
   }));
   unsubs.push(bus.on("story:completed", (ev) => {
@@ -39131,7 +39093,6 @@ var init_pipeline_result_handler = __esm(() => {
 // src/execution/iteration-runner.ts
 import { join as join45 } from "path";
 async function runIteration(ctx, prd, selection, iterations, totalCost, allStoryMetrics) {
-  const logger = getSafeLogger();
   const { story, storiesToExecute, routing, isBatchExecution } = selection;
   if (ctx.dryRun) {
     const dryRunResult = await handleDryRun({
@@ -39255,7 +39216,6 @@ async function runIteration(ctx, prd, selection, iterations, totalCost, allStory
 var _iterationRunnerDeps;
 var init_iteration_runner = __esm(() => {
   init_loader();
-  init_logger2();
   init_runner();
   init_stages();
   init_prd();
@@ -39373,7 +39333,7 @@ async function executeStoryInWorktree(story, worktreePath, context, routing, eve
     };
   }
 }
-async function executeParallelBatch(stories, projectRoot, config2, context, worktreePaths, maxConcurrency, eventEmitter, storyEffectiveConfigs) {
+async function executeParallelBatch(stories, _projectRoot, config2, context, worktreePaths, maxConcurrency, eventEmitter, storyEffectiveConfigs) {
   const logger = getSafeLogger();
   const results = {
     pipelinePassed: [],
@@ -40524,7 +40484,7 @@ function detectType(pkg) {
     return "cli";
   return;
 }
-async function detectTestFramework(workdir, language, pkg) {
+async function detectTestFramework(_workdir, language, pkg) {
   if (language === "go")
     return "go-test";
   if (language === "rust")
@@ -40984,7 +40944,6 @@ import { join as join50 } from "path";
 async function reconcileState(prd, prdPath, workdir, config2) {
   const logger = getSafeLogger();
   let reconciledCount = 0;
-  let modified = false;
   for (const story of prd.userStories) {
     if (story.status !== "failed")
       continue;
@@ -41019,7 +40978,6 @@ async function reconcileState(prd, prdPath, workdir, config2) {
     });
     markStoryPassed(prd, story.id);
     reconciledCount++;
-    modified = true;
   }
   if (reconciledCount > 0) {
     logger?.info("reconciliation", `Reconciled ${reconciledCount} failed stories from git history`);
@@ -41031,7 +40989,6 @@ async function checkAgentInstalled(config2, dryRun, agentGetFn) {
   if (dryRun)
     return;
   const logger = getSafeLogger();
-  const { getAgent: getAgent2 } = await Promise.resolve().then(() => (init_agents(), exports_agents));
   const agent = (agentGetFn ?? _reconcileDeps.getAgent)(config2.autoMode.defaultAgent);
   if (!agent) {
     logger?.error("execution", "Agent not found", {
@@ -74892,7 +74849,7 @@ import { existsSync as existsSync22, readdirSync as readdirSync6 } from "fs";
 import { join as join30 } from "path";
 
 // src/cli/diagnose-analysis.ts
-function detectFailurePattern(story, prd, status) {
+function detectFailurePattern(story, _prd, status) {
   if (story.status === "passed" && story.priorErrors?.some((err) => err.toLowerCase().includes("greenfield-no-tests"))) {
     return "AUTO_RECOVERED";
   }
@@ -76838,7 +76795,6 @@ async function run(options) {
   let totalCost = 0;
   let runCompleted = false;
   const allStoryMetrics = [];
-  const logger = getSafeLogger();
   const registry2 = createAgentRegistry(config2);
   const agentGetFn = registry2.getAgent.bind(registry2);
   let prd;
@@ -76939,20 +76895,20 @@ async function run(options) {
       durationMs
     };
   } finally {
-    const logger2 = getSafeLogger();
-    logger2?.debug("execution", "Runner finally block \u2014 starting cleanup");
+    const logger = getSafeLogger();
+    logger?.debug("execution", "Runner finally block \u2014 starting cleanup");
     stopHeartbeat();
     cleanupCrashHandlers();
-    logger2?.debug("execution", "Runner finally \u2014 sweeping ACP sessions");
+    logger?.debug("execution", "Runner finally \u2014 sweeping ACP sessions");
     await sweepFeatureSessions(workdir, feature).catch(() => {});
-    logger2?.debug("execution", "Runner finally \u2014 ACP sweep done");
+    logger?.debug("execution", "Runner finally \u2014 ACP sweep done");
     let branch = "";
     try {
       const { stdout, exitCode } = await gitWithTimeout(["branch", "--show-current"], workdir);
       if (exitCode === 0)
         branch = stdout.trim();
     } catch {}
-    logger2?.debug("execution", "Runner finally \u2014 running cleanupRun");
+    logger?.debug("execution", "Runner finally \u2014 running cleanupRun");
     const { cleanupRun: cleanupRun2 } = await Promise.resolve().then(() => (init_run_cleanup(), exports_run_cleanup));
     await cleanupRun2({
       runId,
@@ -76969,7 +76925,7 @@ async function run(options) {
       version: NAX_VERSION,
       runCompleted
     });
-    logger2?.debug("execution", "Runner finally \u2014 cleanupRun done, run() returning");
+    logger?.debug("execution", "Runner finally \u2014 cleanupRun done, run() returning");
   }
 }
 
@@ -83955,7 +83911,6 @@ function usePty(options) {
     isRunning: false
   }));
   const [handle, setHandle] = import_react33.useState(null);
-  const [ptyProcess, setPtyProcess] = import_react33.useState(null);
   const command = options?.command;
   const argsJson = JSON.stringify(options?.args);
   const cwd2 = options?.cwd;
@@ -83971,7 +83926,6 @@ function usePty(options) {
       stdout: "pipe",
       stderr: "inherit"
     });
-    setPtyProcess(proc);
     setState((prev) => ({ ...prev, isRunning: true }));
     (async () => {
       let currentLine = "";

package/package.json

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