agentxchain 2.155.41 → 2.155.43

package/package.json

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

package/src/lib/dispatch-bundle.js

@@ -494,7 +494,7 @@ function renderPrompt(role, roleId, turn, state, config, root) {
   lines.push('- `summary`: concise description of what you did this turn');
   lines.push('- `decisions[].id`: pattern `DEC-NNN` (increment from previous turn)');
   lines.push('- `decisions[].category`: one of `implementation`, `architecture`, `scope`, `process`, `quality`, `release`');
-  lines.push('- `objections[].id`: pattern `OBJ-NNN`');
+  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`');
   lines.push('- `verification.status`: one of `pass`, `fail`, `skipped`');
   lines.push('- `verification.status: "pass"` is valid only when every `verification.machine_evidence[].exit_code` is `0`');

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

@@ -1048,26 +1048,45 @@ export function normalizeTurnResult(tr, config, context = {}) {
       if (objection === null || typeof objection !== 'object' || Array.isArray(objection)) {
         return objection;
       }
-      const statement = typeof objection.statement === 'string' ? objection.statement.trim() : '';
+
+      let patched = objection;
+
+      // ── BUG-89: normalize invalid objection IDs to OBJ-NNN ──────────
+      const validIdPattern = /^OBJ-\d+$/;
+      const currentId = typeof patched.id === 'string' ? patched.id : '';
+      if (!validIdPattern.test(currentId)) {
+        const normalizedId = `OBJ-${String(index + 1).padStart(3, '0')}`;
+        corrections.push(`objections[${index}].id: rewritten "${currentId || '(missing)'}" → ${normalizedId}`);
+        normalizationEvents.push({
+          field: `objections[${index}].id`,
+          original_value: patched.id ?? null,
+          normalized_value: normalizedId,
+          rationale: 'invalid_objection_id_rewritten',
+        });
+        patched = { ...patched, id: normalizedId };
+      }
+
+      // ── BUG-79: normalize missing statement from summary/detail ─────
+      const statement = typeof patched.statement === 'string' ? patched.statement.trim() : '';
       if (statement) {
-        return objection;
+        return patched;
       }
-      const summary = typeof objection.summary === 'string' ? objection.summary.trim() : '';
-      const detail = typeof objection.detail === 'string' ? objection.detail.trim() : '';
+      const summary = typeof patched.summary === 'string' ? patched.summary.trim() : '';
+      const detail = typeof patched.detail === 'string' ? patched.detail.trim() : '';
       const sourceField = summary ? 'summary' : detail ? 'detail' : null;
       const sourceValue = summary || detail;
       if (!sourceField) {
-        return objection;
+        return patched;
       }
       corrections.push(`objections[${index}].statement: copied from ${sourceField}`);
       normalizationEvents.push({
         field: `objections[${index}].statement`,
-        original_value: objection.statement ?? null,
+        original_value: patched.statement ?? null,
         normalized_value: sourceValue,
         rationale: `copied_from_${sourceField}`,
       });
       return {
-        ...objection,
+        ...patched,
         statement: sourceValue,
       };
     });