agentxchain 2.155.49 → 2.155.51

package/package.json

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

package/src/lib/dispatch-bundle.js

@@ -489,7 +489,7 @@ function renderPrompt(role, roleId, turn, state, config, root) {
   lines.push('### Field Rules');
   lines.push('');
   lines.push('- `schema_version`: always `"1.0"`');
-  lines.push('- `run_id`, `turn_id`, `role`, `runtime_id`: must match the values above exactly');
+  lines.push('- `run_id`, `turn_id`, `role`, `runtime_id`: must match the current assignment values above exactly. Do NOT copy `run_id` from old reports, history entries, previous dispatch bundles, or retained staging JSON.');
   lines.push('- `status`: one of `completed`, `blocked`, `needs_human`, `failed`. Do NOT use `complete`, `success`, `done`, or any other synonym — use the exact enum value `completed`.');
   lines.push('- `decisions`: REQUIRED array. Use `[]` when no new decisions were made; do not omit the field.');
   lines.push('- `objections`: REQUIRED array. Use `[]` when no objections are raised; review_only roles must include at least one objection.');
@@ -499,6 +499,7 @@ function renderPrompt(role, roleId, turn, state, config, root) {
   lines.push('- `proposed_next_role`: **REQUIRED**. Must be in allowed_next_roles for the current phase, or `"human"`.');
   lines.push('- `decisions[].id`: pattern `DEC-NNN` where NNN is digits only (e.g. `DEC-001`, `DEC-002`). Do NOT use `D1`, `D2`, or freeform IDs.');
   lines.push('- `decisions[].statement`: non-empty string describing the decision. Do NOT use `decision` or `description` as the field name — the field is `statement`.');
+  lines.push('- `decisions[].rationale`: REQUIRED non-empty string explaining why the decision was made. Do NOT omit this field.');
   lines.push('- `decisions[].category`: one of `implementation`, `architecture`, `scope`, `process`, `quality`, `release`');
   lines.push('- `objections[].id`: pattern `OBJ-NNN` where NNN is digits only (e.g. `OBJ-001`, `OBJ-002`). Do NOT append extra suffixes like `-M31` or use non-numeric characters after `OBJ-`.');
   lines.push('- `objections[].severity`: one of `low`, `medium`, `high`, `blocking`');

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

@@ -78,6 +78,9 @@ export function validateStagedTurnResult(root, state, config, opts = {}) {
   const normContext = {};
   if (state) {
     normContext.phase = state.phase;
+    if (state.run_id) {
+      normContext.runId = state.run_id;
+    }
     // Prefer active_turns (the persisted schema field); fall back to the
     // current_turn compatibility alias for callers that pass a state shape
     // built outside loadProjectState() (e.g. raw fixtures). Both surfaces are
@@ -85,6 +88,12 @@ export function validateStagedTurnResult(root, state, config, opts = {}) {
     if (activeTurn) {
       const roleKey = activeTurn.assigned_role || activeTurn.role;
       normContext.assignedRole = roleKey;
+      if (activeTurn.turn_id) {
+        normContext.turnId = activeTurn.turn_id;
+      }
+      if (activeTurn.run_id) {
+        normContext.activeTurnRunId = activeTurn.run_id;
+      }
       if (activeTurn.runtime_id) {
         normContext.runtimeId = activeTurn.runtime_id;
       }
@@ -1006,6 +1015,29 @@ export function normalizeTurnResult(tr, config, context = {}) {
 
   const normalized = { ...tr };
 
+  // ── BUG-97: recover stale run_id for retained active turns ────────────
+  // run_id is state-owned identity. A staged result may safely inherit it
+  // only when the staged turn_id proves it belongs to the active retained
+  // turn and the active-turn/state run identity is coherent.
+  const stagedTurnMatchesActive = Boolean(
+    context.runId
+    && context.turnId
+    && typeof normalized.turn_id === 'string'
+    && normalized.turn_id === context.turnId
+  );
+  const activeTurnRunIdCoherent = !context.activeTurnRunId || context.activeTurnRunId === context.runId;
+  const stagedRunId = typeof normalized.run_id === 'string' ? normalized.run_id.trim() : '';
+  if (stagedTurnMatchesActive && activeTurnRunIdCoherent && stagedRunId !== context.runId) {
+    corrections.push(`run_id: rewritten from "${stagedRunId || '(missing)'}" to active state run_id "${context.runId}"`);
+    normalizationEvents.push({
+      field: 'run_id',
+      original_value: normalized.run_id ?? null,
+      normalized_value: context.runId,
+      rationale: 'run_id_rewritten_from_active_turn_context',
+    });
+    normalized.run_id = context.runId;
+  }
+
   // ── BUG-95: rename synonym fields before variable computation ────────
   // files_modified is an unambiguous synonym for files_changed.
   if (!('files_changed' in normalized) && Array.isArray(normalized.files_modified)) {
@@ -1199,6 +1231,32 @@ export function normalizeTurnResult(tr, config, context = {}) {
         }
       }
 
+      // Normalize missing rationale from existing decision text. Do not invent
+      // rationale when the decision object has no meaningful source material.
+      const rationale = typeof patched.rationale === 'string' ? patched.rationale.trim() : '';
+      if (!rationale) {
+        const rationaleSources = [
+          ['reason', patched.reason],
+          ['why', patched.why],
+          ['description', patched.description],
+          ['decision', patched.decision],
+          ['statement', patched.statement],
+        ];
+        const source = rationaleSources.find(([, value]) => typeof value === 'string' && value.trim());
+        if (source) {
+          const [srcField, srcValue] = source;
+          const alt = srcValue.trim();
+          corrections.push(`decisions[${index}].rationale: copied from ${srcField}`);
+          normalizationEvents.push({
+            field: `decisions[${index}].rationale`,
+            original_value: patched.rationale ?? null,
+            normalized_value: alt,
+            rationale: `copied_from_${srcField}`,
+          });
+          patched = { ...patched, rationale: alt };
+        }
+      }
+
       // Default missing category to 'implementation'
       if (!patched.category || !VALID_CATEGORIES.includes(patched.category)) {
         const defaultCat = 'implementation';