fraim 2.0.286 → 2.0.288

package/dist/src/cli/doctor/checks/agent-cli-health-checks.js

@@ -97,6 +97,27 @@ async function runAgentCliHealthCheck(cli) {
         : managedPath === ambientPath
             ? ambientVersion
             : probeVersion(managedPath);
+    // Issue #1412: a resolved file that fails to execute at all (e.g. an
+    // interrupted update left a non-executable fallback launcher) is a harder
+    // failure than drift — `getSystemCommandPath()` only proves the file
+    // exists, not that it can run. Without this check, a single broken install
+    // with nothing else on PATH to disagree with (ambientPath === managedPath,
+    // both versions null) falls straight through the mismatch check below and
+    // reads as "consistent". Check this before computing npm-global/drift
+    // details that don't matter once the CLI can't run at all.
+    const brokenPath = ambientPath && !ambientVersion
+        ? ambientPath
+        : managedPath && !managedVersion
+            ? managedPath
+            : null;
+    if (brokenPath) {
+        return {
+            status: 'error',
+            message: `${cli.label} is installed at ${brokenPath} but failed to run (\`${cli.command} --version\` produced no output). An interrupted or partial update may have left a non-functional launcher.`,
+            suggestion: `Reinstall ${cli.label}, then run "${cli.command} --version" again to confirm it's fixed.`,
+            details: { ambientPath, ambientVersion, managedPath, managedVersion },
+        };
+    }
     const npmGlobalBinDirs = (0, managed_agent_paths_1.resolveNpmGlobalBinDirs)();
     const npmGlobalPath = npmGlobalBinDirs.length > 0
         ? (0, command_resolution_1.getSystemCommandPath)(cli.command, npmGlobalBinDirs.join(path_1.default.delimiter))

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

@@ -29,7 +29,10 @@ 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.'
+    '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.',
+    'Exception — troubleshooting FRAIM itself: if the user\'s request is to diagnose or fix a FRAIM issue and FRAIM MCP is unavailable, do NOT stop.',
+    'Instead, read `~/.fraim/docs/TROUBLESHOOTING.md` directly from disk (it is always synced in full by `fraim sync` and is available without MCP) and follow its guidance to diagnose the issue.',
+    'After completing the local diagnosis, prompt the user to re-run `fraim sync` and restart their agent session, then offer to run the `troubleshoot-fraim` FRAIM job for a structured investigation once MCP is restored.'
 ].join(' ');
 function buildDeferredToolBootstrapSection(profile) {
     if (profile === 'none') {

package/dist/src/config/persona-capability-bundles.js

@@ -291,10 +291,20 @@ for (const bundle of Object.values(exports.PERSONA_CAPABILITY_BUNDLES)) {
 // named specialist persona. They resolve through ownership (never short-circuited
 // as "free") so the Hub attributes them to FRAIMworker, but they are never
 // hire-gated because FRAIMworker is not a purchasable persona.
+//
+// Issue #1398: manager-agreements / organization-onboarding / organizational-learning-
+// synthesis are Manager/Company area-level jobs, not tied to a named specialist — they
+// were falling through to DEFAULT_UNASSIGNED_PERSONA_KEY ('mandy'), which misattributed
+// them to an employee who never runs them and left them outside the Manager/Company
+// employee rail's FRAIMworker group.
 const GENERIC_WORKER_OWNED_JOBS = new Set([
     'contribute-to-fraim',
     'file-fraim-issue',
     'praise-fraim',
+    'troubleshoot-fraim',
+    'manager-agreements',
+    'organization-onboarding',
+    'organizational-learning-synthesis',
 ]);
 function getPersonaCapabilityBundle(personaKey) {
     return exports.PERSONA_CAPABILITY_BUNDLES[personaKey];

package/dist/src/core/fraim-config-schema.generated.js

@@ -603,6 +603,32 @@ exports.FRAIM_CONFIG_SCHEMA = {
                     }
                 }
             }
+        },
+        "artifact_retention_period": {
+            "kind": "object",
+            "properties": {
+                "default": {
+                    "kind": "number"
+                },
+                "learning_archive": {
+                    "kind": "number"
+                },
+                "retrospectives": {
+                    "kind": "number"
+                },
+                "evidence": {
+                    "kind": "number"
+                },
+                "feedback": {
+                    "kind": "number"
+                },
+                "cleanup_manifests": {
+                    "kind": "number"
+                },
+                "learning_usage": {
+                    "kind": "number"
+                }
+            }
         }
     },
     "required": true
@@ -734,5 +760,13 @@ exports.SUPPORTED_FRAIM_CONFIG_PATHS = [
     "automation.support.communication.deliveryMode",
     "automation.support.communication.recipientField",
     "automation.support.communication.includeTemporaryPassword",
-    "automation.support.communication.messageTemplate"
+    "automation.support.communication.messageTemplate",
+    "artifact_retention_period",
+    "artifact_retention_period.default",
+    "artifact_retention_period.learning_archive",
+    "artifact_retention_period.retrospectives",
+    "artifact_retention_period.evidence",
+    "artifact_retention_period.feedback",
+    "artifact_retention_period.cleanup_manifests",
+    "artifact_retention_period.learning_usage"
 ];

package/dist/src/fraim/db-service.js

@@ -170,6 +170,8 @@ class FraimDbService {
         // Issue #563 — shared organization context (FRAIM-cloud backend).
         this.orgArtifactsCollection = this.db.collection('fraim_org_artifacts');
         this.orgAuditCollection = this.db.collection('fraim_org_audit');
+        // Issue #1345 — job execution modes (Coached/Trusted).
+        this.jobExecutionModesCollection = this.db.collection('fraim_job_execution_modes');
     }
     async initializeIndexes() {
         if (!this.db)
@@ -199,6 +201,7 @@ class FraimDbService {
         // requirement — swallow failures like the other Cosmos-sensitive indexes above.
         await this.orgArtifactsCollection.createIndex({ orgId: 1, relativePath: 1 }, { unique: true }).catch(() => { });
         await this.orgAuditCollection.createIndex({ orgId: 1, at: -1 }).catch(() => { });
+        await this.jobExecutionModesCollection.createIndex({ userId: 1, jobName: 1 }, { unique: true }).catch(() => { });
         await this.pendingVerificationsCollection.createIndex({ email: 1 });
         // Compound index covers `findOne({email}, { sort: { createdAt: -1 } })` —
         // the request-access flow's lookup of the most-recent pending row per
@@ -286,6 +289,36 @@ class FraimDbService {
             throw new Error('DB not connected');
         return await this.orgAuditCollection.find({ orgId }).sort({ at: -1 }).toArray();
     }
+    async getJobExecutionMode(userId, jobName) {
+        if (!this.jobExecutionModesCollection)
+            return null;
+        return await this.jobExecutionModesCollection.findOne({ userId, jobName }) ?? null;
+    }
+    async upsertJobExecutionMode(userId, update) {
+        if (!this.jobExecutionModesCollection)
+            return;
+        const jobName = String(update['jobName'] ?? '');
+        if (!jobName)
+            return;
+        const existing = await this.jobExecutionModesCollection.findOne({ userId, jobName });
+        let completedRuns = typeof existing?.completedRuns === 'number' ? existing.completedRuns : 0;
+        if (update['incrementRun'] === true)
+            completedRuns += 1;
+        const record = {
+            userId,
+            jobName,
+            mode: (update['mode'] === 'trusted' || update['mode'] === 'coached')
+                ? update['mode']
+                : (existing?.mode ?? 'coached'),
+            completedRuns,
+            trustedSince: typeof update['trustedSince'] === 'string' ? update['trustedSince'] : existing?.trustedSince,
+            trustedAtRun: typeof update['trustedAtRun'] === 'number' ? update['trustedAtRun'] : existing?.trustedAtRun,
+            lastGraduationOfferRun: typeof update['lastGraduationOfferRun'] === 'number' ? update['lastGraduationOfferRun'] : existing?.lastGraduationOfferRun,
+            graduationSuppressedUntilRun: typeof update['graduationSuppressedUntilRun'] === 'number' ? update['graduationSuppressedUntilRun'] : existing?.graduationSuppressedUntilRun,
+            updatedAt: new Date(),
+        };
+        await this.jobExecutionModesCollection.replaceOne({ userId, jobName }, record, { upsert: true });
+    }
     async verifyApiKey(key) {
         if (!this.keysCollection)
             throw new Error('DB not connected');

package/dist/src/local-mcp-server/stdio-server.js

@@ -52,6 +52,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 };
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.FraimLocalMCPServer = exports.FraimTemplateEngine = void 0;
+exports.parseExecutionModeFromResponse = parseExecutionModeFromResponse;
 const fs_1 = require("fs");
 const path_1 = require("path");
 const os_1 = require("os");
@@ -425,6 +426,29 @@ FraimTemplateEngine.ISSUE_ACTIONS = new Set([
     'close_issue',
     'list_issues'
 ]);
+function parseExecutionModeFromResponse(text) {
+    const marker = '**Execution Mode Context:**';
+    const idx = text.indexOf(marker);
+    if (idx < 0)
+        return null;
+    const jsonStart = text.indexOf('```json', idx);
+    const jsonEnd = text.indexOf('```', jsonStart + 7);
+    if (jsonStart < 0 || jsonEnd < 0)
+        return null;
+    try {
+        const parsed = JSON.parse(text.slice(jsonStart + 7, jsonEnd).trim());
+        const em = parsed.executionMode;
+        if (!em || typeof em.mode !== 'string')
+            return null;
+        return {
+            mode: em.mode === 'trusted' ? 'trusted' : 'coached',
+            completedRuns: typeof em.completedRuns === 'number' ? em.completedRuns : 0,
+        };
+    }
+    catch {
+        return null;
+    }
+}
 class FraimLocalMCPServer {
     constructor(writer) {
         this.config = null;
@@ -1395,6 +1419,18 @@ class FraimLocalMCPServer {
                 }
                 this.recordUsageOffers(workspaceRoot, userEmail, true, jobDomain, String(args.job ?? 'unknown'), requestId);
             }
+            // Issue #1345: forward executionMode from get_fraim_job response to Hub so
+            // the chip reads from run data (no local file read, no separate endpoint needed).
+            const responseText = finalizedResponse.result?.content?.[0]?.text;
+            if (typeof responseText === 'string' && requestSessionId) {
+                const hubBase = process.env.FRAIM_HUB_BASE_URL;
+                const em = parseExecutionModeFromResponse(responseText);
+                if (hubBase && em) {
+                    axios_1.default.post(`${hubBase}/api/ai-hub/runs/by-session/${encodeURIComponent(requestSessionId)}/execution-mode`, em, { headers: { 'Content-Type': 'application/json' } }).catch((err) => {
+                        this.log(`[req:${requestId}] execution-mode forward failed: ${err instanceof Error ? err.message : String(err)}`);
+                    });
+                }
+            }
         }
         return this.processResponseWithHydration(finalizedResponse, requestSessionId);
     }

package/dist/src/services/mcp-service.js

@@ -34,6 +34,7 @@ var __importStar = (this && this.__importStar) || (function () {
 })();
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.McpService = exports.REQUIRED_QUALITY_FIELDS = exports.QUALITY_SCORE_JOBS = exports.QUALITY_PRODUCING_JOBS = exports.DEFAULT_LAUNCH_PHRASE_MAPPINGS = void 0;
+exports.buildTrainingModeBlock = buildTrainingModeBlock;
 const fs_1 = require("fs");
 const crypto_1 = require("crypto");
 const job_parser_1 = require("../core/utils/job-parser");
@@ -60,6 +61,11 @@ exports.DEFAULT_LAUNCH_PHRASE_MAPPINGS = {
 exports.QUALITY_PRODUCING_JOBS = quality_evidence_1.QUALITY_PRODUCING_JOBS;
 exports.QUALITY_SCORE_JOBS = quality_evidence_1.QUALITY_SCORE_JOBS;
 exports.REQUIRED_QUALITY_FIELDS = quality_evidence_1.REQUIRED_QUALITY_FIELDS;
+function buildTrainingModeBlock(mode) {
+    if (mode === 'trusted')
+        return '';
+    return `\n\n---\n\n**Training Mode Active (In Training):** Apply these instructions throughout all phases:\n\n1. **Outline checkpoint** (phases with 2 or more output components): Before drafting any content, emit a concise outline listing all components and counts. Wait for manager confirmation before drafting. Do not call \`seekMentoring\` for this pause.\n\n2. **Phase direction check** (all phases): After completing a phase's substantive work, write one sentence summarizing what was done and ask "Does this look right before I move to [next phase]?" Wait for manager approval before calling \`seekMentoring\`.\n\n3. **Training notes**: After each approved phase, append a structured note to the retrospective draft: phase name, what was done, manager preferences observed, questions resolved.`;
+}
 class McpService {
     constructor(registryService, sessionManager, aiMentor, dbService, analyticsServiceOrVersion, serverVersion) {
         this.registryService = registryService;
@@ -381,6 +387,26 @@ class McpService {
         if (!result.isSimple) {
             response += `\n\n---\n\n**Job ID:** \`${jobId}\`\n\n**This job has phases.** Use \`seekMentoring\` with the jobId above to get phase-specific instructions.`;
         }
+        // Issue #1345: append executionMode and inject training instructions when coached.
+        if (effectiveUserId) {
+            try {
+                const modeRecord = await this.dbService.getJobExecutionMode(effectiveUserId, jobName);
+                const mode = modeRecord?.mode ?? 'coached';
+                const executionMode = {
+                    mode,
+                    completedRuns: modeRecord?.completedRuns ?? 0,
+                };
+                if (modeRecord?.lastGraduationOfferRun != null)
+                    executionMode['lastGraduationOfferRun'] = modeRecord.lastGraduationOfferRun;
+                if (modeRecord?.graduationSuppressedUntilRun != null)
+                    executionMode['graduationSuppressedUntilRun'] = modeRecord.graduationSuppressedUntilRun;
+                response += `\n\n---\n\n**Execution Mode Context:** \`\`\`json\n${JSON.stringify({ executionMode })}\n\`\`\``;
+                response += buildTrainingModeBlock(mode);
+            }
+            catch (e) {
+                console.error(`❌ McpService: Failed to fetch executionMode for ${jobName}:`, e);
+            }
+        }
         return {
             content: [{
                     type: 'text',
@@ -737,6 +763,16 @@ class McpService {
                 console.error(`❌ McpService: Failed to log mentoring/completion:`, e);
             }
         }
+        // Issue #1345: persist executionModeUpdate from findings when present.
+        const executionModeUpdate = args.findings?.executionModeUpdate;
+        if (userId && executionModeUpdate && typeof executionModeUpdate['jobName'] === 'string') {
+            try {
+                await this.dbService.upsertJobExecutionMode(userId, executionModeUpdate);
+            }
+            catch (e) {
+                console.error(`❌ McpService: Failed to upsert executionMode:`, e);
+            }
+        }
         return {
             content: [{
                     type: 'text',

package/package.json

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