fraim 2.0.240 → 2.0.241

package/dist/src/core/handoff-contracts.js

@@ -152,6 +152,38 @@ function validateDelegationLedger(value) {
     else if (!Array.isArray(obj.tasks)) {
         errors.push('evidence.delegationLedger.tasks must be an array');
     }
+    else {
+        // Issue #1021: validate each task, not just the array's existence. A task naming
+        // no job used to pass here and then be skipped by a bare `continue` in
+        // maybeStartDelegatedChildRuns, so a whole workstream could vanish with nothing
+        // manager-visible. Rejecting at emission lets the agent retry in one round.
+        //
+        // Deliberately NOT required: `taskId`. normalizeDelegationLedger synthesizes one
+        // from the title, so requiring it would reject ledgers the runtime handles fine.
+        // The title requirement accepts `briefing`/`instructions` because the group schema
+        // supplies those instead — this mirrors exactly what the normalizer needs to keep
+        // a task, so the contract can no longer be weaker than the parser.
+        obj.tasks.forEach((task, index) => {
+            const label = `evidence.delegationLedger.tasks[${index}]`;
+            if (!task || typeof task !== 'object' || Array.isArray(task)) {
+                errors.push(`${label} must be an object`);
+                return;
+            }
+            const entry = task;
+            const nonEmpty = (value) => typeof value === 'string' && value.trim().length > 0;
+            if (!nonEmpty(entry.title) && !nonEmpty(entry.briefing) && !nonEmpty(entry.instructions)) {
+                errors.push(`${label} needs a title (or briefing/instructions to derive one from)`);
+            }
+            if (!nonEmpty(entry.jobId) && !nonEmpty(entry.job) && !nonEmpty(entry.job_id)) {
+                errors.push(`${label}.jobId is missing; every delegated task must name an existing FRAIM job`);
+            }
+            const issueNumber = entry.issueNumber ?? entry.issue_number ?? entry.issue;
+            if (issueNumber !== undefined && issueNumber !== null
+                && typeof issueNumber !== 'string' && typeof issueNumber !== 'number') {
+                errors.push(`${label}.issueNumber must be a string or number when present (got ${typeof issueNumber})`);
+            }
+        });
+    }
     return errors.length > 0 ? errors : null;
 }
 // ---------------------------------------------------------------------------

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "fraim",
-  "version": "2.0.240",
+  "version": "2.0.241",
   "description": "FRAIM core CLI and MCP package.",
   "main": "index.js",
   "bin": {