nexus-agents 8.46.15 → 8.47.0

package/dist/{chunk-P6LLNTKM.js → chunk-6GGWO4B3.js}

@@ -21,7 +21,7 @@ import {
 import {
   DEFAULT_TASK_TTL_MS,
   clampTaskTtl
-} from "./chunk-AOVMGKUI.js";
+} from "./chunk-AR3RVMX7.js";
 import {
   executeExpert
 } from "./chunk-QLF22ILR.js";
@@ -55,7 +55,7 @@ import {
   toJobSummary,
   updateStage,
   writeJobCancelled
-} from "./chunk-R2MAX7UL.js";
+} from "./chunk-ISUPN5U4.js";
 import {
   allOf,
   anyOf
@@ -75,7 +75,7 @@ import {
 } from "./chunk-43ONDTAK.js";
 import {
   withPrerequisite
-} from "./chunk-GUS6PTRG.js";
+} from "./chunk-CEB525JC.js";
 import {
   NOOP_NOTIFIER,
   RateLimiter,
@@ -100,7 +100,7 @@ import {
   withAuditTrail,
   withProgressHeartbeat,
   wrapToolWithTimeout
-} from "./chunk-L4EZIS3F.js";
+} from "./chunk-RSC54OHX.js";
 import {
   createAuditTrail,
   createGraphAuditBridge,
@@ -29636,7 +29636,7 @@ function formatCompileError(error) {
     case "no_entry":
       return `No entry point: ${error.message}`;
     case "unreachable_node":
-      return `Node '${error.nodeId}' is unreachable from START`;
+      return `Node '${error.nodeId}' is unreachable from START. If it is entered by Command.goto, declare it: addNode(source, fn, { gotoTargets: ['${error.nodeId}'] })`;
     case "missing_reducer":
       return `State field '${error.field}' has no reducer defined`;
   }
@@ -30562,6 +30562,9 @@ async function withTimeout3(promise, ms) {
 }
 
 // src/orchestration/graph/graph-builder.ts
+function definedNodeOptions(opts) {
+  return Object.fromEntries(Object.entries(opts ?? {}).filter(([, value]) => value !== void 0));
+}
 var GraphBuilder = class {
   nodes = /* @__PURE__ */ new Map();
   edges = [];
@@ -30578,14 +30581,7 @@ var GraphBuilder = class {
    * Supports optional precondition hooks (Issue #997) and verify hook (Issue #994).
    */
   addNode(id, handler3, opts) {
-    const node = {
-      id,
-      handler: handler3,
-      ...opts?.timeout !== void 0 ? { timeout: opts.timeout } : {},
-      ...opts?.retries !== void 0 ? { retries: opts.retries } : {},
-      ...opts?.preconditions !== void 0 ? { preconditions: opts.preconditions } : {},
-      ...opts?.verify !== void 0 ? { verify: opts.verify } : {}
-    };
+    const node = { id, handler: handler3, ...definedNodeOptions(opts) };
     this.nodes.set(id, node);
     return this;
   }
@@ -30623,6 +30619,8 @@ var GraphBuilder = class {
     if (entryError !== void 0) return err(entryError);
     const cycleError = this.checkCycles();
     if (cycleError !== void 0) return err(cycleError);
+    const gotoError = this.checkGotoTargets();
+    if (gotoError !== void 0) return err(gotoError);
     const reachError = this.checkReachability();
     if (reachError !== void 0) return err(reachError);
     const entryEdges = this.edges.filter((e) => e.from === START);
@@ -30722,7 +30720,39 @@ var GraphBuilder = class {
     }
     return void 0;
   }
-  /** BFS from START to find all reachable node IDs. */
+  /**
+   * Every declared `gotoTargets` entry must name a real node (#5727). Checked
+   * before reachability so the error names the DECLARING node, rather than
+   * surfacing later as a confusing `unreachable_node` about something else.
+   *
+   * Reports the EXISTING `missing_node` rather than a new error variant: a
+   * declared goto target is an edge reference, so `missing_node`'s own message
+   * ("Edge references non-existent node 'x' (from 'y')") already reads correctly.
+   * Adding a variant would widen a union the library RETURNS, which breaks any
+   * consumer switching exhaustively over `GraphCompileError` — a breaking change
+   * needing a unanimous panel, for a diagnostic distinction nothing consumes.
+   */
+  checkGotoTargets() {
+    for (const [nodeId, node] of this.nodes) {
+      for (const target of node.gotoTargets ?? []) {
+        if (target === END) continue;
+        if (!this.nodes.has(target)) {
+          return { type: "missing_node", nodeId: target, referencedBy: nodeId };
+        }
+      }
+    }
+    return void 0;
+  }
+  /**
+   * BFS from START over static edges PLUS declared `Command.goto` targets
+   * (#5727).
+   *
+   * Goto targets are followed from the DECLARING node, exactly like an edge —
+   * not seeded from START. That distinction is load-bearing: seeding would make
+   * a target declared on an itself-unreachable node read as reachable, so a
+   * whole orphaned subgraph could bless itself and `unreachable_node` would stop
+   * being able to fail for the graphs that use goto.
+   */
   findReachableNodes() {
     const reachable = /* @__PURE__ */ new Set();
     const queue = this.getStartTargets();
@@ -30731,6 +30761,7 @@ var GraphBuilder = class {
       if (current === void 0 || current === END || reachable.has(current)) continue;
       reachable.add(current);
       queue.push(...this.getEdgeTargets(current, reachable));
+      queue.push(...this.nodes.get(current)?.gotoTargets ?? []);
     }
     return reachable;
   }
@@ -46107,7 +46138,7 @@ function buildVotingInput(plan, config) {
 }
 async function executeSingleVote(plan, config, log) {
   try {
-    const { executeVoting } = await import("./consensus-vote-ATICPOXQ.js");
+    const { executeVoting } = await import("./consensus-vote-7GGJYVBF.js");
     const input = buildVotingInput(plan, config);
     const result = await executeVoting(input, log);
     return parseVotingResult(result);
@@ -47902,7 +47933,7 @@ ${contextBlock}`;
       const strategy = config.votingStrategy ?? "higher_order";
       await postProgress(config, "Vote", `Running consensus with ${strategy} strategy...`);
       try {
-        const { executeVoting } = await import("./consensus-vote-ATICPOXQ.js");
+        const { executeVoting } = await import("./consensus-vote-7GGJYVBF.js");
         const votingResult = await executeVoting(
           {
             proposal: buildVoteProposal(plan, research),
@@ -48994,7 +49025,7 @@ function securityFirstPack(files, budget) {
   };
 }
 function packDiffForReview(prDiff, budget) {
-  if (prDiff.length <= budget) {
+  if (byteLen(prDiff) <= budget) {
     return { coverage: void 0, packedDiff: prDiff, note: "" };
   }
   const pack = securityFirstPack(splitByFile(prDiff), budget);
@@ -56643,4 +56674,4 @@ export {
   startPipelineEventBridge,
   shutdownPipelineEventBridge
 };
-//# sourceMappingURL=chunk-P6LLNTKM.js.map
+//# sourceMappingURL=chunk-6GGWO4B3.js.map