@skillsmith/mcp-server 0.4.13 → 0.5.1

package/dist/src/audit/audit-report-writer.d.ts

@@ -0,0 +1,82 @@
+/**
+ * @fileoverview Markdown audit-report writer for SMI-4587 Wave 1 Step 7/8.
+ * @module @skillsmith/mcp-server/audit/audit-report-writer
+ *
+ * Renders an `InventoryAuditResult` into the markdown report stored at
+ * `~/.skillsmith/audits/<auditId>/report.md`. Atomic via tmp-file +
+ * `fs.rename` (mirrors `audit-history.ts`).
+ *
+ * Sections, in order (plan §446):
+ *   1. Summary header — auditId, generated-at, totals
+ *   2. CLAUDE.md scan caveat — only when any inventory entry is
+ *      `kind: 'claude_md_rule'` (D-ANTI-1)
+ *   3. Exact collisions — each lists involved entries with absolute paths
+ *   4. Generic flags — matched tokens, suggested rename if any
+ *   5. Semantic collisions — cosine score, overlapping phrases
+ *   6. Recommended edits — Wave 3 plumbing; Wave 1 emits a placeholder
+ *
+ * Wave 2/4 import this writer via `@skillsmith/mcp-server/audit` (Step 9
+ * barrel).
+ */
+import type { InventoryAuditResult } from './collision-detector.types.js';
+import type { RenameSuggestion } from './rename-engine.types.js';
+import type { RecommendedEdit } from './edit-suggester.types.js';
+export interface AuditReportRenderOptions {
+    /**
+     * Override the report's "Generated" timestamp. Defaults to a fresh
+     * `new Date()` at render time. Tests pin this to keep snapshots
+     * deterministic; production callers pass nothing.
+     */
+    generatedAt?: Date;
+    /**
+     * Per-collision rename suggestions to render in the "Recommended edits"
+     * section (SMI-4588 Wave 2 PR #4 / Step 8). When provided AND non-empty,
+     * the writer replaces the Wave 1 placeholder with a table of
+     * `currentName → suggested` pairs plus a copy-paste-ready CLI
+     * invocation per row. Pass nothing (or an empty array) to keep the
+     * Wave 1 placeholder behavior — backward-compatible with existing
+     * audit-report consumers.
+     *
+     * Wave 4 wires this from `runInstallPreflight` /
+     * `generateRenameSuggestions` outputs; Wave 2 ships only the
+     * rendering surface.
+     */
+    renameSuggestions?: ReadonlyArray<RenameSuggestion>;
+    /**
+     * Recommended prose edits to render in the "Recommended Edits"
+     * section (SMI-4589 Wave 3). When provided AND non-empty, the writer
+     * renders each edit as a `diff` fenced markdown block per plan §2.
+     * Pass nothing (or an empty array) to omit the section entirely.
+     *
+     * Wave 4 wires this from `runEditSuggester` outputs; Wave 3 ships
+     * only the rendering surface here. Per the per-template gate
+     * ratified 2026-05-01, only `add_domain_qualifier`-pattern edits
+     * surface in v1; failing-template edits are absent from
+     * `runEditSuggester`'s output entirely.
+     */
+    recommendedEdits?: ReadonlyArray<RecommendedEdit>;
+}
+export interface AuditReportWriteOptions extends AuditReportRenderOptions {
+    /**
+     * Per-audit directory (sibling of `result.json`). The writer assumes the
+     * directory already exists — `writeAuditHistory` creates it.
+     */
+    auditDir: string;
+}
+export interface AuditReportWriteResult {
+    /** Absolute path to the rendered `report.md`. */
+    reportPath: string;
+}
+/**
+ * Render an `InventoryAuditResult` into the audit-report markdown body.
+ * Pure — no IO. Exposed so tests can inspect the output without round-
+ * tripping through the filesystem.
+ */
+export declare function renderAuditReport(result: InventoryAuditResult, opts?: AuditReportRenderOptions): string;
+/**
+ * Persist the rendered report to `<auditDir>/report.md`. Atomic via
+ * tmp-file + `fs.rename`, matching the audit-history writer's contract so
+ * concurrent readers never observe a partially-written file.
+ */
+export declare function writeAuditReport(result: InventoryAuditResult, opts: AuditReportWriteOptions): Promise<AuditReportWriteResult>;
+//# sourceMappingURL=audit-report-writer.d.ts.map

package/dist/src/audit/audit-report-writer.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"audit-report-writer.d.ts","sourceRoot":"","sources":["../../../src/audit/audit-report-writer.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;GAmBG;AAKH,OAAO,KAAK,EAIV,oBAAoB,EAErB,MAAM,+BAA+B,CAAA;AAEtC,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,0BAA0B,CAAA;AAChE,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,2BAA2B,CAAA;AAEhE,MAAM,WAAW,wBAAwB;IACvC;;;;OAIG;IACH,WAAW,CAAC,EAAE,IAAI,CAAA;IAClB;;;;;;;;;;;;OAYG;IACH,iBAAiB,CAAC,EAAE,aAAa,CAAC,gBAAgB,CAAC,CAAA;IACnD;;;;;;;;;;;OAWG;IACH,gBAAgB,CAAC,EAAE,aAAa,CAAC,eAAe,CAAC,CAAA;CAClD;AAED,MAAM,WAAW,uBAAwB,SAAQ,wBAAwB;IACvE;;;OAGG;IACH,QAAQ,EAAE,MAAM,CAAA;CACjB;AAED,MAAM,WAAW,sBAAsB;IACrC,iDAAiD;IACjD,UAAU,EAAE,MAAM,CAAA;CACnB;AAED;;;;GAIG;AACH,wBAAgB,iBAAiB,CAC/B,MAAM,EAAE,oBAAoB,EAC5B,IAAI,GAAE,wBAA6B,GAClC,MAAM,CAoCR;AAED;;;;GAIG;AACH,wBAAsB,gBAAgB,CACpC,MAAM,EAAE,oBAAoB,EAC5B,IAAI,EAAE,uBAAuB,GAC5B,OAAO,CAAC,sBAAsB,CAAC,CAYjC"}

package/dist/src/audit/audit-report-writer.js

@@ -0,0 +1,241 @@
+/**
+ * @fileoverview Markdown audit-report writer for SMI-4587 Wave 1 Step 7/8.
+ * @module @skillsmith/mcp-server/audit/audit-report-writer
+ *
+ * Renders an `InventoryAuditResult` into the markdown report stored at
+ * `~/.skillsmith/audits/<auditId>/report.md`. Atomic via tmp-file +
+ * `fs.rename` (mirrors `audit-history.ts`).
+ *
+ * Sections, in order (plan §446):
+ *   1. Summary header — auditId, generated-at, totals
+ *   2. CLAUDE.md scan caveat — only when any inventory entry is
+ *      `kind: 'claude_md_rule'` (D-ANTI-1)
+ *   3. Exact collisions — each lists involved entries with absolute paths
+ *   4. Generic flags — matched tokens, suggested rename if any
+ *   5. Semantic collisions — cosine score, overlapping phrases
+ *   6. Recommended edits — Wave 3 plumbing; Wave 1 emits a placeholder
+ *
+ * Wave 2/4 import this writer via `@skillsmith/mcp-server/audit` (Step 9
+ * barrel).
+ */
+import * as fs from 'node:fs/promises';
+import * as path from 'node:path';
+/**
+ * Render an `InventoryAuditResult` into the audit-report markdown body.
+ * Pure — no IO. Exposed so tests can inspect the output without round-
+ * tripping through the filesystem.
+ */
+export function renderAuditReport(result, opts = {}) {
+    const generatedAt = opts.generatedAt ?? new Date();
+    const sections = [];
+    sections.push(renderSummaryHeader(result, generatedAt));
+    if (containsClaudeMdRule(result)) {
+        sections.push(renderClaudeMdCaveat());
+    }
+    if (result.exactCollisions.length > 0) {
+        sections.push(renderExactCollisions(result.exactCollisions));
+    }
+    if (result.genericFlags.length > 0) {
+        sections.push(renderGenericFlags(result.genericFlags));
+    }
+    if (result.semanticCollisions.length > 0) {
+        sections.push(renderSemanticCollisions(result.semanticCollisions));
+    }
+    sections.push(renderRecommendedEdits(opts.renameSuggestions, result.auditId));
+    // SMI-4589 Wave 3: prose-edit suggestions render in their own section
+    // immediately after the Wave 2 rename-suggestion section. Empty input
+    // omits the section entirely (no placeholder text — keeps the report
+    // tight when no semantic collisions fired).
+    if (opts.recommendedEdits && opts.recommendedEdits.length > 0) {
+        sections.push(renderRecommendedEditsSection(opts.recommendedEdits));
+    }
+    // Single trailing newline. `trimEnd()` (not `/\n+$/`) is intentional —
+    // see SMI-4733: any non-`\n` trailing whitespace that leaks in is
+    // acceptable to strip in an ASCII report, and the non-regex form
+    // sidesteps polynomial backtracking flagged by CodeQL.
+    return sections.join('\n').trimEnd() + '\n';
+}
+/**
+ * Persist the rendered report to `<auditDir>/report.md`. Atomic via
+ * tmp-file + `fs.rename`, matching the audit-history writer's contract so
+ * concurrent readers never observe a partially-written file.
+ */
+export async function writeAuditReport(result, opts) {
+    const reportPath = path.join(opts.auditDir, 'report.md');
+    const tmpPath = `${reportPath}.tmp`;
+    await fs.mkdir(opts.auditDir, { recursive: true });
+    const body = renderAuditReport(result, {
+        generatedAt: opts.generatedAt,
+        renameSuggestions: opts.renameSuggestions,
+        recommendedEdits: opts.recommendedEdits,
+    });
+    await fs.writeFile(tmpPath, body, 'utf-8');
+    await fs.rename(tmpPath, reportPath);
+    return { reportPath };
+}
+// ---------------------------------------------------------------------------
+// Section renderers
+// ---------------------------------------------------------------------------
+function renderSummaryHeader(result, generatedAt) {
+    const lines = [];
+    lines.push(`# Skillsmith Namespace Audit — ${result.auditId}`);
+    lines.push('');
+    lines.push(`- Generated: ${generatedAt.toISOString()}`);
+    lines.push(`- Total entries scanned: ${result.summary.totalEntries}`);
+    lines.push(`- Total flags: ${result.summary.totalFlags}`);
+    lines.push(`  - Errors (exact collisions): ${result.summary.errorCount}`);
+    lines.push(`  - Warnings (generic + semantic): ${result.summary.warningCount}`);
+    lines.push(`- Audit duration: ${result.summary.durationMs.toFixed(2)}ms`);
+    lines.push('');
+    return lines.join('\n');
+}
+function renderClaudeMdCaveat() {
+    const lines = [];
+    lines.push('## CLAUDE.md scan caveat');
+    lines.push('');
+    lines.push('This audit included trigger phrases extracted from `CLAUDE.md` rules. ');
+    lines.push('Phrase extraction is heuristic — false-positives are possible when a ');
+    lines.push('rule is phrased imperatively ("when X, do Y") without intending to ');
+    lines.push('overlap with a skill trigger. Review collisions involving ');
+    lines.push('`claude_md_rule` entries before applying any rename.');
+    lines.push('');
+    return lines.join('\n');
+}
+function renderExactCollisions(flags) {
+    const lines = [];
+    lines.push('## Exact collisions');
+    lines.push('');
+    for (const flag of flags) {
+        lines.push(`### \`${flag.identifier}\` (${flag.entries.length} entries)`);
+        lines.push('');
+        lines.push(`- Severity: **${flag.severity}**`);
+        lines.push(`- Collision id: \`${flag.collisionId}\``);
+        lines.push(`- Reason: ${flag.reason}`);
+        lines.push('- Involved entries:');
+        for (const entry of flag.entries) {
+            lines.push(`  - [${entry.kind}] ${entry.source_path}`);
+        }
+        lines.push('');
+    }
+    return lines.join('\n');
+}
+function renderGenericFlags(flags) {
+    const lines = [];
+    lines.push('## Generic-token flags');
+    lines.push('');
+    for (const flag of flags) {
+        lines.push(`### \`${flag.identifier}\``);
+        lines.push('');
+        lines.push(`- Severity: **${flag.severity}**`);
+        lines.push(`- Collision id: \`${flag.collisionId}\``);
+        lines.push(`- Source: ${flag.entry.source_path}`);
+        lines.push(`- Matched generic tokens: ${formatTokens(flag.matchedTokens)}`);
+        lines.push(`- Reason: ${flag.reason}`);
+        lines.push('');
+    }
+    return lines.join('\n');
+}
+function renderSemanticCollisions(flags) {
+    const lines = [];
+    lines.push('## Semantic collisions');
+    lines.push('');
+    for (const flag of flags) {
+        lines.push(`### ${describeEntry(flag.entryA)} ↔ ${describeEntry(flag.entryB)}`);
+        lines.push('');
+        lines.push(`- Severity: **${flag.severity}**`);
+        lines.push(`- Collision id: \`${flag.collisionId}\``);
+        lines.push(`- Cosine score: ${flag.cosineScore.toFixed(3)}`);
+        lines.push(`- Reason: ${flag.reason}`);
+        lines.push(`- Entry A: ${flag.entryA.source_path}`);
+        lines.push(`- Entry B: ${flag.entryB.source_path}`);
+        if (flag.overlappingPhrases.length > 0) {
+            lines.push('- Overlapping phrases:');
+            for (const pair of flag.overlappingPhrases) {
+                lines.push(`  - "${pair.phrase1}" ↔ "${pair.phrase2}" (sim ${pair.similarity.toFixed(3)})`);
+            }
+        }
+        lines.push('');
+    }
+    return lines.join('\n');
+}
+function renderRecommendedEdits(suggestions, auditId) {
+    const lines = [];
+    lines.push('## Recommended edits');
+    lines.push('');
+    if (!suggestions || suggestions.length === 0) {
+        // Wave 1 placeholder preserved for backward compatibility — no
+        // suggestion data was passed in.
+        lines.push('_No automated edits suggested in Wave 1._');
+        lines.push('');
+        return lines.join('\n');
+    }
+    // Wave 2 PR #4 (Step 8): render a markdown table of rename suggestions
+    // with copy-paste-ready CLI invocations. Wave 4 wires the actual CLI;
+    // this writer just renders the suggestion text.
+    lines.push('| Current name | Suggested rename | Apply action | Apply command |');
+    lines.push('|---|---|---|---|');
+    for (const suggestion of suggestions) {
+        const cmd = `\`sklx audit collisions apply ${auditId} ${suggestion.collisionId}\``;
+        lines.push(`| \`${suggestion.currentName}\` | \`${suggestion.suggested}\` | ${suggestion.applyAction} | ${cmd} |`);
+    }
+    lines.push('');
+    for (const suggestion of suggestions) {
+        lines.push(`### \`${suggestion.currentName}\` → \`${suggestion.suggested}\``);
+        lines.push('');
+        lines.push(`- Collision id: \`${suggestion.collisionId}\``);
+        lines.push(`- Reason: ${suggestion.reason}`);
+        lines.push(`- Source: ${suggestion.entry.source_path}`);
+        lines.push('');
+    }
+    return lines.join('\n');
+}
+/**
+ * SMI-4589 Wave 3: render the prose-edit suggestions section. Each edit
+ * becomes a markdown block with file/lineRange metadata and a `diff`
+ * fenced code block showing the before/after pair with `-`/`+` line
+ * prefixes — renders with syntax highlighting in GitHub / VSCode.
+ *
+ * Plan §2 mandates the diff block format over separate before/after
+ * plain-text blocks because the unified-diff form gives free
+ * highlighting and a familiar review surface.
+ */
+function renderRecommendedEditsSection(edits) {
+    const lines = [];
+    lines.push('## Recommended Edits');
+    lines.push('');
+    for (const edit of edits) {
+        lines.push(`### Recommended edit: differentiate from \`${edit.otherEntry.identifier}\``);
+        lines.push('');
+        lines.push(`**File**: \`${edit.filePath}\``);
+        lines.push(`**Lines**: ${edit.lineRange.start}-${edit.lineRange.end}`);
+        lines.push(`**Pattern**: \`${edit.pattern}\` (${edit.applyMode})`);
+        lines.push('');
+        lines.push('```diff');
+        for (const beforeLine of edit.before.split('\n')) {
+            lines.push(`-${beforeLine}`);
+        }
+        for (const afterLine of edit.after.split('\n')) {
+            lines.push(`+${afterLine}`);
+        }
+        lines.push('```');
+        lines.push('');
+        lines.push(`**Why**: ${edit.rationale}`);
+        lines.push('');
+    }
+    return lines.join('\n');
+}
+// ---------------------------------------------------------------------------
+// Helpers
+// ---------------------------------------------------------------------------
+function containsClaudeMdRule(result) {
+    return result.inventory.some((e) => e.kind === 'claude_md_rule');
+}
+function describeEntry(entry) {
+    return `\`${entry.identifier}\` (${entry.kind})`;
+}
+function formatTokens(tokens) {
+    if (tokens.length === 0)
+        return '_none_';
+    return tokens.map((t) => `\`${t}\``).join(', ');
+}
+//# sourceMappingURL=audit-report-writer.js.map

package/dist/src/audit/audit-report-writer.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"audit-report-writer.js","sourceRoot":"","sources":["../../../src/audit/audit-report-writer.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;GAmBG;AAEH,OAAO,KAAK,EAAE,MAAM,kBAAkB,CAAA;AACtC,OAAO,KAAK,IAAI,MAAM,WAAW,CAAA;AA8DjC;;;;GAIG;AACH,MAAM,UAAU,iBAAiB,CAC/B,MAA4B,EAC5B,OAAiC,EAAE;IAEnC,MAAM,WAAW,GAAG,IAAI,CAAC,WAAW,IAAI,IAAI,IAAI,EAAE,CAAA;IAClD,MAAM,QAAQ,GAAa,EAAE,CAAA;IAC7B,QAAQ,CAAC,IAAI,CAAC,mBAAmB,CAAC,MAAM,EAAE,WAAW,CAAC,CAAC,CAAA;IAEvD,IAAI,oBAAoB,CAAC,MAAM,CAAC,EAAE,CAAC;QACjC,QAAQ,CAAC,IAAI,CAAC,oBAAoB,EAAE,CAAC,CAAA;IACvC,CAAC;IAED,IAAI,MAAM,CAAC,eAAe,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QACtC,QAAQ,CAAC,IAAI,CAAC,qBAAqB,CAAC,MAAM,CAAC,eAAe,CAAC,CAAC,CAAA;IAC9D,CAAC;IAED,IAAI,MAAM,CAAC,YAAY,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QACnC,QAAQ,CAAC,IAAI,CAAC,kBAAkB,CAAC,MAAM,CAAC,YAAY,CAAC,CAAC,CAAA;IACxD,CAAC;IAED,IAAI,MAAM,CAAC,kBAAkB,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QACzC,QAAQ,CAAC,IAAI,CAAC,wBAAwB,CAAC,MAAM,CAAC,kBAAkB,CAAC,CAAC,CAAA;IACpE,CAAC;IAED,QAAQ,CAAC,IAAI,CAAC,sBAAsB,CAAC,IAAI,CAAC,iBAAiB,EAAE,MAAM,CAAC,OAAO,CAAC,CAAC,CAAA;IAE7E,sEAAsE;IACtE,sEAAsE;IACtE,qEAAqE;IACrE,4CAA4C;IAC5C,IAAI,IAAI,CAAC,gBAAgB,IAAI,IAAI,CAAC,gBAAgB,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QAC9D,QAAQ,CAAC,IAAI,CAAC,6BAA6B,CAAC,IAAI,CAAC,gBAAgB,CAAC,CAAC,CAAA;IACrE,CAAC;IAED,uEAAuE;IACvE,kEAAkE;IAClE,iEAAiE;IACjE,uDAAuD;IACvD,OAAO,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,OAAO,EAAE,GAAG,IAAI,CAAA;AAC7C,CAAC;AAED;;;;GAIG;AACH,MAAM,CAAC,KAAK,UAAU,gBAAgB,CACpC,MAA4B,EAC5B,IAA6B;IAE7B,MAAM,UAAU,GAAG,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,QAAQ,EAAE,WAAW,CAAC,CAAA;IACxD,MAAM,OAAO,GAAG,GAAG,UAAU,MAAM,CAAA;IACnC,MAAM,EAAE,CAAC,KAAK,CAAC,IAAI,CAAC,QAAQ,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAA;IAClD,MAAM,IAAI,GAAG,iBAAiB,CAAC,MAAM,EAAE;QACrC,WAAW,EAAE,IAAI,CAAC,WAAW;QAC7B,iBAAiB,EAAE,IAAI,CAAC,iBAAiB;QACzC,gBAAgB,EAAE,IAAI,CAAC,gBAAgB;KACxC,CAAC,CAAA;IACF,MAAM,EAAE,CAAC,SAAS,CAAC,OAAO,EAAE,IAAI,EAAE,OAAO,CAAC,CAAA;IAC1C,MAAM,EAAE,CAAC,MAAM,CAAC,OAAO,EAAE,UAAU,CAAC,CAAA;IACpC,OAAO,EAAE,UAAU,EAAE,CAAA;AACvB,CAAC;AAED,8EAA8E;AAC9E,oBAAoB;AACpB,8EAA8E;AAE9E,SAAS,mBAAmB,CAAC,MAA4B,EAAE,WAAiB;IAC1E,MAAM,KAAK,GAAa,EAAE,CAAA;IAC1B,KAAK,CAAC,IAAI,CAAC,kCAAkC,MAAM,CAAC,OAAO,EAAE,CAAC,CAAA;IAC9D,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAA;IACd,KAAK,CAAC,IAAI,CAAC,gBAAgB,WAAW,CAAC,WAAW,EAAE,EAAE,CAAC,CAAA;IACvD,KAAK,CAAC,IAAI,CAAC,4BAA4B,MAAM,CAAC,OAAO,CAAC,YAAY,EAAE,CAAC,CAAA;IACrE,KAAK,CAAC,IAAI,CAAC,kBAAkB,MAAM,CAAC,OAAO,CAAC,UAAU,EAAE,CAAC,CAAA;IACzD,KAAK,CAAC,IAAI,CAAC,kCAAkC,MAAM,CAAC,OAAO,CAAC,UAAU,EAAE,CAAC,CAAA;IACzE,KAAK,CAAC,IAAI,CAAC,sCAAsC,MAAM,CAAC,OAAO,CAAC,YAAY,EAAE,CAAC,CAAA;IAC/E,KAAK,CAAC,IAAI,CAAC,qBAAqB,MAAM,CAAC,OAAO,CAAC,UAAU,CAAC,OAAO,CAAC,CAAC,CAAC,IAAI,CAAC,CAAA;IACzE,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAA;IACd,OAAO,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;AACzB,CAAC;AAED,SAAS,oBAAoB;IAC3B,MAAM,KAAK,GAAa,EAAE,CAAA;IAC1B,KAAK,CAAC,IAAI,CAAC,0BAA0B,CAAC,CAAA;IACtC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAA;IACd,KAAK,CAAC,IAAI,CAAC,wEAAwE,CAAC,CAAA;IACpF,KAAK,CAAC,IAAI,CAAC,uEAAuE,CAAC,CAAA;IACnF,KAAK,CAAC,IAAI,CAAC,qEAAqE,CAAC,CAAA;IACjF,KAAK,CAAC,IAAI,CAAC,4DAA4D,CAAC,CAAA;IACxE,KAAK,CAAC,IAAI,CAAC,sDAAsD,CAAC,CAAA;IAClE,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAA;IACd,OAAO,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;AACzB,CAAC;AAED,SAAS,qBAAqB,CAAC,KAAwC;IACrE,MAAM,KAAK,GAAa,EAAE,CAAA;IAC1B,KAAK,CAAC,IAAI,CAAC,qBAAqB,CAAC,CAAA;IACjC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAA;IACd,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,KAAK,CAAC,IAAI,CAAC,SAAS,IAAI,CAAC,UAAU,OAAO,IAAI,CAAC,OAAO,CAAC,MAAM,WAAW,CAAC,CAAA;QACzE,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAA;QACd,KAAK,CAAC,IAAI,CAAC,iBAAiB,IAAI,CAAC,QAAQ,IAAI,CAAC,CAAA;QAC9C,KAAK,CAAC,IAAI,CAAC,qBAAqB,IAAI,CAAC,WAAW,IAAI,CAAC,CAAA;QACrD,KAAK,CAAC,IAAI,CAAC,aAAa,IAAI,CAAC,MAAM,EAAE,CAAC,CAAA;QACtC,KAAK,CAAC,IAAI,CAAC,qBAAqB,CAAC,CAAA;QACjC,KAAK,MAAM,KAAK,IAAI,IAAI,CAAC,OAAO,EAAE,CAAC;YACjC,KAAK,CAAC,IAAI,CAAC,QAAQ,KAAK,CAAC,IAAI,KAAK,KAAK,CAAC,WAAW,EAAE,CAAC,CAAA;QACxD,CAAC;QACD,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAA;IAChB,CAAC;IACD,OAAO,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;AACzB,CAAC;AAED,SAAS,kBAAkB,CAAC,KAAsC;IAChE,MAAM,KAAK,GAAa,EAAE,CAAA;IAC1B,KAAK,CAAC,IAAI,CAAC,wBAAwB,CAAC,CAAA;IACpC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAA;IACd,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,KAAK,CAAC,IAAI,CAAC,SAAS,IAAI,CAAC,UAAU,IAAI,CAAC,CAAA;QACxC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAA;QACd,KAAK,CAAC,IAAI,CAAC,iBAAiB,IAAI,CAAC,QAAQ,IAAI,CAAC,CAAA;QAC9C,KAAK,CAAC,IAAI,CAAC,qBAAqB,IAAI,CAAC,WAAW,IAAI,CAAC,CAAA;QACrD,KAAK,CAAC,IAAI,CAAC,aAAa,IAAI,CAAC,KAAK,CAAC,WAAW,EAAE,CAAC,CAAA;QACjD,KAAK,CAAC,IAAI,CAAC,6BAA6B,YAAY,CAAC,IAAI,CAAC,aAAa,CAAC,EAAE,CAAC,CAAA;QAC3E,KAAK,CAAC,IAAI,CAAC,aAAa,IAAI,CAAC,MAAM,EAAE,CAAC,CAAA;QACtC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAA;IAChB,CAAC;IACD,OAAO,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;AACzB,CAAC;AAED,SAAS,wBAAwB,CAAC,KAA2C;IAC3E,MAAM,KAAK,GAAa,EAAE,CAAA;IAC1B,KAAK,CAAC,IAAI,CAAC,wBAAwB,CAAC,CAAA;IACpC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAA;IACd,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,KAAK,CAAC,IAAI,CAAC,OAAO,aAAa,CAAC,IAAI,CAAC,MAAM,CAAC,MAAM,aAAa,CAAC,IAAI,CAAC,MAAM,CAAC,EAAE,CAAC,CAAA;QAC/E,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAA;QACd,KAAK,CAAC,IAAI,CAAC,iBAAiB,IAAI,CAAC,QAAQ,IAAI,CAAC,CAAA;QAC9C,KAAK,CAAC,IAAI,CAAC,qBAAqB,IAAI,CAAC,WAAW,IAAI,CAAC,CAAA;QACrD,KAAK,CAAC,IAAI,CAAC,mBAAmB,IAAI,CAAC,WAAW,CAAC,OAAO,CAAC,CAAC,CAAC,EAAE,CAAC,CAAA;QAC5D,KAAK,CAAC,IAAI,CAAC,aAAa,IAAI,CAAC,MAAM,EAAE,CAAC,CAAA;QACtC,KAAK,CAAC,IAAI,CAAC,cAAc,IAAI,CAAC,MAAM,CAAC,WAAW,EAAE,CAAC,CAAA;QACnD,KAAK,CAAC,IAAI,CAAC,cAAc,IAAI,CAAC,MAAM,CAAC,WAAW,EAAE,CAAC,CAAA;QACnD,IAAI,IAAI,CAAC,kBAAkB,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YACvC,KAAK,CAAC,IAAI,CAAC,wBAAwB,CAAC,CAAA;YACpC,KAAK,MAAM,IAAI,IAAI,IAAI,CAAC,kBAAkB,EAAE,CAAC;gBAC3C,KAAK,CAAC,IAAI,CAAC,QAAQ,IAAI,CAAC,OAAO,QAAQ,IAAI,CAAC,OAAO,UAAU,IAAI,CAAC,UAAU,CAAC,OAAO,CAAC,CAAC,CAAC,GAAG,CAAC,CAAA;YAC7F,CAAC;QACH,CAAC;QACD,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAA;IAChB,CAAC;IACD,OAAO,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;AACzB,CAAC;AAED,SAAS,sBAAsB,CAC7B,WAAwD,EACxD,OAAgB;IAEhB,MAAM,KAAK,GAAa,EAAE,CAAA;IAC1B,KAAK,CAAC,IAAI,CAAC,sBAAsB,CAAC,CAAA;IAClC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAA;IAEd,IAAI,CAAC,WAAW,IAAI,WAAW,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QAC7C,+DAA+D;QAC/D,iCAAiC;QACjC,KAAK,CAAC,IAAI,CAAC,2CAA2C,CAAC,CAAA;QACvD,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAA;QACd,OAAO,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;IACzB,CAAC;IAED,uEAAuE;IACvE,sEAAsE;IACtE,gDAAgD;IAChD,KAAK,CAAC,IAAI,CAAC,oEAAoE,CAAC,CAAA;IAChF,KAAK,CAAC,IAAI,CAAC,mBAAmB,CAAC,CAAA;IAC/B,KAAK,MAAM,UAAU,IAAI,WAAW,EAAE,CAAC;QACrC,MAAM,GAAG,GAAG,iCAAiC,OAAO,IAAI,UAAU,CAAC,WAAW,IAAI,CAAA;QAClF,KAAK,CAAC,IAAI,CACR,OAAO,UAAU,CAAC,WAAW,UAAU,UAAU,CAAC,SAAS,QAAQ,UAAU,CAAC,WAAW,MAAM,GAAG,IAAI,CACvG,CAAA;IACH,CAAC;IACD,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAA;IACd,KAAK,MAAM,UAAU,IAAI,WAAW,EAAE,CAAC;QACrC,KAAK,CAAC,IAAI,CAAC,SAAS,UAAU,CAAC,WAAW,UAAU,UAAU,CAAC,SAAS,IAAI,CAAC,CAAA;QAC7E,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAA;QACd,KAAK,CAAC,IAAI,CAAC,qBAAqB,UAAU,CAAC,WAAW,IAAI,CAAC,CAAA;QAC3D,KAAK,CAAC,IAAI,CAAC,aAAa,UAAU,CAAC,MAAM,EAAE,CAAC,CAAA;QAC5C,KAAK,CAAC,IAAI,CAAC,aAAa,UAAU,CAAC,KAAK,CAAC,WAAW,EAAE,CAAC,CAAA;QACvD,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAA;IAChB,CAAC;IACD,OAAO,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;AACzB,CAAC;AAED;;;;;;;;;GASG;AACH,SAAS,6BAA6B,CAAC,KAAqC;IAC1E,MAAM,KAAK,GAAa,EAAE,CAAA;IAC1B,KAAK,CAAC,IAAI,CAAC,sBAAsB,CAAC,CAAA;IAClC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAA;IACd,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,KAAK,CAAC,IAAI,CAAC,8CAA8C,IAAI,CAAC,UAAU,CAAC,UAAU,IAAI,CAAC,CAAA;QACxF,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAA;QACd,KAAK,CAAC,IAAI,CAAC,eAAe,IAAI,CAAC,QAAQ,IAAI,CAAC,CAAA;QAC5C,KAAK,CAAC,IAAI,CAAC,cAAc,IAAI,CAAC,SAAS,CAAC,KAAK,IAAI,IAAI,CAAC,SAAS,CAAC,GAAG,EAAE,CAAC,CAAA;QACtE,KAAK,CAAC,IAAI,CAAC,kBAAkB,IAAI,CAAC,OAAO,OAAO,IAAI,CAAC,SAAS,GAAG,CAAC,CAAA;QAClE,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAA;QACd,KAAK,CAAC,IAAI,CAAC,SAAS,CAAC,CAAA;QACrB,KAAK,MAAM,UAAU,IAAI,IAAI,CAAC,MAAM,CAAC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC;YACjD,KAAK,CAAC,IAAI,CAAC,IAAI,UAAU,EAAE,CAAC,CAAA;QAC9B,CAAC;QACD,KAAK,MAAM,SAAS,IAAI,IAAI,CAAC,KAAK,CAAC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC;YAC/C,KAAK,CAAC,IAAI,CAAC,IAAI,SAAS,EAAE,CAAC,CAAA;QAC7B,CAAC;QACD,KAAK,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;QACjB,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAA;QACd,KAAK,CAAC,IAAI,CAAC,YAAY,IAAI,CAAC,SAAS,EAAE,CAAC,CAAA;QACxC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAA;IAChB,CAAC;IACD,OAAO,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;AACzB,CAAC;AAED,8EAA8E;AAC9E,UAAU;AACV,8EAA8E;AAE9E,SAAS,oBAAoB,CAAC,MAA4B;IACxD,OAAO,MAAM,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,KAAK,gBAAgB,CAAC,CAAA;AAClE,CAAC;AAED,SAAS,aAAa,CAAC,KAAqB;IAC1C,OAAO,KAAK,KAAK,CAAC,UAAU,OAAO,KAAK,CAAC,IAAI,GAAG,CAAA;AAClD,CAAC;AAED,SAAS,YAAY,CAAC,MAA6B;IACjD,IAAI,MAAM,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,QAAQ,CAAA;IACxC,OAAO,MAAM,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;AACjD,CAAC"}

package/dist/src/audit/audit-suggestions.d.ts

@@ -0,0 +1,52 @@
+/**
+ * @fileoverview Per-audit suggestion persistence (SMI-4590 Wave 4 PR 4).
+ * @module @skillsmith/mcp-server/audit/audit-suggestions
+ *
+ * Persists `RenameSuggestion[]` (Wave 2) + `RecommendedEdit[]` (Wave 3)
+ * alongside the existing `~/.skillsmith/audits/<auditId>/result.json`
+ * (Wave 1, see `audit-history.ts`). The `apply_namespace_rename` and
+ * `apply_recommended_edit` MCP tools (this PR) read this file to look up
+ * the suggestion that corresponds to a `(auditId, collisionId)` pair.
+ *
+ * File layout: `<auditDir>/suggestions.json` — atomic via tmp-file +
+ * `fs.rename`, mirroring `audit-history.ts`. Schema is versioned so a
+ * future PR can extend the persisted shape without breaking older
+ * audit dirs.
+ *
+ * Why a sibling file (vs. extending `result.json`):
+ *   - `result.json` is keyed by `InventoryAuditResult` shape (Wave 1
+ *     barrel surface). Extending it pulls Wave 2/3 types into Wave 1's
+ *     persistence layer.
+ *   - The CLI (PR 5) wants to render `result.json` as-is for
+ *     `--report-only` callers; suggestions live alongside but aren't
+ *     part of the inventory snapshot.
+ */
+import type { RecommendedEdit } from './edit-suggester.types.js';
+import type { RenameSuggestion } from './rename-engine.types.js';
+/** Persisted shape. `version: 1` is the only supported schema. */
+export interface AuditSuggestionsFile {
+    version: 1;
+    auditId: string;
+    renameSuggestions: RenameSuggestion[];
+    recommendedEdits: RecommendedEdit[];
+}
+export interface AuditSuggestionsOptions {
+    /** Override the audits root (default `~/.skillsmith/audits`). */
+    auditsDir?: string;
+}
+/**
+ * Persist the suggestion arrays for `auditId` to
+ * `<auditsDir>/<auditId>/suggestions.json`. Atomic. Creates the per-audit
+ * directory if missing (matches `writeAuditHistory` semantics).
+ */
+export declare function writeAuditSuggestions(auditId: string, renameSuggestions: ReadonlyArray<RenameSuggestion>, recommendedEdits: ReadonlyArray<RecommendedEdit>, opts?: AuditSuggestionsOptions): Promise<{
+    suggestionsPath: string;
+}>;
+/**
+ * Read back the persisted suggestions for `auditId`. Returns `null` for
+ * unknown / missing audit dirs and for malformed / wrong-version files.
+ * Callers treat the absence as "no suggestions"; the apply-tools surface
+ * a typed error when the lookup is required but returns null.
+ */
+export declare function readAuditSuggestions(auditId: string, opts?: AuditSuggestionsOptions): Promise<AuditSuggestionsFile | null>;
+//# sourceMappingURL=audit-suggestions.d.ts.map

package/dist/src/audit/audit-suggestions.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"audit-suggestions.d.ts","sourceRoot":"","sources":["../../../src/audit/audit-suggestions.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;GAsBG;AAMH,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,2BAA2B,CAAA;AAChE,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,0BAA0B,CAAA;AAIhE,kEAAkE;AAClE,MAAM,WAAW,oBAAoB;IACnC,OAAO,EAAE,CAAC,CAAA;IACV,OAAO,EAAE,MAAM,CAAA;IACf,iBAAiB,EAAE,gBAAgB,EAAE,CAAA;IACrC,gBAAgB,EAAE,eAAe,EAAE,CAAA;CACpC;AAED,MAAM,WAAW,uBAAuB;IACtC,iEAAiE;IACjE,SAAS,CAAC,EAAE,MAAM,CAAA;CACnB;AAED;;;;GAIG;AACH,wBAAsB,qBAAqB,CACzC,OAAO,EAAE,MAAM,EACf,iBAAiB,EAAE,aAAa,CAAC,gBAAgB,CAAC,EAClD,gBAAgB,EAAE,aAAa,CAAC,eAAe,CAAC,EAChD,IAAI,GAAE,uBAA4B,GACjC,OAAO,CAAC;IAAE,eAAe,EAAE,MAAM,CAAA;CAAE,CAAC,CAkBtC;AAED;;;;;GAKG;AACH,wBAAsB,oBAAoB,CACxC,OAAO,EAAE,MAAM,EACf,IAAI,GAAE,uBAA4B,GACjC,OAAO,CAAC,oBAAoB,GAAG,IAAI,CAAC,CAoBtC"}

package/dist/src/audit/audit-suggestions.js

@@ -0,0 +1,90 @@
+/**
+ * @fileoverview Per-audit suggestion persistence (SMI-4590 Wave 4 PR 4).
+ * @module @skillsmith/mcp-server/audit/audit-suggestions
+ *
+ * Persists `RenameSuggestion[]` (Wave 2) + `RecommendedEdit[]` (Wave 3)
+ * alongside the existing `~/.skillsmith/audits/<auditId>/result.json`
+ * (Wave 1, see `audit-history.ts`). The `apply_namespace_rename` and
+ * `apply_recommended_edit` MCP tools (this PR) read this file to look up
+ * the suggestion that corresponds to a `(auditId, collisionId)` pair.
+ *
+ * File layout: `<auditDir>/suggestions.json` — atomic via tmp-file +
+ * `fs.rename`, mirroring `audit-history.ts`. Schema is versioned so a
+ * future PR can extend the persisted shape without breaking older
+ * audit dirs.
+ *
+ * Why a sibling file (vs. extending `result.json`):
+ *   - `result.json` is keyed by `InventoryAuditResult` shape (Wave 1
+ *     barrel surface). Extending it pulls Wave 2/3 types into Wave 1's
+ *     persistence layer.
+ *   - The CLI (PR 5) wants to render `result.json` as-is for
+ *     `--report-only` callers; suggestions live alongside but aren't
+ *     part of the inventory snapshot.
+ */
+import * as fs from 'node:fs/promises';
+import * as os from 'node:os';
+import * as path from 'node:path';
+const DEFAULT_AUDITS_DIR = path.join(os.homedir(), '.skillsmith', 'audits');
+/**
+ * Persist the suggestion arrays for `auditId` to
+ * `<auditsDir>/<auditId>/suggestions.json`. Atomic. Creates the per-audit
+ * directory if missing (matches `writeAuditHistory` semantics).
+ */
+export async function writeAuditSuggestions(auditId, renameSuggestions, recommendedEdits, opts = {}) {
+    const auditsDir = opts.auditsDir ?? DEFAULT_AUDITS_DIR;
+    const auditDir = path.join(auditsDir, auditId);
+    const suggestionsPath = path.join(auditDir, 'suggestions.json');
+    const tmpPath = `${suggestionsPath}.tmp`;
+    await fs.mkdir(auditDir, { recursive: true });
+    const payload = {
+        version: 1,
+        auditId,
+        renameSuggestions: [...renameSuggestions],
+        recommendedEdits: [...recommendedEdits],
+    };
+    await fs.writeFile(tmpPath, JSON.stringify(payload, null, 2), 'utf-8');
+    await fs.rename(tmpPath, suggestionsPath);
+    return { suggestionsPath };
+}
+/**
+ * Read back the persisted suggestions for `auditId`. Returns `null` for
+ * unknown / missing audit dirs and for malformed / wrong-version files.
+ * Callers treat the absence as "no suggestions"; the apply-tools surface
+ * a typed error when the lookup is required but returns null.
+ */
+export async function readAuditSuggestions(auditId, opts = {}) {
+    const auditsDir = opts.auditsDir ?? DEFAULT_AUDITS_DIR;
+    const suggestionsPath = path.join(auditsDir, auditId, 'suggestions.json');
+    let raw;
+    try {
+        raw = await fs.readFile(suggestionsPath, 'utf-8');
+    }
+    catch {
+        return null;
+    }
+    let parsed;
+    try {
+        parsed = JSON.parse(raw);
+    }
+    catch {
+        return null;
+    }
+    if (!isAuditSuggestionsFile(parsed))
+        return null;
+    return parsed;
+}
+function isAuditSuggestionsFile(v) {
+    if (!v || typeof v !== 'object')
+        return false;
+    const obj = v;
+    if (obj.version !== 1)
+        return false;
+    if (typeof obj.auditId !== 'string' || obj.auditId.length === 0)
+        return false;
+    if (!Array.isArray(obj.renameSuggestions))
+        return false;
+    if (!Array.isArray(obj.recommendedEdits))
+        return false;
+    return true;
+}
+//# sourceMappingURL=audit-suggestions.js.map

package/dist/src/audit/audit-suggestions.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"audit-suggestions.js","sourceRoot":"","sources":["../../../src/audit/audit-suggestions.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;GAsBG;AAEH,OAAO,KAAK,EAAE,MAAM,kBAAkB,CAAA;AACtC,OAAO,KAAK,EAAE,MAAM,SAAS,CAAA;AAC7B,OAAO,KAAK,IAAI,MAAM,WAAW,CAAA;AAKjC,MAAM,kBAAkB,GAAG,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC,OAAO,EAAE,EAAE,aAAa,EAAE,QAAQ,CAAC,CAAA;AAe3E;;;;GAIG;AACH,MAAM,CAAC,KAAK,UAAU,qBAAqB,CACzC,OAAe,EACf,iBAAkD,EAClD,gBAAgD,EAChD,OAAgC,EAAE;IAElC,MAAM,SAAS,GAAG,IAAI,CAAC,SAAS,IAAI,kBAAkB,CAAA;IACtD,MAAM,QAAQ,GAAG,IAAI,CAAC,IAAI,CAAC,SAAS,EAAE,OAAO,CAAC,CAAA;IAC9C,MAAM,eAAe,GAAG,IAAI,CAAC,IAAI,CAAC,QAAQ,EAAE,kBAAkB,CAAC,CAAA;IAC/D,MAAM,OAAO,GAAG,GAAG,eAAe,MAAM,CAAA;IAExC,MAAM,EAAE,CAAC,KAAK,CAAC,QAAQ,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAA;IAE7C,MAAM,OAAO,GAAyB;QACpC,OAAO,EAAE,CAAC;QACV,OAAO;QACP,iBAAiB,EAAE,CAAC,GAAG,iBAAiB,CAAC;QACzC,gBAAgB,EAAE,CAAC,GAAG,gBAAgB,CAAC;KACxC,CAAA;IACD,MAAM,EAAE,CAAC,SAAS,CAAC,OAAO,EAAE,IAAI,CAAC,SAAS,CAAC,OAAO,EAAE,IAAI,EAAE,CAAC,CAAC,EAAE,OAAO,CAAC,CAAA;IACtE,MAAM,EAAE,CAAC,MAAM,CAAC,OAAO,EAAE,eAAe,CAAC,CAAA;IAEzC,OAAO,EAAE,eAAe,EAAE,CAAA;AAC5B,CAAC;AAED;;;;;GAKG;AACH,MAAM,CAAC,KAAK,UAAU,oBAAoB,CACxC,OAAe,EACf,OAAgC,EAAE;IAElC,MAAM,SAAS,GAAG,IAAI,CAAC,SAAS,IAAI,kBAAkB,CAAA;IACtD,MAAM,eAAe,GAAG,IAAI,CAAC,IAAI,CAAC,SAAS,EAAE,OAAO,EAAE,kBAAkB,CAAC,CAAA;IAEzE,IAAI,GAAW,CAAA;IACf,IAAI,CAAC;QACH,GAAG,GAAG,MAAM,EAAE,CAAC,QAAQ,CAAC,eAAe,EAAE,OAAO,CAAC,CAAA;IACnD,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,IAAI,CAAA;IACb,CAAC;IAED,IAAI,MAAe,CAAA;IACnB,IAAI,CAAC;QACH,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,CAAA;IAC1B,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,IAAI,CAAA;IACb,CAAC;IAED,IAAI,CAAC,sBAAsB,CAAC,MAAM,CAAC;QAAE,OAAO,IAAI,CAAA;IAChD,OAAO,MAAM,CAAA;AACf,CAAC;AAED,SAAS,sBAAsB,CAAC,CAAU;IACxC,IAAI,CAAC,CAAC,IAAI,OAAO,CAAC,KAAK,QAAQ;QAAE,OAAO,KAAK,CAAA;IAC7C,MAAM,GAAG,GAAG,CAA4B,CAAA;IACxC,IAAI,GAAG,CAAC,OAAO,KAAK,CAAC;QAAE,OAAO,KAAK,CAAA;IACnC,IAAI,OAAO,GAAG,CAAC,OAAO,KAAK,QAAQ,IAAI,GAAG,CAAC,OAAO,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,KAAK,CAAA;IAC7E,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,GAAG,CAAC,iBAAiB,CAAC;QAAE,OAAO,KAAK,CAAA;IACvD,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,GAAG,CAAC,gBAAgB,CAAC;QAAE,OAAO,KAAK,CAAA;IACtD,OAAO,IAAI,CAAA;AACb,CAAC"}

package/dist/src/audit/bootstrap-unmanaged.d.ts

@@ -0,0 +1,66 @@
+/**
+ * @fileoverview Bootstrap unmanaged skills for SMI-4587 Wave 1 Step 6a.
+ * @module @skillsmith/mcp-server/audit/bootstrap-unmanaged
+ *
+ * Iterates over inventory entries and registers any unmanaged SKILL.md
+ * via `index_local`. Failures are converted into typed `warnings[]`
+ * entries (`namespace.inventory.bootstrap_failed`) — never thrown — so
+ * the audit run can complete even when individual bootstrap calls fail
+ * (decision #12 in plan).
+ *
+ * Wave 1 PR #4 (Step 8 / NEW-E-2) wires the real `indexLocalSkill` core
+ * helper as the default `bootstrapFn` — replacing PR #3's no-op stub. The
+ * MCP-side `parseFrontmatter` is injected so behaviour matches the
+ * existing `LocalIndexer` path bit-for-bit.
+ *
+ * "Unmanaged" = `kind: 'skill'` AND `meta.author` is undefined (i.e.,
+ * the skill is not registered in `~/.skillsmith/manifest.json`).
+ */
+import type { InventoryEntry, ScanWarning } from '../utils/local-inventory.types.js';
+/**
+ * Per-entry bootstrap callback. Implementations register the skill with
+ * `index_local` (or the future `indexLocalSkill` core helper) and return
+ * void on success. Throwing converts to a typed warning, never a hard
+ * fail.
+ */
+export type BootstrapFn = (entry: InventoryEntry) => Promise<void>;
+export interface BootstrapUnmanagedOptions {
+    /**
+     * Bootstrap callback. Defaults to {@link defaultIndexLocalSkillBootstrap}
+     * (PR #4 / NEW-E-2 wiring) which delegates to the real `indexLocalSkill`
+     * core helper. Tests can substitute a vi.fn() to assert call counts and
+     * exercise the failure-warning path.
+     */
+    bootstrapFn?: BootstrapFn;
+    /** Optional logger sink for non-fatal diagnostics (debug-level). */
+    logger?: {
+        debug?: (msg: string, meta?: unknown) => void;
+    };
+}
+export interface BootstrapUnmanagedResult {
+    /** Number of entries that were considered unmanaged candidates. */
+    attempted: number;
+    /** Subset of `attempted` that completed without throwing. */
+    succeeded: number;
+    /** Warnings produced by failed bootstrap attempts (one per failure). */
+    warnings: ScanWarning[];
+}
+/**
+ * Identify unmanaged SKILL.md entries — `kind: 'skill'` with no
+ * `meta.author` (i.e., not registered in `~/.skillsmith/manifest.json`).
+ * Exported so tests + future callers can re-use the predicate.
+ */
+export declare function isUnmanagedSkill(entry: InventoryEntry): boolean;
+/**
+ * Run the bootstrap pass over an inventory snapshot.
+ *
+ * Contract:
+ *   - Never throws. Per-entry failures convert to a `ScanWarning` with
+ *     code `namespace.inventory.bootstrap_failed`.
+ *   - Always returns; callers can append `result.warnings` onto the
+ *     audit-level warnings array.
+ *   - Pure aside from the `bootstrapFn` side effect; safe to invoke in
+ *     dry-run / unit-test mode by supplying a no-op `bootstrapFn`.
+ */
+export declare function bootstrapUnmanagedSkills(inventory: ReadonlyArray<InventoryEntry>, opts?: BootstrapUnmanagedOptions): Promise<BootstrapUnmanagedResult>;
+//# sourceMappingURL=bootstrap-unmanaged.d.ts.map

package/dist/src/audit/bootstrap-unmanaged.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"bootstrap-unmanaged.d.ts","sourceRoot":"","sources":["../../../src/audit/bootstrap-unmanaged.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;GAiBG;AAIH,OAAO,KAAK,EAAE,cAAc,EAAE,WAAW,EAAE,MAAM,mCAAmC,CAAA;AAIpF;;;;;GAKG;AACH,MAAM,MAAM,WAAW,GAAG,CAAC,KAAK,EAAE,cAAc,KAAK,OAAO,CAAC,IAAI,CAAC,CAAA;AAElE,MAAM,WAAW,yBAAyB;IACxC;;;;;OAKG;IACH,WAAW,CAAC,EAAE,WAAW,CAAA;IACzB,oEAAoE;IACpE,MAAM,CAAC,EAAE;QAAE,KAAK,CAAC,EAAE,CAAC,GAAG,EAAE,MAAM,EAAE,IAAI,CAAC,EAAE,OAAO,KAAK,IAAI,CAAA;KAAE,CAAA;CAC3D;AAED,MAAM,WAAW,wBAAwB;IACvC,mEAAmE;IACnE,SAAS,EAAE,MAAM,CAAA;IACjB,6DAA6D;IAC7D,SAAS,EAAE,MAAM,CAAA;IACjB,wEAAwE;IACxE,QAAQ,EAAE,WAAW,EAAE,CAAA;CACxB;AAED;;;;GAIG;AACH,wBAAgB,gBAAgB,CAAC,KAAK,EAAE,cAAc,GAAG,OAAO,CAE/D;AAED;;;;;;;;;;GAUG;AACH,wBAAsB,wBAAwB,CAC5C,SAAS,EAAE,aAAa,CAAC,cAAc,CAAC,EACxC,IAAI,GAAE,yBAA8B,GACnC,OAAO,CAAC,wBAAwB,CAAC,CA+BnC"}

package/dist/src/audit/bootstrap-unmanaged.js

@@ -0,0 +1,91 @@
+/**
+ * @fileoverview Bootstrap unmanaged skills for SMI-4587 Wave 1 Step 6a.
+ * @module @skillsmith/mcp-server/audit/bootstrap-unmanaged
+ *
+ * Iterates over inventory entries and registers any unmanaged SKILL.md
+ * via `index_local`. Failures are converted into typed `warnings[]`
+ * entries (`namespace.inventory.bootstrap_failed`) — never thrown — so
+ * the audit run can complete even when individual bootstrap calls fail
+ * (decision #12 in plan).
+ *
+ * Wave 1 PR #4 (Step 8 / NEW-E-2) wires the real `indexLocalSkill` core
+ * helper as the default `bootstrapFn` — replacing PR #3's no-op stub. The
+ * MCP-side `parseFrontmatter` is injected so behaviour matches the
+ * existing `LocalIndexer` path bit-for-bit.
+ *
+ * "Unmanaged" = `kind: 'skill'` AND `meta.author` is undefined (i.e.,
+ * the skill is not registered in `~/.skillsmith/manifest.json`).
+ */
+import { indexLocalSkill } from '@skillsmith/core';
+import { WARNING_CODES } from '../utils/local-inventory.helpers.js';
+import { parseFrontmatter } from '../indexer/FrontmatterParser.js';
+/**
+ * Identify unmanaged SKILL.md entries — `kind: 'skill'` with no
+ * `meta.author` (i.e., not registered in `~/.skillsmith/manifest.json`).
+ * Exported so tests + future callers can re-use the predicate.
+ */
+export function isUnmanagedSkill(entry) {
+    return entry.kind === 'skill' && !entry.meta?.author;
+}
+/**
+ * Run the bootstrap pass over an inventory snapshot.
+ *
+ * Contract:
+ *   - Never throws. Per-entry failures convert to a `ScanWarning` with
+ *     code `namespace.inventory.bootstrap_failed`.
+ *   - Always returns; callers can append `result.warnings` onto the
+ *     audit-level warnings array.
+ *   - Pure aside from the `bootstrapFn` side effect; safe to invoke in
+ *     dry-run / unit-test mode by supplying a no-op `bootstrapFn`.
+ */
+export async function bootstrapUnmanagedSkills(inventory, opts = {}) {
+    const bootstrapFn = opts.bootstrapFn ?? defaultIndexLocalSkillBootstrap;
+    const warnings = [];
+    let attempted = 0;
+    let succeeded = 0;
+    for (const entry of inventory) {
+        if (!isUnmanagedSkill(entry))
+            continue;
+        attempted++;
+        try {
+            await bootstrapFn(entry);
+            succeeded++;
+        }
+        catch (err) {
+            const message = err instanceof Error ? err.message : String(err);
+            warnings.push({
+                code: WARNING_CODES.BOOTSTRAP_FAILED,
+                message: `bootstrap failed for ${entry.source_path}: ${message}`,
+                context: {
+                    source_path: entry.source_path,
+                    identifier: entry.identifier,
+                    error: message,
+                },
+            });
+            opts.logger?.debug?.('bootstrap_failed', {
+                source_path: entry.source_path,
+                error: message,
+            });
+        }
+    }
+    return { attempted, succeeded, warnings };
+}
+/**
+ * Default bootstrap implementation (SMI-4587 Wave 1 PR #4 / NEW-E-2).
+ *
+ * Delegates to `indexLocalSkill` from `@skillsmith/core/skills/index-local`
+ * with the MCP-side `parseFrontmatter` injected so we keep parity with the
+ * existing `LocalIndexer` path. Synchronous filesystem reads inside the
+ * core helper are wrapped in `Promise.resolve()` so the audit pipeline can
+ * await uniformly.
+ *
+ * Errors propagate upward — `bootstrapUnmanagedSkills` converts them into
+ * typed `ScanWarning` entries with code
+ * `namespace.inventory.bootstrap_failed`.
+ */
+async function defaultIndexLocalSkillBootstrap(entry) {
+    await Promise.resolve(indexLocalSkill(entry.source_path, {
+        parseFrontmatter,
+    }));
+}
+//# sourceMappingURL=bootstrap-unmanaged.js.map