fraim 2.0.346 → 2.0.349

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

@@ -1,17 +1,12 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.CATEGORY_TO_FILETYPE = exports.BRAIN_LEARNING_FILETYPE_TO_REGION = exports.TEAM_CONTEXT_RELATIVE_PATHS = exports.LEARNING_ENTRY_ID_REGEX = exports.LEARNING_PRIORITIES = exports.DEFAULT_THRESHOLD = void 0;
-exports.learningEntryHeadingRegex = learningEntryHeadingRegex;
-exports.learningEntryIdLineRegex = learningEntryIdLineRegex;
-exports.generateLearningEntryId = generateLearningEntryId;
+exports.CATEGORY_TO_FILETYPE = exports.BRAIN_LEARNING_FILETYPE_TO_REGION = exports.TEAM_CONTEXT_RELATIVE_PATHS = exports.generateLearningEntryId = exports.learningEntryIdLineRegex = exports.learningEntryHeadingRegex = exports.LEARNING_ENTRY_ID_REGEX = exports.LEARNING_PRIORITIES = exports.DEFAULT_THRESHOLD = void 0;
 exports.getScoreThreshold = getScoreThreshold;
 exports.computeEffectiveScore = computeEffectiveScore;
 exports.getLearningInputHomes = getLearningInputHomes;
 exports.readLearningSourceFile = readLearningSourceFile;
 exports.mergeLearningEntriesAcrossTiers = mergeLearningEntriesAcrossTiers;
 exports.buildLearningContextSection = buildLearningContextSection;
-exports.collectOfferedLearningEntries = collectOfferedLearningEntries;
-exports.collectOfferedRuleFiles = collectOfferedRuleFiles;
 exports.buildTeamContextSection = buildTeamContextSection;
 exports.resolveTeamContextFiles = resolveTeamContextFiles;
 exports.isTeamContextKey = isTeamContextKey;
@@ -33,10 +28,11 @@ const artifact_paths_1 = require("../../registry/scripts/artifact-paths");
  * workspace root on the user's machine.
  */
 const fs_1 = require("fs");
-const crypto_1 = require("crypto");
 const path_1 = require("path");
 const learning_source_record_1 = require("./learning-source-record");
 const completion_state_1 = require("../../registry/scripts/learning/completion-state");
+const outcome_types_1 = require("../../registry/scripts/learning/outcome-types");
+const owner_identity_1 = require("../../registry/scripts/learning/owner-identity");
 const project_fraim_paths_1 = require("../core/utils/project-fraim-paths");
 const brand_store_1 = require("../core/brand-store");
 // Issue #1043 round 2: org and manager content lives at one home named by
@@ -44,10 +40,6 @@ const brand_store_1 = require("../core/brand-store");
 // cache, which silently returned nothing once that cache stopped existing.
 const pack_home_1 = require("../cli/utils/pack-home");
 const learning_domains_1 = require("../config/learning-domains");
-// Key construction only. The store's read/write path is still required lazily
-// below, because that side pulls in the retention config and a static cycle
-// through this module would be fragile.
-const learning_usage_store_1 = require("./learning-usage-store");
 const REPO_LEARNINGS_REL = (0, project_fraim_paths_1.getWorkspaceFraimDisplayPath)('personalized-employee/learnings').replace(/\/$/, '');
 /**
  * The score at or above which an entry is injected. Exported so a test asserts
@@ -70,41 +62,13 @@ const L0_SLEEP_ON_LEARNINGS_PROMPT_MIN = 5;
 // from LEARNING_PRIORITIES, and a build-time contract check
 // (scripts/validate-learning-format-contract.ts, run by `npm run build`) asserts
 // all three sides agree. Issue #1103 added the `**Id**` line to the same contract.
-exports.LEARNING_PRIORITIES = ['P-CRITICAL', 'P-HIGH', 'P-MED', 'P-LOW'];
-/** Canonical entry-heading matcher: `##`..`######` + `[P-X]` + optional title. */
-function learningEntryHeadingRegex(flags = '') {
-    return new RegExp(`^#{2,}\\s+\\[(${exports.LEARNING_PRIORITIES.join('|')})\\]\\s*(.*)$`, flags);
-}
-/**
- * Issue #1103 R5: the entry's stable identifier, written as a `**Id**: L-…` line.
- *
- * A firing record has to survive a later title edit, which a title-derived key
- * cannot do — so the identity is persisted on the entry rather than computed at
- * read time. Generation is deterministic from (family, original title) only so a
- * re-run of the migration never renumbers the corpus; once written, the id never
- * changes, including when the title does.
- */
-/** What `generateLearningEntryId` emits: `L-` plus exactly ten hex characters. */
-exports.LEARNING_ENTRY_ID_REGEX = /^L-[0-9a-f]{10}$/;
-/**
- * What the reader accepts. Deliberately wider than what the generator emits: a
- * hand-written or hand-edited id must be honoured rather than silently ignored,
- * because ignoring it would leave the entry looking un-stamped and route its
- * firings to a title key without saying so.
- */
-const LEARNING_ENTRY_ID_LINE = /^\*\*Id\*\*:\s*(L-[0-9A-Za-z]{6,24})\s*$/;
-function learningEntryIdLineRegex() {
-    return new RegExp(LEARNING_ENTRY_ID_LINE.source);
-}
-function generateLearningEntryId(fileType, title) {
-    // Ten hex characters: short enough for an agent to type into a retrospective
-    // by hand, wide enough that a collision across one manager's corpus is not a
-    // practical concern.
-    const digest = (0, crypto_1.createHash)('sha1')
-        .update(`${fileType}|${title.trim().replace(/\s+/g, ' ').toLowerCase()}`)
-        .digest('hex');
-    return `L-${digest.slice(0, 10)}`;
-}
+const learning_entry_format_1 = require("./learning-entry-format");
+var learning_entry_format_2 = require("./learning-entry-format");
+Object.defineProperty(exports, "LEARNING_PRIORITIES", { enumerable: true, get: function () { return learning_entry_format_2.LEARNING_PRIORITIES; } });
+Object.defineProperty(exports, "LEARNING_ENTRY_ID_REGEX", { enumerable: true, get: function () { return learning_entry_format_2.LEARNING_ENTRY_ID_REGEX; } });
+Object.defineProperty(exports, "learningEntryHeadingRegex", { enumerable: true, get: function () { return learning_entry_format_2.learningEntryHeadingRegex; } });
+Object.defineProperty(exports, "learningEntryIdLineRegex", { enumerable: true, get: function () { return learning_entry_format_2.learningEntryIdLineRegex; } });
+Object.defineProperty(exports, "generateLearningEntryId", { enumerable: true, get: function () { return learning_entry_format_2.generateLearningEntryId; } });
 /**
  * The manager location that is NOT the portable authoring base.
  *
@@ -348,7 +312,7 @@ function getScoreThreshold(workspaceRoot) {
  * @param now Optional override for "now" (for forward-looking aging-risk
  *            calculations). Defaults to the current wall clock.
  */
-function computeEffectiveScore(severity, lastSeenDate, recurrences, fileType, now = new Date(), lastFiredDate) {
+function computeEffectiveScore(severity, lastSeenDate, recurrences, fileType, now = new Date()) {
     const baseScore = severity === 'P-CRITICAL' ? 10 : severity === 'P-HIGH' ? 8 : severity === 'P-MED' ? 5 : 3;
     // Mistake patterns decay faster (90d) — they're tied to environments that change.
     // Preferences, manager-coaching, and validated-patterns express durable judgment (180d half-life).
@@ -361,55 +325,12 @@ function computeEffectiveScore(severity, lastSeenDate, recurrences, fileType, no
     catch {
         // Treat as fresh.
     }
-    // Issue #1103 R8: the clock is max(last_seen, last_fired). `Last seen` only
-    // advances when the same signal recurs, which for a mistake pattern means the
-    // mistake happened again — so an entry that fires and prevents its mistake
-    // generates no signal and decays *because* it worked. Reading the newer of the
-    // two dates fixes that inversion with one changed input: no new term, the same
-    // half-lives, the same threshold. The newer date is the smaller elapsed time,
-    // hence min() here. An absent or unparseable firing date contributes nothing,
-    // so it can never silently resurrect a dormant entry, and an entry with no
-    // usage record at all scores exactly as it did before.
-    let daysSinceClock = daysSinceLastSeen;
-    const lastFiredMs = parseClockDate(lastFiredDate);
-    if (lastFiredMs > 0 && !isImplausiblyFuture(lastFiredMs, now)) {
-        const daysSinceFired = Math.max(0, (now.getTime() - lastFiredMs) / (1000 * 60 * 60 * 24));
-        daysSinceClock = Number.isNaN(daysSinceClock) ? daysSinceFired : Math.min(daysSinceClock, daysSinceFired);
-    }
-    const decay = Math.pow(0.5, daysSinceClock / halfLife);
+    // Selection uses recorded learning experience, independently of optional usage history.
+    const decay = Math.pow(0.5, daysSinceLastSeen / halfLife);
     const recurrenceBoost = Math.log2(Math.max(1, recurrences) + 1);
     return baseScore * decay * recurrenceBoost;
 }
-/** Clock skew tolerance for a firing date, in milliseconds. */
-const FUTURE_FIRING_TOLERANCE_MS = 24 * 60 * 60 * 1000;
-/**
- * Is this firing date far enough ahead of now to be untrustworthy?
- *
- * `lastFiredAt` is stamped by whichever machine ran the job, and the usage store
- * lives in the manager home, which is commonly a folder synced across machines.
- * A machine running fast therefore writes a date another machine reads as the
- * future, and elapsed time clamps to zero, so decay stops and the entry stays at
- * full score indefinitely. That is the opposite of a decay model.
- *
- * A day of tolerance absorbs ordinary skew, where treating the firing as "just
- * now" is right anyway. Beyond that the date is not believable, so it is ignored
- * exactly as an unparseable one is, leaving the recurrence date in charge.
- */
-function isImplausiblyFuture(timestampMs, now) {
-    return timestampMs - now.getTime() > FUTURE_FIRING_TOLERANCE_MS;
-}
-/**
- * Parse one side of the decay clock. An absent or unparseable date contributes
- * nothing, so a malformed `last fired` can never silently resurrect a dormant
- * entry: it just leaves the recurrence date in charge.
- */
-function parseClockDate(value) {
-    if (!value)
-        return 0;
-    const parsed = new Date(value).getTime();
-    return Number.isNaN(parsed) ? 0 : parsed;
-}
-function scanMistakePatternFile(filePath, threshold, fileType = 'mistake-patterns', usageClock) {
+function scanMistakePatternFile(filePath, threshold, fileType = 'mistake-patterns') {
     const empty = { active: 0, dormant: 0, agingRisk: 0 };
     if (!(0, fs_1.existsSync)(filePath))
         return empty;
@@ -429,20 +350,17 @@ function scanMistakePatternFile(filePath, threshold, fileType = 'mistake-pattern
     let scanned = 0;
     let inEntry = false;
     let severity = null;
-    let title = '';
-    let entryId = null;
     let lastSeen = '';
     let recurrences = 1;
     const flush = () => {
         if (!severity)
             return;
         scanned++;
-        const lastFired = usageClock ? usageClock(entryId, fileType, title) : null;
-        const today = computeEffectiveScore(severity, lastSeen, recurrences, fileType, now, lastFired);
+        const today = computeEffectiveScore(severity, lastSeen, recurrences, fileType, now);
         if (today >= threshold) {
             active++;
             if (lastSeen) {
-                const future = computeEffectiveScore(severity, lastSeen, recurrences, fileType, horizon, lastFired);
+                const future = computeEffectiveScore(severity, lastSeen, recurrences, fileType, horizon);
                 if (future < threshold)
                     agingRisk++;
             }
@@ -454,24 +372,17 @@ function scanMistakePatternFile(filePath, threshold, fileType = 'mistake-pattern
     for (const line of lines) {
         if (scanned >= MAX_ENTRIES_SCANNED)
             break;
-        const headerMatch = line.match(learningEntryHeadingRegex());
+        const headerMatch = line.match((0, learning_entry_format_1.learningEntryHeadingRegex)());
         if (headerMatch) {
             flush();
             inEntry = true;
             severity = headerMatch[1];
-            title = headerMatch[2].trim();
-            entryId = null;
             lastSeen = '';
             recurrences = 1;
             continue;
         }
         if (!inEntry)
             continue;
-        const idMatch = line.trim().match(LEARNING_ENTRY_ID_LINE);
-        if (idMatch) {
-            entryId = idMatch[1];
-            continue;
-        }
         const lastSeenMatch = line.match(/^\*\*Last seen\*\*:\s*(.+)/);
         if (lastSeenMatch) {
             lastSeen = lastSeenMatch[1].trim();
@@ -485,22 +396,8 @@ function scanMistakePatternFile(filePath, threshold, fileType = 'mistake-pattern
     flush();
     return { active, dormant, agingRisk };
 }
-/**
- * Build the usage clock from the store, or return undefined when there is no
- * usage data at all. Returning undefined rather than an always-null function is
- * deliberate: it keeps the "no usage record" path identical to pre-#1103
- * behaviour instead of merely equivalent.
- */
-function resolveUsageClock(workspaceRoot) {
-    return resolveUsageViews(workspaceRoot).clock;
-}
-/**
- * Issue #1755: one store read, two views of it: the clock the scorer needs and
- * the counters a surface renders. They are resolved together because reading the
- * store twice for the same pass would let the two disagree, and because the
- * identity resolution that finds a record is the projection's, not each caller's.
- */
-function resolveUsageViews(workspaceRoot) {
+/** Historical counters are only read for the human library view. */
+function resolveUsageStats() {
     try {
         // Required lazily: the store imports the retention config, which imports the
         // workspace path helpers, and a cycle through this module would be fragile.
@@ -508,31 +405,28 @@ function resolveUsageViews(workspaceRoot) {
         const projection = require('./learning-usage-projection');
         const loaded = store.readUsageStore();
         if (Object.keys(loaded.entries).length === 0)
-            return {};
+            return undefined;
         const lookup = projection.buildUsageLookup(loaded);
-        return {
-            clock: (id, fileType, title) => projection.lookupUsage(lookup, id, fileType, title)?.lastFired ?? null,
-            stats: (id, fileType, title) => {
-                const found = projection.lookupUsage(lookup, id, fileType, title);
-                if (!found)
-                    return null;
-                return {
-                    offered: found.offered,
-                    offeredAboveThreshold: found.offeredAboveThreshold,
-                    applied: found.byOutcome.applied || 0,
-                    prevented: found.byOutcome.prevented || 0,
-                    ignored: found.byOutcome.ignored || 0,
-                    notApplicable: found.byOutcome['not-applicable'] || 0,
-                    lastOfferedAt: found.lastOffered,
-                    lastFiredAt: found.lastFired,
-                };
-            },
+        return (id, fileType, title) => {
+            const found = projection.lookupUsage(lookup, id, fileType, title);
+            if (!found)
+                return null;
+            return {
+                historical: true,
+                countingStatus: 'deferred',
+                lastObservedAt: found.lastObservedAt,
+                applied: found.byOutcome.applied || 0,
+                prevented: found.byOutcome.prevented || 0,
+                ignored: found.byOutcome.ignored || 0,
+                notApplicable: found.byOutcome['not-applicable'] || 0,
+                lastFiredAt: found.lastFired,
+            };
         };
     }
     catch {
         // Usage data is derived and optional. A failure here must never change what
         // the agent is given.
-        return {};
+        return undefined;
     }
 }
 function readFrontmatter(content) {
@@ -552,34 +446,6 @@ function readFrontmatter(content) {
     }
     return frontmatter;
 }
-/**
- * Issue #1857 R11: how many proposals are waiting for the manager's review.
- *
- * This counts directory entries and nothing else. It does not open a proposal, and
- * deliberately has no access to one: R10 is an invariant about influence rather
- * than storage, so the employee learns that a backlog exists and which job clears
- * it, never what any pending claim says.
- *
- * The same shape as the pending-L0 line it sits beside, for the same reason: a
- * count plus the job that clears it is actionable; a list of contents is not.
- */
-function countPendingProposals(roots, resolvedUserId) {
-    let total = 0;
-    for (const base of [roots.repoLearningsBase, roots.globalPersonalBase]) {
-        if (!base)
-            continue;
-        const queueDir = (0, path_1.join)(base, 'review-queue', resolvedUserId);
-        if (!(0, fs_1.existsSync)(queueDir))
-            continue;
-        try {
-            total += (0, fs_1.readdirSync)(queueDir).filter((name) => name.endsWith('.json')).length;
-        }
-        catch {
-            // An unreadable queue is not worth failing a job's startup over.
-        }
-    }
-    return total;
-}
 /** Configured owning homes only; discovery never provisions or infers a home. */
 function getLearningInputHomes(workspaceRoot) {
     const homes = [{ root: workspaceRoot, contentRoot: (0, path_1.dirname)((0, project_fraim_paths_1.getWorkspaceLearningsDir)(workspaceRoot)) },
@@ -736,15 +602,7 @@ const CONTEXT_CATEGORIES = {
     coach: { ft: 'manager-coaching', gated: false, label: '(manager-facing; all entries)' },
     validated: { ft: 'validated-patterns', gated: true, label: '(entries above score threshold)' },
 };
-/**
- * Resolve exactly the files the injected context block lists, in the order it lists
- * them.
- *
- * Extracted from `buildLearningContextSection` for issue #1103: the block renders
- * these as text, and the offer recorder enumerates their entries. Both have to see
- * the identical set, so there is one resolution and two consumers rather than two
- * resolutions that can drift.
- */
+/** Resolve the files listed in injected context, in delivery order. */
 function resolveContextLearningFiles(workspaceRoot, resolvedUserId, roots, forJob, domain) {
     // Domain axis (issue #806) plus the connect/job single-injection contract (issue
     // #860). Session startup stays broad enough for stable preferences plus L0, while
@@ -839,14 +697,13 @@ function buildLearningContextSection(workspaceRoot, userId, forJob, domain) {
     const roots = getLearningRoots(workspaceRoot);
     const resolvedUserId = resolveLearningUserId(workspaceRoot, userId, roots);
     const threshold = getScoreThreshold(workspaceRoot);
-    const usageClock = resolveUsageClock(workspaceRoot);
     const resolved = resolveContextLearningFiles(workspaceRoot, resolvedUserId, roots, forJob, domain);
     const multiTier = resolved.multiTier;
     const toListed = (f) => ({
         displayPath: f.displayPath,
         label: f.label,
         dormant: f.gated
-            ? scanMistakePatternFile(f.path, threshold, f.fileType, usageClock).dormant
+            ? scanMistakePatternFile(f.path, threshold, f.fileType).dormant
             : 0,
         isCoach: f.isCoach,
     });
@@ -857,11 +714,9 @@ function buildLearningContextSection(workspaceRoot, userId, forJob, domain) {
     const l0RetroCount = pendingL0Sources.filter(source => source.kind === 'retrospective').length;
     const hasL2 = l2Files.length > 0;
     const hasL1 = l1Files.length > 0;
-    // Issue #1857 R11. Counted before the gate on purpose: a manager with a full
-    // review queue and an empty corpus is exactly the case where the backlog most
-    // needs saying, and computing it after the early return would silence it there.
-    const pendingProposals = countPendingProposals(roots, resolvedUserId);
-    const hasContent = hasL2 || hasL1 || l0CoachingCount > 0 || l0RetroCount > 0 || pendingProposals > 0;
+    // Saved proposal copies do not establish an outstanding human decision.
+    // The originating run's current review state owns reminders.
+    const hasContent = hasL2 || hasL1 || l0CoachingCount > 0 || l0RetroCount > 0;
     if (!hasContent)
         return '';
     let section = forJob
@@ -903,10 +758,6 @@ function buildLearningContextSection(workspaceRoot, userId, forJob, domain) {
     // Issue #1857 R11. The count, never the content: an unapproved proposal does
     // not influence this job, and the employee is told only that the manager has a
     // backlog and which job clears it.
-    if (pendingProposals > 0) {
-        const plural = pendingProposals !== 1;
-        section += `${pendingProposals} proposal${plural ? 's are' : ' is'} waiting for your manager's review; run \`sleep-on-learnings\` to review ${plural ? 'them' : 'it'}.\n\n`;
-    }
     const coachPresent = resolved.coachPresent;
     if (forJob) {
         if (hasL2 || hasL1) {
@@ -930,159 +781,6 @@ function buildLearningContextSection(workspaceRoot, userId, forJob, domain) {
     }
     return section;
 }
-/** Enumerate one learning file's entries with their above-threshold state. */
-function enumerateFileEntries(file, threshold, usageClock, now) {
-    if (!(0, fs_1.existsSync)(file.path))
-        return [];
-    let content;
-    try {
-        content = (0, fs_1.readFileSync)(file.path, 'utf8');
-    }
-    catch {
-        return [];
-    }
-    const out = [];
-    const headingRe = learningEntryHeadingRegex();
-    let current = null;
-    const flush = () => {
-        if (!current)
-            return;
-        // A non-gated family (preferences, manager-coaching) is delivered whole, so
-        // every entry in it is active guidance. A gated family delivers only entries
-        // at or above threshold; the rest are the dormant rollup.
-        let aboveThreshold = true;
-        if (file.gated) {
-            if (current.lastSeen) {
-                const lastFired = usageClock ? usageClock(current.id, file.fileType, current.title) : null;
-                aboveThreshold = computeEffectiveScore(current.severity, current.lastSeen, current.recurrences, file.fileType, now, lastFired) >= threshold;
-            }
-            // No date metadata: parseLearningEntries treats that as fresh and injects it,
-            // so the offer record has to agree.
-        }
-        out.push({
-            // The store owns key construction. Building the composite key by hand here
-            // would be a second definition of what an entry's identity is, and the two
-            // would drift the first time normalization changed. Issue #1752 A1 made
-            // that one function rather than four copies of the same expression.
-            key: (0, learning_usage_store_1.resolveEntryIdentity)({ id: current.id, fileType: file.fileType, title: current.title }).key,
-            family: 'learning',
-            id: current.id,
-            title: current.title,
-            fileType: file.fileType,
-            level: file.level,
-            displayPath: file.displayPath,
-            severity: current.severity,
-            aboveThreshold,
-        });
-        current = null;
-    };
-    for (const line of content.split(/\r?\n/)) {
-        const header = line.match(headingRe);
-        if (header) {
-            flush();
-            current = { severity: header[1], title: header[2].trim(), id: null, lastSeen: '', recurrences: 1 };
-            continue;
-        }
-        if (!current)
-            continue;
-        const trimmed = line.trim();
-        const idMatch = trimmed.match(LEARNING_ENTRY_ID_LINE);
-        if (idMatch) {
-            current.id = idMatch[1];
-            continue;
-        }
-        const lastSeenMatch = trimmed.match(/^\*\*Last seen\*\*:\s*(.+)/i);
-        if (lastSeenMatch) {
-            current.lastSeen = lastSeenMatch[1].trim();
-            continue;
-        }
-        const recurrenceMatch = trimmed.match(/^\*\*Recurrences\*\*:\s*(\d+)/i);
-        if (recurrenceMatch) {
-            current.recurrences = parseInt(recurrenceMatch[1], 10) || 1;
-        }
-    }
-    flush();
-    return out;
-}
-/**
- * Issue #1103 R1 and R2: every learning entry the runtime just delivered.
- *
- * "Delivered" is file-level, because the injected block lists file paths rather
- * than entry titles: an entry is offered when it sat inside a file the block
- * listed. `aboveThreshold` is what separates active guidance from the dormant
- * rollup. Nothing here depends on the agent reading anything.
- *
- * Offers use stable identity, independently of ownership reconciliation. A repeated
- * ID records one offer, but equal titles with different IDs remain distinct.
- * Unstamped legacy entries retain their existing title usage key; that fallback
- * never grants a stamped entry authority over a different identity.
- */
-function collectOfferedLearningEntries(workspaceRoot, userId, forJob, domain) {
-    const roots = getLearningRoots(workspaceRoot);
-    const resolvedUserId = resolveLearningUserId(workspaceRoot, userId, roots);
-    const threshold = getScoreThreshold(workspaceRoot);
-    const usageClock = resolveUsageClock(workspaceRoot);
-    const now = new Date();
-    const resolved = resolveContextLearningFiles(workspaceRoot, resolvedUserId, roots, forJob, domain);
-    const byLesson = new Map();
-    for (const file of [...resolved.l2, ...resolved.l1]) {
-        for (const entry of enumerateFileEntries(file, threshold, usageClock, now)) {
-            const lessonKey = entry.id ?? (0, learning_usage_store_1.titleUsageKey)(entry.fileType, entry.title);
-            const group = byLesson.get(lessonKey);
-            if (group)
-                group.push(entry);
-            else
-                byLesson.set(lessonKey, [entry]);
-        }
-    }
-    const out = [];
-    for (const group of byLesson.values()) {
-        // This is an offer counter, not a choice of authoritative guidance.
-        const entry = group[0];
-        out.push({ ...entry, aboveThreshold: group.some(item => item.aboveThreshold) });
-    }
-    return out;
-}
-/**
- * Issue #1103 R3: the auto-loaded Team Context rule files.
- *
- * These are the most influential instructions in the system and produce no usage
- * signal today, because the agent reads them with its own file tool and the proxy
- * never sees a `get_fraim_file` call for them. Recording the offer at composition
- * time closes that gap. A rule is unconditional, so it is always above threshold.
- */
-function collectOfferedRuleFiles(workspaceRoot, forJob) {
-    const resolved = resolveTeamContextFiles(workspaceRoot);
-    // The three rule files by name, not by path sniffing: these are the ones R3
-    // names, and the other Team Context files are context rather than instruction,
-    // so recording them as offered guidance would misreport what they are.
-    const ruleFiles = [
-        { presence: resolved.orgRules, level: 'org' },
-        { presence: resolved.managerRules, level: 'manager' },
-        { presence: resolved.projectRules, level: 'project' },
-    ];
-    const out = [];
-    for (const { presence, level } of ruleFiles) {
-        if (!presence.present || !presence.displayPath)
-            continue;
-        const displayPath = presence.displayPath.replace(/\\/g, '/');
-        out.push({
-            key: (0, learning_usage_store_1.ruleUsageKey)(displayPath),
-            family: 'rule',
-            id: null,
-            title: displayPath.split('/').pop() ?? displayPath,
-            fileType: 'rules',
-            level,
-            displayPath,
-            severity: null,
-            aboveThreshold: true,
-        });
-    }
-    // `forJob` is accepted so both injection points call this identically; the Team
-    // Context block lists the same rule files in either frame.
-    void forJob;
-    return out;
-}
 /**
  * Resolve an organization/manager-scope context file (issue #563 order, R3.1):
  *   1. repo-local override (`fraim/personalized-employee/…`) — wins when present
@@ -1390,7 +1088,7 @@ function countLearningEntries(filePath) {
         return 0;
     }
     let count = 0;
-    const headingRe = learningEntryHeadingRegex();
+    const headingRe = (0, learning_entry_format_1.learningEntryHeadingRegex)();
     for (const line of content.split(/\r?\n/)) {
         if (headingRe.test(line))
             count++;
@@ -1413,7 +1111,7 @@ function readLearningEntryTitles(filePath) {
         return [];
     }
     const titles = [];
-    const headingRe = learningEntryHeadingRegex();
+    const headingRe = (0, learning_entry_format_1.learningEntryHeadingRegex)();
     for (const line of content.split(/\r?\n/)) {
         const m = line.match(headingRe);
         if (m)
@@ -1533,8 +1231,6 @@ const PENDING_KIND_LABEL = {
 };
 /** Filename infix that separates hot (.md) from cold-tier (.cold.md) learning files (#861). */
 const COLD_FILE_SUFFIX = '.cold';
-/** R11's "fired recently" window. */
-const FIRED_RECENT_DAYS = 30;
 // Parse `## [P-…] Title` entries out of a learning file WITH their effective
 // score + last-seen (unlike parseLearningEntries, which drops that bookkeeping
 // for the Hub text display). Used only by the brain-dot projection.
@@ -1549,7 +1245,7 @@ function parseBrainLearningEntries(filePath, fileType, usage) {
         return [];
     }
     const out = [];
-    const headingRe = learningEntryHeadingRegex();
+    const headingRe = (0, learning_entry_format_1.learningEntryHeadingRegex)();
     const now = new Date();
     let current = null;
     const flush = () => {
@@ -1557,7 +1253,7 @@ function parseBrainLearningEntries(filePath, fileType, usage) {
             return;
         const projected = usage ? usage(current.id, fileType, current.title) : undefined;
         const rawScore = current.lastSeen
-            ? computeEffectiveScore(current.severity, current.lastSeen, current.recurrences, fileType, now, projected?.lastFired ?? null)
+            ? computeEffectiveScore(current.severity, current.lastSeen, current.recurrences, fileType, now)
             : null;
         out.push({
             title: current.title,
@@ -1578,7 +1274,7 @@ function parseBrainLearningEntries(filePath, fileType, usage) {
         }
         if (!current)
             continue;
-        const idMatch = line.trim().match(LEARNING_ENTRY_ID_LINE);
+        const idMatch = line.trim().match(learning_entry_format_1.LEARNING_ENTRY_ID_LINE);
         if (idMatch) {
             current.id = idMatch[1];
             continue;
@@ -1609,26 +1305,19 @@ function resolveBrainUsageProjector() {
         if (Object.keys(loaded.entries).length === 0)
             return undefined;
         const lookup = projection.buildUsageLookup(loaded);
-        const now = Date.now();
         return (id, fileType, title) => {
             const value = projection.lookupUsage(lookup, id, fileType, title);
             if (!value)
                 return undefined;
-            let usageState;
-            if (value.lastFired) {
-                const ageDays = (now - new Date(value.lastFired).getTime()) / 86_400_000;
-                usageState = ageDays <= FIRED_RECENT_DAYS ? 'fired-recent' : 'fired-aging';
-            }
-            else {
-                usageState = 'never-fired';
-            }
             return {
-                offered: value.offered,
+                historical: true,
+                countingStatus: 'deferred',
+                lastObservedAt: value.lastObservedAt,
                 fired: value.fired,
                 lastFired: value.lastFired,
                 firedUnder: value.firedUnder,
                 fireLog: value.log.map((l) => ({ date: l.date, outcome: l.outcome, job: l.job, note: l.note })),
-                usageState,
+                usageState: 'historical',
             };
         };
     }
@@ -1726,7 +1415,7 @@ function listDomainLearningFiles(dir, prefix, fileType) {
 // 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 = exports.DEFAULT_THRESHOLD, usageClock, usageStats) {
+function parseLearningEntries(filePath, displayPath, category, level, domain = 'global', tier, threshold = exports.DEFAULT_THRESHOLD, usageStats) {
     if (!(0, fs_1.existsSync)(filePath))
         return [];
     let content;
@@ -1748,14 +1437,11 @@ function parseLearningEntries(filePath, displayPath, category, level, domain = '
     const flush = () => {
         if (current) {
             current.body = bodyLines.join('\n').replace(/\n{3,}/g, '\n\n').trim();
-            // Issue #1755: resolve usage once per entry, before scoring, so the
-            // counters a surface renders and the clock the score uses come from
-            // the same record rather than two independent lookups.
+            // Historical observations decorate the library; they never select guidance.
             current.usage = usageStats ? usageStats(current.id, fileType, current.title) : null;
             if (isScoreGated) {
                 if (curLastSeen !== null) {
-                    const lastFired = usageClock ? usageClock(current.id, fileType, current.title) : null;
-                    const s = computeEffectiveScore(current.severity, curLastSeen, curRecurrences, fileType, new Date(), lastFired);
+                    const s = computeEffectiveScore(current.severity, curLastSeen, curRecurrences, fileType, new Date());
                     current.score = s;
                     current.injected = s >= threshold;
                 }
@@ -1778,7 +1464,7 @@ function parseLearningEntries(filePath, displayPath, category, level, domain = '
         curLastSeen = null;
         curRecurrences = 1;
     };
-    const headingRe = learningEntryHeadingRegex();
+    const headingRe = (0, learning_entry_format_1.learningEntryHeadingRegex)();
     for (const line of content.split(/\r?\n/)) {
         const header = line.match(headingRe);
         if (header) {
@@ -1803,7 +1489,7 @@ function parseLearningEntries(filePath, displayPath, category, level, domain = '
         }
         const t = line.trim();
         // Capture scoring metadata but exclude from body.
-        const idMatch = t.match(LEARNING_ENTRY_ID_LINE);
+        const idMatch = t.match(learning_entry_format_1.LEARNING_ENTRY_ID_LINE);
         if (idMatch) {
             current.id = idMatch[1];
             continue;
@@ -1849,6 +1535,7 @@ function readRejectedProposals(workspaceRoot, userId) {
         return [];
     const out = [];
     const seen = new Set();
+    const projectOwnerPath = (0, owner_identity_1.ownerPath)((0, project_fraim_paths_1.getWorkspaceFraimDir)(workspaceRoot));
     const headingRe = /^## (Rejected|Superseded): (.+)$/m;
     const reasonRe = /^Reason: (.+)$/m;
     const revisitRe = /^Revisit when: (.+)$/m;
@@ -1881,7 +1568,9 @@ function readRejectedProposals(workspaceRoot, userId) {
             let proposal = {};
             if (json) {
                 try {
-                    proposal = JSON.parse(json[1]);
+                    const parsed = JSON.parse(json[1]);
+                    if (parsed && typeof parsed === 'object' && !Array.isArray(parsed))
+                        proposal = parsed;
                 }
                 catch {
                     // Keep the human-readable half; the JSON only enriches it.
@@ -1890,12 +1579,24 @@ function readRejectedProposals(workspaceRoot, userId) {
             const decision = (proposal.decision || {});
             const content = (proposal.content || {});
             const trace = (content.trace || {});
+            let targetOwner;
+            try {
+                const owner = proposal.target?.owner;
+                (0, owner_identity_1.validateOwner)(owner);
+                targetOwner = { level: owner.level, path: owner.path };
+            }
+            catch {
+                // Older/damaged targets remain visible as unknown; never infer
+                // their project from the shared history file's location.
+            }
+            if (targetOwner?.level === 'Project' && targetOwner.path !== projectOwnerPath)
+                continue;
             const claimKey = typeof proposal.claimKey === 'string' && proposal.claimKey
                 ? proposal.claimKey
                 : parts[1] || heading[2];
             // The same claim can appear in more than one configured base. The first
             // base supplies display metadata for that same recorded decision.
-            const key = `${claimKey}:${String(decision.at || '')}`;
+            const key = JSON.stringify([claimKey, String(decision.at || ''), targetOwner?.level, targetOwner?.path]);
             if (seen.has(key))
                 continue;
             seen.add(key);
@@ -1910,6 +1611,7 @@ function readRejectedProposals(workspaceRoot, userId) {
                 rejectedAt: typeof decision.at === 'string' ? decision.at : '',
                 ...(typeof trace.recurrences === 'number' ? { recurrencesAtRejection: trace.recurrences } : {}),
                 displayPath,
+                ...(targetOwner ? { targetOwner } : {}),
             });
         }
     }
@@ -1919,16 +1621,44 @@ function readRejectedProposals(workspaceRoot, userId) {
 function readPreservedLearnings(workspaceRoot, userId, scope, level = 'machine') {
     const roots = getLearningRoots(workspaceRoot);
     const threshold = getScoreThreshold(workspaceRoot);
-    // Issue #1103 R8: the Hub's injected flag must agree with the loader's, so both
-    // read the same clock. One store read serves the whole call.
-    // Issue #1755: the same read also yields the counters the Hub renders.
-    const { clock: usageClock, stats: usageStats } = resolveUsageViews(workspaceRoot);
+    const usageStats = resolveUsageStats();
+    // Only the human library reads these observations. An unavailable store must
+    // fail this read, so a client can retain known history rather than invent zeroes.
+    const resolvedUserId = scope === 'org' ? userId : resolveLearningUserId(workspaceRoot, userId, roots);
+    const managerHome = (0, pack_home_1.resolvePackHome)('manager', { migrateLegacy: false });
+    const orgHome = scope === 'org' ? (0, pack_home_1.resolvePackHome)('org', { migrateLegacy: false }) : null;
+    const assessmentHomes = [
+        { level: 'Project', root: (0, project_fraim_paths_1.getWorkspaceFraimDir)(workspaceRoot), contentRoot: (0, path_1.join)((0, project_fraim_paths_1.getWorkspaceFraimDir)(workspaceRoot), 'personalized-employee') },
+        { level: 'Manager', root: managerHome.root, contentRoot: managerHome.contentRoot },
+        ...(orgHome ? [{ level: 'Org', root: orgHome.root, contentRoot: orgHome.contentRoot }] : []),
+    ];
+    const requestedHomes = assessmentHomes.filter(home => scope === 'org'
+        ? home.level !== 'Manager' : home.level === (level === 'project' ? 'Project' : 'Manager'));
+    const assessed = require('./learning-usage-store').readAssessedOutcomes(undefined, {
+        owners: requestedHomes.map(home => ({ level: home.level, path: (0, owner_identity_1.ownerPath)(home.root) })), principal: userId,
+    });
+    const decorateAssessment = (entry, file) => {
+        const home = assessmentHomes.find(candidate => {
+            if (scope !== 'org' && candidate.level !== (entry.level === 'project' ? 'Project' : 'Manager'))
+                return false;
+            const child = (0, path_1.relative)(candidate.contentRoot, file);
+            return child !== '..' && !child.startsWith('..' + path_1.sep) && !(0, path_1.isAbsolute)(child);
+        });
+        entry.assessment = home && entry.id ? (0, outcome_types_1.summarizeLearningOutcomes)(assessed, {
+            owner: { level: home.level, path: (0, owner_identity_1.ownerPath)(home.root) },
+            relativePath: (0, path_1.relative)(home.contentRoot, file).replace(/\\/g, '/'),
+            // A legacy filename alias is not the authenticated learning subject.
+            entryId: entry.id, learningSubject: userId,
+        }) ?? null : null;
+        return entry;
+    };
     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, usageClock, usageStats),
+        entries: parseLearningEntries(t.path, t.displayPath, cat, tierLevel(t), domainVal ?? 'global', tierVal, threshold, usageStats)
+            .map(entry => decorateAssessment(entry, t.path)),
     })));
     if (scope === 'org') {
         for (const cat of ['avoid', 'preference', 'repeat', 'coaching']) {
@@ -1949,7 +1679,8 @@ function readPreservedLearnings(workspaceRoot, userId, scope, level = 'machine')
                 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, usageClock, usageStats));
+                    out.push(...parseLearningEntries((0, path_1.join)(base, fileName), `${REPO_LEARNINGS_REL}/${fileName}`, cat, 'org', domain, undefined, threshold, usageStats)
+                        .map(entry => decorateAssessment(entry, (0, path_1.join)(base, fileName))));
                 }
                 else {
                     out.push(...parseTiers(domainTiers, cat, undefined, domain));
@@ -1959,7 +1690,6 @@ function readPreservedLearnings(workspaceRoot, userId, scope, level = 'machine')
         return out;
     }
     // The library is an ownership view, not the combined runtime context.
-    const resolvedUserId = resolveLearningUserId(workspaceRoot, userId, roots);
     const cats = scope === 'reverse' ? ['coaching'] : ['avoid', 'preference', 'repeat'];
     for (const cat of cats) {
         const fileType = exports.CATEGORY_TO_FILETYPE[cat];
@@ -2016,7 +1746,7 @@ function applyLearningEntryChange(workspaceRoot, userId, ref, action, payload) {
     const existing = (0, fs_1.existsSync)(filePath) ? (0, fs_1.readFileSync)(filePath, 'utf8') : '';
     const nl = existing.includes('\r\n') ? '\r\n' : '\n';
     const lines = existing.length ? existing.split(/\r?\n/) : [];
-    const headingRe = learningEntryHeadingRegex();
+    const headingRe = (0, learning_entry_format_1.learningEntryHeadingRegex)();
     const starts = [];
     lines.forEach((l, i) => { if (headingRe.test(l))
         starts.push(i); });
@@ -2043,7 +1773,7 @@ function applyLearningEntryChange(workspaceRoot, userId, ref, action, payload) {
     const matches = starts.map((start, index) => ({ start, index })).filter(({ start, index }) => {
         if (payload.originalId !== undefined) {
             const end = index + 1 < starts.length ? starts[index + 1] : lines.length;
-            const ids = lines.slice(start + 1, end).map(line => line.trim().match(LEARNING_ENTRY_ID_LINE)).filter(Boolean);
+            const ids = lines.slice(start + 1, end).map(line => line.trim().match(learning_entry_format_1.LEARNING_ENTRY_ID_LINE)).filter(Boolean);
             return ids.length === 1 && ids[0][1] === payload.originalId;
         }
         return lines[start].match(headingRe)?.[2].trim() === (payload.originalTitle || '').trim();