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

package/dist/doctor-verdict-kind.js

@@ -0,0 +1,19 @@
+/**
+ * How one doctor run settles — THE reading of an `IDoctorResult` that every
+ * surface shares (`shrk doctor`, `shrk check`, the dashboard, MCP
+ * `inspect_sharkcraft_setup`), so none of them can call a setup ready that
+ * another calls not verified.
+ */
+export var DoctorVerdictKind;
+(function (DoctorVerdictKind) {
+    /** No error check, and every scope the doctor was asked to verify was verified. */
+    DoctorVerdictKind["Ready"] = "ready";
+    /**
+     * No error check, but part of the setup could not be verified (today: a
+     * compiled pack build with no build record, never compared with its
+     * source). Never "ready".
+     */
+    DoctorVerdictKind["NotVerified"] = "not-verified";
+    /** At least one error-severity check. */
+    DoctorVerdictKind["Errors"] = "errors";
+})(DoctorVerdictKind || (DoctorVerdictKind = {}));

package/dist/doctor-verdict.d.ts

@@ -0,0 +1,18 @@
+import type { IDoctorResult } from './doctor-result.js';
+import type { IDoctorVerdict } from './i-doctor-verdict.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 declare function doctorVerdict(result: IDoctorResult): IDoctorVerdict;
+//# sourceMappingURL=doctor-verdict.d.ts.map

package/dist/doctor-verdict.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"doctor-verdict.d.ts","sourceRoot":"","sources":["../src/doctor-verdict.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,oBAAoB,CAAC;AAExD,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,uBAAuB,CAAC;AAE5D;;;;;;;;;;;;;GAaG;AACH,wBAAgB,aAAa,CAAC,MAAM,EAAE,aAAa,GAAG,cAAc,CAWnE"}

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;AAE7E,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,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,7 @@
  */
 import { existsSync, readFileSync } from 'node:fs';
 import * as nodePath from 'node:path';
-import { importModuleViaLoader } from '@shrkcrft/core';
+import { importModuleViaLoader, RejectionCause } from '@shrkcrft/core';
 export const FEEDBACK_INGESTION_SCHEMA = 'sharkcraft.feedback-ingestion/v1';
 export var FeedbackBucket;
 (function (FeedbackBucket) {
@@ -141,7 +141,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 +305,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/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":"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,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

@@ -237,13 +237,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 +254,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({

package/dist/framework-extractor-outcomes.d.ts

@@ -0,0 +1,27 @@
+import { type IRejectedEntry } from '@shrkcrft/core';
+import type { IContributionFileIssue } from './i-contribution-file-issue.js';
+import type { ISharkcraftInspection } from './sharkcraft-inspector.js';
+/**
+ * Every pack `frameworkExtractorFiles` candidate, read through THE shared
+ * reading (`frameworkExtractorExports`) and shape predicate
+ * (`frameworkExtractorRejectionReasons`, both @shrkcrft/plugin-api) — the
+ * same two the runtime loader (`loadPackExtractors`, @shrkcrft/framework-
+ * scanners) applies. The inspector cannot import that loader (it sits above
+ * this layer via @shrkcrft/graph), so this is how framework extractors join
+ * THE rejection channel (round 12, 12.1).
+ *
+ * A candidate whose `framework` collides with a BUILT-IN extractor (nestjs,
+ * react, …) is refused only by the runtime loader, which alone knows the
+ * built-in registry; a collision between two contributed extractors is
+ * refused here too (the runtime keeps the first).
+ */
+export declare function loadFrameworkExtractorOutcomes(inspection: ISharkcraftInspection): Promise<{
+    readonly accepted: readonly {
+        readonly id: string;
+        readonly file: string;
+        readonly packageName: string;
+    }[];
+    readonly rejected: readonly IRejectedEntry[];
+    readonly issues: readonly IContributionFileIssue[];
+}>;
+//# sourceMappingURL=framework-extractor-outcomes.d.ts.map

package/dist/framework-extractor-outcomes.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"framework-extractor-outcomes.d.ts","sourceRoot":"","sources":["../src/framework-extractor-outcomes.ts"],"names":[],"mappings":"AAEA,OAAO,EAAyC,KAAK,cAAc,EAAE,MAAM,gBAAgB,CAAC;AAE5F,OAAO,KAAK,EAAE,sBAAsB,EAAE,MAAM,gCAAgC,CAAC;AAC7E,OAAO,KAAK,EAAE,qBAAqB,EAAE,MAAM,2BAA2B,CAAC;AAEvE;;;;;;;;;;;;;GAaG;AACH,wBAAsB,8BAA8B,CAAC,UAAU,EAAE,qBAAqB,GAAG,OAAO,CAAC;IAC/F,QAAQ,CAAC,QAAQ,EAAE,SAAS;QAAE,QAAQ,CAAC,EAAE,EAAE,MAAM,CAAC;QAAC,QAAQ,CAAC,IAAI,EAAE,MAAM,CAAC;QAAC,QAAQ,CAAC,WAAW,EAAE,MAAM,CAAA;KAAE,EAAE,CAAC;IAC3G,QAAQ,CAAC,QAAQ,EAAE,SAAS,cAAc,EAAE,CAAC;IAC7C,QAAQ,CAAC,MAAM,EAAE,SAAS,sBAAsB,EAAE,CAAC;CACpD,CAAC,CA+DD"}

package/dist/framework-extractor-outcomes.js

@@ -0,0 +1,83 @@
+import { existsSync } from 'node:fs';
+import * as nodePath from 'node:path';
+import { importModuleViaLoader, RejectionCause } from '@shrkcrft/core';
+import { frameworkExtractorExports, frameworkExtractorRejectionReasons } from '@shrkcrft/plugin-api';
+/**
+ * Every pack `frameworkExtractorFiles` candidate, read through THE shared
+ * reading (`frameworkExtractorExports`) and shape predicate
+ * (`frameworkExtractorRejectionReasons`, both @shrkcrft/plugin-api) — the
+ * same two the runtime loader (`loadPackExtractors`, @shrkcrft/framework-
+ * scanners) applies. The inspector cannot import that loader (it sits above
+ * this layer via @shrkcrft/graph), so this is how framework extractors join
+ * THE rejection channel (round 12, 12.1).
+ *
+ * A candidate whose `framework` collides with a BUILT-IN extractor (nestjs,
+ * react, …) is refused only by the runtime loader, which alone knows the
+ * built-in registry; a collision between two contributed extractors is
+ * refused here too (the runtime keeps the first).
+ */
+export async function loadFrameworkExtractorOutcomes(inspection) {
+    const accepted = [];
+    const rejected = [];
+    const issues = [];
+    const seen = new Map();
+    for (const pack of inspection.packs.validPacks ?? []) {
+        const files = pack.manifest?.contributions
+            ?.frameworkExtractorFiles;
+        for (const rel of files ?? []) {
+            const file = nodePath.resolve(pack.packageRoot, rel);
+            if (!existsSync(file)) {
+                issues.push({
+                    severity: 'warning',
+                    code: 'missing-file',
+                    message: `Pack ${pack.packageName} declares ${rel} but the file is missing.`,
+                    source: file,
+                    packageName: pack.packageName,
+                });
+                continue;
+            }
+            let mod;
+            try {
+                mod = await importModuleViaLoader(file);
+            }
+            catch (e) {
+                issues.push({
+                    severity: 'warning',
+                    code: 'load-failed',
+                    message: `Pack ${pack.packageName} (${rel}): ${(e.message ?? String(e)).split('\n')[0].trim()}`,
+                    source: file,
+                    packageName: pack.packageName,
+                });
+                continue;
+            }
+            for (const c of frameworkExtractorExports(mod)) {
+                const reasons = frameworkExtractorRejectionReasons(c.value);
+                const framework = c.value?.framework;
+                const at = { file, index: c.index, exportName: c.exportName };
+                if (reasons.length > 0) {
+                    rejected.push({
+                        ...at,
+                        ...(typeof framework === 'string' ? { entryId: framework } : {}),
+                        reasons,
+                        cause: RejectionCause.Invalid,
+                    });
+                    continue;
+                }
+                const name = framework;
+                const prev = seen.get(name);
+                if (prev !== undefined) {
+                    rejected.push({
+                        ...at,
+                        entryId: name,
+                        reasons: [`framework: "${name}" is already registered by ${prev}`],
+                        cause: RejectionCause.DuplicateId,
+                    });
+                    continue;
+                }
+                seen.set(name, `${pack.packageName} (${rel})`);
+                accepted.push({ id: name, file, packageName: pack.packageName });
+            }
+        }
+    }
+    return { accepted, rejected, issues };
+}

package/dist/fuzzy-impact.js

@@ -379,7 +379,7 @@ function buildResolution(inspection, query, resolution, opts) {
             resolvedLabel: best.label,
             alternatives,
             shouldRunImpact: false,
-            followUpCommands: [`shrk commands get ${best.id}`, `shrk ${best.id} --help`],
+            followUpCommands: [`shrk help ${best.id}`, `shrk ${best.id} --help`],
             diagnostics: [
                 `"${best.id}" resolves to a CLI command, not a code construct. Use \`shrk impact <file>\` to analyse files this command would touch.`,
             ],

package/dist/healing-plan.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"healing-plan.d.ts","sourceRoot":"","sources":["../src/healing-plan.ts"],"names":[],"mappings":"AAOA,OAAO,EAIL,KAAK,kBAAkB,EACxB,MAAM,0BAA0B,CAAC;AAGlC,eAAO,MAAM,mBAAmB,+BAA+B,CAAC;AAEhE,oBAAY,gBAAgB;IAC1B,KAAK,UAAU;IACf,IAAI,SAAS;IACb,MAAM,WAAW;IACjB,OAAO,YAAY;CACpB;AAED,MAAM,WAAW,YAAY;IAC3B,MAAM,EAAE,OAAO,mBAAmB,CAAC;IACnC,WAAW,EAAE,MAAM,CAAC;IACpB,SAAS,EAAE,gBAAgB,CAAC;IAC5B,YAAY,EAAE,MAAM,CAAC;IACrB,mBAAmB,EAAE,SAAS,kBAAkB,EAAE,CAAC;IACnD,UAAU,EAAE,KAAK,GAAG,QAAQ,GAAG,MAAM,CAAC;IACtC,YAAY,EAAE,SAAS,MAAM,EAAE,CAAC;IAChC,iBAAiB,EAAE,SAAS,MAAM,EAAE,CAAC;IACrC,mBAAmB,EAAE,SAAS,MAAM,EAAE,CAAC;IACvC,mBAAmB,EAAE,SAAS,MAAM,EAAE,CAAC;IACvC,iBAAiB,EAAE,SAAS,MAAM,EAAE,CAAC;IACrC,WAAW,EAAE,SAAS,MAAM,EAAE,CAAC;IAC/B,qBAAqB,EAAE,OAAO,CAAC;IAC/B,oBAAoB,EAAE,OAAO,CAAC;IAC9B,iBAAiB,EAAE,MAAM,CAAC;IAC1B,KAAK,EAAE,SAAS,MAAM,EAAE,CAAC;CAC1B;AAoQD,wBAAgB,yBAAyB,CAAC,SAAS,EAAE,MAAM,GAAG,YAAY,CAMzE;AAED,wBAAgB,wBAAwB,CAAC,QAAQ,EAAE,MAAM,GAAG,YAAY,CAcvE;AAED,wBAAgB,0BAA0B,CAAC,UAAU,EAAE,MAAM,GAAG,YAAY,CAqB3E;AAED,wBAAgB,2BAA2B,CACzC,OAAO,EAAE,MAAM,EACf,QAAQ,EAAE,MAAM,EAChB,UAAU,EAAE,MAAM,GACjB,YAAY,CAQd;AAID,wBAAgB,qBAAqB,CAAC,CAAC,EAAE,YAAY,GAAG,MAAM,CAsB7D;AAED,wBAAgB,yBAAyB,CAAC,CAAC,EAAE,YAAY,GAAG,MAAM,CAiBjE"}
+{"version":3,"file":"healing-plan.d.ts","sourceRoot":"","sources":["../src/healing-plan.ts"],"names":[],"mappings":"AAOA,OAAO,EAIL,KAAK,kBAAkB,EACxB,MAAM,0BAA0B,CAAC;AAGlC,eAAO,MAAM,mBAAmB,+BAA+B,CAAC;AAEhE,oBAAY,gBAAgB;IAC1B,KAAK,UAAU;IACf,IAAI,SAAS;IACb,MAAM,WAAW;IACjB,OAAO,YAAY;CACpB;AAED,MAAM,WAAW,YAAY;IAC3B,MAAM,EAAE,OAAO,mBAAmB,CAAC;IACnC,WAAW,EAAE,MAAM,CAAC;IACpB,SAAS,EAAE,gBAAgB,CAAC;IAC5B,YAAY,EAAE,MAAM,CAAC;IACrB,mBAAmB,EAAE,SAAS,kBAAkB,EAAE,CAAC;IACnD,UAAU,EAAE,KAAK,GAAG,QAAQ,GAAG,MAAM,CAAC;IACtC,YAAY,EAAE,SAAS,MAAM,EAAE,CAAC;IAChC,iBAAiB,EAAE,SAAS,MAAM,EAAE,CAAC;IACrC,mBAAmB,EAAE,SAAS,MAAM,EAAE,CAAC;IACvC,mBAAmB,EAAE,SAAS,MAAM,EAAE,CAAC;IACvC,iBAAiB,EAAE,SAAS,MAAM,EAAE,CAAC;IACrC,WAAW,EAAE,SAAS,MAAM,EAAE,CAAC;IAC/B,qBAAqB,EAAE,OAAO,CAAC;IAC/B,oBAAoB,EAAE,OAAO,CAAC;IAC9B,iBAAiB,EAAE,MAAM,CAAC;IAC1B,KAAK,EAAE,SAAS,MAAM,EAAE,CAAC;CAC1B;AAmQD,wBAAgB,yBAAyB,CAAC,SAAS,EAAE,MAAM,GAAG,YAAY,CAMzE;AAED,wBAAgB,wBAAwB,CAAC,QAAQ,EAAE,MAAM,GAAG,YAAY,CAcvE;AAED,wBAAgB,0BAA0B,CAAC,UAAU,EAAE,MAAM,GAAG,YAAY,CAqB3E;AAED,wBAAgB,2BAA2B,CACzC,OAAO,EAAE,MAAM,EACf,QAAQ,EAAE,MAAM,EAChB,UAAU,EAAE,MAAM,GACjB,YAAY,CAQd;AAID,wBAAgB,qBAAqB,CAAC,CAAC,EAAE,YAAY,GAAG,MAAM,CAsB7D;AAED,wBAAgB,yBAAyB,CAAC,CAAC,EAAE,YAAY,GAAG,MAAM,CAiBjE"}

package/dist/healing-plan.js

@@ -15,7 +15,7 @@ export var HealingInputKind;
     HealingInputKind["Report"] = "report";
     HealingInputKind["Command"] = "command";
 })(HealingInputKind || (HealingInputKind = {}));
-const GENERIC_NEXT = 'shrk diagnostics suggest "<paste error text here>"';
+const GENERIC_NEXT = 'shrk recommend --from-error <stderr-file>';
 function inferFromKeywords(text) {
     const causes = [];
     const recover = [];
@@ -58,8 +58,7 @@ function inferFromKeywords(text) {
     if (lower.includes('migration') && (lower.includes('readiness') || lower.includes('blocker'))) {
         causes.push('Migration readiness gate reported a blocker.');
         recover.push('Resolve each blocker before re-running the gate.');
-        cmds.push('shrk migration readiness --profile <id>');
-        cmds.push('shrk migration profiles');
+        cmds.push('shrk migrate plan <id>');
         docs.push('docs/migration-readiness.md');
     }
     // Unknown command.
@@ -202,7 +201,7 @@ function buildPlan(ctx) {
     const safeRecoverySteps = [
         'Re-read the failing command output carefully (do not skip).',
         'Reproduce the failure locally before changing code.',
-        'If unsure, run `shrk doctor` and `shrk diagnostics suggest`.',
+        'If unsure, run `shrk doctor` and `shrk recommend --from-error <stderr-file>`.',
     ];
     for (const r of inferred.recover)
         safeRecoverySteps.push(r);

package/dist/helper-catalog.d.ts

@@ -0,0 +1,25 @@
+import { type IRejectedEntry } from '@shrkcrft/core';
+import type { IHelperView } from './helper-view.js';
+import { type IPackHelperDoctorIssue } from './pack-helper-registry.js';
+import type { ISharkcraftInspection } from './sharkcraft-inspector.js';
+export interface IHelperCatalog {
+    /** Every helper, deduped by id: built-ins first, then local, then pack (loader order). */
+    readonly entries: readonly IHelperView[];
+    /** load-failed / missing-file / invalid-helper / duplicate-id / helper-warning issues. */
+    readonly issues: readonly IPackHelperDoctorIssue[];
+    /** Every helper FILE considered (local `helpers.ts` + pack `helperFiles[]`) and whether it loaded. */
+    readonly files: readonly {
+        readonly file: string;
+        readonly status: 'loaded' | 'failed' | 'missing';
+    }[];
+    /**
+     * Every declared helper that did not make the catalog (round 12, 12.1): the
+     * loader's refusals plus a contributed helper shadowed by a built-in id.
+     */
+    readonly rejected: readonly IRejectedEntry[];
+}
+/** Build the catalog. One pack-helper load per call (it returns entries AND issues). */
+export declare function listAllHelpers(inspection: ISharkcraftInspection): Promise<IHelperCatalog>;
+/** One helper by id, from the same catalog `helper list` prints. */
+export declare function findHelper(inspection: ISharkcraftInspection, id: string): Promise<IHelperView | null>;
+//# sourceMappingURL=helper-catalog.d.ts.map

package/dist/helper-catalog.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"helper-catalog.d.ts","sourceRoot":"","sources":["../src/helper-catalog.ts"],"names":[],"mappings":"AAWA,OAAO,EAAkB,KAAK,cAAc,EAAE,MAAM,gBAAgB,CAAC;AAErE,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,kBAAkB,CAAC;AACpD,OAAO,EAAmB,KAAK,sBAAsB,EAAE,MAAM,2BAA2B,CAAC;AACzF,OAAO,KAAK,EAAE,qBAAqB,EAAE,MAAM,2BAA2B,CAAC;AAEvE,MAAM,WAAW,cAAc;IAC7B,0FAA0F;IAC1F,QAAQ,CAAC,OAAO,EAAE,SAAS,WAAW,EAAE,CAAC;IACzC,0FAA0F;IAC1F,QAAQ,CAAC,MAAM,EAAE,SAAS,sBAAsB,EAAE,CAAC;IACnD,sGAAsG;IACtG,QAAQ,CAAC,KAAK,EAAE,SAAS;QAAE,QAAQ,CAAC,IAAI,EAAE,MAAM,CAAC;QAAC,QAAQ,CAAC,MAAM,EAAE,QAAQ,GAAG,QAAQ,GAAG,SAAS,CAAA;KAAE,EAAE,CAAC;IACvG;;;OAGG;IACH,QAAQ,CAAC,QAAQ,EAAE,SAAS,cAAc,EAAE,CAAC;CAC9C;AAED,wFAAwF;AACxF,wBAAsB,cAAc,CAAC,UAAU,EAAE,qBAAqB,GAAG,OAAO,CAAC,cAAc,CAAC,CAsE/F;AAED,oEAAoE;AACpE,wBAAsB,UAAU,CAC9B,UAAU,EAAE,qBAAqB,EACjC,EAAE,EAAE,MAAM,GACT,OAAO,CAAC,WAAW,GAAG,IAAI,CAAC,CAG7B"}