fraim 2.0.346 → 2.0.349

package/dist/src/local-mcp-server/learning-usage-verify.js

@@ -4,28 +4,8 @@ exports.COMPACTION_ADVISORY_BYTES = void 0;
 exports.listUnmergedStoreCopies = listUnmergedStoreCopies;
 exports.findTwinPairs = findTwinPairs;
 exports.buildVerifyReport = buildVerifyReport;
-/**
- * Issue #1752 E2 — the integrity picture for the usage record, in one output.
- *
- * Every defect in issue #1752 was found by a manager reading raw JSON during a
- * `sleep-on-learnings` run, one at a time, over one session. This is the command
- * that answers "can I trust what the record just told me" before acting on a
- * recommendation, and it is the regression guard that makes every defect class
- * visible in one place instead of one at a time.
- *
- * It reports and never blocks. Nothing here treats a finding as a gate: it states
- * what it found, and the manager decides.
- *
- * ## Recovery boundary
- *
- * Rebuild only the snapshot, keeping the event directory intact. The retained
- * events recover the offers and firings they cover; retrospective backfill can
- * supplement older firings but cannot recreate offers. Normal compaction keeps
- * lifetime aggregate totals in the snapshot, so a backup is required before
- * discarding it if history already pruned from events must remain inspectable.
- * Rebuilding cannot invent missing history or fix identities still present in
- * the authoritative events. Verify reports those limitations rather than
- * claiming that deleting the usage directory is a lossless repair.
+/** Explicit historical-store diagnostics. Snapshot/checkpoint/journal provenance must
+ * establish recovery; no diagnostic can reconstruct already discarded observations.
  */
 const fs_1 = require("fs");
 const path_1 = require("path");
@@ -134,22 +114,12 @@ function findTwinPairs(entries) {
  * Split out of `buildVerifyReport` so that function reads as what it is: an
  * assembly of independently-computed parts.
  */
-function scanRecordsForIntegrity(records, offersEraStart) {
-    const invariantViolations = [];
+function scanRecordsForIntegrity(records) {
     const caseVariantModelBuckets = [];
     const unknownModelAttribution = [];
     for (const record of records) {
         const title = record.titles[record.titles.length - 1] ?? record.key;
-        const violation = record.integrity ?? (0, learning_usage_store_1.integrityViolation)(record, offersEraStart);
-        if (violation) {
-            invariantViolations.push({
-                key: record.key, title,
-                offered: record.offered.total,
-                firedInEra: record.offered.total + violation.count,
-                detail: violation.detail,
-            });
-        }
-        for (const buckets of [record.offered.byModel, record.fired.byModel]) {
+        for (const buckets of [record.fired.byModel]) {
             const byNormalized = new Map();
             for (const raw of Object.keys(buckets)) {
                 const normalized = (0, learning_usage_store_1.normalizeModelName)(raw) ?? raw;
@@ -161,15 +131,15 @@ function scanRecordsForIntegrity(records, offersEraStart) {
                 }
             }
         }
-        // Only a clock-advancing firing that produced no bucket means the model went
+        // Only a positive outcome that produced no bucket means the model went
         // unrecorded. An entry whose outcomes are all `ignored` has an empty bucket
         // set correctly, and that is information rather than a gap.
-        const clockAdvancing = record.fired.byOutcome.prevented + record.fired.byOutcome.applied;
+        const positiveOutcomes = record.fired.byOutcome.prevented + record.fired.byOutcome.applied;
         if ((0, learning_usage_eligibility_1.modelAttributionIncomplete)(record)) {
-            unknownModelAttribution.push({ key: record.key, title, fired: clockAdvancing });
+            unknownModelAttribution.push({ key: record.key, title, fired: positiveOutcomes });
         }
     }
-    return { invariantViolations, caseVariantModelBuckets, unknownModelAttribution };
+    return { caseVariantModelBuckets, unknownModelAttribution };
 }
 function buildVerifyReport(options) {
     const now = options.now ?? new Date();
@@ -178,7 +148,7 @@ function buildVerifyReport(options) {
     const snapshot = (0, learning_usage_store_1.readUsageSnapshot)(options.snapshotPath);
     const logs = (0, learning_usage_events_1.listEventLogs)(eventDir);
     const records = Object.values(store.entries);
-    const { invariantViolations, caseVariantModelBuckets, unknownModelAttribution } = scanRecordsForIntegrity(records, store.offersEraStart);
+    const { caseVariantModelBuckets, unknownModelAttribution } = scanRecordsForIntegrity(records);
     const twins = findTwinPairs(store.entries);
     const storeCopies = listUnmergedStoreCopies(options.snapshotPath);
     const checkpoints = snapshot.foldedThroughBytes ?? {};
@@ -191,7 +161,6 @@ function buildVerifyReport(options) {
         entries: records.length,
         twinPairs: twins.pairs,
         unmatchedTitleKeys: twins.unmatched,
-        invariantViolations,
         caseVariantModelBuckets,
         unknownModelAttribution,
         storeCopies,

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

@@ -77,7 +77,6 @@ const usage_collector_js_1 = require("./usage-collector.js");
 const otlp_metrics_receiver_js_1 = require("./otlp-metrics-receiver.js");
 const token_adapter_registry_js_1 = require("./token-adapter-registry.js");
 const learning_context_builder_js_1 = require("./learning-context-builder.js");
-const learning_usage_store_js_1 = require("./learning-usage-store.js");
 const learning_domains_js_1 = require("../config/learning-domains.js");
 const skill_include_dedup_js_1 = require("./skill-include-dedup.js");
 const git_utils_1 = require("../core/utils/git-utils");
@@ -1658,8 +1657,7 @@ class FraimLocalMCPServer {
         if (request.method !== 'tools/call' || typeof toolName !== 'string') {
             return this.processResponseWithHydration(response, requestSessionId);
         }
-        const projectRoot = this.findProjectRoot();
-        const resolver = projectRoot ? this.getRegistryResolver(requestSessionId) : null;
+        const resolver = this.getRegistryResolver(requestSessionId);
         let finalizedResponse = response;
         // 1. Resolve top-level override if this tool maps to a specific file.
         if (resolver) {
@@ -1671,14 +1669,14 @@ class FraimLocalMCPServer {
             else if (toolName === 'get_fraim_job')
                 registryPath = await resolver.findRegistryPath('jobs', args.job);
             if (registryPath) {
-                const localOverride = resolver.hasLocalOverride(registryPath);
+                const localOverride = resolver.hasPersonalizedOverride(registryPath);
                 const remoteFailed = Boolean(response.error);
                 if (response.error) {
                     this.log(`[req:${requestId}] Remote error for ${registryPath}: code=${response.error.code}`);
                 }
                 if (localOverride || remoteFailed) {
                     try {
-                        // personalized-employee/ takes priority; when remote fails, resolver falls back to synced/cached content
+                        // Every configured personalized layer takes priority over the remote baseline.
                         const resolved = await resolver.resolveFile(registryPath).catch(() => null);
                         if (resolved?.content) {
                             const newResponse = {
@@ -1731,7 +1729,6 @@ class FraimLocalMCPServer {
                         responseText += teamContextSection;
                         this.log(`[req:${requestId}] Injected team context from ${workspaceRoot}`);
                     }
-                    this.recordUsageOffers(workspaceRoot, userEmail, false, null, 'session', requestId);
                 }
                 if (this.latestConnectSyncWarning) {
                     responseText += `\n\n## Local Catalog\n${this.latestConnectSyncWarning}`;
@@ -1774,54 +1771,8 @@ class FraimLocalMCPServer {
             text += `\n\n---` + learningSection + teamContextSection;
             this.log(`[req:${requestId}] Injected job-focus learning/team context for ${userEmail} (domain: ${jobDomain ?? 'global'})`);
         }
-        this.recordUsageOffers(workspaceRoot, userEmail, true, jobDomain, String(jobName ?? 'unknown'), requestId);
         return text;
     }
-    /**
-     * Issue #1103 R1, R2 and R3: record what this context injection delivered.
-     *
-     * Written here because this is the only place that knows all of it at once: the
-     * files that were listed, the job, the agent, and the model. Nothing depends on
-     * the agent reading anything, which is R2, and the three auto-loaded rule files
-     * get a record for the first time, which is R3.
-     *
-     * Never throws and never blocks the response. Usage data is derived and optional:
-     * a failure here must not change what the agent is given.
-     */
-    recordUsageOffers(workspaceRoot, userEmail, forJob, domain, job, requestId) {
-        try {
-            const offered = [
-                ...(0, learning_context_builder_js_1.collectOfferedLearningEntries)(workspaceRoot, userEmail, forJob, domain),
-                ...(0, learning_context_builder_js_1.collectOfferedRuleFiles)(workspaceRoot, forJob),
-            ];
-            if (offered.length === 0)
-                return;
-            const collector = this.usageCollector;
-            const date = new Date().toISOString().slice(0, 10);
-            const limits = (0, learning_usage_store_js_1.resolveUsageLimits)(workspaceRoot);
-            const result = (0, learning_usage_store_js_1.recordOffers)(offered.map((entry) => ({
-                key: entry.key,
-                family: entry.family,
-                title: entry.title,
-                fileType: entry.fileType,
-                level: entry.level,
-                aboveThreshold: entry.aboveThreshold,
-                job,
-                date,
-                agent: collector?.getAgentName() ?? null,
-                // Issue #1752 C2: one model, one bucket. `GPT-5` and `gpt-5` were two,
-                // splitting the denominator of every per-model conclusion about it.
-                model: (0, learning_usage_store_js_1.normalizeModelName)(collector?.getAgentModel()),
-            })), { limits });
-            this.log(`[req:${requestId}] Recorded ${result.recorded} usage offers for job ${job} (write mode: append)`);
-        }
-        catch (error) {
-            // Issue #1752: name the log and the write mode. During the migration window
-            // "which path did this write take" is the first question on any report of
-            // missing data, and the answer was not in the logs.
-            this.log(`[req:${requestId}] Usage offer recording skipped (append, log ${(0, learning_usage_store_js_1.resolveEventDir)((0, learning_usage_store_js_1.resolveUsageStorePath)())}): ${error instanceof Error ? error.message : String(error)}`);
-        }
-    }
     /**
      * Resolve the learning domain for a `get_fraim_job` request (issue #806) from
      * the job's registry path, so the loader injects global + that domain's
@@ -2038,75 +1989,60 @@ class FraimLocalMCPServer {
         const { getEffectiveFraimDir } = require('../core/utils/project-fraim-paths');
         const effectiveFraimDir = projectRoot
             ? getEffectiveFraimDir(projectRoot)
-            : getEffectiveFraimDir(process.cwd());
+            : (0, project_fraim_paths_1.getUserFraimDirPath)();
         if (effectiveFraimDir) {
             this.log(`📂 Effective FRAIM dir: ${effectiveFraimDir}`);
         }
-        if (!projectRoot && !effectiveFraimDir) {
-            this.log('⚠️ No project root or user-level FRAIM found, override resolution disabled');
-            // Return a resolver that always falls back to remote
-            return new local_registry_resolver_1.LocalRegistryResolver({
-                workspaceRoot: process.cwd(),
-                // Issue #1043 round 2: overlay roots are the layers' homes. There is no
-                // separate cache to point at any more.
-                managerCacheRoot: (0, pack_home_1.resolvePackHome)('manager').contentRoot,
-                orgCacheRoot: (0, pack_home_1.resolvePackHome)('org').contentRoot,
-                remoteContentResolver: async (_path) => {
-                    throw new Error('No project root available');
-                },
-                shouldFilter: (content) => this.isStub(content)
-            });
-        }
-        else {
-            // Determine if we need effectiveFraimDir override.
-            // If the effective dir is user-level (~/.fraim/), it's NOT under projectRoot/fraim/,
-            // so we pass it as effectiveFraimDir to bypass the getWorkspaceFraimPath logic.
-            const workspaceRoot = projectRoot || process.cwd();
-            const projectFraimDir = projectRoot ? (0, path_1.join)(projectRoot, 'fraim') : '';
-            const needsEffectiveDirOverride = effectiveFraimDir && effectiveFraimDir !== projectFraimDir;
-            return new local_registry_resolver_1.LocalRegistryResolver({
-                workspaceRoot,
-                ...(needsEffectiveDirOverride ? { effectiveFraimDir } : {}),
-                // Issue #1043 round 2: overlay roots are the layers' homes.
-                managerCacheRoot: (0, pack_home_1.resolvePackHome)('manager').contentRoot,
-                orgCacheRoot: (0, pack_home_1.resolvePackHome)('org').contentRoot,
-                shouldFilter: (content) => this.isStub(content),
-                remoteContentResolver: async (path) => {
-                    // Fetch parent content from remote for inheritance
-                    this.log(`🔄 Remote content resolver: fetching ${path}`);
-                    this.log(`🔄 Fetching raw file content: ${path}`);
-                    const request = {
-                        jsonrpc: '2.0',
-                        id: (0, crypto_1.randomUUID)(),
-                        method: 'tools/call',
-                        params: {
-                            name: 'get_fraim_file',
-                            arguments: {
-                                path,
-                                raw: true
-                            }
+        // Determine if we need effectiveFraimDir override.
+        // If the effective dir is user-level (~/.fraim/), it's NOT under projectRoot/fraim/,
+        // so we pass it as effectiveFraimDir to bypass the getWorkspaceFraimPath logic.
+        const workspaceRoot = projectRoot || (0, project_fraim_paths_1.getUserFraimDirPath)();
+        const projectFraimDir = projectRoot ? (0, path_1.join)(projectRoot, 'fraim') : '';
+        const needsEffectiveDirOverride = effectiveFraimDir && effectiveFraimDir !== projectFraimDir;
+        return new local_registry_resolver_1.LocalRegistryResolver({
+            workspaceRoot,
+            userLevelOnly: !projectRoot,
+            ...(needsEffectiveDirOverride ? { effectiveFraimDir } : {}),
+            // Issue #1043 round 2: overlay roots are the layers' homes.
+            managerLocalRoot: (0, pack_home_1.resolvePackHome)('manager', { migrateLegacy: false }).contentRoot,
+            managerCacheRoot: (0, pack_home_1.resolvePackHome)('manager', { migrateLegacy: false }).contentRoot,
+            orgCacheRoot: (0, pack_home_1.resolvePackHome)('org', { migrateLegacy: false }).contentRoot,
+            shouldFilter: (content) => this.isStub(content),
+            remoteContentResolver: async (path) => {
+                // Fetch parent content from remote for inheritance
+                this.log(`🔄 Remote content resolver: fetching ${path}`);
+                this.log(`🔄 Fetching raw file content: ${path}`);
+                const request = {
+                    jsonrpc: '2.0',
+                    id: (0, crypto_1.randomUUID)(),
+                    method: 'tools/call',
+                    params: {
+                        name: 'get_fraim_file',
+                        arguments: {
+                            path,
+                            raw: true
                         }
-                    };
-                    this.applyRequestSessionId(request, requestSessionId);
-                    const response = await this._doProxyToRemote(request);
-                    if (response.error) {
-                        this.logError(`❌ Remote content resolver failed: ${response.error.message}`);
-                        throw new Error(`Failed to fetch parent: ${response.error.message}`);
                     }
-                    // Extract content from MCP response format
-                    if (response.result?.content && Array.isArray(response.result.content)) {
-                        const textContent = response.result.content.find((c) => c.type === 'text');
-                        if (textContent?.text) {
-                            this.log(`✅ Remote content resolver: fetched ${textContent.text.length} chars`);
-                            return textContent.text;
-                        }
+                };
+                this.applyRequestSessionId(request, requestSessionId);
+                const response = await this._doProxyToRemote(request);
+                if (response.error) {
+                    this.logError(`❌ Remote content resolver failed: ${response.error.message}`);
+                    throw new Error(`Failed to fetch parent: ${response.error.message}`);
+                }
+                // Extract content from MCP response format
+                if (response.result?.content && Array.isArray(response.result.content)) {
+                    const textContent = response.result.content.find((c) => c.type === 'text');
+                    if (textContent?.text) {
+                        this.log(`✅ Remote content resolver: fetched ${textContent.text.length} chars`);
+                        return textContent.text;
                     }
-                    this.logError(`❌ Remote content resolver: no content in response for ${path}`);
-                    this.logError(`Response: ${JSON.stringify(response, null, 2)}`);
-                    throw new Error(`No content in remote response for ${path}`);
                 }
-            });
-        }
+                this.logError(`❌ Remote content resolver: no content in response for ${path}`);
+                this.logError(`Response: ${JSON.stringify(response, null, 2)}`);
+                throw new Error(`No content in remote response for ${path}`);
+            }
+        });
     }
     /**
      * Check if content is a "stub" indicating local version should be ignored.

package/dist/src/mcp/tool-schemas.js

@@ -201,6 +201,7 @@ Example:
             description: `Get a specific skill, rule, or reference file from the FRAIM registry by path.
 
 For running FRAIM jobs, use get_fraim_job instead — do NOT call get_fraim_file for job execution.
+For read-only diagnosis or delivery verification, use get_fraim_file({ path: "jobs/<category>/<job>.md", raw: true }). This does not start a job; the local proxy resolves effective overrides, inheritance, includes, and template values.
 
 Examples:
 - get_fraim_file({ path: "skills/communication/active-listening.md" })
@@ -226,7 +227,7 @@ Examples:
             name: 'get_fraim_job',
             description: `Execute a named FRAIM job after explicit FRAIM activation. Call this tool whenever the user asks to run, start, or execute a named FRAIM job. Returns phased instructions — follow each phase and call seekMentoring to advance phases.
 
-Do NOT use get_fraim_file to load job content. Always use get_fraim_job for job execution.
+Use this tool to start job execution, not to inspect candidate jobs. Read-only inspection uses get_fraim_file with the catalog job path and raw: true.
 
 Examples:
 - get_fraim_job({ job: "feature-specification" })

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "fraim",
-  "version": "2.0.346",
+  "version": "2.0.349",
   "description": "FRAIM core CLI and MCP package.",
   "main": "index.js",
   "bin": {
@@ -13,7 +13,15 @@
     "dist/registry/scripts/learning/entry-format.js",
     "dist/registry/scripts/learning/completion-state.js",
     "dist/registry/scripts/learning/corpus-audit.js",
+    "dist/registry/scripts/learning/entry-authoring.js",
     "dist/registry/scripts/learning/owner-identity.js",
+    "dist/registry/scripts/learning/outcome-types.js",
+    "dist/registry/scripts/learning/outcome-assessment.js",
+    "dist/registry/scripts/learning/proposal-queue.js",
+    "dist/registry/scripts/learning/run-record.js",
+    "dist/registry/scripts/learning/run-application.js",
+    "dist/registry/scripts/learning/run-publication.js",
+    "dist/registry/scripts/learning-usage/learning-source-record.js",
     "dist/src/local-mcp-server/",
     "dist/src/cli/",
     "!dist/src/cli/fraim-hub.js",

package/dist/src/local-mcp-server/learning-firing-parser.js

@@ -1,247 +0,0 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.extractLegacyTitle = extractLegacyTitle;
-exports.parseFiringSection = parseFiringSection;
-/**
- * Issue #1103 — reading the firing section a retrospective attests.
- *
- * Pure parsing, no I/O and no store access: it turns the markdown of
- * `## Where Past Learnings Actually Fired` into typed items plus an explicit
- * section state, so silence, an unfilled template, and a genuinely empty section
- * are three different answers rather than one.
- *
- * Extracted from `learning-usage-attestation.ts`, which now owns resolving those
- * items to entries and writing the record. The split is along the boundary the
- * file already had: what a retrospective says, versus what that means for the
- * corpus.
- *
- * Matching is deliberately strict elsewhere; extraction here is deliberately
- * generous. R15 requires exact-title matching, and says nothing about how
- * tolerantly the title is read off the line. A narrow extractor would report most
- * of the corpus as unmatched for a formatting reason dressed up as an identity
- * one.
- */
-const learning_usage_store_1 = require("./learning-usage-store");
-const FIRING_SECTION_HEADING = /^#{1,6}\s+Where Past Learnings Actually Fired\s*$/i;
-const NEXT_SECTION_HEADING = /^#{1,6}\s+\S/;
-/**
- * The structured form R4 requires:
- *   `1. **[L-4f2a91c0de] Entry Title** - outcome: \`prevented\` - one sentence.`
- * The separator may be a hyphen, an en dash, or an em dash, because a human editing
- * a retrospective will not be careful about which.
- */
-// The id pattern matches what the entry reader accepts, not only what the generator
-// emits: a hand-written id in a learning file has to be referenceable from a
-// retrospective, or the two sides disagree about what an id is.
-const STRUCTURED_ITEM = /^\d+\.\s*\*\*\[(L-[0-9A-Za-z]{6,24})\]\s*(.+?)\*\*\s*[-–—:]+\s*outcome:\s*`?([A-Za-z-]+)`?\s*[-–—:]*\s*(.*)$/i;
-/** The same, without an id, for an entry the id migration has not reached. */
-const STRUCTURED_ITEM_NO_ID = /^\d+\.\s*\*\*(.+?)\*\*\s*[-–—:]+\s*outcome:\s*`?([A-Za-z-]+)`?\s*[-–—:]*\s*(.*)$/i;
-/**
- * The legacy form: a numbered item whose leading bold segment names the entry.
- *
- * Measured across the real corpus (346 retrospectives, 828 numbered items in this
- * section, 819 of them with a leading bold segment), the shapes in use are:
- *
- *   1. **Pattern: "Entry Title" (graduated standing rule)** - prose
- *   1. **`[P-HIGH] Entry Title` (score 19.0, PRs #507 and #518)** - prose
- *   1. **Entry Title**: prose
- *   1. **`someIdentifier` gate**: prose
- *
- * Extraction is deliberately generous and matching is deliberately strict. R15
- * requires exact-title matching; it says nothing about how tolerantly the title is
- * read off the line. A narrow extractor would report most of the corpus as
- * unmatched for a formatting reason rather than an identity one, which is a
- * measurement error dressed up as a finding.
- */
-const LEGACY_ITEM = /^\d+\.\s*(.*)$/;
-const LEGACY_BOLD_SEGMENT = /^\*\*(.+?)\*\*\s*(.*)$/;
-/** An explicit statement that nothing fired (R4's second criterion). */
-const DECLARED_NONE = /^[-*]?\s*none\b/i;
-/** Placeholder text from the unfilled template, which must never be read as a firing. */
-const TEMPLATE_PLACEHOLDER = /<[^>]*>|\{[^}]*\}/;
-/** Split a bare `<title><separator><prose>` string on the first colon or spaced dash. */
-function splitTitleAndNote(text) {
-    const split = text.match(/^(.+?)\s*(?::|\s[-–—]\s)\s*(.*)$/);
-    return {
-        title: extractLegacyTitle(split ? split[1] : text),
-        note: split ? split[2].trim() : '',
-    };
-}
-/**
- * Read a legacy numbered item into a title and a one-sentence note.
- *
- * The leading bold segment usually names the entry, but in the oldest form the
- * bold segment is only the template's field label (`**Pattern**:`) and the title
- * follows it. Both are handled, because both are in the corpus.
- */
-/** The old template's field label. Only this justifies reading the title from what follows. */
-const LEGACY_FIELD_LABEL = /^pattern\s*:?\s*$/i;
-function parseLegacyItem(rest) {
-    const bold = rest.match(LEGACY_BOLD_SEGMENT);
-    if (bold) {
-        const remainder = bold[2].replace(/^\s*[-–—:]+\s*/, '').trim();
-        const boldTitle = extractLegacyTitle(bold[1]);
-        if (boldTitle)
-            return { title: boldTitle, note: remainder };
-        // The bold segment reduced to nothing. Read the title from what follows only
-        // when the segment was the template's field label; otherwise the bold text was
-        // something else that stripped away (a bare score parenthetical, say) and
-        // taking the prose after it as a title would attribute a firing to a sentence.
-        if (!LEGACY_FIELD_LABEL.test(bold[1].trim()))
-            return null;
-        const fromRemainder = splitTitleAndNote(remainder);
-        return fromRemainder.title ? { title: fromRemainder.title, note: fromRemainder.note } : null;
-    }
-    const bare = splitTitleAndNote(rest);
-    return bare.title ? { title: bare.title, note: bare.note } : null;
-}
-function extractSection(content) {
-    const lines = content.split(/\r?\n/);
-    const start = lines.findIndex((line) => FIRING_SECTION_HEADING.test(line.trim()));
-    if (start < 0)
-        return null;
-    const body = [];
-    for (let i = start + 1; i < lines.length; i++) {
-        const line = lines[i];
-        if (NEXT_SECTION_HEADING.test(line.trim()) && !FIRING_SECTION_HEADING.test(line.trim()))
-            break;
-        body.push(line);
-    }
-    return body;
-}
-function normalizeOutcome(raw) {
-    const candidate = raw.trim().toLowerCase();
-    return learning_usage_store_1.FIRING_OUTCOMES.includes(candidate) ? candidate : null;
-}
-/**
- * Read the entry title a legacy item is naming, stripping the decoration the
- * corpus actually carries: a `Pattern:` label, backticks, a severity or graduation
- * tag, a trailing score parenthetical, and surrounding quotes.
- *
- * Returns null when nothing usable is left, which is a reportable outcome rather
- * than a guess.
- */
-function extractLegacyTitle(boldSegment) {
-    let title = boldSegment.trim();
-    // A `Pattern:` or `Pattern` label, which is the old template's field name.
-    title = title.replace(/^pattern\s*:?\s*/i, '');
-    title = title.replace(/`/g, '');
-    // Trailing parentheticals: score, recurrence counts, PR references, corpus notes.
-    // Repeated because some items carry more than one.
-    let previous;
-    do {
-        previous = title;
-        title = title.replace(/\s*\([^()]*\)\s*$/, '').trim();
-    } while (title !== previous);
-    // A severity or graduation tag prefix.
-    title = title.replace(/^\[(P-CRITICAL|P-HIGH|P-MED|P-LOW|GRADUATED)\]\s*/i, '');
-    // Surrounding quotes, straight or curly.
-    title = title.replace(/^["'“‘](.*)["'”’]$/, '$1');
-    // A trailing corpus qualifier the corpus writes after a comma or dash, such as
-    // "(engineering corpus)" already removed above, or "- L2 org".
-    title = title.replace(/\s*[-–—]\s*(L2 org|L1|org)\s*$/i, '').trim();
-    return title.length > 0 ? title : null;
-}
-function parseFiringSection(content, options) {
-    const body = extractSection(content);
-    if (body === null) {
-        return { sectionState: 'absent', declaredNone: false, items: [], rejected: [] };
-    }
-    const lines = body.map((l) => l.trim()).filter(Boolean);
-    if (lines.length === 0) {
-        return { sectionState: 'empty', declaredNone: false, items: [], rejected: [] };
-    }
-    const items = [];
-    const rejected = [];
-    let sawDeclaredNone = false;
-    let sawPlaceholderItem = false;
-    let inFence = false;
-    for (const line of lines) {
-        // A fenced block in this section is illustration, not attestation. The template
-        // carries a worked example in one, and copying the template must not create
-        // firings for the entries the example names.
-        if (/^```/.test(line)) {
-            inFence = !inFence;
-            continue;
-        }
-        if (inFence)
-            continue;
-        if (DECLARED_NONE.test(line)) {
-            sawDeclaredNone = true;
-            continue;
-        }
-        // A line still carrying template placeholders is the unfilled template, not an
-        // attestation. Recording it would create a firing for an entry named "<entry
-        // title>", which is exactly the silent-garbage case R4's second criterion
-        // exists to make visible.
-        const isPlaceholder = TEMPLATE_PLACEHOLDER.test(line);
-        const structured = line.match(STRUCTURED_ITEM);
-        if (structured) {
-            if (isPlaceholder) {
-                sawPlaceholderItem = true;
-                continue;
-            }
-            const outcome = normalizeOutcome(structured[3]);
-            if (!outcome) {
-                rejected.push({ raw: line, reason: `Unknown outcome "${structured[3]}". Use one of: ${learning_usage_store_1.FIRING_OUTCOMES.join(', ')}.` });
-                continue;
-            }
-            items.push({ id: structured[1], title: structured[2].trim(), outcome, note: structured[4].trim(), format: 'structured', raw: line });
-            continue;
-        }
-        const noId = line.match(STRUCTURED_ITEM_NO_ID);
-        if (noId) {
-            if (isPlaceholder) {
-                sawPlaceholderItem = true;
-                continue;
-            }
-            // `**Pattern**: …` also matches this shape; keep it in the legacy lane.
-            if (/^pattern$/i.test(noId[1].trim())) {
-                rejected.push({ raw: line, reason: 'Legacy `**Pattern**:` item carrying an outcome is ambiguous. Write it in the structured form.' });
-                continue;
-            }
-            const outcome = normalizeOutcome(noId[2]);
-            if (!outcome) {
-                rejected.push({ raw: line, reason: `Unknown outcome "${noId[2]}". Use one of: ${learning_usage_store_1.FIRING_OUTCOMES.join(', ')}.` });
-                continue;
-            }
-            items.push({ id: null, title: noId[1].trim(), outcome, note: noId[3].trim(), format: 'structured-no-id', raw: line });
-            continue;
-        }
-        const legacy = options.legacy ? line.match(LEGACY_ITEM) : null;
-        if (legacy) {
-            if (isPlaceholder) {
-                sawPlaceholderItem = true;
-                continue;
-            }
-            const parsedLegacy = parseLegacyItem(legacy[1].trim());
-            if (!parsedLegacy) {
-                rejected.push({ raw: line, reason: 'No entry title could be read off this item.' });
-                continue;
-            }
-            const { title, note } = parsedLegacy;
-            // Membership in this section is the attestation that the entry fired and
-            // changed something, which is what the section's own template line says. The
-            // outcome is not inferred from the prose.
-            items.push({ id: null, title, outcome: 'applied', note, format: 'legacy', raw: line });
-            continue;
-        }
-        // A list item still carrying template placeholders is the unfilled template
-        // rather than an attestation, and is what distinguishes it from a genuinely
-        // empty section.
-        if (isPlaceholder && /^(\d+\.|[-*])\s/.test(line))
-            sawPlaceholderItem = true;
-    }
-    // A real item wins over a stray "None" line, because a template that has been
-    // filled in often still carries the guidance line underneath the items.
-    let sectionState;
-    if (items.length > 0 || rejected.length > 0)
-        sectionState = 'filled';
-    else if (sawDeclaredNone)
-        sectionState = 'declared-none';
-    else if (sawPlaceholderItem)
-        sectionState = 'unfilled-template';
-    else
-        sectionState = 'empty';
-    return { sectionState, declaredNone: sectionState === 'declared-none', items, rejected };
-}