@nathapp/nax 0.54.2 → 0.54.3

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.3",
     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("5acee1f"))
+        return "5acee1f";
     } catch {}
     try {
       const result = Bun.spawnSync(["git", "rev-parse", "--short", "HEAD"], {
@@ -23627,7 +23670,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 +24414,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 +24452,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 +24489,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 +24957,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 +32888,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 +33072,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 +33096,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,
@@ -34829,7 +34893,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 +34911,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 +34945,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 +34980,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 +35092,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 +35123,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 +35146,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 +35168,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 +35190,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 +35212,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) {
@@ -68924,7 +69009,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 +71441,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 +71535,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 +71636,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 +71749,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);
@@ -71991,7 +72072,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 +72199,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.3",
   "description": "AI Coding Agent Orchestrator — loops until done",
   "type": "module",
   "bin": {