@nathapp/nax 0.54.2 → 0.54.4

package/dist/nax.js

@@ -18786,7 +18786,7 @@ async function refineAcceptanceCriteria(criteria, context) {
   if (criteria.length === 0) {
     return [];
   }
-  const { storyId, codebaseContext, config: config2, testStrategy, testFramework } = context;
+  const { storyId, featureName, workdir, codebaseContext, config: config2, testStrategy, testFramework } = context;
   const logger = getLogger();
   const modelTier = config2.acceptance?.model ?? "fast";
   const modelEntry = config2.models[modelTier] ?? config2.models.fast;
@@ -18801,7 +18801,11 @@ async function refineAcceptanceCriteria(criteria, context) {
       jsonMode: true,
       maxTokens: 4096,
       model: modelDef.model,
-      config: config2
+      config: config2,
+      featureName,
+      storyId,
+      workdir,
+      sessionRole: "refine"
     });
   } catch (error48) {
     const reason = errorMessage(error48);
@@ -18940,27 +18944,58 @@ Rules:
 - **NEVER use placeholder assertions** \u2014 no always-passing or always-failing stubs, no TODO comments as the only content, no empty test bodies
 - Every test MUST have real assertions that PASS when the feature is correctly implemented and FAIL when it is broken
 - **Prefer behavioral tests** \u2014 import functions and call them rather than reading source files. For example, to verify "getPostRunActions() returns empty array", import PluginRegistry and call getPostRunActions(), don't grep the source file for the method name.
-- Output raw code only \u2014 no markdown fences, start directly with the language's import or package declaration
-- **Path anchor (CRITICAL)**: This test file lives at \`<package-root>/${acceptanceTestFilename(options.language)}\` and runs from the package root. Import from package sources using relative paths like \`./src/...\`. No deep \`../../../../\` traversal needed.`;
+- **File output (REQUIRED)**: Write the acceptance test file DIRECTLY to the path shown below. Do NOT output the test code in your response. After writing the file, reply with a brief confirmation.
+- **Path anchor (CRITICAL)**: Write the test file to this exact path: \`${options.featureDir}/${acceptanceTestFilename(options.language)}\`. Import from package sources using relative paths like \`./src/...\`. No deep \`../../../../\` traversal needed.`;
   const prompt = basePrompt;
   logger.info("acceptance", "Generating tests from PRD refined criteria", { count: refinedCriteria.length });
   const rawOutput = await (options.adapter ?? _generatorPRDDeps.adapter).complete(prompt, {
     model: options.modelDef.model,
     config: options.config,
     timeoutMs: options.config?.acceptance?.timeoutMs ?? 1800000,
-    workdir: options.workdir
+    workdir: options.workdir,
+    featureName: options.featureName,
+    sessionRole: "acceptance-gen"
   });
   let testCode = extractTestCode(rawOutput);
+  logger.debug("acceptance", "Received raw output from LLM", {
+    hasCode: testCode !== null,
+    outputLength: rawOutput.length,
+    outputPreview: rawOutput.slice(0, 300)
+  });
   if (!testCode) {
     const targetPath = join2(options.featureDir, acceptanceTestFilename(options.language));
+    let recoveryFailed = false;
+    logger.debug("acceptance", "BUG-076 recovery: checking for agent-written file", { targetPath });
     try {
       const existing = await Bun.file(targetPath).text();
       const recovered = extractTestCode(existing);
+      logger.debug("acceptance", "BUG-076 recovery: file check result", {
+        fileSize: existing.length,
+        extractedCode: recovered !== null,
+        filePreview: existing.slice(0, 300)
+      });
       if (recovered) {
         logger.info("acceptance", "Acceptance test written directly by agent \u2014 using existing file", { targetPath });
         testCode = recovered;
+      } else {
+        recoveryFailed = true;
+        logger.error("acceptance", "BUG-076: ACP adapter wrote file but no code extractable \u2014 falling back to skeleton", {
+          targetPath,
+          filePreview: existing.slice(0, 300)
+        });
       }
-    } catch {}
+    } catch {
+      recoveryFailed = true;
+      logger.debug("acceptance", "BUG-076 recovery: no file written by agent, falling back to skeleton", {
+        targetPath,
+        rawOutputPreview: rawOutput.slice(0, 500)
+      });
+    }
+    if (recoveryFailed) {
+      logger.error("acceptance", "BUG-076: LLM returned non-code output and no file was written by agent \u2014 falling back to skeleton", {
+        rawOutputPreview: rawOutput.slice(0, 500)
+      });
+    }
   }
   if (!testCode) {
     logger.warn("acceptance", "LLM returned non-code output for acceptance tests \u2014 falling back to skeleton", {
@@ -19061,7 +19096,9 @@ async function generateAcceptanceTests(adapter, options) {
       model: options.modelDef.model,
       config: options.config,
       timeoutMs: options.config?.acceptance?.timeoutMs ?? 1800000,
-      workdir: options.workdir
+      workdir: options.workdir,
+      featureName: options.featureName,
+      sessionRole: "acceptance-gen"
     });
     const testCode = extractTestCode(output);
     if (!testCode) {
@@ -19312,7 +19349,10 @@ async function generateFixStories(adapter, options) {
     try {
       const fixDescription = await adapter.complete(prompt, {
         model: modelDef.model,
-        config: options.config
+        config: options.config,
+        featureName: options.prd.feature,
+        workdir: options.workdir,
+        sessionRole: "fix-gen"
       });
       fixStories.push({
         id: `US-FIX-${String(i + 1).padStart(3, "0")}`,
@@ -20101,10 +20141,11 @@ class AcpAgentAdapter {
       let session = null;
       let hadError = false;
       try {
+        const completeSessionName = _options?.sessionName ?? buildSessionName(workdir ?? process.cwd(), _options?.featureName, _options?.storyId, _options?.sessionRole);
         session = await client.createSession({
           agentName: this.name,
           permissionMode,
-          sessionName: _options?.sessionName
+          sessionName: completeSessionName
         });
         let timeoutId;
         const timeoutPromise = new Promise((_, reject) => {
@@ -20210,7 +20251,9 @@ class AcpAgentAdapter {
       output = await this.complete(prompt, {
         model,
         jsonMode: true,
-        config: options.config
+        config: options.config,
+        workdir: options.workdir,
+        sessionRole: "decompose"
       });
     } catch (err) {
       const msg = err instanceof Error ? err.message : String(err);
@@ -22305,7 +22348,7 @@ var package_default;
 var init_package = __esm(() => {
   package_default = {
     name: "@nathapp/nax",
-    version: "0.54.2",
+    version: "0.54.4",
     description: "AI Coding Agent Orchestrator \u2014 loops until done",
     type: "module",
     bin: {
@@ -22382,8 +22425,8 @@ var init_version = __esm(() => {
   NAX_VERSION = package_default.version;
   NAX_COMMIT = (() => {
     try {
-      if (/^[0-9a-f]{6,10}$/.test("18dd8fc"))
-        return "18dd8fc";
+      if (/^[0-9a-f]{6,10}$/.test("d0da600"))
+        return "d0da600";
     } catch {}
     try {
       const result = Bun.spawnSync(["git", "rev-parse", "--short", "HEAD"], {
@@ -23101,28 +23144,38 @@ class TelegramInteractionPlugin {
     if (!this.botToken || !this.chatId) {
       throw new Error("Telegram plugin not initialized");
     }
-    const text = this.formatMessage(request);
+    const header = this.buildHeader(request);
     const keyboard = this.buildKeyboard(request);
+    const body = this.buildBody(request);
+    const chunks = this.splitText(body, MAX_MESSAGE_CHARS - header.length - 10);
     try {
-      const response = await fetch(`https://api.telegram.org/bot${this.botToken}/sendMessage`, {
-        method: "POST",
-        headers: { "Content-Type": "application/json" },
-        body: JSON.stringify({
-          chat_id: this.chatId,
-          text,
-          reply_markup: keyboard ? { inline_keyboard: keyboard } : undefined,
-          parse_mode: "Markdown"
-        })
-      });
-      if (!response.ok) {
-        const errorBody = await response.text().catch(() => "");
-        throw new Error(`Telegram API error (${response.status}): ${errorBody || response.statusText}`);
-      }
-      const data = await response.json();
-      if (!data.ok) {
-        throw new Error(`Telegram API returned ok=false: ${JSON.stringify(data)}`);
+      const sentIds = [];
+      for (let i = 0;i < chunks.length; i++) {
+        const isLast = i === chunks.length - 1;
+        const partLabel = chunks.length > 1 ? `[${i + 1}/${chunks.length}] ` : "";
+        const text = `${header}
+${partLabel}${chunks[i]}`;
+        const response = await fetch(`https://api.telegram.org/bot${this.botToken}/sendMessage`, {
+          method: "POST",
+          headers: { "Content-Type": "application/json" },
+          body: JSON.stringify({
+            chat_id: this.chatId,
+            text,
+            reply_markup: isLast && keyboard ? { inline_keyboard: keyboard } : undefined,
+            parse_mode: "Markdown"
+          })
+        });
+        if (!response.ok) {
+          const errorBody = await response.text().catch(() => "");
+          throw new Error(`Telegram API error (${response.status}): ${errorBody || response.statusText}`);
+        }
+        const data = await response.json();
+        if (!data.ok) {
+          throw new Error(`Telegram API returned ok=false: ${JSON.stringify(data)}`);
+        }
+        sentIds.push(data.result.message_id);
       }
-      this.pendingMessages.set(request.id, data.result.message_id);
+      this.pendingMessages.set(request.id, sentIds);
     } catch (err) {
       const msg = err instanceof Error ? err.message : String(err);
       throw new Error(`Failed to send Telegram message: ${msg}`);
@@ -23159,10 +23212,9 @@ class TelegramInteractionPlugin {
     await this.sendTimeoutMessage(requestId);
     this.pendingMessages.delete(requestId);
   }
-  formatMessage(request) {
+  buildHeader(request) {
     const emoji3 = this.getStageEmoji(request.stage);
     let text = `${emoji3} *${request.stage.toUpperCase()}*
-
 `;
     text += `*Feature:* ${request.featureName}
 `;
@@ -23171,11 +23223,15 @@ class TelegramInteractionPlugin {
 `;
     }
     text += `
-${request.summary}
+`;
+    return text;
+  }
+  buildBody(request) {
+    let text = `${this.sanitizeMarkdown(request.summary)}
 `;
     if (request.detail) {
       text += `
-${request.detail}
+${this.sanitizeMarkdown(request.detail)}
 `;
     }
     if (request.options && request.options.length > 0) {
@@ -23183,8 +23239,8 @@ ${request.detail}
 *Options:*
 `;
       for (const opt of request.options) {
-        const desc = opt.description ? ` \u2014 ${opt.description}` : "";
-        text += `  \u2022 ${opt.label}${desc}
+        const desc = opt.description ? ` - ${this.sanitizeMarkdown(opt.description)}` : "";
+        text += `  - ${opt.label}${desc}
 `;
       }
     }
@@ -23195,6 +23251,30 @@ ${request.detail}
     }
     return text;
   }
+  sanitizeMarkdown(text) {
+    return text.replace(/\\(?=[_*`\[])/g, "\\\\").replace(/_/g, "\\_").replace(/`/g, "\\`").replace(/\*/g, "\\*").replace(/\[/g, "\\[");
+  }
+  splitText(text, maxChars) {
+    if (text.length <= maxChars)
+      return [text];
+    const chunks = [];
+    let remaining = text;
+    while (remaining.length > maxChars) {
+      const slice = remaining.slice(0, maxChars);
+      const lastNewline = slice.lastIndexOf(`
+`);
+      if (lastNewline > maxChars * 0.5) {
+        chunks.push(remaining.slice(0, lastNewline));
+        remaining = remaining.slice(lastNewline + 1);
+      } else {
+        chunks.push(slice);
+        remaining = remaining.slice(maxChars);
+      }
+    }
+    if (remaining.length > 0)
+      chunks.push(remaining);
+    return chunks;
+  }
   buildKeyboard(request) {
     switch (request.type) {
       case "confirm":
@@ -23302,8 +23382,11 @@ ${request.detail}
       };
     }
     if (update.message?.text) {
-      const messageId = this.pendingMessages.get(requestId);
-      if (!messageId)
+      const messageIds = this.pendingMessages.get(requestId);
+      if (!messageIds)
+        return null;
+      const replyToId = update.message.reply_to_message?.message_id;
+      if (replyToId !== undefined && !messageIds.includes(replyToId))
         return null;
       return {
         requestId,
@@ -23329,20 +23412,20 @@ ${request.detail}
     } catch {}
   }
   async sendTimeoutMessage(requestId) {
-    const messageId = this.pendingMessages.get(requestId);
-    if (!messageId || !this.botToken || !this.chatId) {
+    const messageIds = this.pendingMessages.get(requestId);
+    if (!messageIds || !this.botToken || !this.chatId) {
       this.pendingMessages.delete(requestId);
       return;
     }
+    const lastId = messageIds[messageIds.length - 1];
     try {
       await fetch(`https://api.telegram.org/bot${this.botToken}/editMessageText`, {
         method: "POST",
         headers: { "Content-Type": "application/json" },
         body: JSON.stringify({
           chat_id: this.chatId,
-          message_id: messageId,
-          text: "\u23F1 *EXPIRED* \u2014 Interaction timed out",
-          parse_mode: "Markdown"
+          message_id: lastId,
+          text: "\u23F1 EXPIRED \u2014 Interaction timed out"
         })
       });
     } catch {} finally {
@@ -23350,7 +23433,7 @@ ${request.detail}
     }
   }
 }
-var TelegramConfigSchema;
+var MAX_MESSAGE_CHARS = 4000, TelegramConfigSchema;
 var init_telegram = __esm(() => {
   init_zod();
   TelegramConfigSchema = exports_external.object({
@@ -23627,7 +23710,10 @@ class AutoInteractionPlugin {
     const output = await adapter.complete(prompt, {
       ...modelArg && { model: modelArg },
       jsonMode: true,
-      ...this.config.naxConfig && { config: this.config.naxConfig }
+      ...this.config.naxConfig && { config: this.config.naxConfig },
+      featureName: request.featureName,
+      storyId: request.storyId,
+      sessionRole: "auto"
     });
     return this.parseResponse(output);
   }
@@ -24368,31 +24454,33 @@ ${stderr}` };
       }
       let totalCriteria = 0;
       let testableCount = 0;
-      const existsResults = await Promise.all(testPaths.map(({ testPath }) => _acceptanceSetupDeps.fileExists(testPath)));
-      const anyFileMissing = existsResults.some((exists) => !exists);
-      let shouldGenerate = anyFileMissing;
-      if (!anyFileMissing) {
-        const fingerprint = computeACFingerprint(allCriteria);
-        const meta3 = await _acceptanceSetupDeps.readMeta(metaPath);
-        getSafeLogger()?.debug("acceptance-setup", "Fingerprint check", {
-          currentFingerprint: fingerprint,
-          storedFingerprint: meta3?.acFingerprint ?? "none",
-          match: meta3?.acFingerprint === fingerprint
-        });
-        if (!meta3 || meta3.acFingerprint !== fingerprint) {
+      const fingerprint = computeACFingerprint(allCriteria);
+      const meta3 = await _acceptanceSetupDeps.readMeta(metaPath);
+      getSafeLogger()?.debug("acceptance-setup", "Fingerprint check", {
+        currentFingerprint: fingerprint,
+        storedFingerprint: meta3?.acFingerprint ?? "none",
+        match: meta3?.acFingerprint === fingerprint
+      });
+      let shouldGenerate = false;
+      if (!meta3 || meta3.acFingerprint !== fingerprint) {
+        if (!meta3) {
+          getSafeLogger()?.info("acceptance-setup", "No acceptance meta \u2014 generating acceptance tests");
+        } else {
           getSafeLogger()?.info("acceptance-setup", "ACs changed \u2014 regenerating acceptance tests", {
-            reason: !meta3 ? "no meta file" : "fingerprint mismatch"
+            reason: "fingerprint mismatch",
+            currentFingerprint: fingerprint,
+            storedFingerprint: meta3.acFingerprint
           });
-          for (const { testPath } of testPaths) {
-            if (await _acceptanceSetupDeps.fileExists(testPath)) {
-              await _acceptanceSetupDeps.copyFile(testPath, `${testPath}.bak`);
-              await _acceptanceSetupDeps.deleteFile(testPath);
-            }
+        }
+        for (const { testPath } of testPaths) {
+          if (await _acceptanceSetupDeps.fileExists(testPath)) {
+            await _acceptanceSetupDeps.copyFile(testPath, `${testPath}.bak`);
+            await _acceptanceSetupDeps.deleteFile(testPath);
           }
-          shouldGenerate = true;
-        } else {
-          getSafeLogger()?.info("acceptance-setup", "Reusing existing acceptance tests (fingerprint match)");
         }
+        shouldGenerate = true;
+      } else {
+        getSafeLogger()?.info("acceptance-setup", "Reusing existing acceptance tests (fingerprint match)");
       }
       if (shouldGenerate) {
         totalCriteria = allCriteria.length;
@@ -24404,6 +24492,8 @@ ${stderr}` };
           for (const story of nonFixStories) {
             const storyRefined = await _acceptanceSetupDeps.refine(story.acceptanceCriteria, {
               storyId: story.id,
+              featureName: ctx.prd.feature,
+              workdir: ctx.workdir,
               codebaseContext: "",
               config: ctx.config,
               testStrategy: ctx.config.acceptance.testStrategy,
@@ -24439,10 +24529,10 @@ ${stderr}` };
           });
           await _acceptanceSetupDeps.writeFile(testPath, result.testCode);
         }
-        const fingerprint = computeACFingerprint(allCriteria);
+        const fingerprint2 = computeACFingerprint(allCriteria);
         await _acceptanceSetupDeps.writeMeta(metaPath, {
           generatedAt: new Date().toISOString(),
-          acFingerprint: fingerprint,
+          acFingerprint: fingerprint2,
           storyCount: ctx.prd.userStories.length,
           acCount: totalCriteria,
           generator: "nax"
@@ -24907,6 +24997,16 @@ async function runSemanticReview(workdir, storyGitRef, story, semanticConfig, mo
       storyId: story.id,
       durationMs: durationMs2
     });
+    logger?.debug("review", "Semantic review findings", {
+      storyId: story.id,
+      findings: parsed.findings.map((f) => ({
+        severity: f.severity,
+        file: f.file,
+        line: f.line,
+        issue: f.issue,
+        suggestion: f.suggestion
+      }))
+    });
     const output = `Semantic review failed:
 
 ${formatFindings(parsed.findings)}`;
@@ -32828,36 +32928,32 @@ import { appendFileSync as appendFileSync2 } from "fs";
 function startHeartbeat(statusWriter, getTotalCost, getIterations, jsonlFilePath) {
   const logger = getSafeLogger();
   stopHeartbeat();
-  heartbeatTimer = setInterval(async () => {
-    logger?.debug("crash-recovery", "Heartbeat");
-    if (jsonlFilePath) {
+  heartbeatTimer = setInterval(() => {
+    (async () => {
       try {
-        const heartbeatEntry = {
-          timestamp: new Date().toISOString(),
-          level: "debug",
-          stage: "heartbeat",
-          message: "Process alive",
-          data: {
-            pid: process.pid,
-            memoryUsageMB: Math.round(process.memoryUsage().heapUsed / 1024 / 1024)
-          }
-        };
-        const line = `${JSON.stringify(heartbeatEntry)}
+        logger?.debug("crash-recovery", "Heartbeat");
+        if (jsonlFilePath) {
+          const heartbeatEntry = {
+            timestamp: new Date().toISOString(),
+            level: "debug",
+            stage: "heartbeat",
+            message: "Process alive",
+            data: {
+              pid: process.pid,
+              memoryUsageMB: Math.round(process.memoryUsage().heapUsed / 1024 / 1024)
+            }
+          };
+          const line = `${JSON.stringify(heartbeatEntry)}
 `;
-        appendFileSync2(jsonlFilePath, line);
+          appendFileSync2(jsonlFilePath, line);
+        }
+        await statusWriter.update(getTotalCost(), getIterations(), {
+          lastHeartbeat: new Date().toISOString()
+        });
       } catch (err) {
-        logger?.warn("crash-recovery", "Failed to write heartbeat", { error: err.message });
+        logger?.warn("crash-recovery", "Failed during heartbeat", { error: err.message });
       }
-    }
-    try {
-      await statusWriter.update(getTotalCost(), getIterations(), {
-        lastHeartbeat: new Date().toISOString()
-      });
-    } catch (err) {
-      logger?.warn("crash-recovery", "Failed to update status during heartbeat", {
-        error: err.message
-      });
-    }
+    })().catch(() => {});
   }, 60000);
   logger?.debug("crash-recovery", "Heartbeat started (60s interval)");
 }
@@ -33016,6 +33112,10 @@ function createSignalHandler(ctx) {
 }
 function createUncaughtExceptionHandler(ctx) {
   return async (error48) => {
+    process.stderr.write(`
+[nax crash] Uncaught exception: ${error48.message}
+${error48.stack ?? ""}
+`);
     const logger = getSafeLogger();
     logger?.error("crash-recovery", "Uncaught exception", {
       error: error48.message,
@@ -33036,6 +33136,10 @@ function createUncaughtExceptionHandler(ctx) {
 function createUnhandledRejectionHandler(ctx) {
   return async (reason) => {
     const error48 = reason instanceof Error ? reason : new Error(String(reason));
+    process.stderr.write(`
+[nax crash] Unhandled rejection: ${error48.message}
+${error48.stack ?? ""}
+`);
     const logger = getSafeLogger();
     logger?.error("crash-recovery", "Unhandled promise rejection", {
       error: error48.message,
@@ -34417,7 +34521,7 @@ __export(exports_parallel_executor_rectify, {
 });
 import path15 from "path";
 async function rectifyConflictedStory(options) {
-  const { storyId, workdir, config: config2, hooks, pluginRegistry, prd, eventEmitter } = options;
+  const { storyId, workdir, config: config2, hooks, pluginRegistry, prd, eventEmitter, agentGetFn } = options;
   const logger = getSafeLogger();
   logger?.info("parallel", "Rectifying story on updated base", { storyId, attempt: "rectification" });
   try {
@@ -34449,7 +34553,8 @@ async function rectifyConflictedStory(options) {
       hooks,
       plugins: pluginRegistry,
       storyStartTime: new Date().toISOString(),
-      routing
+      routing,
+      agentGetFn
     };
     const pipelineResult = await runPipeline2(defaultPipeline2, pipelineContext, eventEmitter);
     const cost = pipelineResult.context.agentResult?.estimatedCost ?? 0;
@@ -34485,7 +34590,7 @@ var init_parallel_executor_rectify = __esm(() => {
 // src/execution/parallel-executor-rectification-pass.ts
 async function runRectificationPass(conflictedStories, options, prd, rectifyConflictedStory2) {
   const logger = getSafeLogger();
-  const { workdir, config: config2, hooks, pluginRegistry, eventEmitter } = options;
+  const { workdir, config: config2, hooks, pluginRegistry, eventEmitter, agentGetFn } = options;
   const rectify = rectifyConflictedStory2 || (async (opts) => {
     const { rectifyConflictedStory: importedRectify } = await Promise.resolve().then(() => (init_parallel_executor_rectify(), exports_parallel_executor_rectify));
     return importedRectify(opts);
@@ -34506,7 +34611,8 @@ async function runRectificationPass(conflictedStories, options, prd, rectifyConf
       hooks,
       pluginRegistry,
       prd,
-      eventEmitter
+      eventEmitter,
+      agentGetFn
     });
     additionalCost += result.cost;
     if (result.success) {
@@ -34829,7 +34935,7 @@ function wireEventsWriter(bus, feature, runId, workdir) {
   const eventsFile = join49(eventsDir, "events.jsonl");
   let dirReady = false;
   const write = (line) => {
-    (async () => {
+    return (async () => {
       try {
         if (!dirReady) {
           await mkdir2(eventsDir, { recursive: true });
@@ -34847,16 +34953,30 @@ function wireEventsWriter(bus, feature, runId, workdir) {
   };
   const unsubs = [];
   unsubs.push(bus.on("run:started", (_ev) => {
-    write({ ts: new Date().toISOString(), event: "run:started", runId, feature, project });
+    return write({ ts: new Date().toISOString(), event: "run:started", runId, feature, project });
   }));
   unsubs.push(bus.on("story:started", (ev) => {
-    write({ ts: new Date().toISOString(), event: "story:started", runId, feature, project, storyId: ev.storyId });
+    return write({
+      ts: new Date().toISOString(),
+      event: "story:started",
+      runId,
+      feature,
+      project,
+      storyId: ev.storyId
+    });
   }));
   unsubs.push(bus.on("story:completed", (ev) => {
-    write({ ts: new Date().toISOString(), event: "story:completed", runId, feature, project, storyId: ev.storyId });
+    return write({
+      ts: new Date().toISOString(),
+      event: "story:completed",
+      runId,
+      feature,
+      project,
+      storyId: ev.storyId
+    });
   }));
   unsubs.push(bus.on("story:decomposed", (ev) => {
-    write({
+    return write({
       ts: new Date().toISOString(),
       event: "story:decomposed",
       runId,
@@ -34867,13 +34987,20 @@ function wireEventsWriter(bus, feature, runId, workdir) {
     });
   }));
   unsubs.push(bus.on("story:failed", (ev) => {
-    write({ ts: new Date().toISOString(), event: "story:failed", runId, feature, project, storyId: ev.storyId });
+    return write({
+      ts: new Date().toISOString(),
+      event: "story:failed",
+      runId,
+      feature,
+      project,
+      storyId: ev.storyId
+    });
   }));
   unsubs.push(bus.on("run:completed", (_ev) => {
-    write({ ts: new Date().toISOString(), event: "on-complete", runId, feature, project });
+    return write({ ts: new Date().toISOString(), event: "on-complete", runId, feature, project });
   }));
   unsubs.push(bus.on("run:paused", (ev) => {
-    write({
+    return write({
       ts: new Date().toISOString(),
       event: "run:paused",
       runId,
@@ -34895,44 +35022,44 @@ var init_events_writer = __esm(() => {
 function wireHooks(bus, hooks, workdir, feature) {
   const logger = getSafeLogger();
   const safe = (name, fn) => {
-    fn().catch((err) => logger?.warn("hooks-subscriber", `Hook "${name}" failed`, { error: String(err) }));
+    return fn().catch((err) => logger?.warn("hooks-subscriber", `Hook "${name}" failed`, { error: String(err) })).catch(() => {});
   };
   const unsubs = [];
   unsubs.push(bus.on("run:started", (ev) => {
-    safe("on-start", () => fireHook(hooks, "on-start", hookCtx(feature, { status: "running" }), workdir));
+    return safe("on-start", () => fireHook(hooks, "on-start", hookCtx(feature, { status: "running" }), workdir));
   }));
   unsubs.push(bus.on("story:started", (ev) => {
-    safe("on-story-start", () => fireHook(hooks, "on-story-start", hookCtx(feature, { storyId: ev.storyId, model: ev.modelTier, agent: ev.agent }), workdir));
+    return safe("on-story-start", () => fireHook(hooks, "on-story-start", hookCtx(feature, { storyId: ev.storyId, model: ev.modelTier, agent: ev.agent }), workdir));
   }));
   unsubs.push(bus.on("story:completed", (ev) => {
-    safe("on-story-complete", () => fireHook(hooks, "on-story-complete", hookCtx(feature, { storyId: ev.storyId, status: "passed", cost: ev.cost }), workdir));
+    return safe("on-story-complete", () => fireHook(hooks, "on-story-complete", hookCtx(feature, { storyId: ev.storyId, status: "passed", cost: ev.cost }), workdir));
   }));
   unsubs.push(bus.on("story:decomposed", (ev) => {
-    safe("on-story-complete (decomposed)", () => fireHook(hooks, "on-story-complete", hookCtx(feature, { storyId: ev.storyId, status: "decomposed", subStoryCount: ev.subStoryCount }), workdir));
+    return safe("on-story-complete (decomposed)", () => fireHook(hooks, "on-story-complete", hookCtx(feature, { storyId: ev.storyId, status: "decomposed", subStoryCount: ev.subStoryCount }), workdir));
   }));
   unsubs.push(bus.on("story:failed", (ev) => {
-    safe("on-story-fail", () => fireHook(hooks, "on-story-fail", hookCtx(feature, { storyId: ev.storyId, status: "failed", reason: ev.reason }), workdir));
+    return safe("on-story-fail", () => fireHook(hooks, "on-story-fail", hookCtx(feature, { storyId: ev.storyId, status: "failed", reason: ev.reason }), workdir));
   }));
   unsubs.push(bus.on("story:paused", (ev) => {
-    safe("on-pause (story)", () => fireHook(hooks, "on-pause", hookCtx(feature, { storyId: ev.storyId, reason: ev.reason, cost: ev.cost }), workdir));
+    return safe("on-pause (story)", () => fireHook(hooks, "on-pause", hookCtx(feature, { storyId: ev.storyId, reason: ev.reason, cost: ev.cost }), workdir));
   }));
   unsubs.push(bus.on("run:paused", (ev) => {
-    safe("on-pause (run)", () => fireHook(hooks, "on-pause", hookCtx(feature, { storyId: ev.storyId, reason: ev.reason, cost: ev.cost }), workdir));
+    return safe("on-pause (run)", () => fireHook(hooks, "on-pause", hookCtx(feature, { storyId: ev.storyId, reason: ev.reason, cost: ev.cost }), workdir));
   }));
   unsubs.push(bus.on("run:completed", (ev) => {
-    safe("on-complete", () => fireHook(hooks, "on-complete", hookCtx(feature, { status: "complete", cost: ev.totalCost ?? 0 }), workdir));
+    return safe("on-complete", () => fireHook(hooks, "on-complete", hookCtx(feature, { status: "complete", cost: ev.totalCost ?? 0 }), workdir));
   }));
   unsubs.push(bus.on("run:resumed", (ev) => {
-    safe("on-resume", () => fireHook(hooks, "on-resume", hookCtx(feature, { status: "running" }), workdir));
+    return safe("on-resume", () => fireHook(hooks, "on-resume", hookCtx(feature, { status: "running" }), workdir));
   }));
   unsubs.push(bus.on("story:completed", (ev) => {
-    safe("on-session-end (completed)", () => fireHook(hooks, "on-session-end", hookCtx(feature, { storyId: ev.storyId, status: "passed" }), workdir));
+    return safe("on-session-end (completed)", () => fireHook(hooks, "on-session-end", hookCtx(feature, { storyId: ev.storyId, status: "passed" }), workdir));
   }));
   unsubs.push(bus.on("story:failed", (ev) => {
-    safe("on-session-end (failed)", () => fireHook(hooks, "on-session-end", hookCtx(feature, { storyId: ev.storyId, status: "failed" }), workdir));
+    return safe("on-session-end (failed)", () => fireHook(hooks, "on-session-end", hookCtx(feature, { storyId: ev.storyId, status: "failed" }), workdir));
   }));
   unsubs.push(bus.on("run:errored", (ev) => {
-    safe("on-error", () => fireHook(hooks, "on-error", hookCtx(feature, { reason: ev.reason }), workdir));
+    return safe("on-error", () => fireHook(hooks, "on-error", hookCtx(feature, { reason: ev.reason }), workdir));
   }));
   return () => {
     for (const u of unsubs)
@@ -35007,7 +35134,7 @@ function wireRegistry(bus, feature, runId, workdir) {
   const runDir = join50(homedir6(), ".nax", "runs", `${project}-${feature}-${runId}`);
   const metaFile = join50(runDir, "meta.json");
   const unsub = bus.on("run:started", (_ev) => {
-    (async () => {
+    return (async () => {
       try {
         await mkdir3(runDir, { recursive: true });
         const meta3 = {
@@ -35038,11 +35165,11 @@ var init_registry3 = __esm(() => {
 function wireReporters(bus, pluginRegistry, runId, startTime) {
   const logger = getSafeLogger();
   const safe = (name, fn) => {
-    fn().catch((err) => logger?.warn("reporters-subscriber", `Reporter "${name}" error`, { error: String(err) }));
+    return fn().catch((err) => logger?.warn("reporters-subscriber", `Reporter "${name}" error`, { error: String(err) })).catch(() => {});
   };
   const unsubs = [];
   unsubs.push(bus.on("run:started", (ev) => {
-    safe("onRunStart", async () => {
+    return safe("onRunStart", async () => {
       const reporters = pluginRegistry.getReporters();
       for (const r of reporters) {
         if (r.onRunStart) {
@@ -35061,7 +35188,7 @@ function wireReporters(bus, pluginRegistry, runId, startTime) {
     });
   }));
   unsubs.push(bus.on("story:completed", (ev) => {
-    safe("onStoryComplete(completed)", async () => {
+    return safe("onStoryComplete(completed)", async () => {
       const reporters = pluginRegistry.getReporters();
       for (const r of reporters) {
         if (r.onStoryComplete) {
@@ -35083,7 +35210,7 @@ function wireReporters(bus, pluginRegistry, runId, startTime) {
     });
   }));
   unsubs.push(bus.on("story:failed", (ev) => {
-    safe("onStoryComplete(failed)", async () => {
+    return safe("onStoryComplete(failed)", async () => {
       const reporters = pluginRegistry.getReporters();
       for (const r of reporters) {
         if (r.onStoryComplete) {
@@ -35105,7 +35232,7 @@ function wireReporters(bus, pluginRegistry, runId, startTime) {
     });
   }));
   unsubs.push(bus.on("story:paused", (ev) => {
-    safe("onStoryComplete(paused)", async () => {
+    return safe("onStoryComplete(paused)", async () => {
       const reporters = pluginRegistry.getReporters();
       for (const r of reporters) {
         if (r.onStoryComplete) {
@@ -35127,7 +35254,7 @@ function wireReporters(bus, pluginRegistry, runId, startTime) {
     });
   }));
   unsubs.push(bus.on("run:completed", (ev) => {
-    safe("onRunEnd", async () => {
+    return safe("onRunEnd", async () => {
       const reporters = pluginRegistry.getReporters();
       for (const r of reporters) {
         if (r.onRunEnd) {
@@ -68822,11 +68949,19 @@ function validateStory(raw, index, allIds) {
     ...contextFiles.length > 0 ? { contextFiles } : {}
   };
 }
+function sanitizeInvalidEscapes(text) {
+  let result = text.replace(/\\x([0-9a-fA-F]{1,2})/g, (_, hex3) => `\\u00${hex3.padStart(2, "0")}`);
+  result = result.replace(/\\u([0-9a-fA-F]{1,3})(?![0-9a-fA-F])/g, (_, digits) => `\\u${digits.padStart(4, "0")}`);
+  result = result.replace(/\\u(?![0-9a-fA-F])/g, "\\");
+  result = result.replace(/\\([^"\\\/bfnrtu])/g, "$1");
+  return result;
+}
 function parseRawString(text) {
   const extracted = extractJsonFromMarkdown(text);
   const cleaned = stripTrailingCommas(extracted);
+  const sanitized = sanitizeInvalidEscapes(cleaned);
   try {
-    return JSON.parse(cleaned);
+    return JSON.parse(sanitized);
   } catch (err) {
     const parseErr = err;
     throw new Error(`[schema] Failed to parse JSON: ${parseErr.message}`, { cause: parseErr });
@@ -68924,7 +69059,14 @@ async function planCommand(workdir, config2, options) {
       if (entry)
         autoModel = resolveModel2(entry).model;
     } catch {}
-    rawResponse = await cliAdapter.complete(prompt, { model: autoModel, jsonMode: true, workdir, config: config2 });
+    rawResponse = await cliAdapter.complete(prompt, {
+      model: autoModel,
+      jsonMode: true,
+      workdir,
+      config: config2,
+      featureName: options.feature,
+      sessionRole: "plan"
+    });
     try {
       const envelope = JSON.parse(rawResponse);
       if (envelope?.type === "result" && typeof envelope?.result === "string") {
@@ -71349,7 +71491,7 @@ Runs:
     const summary = await extractRunSummary(filePath);
     const timestamp = file2.replace(".jsonl", "");
     const stories = summary ? `${summary.passed}/${summary.total}` : "?/?";
-    const duration3 = summary ? formatDuration2(summary.durationMs) : "?";
+    const duration3 = summary ? formatDuration(summary.durationMs) : "?";
     const cost = summary ? `$${summary.totalCost.toFixed(4)}` : "$?.????";
     const status = summary ? summary.failed === 0 ? source_default.green("\u2713") : source_default.red("\u2717") : "?";
     console.log(`  ${timestamp}  ${stories.padEnd(7)}  ${duration3.padEnd(8)}  ${cost.padEnd(8)}  ${status}`);
@@ -71443,17 +71585,6 @@ function shouldDisplayEntry(entry, options) {
   }
   return true;
 }
-function formatDuration2(ms) {
-  if (ms < 1000) {
-    return `${ms}ms`;
-  }
-  if (ms < 60000) {
-    return `${(ms / 1000).toFixed(1)}s`;
-  }
-  const minutes = Math.floor(ms / 60000);
-  const seconds = Math.floor(ms % 60000 / 1000);
-  return `${minutes}m${seconds}s`;
-}
 
 // src/commands/logs.ts
 async function logsCommand(options) {
@@ -71555,7 +71686,7 @@ var DEFAULT_LIMIT = 20;
 var _runsCmdDeps = {
   getRunsDir
 };
-function formatDuration3(ms) {
+function formatDuration2(ms) {
   if (ms <= 0)
     return "-";
   const minutes = Math.floor(ms / 60000);
@@ -71668,7 +71799,7 @@ async function runsCommand(options = {}) {
       pad3(row.feature, COL.feature),
       pad3(colored, COL.status + (colored.length - visibleLength(colored))),
       pad3(`${row.passed}/${row.total}`, COL.stories),
-      pad3(formatDuration3(row.durationMs), COL.duration),
+      pad3(formatDuration2(row.durationMs), COL.duration),
       formatDate(row.registeredAt)
     ].join("  ");
     console.log(line);
@@ -71955,7 +72086,8 @@ async function runExecutionPhase(options, prd, pluginRegistry) {
       allStoryMetrics,
       pluginRegistry,
       formatterMode: options.formatterMode,
-      headless: options.headless
+      headless: options.headless,
+      agentGetFn: options.agentGetFn
     }, prd);
     prd = parallelResult.prd;
     totalCost = parallelResult.totalCost;
@@ -71991,7 +72123,8 @@ async function runExecutionPhase(options, prd, pluginRegistry) {
     startTime: options.startTime,
     batchPlan,
     agentGetFn: options.agentGetFn,
-    pidRegistry: options.pidRegistry
+    pidRegistry: options.pidRegistry,
+    interactionChain: options.interactionChain
   }, prd);
   prd = sequentialResult.prd;
   iterations = sequentialResult.iterations;
@@ -72117,7 +72250,8 @@ async function run(options) {
       parallel,
       runParallelExecution: _runnerDeps.runParallelExecution ?? undefined,
       agentGetFn,
-      pidRegistry
+      pidRegistry,
+      interactionChain
     }, prd, pluginRegistry);
     prd = executionResult.prd;
     iterations = executionResult.iterations;

package/package.json

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