@nathapp/nax 0.39.0 → 0.39.1

package/dist/nax.js

@@ -20159,7 +20159,9 @@ async function tryLlmBatchRoute(config2, stories, label = "routing") {
   const logger = getSafeLogger();
   try {
     logger?.debug("routing", `LLM batch routing: ${label}`, { storyCount: stories.length, mode });
-    await routeBatch(stories, { config: config2 });
+    const agentName = config2.execution?.agent ?? "claude";
+    const adapter = getAgent(agentName);
+    await routeBatch(stories, { config: config2, adapter });
     logger?.debug("routing", "LLM batch routing complete", { label });
   } catch (err) {
     logger?.warn("routing", "LLM batch routing failed, falling back to individual routing", {
@@ -20169,6 +20171,7 @@ async function tryLlmBatchRoute(config2, stories, label = "routing") {
   }
 }
 var init_batch_route = __esm(() => {
+  init_registry();
   init_logger2();
   init_llm();
 });
@@ -20793,7 +20796,7 @@ var package_default;
 var init_package = __esm(() => {
   package_default = {
     name: "@nathapp/nax",
-    version: "0.39.0",
+    version: "0.39.1",
     description: "AI Coding Agent Orchestrator \u2014 loops until done",
     type: "module",
     bin: {
@@ -20857,8 +20860,8 @@ var init_version = __esm(() => {
   NAX_VERSION = package_default.version;
   NAX_COMMIT = (() => {
     try {
-      if (/^[0-9a-f]{6,10}$/.test("e6f293e"))
-        return "e6f293e";
+      if (/^[0-9a-f]{6,10}$/.test("91b2a1c"))
+        return "91b2a1c";
     } catch {}
     try {
       const result = Bun.spawnSync(["git", "rev-parse", "--short", "HEAD"], {
@@ -22813,7 +22816,7 @@ async function getChangedFiles(workdir, baseRef) {
 }
 
 class ReviewOrchestrator {
-  async review(reviewConfig, workdir, executionConfig, plugins) {
+  async review(reviewConfig, workdir, executionConfig, plugins, storyGitRef) {
     const logger = getSafeLogger();
     const builtIn = await runReview(reviewConfig, workdir, executionConfig);
     if (!builtIn.success) {
@@ -22822,7 +22825,7 @@ class ReviewOrchestrator {
     if (plugins) {
       const reviewers = plugins.getReviewers();
       if (reviewers.length > 0) {
-        const baseRef = executionConfig?.storyGitRef;
+        const baseRef = storyGitRef ?? executionConfig?.storyGitRef;
         const changedFiles = await getChangedFiles(workdir, baseRef);
         const pluginResults = [];
         for (const reviewer of reviewers) {
@@ -22897,7 +22900,7 @@ var init_review = __esm(() => {
     async execute(ctx) {
       const logger = getLogger();
       logger.info("review", "Running review phase", { storyId: ctx.story.id });
-      const result = await reviewOrchestrator.review(ctx.config.review, ctx.workdir, ctx.config.execution, ctx.plugins);
+      const result = await reviewOrchestrator.review(ctx.config.review, ctx.workdir, ctx.config.execution, ctx.plugins, ctx.storyGitRef);
       ctx.reviewResult = result.builtIn;
       if (!result.success) {
         const allFindings = result.builtIn.pluginReviewers?.flatMap((pr) => pr.findings ?? []) ?? [];
@@ -31644,22 +31647,6 @@ function resolveMaxAttemptsOutcome(failureCategory) {
       return "fail";
   }
 }
-async function tryLlmBatchRoute2(config2, stories, label = "routing") {
-  const mode = config2.routing.llm?.mode ?? "hybrid";
-  if (config2.routing.strategy !== "llm" || mode === "per-story" || stories.length === 0)
-    return;
-  const logger = getSafeLogger();
-  try {
-    logger?.debug("routing", `LLM batch routing: ${label}`, { storyCount: stories.length, mode });
-    await routeBatch(stories, { config: config2 });
-    logger?.debug("routing", "LLM batch routing complete", { label });
-  } catch (err) {
-    logger?.warn("routing", "LLM batch routing failed, falling back to individual routing", {
-      error: err.message,
-      label
-    });
-  }
-}
 function shouldRetrySameTier(verifyResult) {
   return verifyResult?.status === "RUNTIME_CRASH";
 }
@@ -31739,7 +31726,7 @@ async function handleTierEscalation(ctx) {
     clearCacheForStory(story.id);
   }
   if (routingMode === "hybrid") {
-    await tryLlmBatchRoute2(ctx.config, storiesToEscalate, "hybrid-re-route-pipeline");
+    await tryLlmBatchRoute(ctx.config, storiesToEscalate, "hybrid-re-route-pipeline");
   }
   return {
     outcome: "escalated",
@@ -31752,6 +31739,7 @@ var init_tier_escalation = __esm(() => {
   init_hooks();
   init_logger2();
   init_prd();
+  init_batch_route();
   init_llm();
   init_escalation();
   init_helpers();

package/package.json

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

package/src/execution/escalation/tier-escalation.ts

@@ -12,7 +12,8 @@ import { type LoadedHooksConfig, fireHook } from "../../hooks";
 import { getSafeLogger } from "../../logger";
 import type { PRD, StructuredFailure, UserStory } from "../../prd";
 import { markStoryFailed, savePRD } from "../../prd";
-import { clearCacheForStory, routeBatch as llmRouteBatch } from "../../routing/strategies/llm";
+import { tryLlmBatchRoute } from "../../routing/batch-route";
+import { clearCacheForStory } from "../../routing/strategies/llm";
 import type { FailureCategory } from "../../tdd/types";
 import { calculateMaxIterations, escalateTier, getTierConfig } from "../escalation";
 import { hookCtx } from "../helpers";
@@ -172,25 +173,6 @@ export async function preIterationTierCheck(
   return { shouldSkipIteration: true, prdDirty: true, prd: failedPrd };
 }
 
-/**
- * Try LLM batch routing for ready stories. Logs and swallows errors (falls back to per-story routing).
- */
-async function tryLlmBatchRoute(config: NaxConfig, stories: UserStory[], label = "routing"): Promise<void> {
-  const mode = config.routing.llm?.mode ?? "hybrid";
-  if (config.routing.strategy !== "llm" || mode === "per-story" || stories.length === 0) return;
-  const logger = getSafeLogger();
-  try {
-    logger?.debug("routing", `LLM batch routing: ${label}`, { storyCount: stories.length, mode });
-    await llmRouteBatch(stories, { config });
-    logger?.debug("routing", "LLM batch routing complete", { label });
-  } catch (err) {
-    logger?.warn("routing", "LLM batch routing failed, falling back to individual routing", {
-      error: (err as Error).message,
-      label,
-    });
-  }
-}
-
 export interface EscalationHandlerContext {
   story: UserStory;
   storiesToExecute: UserStory[];

package/src/pipeline/stages/review.ts

@@ -25,7 +25,13 @@ export const reviewStage: PipelineStage = {
 
     logger.info("review", "Running review phase", { storyId: ctx.story.id });
 
-    const result = await reviewOrchestrator.review(ctx.config.review, ctx.workdir, ctx.config.execution, ctx.plugins);
+    const result = await reviewOrchestrator.review(
+      ctx.config.review,
+      ctx.workdir,
+      ctx.config.execution,
+      ctx.plugins,
+      ctx.storyGitRef,
+    );
 
     ctx.reviewResult = result.builtIn;
 

package/src/review/orchestrator.ts

@@ -74,6 +74,7 @@ export class ReviewOrchestrator {
     workdir: string,
     executionConfig: NaxConfig["execution"],
     plugins?: PluginRegistry,
+    storyGitRef?: string,
   ): Promise<OrchestratorReviewResult> {
     const logger = getSafeLogger();
 
@@ -87,7 +88,7 @@ export class ReviewOrchestrator {
       const reviewers = plugins.getReviewers();
       if (reviewers.length > 0) {
         // Use the story's start ref if available to capture auto-committed changes
-        const baseRef = executionConfig?.storyGitRef;
+        const baseRef = storyGitRef ?? executionConfig?.storyGitRef;
         const changedFiles = await getChangedFiles(workdir, baseRef);
         const pluginResults: ReviewResult["pluginReviewers"] = [];
 

package/src/routing/batch-route.ts

@@ -2,6 +2,7 @@
  * LLM Batch Routing Helper
  */
 
+import { getAgent } from "../agents/registry";
 import type { NaxConfig } from "../config";
 import { getSafeLogger } from "../logger";
 import type { UserStory } from "../prd";
@@ -21,7 +22,9 @@ export async function tryLlmBatchRoute(config: NaxConfig, stories: UserStory[],
   const logger = getSafeLogger();
   try {
     logger?.debug("routing", `LLM batch routing: ${label}`, { storyCount: stories.length, mode });
-    await llmRouteBatch(stories, { config });
+    const agentName = config.execution?.agent ?? "claude";
+    const adapter = getAgent(agentName);
+    await llmRouteBatch(stories, { config, adapter });
     logger?.debug("routing", "LLM batch routing complete", { label });
   } catch (err) {
     logger?.warn("routing", "LLM batch routing failed, falling back to individual routing", {