agentxchain 2.155.31 → 2.155.33

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agentxchain",
-  "version": "2.155.31",
+  "version": "2.155.33",
   "description": "CLI for AgentXchain — governed multi-agent software delivery",
   "type": "module",
   "bin": {

package/src/lib/governed-state.js

@@ -4701,29 +4701,51 @@ function _acceptGovernedTurnLocked(root, config, opts) {
 
       const gateSemanticMode = config.gate_semantic_coverage_mode || 'strict';
       if (uncoveredFiles.length > 0 && gateSemanticMode === 'strict') {
-        const coverageError = `Gate "${exitGateId}" is failing on ${uncoveredFiles.join(', ')}. Your turn did not modify ${uncoveredFiles.length === 1 ? 'that file' : 'those files'}. Either edit the file(s) to satisfy the gate, or remove the phase transition request.`;
-        transitionToFailedAcceptance(root, state, currentTurn, coverageError, {
-          error_code: 'gate_semantic_coverage',
-          stage: 'gate_semantic_coverage',
-          extra: {
-            gate_id: exitGateId,
-            uncovered_files: uncoveredFiles,
-            declared_files: [...declaredFiles],
-            gate_reasons: preGateResult.reasons,
-          },
-        });
-        return {
-          ok: false,
-          error: coverageError,
-          validation: {
-            ...validation,
-            ok: false,
+        // BUG-81: In planning phase only, auto-strip the phase transition request
+        // instead of permanently blocking.  PM's partial planning work is preserved;
+        // the phase stays in "planning" and the continuous loop re-dispatches PM to
+        // complete gate artifacts.  In implementation+ phases, gate failures still
+        // reject — dev/qa claiming readiness without updating gate files is a real error.
+        if (turnResult.phase_transition_request && state.phase === 'planning') {
+          emitRunEvent(root, 'gate_transition_request_auto_stripped', {
+            run_id: state.run_id,
+            phase: state.phase,
+            status: state.status,
+            turn: { turn_id: currentTurn.turn_id, role_id: currentTurn.assigned_role },
+            payload: {
+              gate_id: exitGateId,
+              uncovered_files: uncoveredFiles,
+              declared_files: [...declaredFiles],
+              rationale: 'Turn did not modify gate-required files; phase transition stripped to preserve partial planning work',
+            },
+          });
+          delete turnResult.phase_transition_request;
+          // Fall through — continue with acceptance without the phase transition
+        } else {
+          const coverageError = `Gate "${exitGateId}" is failing on ${uncoveredFiles.join(', ')}. Your turn did not modify ${uncoveredFiles.length === 1 ? 'that file' : 'those files'}. Either edit the file(s) to satisfy the gate, or remove the phase transition request.`;
+          transitionToFailedAcceptance(root, state, currentTurn, coverageError, {
+            error_code: 'gate_semantic_coverage',
             stage: 'gate_semantic_coverage',
-            error_class: 'gate_coverage_error',
-            errors: uncoveredFiles.map(f => `Gate "${exitGateId}" is failing on "${f}". Your turn did not modify that file.`),
-            warnings: [],
-          },
-        };
+            extra: {
+              gate_id: exitGateId,
+              uncovered_files: uncoveredFiles,
+              declared_files: [...declaredFiles],
+              gate_reasons: preGateResult.reasons,
+            },
+          });
+          return {
+            ok: false,
+            error: coverageError,
+            validation: {
+              ...validation,
+              ok: false,
+              stage: 'gate_semantic_coverage',
+              error_class: 'gate_coverage_error',
+              errors: uncoveredFiles.map(f => `Gate "${exitGateId}" is failing on "${f}". Your turn did not modify that file.`),
+              warnings: [],
+            },
+          };
+        }
       }
     }
   }

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

@@ -1274,6 +1274,34 @@ export function normalizeTurnResult(tr, config, context = {}) {
       );
       normalized.proposed_next_role = fallback;
     }
+  } else if (
+    // BUG-82: When BUG-81's gate auto-strip keeps the session in an earlier
+    // phase (e.g. planning), authoritative roles dispatched in that phase may
+    // propose a next role valid for the phase they *expected* (e.g. "qa" for
+    // implementation) but illegal for the actual phase.  Auto-normalize to a
+    // routing-legal role instead of hard-erroring.
+    isKnownPhase &&
+    !isReviewOnly &&
+    normalized.status === 'completed' &&
+    typeof normalized.proposed_next_role === 'string' &&
+    normalized.proposed_next_role !== 'human' &&
+    !/^<[^>]+>$/.test(normalized.proposed_next_role) && // skip template placeholders — let schema validation catch them
+    allowedNextRoles.length > 0 &&
+    !allowedNextRoles.includes(normalized.proposed_next_role)
+  ) {
+    const fallback = pickAllowedRoleFallback();
+    if (fallback) {
+      corrections.push(
+        `proposed_next_role: corrected "${normalized.proposed_next_role}" to allowed role "${fallback}" (routing-illegal for phase "${currentPhase}")`
+      );
+      normalizationEvents.push({
+        field: 'proposed_next_role',
+        original_value: normalized.proposed_next_role,
+        normalized_value: fallback,
+        rationale: `routing_illegal_for_phase_${currentPhase}`,
+      });
+      normalized.proposed_next_role = fallback;
+    }
   }
 
   return { normalized, corrections, normalizationEvents };