@shrkcrft/inspector 0.1.0-alpha.30 → 0.1.0-alpha.32

package/dist/doctor-verdict.js

@@ -0,0 +1,28 @@
+import { coverageShortfall } from '@shrkcrft/core';
+import { DoctorVerdictKind } from "./doctor-verdict-kind.js";
+/**
+ * THE settlement of one doctor run.
+ *
+ *   an error-severity check          → `errors`
+ *   else any coverage shortfall      → `not-verified` (never ready)
+ *   else                             → `ready`
+ *
+ * `IDoctorResult.passed` stays errors-only (its long-standing meaning); the
+ * coverage the doctor could not verify is read here, through core's
+ * `coverageShortfall`, the same records `shrk doctor` settles its exit on.
+ * `shrk check`, the dashboard and MCP `inspect_sharkcraft_setup` read this
+ * instead of `passed`, so none of them calls a setup ready over a compiled
+ * pack build the doctor could not compare with its source.
+ */
+export function doctorVerdict(result) {
+    const shortfalls = (result.coverage ?? []).flatMap((c) => {
+        const s = coverageShortfall(c);
+        return s === undefined ? [] : [s];
+    });
+    const verdict = !result.passed
+        ? DoctorVerdictKind.Errors
+        : shortfalls.length > 0
+            ? DoctorVerdictKind.NotVerified
+            : DoctorVerdictKind.Ready;
+    return { verdict, ready: verdict === DoctorVerdictKind.Ready, shortfalls };
+}

package/dist/drift.d.ts

@@ -1,3 +1,4 @@
+import { type IVerdictCoverage } from '@shrkcrft/core';
 import type { ISharkcraftInspection } from './sharkcraft-inspector.js';
 export type DriftSeverity = 'error' | 'warning' | 'info';
 export type DriftCategory = 'boundary' | 'preset-reference' | 'preset-composition' | 'missing-pack-asset' | 'config-mismatch' | 'template-relationship' | 'pipeline-template-link';
@@ -19,6 +20,22 @@ export interface IDriftReport {
         warning: number;
         info: number;
     };
+    /**
+     * What the boundary half examined — THE records `check boundaries` settles
+     * on (`boundaryCheckCoverage`). Empty when boundaries were skipped or none
+     * is configured.
+     */
+    coverage: readonly IVerdictCoverage[];
+    /** 0 clean · 1 an error finding · 2 part of a boundary rule's scope was never examined. */
+    exitCode: number;
+    verdict: 'pass' | 'fail' | 'not-verified' | 'usage-error';
+    shortfalls: readonly string[];
+    /**
+     * The settled acceptances (`<rule>: accepted by expectEmpty: …`) — present
+     * only at exit 0, like every acceptance (round 13 review: a boundary rule's
+     * expectEmpty acceptance reached this settle and was never printed).
+     */
+    accepted?: readonly string[];
 }
 export interface IBuildDriftOptions {
     /** When true, run the full boundary scan (slow on large repos). Default: true. */

package/dist/drift.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"drift.d.ts","sourceRoot":"","sources":["../src/drift.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,qBAAqB,EAAE,MAAM,2BAA2B,CAAC;AAGvE,MAAM,MAAM,aAAa,GAAG,OAAO,GAAG,SAAS,GAAG,MAAM,CAAC;AACzD,MAAM,MAAM,aAAa,GACrB,UAAU,GACV,kBAAkB,GAClB,oBAAoB,GACpB,oBAAoB,GACpB,iBAAiB,GACjB,uBAAuB,GACvB,wBAAwB,CAAC;AAE7B,MAAM,WAAW,aAAa;IAC5B,QAAQ,EAAE,aAAa,CAAC;IACxB,QAAQ,EAAE,aAAa,CAAC;IACxB,OAAO,EAAE,MAAM,CAAC;IAChB,QAAQ,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC;IACnC,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,OAAO,CAAC,EAAE;QAAE,IAAI,EAAE,MAAM,GAAG,UAAU,GAAG,MAAM,GAAG,UAAU,GAAG,QAAQ,GAAG,MAAM,CAAC;QAAC,EAAE,EAAE,MAAM,CAAA;KAAE,EAAE,CAAC;CACjG;AAED,MAAM,WAAW,YAAY;IAC3B,QAAQ,EAAE,aAAa,EAAE,CAAC;IAC1B,MAAM,EAAE;QAAE,KAAK,EAAE,MAAM,CAAC;QAAC,OAAO,EAAE,MAAM,CAAC;QAAC,IAAI,EAAE,MAAM,CAAA;KAAE,CAAC;CAC1D;AAED,MAAM,WAAW,kBAAkB;IACjC,kFAAkF;IAClF,aAAa,CAAC,EAAE,OAAO,CAAC;CACzB;AAED;;;;;;;;;GASG;AACH,wBAAgB,gBAAgB,CAC9B,UAAU,EAAE,qBAAqB,EACjC,OAAO,GAAE,kBAAuB,GAC/B,YAAY,CAuGd"}
+{"version":3,"file":"drift.d.ts","sourceRoot":"","sources":["../src/drift.ts"],"names":[],"mappings":"AAAA,OAAO,EAAiB,KAAK,gBAAgB,EAAE,MAAM,gBAAgB,CAAC;AAGtE,OAAO,KAAK,EAAE,qBAAqB,EAAE,MAAM,2BAA2B,CAAC;AAGvE,MAAM,MAAM,aAAa,GAAG,OAAO,GAAG,SAAS,GAAG,MAAM,CAAC;AACzD,MAAM,MAAM,aAAa,GACrB,UAAU,GACV,kBAAkB,GAClB,oBAAoB,GACpB,oBAAoB,GACpB,iBAAiB,GACjB,uBAAuB,GACvB,wBAAwB,CAAC;AAE7B,MAAM,WAAW,aAAa;IAC5B,QAAQ,EAAE,aAAa,CAAC;IACxB,QAAQ,EAAE,aAAa,CAAC;IACxB,OAAO,EAAE,MAAM,CAAC;IAChB,QAAQ,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC;IACnC,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,OAAO,CAAC,EAAE;QAAE,IAAI,EAAE,MAAM,GAAG,UAAU,GAAG,MAAM,GAAG,UAAU,GAAG,QAAQ,GAAG,MAAM,CAAC;QAAC,EAAE,EAAE,MAAM,CAAA;KAAE,EAAE,CAAC;CACjG;AAED,MAAM,WAAW,YAAY;IAC3B,QAAQ,EAAE,aAAa,EAAE,CAAC;IAC1B,MAAM,EAAE;QAAE,KAAK,EAAE,MAAM,CAAC;QAAC,OAAO,EAAE,MAAM,CAAC;QAAC,IAAI,EAAE,MAAM,CAAA;KAAE,CAAC;IACzD;;;;OAIG;IACH,QAAQ,EAAE,SAAS,gBAAgB,EAAE,CAAC;IACtC,2FAA2F;IAC3F,QAAQ,EAAE,MAAM,CAAC;IACjB,OAAO,EAAE,MAAM,GAAG,MAAM,GAAG,cAAc,GAAG,aAAa,CAAC;IAC1D,UAAU,EAAE,SAAS,MAAM,EAAE,CAAC;IAC9B;;;;OAIG;IACH,QAAQ,CAAC,EAAE,SAAS,MAAM,EAAE,CAAC;CAC9B;AAED,MAAM,WAAW,kBAAkB;IACjC,kFAAkF;IAClF,aAAa,CAAC,EAAE,OAAO,CAAC;CACzB;AAED;;;;;;;;;GASG;AACH,wBAAgB,gBAAgB,CAC9B,UAAU,EAAE,qBAAqB,EACjC,OAAO,GAAE,kBAAuB,GAC/B,YAAY,CAoId"}

package/dist/drift.js

@@ -1,5 +1,6 @@
-import { evaluateBoundaries, loadTsconfigPaths, scanImports } from '@shrkcrft/boundaries';
+import { settleVerdict } from '@shrkcrft/core';
 import { resolvePreset, resolvePresetReferences } from '@shrkcrft/presets';
+import { boundaryCheckCoverage, boundaryLoadIssueLabel, runBoundaryCheck } from "./run-boundary-check.js";
 import { inspectionReferenceLookup } from "./reference-lookup.js";
 /**
  * First-cut drift report. Combines:
@@ -13,14 +14,32 @@ import { inspectionReferenceLookup } from "./reference-lookup.js";
  */
 export function buildDriftReport(inspection, options = {}) {
     const findings = [];
-    // 1. Boundary violations.
-    if ((options.runBoundaries ?? true) && inspection.boundaryRegistry.size() > 0) {
-        const scan = scanImports({ projectRoot: inspection.projectRoot });
-        const tsconfigPaths = loadTsconfigPaths(inspection.projectRoot);
-        const evalResult = evaluateBoundaries(scan, inspection.boundaryRegistry.list(), {
-            ...(tsconfigPaths.aliases.size > 0 ? { tsconfigPaths } : {}),
-        });
-        for (const v of evalResult.violations) {
+    // 1. Boundary violations — through THE boundary orchestrator, the call
+    // `check boundaries` makes (round 11 review R11-GAP-3). Drift ran its own
+    // scan + evaluate: an unreadable governed file read `0` here where the gate
+    // read 2, and an errored rule or a stale exception was invisible.
+    let coverage = [];
+    const loadIssues = inspection.boundaryLoadIssues ?? [];
+    if ((options.runBoundaries ?? true) && (inspection.boundaryRegistry.size() > 0 || loadIssues.length > 0)) {
+        const r = runBoundaryCheck(inspection);
+        coverage = boundaryCheckCoverage(r);
+        for (const issue of r.loadIssues) {
+            findings.push({
+                category: 'boundary',
+                severity: 'error',
+                message: `Boundary rule ${boundaryLoadIssueLabel(issue)} did not load — NOT evaluated: ${issue.issues.join('; ')}`,
+                evidence: { file: issue.file, kind: issue.kind, ...(issue.ruleId ? { ruleId: issue.ruleId } : {}) },
+            });
+        }
+        for (const s of r.staleExceptions) {
+            findings.push({
+                category: 'boundary',
+                severity: 'error',
+                message: s.message,
+                evidence: { ruleId: s.ruleId, file: s.file },
+            });
+        }
+        for (const v of r.violations) {
             findings.push({
                 category: 'boundary',
                 severity: v.severity,
@@ -103,5 +122,16 @@ export function buildDriftReport(inspection, options = {}) {
     const counts = { error: 0, warning: 0, info: 0 };
     for (const f of findings)
         counts[f.severity] += 1;
-    return { findings, counts };
+    // Settled with core's one fold: a boundary scope that was not fully examined
+    // turns a clean drift report into NOT VERIFIED (2), never "No drift".
+    const settled = settleVerdict(counts.error > 0 ? 1 : 0, coverage);
+    return {
+        findings,
+        counts,
+        coverage,
+        exitCode: settled.exit,
+        verdict: settled.verdict,
+        shortfalls: settled.shortfalls,
+        accepted: settled.accepted,
+    };
 }

package/dist/entrypoint-matrix.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"entrypoint-matrix.d.ts","sourceRoot":"","sources":["../src/entrypoint-matrix.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;GAeG;AAEH,eAAO,MAAM,wBAAwB,oCAAoC,CAAC;AAE1E,oBAAY,eAAe;IACzB,gBAAgB,sBAAsB;IACtC,QAAQ,cAAc;IACtB,WAAW,iBAAiB;IAC5B,mBAAmB,yBAAyB;CAC7C;AAED,MAAM,WAAW,sBAAsB;IACrC,QAAQ,CAAC,EAAE,EAAE,MAAM,CAAC;IACpB,QAAQ,CAAC,KAAK,EAAE,eAAe,CAAC;IAChC,QAAQ,CAAC,SAAS,EAAE,MAAM,CAAC;IAC3B,QAAQ,CAAC,SAAS,EAAE,MAAM,CAAC;IAC3B,QAAQ,CAAC,MAAM,EAAE,WAAW,GAAG,eAAe,GAAG,eAAe,CAAC;IACjE,QAAQ,CAAC,IAAI,CAAC,EAAE,MAAM,CAAC;IACvB,0EAA0E;IAC1E,QAAQ,CAAC,OAAO,CAAC,EAAE,aAAa,CAAC,MAAM,CAAC,CAAC;CAC1C;AAED,MAAM,WAAW,uBAAuB;IACtC,QAAQ,CAAC,MAAM,EAAE,OAAO,wBAAwB,CAAC;IACjD,QAAQ,CAAC,OAAO,EAAE,aAAa,CAAC,sBAAsB,CAAC,CAAC;IACxD;;;OAGG;IACH,QAAQ,CAAC,YAAY,EAAE,aAAa,CAAC;QACnC,QAAQ,CAAC,IAAI,EAAE,MAAM,CAAC;QACtB,QAAQ,CAAC,GAAG,EAAE,MAAM,CAAC;KACtB,CAAC,CAAC;CACJ;AAoJD,wBAAgB,qBAAqB,IAAI,uBAAuB,CAM/D;AAED,wBAAgB,0BAA0B,CAAC,MAAM,EAAE,uBAAuB,GAAG,MAAM,CA2BlF;AAED;;;;;;;;;;;GAWG;AACH,wBAAgB,gBAAgB,CAC9B,EAAE,EAAE,MAAM,GAAG,SAAS,GAAG,WAAW,GAAG,QAAQ,GAAG,KAAK,GAAG,WAAW,GACpE,MAAM,CAeR"}
+{"version":3,"file":"entrypoint-matrix.d.ts","sourceRoot":"","sources":["../src/entrypoint-matrix.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;GAeG;AAEH,eAAO,MAAM,wBAAwB,oCAAoC,CAAC;AAE1E,oBAAY,eAAe;IACzB,gBAAgB,sBAAsB;IACtC,QAAQ,cAAc;IACtB,WAAW,iBAAiB;IAC5B,mBAAmB,yBAAyB;CAC7C;AAED,MAAM,WAAW,sBAAsB;IACrC,QAAQ,CAAC,EAAE,EAAE,MAAM,CAAC;IACpB,QAAQ,CAAC,KAAK,EAAE,eAAe,CAAC;IAChC,QAAQ,CAAC,SAAS,EAAE,MAAM,CAAC;IAC3B,QAAQ,CAAC,SAAS,EAAE,MAAM,CAAC;IAC3B,QAAQ,CAAC,MAAM,EAAE,WAAW,GAAG,eAAe,GAAG,eAAe,CAAC;IACjE,QAAQ,CAAC,IAAI,CAAC,EAAE,MAAM,CAAC;IACvB,0EAA0E;IAC1E,QAAQ,CAAC,OAAO,CAAC,EAAE,aAAa,CAAC,MAAM,CAAC,CAAC;CAC1C;AAED,MAAM,WAAW,uBAAuB;IACtC,QAAQ,CAAC,MAAM,EAAE,OAAO,wBAAwB,CAAC;IACjD,QAAQ,CAAC,OAAO,EAAE,aAAa,CAAC,sBAAsB,CAAC,CAAC;IACxD;;;OAGG;IACH,QAAQ,CAAC,YAAY,EAAE,aAAa,CAAC;QACnC,QAAQ,CAAC,IAAI,EAAE,MAAM,CAAC;QACtB,QAAQ,CAAC,GAAG,EAAE,MAAM,CAAC;KACtB,CAAC,CAAC;CACJ;AAuJD,wBAAgB,qBAAqB,IAAI,uBAAuB,CAM/D;AAED,wBAAgB,0BAA0B,CAAC,MAAM,EAAE,uBAAuB,GAAG,MAAM,CA2BlF;AAED;;;;;;;;;;;GAWG;AACH,wBAAgB,gBAAgB,CAC9B,EAAE,EAAE,MAAM,GAAG,SAAS,GAAG,WAAW,GAAG,QAAQ,GAAG,KAAK,GAAG,WAAW,GACpE,MAAM,CAeR"}

package/dist/entrypoint-matrix.js

@@ -91,7 +91,10 @@ const MATRIX_ENTRIES = [
     {
         id: 'why',
         class: EntrypointClass.DebugExplainability,
-        callShape: 'shrk why <id> | shrk why-not <id>',
+        // The `shrk why <id>` / `shrk why-not <id>` ranker CLI was removed; the
+        // ranker explainers live behind these MCP tools (`shrk why` is now the
+        // per-FILE constraint explainer).
+        callShape: 'MCP get_ranker_explanation | get_ranker_why_not',
         whenToUse: 'Debug the ranker — why a registry entry was / was not surfaced for a task.',
         safety: 'read-only',
         docs: 'docs/why.md',

package/dist/feedback-ingestion.d.ts

@@ -1,3 +1,5 @@
+import { type IRejectedEntry } from '@shrkcrft/core';
+import type { IContributionFileIssue } from './i-contribution-file-issue.js';
 export declare const FEEDBACK_INGESTION_SCHEMA = "sharkcraft.feedback-ingestion/v1";
 export declare enum FeedbackBucket {
     Good = "good",
@@ -80,17 +82,41 @@ export declare function ingestFeedbackFile(projectRoot: string, file: string, op
  * `sharkcraft/feedback-rules.ts` exporting an `IFeedbackRule[]` default.
  * Packs contribute via `feedbackRuleFiles[]` in their manifest.
  */
-export declare function loadFeedbackRules(inspection: {
+export declare function loadFeedbackRules(inspection: IFeedbackRuleLoadScope): Promise<readonly IFeedbackRule[]>;
+/** What {@link loadFeedbackRules} reads off an inspection (a structural subset, so a caller need not build one). */
+type IFeedbackRuleLoadScope = {
     projectRoot: string;
     sharkcraftDir: string | null;
     packs?: {
         validPacks?: readonly {
             packageRoot: string;
+            packageName?: string;
             manifest?: {
                 contributions?: unknown;
             } | null;
         }[];
     };
-}): Promise<readonly IFeedbackRule[]>;
+};
+/**
+ * THE feedback-rule acceptance predicate (round 12, 12.1): a non-empty string
+ * `id` — `[]` means accepted. An id-less rule used to be dropped with no signal.
+ */
+export declare function feedbackRuleRejectionReasons(raw: unknown): readonly string[];
+/**
+ * {@link loadFeedbackRules} WITH what did not take effect (round 12, 12.1): a
+ * rules file that failed to import (it was swallowed to `[]`), and every
+ * declared rule the loader refused — invalid, or a duplicate id.
+ */
+export declare function loadFeedbackRulesWithIssues(inspection: IFeedbackRuleLoadScope): Promise<{
+    readonly rules: readonly IFeedbackRule[];
+    readonly entries: readonly {
+        readonly id: string;
+        readonly file: string;
+        readonly packageName?: string;
+    }[];
+    readonly issues: readonly IContributionFileIssue[];
+    readonly rejected: readonly IRejectedEntry[];
+}>;
 export declare function renderFeedbackBacklog(report: IFeedbackReport): string;
+export {};
 //# sourceMappingURL=feedback-ingestion.d.ts.map

package/dist/feedback-ingestion.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"feedback-ingestion.d.ts","sourceRoot":"","sources":["../src/feedback-ingestion.ts"],"names":[],"mappings":"AAkBA,eAAO,MAAM,yBAAyB,qCAAqC,CAAC;AAE5E,oBAAY,cAAc;IACxB,IAAI,SAAS;IACb,GAAG,QAAQ;IACX,OAAO,YAAY;IACnB,SAAS,eAAe;IACxB,KAAK,UAAU;CAChB;AAED,oBAAY,gBAAgB;IAC1B,IAAI,SAAS;IACb,KAAK,UAAU;IACf,KAAK,UAAU;IACf,OAAO,YAAY;CACpB;AAED,MAAM,WAAW,gBAAgB;IAC/B,MAAM,EAAE,cAAc,CAAC;IACvB,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,aAAa,CAAC,MAAM,CAAC,CAAC;IAC5B,uFAAuF;IACvF,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,QAAQ,EAAE,gBAAgB,CAAC;IAC3B,oCAAoC;IACpC,iBAAiB,EAAE,aAAa,CAAC,MAAM,CAAC,CAAC;CAC1C;AAED,MAAM,WAAW,eAAe;IAC9B,MAAM,EAAE,OAAO,yBAAyB,CAAC;IACzC,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,WAAW,EAAE,MAAM,CAAC;IACpB,aAAa,EAAE,MAAM,CAAC;IACtB,MAAM,EAAE,MAAM,CAAC,cAAc,EAAE,MAAM,CAAC,CAAC;IACvC,QAAQ,EAAE,aAAa,CAAC,gBAAgB,CAAC,CAAC;IAC1C,kBAAkB,EAAE,aAAa,CAAC,MAAM,CAAC,CAAC;CAC3C;AAUD;;;;;;;GAOG;AACH,eAAO,MAAM,oBAAoB,gCAAgC,CAAC;AAElE,MAAM,WAAW,aAAa;IAC5B,EAAE,EAAE,MAAM,CAAC;IACX,KAAK,EAAE,MAAM,CAAC;IACd,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,oEAAoE;IACpE,QAAQ,CAAC,EAAE,SAAS,MAAM,EAAE,CAAC;IAC7B,0BAA0B;IAC1B,OAAO,CAAC,EAAE,SAAS,MAAM,EAAE,CAAC;IAC5B,kDAAkD;IAClD,OAAO,CAAC,EAAE,SAAS,MAAM,EAAE,CAAC;IAC5B,iEAAiE;IACjE,QAAQ,CAAC,EAAE,MAAM,GAAG,KAAK,GAAG,SAAS,GAAG,YAAY,GAAG,OAAO,CAAC;IAC/D,0DAA0D;IAC1D,GAAG,CAAC,EAAE,MAAM,CAAC;IACb,IAAI,CAAC,EAAE,SAAS,MAAM,EAAE,CAAC;IACzB,sCAAsC;IACtC,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,QAAQ,CAAC,EAAE,gBAAgB,CAAC;IAC5B,4DAA4D;IAC5D,gBAAgB,CAAC,EAAE,SAAS,MAAM,EAAE,CAAC;IACrC,4CAA4C;IAC5C,eAAe,CAAC,EAAE,SAAS,MAAM,EAAE,CAAC;IACpC,gBAAgB,CAAC,EAAE,SAAS,MAAM,EAAE,CAAC;IACrC,gBAAgB,CAAC,EAAE,SAAS,MAAM,EAAE,CAAC;IACrC,gBAAgB,CAAC,EAAE,SAAS,MAAM,EAAE,CAAC;IACrC,uEAAuE;IACvE,WAAW,CAAC,EAAE,SAAS,MAAM,EAAE,CAAC;CACjC;AAED,wBAAgB,kBAAkB,CAAC,CAAC,SAAS,aAAa,EAAE,CAAC,EAAE,CAAC,GAAG,CAAC,CAEnE;AAqMD,MAAM,WAAW,sBAAsB;IACrC,yEAAyE;IACzE,KAAK,CAAC,EAAE,SAAS,aAAa,EAAE,CAAC;CAClC;AAED,wBAAgB,kBAAkB,CAChC,IAAI,EAAE,MAAM,EACZ,UAAU,CAAC,EAAE,MAAM,EACnB,OAAO,GAAE,sBAA2B,GACnC,eAAe,CAiDjB;AAED,wBAAgB,kBAAkB,CAChC,WAAW,EAAE,MAAM,EACnB,IAAI,EAAE,MAAM,EACZ,OAAO,GAAE,sBAA2B,GACnC,eAAe,CAIjB;AAED;;;;GAIG;AACH,wBAAsB,iBAAiB,CAAC,UAAU,EAAE;IAClD,WAAW,EAAE,MAAM,CAAC;IACpB,aAAa,EAAE,MAAM,GAAG,IAAI,CAAC;IAC7B,KAAK,CAAC,EAAE;QAAE,UAAU,CAAC,EAAE,SAAS;YAAE,WAAW,EAAE,MAAM,CAAC;YAAC,QAAQ,CAAC,EAAE;gBAAE,aAAa,CAAC,EAAE,OAAO,CAAA;aAAE,GAAG,IAAI,CAAA;SAAE,EAAE,CAAA;KAAE,CAAC;CAC5G,GAAG,OAAO,CAAC,SAAS,aAAa,EAAE,CAAC,CAuBpC;AAYD,wBAAgB,qBAAqB,CAAC,MAAM,EAAE,eAAe,GAAG,MAAM,CAmCrE"}
+{"version":3,"file":"feedback-ingestion.d.ts","sourceRoot":"","sources":["../src/feedback-ingestion.ts"],"names":[],"mappings":"AAgBA,OAAO,EAAyC,KAAK,cAAc,EAAE,MAAM,gBAAgB,CAAC;AAC5F,OAAO,KAAK,EAAE,sBAAsB,EAAE,MAAM,gCAAgC,CAAC;AAI7E,eAAO,MAAM,yBAAyB,qCAAqC,CAAC;AAE5E,oBAAY,cAAc;IACxB,IAAI,SAAS;IACb,GAAG,QAAQ;IACX,OAAO,YAAY;IACnB,SAAS,eAAe;IACxB,KAAK,UAAU;CAChB;AAED,oBAAY,gBAAgB;IAC1B,IAAI,SAAS;IACb,KAAK,UAAU;IACf,KAAK,UAAU;IACf,OAAO,YAAY;CACpB;AAED,MAAM,WAAW,gBAAgB;IAC/B,MAAM,EAAE,cAAc,CAAC;IACvB,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,aAAa,CAAC,MAAM,CAAC,CAAC;IAC5B,uFAAuF;IACvF,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,QAAQ,EAAE,gBAAgB,CAAC;IAC3B,oCAAoC;IACpC,iBAAiB,EAAE,aAAa,CAAC,MAAM,CAAC,CAAC;CAC1C;AAED,MAAM,WAAW,eAAe;IAC9B,MAAM,EAAE,OAAO,yBAAyB,CAAC;IACzC,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,WAAW,EAAE,MAAM,CAAC;IACpB,aAAa,EAAE,MAAM,CAAC;IACtB,MAAM,EAAE,MAAM,CAAC,cAAc,EAAE,MAAM,CAAC,CAAC;IACvC,QAAQ,EAAE,aAAa,CAAC,gBAAgB,CAAC,CAAC;IAC1C,kBAAkB,EAAE,aAAa,CAAC,MAAM,CAAC,CAAC;CAC3C;AAUD;;;;;;;GAOG;AACH,eAAO,MAAM,oBAAoB,gCAAgC,CAAC;AAElE,MAAM,WAAW,aAAa;IAC5B,EAAE,EAAE,MAAM,CAAC;IACX,KAAK,EAAE,MAAM,CAAC;IACd,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,oEAAoE;IACpE,QAAQ,CAAC,EAAE,SAAS,MAAM,EAAE,CAAC;IAC7B,0BAA0B;IAC1B,OAAO,CAAC,EAAE,SAAS,MAAM,EAAE,CAAC;IAC5B,kDAAkD;IAClD,OAAO,CAAC,EAAE,SAAS,MAAM,EAAE,CAAC;IAC5B,iEAAiE;IACjE,QAAQ,CAAC,EAAE,MAAM,GAAG,KAAK,GAAG,SAAS,GAAG,YAAY,GAAG,OAAO,CAAC;IAC/D,0DAA0D;IAC1D,GAAG,CAAC,EAAE,MAAM,CAAC;IACb,IAAI,CAAC,EAAE,SAAS,MAAM,EAAE,CAAC;IACzB,sCAAsC;IACtC,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,QAAQ,CAAC,EAAE,gBAAgB,CAAC;IAC5B,4DAA4D;IAC5D,gBAAgB,CAAC,EAAE,SAAS,MAAM,EAAE,CAAC;IACrC,4CAA4C;IAC5C,eAAe,CAAC,EAAE,SAAS,MAAM,EAAE,CAAC;IACpC,gBAAgB,CAAC,EAAE,SAAS,MAAM,EAAE,CAAC;IACrC,gBAAgB,CAAC,EAAE,SAAS,MAAM,EAAE,CAAC;IACrC,gBAAgB,CAAC,EAAE,SAAS,MAAM,EAAE,CAAC;IACrC,uEAAuE;IACvE,WAAW,CAAC,EAAE,SAAS,MAAM,EAAE,CAAC;CACjC;AAED,wBAAgB,kBAAkB,CAAC,CAAC,SAAS,aAAa,EAAE,CAAC,EAAE,CAAC,GAAG,CAAC,CAEnE;AAyMD,MAAM,WAAW,sBAAsB;IACrC,yEAAyE;IACzE,KAAK,CAAC,EAAE,SAAS,aAAa,EAAE,CAAC;CAClC;AAED,wBAAgB,kBAAkB,CAChC,IAAI,EAAE,MAAM,EACZ,UAAU,CAAC,EAAE,MAAM,EACnB,OAAO,GAAE,sBAA2B,GACnC,eAAe,CAiDjB;AAED,wBAAgB,kBAAkB,CAChC,WAAW,EAAE,MAAM,EACnB,IAAI,EAAE,MAAM,EACZ,OAAO,GAAE,sBAA2B,GACnC,eAAe,CAIjB;AAED;;;;GAIG;AACH,wBAAsB,iBAAiB,CAAC,UAAU,EAAE,sBAAsB,GAAG,OAAO,CAAC,SAAS,aAAa,EAAE,CAAC,CAE7G;AAED,oHAAoH;AACpH,KAAK,sBAAsB,GAAG;IAC5B,WAAW,EAAE,MAAM,CAAC;IACpB,aAAa,EAAE,MAAM,GAAG,IAAI,CAAC;IAC7B,KAAK,CAAC,EAAE;QACN,UAAU,CAAC,EAAE,SAAS;YACpB,WAAW,EAAE,MAAM,CAAC;YACpB,WAAW,CAAC,EAAE,MAAM,CAAC;YACrB,QAAQ,CAAC,EAAE;gBAAE,aAAa,CAAC,EAAE,OAAO,CAAA;aAAE,GAAG,IAAI,CAAC;SAC/C,EAAE,CAAC;KACL,CAAC;CACH,CAAC;AAEF;;;GAGG;AACH,wBAAgB,4BAA4B,CAAC,GAAG,EAAE,OAAO,GAAG,SAAS,MAAM,EAAE,CAI5E;AAED;;;;GAIG;AACH,wBAAsB,2BAA2B,CAAC,UAAU,EAAE,sBAAsB,GAAG,OAAO,CAAC;IAC7F,QAAQ,CAAC,KAAK,EAAE,SAAS,aAAa,EAAE,CAAC;IACzC,QAAQ,CAAC,OAAO,EAAE,SAAS;QAAE,QAAQ,CAAC,EAAE,EAAE,MAAM,CAAC;QAAC,QAAQ,CAAC,IAAI,EAAE,MAAM,CAAC;QAAC,QAAQ,CAAC,WAAW,CAAC,EAAE,MAAM,CAAA;KAAE,EAAE,CAAC;IAC3G,QAAQ,CAAC,MAAM,EAAE,SAAS,sBAAsB,EAAE,CAAC;IACnD,QAAQ,CAAC,QAAQ,EAAE,SAAS,cAAc,EAAE,CAAC;CAC9C,CAAC,CAwDD;AAaD,wBAAgB,qBAAqB,CAAC,MAAM,EAAE,eAAe,GAAG,MAAM,CAmCrE"}

package/dist/feedback-ingestion.js

@@ -14,7 +14,9 @@
  */
 import { existsSync, readFileSync } from 'node:fs';
 import * as nodePath from 'node:path';
-import { importModuleViaLoader } from '@shrkcrft/core';
+import { importModuleViaLoader, RejectionCause } from '@shrkcrft/core';
+import { knowledgeRenameCommand } from "./knowledge-rename-command.js";
+import { KnowledgeRenameVerb } from "./knowledge-rename-verb.js";
 export const FEEDBACK_INGESTION_SCHEMA = 'sharkcraft.feedback-ingestion/v1';
 export var FeedbackBucket;
 (function (FeedbackBucket) {
@@ -129,7 +131,11 @@ const KEYWORD_RULES = [
         pattern: /\b(rename|move|moved|renamed)\b/i,
         tags: ['rename'],
         targetArea: 'knowledge-rename',
-        suggestedCommands: ['shrk knowledge rename-symbol <old> <new> --dry-run'],
+        // THE rename-command helper: the previews are read-only and take no `--dry-run`.
+        suggestedCommands: [
+            knowledgeRenameCommand(KnowledgeRenameVerb.RenameFile),
+            knowledgeRenameCommand(KnowledgeRenameVerb.RenameSymbol),
+        ],
     },
     {
         pattern: /\b(trace|impact)\b/i,
@@ -141,7 +147,7 @@ const KEYWORD_RULES = [
         pattern: /\b(mcp|tool[s]?)\b/i,
         tags: ['mcp'],
         targetArea: 'mcp',
-        suggestedCommands: ['shrk commands list --filter mcp'],
+        suggestedCommands: ['shrk commands search mcp'],
     },
     {
         pattern: /\b(helper[s]?)\b/i,
@@ -305,41 +311,90 @@ export function ingestFeedbackFile(projectRoot, file, options = {}) {
  * Packs contribute via `feedbackRuleFiles[]` in their manifest.
  */
 export async function loadFeedbackRules(inspection) {
+    return (await loadFeedbackRulesWithIssues(inspection)).rules;
+}
+/**
+ * THE feedback-rule acceptance predicate (round 12, 12.1): a non-empty string
+ * `id` — `[]` means accepted. An id-less rule used to be dropped with no signal.
+ */
+export function feedbackRuleRejectionReasons(raw) {
+    if (!raw || typeof raw !== 'object' || Array.isArray(raw))
+        return ['(entry): must be an object'];
+    const id = raw.id;
+    return typeof id === 'string' && id.length > 0 ? [] : ['id: must be a non-empty string'];
+}
+/**
+ * {@link loadFeedbackRules} WITH what did not take effect (round 12, 12.1): a
+ * rules file that failed to import (it was swallowed to `[]`), and every
+ * declared rule the loader refused — invalid, or a duplicate id.
+ */
+export async function loadFeedbackRulesWithIssues(inspection) {
     const out = [];
-    const seen = new Set();
-    const add = (r) => {
-        if (!r?.id || seen.has(r.id))
+    const entries = [];
+    const issues = [];
+    const rejected = [];
+    const seen = new Map();
+    const readInto = async (file, packageName) => {
+        const r = await importDefaultArray(file);
+        if (r.error !== undefined) {
+            const rel = nodePath.relative(inspection.projectRoot, file) || file;
+            issues.push({
+                severity: 'warning',
+                code: 'load-failed',
+                message: `${packageName ? `Pack ${packageName} (${rel})` : `Failed to load ${rel}`}: ${r.error}`,
+                source: file,
+                ...(packageName ? { packageName } : {}),
+            });
             return;
-        seen.add(r.id);
-        out.push(r);
+        }
+        r.items.forEach((item, index) => {
+            const reasons = feedbackRuleRejectionReasons(item);
+            if (reasons.length > 0) {
+                rejected.push({ file, index, exportName: 'default', reasons, cause: RejectionCause.Invalid });
+                return;
+            }
+            const rule = item;
+            const prev = seen.get(rule.id);
+            if (prev !== undefined) {
+                rejected.push({
+                    file,
+                    index,
+                    exportName: 'default',
+                    entryId: rule.id,
+                    reasons: [`id: "${rule.id}" is already declared in ${prev}`],
+                    cause: RejectionCause.DuplicateId,
+                });
+                return;
+            }
+            seen.set(rule.id, nodePath.relative(inspection.projectRoot, file) || file);
+            out.push(rule);
+            entries.push({ id: rule.id, file, ...(packageName ? { packageName } : {}) });
+        });
     };
     // Local file.
     if (inspection.sharkcraftDir) {
-        const local = nodePath.join(inspection.sharkcraftDir, 'feedback-rules.ts');
-        for (const r of await importDefaultArray(local))
-            add(r);
+        await readInto(nodePath.join(inspection.sharkcraftDir, 'feedback-rules.ts'));
     }
     // Pack contributions.
     const packs = inspection.packs?.validPacks ?? [];
     for (const pack of packs) {
         const c = (pack.manifest?.contributions ?? {});
         for (const rel of c.feedbackRuleFiles ?? []) {
-            const full = nodePath.resolve(pack.packageRoot, rel);
-            for (const r of await importDefaultArray(full))
-                add(r);
+            await readInto(nodePath.resolve(pack.packageRoot, rel), pack.packageName);
         }
     }
-    return out;
+    return { rules: out, entries, issues, rejected };
 }
+/** A rules file's default array; `error` (first line) when the import threw. A missing file is `[]`. */
 async function importDefaultArray(absPath) {
+    if (!existsSync(absPath))
+        return { items: [] };
     try {
-        if (!existsSync(absPath))
-            return [];
         const mod = (await importModuleViaLoader(absPath));
-        return Array.isArray(mod.default) ? mod.default : [];
+        return { items: Array.isArray(mod.default) ? mod.default : [] };
     }
-    catch {
-        return [];
+    catch (e) {
+        return { items: [], error: (e.message ?? String(e)).split('\n')[0].trim() };
     }
 }
 export function renderFeedbackBacklog(report) {

package/dist/file-languages.d.ts

@@ -0,0 +1,9 @@
+import type { IFileLanguage } from './i-file-language.js';
+export declare const FILE_LANGUAGES: readonly IFileLanguage[];
+/** The language of `path` (absolute or project-relative): a basename override, else its extension; `null` when unknown. */
+export declare function fileLanguageOf(path: string): IFileLanguage | null;
+/** The language whose table row lists `extension` (`.ts` or `ts`), or `null`. */
+export declare function fileLanguageForExtension(extension: string): IFileLanguage | null;
+/** Every language id of the table, in table order — THE vocabulary `appliesTo.languages` may name. */
+export declare function fileLanguageIds(): readonly string[];
+//# sourceMappingURL=file-languages.d.ts.map

package/dist/file-languages.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"file-languages.d.ts","sourceRoot":"","sources":["../src/file-languages.ts"],"names":[],"mappings":"AAgBA,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,sBAAsB,CAAC;AAE1D,eAAO,MAAM,cAAc,EAAE,SAAS,aAAa,EAsCvB,CAAC;AAiB7B,2HAA2H;AAC3H,wBAAgB,cAAc,CAAC,IAAI,EAAE,MAAM,GAAG,aAAa,GAAG,IAAI,CASjE;AAED,iFAAiF;AACjF,wBAAgB,wBAAwB,CAAC,SAAS,EAAE,MAAM,GAAG,aAAa,GAAG,IAAI,CAGhF;AAED,sGAAsG;AACtG,wBAAgB,eAAe,IAAI,SAAS,MAAM,EAAE,CAEnD"}

package/dist/file-languages.js

@@ -0,0 +1,91 @@
+/**
+ * THE per-file extension → language authority (round 15, 15.1).
+ *
+ * Promoted from `shrk stats` (`repository-stats.ts`, where it was
+ * module-private), because a convention's `appliesTo.languages` filter needs a
+ * per-file language and the tree had four unrelated maps with four different
+ * vocabularies. This is the widest one and the vocabulary `shrk stats` prints
+ * (`typescript`, `javascript`, `python`, …), so an author can read the ids
+ * they may write off `shrk stats`. `shrk stats`, `conventionApplicability` and
+ * the self-config doctor's language-id check read this table and nothing else.
+ *
+ * (The polyglot `LanguageId` enum, `languages/`, is a different, workspace-level
+ * question — which toolchains a repo runs — and keeps its own seven ids.)
+ */
+import * as nodePath from 'node:path';
+import { CommentSyntax } from "./comment-syntax.js";
+export const FILE_LANGUAGES = Object.freeze([
+    { id: 'typescript', extensions: ['.ts', '.tsx', '.mts', '.cts'], comment: CommentSyntax.CFamily },
+    { id: 'javascript', extensions: ['.js', '.jsx', '.mjs', '.cjs'], comment: CommentSyntax.CFamily },
+    { id: 'java', extensions: ['.java'], comment: CommentSyntax.CFamily },
+    { id: 'kotlin', extensions: ['.kt', '.kts'], comment: CommentSyntax.CFamily },
+    { id: 'scala', extensions: ['.scala'], comment: CommentSyntax.CFamily },
+    { id: 'groovy', extensions: ['.groovy'], comment: CommentSyntax.CFamily },
+    { id: 'csharp', extensions: ['.cs'], comment: CommentSyntax.CFamily },
+    { id: 'cpp', extensions: ['.cpp', '.cc', '.cxx', '.hpp', '.hh', '.hxx'], comment: CommentSyntax.CFamily },
+    { id: 'c', extensions: ['.c', '.h'], comment: CommentSyntax.CFamily },
+    { id: 'go', extensions: ['.go'], comment: CommentSyntax.CFamily },
+    { id: 'rust', extensions: ['.rs'], comment: CommentSyntax.CFamily },
+    { id: 'swift', extensions: ['.swift'], comment: CommentSyntax.CFamily },
+    { id: 'php', extensions: ['.php'], comment: CommentSyntax.CFamily },
+    { id: 'dart', extensions: ['.dart'], comment: CommentSyntax.CFamily },
+    { id: 'python', extensions: ['.py', '.pyi'], comment: CommentSyntax.Hash },
+    { id: 'ruby', extensions: ['.rb'], comment: CommentSyntax.Hash },
+    { id: 'shell', extensions: ['.sh', '.bash', '.zsh', '.fish'], comment: CommentSyntax.Hash },
+    { id: 'perl', extensions: ['.pl', '.pm'], comment: CommentSyntax.Hash },
+    { id: 'r', extensions: ['.r', '.R'], comment: CommentSyntax.Hash },
+    { id: 'yaml', extensions: ['.yaml', '.yml'], comment: CommentSyntax.Hash },
+    { id: 'toml', extensions: ['.toml'], comment: CommentSyntax.Hash },
+    { id: 'ini', extensions: ['.ini', '.cfg', '.conf'], comment: CommentSyntax.Hash },
+    { id: 'dockerfile', extensions: ['.dockerfile'], comment: CommentSyntax.Hash },
+    { id: 'makefile', extensions: ['.mk'], comment: CommentSyntax.Hash },
+    { id: 'html', extensions: ['.html', '.htm'], comment: CommentSyntax.Html },
+    { id: 'xml', extensions: ['.xml', '.xsd', '.xsl'], comment: CommentSyntax.Html },
+    { id: 'vue', extensions: ['.vue'], comment: CommentSyntax.Html },
+    { id: 'svelte', extensions: ['.svelte'], comment: CommentSyntax.Html },
+    { id: 'sql', extensions: ['.sql'], comment: CommentSyntax.Sql },
+    { id: 'css', extensions: ['.css', '.scss', '.sass', '.less'], comment: CommentSyntax.CFamily },
+    { id: 'lua', extensions: ['.lua'], comment: CommentSyntax.Lua },
+    { id: 'elixir', extensions: ['.ex', '.exs'], comment: CommentSyntax.Hash },
+    { id: 'clojure', extensions: ['.clj', '.cljs'], comment: CommentSyntax.Lisp },
+    { id: 'lisp', extensions: ['.lisp', '.lsp', '.el'], comment: CommentSyntax.Lisp },
+    { id: 'json', extensions: ['.json', '.jsonc'], comment: CommentSyntax.None },
+    { id: 'markdown', extensions: ['.md', '.mdx'], comment: CommentSyntax.None },
+    { id: 'text', extensions: ['.txt'], comment: CommentSyntax.None },
+]);
+const EXTENSION_INDEX = (() => {
+    const m = new Map();
+    for (const def of FILE_LANGUAGES) {
+        for (const ext of def.extensions)
+            m.set(ext.toLowerCase(), def);
+    }
+    return m;
+})();
+/** Extension-less files named for their language (case-insensitive basename). */
+const BASENAME_OVERRIDES = new Map([
+    ['dockerfile', 'dockerfile'],
+    ['makefile', 'makefile'],
+    ['gnumakefile', 'makefile'],
+]);
+/** The language of `path` (absolute or project-relative): a basename override, else its extension; `null` when unknown. */
+export function fileLanguageOf(path) {
+    const override = BASENAME_OVERRIDES.get(nodePath.basename(path).toLowerCase());
+    if (override) {
+        const def = FILE_LANGUAGES.find((d) => d.id === override);
+        if (def)
+            return def;
+    }
+    const ext = nodePath.extname(path).toLowerCase();
+    if (!ext)
+        return null;
+    return EXTENSION_INDEX.get(ext) ?? null;
+}
+/** The language whose table row lists `extension` (`.ts` or `ts`), or `null`. */
+export function fileLanguageForExtension(extension) {
+    const ext = (extension.startsWith('.') ? extension : `.${extension}`).toLowerCase();
+    return EXTENSION_INDEX.get(ext) ?? null;
+}
+/** Every language id of the table, in table order — THE vocabulary `appliesTo.languages` may name. */
+export function fileLanguageIds() {
+    return FILE_LANGUAGES.map((d) => d.id);
+}

package/dist/fix-preview.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"fix-preview.d.ts","sourceRoot":"","sources":["../src/fix-preview.ts"],"names":[],"mappings":"AAyBA,OAAO,KAAK,EAAE,qBAAqB,EAAE,MAAM,2BAA2B,CAAC;AAEvE,eAAO,MAAM,kBAAkB,8BAA8B,CAAC;AAE9D,oBAAY,OAAO;IACjB,WAAW,iBAAiB;IAC5B,cAAc,oBAAoB;IAClC,aAAa,mBAAmB;IAChC,QAAQ,aAAa;IACrB,UAAU,eAAe;IACzB,UAAU,gBAAgB;IAC1B,YAAY,kBAAkB;IAC9B,kBAAkB,yBAAyB;IAC3C,kBAAkB,yBAAyB;IAC3C,0BAA0B,iCAAiC;IAC3D,wBAAwB,+BAA+B;IACvD,uBAAuB,8BAA8B;IACrD,wBAAwB,gCAAgC;IACxD,0BAA0B,kCAAkC;IAC5D,qBAAqB,4BAA4B;CAClD;AAED,MAAM,WAAW,qBAAqB;IACpC,IAAI,EAAE,OAAO,CAAC;IACd,QAAQ,EAAE,MAAM,CAAC;IACjB,0CAA0C;IAC1C,QAAQ,EAAE,MAAM,GAAG,SAAS,GAAG,OAAO,CAAC;IACvC,8DAA8D;IAC9D,EAAE,CAAC,EAAE,MAAM,CAAC;IACZ,4EAA4E;IAC5E,UAAU,CAAC,EAAE,MAAM,GAAG,QAAQ,GAAG,KAAK,CAAC;IACvC,kEAAkE;IAClE,mBAAmB,CAAC,EAAE,OAAO,CAAC;IAC9B,oEAAoE;IACpE,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,KAAK,EAAE,MAAM,CAAC;IACd,WAAW,EAAE,MAAM,CAAC;IACpB,uDAAuD;IACvD,YAAY,EAAE,SAAS,MAAM,EAAE,CAAC;IAChC,uEAAuE;IACvE,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,yEAAyE;IACzE,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,yDAAyD;IACzD,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,wEAAwE;IACxE,OAAO,EAAE,OAAO,CAAC;CAClB;AAED,MAAM,WAAW,iBAAiB;IAChC,MAAM,EAAE,OAAO,kBAAkB,CAAC;IAClC,WAAW,EAAE,MAAM,CAAC;IACpB,KAAK,EAAE,SAAS,OAAO,EAAE,CAAC;IAC1B,WAAW,EAAE,SAAS,qBAAqB,EAAE,CAAC;CAC/C;AAED,MAAM,WAAW,kBAAkB;IACjC,KAAK,CAAC,EAAE,SAAS,OAAO,EAAE,CAAC;CAC5B;AAED;;;;GAIG;AACH,MAAM,WAAW,sBAAsB;IACrC,OAAO,EAAE,SAAS;QAAE,YAAY,EAAE,MAAM,CAAC;QAAC,YAAY,EAAE,MAAM,CAAC;QAAC,KAAK,EAAE,MAAM,CAAA;KAAE,EAAE,CAAC;IAClF,OAAO,EAAE,SAAS;QAAE,YAAY,EAAE,MAAM,CAAC;QAAC,MAAM,EAAE,MAAM,CAAA;KAAE,EAAE,CAAC;CAC9D;AAED,wBAAsB,qBAAqB,CACzC,WAAW,EAAE,MAAM,EACnB,MAAM,EAAE,iBAAiB,GACxB,OAAO,CAAC,sBAAsB,CAAC,CAuCjC;AAoCD,wBAAgB,eAAe,CAC7B,UAAU,EAAE,qBAAqB,EACjC,OAAO,GAAE,kBAAuB,GAC/B,iBAAiB,CAoFnB;AA4CD;;4CAE4C;AAC5C,wBAAsB,uBAAuB,CAC3C,UAAU,EAAE,qBAAqB,EACjC,OAAO,GAAE,kBAAuB,GAC/B,OAAO,CAAC,iBAAiB,CAAC,CAkF5B;AAED,wBAAgB,YAAY,IAAI,SAAS;IAAE,IAAI,EAAE,OAAO,CAAC;IAAC,WAAW,EAAE,MAAM,CAAA;CAAE,EAAE,CAkBhF;AAED,wBAAgB,wBAAwB,CAAC,MAAM,EAAE,iBAAiB,GAAG,MAAM,CA+B1E"}
+{"version":3,"file":"fix-preview.d.ts","sourceRoot":"","sources":["../src/fix-preview.ts"],"names":[],"mappings":"AA+BA,OAAO,KAAK,EAAE,qBAAqB,EAAE,MAAM,2BAA2B,CAAC;AAEvE,eAAO,MAAM,kBAAkB,8BAA8B,CAAC;AAE9D,oBAAY,OAAO;IACjB,WAAW,iBAAiB;IAC5B,cAAc,oBAAoB;IAClC,aAAa,mBAAmB;IAChC,QAAQ,aAAa;IACrB,UAAU,eAAe;IACzB,UAAU,gBAAgB;IAC1B,YAAY,kBAAkB;IAC9B,kBAAkB,yBAAyB;IAC3C,kBAAkB,yBAAyB;IAC3C,0BAA0B,iCAAiC;IAC3D,wBAAwB,+BAA+B;IACvD,uBAAuB,8BAA8B;IACrD,wBAAwB,gCAAgC;IACxD,0BAA0B,kCAAkC;IAC5D,qBAAqB,4BAA4B;CAClD;AAED,MAAM,WAAW,qBAAqB;IACpC,IAAI,EAAE,OAAO,CAAC;IACd,QAAQ,EAAE,MAAM,CAAC;IACjB,0CAA0C;IAC1C,QAAQ,EAAE,MAAM,GAAG,SAAS,GAAG,OAAO,CAAC;IACvC,8DAA8D;IAC9D,EAAE,CAAC,EAAE,MAAM,CAAC;IACZ,4EAA4E;IAC5E,UAAU,CAAC,EAAE,MAAM,GAAG,QAAQ,GAAG,KAAK,CAAC;IACvC,kEAAkE;IAClE,mBAAmB,CAAC,EAAE,OAAO,CAAC;IAC9B,oEAAoE;IACpE,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,KAAK,EAAE,MAAM,CAAC;IACd,WAAW,EAAE,MAAM,CAAC;IACpB,uDAAuD;IACvD,YAAY,EAAE,SAAS,MAAM,EAAE,CAAC;IAChC,uEAAuE;IACvE,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,yEAAyE;IACzE,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,yDAAyD;IACzD,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,wEAAwE;IACxE,OAAO,EAAE,OAAO,CAAC;CAClB;AAED,MAAM,WAAW,iBAAiB;IAChC,MAAM,EAAE,OAAO,kBAAkB,CAAC;IAClC,WAAW,EAAE,MAAM,CAAC;IACpB,KAAK,EAAE,SAAS,OAAO,EAAE,CAAC;IAC1B,WAAW,EAAE,SAAS,qBAAqB,EAAE,CAAC;CAC/C;AAED,MAAM,WAAW,kBAAkB;IACjC,KAAK,CAAC,EAAE,SAAS,OAAO,EAAE,CAAC;CAC5B;AAED;;;;GAIG;AACH,MAAM,WAAW,sBAAsB;IACrC,OAAO,EAAE,SAAS;QAAE,YAAY,EAAE,MAAM,CAAC;QAAC,YAAY,EAAE,MAAM,CAAC;QAAC,KAAK,EAAE,MAAM,CAAA;KAAE,EAAE,CAAC;IAClF,OAAO,EAAE,SAAS;QAAE,YAAY,EAAE,MAAM,CAAC;QAAC,MAAM,EAAE,MAAM,CAAA;KAAE,EAAE,CAAC;CAC9D;AAED,wBAAsB,qBAAqB,CACzC,WAAW,EAAE,MAAM,EACnB,MAAM,EAAE,iBAAiB,GACxB,OAAO,CAAC,sBAAsB,CAAC,CAuCjC;AA+GD,wBAAgB,eAAe,CAC7B,UAAU,EAAE,qBAAqB,EACjC,OAAO,GAAE,kBAAuB,GAC/B,iBAAiB,CAyEnB;AA4CD;;4CAE4C;AAC5C,wBAAsB,uBAAuB,CAC3C,UAAU,EAAE,qBAAqB,EACjC,OAAO,GAAE,kBAAuB,GAC/B,OAAO,CAAC,iBAAiB,CAAC,CAsF5B;AAED,wBAAgB,YAAY,IAAI,SAAS;IAAE,IAAI,EAAE,OAAO,CAAC;IAAC,WAAW,EAAE,MAAM,CAAA;CAAE,EAAE,CAkBhF;AAED,wBAAgB,wBAAwB,CAAC,MAAM,EAAE,iBAAiB,GAAG,MAAM,CA+B1E"}

package/dist/fix-preview.js

@@ -15,7 +15,11 @@
  * Read-only at the inspector level. Schema: sharkcraft.fix-preview/v1
  */
 import { diagnoseActionHints, } from "./action-hint-diagnostics.js";
+import { KnowledgeSourceFormat } from '@shrkcrft/knowledge';
 import { buildKnowledgeStaleReport, ReferenceCheckOutcome, } from "./knowledge-stale.js";
+import { knowledgeRenameCommand } from "./knowledge-rename-command.js";
+import { KnowledgeRenameVerb } from "./knowledge-rename-verb.js";
+import { ReferenceFailure } from "./reference-failure.js";
 import { buildTemplateDriftReport } from "./template-drift.js";
 export const FIX_PREVIEW_SCHEMA = 'sharkcraft.fix-preview/v1';
 export var FixKind;
@@ -107,6 +111,73 @@ function buildActionHintFix(issue) {
 function slug(s) {
     return s.toLowerCase().replace(/[^a-z0-9]+/g, '-').replace(/(^-|-$)/g, '');
 }
+/**
+ * One non-ok stale-check row as a fix suggestion (round 15 follow-up, F4).
+ *
+ * Every command it suggests is one the dispatcher RUNS: `knowledge
+ * rename-file` / `rename-symbol` are read-only previews already (they take no
+ * `--dry-run` — the suggestion used to name it, and the dispatcher refused it),
+ * and `fix preview` takes the entry as `--target <id>` (it takes no positional:
+ * `--knowledge-stale <id>` read the id as the flag's value and previewed every
+ * kind). An entry `shrk fix --knowledge-stale --apply` cannot edit — declared
+ * in Markdown (a `references:` frontmatter list), or contributed by a pack —
+ * says so in its description and gets no `--apply` command.
+ */
+function buildKnowledgeStaleFix(c, declaredBy) {
+    const ref = c.reference;
+    // A `command:` reference names its target in `command` (it read `→ <unknown>`).
+    const target = ref.id ?? ref.path ?? ref.symbol ?? ref.command ?? '<unknown>';
+    const failing = c.outcome === ReferenceCheckOutcome.Stale || c.outcome === ReferenceCheckOutcome.Missing;
+    const notAutoFixable = declaredBy?.pack !== undefined
+        ? `not auto-fixable here: pack ${declaredBy.pack} contributes ${c.entryId} (${declaredBy.source}) — fix it upstream in the pack`
+        : declaredBy?.sourceFormat === KnowledgeSourceFormat.Markdown
+            ? `not auto-fixable: ${c.entryId} is declared in Markdown (${declaredBy.source}) — \`shrk fix --knowledge-stale --apply\` edits TypeScript entries only; edit its references: frontmatter list by hand`
+            : undefined;
+    const replaceWith = c.replaceWith;
+    const hasRenameTarget = replaceWith !== undefined &&
+        (replaceWith.path !== undefined || replaceWith.id !== undefined || replaceWith.symbol !== undefined);
+    // A rename answers a MOVED target only — a path that is gone (`rename-file`
+    // matches any reference by its `path`) or a symbol no longer declared in its
+    // file (`rename-symbol`), never a content / count mismatch on a file that is
+    // still there. And only for an entry `--apply` can edit: a rename plan lands
+    // through `shrk fix --knowledge-stale --apply`, which refuses a Markdown or a
+    // pack entry — a pack's reference is fixed in the pack, never renamed onto
+    // this tree (the stale engine offers it no rename candidate either).
+    const renameCommand = !failing || notAutoFixable !== undefined
+        ? undefined
+        : c.failure === ReferenceFailure.PathMissing && typeof ref.path === 'string'
+            ? knowledgeRenameCommand(KnowledgeRenameVerb.RenameFile, ref.path, replaceWith?.path)
+            : c.failure === ReferenceFailure.AnchorMissing && ref.kind === 'symbol' && typeof ref.symbol === 'string'
+                ? knowledgeRenameCommand(KnowledgeRenameVerb.RenameSymbol, ref.symbol, replaceWith?.symbol)
+                : undefined;
+    const applyCommand = failing && notAutoFixable === undefined
+        ? hasRenameTarget
+            ? 'shrk fix --knowledge-stale --apply'
+            : `shrk fix --knowledge-stale --apply ${c.outcome === ReferenceCheckOutcome.Missing ? '--drop-missing' : '--drop-stale'}`
+        : undefined;
+    const what = failing
+        ? 'Stale reference'
+        : c.outcome === ReferenceCheckOutcome.Invalid
+            ? 'Malformed reference'
+            : 'Unverifiable reference';
+    return {
+        kind: FixKind.KnowledgeStale,
+        targetId: c.entryId,
+        severity: c.outcome === ReferenceCheckOutcome.Missing ? 'error' : 'warning',
+        title: `${what} ${c.entryId} → ${target}`,
+        description: notAutoFixable !== undefined ? `${c.message} — ${notAutoFixable}.` : c.message,
+        nextCommands: [
+            `shrk knowledge references ${c.entryId}`,
+            ...(renameCommand !== undefined ? [renameCommand] : []),
+            `shrk fix preview --knowledge-stale --target ${c.entryId}`,
+            ...(applyCommand !== undefined ? [applyCommand] : []),
+        ],
+        ...(c.suggestion ? { draftBody: `// suggestion: ${c.suggestion}` } : {}),
+        ...(notAutoFixable !== undefined ? { humanReviewRequired: true, reason: notAutoFixable } : {}),
+        previewFileName: `knowledge-stale-${slug(c.entryId)}.preview.md`,
+        stubbed: true,
+    };
+}
 export function buildFixPreview(inspection, options = {}) {
     const defaultKinds = [
         FixKind.ActionHints,
@@ -126,25 +197,13 @@ export function buildFixPreview(inspection, options = {}) {
     }
     if (kinds.has(FixKind.KnowledgeStale)) {
         const stale = buildKnowledgeStaleReport(inspection);
+        // The entry's declaring file, format and pack — what decides whether
+        // `--apply` can edit it (round 15 follow-up, F4).
+        const declared = new Map(stale.entryVerdicts.map((v) => [v.entryId, v]));
         for (const c of stale.referenceChecks) {
             if (c.outcome === ReferenceCheckOutcome.Ok)
                 continue;
-            const target = c.reference.id ?? c.reference.path ?? c.reference.symbol ?? '<unknown>';
-            suggestions.push({
-                kind: FixKind.KnowledgeStale,
-                targetId: c.entryId,
-                severity: c.outcome === ReferenceCheckOutcome.Missing ? 'error' : 'warning',
-                title: `Stale reference ${c.entryId} → ${target}`,
-                description: c.message,
-                nextCommands: [
-                    `shrk knowledge references ${c.entryId}`,
-                    `shrk knowledge rename-symbol <old> <new> --dry-run`,
-                    `shrk fix preview --knowledge-stale ${c.entryId}`,
-                ],
-                ...(c.suggestion ? { draftBody: `// suggestion: ${c.suggestion}` } : {}),
-                previewFileName: `knowledge-stale-${slug(c.entryId)}.preview.md`,
-                stubbed: true,
-            });
+            suggestions.push(buildKnowledgeStaleFix(c, declared.get(c.entryId)));
         }
     }
     if (kinds.has(FixKind.TemplateDrift)) {
@@ -162,7 +221,9 @@ export function buildFixPreview(inspection, options = {}) {
                     nextCommands: [
                         `shrk templates get ${e.templateId}`,
                         `shrk paths list`,
-                        `shrk fix preview --template-drift ${e.templateId} --write-preview`,
+                        // `--target <id>`: `fix` takes no positional, so `--template-drift <id>`
+                        // read the id as the flag's value and previewed every kind (F4).
+                        `shrk fix preview --template-drift --target ${e.templateId} --write-preview`,
                     ],
                     ...(i.suggestedFix ? { draftBody: `// suggestion: ${i.suggestedFix}` } : {}),
                     previewFileName: `template-drift-${slug(e.templateId)}-${i.code}.preview.md`,
@@ -237,13 +298,14 @@ export async function buildFixPreviewExtended(inspection, options = {}) {
     const kinds = new Set(options.kinds ?? sync.kinds);
     const suggestions = [...sync.suggestions];
     if (kinds.has(FixKind.SelfConfig)) {
-        const { buildSelfConfigDoctorReport } = await import("./self-config-doctor.js");
-        const report = await buildSelfConfigDoctorReport(inspection);
+        // THE doctor (v2) — the same checks `shrk self-config doctor` runs.
+        const { buildSelfConfigDoctorReportV2 } = await import("./self-config-doctor-v2.js");
+        const report = await buildSelfConfigDoctorReportV2(inspection);
         for (const f of report.findings) {
             const severity = f.severity === 'error' ? 'error' : f.severity === 'warning' ? 'warning' : 'info';
             suggestions.push({
                 kind: FixKind.SelfConfig,
-                targetId: f.referencingId ?? f.referencedId ?? f.code,
+                targetId: f.sourceId || f.targetId || f.code,
                 severity,
                 title: `Self-config: ${f.code}`,
                 description: f.message,
@@ -253,8 +315,11 @@ export async function buildFixPreviewExtended(inspection, options = {}) {
         }
     }
     if (kinds.has(FixKind.PackConflict) || kinds.has(FixKind.StalePackSignature)) {
-        const { buildPackContributionsInventory } = await import("./pack-contributions-inventory.js");
-        const inv = buildPackContributionsInventory(inspection);
+        // The async inventory: loader-backed extraction, so a conflict preview is
+        // built from the ids the registries really load (the sync regex scrape
+        // marks them unverified).
+        const { buildPackContributionsInventoryAsync } = await import("./pack-contributions-inventory.js");
+        const inv = await buildPackContributionsInventoryAsync(inspection);
         for (const c of inv.conflicts) {
             if (c.kind === 'stale-signature' && kinds.has(FixKind.StalePackSignature)) {
                 suggestions.push({