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

package/dist/template-authoring.js

@@ -17,6 +17,7 @@
  * Scope: update + remove only. `scaffold` / `add` already ship at
  * `packages/cli/src/commands/templates.command.ts`.
  */
+import { knowledgeReferences } from '@shrkcrft/knowledge';
 export const TEMPLATE_AUTHORING_SCHEMA = 'sharkcraft.template-authoring/v1';
 export var TemplateAuthoringOperation;
 (function (TemplateAuthoringOperation) {
@@ -214,7 +215,7 @@ function findReverseReferences(templateId, ctx) {
     const out = [];
     // Knowledge entries referencing this template via `references[kind=template]`.
     for (const k of ctx.knowledgeEntries ?? []) {
-        for (const ref of k.references ?? []) {
+        for (const ref of knowledgeReferences(k)) {
             if (ref.kind === 'template' && ref.id === templateId) {
                 out.push({
                     fromKind: 'knowledge',

package/dist/validate-contribution-file.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"validate-contribution-file.d.ts","sourceRoot":"","sources":["../src/validate-contribution-file.ts"],"names":[],"mappings":"AAoCA,OAAO,KAAK,EAAE,2BAA2B,EAAE,MAAM,qCAAqC,CAAC;AAsEvF,uDAAuD;AACvD,wBAAsB,wBAAwB,CAAC,IAAI,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,OAAO,CAAC,2BAA2B,CAAC,CAyI/G"}
+{"version":3,"file":"validate-contribution-file.d.ts","sourceRoot":"","sources":["../src/validate-contribution-file.ts"],"names":[],"mappings":"AAoCA,OAAO,KAAK,EAAE,2BAA2B,EAAE,MAAM,qCAAqC,CAAC;AA+DvF,uDAAuD;AACvD,wBAAsB,wBAAwB,CAAC,IAAI,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,OAAO,CAAC,2BAA2B,CAAC,CAoJ/G"}

package/dist/validate-contribution-file.js

@@ -22,7 +22,7 @@
  */
 import { existsSync } from 'node:fs';
 import { importModuleViaLoader, readContributionExport, RejectionCause } from '@shrkcrft/core';
-import { MarkdownKnowledgeLoader, TypeScriptKnowledgeLoader } from '@shrkcrft/knowledge';
+import { MarkdownKnowledgeLoader, TypeScriptKnowledgeLoader, validateKnowledgeEntries } from '@shrkcrft/knowledge';
 import { loadTemplatesFromFile } from '@shrkcrft/templates';
 import { loadPipelinesFromFile } from '@shrkcrft/pipelines';
 import { loadPresetsFromFile } from '@shrkcrft/presets';
@@ -34,6 +34,7 @@ import { conventionRejectionReasons } from "./convention-registry.js";
 import { decisionRejectionReasons } from "./decision-records.js";
 import { delegateRecipeRejectionReasons } from "./delegate-pack-recipes.js";
 import { feedbackRuleRejectionReasons } from "./feedback-ingestion.js";
+import { isKnowledgeContributionSlot } from "./knowledge-contribution-slots.js";
 import { migrationProfileRejectionReasons } from "./migration-profile-registry.js";
 import { packHelperRejectionReasons } from "./pack-helper-registry.js";
 import { contributionKindForSlot } from "./pack-contributions-inventory.js";
@@ -63,13 +64,6 @@ const REGISTRY_SLOTS = {
     agentTestFiles: { singleObject: false, reasons: testDefinitionRejectionReasons },
     delegateRecipeFiles: { namedKeys: ['delegateRecipes'], reasons: delegateRecipeRejectionReasons, dedupe: true },
 };
-const KNOWLEDGE_SLOTS = new Set([
-    'knowledgeFiles',
-    'ruleFiles',
-    'pathFiles',
-    'pathConventionFiles',
-    'docsFiles',
-]);
 const GATE_PLANE_SLOTS = new Set([
     'wiringRuleFiles',
     'registryFiles',
@@ -102,14 +96,25 @@ export async function validateContributionFile(slot, file) {
     });
     if (!existsSync(file))
         return done([], [], 'the file does not exist');
-    if (KNOWLEDGE_SLOTS.has(slot)) {
+    // THE knowledge slots (round 15 follow-up, F11): the slot says "knowledge";
+    // which knowledge loader reads the file is the inspection's own choice (the
+    // same `canLoad` order). A file neither loader reads is `unvalidated` — the
+    // consumer skips it as an "unsupported contribution file".
+    if (isKnowledgeContributionSlot(slot)) {
         const ts = new TypeScriptKnowledgeLoader();
         const md = new MarkdownKnowledgeLoader();
         const loader = ts.canLoad(file) ? ts : md.canLoad(file) ? md : null;
         if (!loader)
             return { ...done([], []), unvalidated: true };
         const r = await loader.load(file);
-        return done(r.entries.map((e) => e.id), r.rejected ?? [], importError(r.warnings));
+        // …and THE validator the consumer's inspection runs, over what the loader
+        // ACCEPTED (round 15): an issue that keeps the entry — a non-list
+        // `references` crashed every consumer verb while `--load` said "No issues".
+        const entryIssues = validateKnowledgeEntries(r.entries, { isPackContributed: () => true }).issues;
+        return {
+            ...done(r.entries.map((e) => e.id), r.rejected ?? [], importError(r.warnings)),
+            ...(entryIssues.length > 0 ? { entryIssues } : {}),
+        };
     }
     if (slot === 'templateFiles') {
         const r = await loadTemplatesFromFile(file);

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

@@ -1 +1 @@
-{"version":3,"file":"why-file.d.ts","sourceRoot":"","sources":["../src/why-file.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;GAcG;AAMH,OAAO,KAAK,EAAE,qBAAqB,EAAE,MAAM,2BAA2B,CAAC;AAEvE,eAAO,MAAM,eAAe,sBAAsB,CAAC;AAEnD,MAAM,MAAM,aAAa,GAAG,MAAM,GAAG,WAAW,GAAG,SAAS,CAAC;AAE7D,MAAM,WAAW,UAAU;IACzB,QAAQ,CAAC,SAAS,EAAE,MAAM,CAAC;IAC3B,QAAQ,CAAC,YAAY,EAAE,MAAM,CAAC;IAC9B,QAAQ,CAAC,YAAY,EAAE,MAAM,CAAC;IAC9B,QAAQ,CAAC,IAAI,EAAE,aAAa,CAAC;CAC9B;AAED,MAAM,WAAW,kBAAkB;IACjC,QAAQ,CAAC,EAAE,EAAE,MAAM,CAAC;IACpB,QAAQ,CAAC,KAAK,EAAE,MAAM,CAAC;IACvB,QAAQ,CAAC,aAAa,EAAE,MAAM,CAAC;IAC/B,QAAQ,CAAC,MAAM,CAAC,EAAE,MAAM,CAAC;CAC1B;AAED,MAAM,WAAW,QAAQ;IACvB,QAAQ,CAAC,EAAE,EAAE,MAAM,CAAC;IACpB,QAAQ,CAAC,KAAK,EAAE,MAAM,CAAC;IACvB,QAAQ,CAAC,QAAQ,EAAE,MAAM,CAAC;IAC1B,QAAQ,CAAC,KAAK,EAAE,SAAS,MAAM,EAAE,CAAC;IAClC,QAAQ,CAAC,IAAI,EAAE,SAAS,MAAM,EAAE,CAAC;IACjC,QAAQ,CAAC,WAAW,EAAE,SAAS,MAAM,EAAE,CAAC;IACxC,QAAQ,CAAC,MAAM,CAAC,EAAE,MAAM,CAAC;IACzB,QAAQ,CAAC,MAAM,EAAE,MAAM,CAAC;CACzB;AAED,MAAM,WAAW,gBAAgB;IAC/B,QAAQ,CAAC,EAAE,EAAE,MAAM,CAAC;IACpB,QAAQ,CAAC,KAAK,EAAE,MAAM,CAAC;IACvB,qFAAqF;IACrF,QAAQ,CAAC,QAAQ,CAAC,EAAE,MAAM,CAAC;IAC3B,4GAA4G;IAC5G,QAAQ,CAAC,MAAM,CAAC,EAAE,OAAO,CAAC;IAC1B,QAAQ,CAAC,IAAI,EAAE,SAAS,MAAM,EAAE,CAAC;IACjC,QAAQ,CAAC,gBAAgB,CAAC,EAAE,SAAS,MAAM,EAAE,CAAC;IAC9C,QAAQ,CAAC,cAAc,CAAC,EAAE,SAAS,MAAM,EAAE,CAAC;IAC5C,QAAQ,CAAC,MAAM,CAAC,EAAE,MAAM,CAAC;CAC1B;AAED,MAAM,WAAW,aAAa;IAC5B,QAAQ,CAAC,EAAE,EAAE,MAAM,CAAC;IACpB,QAAQ,CAAC,KAAK,EAAE,MAAM,CAAC;IACvB,QAAQ,CAAC,IAAI,EAAE,MAAM,CAAC;IACtB,QAAQ,CAAC,MAAM,CAAC,EAAE,MAAM,CAAC;IACzB,QAAQ,CAAC,MAAM,EAAE,MAAM,CAAC;CACzB;AAED,MAAM,WAAW,UAAU;IACzB,QAAQ,CAAC,MAAM,EAAE,OAAO,eAAe,CAAC;IACxC,QAAQ,CAAC,MAAM,EAAE,UAAU,CAAC;IAC5B,QAAQ,CAAC,eAAe,CAAC,EAAE,MAAM,CAAC;IAClC,QAAQ,CAAC,aAAa,CAAC,EAAE,MAAM,CAAC;IAChC,QAAQ,CAAC,eAAe,EAAE,SAAS,kBAAkB,EAAE,CAAC;IACxD,QAAQ,CAAC,KAAK,EAAE,SAAS,QAAQ,EAAE,CAAC;IACpC,QAAQ,CAAC,UAAU,EAAE,SAAS,gBAAgB,EAAE,CAAC;IACjD,QAAQ,CAAC,SAAS,EAAE,SAAS,aAAa,EAAE,CAAC;IAC7C,QAAQ,CAAC,aAAa,EAAE,SAAS,MAAM,EAAE,CAAC;CAC3C;AAED,MAAM,WAAW,oBAAoB;IACnC,QAAQ,CAAC,UAAU,EAAE,qBAAqB,CAAC;IAC3C,QAAQ,CAAC,WAAW,EAAE,MAAM,CAAC;IAC7B,QAAQ,CAAC,MAAM,EAAE,MAAM,CAAC;IACxB,QAAQ,CAAC,KAAK,CAAC,EAAE,MAAM,CAAC;CACzB;AAED,wBAAgB,cAAc,CAAC,KAAK,EAAE,oBAAoB,GAAG,UAAU,CAuBtE"}
+{"version":3,"file":"why-file.d.ts","sourceRoot":"","sources":["../src/why-file.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;GAcG;AAOH,OAAO,KAAK,EAAE,qBAAqB,EAAE,MAAM,2BAA2B,CAAC;AAEvE,eAAO,MAAM,eAAe,sBAAsB,CAAC;AAEnD,MAAM,MAAM,aAAa,GAAG,MAAM,GAAG,WAAW,GAAG,SAAS,CAAC;AAE7D,MAAM,WAAW,UAAU;IACzB,QAAQ,CAAC,SAAS,EAAE,MAAM,CAAC;IAC3B,QAAQ,CAAC,YAAY,EAAE,MAAM,CAAC;IAC9B,QAAQ,CAAC,YAAY,EAAE,MAAM,CAAC;IAC9B,QAAQ,CAAC,IAAI,EAAE,aAAa,CAAC;CAC9B;AAED,MAAM,WAAW,kBAAkB;IACjC,QAAQ,CAAC,EAAE,EAAE,MAAM,CAAC;IACpB,QAAQ,CAAC,KAAK,EAAE,MAAM,CAAC;IACvB,QAAQ,CAAC,aAAa,EAAE,MAAM,CAAC;IAC/B,QAAQ,CAAC,MAAM,CAAC,EAAE,MAAM,CAAC;CAC1B;AAED,MAAM,WAAW,QAAQ;IACvB,QAAQ,CAAC,EAAE,EAAE,MAAM,CAAC;IACpB,QAAQ,CAAC,KAAK,EAAE,MAAM,CAAC;IACvB,QAAQ,CAAC,QAAQ,EAAE,MAAM,CAAC;IAC1B,QAAQ,CAAC,KAAK,EAAE,SAAS,MAAM,EAAE,CAAC;IAClC,QAAQ,CAAC,IAAI,EAAE,SAAS,MAAM,EAAE,CAAC;IACjC,QAAQ,CAAC,WAAW,EAAE,SAAS,MAAM,EAAE,CAAC;IACxC,QAAQ,CAAC,MAAM,CAAC,EAAE,MAAM,CAAC;IACzB,QAAQ,CAAC,MAAM,EAAE,MAAM,CAAC;CACzB;AAED,MAAM,WAAW,gBAAgB;IAC/B,QAAQ,CAAC,EAAE,EAAE,MAAM,CAAC;IACpB,QAAQ,CAAC,KAAK,EAAE,MAAM,CAAC;IACvB,qFAAqF;IACrF,QAAQ,CAAC,QAAQ,CAAC,EAAE,MAAM,CAAC;IAC3B,4GAA4G;IAC5G,QAAQ,CAAC,MAAM,CAAC,EAAE,OAAO,CAAC;IAC1B,QAAQ,CAAC,IAAI,EAAE,SAAS,MAAM,EAAE,CAAC;IACjC,QAAQ,CAAC,gBAAgB,CAAC,EAAE,SAAS,MAAM,EAAE,CAAC;IAC9C,QAAQ,CAAC,cAAc,CAAC,EAAE,SAAS,MAAM,EAAE,CAAC;IAC5C,QAAQ,CAAC,MAAM,CAAC,EAAE,MAAM,CAAC;CAC1B;AAED,MAAM,WAAW,aAAa;IAC5B,QAAQ,CAAC,EAAE,EAAE,MAAM,CAAC;IACpB,QAAQ,CAAC,KAAK,EAAE,MAAM,CAAC;IACvB,QAAQ,CAAC,IAAI,EAAE,MAAM,CAAC;IACtB,QAAQ,CAAC,MAAM,CAAC,EAAE,MAAM,CAAC;IACzB,QAAQ,CAAC,MAAM,EAAE,MAAM,CAAC;CACzB;AAED,MAAM,WAAW,UAAU;IACzB,QAAQ,CAAC,MAAM,EAAE,OAAO,eAAe,CAAC;IACxC,QAAQ,CAAC,MAAM,EAAE,UAAU,CAAC;IAC5B,QAAQ,CAAC,eAAe,CAAC,EAAE,MAAM,CAAC;IAClC,QAAQ,CAAC,aAAa,CAAC,EAAE,MAAM,CAAC;IAChC,QAAQ,CAAC,eAAe,EAAE,SAAS,kBAAkB,EAAE,CAAC;IACxD,QAAQ,CAAC,KAAK,EAAE,SAAS,QAAQ,EAAE,CAAC;IACpC,QAAQ,CAAC,UAAU,EAAE,SAAS,gBAAgB,EAAE,CAAC;IACjD,QAAQ,CAAC,SAAS,EAAE,SAAS,aAAa,EAAE,CAAC;IAC7C,QAAQ,CAAC,aAAa,EAAE,SAAS,MAAM,EAAE,CAAC;CAC3C;AAED,MAAM,WAAW,oBAAoB;IACnC,QAAQ,CAAC,UAAU,EAAE,qBAAqB,CAAC;IAC3C,QAAQ,CAAC,WAAW,EAAE,MAAM,CAAC;IAC7B,QAAQ,CAAC,MAAM,EAAE,MAAM,CAAC;IACxB,QAAQ,CAAC,KAAK,CAAC,EAAE,MAAM,CAAC;CACzB;AAED,wBAAgB,cAAc,CAAC,KAAK,EAAE,oBAAoB,GAAG,UAAU,CAuBtE"}

package/dist/why-file.js

@@ -17,6 +17,7 @@ import { existsSync, statSync } from 'node:fs';
 import * as nodePath from 'node:path';
 import { boundaryRuleCovers, boundaryRuleSeverity, matchesAny } from '@shrkcrft/boundaries';
 import { deriveApplicability } from '@shrkcrft/rules';
+import { knowledgeAnchors, knowledgeReferences } from '@shrkcrft/knowledge';
 export const WHY_FILE_SCHEMA = 'sharkcraft.why/v1';
 export function buildWhyReport(input) {
     const limit = input.limit ?? 10;
@@ -113,14 +114,14 @@ function matchPathConventions(inspection, target) {
 function ruleReasonForFile(r, target, inferredPackage, inferredLayer) {
     const rel = target.relativePath;
     // 1. Explicit file/directory references that cover the target.
-    for (const ref of r.references ?? []) {
+    for (const ref of knowledgeReferences(r)) {
         const p = (ref.path ?? '').replace(/^\.\//, '');
         if ((ref.kind === 'file' || ref.kind === 'directory') && p && pathCovers(rel, p)) {
             return ref.kind === 'directory' ? `directory reference ${p}` : `file reference ${p}`;
         }
     }
     // 1b. Anchors that point at a covering path.
-    for (const a of r.anchors ?? []) {
+    for (const a of knowledgeAnchors(r)) {
         const p = (a.path ?? '').replace(/^\.\//, '');
         if (p && pathCovers(rel, p))
             return `anchor path ${p}`;
@@ -147,7 +148,7 @@ function ruleReasonForFile(r, target, inferredPackage, inferredLayer) {
     // 3. Package reference / scope-or-tag equal to the inferred package or layer.
     const labels = [...(r.scope ?? []), ...(r.tags ?? [])];
     if (inferredPackage) {
-        for (const ref of r.references ?? []) {
+        for (const ref of knowledgeReferences(r)) {
             if (ref.kind === 'package' && ref.id && samePath(ref.id, inferredPackage)) {
                 return `package reference ${ref.id}`;
             }
@@ -216,7 +217,7 @@ function matchKnowledge(inspection, target, limit) {
     const rel = target.relativePath.toLowerCase();
     const out = [];
     for (const k of inspection.knowledgeEntries) {
-        const refs = (k.references ?? []).map((r) => r.path ?? '');
+        const refs = knowledgeReferences(k).map((r) => r.path ?? '');
         const refMatch = refs.some((p) => p.toLowerCase() === rel || p.toLowerCase().endsWith(`/${basename}`));
         const contentMatch = ((k.content ?? '') + ' ' + (k.title ?? '')).toLowerCase().includes(basename);
         if (!refMatch && !contentMatch)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@shrkcrft/inspector",
-  "version": "0.1.0-alpha.31",
+  "version": "0.1.0-alpha.32",
   "description": "SharkCraft inspector: project overview, doctor checks, AI-agent instructions.",
   "license": "MIT",
   "author": "SharkCraft contributors",
@@ -42,22 +42,22 @@
     "typecheck": "tsc --noEmit -p tsconfig.json"
   },
   "dependencies": {
-    "@shrkcrft/core": "^0.1.0-alpha.31",
-    "@shrkcrft/config": "^0.1.0-alpha.31",
-    "@shrkcrft/workspace": "^0.1.0-alpha.31",
-    "@shrkcrft/knowledge": "^0.1.0-alpha.31",
-    "@shrkcrft/rules": "^0.1.0-alpha.31",
-    "@shrkcrft/paths": "^0.1.0-alpha.31",
-    "@shrkcrft/templates": "^0.1.0-alpha.31",
-    "@shrkcrft/context": "^0.1.0-alpha.31",
-    "@shrkcrft/pipelines": "^0.1.0-alpha.31",
-    "@shrkcrft/packs": "^0.1.0-alpha.31",
-    "@shrkcrft/presets": "^0.1.0-alpha.31",
-    "@shrkcrft/boundaries": "^0.1.0-alpha.31",
-    "@shrkcrft/generator": "^0.1.0-alpha.31",
-    "@shrkcrft/importer": "^0.1.0-alpha.31",
-    "@shrkcrft/plugin-api": "^0.1.0-alpha.31",
-    "@shrkcrft/dashboard-api": "^0.1.0-alpha.31",
+    "@shrkcrft/core": "^0.1.0-alpha.32",
+    "@shrkcrft/config": "^0.1.0-alpha.32",
+    "@shrkcrft/workspace": "^0.1.0-alpha.32",
+    "@shrkcrft/knowledge": "^0.1.0-alpha.32",
+    "@shrkcrft/rules": "^0.1.0-alpha.32",
+    "@shrkcrft/paths": "^0.1.0-alpha.32",
+    "@shrkcrft/templates": "^0.1.0-alpha.32",
+    "@shrkcrft/context": "^0.1.0-alpha.32",
+    "@shrkcrft/pipelines": "^0.1.0-alpha.32",
+    "@shrkcrft/packs": "^0.1.0-alpha.32",
+    "@shrkcrft/presets": "^0.1.0-alpha.32",
+    "@shrkcrft/boundaries": "^0.1.0-alpha.32",
+    "@shrkcrft/generator": "^0.1.0-alpha.32",
+    "@shrkcrft/importer": "^0.1.0-alpha.32",
+    "@shrkcrft/plugin-api": "^0.1.0-alpha.32",
+    "@shrkcrft/dashboard-api": "^0.1.0-alpha.32",
     "typescript": "^5.6.0"
   },
   "publishConfig": {