fraim-hub 2.0.263 → 2.0.264

package/dist/src/ai-hub/restart-recovery-policy.js

@@ -57,7 +57,11 @@ class RestartRecoveryPolicy {
         if (conversation.status !== 'running')
             return { action: 'skip', reason: 'not_running' };
         const pauseReason = typeof conversation.pauseReason === 'string' ? conversation.pauseReason : '';
-        if (['stopped', 'done', 'awaiting_review', 'awaiting_user', 'error'].includes(pauseReason)) {
+        // Issue #1076: `error` only means terminal failure when the durable status
+        // has actually left running. A running conversation can carry a stale
+        // pauseReason from an earlier transient tool/process error, and restart
+        // recovery must reattach it instead of promoting that stale field to Failed.
+        if (['stopped', 'done', 'awaiting_review', 'awaiting_user'].includes(pauseReason)) {
             return { action: 'skip', reason: `pause_${pauseReason}` };
         }
         if (!conversation.sessionId || typeof conversation.sessionId !== 'string' || !conversation.sessionId.trim()) {

package/dist/src/ai-hub/server.js

@@ -3665,8 +3665,10 @@ class AiHubServer {
                 if (scope !== 'org' && !userEmail) {
                     return res.json({ scope, level, entries: [] });
                 }
-                const entries = (0, learning_context_builder_1.readPreservedLearnings)(resolveProjectPath(req), userEmail, scope, level);
-                return res.json({ scope, level, entries });
+                const projectPath = resolveProjectPath(req);
+                const entries = (0, learning_context_builder_1.readPreservedLearnings)(projectPath, userEmail, scope, level);
+                const threshold = (0, learning_context_builder_1.getScoreThreshold)(projectPath);
+                return res.json({ scope, level, entries, threshold });
             }
             catch (error) {
                 return res.status(500).json({ error: error instanceof Error ? error.message : 'Could not read learnings.' });

package/dist/src/local-mcp-server/learning-context-builder.js

@@ -8,6 +8,7 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.CATEGORY_TO_FILETYPE = exports.BRAIN_LEARNING_FILETYPE_TO_REGION = exports.LEARNING_PRIORITIES = void 0;
 exports.learningEntryHeadingRegex = learningEntryHeadingRegex;
+exports.getScoreThreshold = getScoreThreshold;
 exports.computeEffectiveScore = computeEffectiveScore;
 exports.mergeLearningEntriesAcrossTiers = mergeLearningEntriesAcrossTiers;
 exports.buildLearningContextSection = buildLearningContextSection;
@@ -1195,9 +1196,14 @@ function listDomainLearningFiles(dir, prefix, fileType) {
     return out;
 }
 // Parse the `## [P-…] Title` entries (and their body prose) out of one learning
-// file. Score/Last seen/Recurrences metadata lines are dropped — the Hub shows
-// the human-readable learning, not the bookkeeping.
-function parseLearningEntries(filePath, displayPath, category, level, domain = 'global', tier) {
+// file. Score metadata (Last seen / Recurrences) is captured to compute the
+// effective decayed score and the injected flag; those bookkeeping lines are
+// still excluded from the rendered body.
+//
+// threshold: entries with score >= threshold (or in a non-gated category) are
+//   marked injected:true. Callers that don't need scoring pass Infinity so
+//   all entries are injected.
+function parseLearningEntries(filePath, displayPath, category, level, domain = 'global', tier, threshold = DEFAULT_THRESHOLD) {
     if (!(0, fs_1.existsSync)(filePath))
         return [];
     let content;
@@ -1207,23 +1213,52 @@ function parseLearningEntries(filePath, displayPath, category, level, domain = '
     catch {
         return [];
     }
+    const fileType = exports.CATEGORY_TO_FILETYPE[category];
+    // preferences and manager-coaching are threshold-free — always injected.
+    const isScoreGated = fileType === 'mistake-patterns' || fileType === 'validated-patterns';
     const out = [];
     let current = null;
     let bodyLines = [];
+    // Scoring metadata for the current entry.
+    let curLastSeen = null;
+    let curRecurrences = 1;
     const flush = () => {
         if (current) {
             current.body = bodyLines.join('\n').replace(/\n{3,}/g, '\n\n').trim();
+            if (isScoreGated) {
+                if (curLastSeen !== null) {
+                    const s = computeEffectiveScore(current.severity, curLastSeen, curRecurrences, fileType);
+                    current.score = s;
+                    current.injected = s >= threshold;
+                }
+                else {
+                    current.score = null;
+                    // No date metadata: treat as fresh / always inject.
+                    current.injected = true;
+                }
+            }
+            else {
+                current.score = null;
+                current.injected = true;
+            }
             out.push(current);
         }
         current = null;
         bodyLines = [];
+        curLastSeen = null;
+        curRecurrences = 1;
     };
     const headingRe = learningEntryHeadingRegex();
     for (const line of content.split(/\r?\n/)) {
         const header = line.match(headingRe);
         if (header) {
             flush();
-            const entry = { severity: header[1], title: header[2].trim(), body: '', source: displayPath, category, level, domain };
+            const entry = {
+                severity: header[1],
+                title: header[2].trim(),
+                body: '', source: displayPath, category, level, domain,
+                score: null, injected: true,
+            };
             if (tier !== undefined)
                 entry.tier = tier;
             current = entry;
@@ -1235,10 +1270,19 @@ function parseLearningEntries(filePath, displayPath, category, level, domain = '
             flush();
             continue;
         }
-        // Drop the bookkeeping lines — the Hub shows the human-readable learning,
-        // not the scoring metadata.
         const t = line.trim();
-        if (/^\*\*(Score|Last seen|Recurrences|Technical trace|Users|First synthesized)\*\*:/i.test(t))
+        // Capture scoring metadata but exclude from body.
+        const lastSeenMatch = t.match(/^\*\*Last seen\*\*:\s*(.+)/i);
+        if (lastSeenMatch) {
+            curLastSeen = lastSeenMatch[1].trim();
+            continue;
+        }
+        const recurrencesMatch = t.match(/^\*\*Recurrences\*\*:\s*(\d+)/i);
+        if (recurrencesMatch) {
+            curRecurrences = parseInt(recurrencesMatch[1], 10) || 1;
+            continue;
+        }
+        if (/^\*\*(Score|Technical trace|Users|First synthesized)\*\*:/i.test(t))
             continue;
         if (/^(First|Last) synthesized:/i.test(t))
             continue;
@@ -1247,44 +1291,85 @@ function parseLearningEntries(filePath, displayPath, category, level, domain = '
     flush();
     return out;
 }
-function levelDir(roots, level) {
-    if (level === 'machine') {
-        // Issue #1070: use the configured manager home, not the portable authoring base.
-        // For single-machine backends managerHomeBase == globalPersonalBase (no change).
-        return { dir: roots.managerHomeBase, displayBase: roots.managerHomeDisplayBase.replace(/\/$/, '') };
-    }
-    return { dir: roots.repoLearningsBase, displayBase: REPO_LEARNINGS_REL };
-}
 function readPreservedLearnings(workspaceRoot, userId, scope, level = 'machine') {
     const roots = getLearningRoots(workspaceRoot);
+    const threshold = getScoreThreshold(workspaceRoot);
     const out = [];
+    // Helper: parse + merge a list of tier paths for one file family.
+    // ResolvedLearningTier uses 'manager' to mean machine-level personal; map it to 'machine'.
+    const tierLevel = (t) => t.level === 'manager' ? 'machine' : t.level;
+    const parseTiers = (tiers, cat, tierVal, domainVal) => mergeLearningEntriesAcrossTiers(tiers.map((t) => ({
+        entries: parseLearningEntries(t.path, t.displayPath, cat, tierLevel(t), domainVal ?? 'global', tierVal, threshold),
+    })));
     if (scope === 'org') {
         for (const cat of ['avoid', 'preference', 'repeat', 'coaching']) {
             const fileType = exports.CATEGORY_TO_FILETYPE[cat];
             const f = `org-${fileType}.md`;
-            out.push(...parseLearningEntries((0, path_1.join)(roots.repoLearningsBase, f), `${REPO_LEARNINGS_REL}/${f}`, cat, 'org'));
-            // #806: domain-scoped org learnings alongside the flat global file.
-            for (const { fileName, domain } of listDomainLearningFiles(roots.repoLearningsBase, 'org', fileType)) {
-                out.push(...parseLearningEntries((0, path_1.join)(roots.repoLearningsBase, fileName), `${REPO_LEARNINGS_REL}/${fileName}`, cat, 'org', domain));
+            // org pack home first, repo-local override second; repo-local wins on title collision.
+            out.push(...parseTiers(resolveOrgLearningFileTiers(roots.repoLearningsBase, f), cat));
+            // #806: domain-scoped org learnings — union from both locations.
+            const allDomainFiles = [
+                ...listDomainLearningFiles(roots.repoLearningsBase, 'org', fileType).map((d) => ({ ...d, base: roots.repoLearningsBase })),
+                ...listDomainLearningFiles((0, path_1.join)((0, pack_home_1.resolvePackHome)('org').contentRoot, 'learnings'), 'org', fileType)
+                    .map((d) => ({ ...d, base: (0, path_1.join)((0, pack_home_1.resolvePackHome)('org').contentRoot, 'learnings') })),
+            ];
+            const seenDomainFile = new Set();
+            for (const { fileName, domain, base } of allDomainFiles) {
+                if (seenDomainFile.has(fileName))
+                    continue;
+                seenDomainFile.add(fileName);
+                const domainTiers = resolveOrgLearningFileTiers(roots.repoLearningsBase, fileName);
+                if (domainTiers.length === 0) {
+                    out.push(...parseLearningEntries((0, path_1.join)(base, fileName), `${REPO_LEARNINGS_REL}/${fileName}`, cat, 'org', domain, undefined, threshold));
+                }
+                else {
+                    out.push(...parseTiers(domainTiers, cat, undefined, domain));
+                }
             }
         }
         return out;
     }
+    // manager + reverse: use the same three-tier resolution as buildLearningContextSection.
     const resolvedUserId = resolveLearningUserId(workspaceRoot, userId, roots);
-    const { dir, displayBase } = levelDir(roots, level);
     const cats = scope === 'reverse' ? ['coaching'] : ['avoid', 'preference', 'repeat'];
+    if (level === 'project') {
+        // Project level: repo-local only (single tier, no merge needed).
+        for (const cat of cats) {
+            const fileType = exports.CATEGORY_TO_FILETYPE[cat];
+            const f = `${resolvedUserId}-${fileType}.md`;
+            out.push(...parseLearningEntries((0, path_1.join)(roots.repoLearningsBase, f), `${REPO_LEARNINGS_REL}/${f}`, cat, 'project', 'global', 'hot', threshold));
+            const coldF = `${resolvedUserId}-${fileType}${COLD_FILE_SUFFIX}.md`;
+            if ((0, fs_1.existsSync)((0, path_1.join)(roots.repoLearningsBase, coldF))) {
+                out.push(...parseLearningEntries((0, path_1.join)(roots.repoLearningsBase, coldF), `${REPO_LEARNINGS_REL}/${coldF}`, cat, 'project', 'global', 'cold', threshold));
+            }
+            for (const { fileName, domain } of listDomainLearningFiles(roots.repoLearningsBase, resolvedUserId, fileType)) {
+                out.push(...parseLearningEntries((0, path_1.join)(roots.repoLearningsBase, fileName), `${REPO_LEARNINGS_REL}/${fileName}`, cat, 'project', domain, 'hot', threshold));
+            }
+        }
+        return out;
+    }
+    // machine level: all three personal tiers (managerCacheBase, globalPersonalBase, repoLearningsBase).
+    // Cold siblings are read from each tier that has the hot file.
     for (const cat of cats) {
         const fileType = exports.CATEGORY_TO_FILETYPE[cat];
         const f = `${resolvedUserId}-${fileType}.md`;
-        out.push(...parseLearningEntries((0, path_1.join)(dir, f), `${displayBase}/${f}`, cat, level, 'global', 'hot'));
-        // Cold sibling — only read when it exists alongside the hot file (#861).
+        const tiers = resolvePersonalLearningFileTiers(roots.repoLearningsBase, roots.managerCacheBase, roots.managerCacheDisplayBase, roots.globalPersonalBase, roots.globalPersonalDisplayBase, f);
+        out.push(...parseTiers(tiers, cat, 'hot'));
+        // Cold sibling from each tier that has the hot file.
         const coldF = `${resolvedUserId}-${fileType}${COLD_FILE_SUFFIX}.md`;
-        if ((0, fs_1.existsSync)((0, path_1.join)(dir, coldF))) {
-            out.push(...parseLearningEntries((0, path_1.join)(dir, coldF), `${displayBase}/${coldF}`, cat, level, 'global', 'cold'));
-        }
-        // #806: domain-scoped personal learnings alongside the flat global file.
-        for (const { fileName, domain } of listDomainLearningFiles(dir, resolvedUserId, fileType)) {
-            out.push(...parseLearningEntries((0, path_1.join)(dir, fileName), `${displayBase}/${fileName}`, cat, level, domain, 'hot'));
+        const coldTiers = resolvePersonalLearningFileTiers(roots.repoLearningsBase, roots.managerCacheBase, roots.managerCacheDisplayBase, roots.globalPersonalBase, roots.globalPersonalDisplayBase, coldF);
+        out.push(...parseTiers(coldTiers, cat, 'cold'));
+        // #806: domain files from managerHomeBase and repo base.
+        const domainDirs = [roots.globalPersonalBase, roots.managerCacheBase, roots.repoLearningsBase];
+        const seenDomain = new Set();
+        for (const dir of domainDirs) {
+            for (const { fileName, domain } of listDomainLearningFiles(dir, resolvedUserId, fileType)) {
+                if (seenDomain.has(fileName))
+                    continue;
+                seenDomain.add(fileName);
+                const domTiers = resolvePersonalLearningFileTiers(roots.repoLearningsBase, roots.managerCacheBase, roots.managerCacheDisplayBase, roots.globalPersonalBase, roots.globalPersonalDisplayBase, fileName);
+                out.push(...parseTiers(domTiers, cat, 'hot', domain));
+            }
         }
     }
     return out;
@@ -1292,8 +1377,12 @@ function readPreservedLearnings(workspaceRoot, userId, scope, level = 'machine')
 function resolveLearningFilePath(workspaceRoot, userId, ref) {
     const roots = getLearningRoots(workspaceRoot);
     const fileType = exports.CATEGORY_TO_FILETYPE[ref.category];
-    if (ref.scope === 'org')
-        return (0, path_1.join)(roots.repoLearningsBase, `org-${fileType}.md`);
+    if (ref.scope === 'org') {
+        // Write org entries to the org pack home — same location share-with-others uses.
+        // The Hub read path already merges pack home + repo-local, so edits land where
+        // the agent reads them without needing a repo commit.
+        return (0, path_1.join)((0, pack_home_1.resolvePackHome)('org').contentRoot, 'learnings', `org-${fileType}.md`);
+    }
     const resolvedUserId = resolveLearningUserId(workspaceRoot, userId, roots);
     // Issue #1070: for the 'machine' level use the configured manager home.
     const base = ref.level === 'project' ? roots.repoLearningsBase : roots.managerHomeBase;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "fraim-hub",
-  "version": "2.0.263",
+  "version": "2.0.264",
   "description": "FRAIM Hub local companion package.",
   "bin": {
     "fraim-hub": "bin/fraim-hub.js",
@@ -163,7 +163,7 @@
     "electron": "^41.2.2",
     "electron-updater": "^6.8.9",
     "express": "^5.2.1",
-    "fraim": "2.0.263",
+    "fraim": "2.0.264",
     "mongodb": "^7.0.0",
     "node-cron": "4.2.1",
     "node-edge-tts": "^1.2.10",

package/public/ai-hub/script.js

@@ -1332,6 +1332,8 @@ async function bgRefreshConversations() {
       state.conversations[state.projectPath] = existing;
       renderRail();
       renderActive();
+      // Issue #1075: keep overview run/status counts current after background refresh.
+      if (typeof tfRenderOverview === 'function' && typeof tf !== 'undefined' && tf.projectView === 'overview') tfRenderOverview();
     }
     ensureActiveConversationBody();
     // #866 R4: the 30s header poll may be the first to observe an onboarding run
@@ -8799,6 +8801,12 @@ function wireEvents() {
       tfOpenRequestedConnectedSurface();
     }
     await hydrateConversationsFromServer();
+    // Issue #1075: re-render overview so run counts reflect the freshly hydrated
+    // conversations. Deferred via setTimeout so the rail/active renders that
+    // complete hydration are not blocked by the overview card iteration.
+    setTimeout(() => {
+      if (typeof tfRenderOverview === 'function' && typeof tf !== 'undefined' && tf.projectView === 'overview') tfRenderOverview();
+    }, 0);
     startBgConvPoll();
     // #769: post-Stripe checkout redirect — start hire-pending polling so the new
     // employee appears as soon as the webhook completes, without a manual reload.
@@ -10242,8 +10250,18 @@ function tfRenderOverview() {
     const briefText = proj.brief || proj.intent || '';
     const team = document.createElement('div');
     team.className = 'proj-team';
+    // Look up conversations for this project; fall back to state.projectPath for the active project
+    const convKey = (state.conversations[proj.folderPath] ? proj.folderPath
+      : (tf.activeProjectId === proj.id ? state.projectPath : null));
+    const projConvs = (convKey ? state.conversations[convKey] : []).filter((c) => !c.managedByRunId);
+    // Issue #1075 R3: show only employees who have actual runs in this project, not all hired personas.
+    // Derive from conv.personaKey to scope avatars to who actually worked here. Fall back to
+    // project assignments when no conversations exist yet.
+    const convPersonaKeys = Array.from(new Set(projConvs.map((c) => c.personaKey).filter(Boolean)));
     const assigned = tfProjectAssignments(proj.id);
-    const empKeys = Array.from(new Set([...tfProjectTeamKeys(), ...assigned.map((a) => a.employeeKey).filter(Boolean)]));
+    const empKeys = convPersonaKeys.length
+      ? convPersonaKeys
+      : Array.from(new Set(assigned.map((a) => a.employeeKey).filter(Boolean)));
     empKeys.slice(0, 6).forEach((key, i) => {
       const persona = tfPersonaByKey(key);
       const av = tfAvatarFor(persona ? persona.displayName : key, i);
@@ -10259,14 +10277,9 @@ function tfRenderOverview() {
       chip.appendChild(pip);
       team.appendChild(chip);
     });
-    // Look up conversations for this project; fall back to state.projectPath for the active project
-    const convKey = (state.conversations[proj.folderPath] ? proj.folderPath
-      : (tf.activeProjectId === proj.id ? state.projectPath : null));
-    const projConvs = (convKey ? state.conversations[convKey] : []).filter((c) => !c.managedByRunId);
     const runCount = projConvs.filter((c) => conversationUiState(c) === 'working').length;
     const waitCount = projConvs.filter((c) => ['waiting', 'stopped'].includes(conversationUiState(c))).length;
     const blockCount = projConvs.filter((c) => c.blocked).length;
-    const totalRuns = projConvs.length;
     const parts = [];
     if (blockCount) parts.push(blockCount + ' blocked');
     if (runCount) parts.push(runCount + ' in progress');
@@ -10274,9 +10287,8 @@ function tfRenderOverview() {
     const badge = document.createElement('span');
     const needsAttention = waitCount > 0 || blockCount > 0;
     badge.className = 'proj-badge' + (needsAttention ? '' : ' quiet');
-    badge.textContent = parts.length ? parts.join(' · ')
-      : totalRuns ? totalRuns + (totalRuns === 1 ? ' run' : ' runs')
-      : 'No runs yet';
+    // Issue #1075 R4: show only active run counts; omit total run count when everything is done.
+    badge.textContent = parts.length ? parts.join(' · ') : '';
     team.appendChild(badge);
     card.appendChild(name);
     if (briefText) {
@@ -12652,6 +12664,7 @@ async function tfRunShareLearnings(scope) {
 // GET /api/ai-hub/learnings, so each section can DISPLAY and edit them. Keyed by
 // `${scope}:${level}` (e.g. manager:machine, manager:project); `undefined` = unloaded.
 function tfPreservedCache() { if (!state._preservedLearnings) state._preservedLearnings = {}; return state._preservedLearnings; }
+function tfThresholdCache() { if (!state._learningThresholds) state._learningThresholds = {}; return state._learningThresholds; }
 function tfLearnKey(scope, level) { return scope + ':' + level; }
 async function tfFetchPreservedLearnings(scope, level) {
   const params = new URLSearchParams({ scope, level });
@@ -12659,6 +12672,7 @@ async function tfFetchPreservedLearnings(scope, level) {
   try {
     const data = await requestJson('/api/ai-hub/learnings?' + params.toString());
     tfPreservedCache()[tfLearnKey(scope, level)] = Array.isArray(data.entries) ? data.entries : [];
+    if (typeof data.threshold === 'number') tfThresholdCache()[tfLearnKey(scope, level)] = data.threshold;
   } catch {
     tfPreservedCache()[tfLearnKey(scope, level)] = [];
   }
@@ -12697,6 +12711,7 @@ async function tfWriteLearning(payload, scope, level) {
     method: 'POST', headers: { 'Content-Type': 'application/json' }, body: JSON.stringify(body),
   });
   delete tfPreservedCache()[tfLearnKey(scope, level)];
+  delete tfThresholdCache()[tfLearnKey(scope, level)];
   tfReRenderScope(scope, level);
 }
 
@@ -12868,14 +12883,37 @@ function tfRenderLearningsList(host, scope, level, emptyText) {
   if (Array.isArray(preserved)) {
     const hotEntries = preserved.filter((e) => e.tier !== 'cold');
     const coldEntries = preserved.filter((e) => e.tier === 'cold');
-    for (const e of hotEntries) list.appendChild(tfPreservedCard(e, scope, level, shareMode, onSelectionChange));
+
+    // Split hot entries into injected (would be fed to agent) and below-threshold.
+    // Preferences and coaching have injected:true on all entries. Only
+    // mistake-patterns and validated-patterns have a score gate.
+    const injectedEntries = hotEntries.filter((e) => e.injected !== false);
+    const dormantHotEntries = hotEntries.filter((e) => e.injected === false);
+
+    for (const e of injectedEntries) list.appendChild(tfPreservedCard(e, scope, level, shareMode, onSelectionChange));
+
+    if (dormantHotEntries.length) {
+      // Cut line: learnings below the score threshold. Agent won't see these.
+      const cut = document.createElement('details');
+      cut.className = 'learn-dormant-section learn-cutline-section';
+      cut.setAttribute('data-testid', 'cutline-section');
+      const cutSummary = document.createElement('summary');
+      cutSummary.className = 'learn-dormant-header';
+      const threshold = tfThresholdCache()[key];
+      const thresholdLabel = typeof threshold === 'number' ? ` (score < ${threshold})` : '';
+      cutSummary.textContent = `Below threshold${thresholdLabel} — not injected into agent context (${dormantHotEntries.length})`;
+      cut.appendChild(cutSummary);
+      for (const e of dormantHotEntries) cut.appendChild(tfPreservedCard(e, scope, level, shareMode, onSelectionChange));
+      list.appendChild(cut);
+    }
+
     if (coldEntries.length) {
       const dormant = document.createElement('details');
       dormant.className = 'learn-dormant-section';
       dormant.setAttribute('data-testid', 'dormant-section');
       const summary = document.createElement('summary');
       summary.className = 'learn-dormant-header';
-      summary.textContent = `Dormant (${coldEntries.length})`;
+      summary.textContent = `Archived (${coldEntries.length})`;
       dormant.appendChild(summary);
       for (const e of coldEntries) dormant.appendChild(tfPreservedCard(e, scope, level, shareMode, onSelectionChange));
       list.appendChild(dormant);