@nathapp/nax 0.33.0 → 0.34.0

package/dist/nax.js

@@ -20135,8 +20135,8 @@ function isStalled(prd) {
   const remaining = prd.userStories.filter((s) => s.status !== "passed" && s.status !== "skipped");
   if (remaining.length === 0)
     return false;
-  const blockedIds = new Set(prd.userStories.filter((s) => s.status === "blocked" || s.status === "failed" || s.status === "paused").map((s) => s.id));
-  return remaining.every((s) => s.status === "blocked" || s.status === "failed" || s.status === "paused" || s.dependencies.some((dep) => blockedIds.has(dep)));
+  const blockedIds = new Set(prd.userStories.filter((s) => s.status === "blocked" || s.status === "failed" || s.status === "paused" || s.status === "regression-failed").map((s) => s.id));
+  return remaining.every((s) => s.status === "blocked" || s.status === "failed" || s.status === "paused" || s.status === "regression-failed" || s.dependencies.some((dep) => blockedIds.has(dep)));
 }
 
 // src/prd/index.ts
@@ -20194,8 +20194,8 @@ function countStories(prd) {
   return {
     total: prd.userStories.length,
     passed: prd.userStories.filter((s) => s.passes || s.status === "passed").length,
-    failed: prd.userStories.filter((s) => s.status === "failed").length,
-    pending: prd.userStories.filter((s) => !s.passes && s.status !== "passed" && s.status !== "failed" && s.status !== "skipped" && s.status !== "blocked" && s.status !== "paused" && s.status !== "decomposed").length,
+    failed: prd.userStories.filter((s) => s.status === "failed" || s.status === "regression-failed").length,
+    pending: prd.userStories.filter((s) => !s.passes && s.status !== "passed" && s.status !== "failed" && s.status !== "skipped" && s.status !== "blocked" && s.status !== "paused" && s.status !== "regression-failed" && s.status !== "decomposed").length,
     skipped: prd.userStories.filter((s) => s.status === "skipped").length,
     blocked: prd.userStories.filter((s) => s.status === "blocked").length,
     paused: prd.userStories.filter((s) => s.status === "paused").length,
@@ -20242,7 +20242,7 @@ var package_default;
 var init_package = __esm(() => {
   package_default = {
     name: "@nathapp/nax",
-    version: "0.33.0",
+    version: "0.34.0",
     description: "AI Coding Agent Orchestrator \u2014 loops until done",
     type: "module",
     bin: {
@@ -20304,8 +20304,8 @@ var init_version = __esm(() => {
   NAX_VERSION = package_default.version;
   NAX_COMMIT = (() => {
     try {
-      if (/^[0-9a-f]{6,10}$/.test("f154976"))
-        return "f154976";
+      if (/^[0-9a-f]{6,10}$/.test("a679961"))
+        return "a679961";
     } catch {}
     try {
       const result = Bun.spawnSync(["git", "rev-parse", "--short", "HEAD"], {
@@ -20384,6 +20384,8 @@ function collectStoryMetrics(ctx, storyStartTime) {
   const modelDef = modelEntry ? resolveModel(modelEntry) : null;
   const modelUsed = modelDef?.model || routing.modelTier;
   const initialComplexity = story.routing?.initialComplexity ?? routing.complexity;
+  const isTddStrategy = routing.testStrategy === "three-session-tdd" || routing.testStrategy === "three-session-tdd-lite";
+  const fullSuiteGatePassed = isTddStrategy ? ctx.fullSuiteGatePassed ?? false : false;
   return {
     storyId: story.id,
     complexity: routing.complexity,
@@ -20397,7 +20399,8 @@ function collectStoryMetrics(ctx, storyStartTime) {
     durationMs: agentResult?.durationMs || 0,
     firstPassSuccess,
     startedAt: storyStartTime,
-    completedAt: new Date().toISOString()
+    completedAt: new Date().toISOString(),
+    fullSuiteGatePassed
   };
 }
 function collectBatchMetrics(ctx, storyStartTime) {
@@ -20426,7 +20429,8 @@ function collectBatchMetrics(ctx, storyStartTime) {
       durationMs: durationPerStory,
       firstPassSuccess: true,
       startedAt: storyStartTime,
-      completedAt: new Date().toISOString()
+      completedAt: new Date().toISOString(),
+      fullSuiteGatePassed: false
     };
   });
 }
@@ -28022,6 +28026,181 @@ var init_precheck = __esm(() => {
   };
 });
 
+// src/hooks/runner.ts
+import { existsSync as existsSync25 } from "fs";
+import { join as join30 } from "path";
+async function loadHooksConfig(projectDir, globalDir) {
+  let globalHooks = { hooks: {} };
+  let projectHooks = { hooks: {} };
+  let skipGlobal = false;
+  const projectPath = join30(projectDir, "hooks.json");
+  if (existsSync25(projectPath)) {
+    try {
+      const projectData = await Bun.file(projectPath).json();
+      projectHooks = projectData;
+      skipGlobal = projectData.skipGlobal ?? false;
+    } catch (err) {
+      const logger = getLogger();
+      logger.warn("hooks", "Failed to parse project hooks.json", { path: projectPath, error: String(err) });
+    }
+  }
+  if (!skipGlobal && globalDir) {
+    const globalPath = join30(globalDir, "hooks.json");
+    if (existsSync25(globalPath)) {
+      try {
+        const globalData = await Bun.file(globalPath).json();
+        globalHooks = globalData;
+      } catch (err) {
+        const logger = getLogger();
+        logger.warn("hooks", "Failed to parse global hooks.json", { path: globalPath, error: String(err) });
+      }
+    }
+  }
+  return {
+    ...projectHooks,
+    _global: skipGlobal ? undefined : globalHooks,
+    _skipGlobal: skipGlobal
+  };
+}
+function escapeEnvValue(value) {
+  return value.replace(/\0/g, "").replace(/\n/g, " ").replace(/\r/g, "");
+}
+function buildEnv(ctx) {
+  const env2 = {
+    NAX_EVENT: escapeEnvValue(ctx.event),
+    NAX_FEATURE: escapeEnvValue(ctx.feature)
+  };
+  if (ctx.storyId)
+    env2.NAX_STORY_ID = escapeEnvValue(ctx.storyId);
+  if (ctx.status)
+    env2.NAX_STATUS = escapeEnvValue(ctx.status);
+  if (ctx.reason)
+    env2.NAX_REASON = escapeEnvValue(ctx.reason);
+  if (ctx.cost !== undefined)
+    env2.NAX_COST = ctx.cost.toFixed(4);
+  if (ctx.model)
+    env2.NAX_MODEL = escapeEnvValue(ctx.model);
+  if (ctx.agent)
+    env2.NAX_AGENT = escapeEnvValue(ctx.agent);
+  if (ctx.iteration !== undefined)
+    env2.NAX_ITERATION = String(ctx.iteration);
+  return env2;
+}
+function hasShellOperators(command) {
+  const shellOperators = /[|&;$`<>(){}]/;
+  return shellOperators.test(command);
+}
+function validateHookCommand(command) {
+  const dangerousPatterns = [
+    /\$\(.*\)/,
+    /`.*`/,
+    /\|\s*bash/,
+    /\|\s*sh/,
+    /;\s*rm\s+-rf/,
+    /&&\s*rm\s+-rf/,
+    /\beval\s+/,
+    /curl\s+[^|]*\|\s*/,
+    /python\s+-c/
+  ];
+  for (const pattern of dangerousPatterns) {
+    if (pattern.test(command)) {
+      throw new Error(`Hook command contains dangerous pattern: ${pattern.source}`);
+    }
+  }
+}
+function parseCommandToArgv(command) {
+  const home = process.env.HOME ?? process.env.USERPROFILE ?? "";
+  return command.trim().split(/\s+/).map((token) => token.startsWith("~/") ? home + token.slice(1) : token);
+}
+async function executeHook(hookDef, ctx, workdir) {
+  if (hookDef.enabled === false) {
+    return { success: true, output: "(disabled)" };
+  }
+  try {
+    validateHookCommand(hookDef.command);
+  } catch (err) {
+    return {
+      success: false,
+      output: `Security validation failed: ${err}`
+    };
+  }
+  const logger = getLogger();
+  if (hasShellOperators(hookDef.command)) {
+    logger.warn("hooks", "[SECURITY] Hook command contains shell operators", {
+      command: hookDef.command,
+      warning: "Shell operators may enable injection attacks. Consider using simple commands only."
+    });
+  }
+  const timeout = hookDef.timeout ?? DEFAULT_TIMEOUT;
+  const env2 = buildEnv(ctx);
+  const contextJson = JSON.stringify(ctx);
+  const argv = parseCommandToArgv(hookDef.command);
+  if (argv.length === 0) {
+    return { success: false, output: "Empty command" };
+  }
+  const proc = Bun.spawn(argv, {
+    cwd: workdir,
+    stdin: new Response(contextJson),
+    stdout: "pipe",
+    stderr: "pipe",
+    env: { ...process.env, ...env2 }
+  });
+  const timeoutId = setTimeout(() => {
+    proc.kill("SIGTERM");
+  }, timeout);
+  const exitCode = await proc.exited;
+  clearTimeout(timeoutId);
+  const stdout = await new Response(proc.stdout).text();
+  const stderr = await new Response(proc.stderr).text();
+  const output = (stdout + stderr).trim();
+  if (exitCode !== 0 && output === "") {
+    return {
+      success: false,
+      output: `Hook timed out after ${timeout}ms`
+    };
+  }
+  return {
+    success: exitCode === 0,
+    output
+  };
+}
+async function fireHook(config2, event, ctx, workdir) {
+  const logger = getLogger();
+  if (config2._global && !config2._skipGlobal) {
+    const globalHookDef = config2._global.hooks[event];
+    if (globalHookDef && globalHookDef.enabled !== false) {
+      try {
+        const result = await executeHook(globalHookDef, { ...ctx, event }, workdir);
+        if (!result.success) {
+          logger.warn("hooks", `Global hook ${event} failed`, { event, output: result.output });
+        }
+      } catch (err) {
+        logger.warn("hooks", `Global hook ${event} error`, { event, error: String(err) });
+      }
+    }
+  }
+  const projectHookDef = config2.hooks[event];
+  if (projectHookDef && projectHookDef.enabled !== false) {
+    try {
+      const result = await executeHook(projectHookDef, { ...ctx, event }, workdir);
+      if (!result.success) {
+        logger.warn("hooks", `Project hook ${event} failed`, { event, output: result.output });
+      }
+    } catch (err) {
+      logger.warn("hooks", `Project hook ${event} error`, { event, error: String(err) });
+    }
+  }
+}
+var DEFAULT_TIMEOUT = 5000;
+var init_runner3 = __esm(() => {
+  init_logger2();
+});
+
+// src/hooks/index.ts
+var init_hooks = __esm(() => {
+  init_runner3();
+});
+
 // src/execution/crash-recovery.ts
 import { appendFileSync as appendFileSync2 } from "fs";
 async function writeFatalLog(jsonlFilePath, signal, error48) {
@@ -28267,181 +28446,6 @@ function calculateMaxIterations(tierOrder) {
   return tierOrder.reduce((sum, t) => sum + t.attempts, 0);
 }
 
-// src/hooks/runner.ts
-import { existsSync as existsSync25 } from "fs";
-import { join as join30 } from "path";
-async function loadHooksConfig(projectDir, globalDir) {
-  let globalHooks = { hooks: {} };
-  let projectHooks = { hooks: {} };
-  let skipGlobal = false;
-  const projectPath = join30(projectDir, "hooks.json");
-  if (existsSync25(projectPath)) {
-    try {
-      const projectData = await Bun.file(projectPath).json();
-      projectHooks = projectData;
-      skipGlobal = projectData.skipGlobal ?? false;
-    } catch (err) {
-      const logger = getLogger();
-      logger.warn("hooks", "Failed to parse project hooks.json", { path: projectPath, error: String(err) });
-    }
-  }
-  if (!skipGlobal && globalDir) {
-    const globalPath = join30(globalDir, "hooks.json");
-    if (existsSync25(globalPath)) {
-      try {
-        const globalData = await Bun.file(globalPath).json();
-        globalHooks = globalData;
-      } catch (err) {
-        const logger = getLogger();
-        logger.warn("hooks", "Failed to parse global hooks.json", { path: globalPath, error: String(err) });
-      }
-    }
-  }
-  return {
-    ...projectHooks,
-    _global: skipGlobal ? undefined : globalHooks,
-    _skipGlobal: skipGlobal
-  };
-}
-function escapeEnvValue(value) {
-  return value.replace(/\0/g, "").replace(/\n/g, " ").replace(/\r/g, "");
-}
-function buildEnv(ctx) {
-  const env2 = {
-    NAX_EVENT: escapeEnvValue(ctx.event),
-    NAX_FEATURE: escapeEnvValue(ctx.feature)
-  };
-  if (ctx.storyId)
-    env2.NAX_STORY_ID = escapeEnvValue(ctx.storyId);
-  if (ctx.status)
-    env2.NAX_STATUS = escapeEnvValue(ctx.status);
-  if (ctx.reason)
-    env2.NAX_REASON = escapeEnvValue(ctx.reason);
-  if (ctx.cost !== undefined)
-    env2.NAX_COST = ctx.cost.toFixed(4);
-  if (ctx.model)
-    env2.NAX_MODEL = escapeEnvValue(ctx.model);
-  if (ctx.agent)
-    env2.NAX_AGENT = escapeEnvValue(ctx.agent);
-  if (ctx.iteration !== undefined)
-    env2.NAX_ITERATION = String(ctx.iteration);
-  return env2;
-}
-function hasShellOperators(command) {
-  const shellOperators = /[|&;$`<>(){}]/;
-  return shellOperators.test(command);
-}
-function validateHookCommand(command) {
-  const dangerousPatterns = [
-    /\$\(.*\)/,
-    /`.*`/,
-    /\|\s*bash/,
-    /\|\s*sh/,
-    /;\s*rm\s+-rf/,
-    /&&\s*rm\s+-rf/,
-    /\beval\s+/,
-    /curl\s+[^|]*\|\s*/,
-    /python\s+-c/
-  ];
-  for (const pattern of dangerousPatterns) {
-    if (pattern.test(command)) {
-      throw new Error(`Hook command contains dangerous pattern: ${pattern.source}`);
-    }
-  }
-}
-function parseCommandToArgv(command) {
-  const home = process.env.HOME ?? process.env.USERPROFILE ?? "";
-  return command.trim().split(/\s+/).map((token) => token.startsWith("~/") ? home + token.slice(1) : token);
-}
-async function executeHook(hookDef, ctx, workdir) {
-  if (hookDef.enabled === false) {
-    return { success: true, output: "(disabled)" };
-  }
-  try {
-    validateHookCommand(hookDef.command);
-  } catch (err) {
-    return {
-      success: false,
-      output: `Security validation failed: ${err}`
-    };
-  }
-  const logger = getLogger();
-  if (hasShellOperators(hookDef.command)) {
-    logger.warn("hooks", "[SECURITY] Hook command contains shell operators", {
-      command: hookDef.command,
-      warning: "Shell operators may enable injection attacks. Consider using simple commands only."
-    });
-  }
-  const timeout = hookDef.timeout ?? DEFAULT_TIMEOUT;
-  const env2 = buildEnv(ctx);
-  const contextJson = JSON.stringify(ctx);
-  const argv = parseCommandToArgv(hookDef.command);
-  if (argv.length === 0) {
-    return { success: false, output: "Empty command" };
-  }
-  const proc = Bun.spawn(argv, {
-    cwd: workdir,
-    stdin: new Response(contextJson),
-    stdout: "pipe",
-    stderr: "pipe",
-    env: { ...process.env, ...env2 }
-  });
-  const timeoutId = setTimeout(() => {
-    proc.kill("SIGTERM");
-  }, timeout);
-  const exitCode = await proc.exited;
-  clearTimeout(timeoutId);
-  const stdout = await new Response(proc.stdout).text();
-  const stderr = await new Response(proc.stderr).text();
-  const output = (stdout + stderr).trim();
-  if (exitCode !== 0 && output === "") {
-    return {
-      success: false,
-      output: `Hook timed out after ${timeout}ms`
-    };
-  }
-  return {
-    success: exitCode === 0,
-    output
-  };
-}
-async function fireHook(config2, event, ctx, workdir) {
-  const logger = getLogger();
-  if (config2._global && !config2._skipGlobal) {
-    const globalHookDef = config2._global.hooks[event];
-    if (globalHookDef && globalHookDef.enabled !== false) {
-      try {
-        const result = await executeHook(globalHookDef, { ...ctx, event }, workdir);
-        if (!result.success) {
-          logger.warn("hooks", `Global hook ${event} failed`, { event, output: result.output });
-        }
-      } catch (err) {
-        logger.warn("hooks", `Global hook ${event} error`, { event, error: String(err) });
-      }
-    }
-  }
-  const projectHookDef = config2.hooks[event];
-  if (projectHookDef && projectHookDef.enabled !== false) {
-    try {
-      const result = await executeHook(projectHookDef, { ...ctx, event }, workdir);
-      if (!result.success) {
-        logger.warn("hooks", `Project hook ${event} failed`, { event, output: result.output });
-      }
-    } catch (err) {
-      logger.warn("hooks", `Project hook ${event} error`, { event, error: String(err) });
-    }
-  }
-}
-var DEFAULT_TIMEOUT = 5000;
-var init_runner3 = __esm(() => {
-  init_logger2();
-});
-
-// src/hooks/index.ts
-var init_hooks = __esm(() => {
-  init_runner3();
-});
-
 // src/execution/escalation/tier-outcome.ts
 async function handleNoTierAvailable(ctx, failureCategory) {
   const logger = getSafeLogger();
@@ -29924,7 +29928,8 @@ var init_parallel_lifecycle = __esm(() => {
 // src/execution/parallel-executor.ts
 var exports_parallel_executor = {};
 __export(exports_parallel_executor, {
-  runParallelExecution: () => runParallelExecution
+  runParallelExecution: () => runParallelExecution,
+  _parallelExecutorDeps: () => _parallelExecutorDeps
 });
 import * as os5 from "os";
 import path15 from "path";
@@ -30001,7 +30006,8 @@ async function runParallelExecution(options, initialPrd) {
       feature,
       totalCost
     });
-    await fireHook(hooks, "on-complete", hookCtx(feature, { status: "complete", cost: totalCost }), workdir);
+    await _parallelExecutorDeps.fireHook(hooks, "on-all-stories-complete", hookCtx(feature, { status: "passed", cost: totalCost }), workdir);
+    await _parallelExecutorDeps.fireHook(hooks, "on-complete", hookCtx(feature, { status: "complete", cost: totalCost }), workdir);
     const durationMs = Date.now() - startTime;
     const runCompletedAt = new Date().toISOString();
     const { handleParallelCompletion: handleParallelCompletion2 } = await Promise.resolve().then(() => (init_parallel_lifecycle(), exports_parallel_lifecycle));
@@ -30049,12 +30055,16 @@ async function runParallelExecution(options, initialPrd) {
   }
   return { prd, totalCost, storiesCompleted, completed: false };
 }
+var _parallelExecutorDeps;
 var init_parallel_executor = __esm(() => {
   init_hooks();
   init_logger2();
   init_prd();
   init_helpers();
   init_parallel();
+  _parallelExecutorDeps = {
+    fireHook
+  };
 });
 
 // src/pipeline/subscribers/events-writer.ts
@@ -30444,11 +30454,13 @@ async function handlePipelineSuccess(ctx, pipelineResult) {
   }
   const storiesCompletedDelta = ctx.storiesToExecute.length;
   for (const completedStory of ctx.storiesToExecute) {
+    const now = Date.now();
     logger?.info("story.complete", "Story completed successfully", {
       storyId: completedStory.id,
       storyTitle: completedStory.title,
       totalCost: ctx.totalCost + costDelta,
-      durationMs: Date.now() - ctx.startTime
+      durationMs: now - ctx.startTime,
+      storyDurationMs: ctx.storyStartTime ? now - ctx.storyStartTime : undefined
     });
     pipelineEventBus.emit({
       type: "story:completed",
@@ -30469,7 +30481,8 @@ async function handlePipelineSuccess(ctx, pipelineResult) {
     pendingStories: updatedCounts.pending,
     totalCost: ctx.totalCost + costDelta,
     costLimit: ctx.config.execution.costLimit,
-    elapsedMs: Date.now() - ctx.startTime
+    elapsedMs: Date.now() - ctx.startTime,
+    storyDurationMs: ctx.storyStartTime ? Date.now() - ctx.storyStartTime : undefined
   });
   return { storiesCompletedDelta, costDelta, prd, prdDirty: true };
 }
@@ -30575,6 +30588,7 @@ async function runIteration(ctx, prd, selection, iterations, totalCost, allStory
       prdDirty: dryRunResult.prdDirty
     };
   }
+  const storyStartTime = Date.now();
   const storyGitRef = await captureGitRef(ctx.workdir);
   const pipelineContext = {
     config: ctx.config,
@@ -30622,7 +30636,8 @@ async function runIteration(ctx, prd, selection, iterations, totalCost, allStory
     isBatchExecution,
     allStoryMetrics,
     storyGitRef,
-    interactionChain: ctx.interactionChain
+    interactionChain: ctx.interactionChain,
+    storyStartTime
   };
   if (pipelineResult.success) {
     const r2 = await handlePipelineSuccess(handlerCtx, pipelineResult);
@@ -30751,14 +30766,6 @@ async function executeSequential(ctx, initialPrd) {
             return buildResult("pre-merge-aborted");
           }
         }
-        pipelineEventBus.emit({
-          type: "run:completed",
-          totalStories: 0,
-          passedStories: 0,
-          failedStories: 0,
-          durationMs: Date.now() - ctx.startTime,
-          totalCost
-        });
         return buildResult("completed");
       }
       const selected = selectNextStories(prd, ctx.config, ctx.batchPlan, currentBatchIndex, lastStoryId, ctx.useBatch);
@@ -31242,6 +31249,16 @@ __export(exports_run_completion, {
   handleRunCompletion: () => handleRunCompletion,
   _runCompletionDeps: () => _runCompletionDeps
 });
+function shouldSkipDeferredRegression(allStoryMetrics, isSequential) {
+  const effectiveSequential = isSequential !== false;
+  if (!effectiveSequential) {
+    return false;
+  }
+  if (allStoryMetrics.length === 0) {
+    return false;
+  }
+  return allStoryMetrics.every((m) => m.fullSuiteGatePassed === true);
+}
 async function handleRunCompletion(options) {
   const logger = getSafeLogger();
   const {
@@ -31256,23 +31273,56 @@ async function handleRunCompletion(options) {
     startTime,
     workdir,
     statusWriter,
-    config: config2
+    config: config2,
+    hooksConfig,
+    isSequential
   } = options;
   const regressionMode = config2.execution.regressionGate?.mode;
   if (regressionMode === "deferred" && config2.quality.commands.test) {
-    const regressionResult = await _runCompletionDeps.runDeferredRegression({
-      config: config2,
-      prd,
-      workdir
-    });
-    logger?.info("regression", "Deferred regression gate completed", {
-      success: regressionResult.success,
-      failedTests: regressionResult.failedTests,
-      affectedStories: regressionResult.affectedStories
-    });
+    if (shouldSkipDeferredRegression(allStoryMetrics, isSequential)) {
+      logger?.info("regression", "Smart-skip: skipping deferred regression (all stories passed full-suite gate in sequential mode)");
+    } else {
+      const regressionResult = await _runCompletionDeps.runDeferredRegression({
+        config: config2,
+        prd,
+        workdir
+      });
+      logger?.info("regression", "Deferred regression gate completed", {
+        success: regressionResult.success,
+        failedTests: regressionResult.failedTests,
+        affectedStories: regressionResult.affectedStories
+      });
+      if (!regressionResult.success) {
+        for (const storyId of regressionResult.affectedStories) {
+          const story = prd.userStories.find((s) => s.id === storyId);
+          if (story) {
+            story.status = "regression-failed";
+          }
+        }
+        statusWriter.setRunStatus("failed");
+        if (hooksConfig) {
+          await _runCompletionDeps.fireHook(hooksConfig, "on-final-regression-fail", {
+            event: "on-final-regression-fail",
+            feature,
+            status: "failed",
+            failedTests: regressionResult.failedTests,
+            affectedStories: regressionResult.affectedStories
+          }, workdir);
+        }
+      }
+    }
   }
   const durationMs = Date.now() - startTime;
   const runCompletedAt = new Date().toISOString();
+  const finalCounts = countStories(prd);
+  pipelineEventBus.emit({
+    type: "run:completed",
+    totalStories: finalCounts.total,
+    passedStories: finalCounts.passed,
+    failedStories: finalCounts.failed,
+    durationMs,
+    totalCost
+  });
   const runMetrics = {
     runId,
     feature,
@@ -31281,12 +31331,15 @@ async function handleRunCompletion(options) {
     totalCost,
     totalStories: allStoryMetrics.length,
     storiesCompleted,
-    storiesFailed: countStories(prd).failed,
+    storiesFailed: finalCounts.failed,
     totalDurationMs: durationMs,
     stories: allStoryMetrics
   };
-  await saveRunMetrics(workdir, runMetrics);
-  const finalCounts = countStories(prd);
+  try {
+    await saveRunMetrics(workdir, runMetrics);
+  } catch (err) {
+    logger?.warn("run.complete", "Failed to save run metrics", { error: String(err) });
+  }
   const storyMetricsSummary = allStoryMetrics.map((sm) => ({
     storyId: sm.storyId,
     complexity: sm.complexity,
@@ -31330,12 +31383,15 @@ async function handleRunCompletion(options) {
 }
 var _runCompletionDeps;
 var init_run_completion = __esm(() => {
+  init_runner3();
   init_logger2();
   init_metrics();
+  init_event_bus();
   init_prd();
   init_run_regression();
   _runCompletionDeps = {
-    runDeferredRegression
+    runDeferredRegression,
+    fireHook
   };
 });
 
@@ -65486,6 +65542,7 @@ async function unlockCommand(options) {
 init_config();
 
 // src/execution/runner.ts
+init_hooks();
 init_logger2();
 init_prd();
 init_batch_route();
@@ -65530,8 +65587,12 @@ function precomputeBatchPlan(stories, maxBatchSize = DEFAULT_MAX_BATCH_SIZE) {
 // src/execution/runner.ts
 init_crash_recovery();
 init_helpers();
+init_story_context();
 init_escalation();
 init_escalation();
+var _runnerDeps = {
+  fireHook
+};
 async function run(options) {
   const {
     prdPath,
@@ -65713,6 +65774,9 @@ async function run(options) {
       iterations = acceptanceResult.iterations;
       storiesCompleted = acceptanceResult.storiesCompleted;
     }
+    if (isComplete(prd)) {
+      await _runnerDeps.fireHook(hooks, "on-all-stories-complete", hookCtx(feature, { status: "passed", cost: totalCost }), workdir);
+    }
     const { handleRunCompletion: handleRunCompletion2 } = await Promise.resolve().then(() => (init_run_completion(), exports_run_completion));
     const completionResult = await handleRunCompletion2({
       runId,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nathapp/nax",
-  "version": "0.33.0",
+  "version": "0.34.0",
   "description": "AI Coding Agent Orchestrator \u2014 loops until done",
   "type": "module",
   "bin": {

package/src/execution/crash-recovery.ts

@@ -409,3 +409,11 @@ export function resetCrashHandlers(): void {
   handlersInstalled = false;
   stopHeartbeat();
 }
+
+/**
+ * Returns true if heartbeat timer is currently active.
+ * @internal - test use only.
+ */
+export function _isHeartbeatActive(): boolean {
+  return heartbeatTimer !== null;
+}