agentxchain 2.91.0 → 2.93.0

package/src/lib/run-loop.js

@@ -217,7 +217,16 @@ async function executeParallelTurns(root, config, state, maxConcurrent, callback
 
     // If selectRole returns a role we already tried (or assigned), try
     // other eligible roles from the routing before giving up.
+    // Exception: when delegation queue is driving resolution, do not fill
+    // extra slots with non-delegation roles via the fallback — those roles
+    // would execute without delegation context and corrupt the lifecycle.
     if (triedRoles.has(roleId)) {
+      const hasPendingDelegations = Array.isArray(state?.delegation_queue) &&
+        state.delegation_queue.some(d => d.status === 'pending' || d.status === 'active');
+      const hasPendingReview = !!state?.pending_delegation_review;
+      if (hasPendingDelegations || hasPendingReview) {
+        break;
+      }
       const phase = state.phase;
       const allowed = config?.routing?.[phase]?.allowed_next_roles || [];
       const alternateFound = allowed.some((alt) => {
@@ -248,6 +257,13 @@ async function executeParallelTurns(root, config, state, maxConcurrent, callback
     turnsToDispatch.push({ turn: assignResult.turn, state: assignResult.state });
     emit({ type: 'turn_assigned', turn: assignResult.turn, role: roleId, state: assignResult.state });
 
+    // Delegation review is a coordination checkpoint — do not fill additional
+    // slots alongside it.  The review must execute alone so it can assess all
+    // delegation results before the run continues.
+    if (assignResult.turn.delegation_review) {
+      break;
+    }
+
     // Reload state after assignment to get accurate active count
     state = loadState(root, config);
     activeCount = getActiveTurnCount(state);

package/src/lib/schemas/turn-result.schema.json

@@ -76,6 +76,16 @@
           "rationale": {
             "type": "string",
             "minLength": 1
+          },
+          "durability": {
+            "type": "string",
+            "enum": ["run", "repo"],
+            "description": "Decision persistence scope. 'run' (default) dies with the run; 'repo' persists across runs as a binding constraint."
+          },
+          "overrides": {
+            "type": "string",
+            "pattern": "^DEC-\\d+$",
+            "description": "ID of an active repo-durable decision this decision supersedes."
           }
         }
       }

package/src/lib/turn-result-validator.js

@@ -325,6 +325,8 @@ function checkPlaceholder(errors, fieldPath, value) {
   }
 }
 
+const VALID_DURABILITIES = ['run', 'repo'];
+
 function validateDecision(dec, index) {
   const errors = [];
   const prefix = `decisions[${index}]`;
@@ -344,6 +346,17 @@ function validateDecision(dec, index) {
   if (typeof dec.rationale !== 'string' || !dec.rationale.trim()) {
     errors.push(`${prefix}.rationale must be a non-empty string.`);
   }
+  if (dec.durability !== undefined && !VALID_DURABILITIES.includes(dec.durability)) {
+    errors.push(`${prefix}.durability must be one of: ${VALID_DURABILITIES.join(', ')}.`);
+  }
+  if (dec.overrides !== undefined) {
+    if (typeof dec.overrides !== 'string' || !/^DEC-\d+$/.test(dec.overrides)) {
+      errors.push(`${prefix}.overrides must match pattern DEC-NNN.`);
+    }
+    if (dec.overrides === dec.id) {
+      errors.push(`${prefix}.overrides cannot reference itself.`);
+    }
+  }
   return errors;
 }