fraim 2.0.345 → 2.0.348

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.
  *
@@ -186,7 +150,7 @@ function resolvePersonalLearningFile(repoBase, managerCacheBase, managerCacheDis
  * before the change, and the same shadowing applied to company families.
  *
  * Returning all tiers lets the context block list each present path and lets the
- * Hub merge entries with the project entry winning only on a title collision.
+ * Hub retain each current occurrence without treating title collisions as authority.
  */
 function resolvePersonalLearningFileTiers(repoBase, managerCacheBase, managerCacheDisplayBase, globalBase, globalDisplayBase, fileName) {
     const tiers = [];
@@ -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)) },
@@ -723,19 +589,12 @@ function resolveOrgLearningFileTiers(repoLearningsBase, fileName) {
     return tiers;
 }
 /**
- * Issue #1002 R4: merge entries across tiers. Lowest precedence first in, so a
- * later tier overwrites on an identical title and nothing is duplicated. Entries
- * whose titles do not collide all survive, which is the whole point: promoting
- * one entry must not hide the rest of its family.
+ * Keep every current occurrence visible. A title is not identity and neither
+ * tier order nor equal IDs authorize discarding a conflicting revision.
+ * Reconciliation belongs to the reviewed learning application flow.
  */
 function mergeLearningEntriesAcrossTiers(tiers) {
-    const byTitle = new Map();
-    for (const tier of tiers) {
-        for (const entry of tier.entries) {
-            byTitle.set(entry.title.trim().toLowerCase(), entry);
-        }
-    }
-    return [...byTitle.values()];
+    return tiers.flatMap(tier => tier.entries);
 }
 const CONTEXT_CATEGORIES = {
     mistake: { ft: 'mistake-patterns', gated: true, label: '(entries above score threshold)' },
@@ -743,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
@@ -846,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,
     });
@@ -864,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
@@ -910,16 +758,12 @@ 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) {
             section += 'Read the listed synthesized learning files before continuing, then apply the relevant patterns and preferences in this job.\n';
             if (multiTier) {
-                section += 'A family listed at more than one level is combined, not replaced: apply every entry, and where the same entry title appears at two levels apply the more specific one (project over manager).\n';
+                section += 'A family listed at more than one level is combined, not replaced. Stable IDs identify lessons; equal titles do not establish identity. Conflicting copies require explicit reconciliation: do not choose a revision by scope, timestamp, count, or title. Preserve approved decisions and useful refinements; unresolved conflicting claims are not authority for action.\n';
             }
             if (coachPresent) {
                 section += 'Treat manager-coaching as feedback for how the manager should continue or improve managing AI, not as agent instruction.\n';
@@ -929,7 +773,7 @@ function buildLearningContextSection(workspaceRoot, userId, forJob, domain) {
     else if (hasL2 || hasL1) {
         section += 'Read the listed synthesized learning files before continuing, then use this synthesized learning context throughout the session.\n';
         if (multiTier) {
-            section += 'A family listed at more than one level is combined, not replaced: apply every entry, and where the same entry title appears at two levels apply the more specific one (project over manager).\n';
+            section += 'A family listed at more than one level is combined, not replaced. Stable IDs identify lessons; equal titles do not establish identity. Conflicting copies require explicit reconciliation: do not choose a revision by scope, timestamp, count, or title. Preserve approved decisions and useful refinements; unresolved conflicting claims are not authority for action.\n';
         }
         if (coachPresent) {
             section += 'Manager-coaching entries are manager-facing feedback, not instructions for the AI to follow.\n';
@@ -937,187 +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.
- *
- * ## Issue #1752 A2 — one lesson, one offer, stamped or not
- *
- * This deduped by usage key, which sounds like the same thing as the context
- * reader's rule and is not. `resolveOrgLearningFileTiers` deliberately returns
- * both the org pack copy and the repo-local copy of an org family, because #1002
- * R4 requires that a promoted entry not hide the rest of its family. The context
- * reader then merges those tiers with `mergeLearningEntriesAcrossTiers`, keyed on
- * the normalized title, so the agent sees one lesson. Keying the offer record on
- * `id ?? titleUsageKey(...)` instead meant that a lesson carrying an `**Id**` line
- * in one tier and not the other produced two keys, and both were recorded.
- *
- * That is defect 4 in #1752, and it is why `standing --changes-only` could
- * recommend promoting and demoting the same lesson in one response: the
- * title-keyed twin collected the offers while the id-keyed one collected the
- * firings (the firing path never split, because `buildEntryIndex` dedups by
- * identity first-wins). Measured live: five titles emitting two keys each in a
- * single derived offer batch.
- *
- * So group by the same normalized title the context reader groups by, resolve one
- * identity per group preferring whichever tier carries an id, and let the later
- * tier win on content exactly as before.
- */
-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);
-    // Grouped by family plus normalized title, which is what `titleUsageKey`
-    // already computes. Two families holding the same title are two lessons, and
-    // that is the existing rule rather than a new one.
-    const byLesson = new Map();
-    for (const file of [...resolved.l2, ...resolved.l1]) {
-        for (const entry of enumerateFileEntries(file, threshold, usageClock, now)) {
-            const lessonKey = (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()) {
-        // Later tier wins for content, unchanged precedence.
-        const winner = group[group.length - 1];
-        // The id comes from whichever tier carries one, because `stamp-ids` has not
-        // reached the org and manager pack homes for every corpus yet. Latest wins
-        // when more than one tier is stamped.
-        const id = group.reduce((found, entry) => entry.id ?? found, null);
-        out.push({
-            ...winner,
-            id,
-            key: (0, learning_usage_store_1.resolveEntryIdentity)({ id, fileType: winner.fileType, title: winner.title }).key,
-        });
-    }
-    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
@@ -1425,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++;
@@ -1448,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)
@@ -1461,7 +1124,7 @@ function readLearningEntryTitles(filePath) {
  *
  * Issue #1002: two changes from the original.
  *
- * R4: counts sum across every level that holds a family, deduped by entry title,
+ * R4: counts sum across every level that holds a family,
  * rather than counting only the winning file. Under the old first-wins read a
  * family present at two levels was counted once, so the Brain under-reported
  * exactly the case this issue introduces.
@@ -1483,26 +1146,27 @@ function countPreservedLearnings(workspaceRoot, userId) {
         }
         return titles.size;
     };
-    // Same for a personal family, and reported per level so the Brain can show
-    // where each entry actually lives. A title present at both levels counts
-    // once, at the project level, matching which entry the agent applies.
+    // Count current occurrences at their actual owner; reconciliation, not a
+    // title match, determines whether a redundant copy may be removed.
     const countPersonalFamilyByLevel = (fileName) => {
         const tiers = resolvePersonalLearningFileTiers(roots.repoLearningsBase, roots.managerCacheBase, roots.managerCacheDisplayBase, roots.globalPersonalBase, roots.globalPersonalDisplayBase, fileName);
-        const projectTitles = new Set();
-        const managerTitles = new Set();
+        const seenFiles = new Set();
+        const counts = { manager: 0, project: 0 };
         for (const tier of tiers) {
-            for (const t of readLearningEntryTitles(tier.path)) {
-                if (tier.level === 'project')
-                    projectTitles.add(t);
-                else
-                    managerTitles.add(t);
+            let file;
+            try {
+                file = (0, fs_1.realpathSync)(tier.path);
+            }
+            catch {
+                continue;
             }
+            if (seenFiles.has(file))
+                continue;
+            seenFiles.add(file);
+            const owner = tier.level === 'project' ? 'project' : 'manager';
+            counts[owner] += countLearningEntries(tier.path);
         }
-        // A promoted entry lives at the project level; do not also count it as the
-        // manager's, or the two tiles would double-report the same lesson.
-        for (const t of projectTitles)
-            managerTitles.delete(t);
-        return { manager: managerTitles.size, project: projectTitles.size };
+        return counts;
     };
     const PERSONAL_WORK_TYPES = ['mistake-patterns', 'preferences', 'validated-patterns'];
     const ORG_TYPES = ['mistake-patterns', 'preferences', 'manager-coaching', 'validated-patterns'];
@@ -1567,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.
@@ -1583,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 = () => {
@@ -1591,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,
@@ -1612,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;
@@ -1643,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',
             };
         };
     }
@@ -1760,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;
@@ -1782,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;
                 }
@@ -1803,6 +1455,8 @@ function parseLearningEntries(filePath, displayPath, category, level, domain = '
                 current.score = null;
                 current.injected = true;
             }
+            if (tier === 'cold')
+                current.injected = false;
             out.push(current);
         }
         current = null;
@@ -1810,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) {
@@ -1835,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;
@@ -1926,7 +1580,7 @@ function readRejectedProposals(workspaceRoot, userId) {
                 ? proposal.claimKey
                 : parts[1] || heading[2];
             // The same claim can appear in more than one configured base. The first
-            // base that carries it wins, matching how the tier merge resolves.
+            // base supplies display metadata for that same recorded decision.
             const key = `${claimKey}:${String(decision.at || '')}`;
             if (seen.has(key))
                 continue;
@@ -1951,22 +1605,50 @@ 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']) {
             const fileType = exports.CATEGORY_TO_FILETYPE[cat];
             const f = `org-${fileType}.md`;
-            // org pack home first, repo-local override second; repo-local wins on title collision.
+            // Keep occurrences from both homes; title matches do not grant authority.
             out.push(...parseTiers(resolveOrgLearningFileTiers(roots.repoLearningsBase, f), cat));
             // #806: domain-scoped org learnings — union from both locations.
             const allDomainFiles = [
@@ -1981,7 +1663,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));
@@ -1990,46 +1673,17 @@ function readPreservedLearnings(workspaceRoot, userId, scope, level = 'machine')
         }
         return out;
     }
-    // manager + reverse: use the same three-tier resolution as buildLearningContextSection.
-    const resolvedUserId = resolveLearningUserId(workspaceRoot, userId, roots);
+    // The library is an ownership view, not the combined runtime context.
     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, usageClock, usageStats));
-            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, usageClock, usageStats));
-            }
-            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, usageClock, usageStats));
-            }
-        }
-        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`;
-        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`;
-        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));
+        for (const domain of ['global', ...learning_domains_1.LEARNING_DOMAINS]) {
+            const domainPart = domain === 'global' ? '' : `-${domain}`;
+            for (const storageTier of ['hot', 'cold']) {
+                const suffix = storageTier === 'cold' ? COLD_FILE_SUFFIX : '';
+                const fileName = `${resolvedUserId}${domainPart}-${fileType}${suffix}.md`;
+                const tiers = resolvePersonalLearningFileTiers(roots.repoLearningsBase, roots.managerCacheBase, roots.managerCacheDisplayBase, roots.globalPersonalBase, roots.globalPersonalDisplayBase, fileName).filter(t => level === 'project' ? t.level === 'project' : t.level !== 'project');
+                out.push(...parseTiers(tiers, cat, storageTier, domain));
             }
         }
     }
@@ -2076,7 +1730,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); });
@@ -2097,12 +1751,22 @@ function applyLearningEntryChange(workspaceRoot, userId, ref, action, payload) {
         write(preamble + nl + nl + block);
         return { ok: true, path: filePath };
     }
-    const targetIdx = starts.findIndex((s) => {
-        const m = lines[s].match(headingRe);
-        return m && m[2].trim() === (payload.originalTitle || '').trim();
+    if (payload.originalId !== undefined && (typeof payload.originalId !== 'string' || !/^L-[0-9A-Za-z]{6,24}$/.test(payload.originalId))) {
+        throw new Error('Invalid learning entry identity');
+    }
+    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_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();
     });
-    if (targetIdx === -1)
-        throw new Error(`Learning entry not found: "${payload.originalTitle}"`);
+    if (!matches.length)
+        throw new Error(`Learning entry not found: "${payload.originalId ?? payload.originalTitle}"`);
+    if (matches.length !== 1)
+        throw new Error('Learning entry is ambiguous; reconcile duplicate identities or select a unique entry');
+    const targetIdx = matches[0].index;
     const start = starts[targetIdx];
     const end = targetIdx + 1 < starts.length ? starts[targetIdx + 1] : lines.length;
     if (action === 'delete') {