fraim 2.0.240 → 2.0.242

package/dist/src/cli/setup/ide-invocation-surfaces.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.CODEX_SKILL_FRONTMATTER = exports.FRAIM_INVOCATION_BODY = exports.FRAIM_DEFERRED_TOOL_PRELOAD = exports.CURSOR_MDC_FRONTMATTER = exports.FRAIM_LAUNCH_PHRASE = void 0;
+exports.CODEX_SKILL_FRONTMATTER = exports.FRAIM_INVOCATION_BODY = exports.FRAIM_MCP_UNAVAILABLE_MANAGER_GUIDANCE = exports.FRAIM_DEFERRED_TOOL_PRELOAD = exports.CURSOR_MDC_FRONTMATTER = exports.FRAIM_LAUNCH_PHRASE = void 0;
 exports.buildFraimInvocationBody = buildFraimInvocationBody;
 exports.buildClaudeSkillContent = buildClaudeSkillContent;
 exports.buildClaudeCommandShimContent = buildClaudeCommandShimContent;
@@ -25,6 +25,12 @@ exports.FRAIM_DEFERRED_TOOL_PRELOAD = [
     'get_fraim_file',
     'seekMentoring'
 ];
+exports.FRAIM_MCP_UNAVAILABLE_MANAGER_GUIDANCE = [
+    'If required FRAIM MCP tools remain unavailable after the deferred-tool preload/retry step, stop the FRAIM job.',
+    'Do not continue from memory, local stubs, cached instructions, or prior context.',
+    'In the blocker sentence, state only the work-focused blocker and what outcome is blocked; do not narrate ToolSearch/tool_search, individual tool names, server internals, or retry mechanics.',
+    'Always help troubleshoot the unavailable FRAIM MCP connection: tell the user to reconnect or enable the FRAIM MCP server in their IDE/agent, restart or refresh the agent session, retry the FRAIM job, and if it still fails run `fraim doctor --test-mcp` and share the failing check output.'
+].join(' ');
 function buildDeferredToolBootstrapSection(profile) {
     if (profile === 'none') {
         return '';
@@ -72,6 +78,7 @@ ${buildDeferredToolBootstrapSection(profile)}1. **Confirm FRAIM activation**:
 5. **Execute**:
    - For jobs, follow the phased instructions and use \`seekMentoring\` when the job requires phase transitions.
    - For skills, apply the skill steps directly to the user's current context.
+   - ${exports.FRAIM_MCP_UNAVAILABLE_MANAGER_GUIDANCE}
 `;
 }
 exports.FRAIM_INVOCATION_BODY = buildFraimInvocationBody();

package/dist/src/cli/utils/agent-adapters.js

@@ -78,6 +78,7 @@ This repository uses FRAIM.
 - For deeper capability detail, call \`get_fraim_file({ path: "skills/<category>/<skill-name>.md" })\` or \`get_fraim_file({ path: "rules/<category>/<rule-name>.md" })\`.
 - Read \`${projectRulesPath}\` if it exists before doing work.
 - When users ask for next step recommendations, use recommend-next-job skill under \`${employeeSkillsPath}/\` to gather context before suggesting jobs.
+- ${ide_invocation_surfaces_1.FRAIM_MCP_UNAVAILABLE_MANAGER_GUIDANCE}
 
 > [!IMPORTANT]
 > **Job stubs are for discovery only.** When a user mentions or references any file under \`${employeeJobsPath}/\` or \`${managerJobsPath}/\`, do NOT attempt to execute the job from the stub content. The stub only shows intent and phase names. Always call \`get_fraim_job({ job: "<job-name>" })\` first to get the full phased instructions before doing any work.
@@ -101,6 +102,7 @@ ${(0, ide_invocation_surfaces_1.buildFraimInvocationBody)('generic-tool-discover
 - When FRAIM routing is active but the job is not exact, use local stubs to identify which job to invoke before fetching full content with FRAIM MCP tools.
 - If no exact or high-confidence job match exists, say that no FRAIM job matches and continue with normal tools or ask one concise clarification.
 - **Job stubs are for discovery only.** Never execute a job from stub content - always call \`get_fraim_job({ job: "<job-name>" })\` first.
+- ${ide_invocation_surfaces_1.FRAIM_MCP_UNAVAILABLE_MANAGER_GUIDANCE}
 `);
     const fraimReadme = `# FRAIM Catalog
 

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.242",
   "description": "FRAIM core CLI and MCP package.",
   "main": "index.js",
   "bin": {