fraim-framework 2.0.160 → 2.0.162

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

@@ -472,6 +472,33 @@ class FraimLocalMCPServer {
         }
         return '';
     }
+    // #533: persist the authenticated user email into ~/.fraim/preferences.json so
+    // the AI Hub (a separate process) can resolve personal learnings + show the real
+    // profile. Merge-and-write; never throw out of the connect path.
+    persistUserEmail(userEmail) {
+        try {
+            const fs = require('fs');
+            const path = require('path');
+            const dir = (0, project_fraim_paths_1.getUserFraimDirPath)();
+            const prefsPath = path.join(dir, 'preferences.json');
+            let prefs = {};
+            try {
+                prefs = JSON.parse(fs.readFileSync(prefsPath, 'utf8'));
+            }
+            catch {
+                prefs = {};
+            }
+            if (prefs.userEmail === userEmail)
+                return; // already current
+            (0, fs_1.mkdirSync)(dir, { recursive: true });
+            prefs.userEmail = userEmail;
+            (0, fs_1.writeFileSync)(prefsPath, JSON.stringify(prefs, null, 2), 'utf8');
+            this.log(`Persisted userEmail to ${prefsPath}`);
+        }
+        catch (e) {
+            this.log(`Could not persist userEmail: ${e instanceof Error ? e.message : String(e)}`);
+        }
+    }
     log(message) {
         // Log to stderr (stdout is reserved for MCP protocol)
         const key = this.apiKey || 'MISSING_API_KEY';
@@ -663,14 +690,28 @@ class FraimLocalMCPServer {
     }
     async performLocalCatalogSync(projectRoot) {
         const { runSync } = await Promise.resolve().then(() => __importStar(require('../cli/commands/sync')));
-        await runSync({
-            projectRoot,
-            skipUpdates: true,
-            local: this.shouldUseLocalSyncTarget(),
-            failHard: 'throw'
+        await this.runWithConsoleLogRedirectedToStderr(async () => {
+            await runSync({
+                projectRoot,
+                skipUpdates: true,
+                local: this.shouldUseLocalSyncTarget(),
+                failHard: 'throw'
+            });
         });
         this.writeLocalCatalogMetadata(projectRoot);
     }
+    async runWithConsoleLogRedirectedToStderr(operation) {
+        const originalLog = console.log;
+        console.log = (...args) => {
+            console.error(...args);
+        };
+        try {
+            return await operation();
+        }
+        finally {
+            console.log = originalLog;
+        }
+    }
     async ensureFreshLocalCatalogOnConnect(requestId) {
         if (process.env.FRAIM_DISABLE_SYNC_ON_CONNECT === '1') {
             this.latestConnectSyncWarning = null;
@@ -1185,6 +1226,11 @@ class FraimLocalMCPServer {
                     const userEmail = emailMatch[1].trim();
                     this.ensureEngine().setUserEmail(userEmail);
                     this.log(`[req:${requestId}] Captured user email for template substitution: ${userEmail}`);
+                    // #533: persist the authenticated email locally so the AI Hub (a separate
+                    // server that never sees fraim_connect) can resolve personal/manager
+                    // learnings and show the real identity in its profile card. Merge into
+                    // the existing preferences.json rather than overwriting it.
+                    this.persistUserEmail(userEmail);
                     // Inject learning context from the local workspace (RFC 177: files live on disk, not server).
                     const workspaceRoot = this.findProjectRoot() || process.cwd();
                     const learningSection = (0, learning_context_builder_js_1.buildLearningContextSection)(workspaceRoot, userEmail, false);
@@ -1192,6 +1238,11 @@ class FraimLocalMCPServer {
                         responseText += learningSection;
                         this.log(`[req:${requestId}] Injected learning context for ${userEmail} from ${workspaceRoot}`);
                     }
+                    const teamContextSection = (0, learning_context_builder_js_1.buildTeamContextSection)(workspaceRoot, false);
+                    if (teamContextSection) {
+                        responseText += teamContextSection;
+                        this.log(`[req:${requestId}] Injected team context from ${workspaceRoot}`);
+                    }
                 }
                 if (this.latestConnectSyncWarning) {
                     responseText += `\n\n## Local Catalog\n${this.latestConnectSyncWarning}`;
@@ -1206,9 +1257,10 @@ class FraimLocalMCPServer {
             if (typeof text === 'string' && userEmail) {
                 const workspaceRoot = this.findProjectRoot() || process.cwd();
                 const learningSection = (0, learning_context_builder_js_1.buildLearningContextSection)(workspaceRoot, userEmail, true);
-                if (learningSection) {
-                    finalizedResponse.result.content[0].text = text + `\n\n---` + learningSection;
-                    this.log(`[req:${requestId}] Injected job-focus learning context for ${userEmail}`);
+                const teamContextSection = (0, learning_context_builder_js_1.buildTeamContextSection)(workspaceRoot, true);
+                if (learningSection || teamContextSection) {
+                    finalizedResponse.result.content[0].text = text + `\n\n---` + learningSection + teamContextSection;
+                    this.log(`[req:${requestId}] Injected job-focus learning/team context for ${userEmail}`);
                 }
             }
         }
@@ -1863,7 +1915,7 @@ class FraimLocalMCPServer {
                     //
                     // When a QUALITY_PRODUCING_JOBS job reaches its final phase
                     // (nextPhase === null) with status === 'complete', the agent MUST
-                    // include a valid `evidence.quality` object. If invalid, swap the
+                    // include `evidence.artifactPath` and a valid `evidence.quality` object. If invalid, swap the
                     // accomplishment message for a rejection and DO NOT emit. If valid,
                     // fire-and-forget POST to /api/analytics/quality-score so the row
                     // lands in fraim_quality_scores.
@@ -1871,8 +1923,13 @@ class FraimLocalMCPServer {
                     const isFinalCompletion = args.status === 'complete' &&
                         (tutoringResponse.nextPhase === null || tutoringResponse.nextPhase === undefined);
                     if (isQualityJob && isFinalCompletion) {
-                        const qualityErrors = (0, quality_evidence_1.validateQualityEvidence)(args.evidence?.quality, args.jobName);
-                        if (qualityErrors) {
+                        const qualityErrors = [
+                            ...((0, quality_evidence_1.validateQualityEvidence)(args.evidence?.quality, args.jobName) || []),
+                            ...(typeof args.evidence?.artifactPath === 'string' && args.evidence.artifactPath.trim()
+                                ? []
+                                : ['evidence.artifactPath is required'])
+                        ];
+                        if (qualityErrors.length > 0) {
                             this.log(`❌ Quality enforcement rejected ${args.jobName} completion: ${qualityErrors.join('; ')}`);
                             const rejection = (0, quality_evidence_1.buildQualityRejectionMessage)(args.jobName, args.currentPhase, qualityErrors);
                             return await this.finalizeLocalToolTextResponse(request, requestSessionId, requestId, rejection);
@@ -1886,7 +1943,7 @@ class FraimLocalMCPServer {
                             jobId: args.jobId,
                             sessionId: requestSessionId || args.sessionId || 'unknown',
                             quality: args.evidence.quality,
-                            artifactPath: args.evidence.quality.artifactPath,
+                            artifactPath: args.evidence?.artifactPath,
                             repoIdentifier: args.evidence?.reviewContext?.repoIdentifier || this.repoInfo?.url,
                             reviewContext: {
                                 ...(args.evidence?.reviewContext || {}),
@@ -1923,9 +1980,10 @@ class FraimLocalMCPServer {
                             if (userEmail) {
                                 const workspaceRoot = this.findProjectRoot() || process.cwd();
                                 const learningSection = (0, learning_context_builder_js_1.buildLearningContextSection)(workspaceRoot, userEmail, true);
-                                if (learningSection) {
-                                    responseText += `\n\n---` + learningSection;
-                                    this.log(`✅ Injected job-focus learning context for ${userEmail} (local override path)`);
+                                const teamContextSection = (0, learning_context_builder_js_1.buildTeamContextSection)(workspaceRoot, true);
+                                if (learningSection || teamContextSection) {
+                                    responseText += `\n\n---` + learningSection + teamContextSection;
+                                    this.log(`✅ Injected job-focus learning/team context for ${userEmail} (local override path)`);
                                 }
                             }
                             return await this.finalizeLocalToolTextResponse(request, requestSessionId, requestId, responseText);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "fraim-framework",
-  "version": "2.0.160",
+  "version": "2.0.162",
   "description": "FRAIM: AI Workforce Infrastructure — the organizational capability that turns AI agents into an accountable workforce, their operators into capable AI managers, and executives into leaders with clear optics on AI proficiency.",
   "main": "index.js",
   "bin": {
@@ -9,12 +9,13 @@
   "scripts": {
     "dev": "tsx --watch src/fraim-mcp-server.ts > server.log 2>&1",
     "dev:prod": "npm run build && node dist/src/fraim-mcp-server.js > server.log 2>&1",
-    "build": "tsx scripts/build-fraim-config-schema-template.ts && npm run typecheck:scripts && tsc && npm run build:stubs && npm run build:fraim-brain && node scripts/copy-registry.js && npm run validate:registry && npm run validate:fraim-pro-assets && npm run validate:employee-catalog && tsx scripts/validate-purity.ts",
+    "build": "tsx scripts/build-fraim-config-schema-template.ts && npm run typecheck:scripts && tsc && npm run build:stubs && npm run build:fraim-brain && node scripts/copy-registry.js && npm run validate:registry && npm run validate:fraim-pro-assets && npm run validate:employee-catalog && npm run validate:learning-format-contract && tsx scripts/validate-purity.ts",
+    "validate:learning-format-contract": "tsx scripts/validate-learning-format-contract.ts",
     "build:stubs": "tsx scripts/build-stub-registry.ts",
     "build:fraim-brain": "node scripts/generate-fraim-brain.js",
-    "test-all": "npm run test && npm run test:isolated && npm run test:ui",
+    "test-all": "npm run test && npm run test:isolated tests/isolated/test-*.ts && npm run test:ui",
     "test": "node scripts/test-with-server.js",
-    "test:isolated": "npx tsx --test --test-concurrency=1 --test-reporter=spec tests/isolated/test-*.ts",
+    "test:isolated": "npx tsx --test --test-concurrency=1 --test-reporter=spec ",
     "test:smoke": "node scripts/test-with-server.js --tags=smoke",
     "test:coverage": "node scripts/test-with-server.js --tags=smoke --coverage",
     "test:stripe": "node scripts/test-with-server.js tests/test-stripe-payment-complete.ts",