@skill-map/cli 0.25.0 → 0.26.0

package/dist/cli.js

@@ -748,19 +748,19 @@ var annotationsExtractor = {
   pluginId: "core",
   kind: "extractor",
   version: "1.0.0",
-  description: "Turns the `supersedes`, `requires`, `related`, `conflictsWith`, and `supersededBy` entries you write in a node's `.sm` sidecar into the arrows (edges) shown between nodes in the graph.",
+  description: "Turns the `supersedes` and `supersededBy` entries you write in a node's `.sm` sidecar into the arrows (edges) shown between nodes in the graph.",
   stability: "stable",
-  emitsLinkKinds: ["supersedes", "references"],
+  emitsLinkKinds: ["supersedes"],
   defaultConfidence: "high",
   scope: "frontmatter",
   extract(ctx) {
     const sourcePath = ctx.node.path;
     const seen = /* @__PURE__ */ new Set();
-    function emit(source, target, kind) {
-      const key = `${source} ${target} ${kind}`;
+    function emit(source, target) {
+      const key = `${source} ${target}`;
       if (seen.has(key)) return;
       seen.add(key);
-      ctx.emitLink(link(source, target, kind));
+      ctx.emitLink(link(source, target));
     }
     const ann = pickAnnotations(ctx.node);
     if (ann) processBlock(ann, sourcePath, emit);
@@ -768,20 +768,11 @@ var annotationsExtractor = {
 };
 function processBlock(block, sourcePath, emit) {
   for (const target of stringArray(block["supersedes"])) {
-    emit(sourcePath, target, "supersedes");
+    emit(sourcePath, target);
   }
   const supersededBy = block["supersededBy"];
   if (typeof supersededBy === "string" && supersededBy.length > 0) {
-    emit(supersededBy, sourcePath, "supersedes");
-  }
-  for (const target of stringArray(block["requires"])) {
-    emit(sourcePath, target, "references");
-  }
-  for (const target of stringArray(block["related"])) {
-    emit(sourcePath, target, "references");
-  }
-  for (const target of stringArray(block["conflictsWith"])) {
-    emit(sourcePath, target, "references");
+    emit(supersededBy, sourcePath);
   }
 }
 function pickAnnotations(node) {
@@ -797,11 +788,11 @@ function stringArray(value) {
   if (!Array.isArray(value)) return [];
   return value.filter((v) => typeof v === "string" && v.length > 0);
 }
-function link(source, target, kind) {
+function link(source, target) {
   return {
     source,
     target,
-    kind,
+    kind: "supersedes",
     confidence: "high",
     sources: [ID]
   };
@@ -2318,7 +2309,13 @@ var VALIDATE_ALL_TEXTS = {
   /** `Node <path> failed schema validation: <errors>` */
   nodeFailure: "Node {{path}} failed schema validation: {{errors}}",
   /** `Link <source> → <target> failed schema validation: <errors>` */
-  linkFailure: "Link {{source}} \u2192 {{target}} failed schema validation: {{errors}}"
+  linkFailure: "Link {{source}} \u2192 {{target}} failed schema validation: {{errors}}",
+  /** `Node <path> is missing required frontmatter fields: <missing>` */
+  frontmatterBaseFailure: "Node {{path}} is missing required frontmatter fields: {{missing}}.",
+  /** Singular tooltip on the alert / chip when a node has exactly one validation failure. */
+  alertTooltipSingle: "Frontmatter or schema validation failed.",
+  /** Plural tooltip; `{{count}}` capped at 99 in the chip badge but the tooltip text shows the raw count. */
+  alertTooltipMany: "{{count}} schema validation issues on this node."
 };
 
 // built-in-plugins/analyzers/validate-all/index.ts
@@ -2331,15 +2328,56 @@ var validateAllAnalyzer = {
   description: "Detects and flags nodes or links violating the project schemas.",
   stability: "stable",
   mode: "deterministic",
+  viewContributions: {
+    // Corner badge on the graph card; surfaces when the node body /
+    // frontmatter fails schema validation (parse error, missing
+    // `name`/`description`, malformed YAML, etc.). Same visual
+    // chassis as `core/broken-ref`, danger severity.
+    alert: {
+      slot: "graph.node.alert",
+      icon: "fa-solid fa-triangle-exclamation",
+      emitWhenEmpty: false
+    },
+    // Footer chip that mirrors the corner alert with the actual
+    // count so the operator can scan the cards and prioritise.
+    // Outlined (vs the filled corner alert) per the broken-ref
+    // pattern: two beats of the same signal.
+    chip: {
+      slot: "card.footer.right",
+      icon: "fa-regular fa-triangle-exclamation",
+      emitWhenEmpty: false,
+      priority: 35
+    }
+  },
   evaluate(ctx) {
     const validators = loadSchemaValidators();
     const findings = [];
+    const perNode = /* @__PURE__ */ new Map();
     for (const node of ctx.nodes) {
+      const before = findings.length;
       collectNodeFindings(validators, node, findings);
+      collectFrontmatterBaseFindings(node, findings);
+      if (findings.length > before) {
+        perNode.set(node.path, (perNode.get(node.path) ?? 0) + (findings.length - before));
+      }
     }
     for (const link2 of ctx.links) {
       collectLinkFindings(validators, link2, findings);
     }
+    for (const [nodePath, count] of perNode) {
+      const tooltip = count === 1 ? VALIDATE_ALL_TEXTS.alertTooltipSingle : tx(VALIDATE_ALL_TEXTS.alertTooltipMany, { count });
+      const capped = Math.min(count, 99);
+      ctx.emitContribution(nodePath, "alert", {
+        icon: "fa-solid fa-triangle-exclamation",
+        severity: "danger",
+        tooltip
+      });
+      ctx.emitContribution(nodePath, "chip", {
+        value: capped,
+        severity: "danger",
+        tooltip
+      });
+    }
     return findings;
   }
 };
@@ -2357,6 +2395,33 @@ function collectNodeFindings(v, node, out) {
     data: { target: "node", path: node.path }
   });
 }
+function collectFrontmatterBaseFindings(node, out) {
+  if (node.provider === "markdown") return;
+  if (node.bytes.frontmatter === 0) return;
+  const fm = node.frontmatter ?? {};
+  const missing = [];
+  if (isMissingStringField(fm, "name")) missing.push("name");
+  if (isMissingStringField(fm, "description")) missing.push("description");
+  if (missing.length === 0) return;
+  out.push({
+    analyzerId: ID19,
+    // `warn` (not `error`) so the default `sm scan` exit code stays
+    // 0 even when nodes are missing frontmatter base fields. Strict
+    // mode (`sm scan --strict`) still escalates to exit 1. Matches
+    // the `frontmatter-invalid` severity policy of the orchestrator.
+    severity: "warn",
+    nodeIds: [node.path],
+    message: tx(VALIDATE_ALL_TEXTS.frontmatterBaseFailure, {
+      path: node.path,
+      missing: missing.join(", ")
+    }),
+    data: { target: "frontmatter", path: node.path, missing }
+  });
+}
+function isMissingStringField(fm, field) {
+  const v = fm[field];
+  return typeof v !== "string" || v.length === 0;
+}
 function collectLinkFindings(v, link2, out) {
   const result = v.validate("link", toLinkForSchema(link2));
   if (result.ok) return;
@@ -2401,6 +2466,54 @@ function toLinkForSchema(link2) {
   };
 }
 
+// kernel/util/trigger-resolve.ts
+function buildNameIndex(nodes) {
+  const out = /* @__PURE__ */ new Map();
+  indexByCanonicalName(nodes, out);
+  fillIndexWithPathBasename(nodes, out);
+  return out;
+}
+function indexByCanonicalName(nodes, out) {
+  for (const node of nodes) {
+    const raw = canonicalName(node);
+    if (raw === null) continue;
+    const key = normalizeTrigger(raw);
+    if (!out.has(key)) out.set(key, node.path);
+  }
+}
+function fillIndexWithPathBasename(nodes, out) {
+  for (const node of nodes) {
+    if (canonicalName(node) !== null) continue;
+    const derived = pathBasenameForLink(node.path);
+    if (derived.length === 0) continue;
+    const key = normalizeTrigger(derived);
+    if (!out.has(key)) out.set(key, node.path);
+  }
+}
+function canonicalName(node) {
+  const raw = node.frontmatter?.["name"];
+  if (typeof raw !== "string" || raw.length === 0) return null;
+  return raw;
+}
+function pathBasenameForLink(path) {
+  const segments = path.split("/").filter((s) => s.length > 0);
+  if (segments.length === 0) return path;
+  const last = segments[segments.length - 1];
+  if (last === "SKILL.md" && segments.length >= 2) {
+    return segments[segments.length - 2];
+  }
+  return last.replace(/\.md$/, "");
+}
+function resolveLinkTargetToPath(link2, nameIndex) {
+  const raw = link2.target;
+  const sigil = raw.charAt(0);
+  if (sigil !== "/" && sigil !== "@") return raw;
+  const normalizedTrigger = link2.trigger?.normalizedTrigger;
+  const normalized = typeof normalizedTrigger === "string" ? normalizedTrigger.replace(/^[/@]/, "").trim() : normalizeTrigger(raw.slice(1));
+  const resolved = nameIndex.get(normalized);
+  return resolved ?? raw;
+}
+
 // built-in-plugins/analyzers/link-counts/index.ts
 var ID20 = "link-counts";
 var linkCountsAnalyzer = {
@@ -2428,10 +2541,12 @@ var linkCountsAnalyzer = {
     }
   },
   evaluate(ctx) {
+    const nameIndex = buildNameIndex(ctx.nodes);
     const perTarget = /* @__PURE__ */ new Map();
     const perSource = /* @__PURE__ */ new Map();
     for (const link2 of ctx.links) {
-      bump(perTarget, link2.target, link2.kind);
+      const resolvedTarget = resolveLinkTargetToPath(link2, nameIndex);
+      bump(perTarget, resolvedTarget, link2.kind);
       bump(perSource, link2.source, link2.kind);
     }
     for (const node of ctx.nodes) {
@@ -5747,7 +5862,7 @@ var UPDATE_CHECK_TEXTS = {
 // package.json
 var package_default = {
   name: "@skill-map/cli",
-  version: "0.25.0",
+  version: "0.26.0",
   description: "skill-map reference implementation \u2014 kernel + CLI + adapters.",
   license: "MIT",
   type: "module",
@@ -6705,16 +6820,25 @@ var BUMP_TEXTS = {
   // --- failures -------------------------------------------------------------
   bumpFailed: "{{glyph}}  sm bump: {{message}}\n",
   storeFailedDetail: "sidecar write failed for {{path}}: {{message}}",
-  resolveAbsPathFailed: "cannot resolve absolute path for {{nodePath}}: {{message}}",
+  resolveAbsPathFailed: "cannot resolve absolute path for {{nodePath}}: {{message}}"
   // --- .sm consent gate ---------------------------------------------------
+  // The shared strings live in `consent.texts.ts` (CONSENT_TEXTS); they
+  // are used by every verb that writes a sidecar (`sm bump`,
+  // `sm sidecar refresh`, `sm sidecar annotate`) with a `{{verb}}`
+  // placeholder for the directed prefix.
+};
+
+// cli/i18n/consent.texts.ts
+var CONSENT_TEXTS = {
   /**
    * Pre-prompt context shown before the interactive `confirm()` so the
-   * operator sees what they are about to opt into. `.skill-map/settings.local.json`
-   * is gitignored, the choice is saved per-checkout, never travels via the repo.
+   * operator sees what they are about to opt into.
+   * `.skill-map/settings.local.json` is gitignored, the choice is saved
+   * per-checkout, never travels via the repo.
    */
-  consentPrompt: "skill-map needs your consent to create .sm sidecar files next to your\nsource files in this project. The choice is saved to\n.skill-map/settings.local.json (gitignored, per-checkout) so this prompt\nnever appears again. Decline to abort without persisting the rejection.\n\nAllow .sm sidecar writes in this project?",
-  consentAborted: "{{glyph}}  sm bump: aborted by user. No .sm sidecar files were written.\n",
-  consentRequiredNonTty: "{{glyph}}  sm bump: consent required to write .sm sidecar files in this project.\n   {{hint}}\n",
+  consentPrompt: "{{glyph}}  skill-map needs consent to create .sm sidecar files next to your\n   .md sources. Your choice is saved to .skill-map/settings.local.json\n   (gitignored) and this prompt will not appear again.\n\nAllow .sm sidecar writes in this project?",
+  consentAborted: "{{glyph}}  {{verb}}: aborted by user. No .sm sidecar files were written.\n",
+  consentRequiredNonTty: "{{glyph}}  {{verb}}: consent required to write .sm sidecar files in this project.\n   {{hint}}\n",
   consentRequiredNonTtyHint: "Pass --yes to grant (writes to .skill-map/settings.local.json, gitignored)."
 };
 
@@ -7112,21 +7236,25 @@ var BumpCommand = class extends SmCommand {
       const isTTY = stdin.isTTY === true;
       if (!isTTY || this.yes) {
         this.printer.error(
-          tx(BUMP_TEXTS.consentRequiredNonTty, {
+          tx(CONSENT_TEXTS.consentRequiredNonTty, {
             glyph: ansi.red("\u2715"),
-            hint: ansi.dim(BUMP_TEXTS.consentRequiredNonTtyHint)
+            verb: "sm bump",
+            hint: ansi.dim(CONSENT_TEXTS.consentRequiredNonTtyHint)
           })
         );
         return ExitCode.Error;
       }
       const ok = await confirm(
-        BUMP_TEXTS.consentPrompt,
+        tx(CONSENT_TEXTS.consentPrompt, { glyph: ansi.cyan("\u2139") }),
         { stdin, stderr },
         { defaultAnswer: "yes" }
       );
       if (!ok) {
         this.printer.info(
-          tx(BUMP_TEXTS.consentAborted, { glyph: ansi.cyan("\u2139") })
+          tx(CONSENT_TEXTS.consentAborted, {
+            glyph: ansi.cyan("\u2139"),
+            verb: "sm bump"
+          })
         );
         return ExitCode.Error;
       }
@@ -12502,6 +12630,26 @@ function validateLink(extractor, link2, emitter) {
   const confidence = link2.confidence ?? extractor.defaultConfidence;
   return { ...link2, confidence };
 }
+function dedupeLinks(links) {
+  const out = /* @__PURE__ */ new Map();
+  for (const link2 of links) {
+    const trigger = link2.trigger?.normalizedTrigger ?? "";
+    const key = `${link2.source}\0${link2.target}\0${link2.kind}\0${trigger}`;
+    const existing = out.get(key);
+    if (existing) {
+      const seen = new Set(existing.sources);
+      for (const src of link2.sources) {
+        if (!seen.has(src)) {
+          seen.add(src);
+          existing.sources = [...existing.sources, src];
+        }
+      }
+      continue;
+    }
+    out.set(key, link2);
+  }
+  return [...out.values()];
+}
 function recomputeLinkCounts(nodes, links) {
   const byPath3 = /* @__PURE__ */ new Map();
   for (const node of nodes) {
@@ -13492,6 +13640,7 @@ async function runScanInternal(_kernel, options) {
     providerFrontmatter: setup.providerFrontmatter,
     pluginStores: options.pluginStores
   });
+  walked.internalLinks = dedupeLinks(walked.internalLinks);
   recomputeLinkCounts(walked.nodes, walked.internalLinks);
   recomputeExternalRefsCount(walked.nodes, walked.externalLinks, walked.cachedPaths);
   await dispatchExtractorCompleted(exts.extractors, emitter, hookDispatcher);
@@ -22623,17 +22772,12 @@ var SIDECAR_TEXTS = {
   annotateCreated: "{{glyph}}  Created {{sidecarPath}}. Edit it, then run `sm bump {{nodePath}}` to commit the version.\n",
   /** Trailing dim tag for sidecar prune dry-run (matches the orphans pattern). */
   sidecarDryRunTag: "  (no changes made)",
-  annotateFailed: "{{glyph}}  sm sidecar annotate: {{message}}\n",
-  // --- .sm consent gate (shared across refresh + annotate) -----------------
-  /**
-   * Pre-prompt context shown before the interactive `confirm()` so the
-   * operator sees what they are about to opt into. `.skill-map/settings.local.json`
-   * is gitignored, the choice is saved per-checkout, never travels via the repo.
-   */
-  consentPrompt: "skill-map needs your consent to create .sm sidecar files next to your\nsource files in this project. The choice is saved to\n.skill-map/settings.local.json (gitignored, per-checkout) so this prompt\nnever appears again. Decline to abort without persisting the rejection.\n\nAllow .sm sidecar writes in this project?",
-  consentAborted: "{{glyph}}  sm sidecar: aborted by user. No .sm sidecar files were written.\n",
-  consentRequiredNonTty: "{{glyph}}  sm sidecar: consent required to write .sm sidecar files in this project.\n   {{hint}}\n",
-  consentRequiredNonTtyHint: "Pass --yes to grant (writes to .skill-map/settings.local.json, gitignored)."
+  annotateFailed: "{{glyph}}  sm sidecar annotate: {{message}}\n"
+  // --- .sm consent gate ---------------------------------------------------
+  // The shared strings live in `consent.texts.ts` (CONSENT_TEXTS); they
+  // are used by every verb that writes a sidecar (`sm bump`,
+  // `sm sidecar refresh`, `sm sidecar annotate`) with a `{{verb}}`
+  // placeholder for the directed prefix.
 };
 
 // cli/commands/sidecar.ts
@@ -22646,21 +22790,25 @@ async function runWithSidecarConsent(bag, ansi, dispatch) {
     if (!isTTY || bag.yes) {
       const errGlyph = ansi.red("\u2715");
       bag.printError(
-        tx(SIDECAR_TEXTS.consentRequiredNonTty, {
+        tx(CONSENT_TEXTS.consentRequiredNonTty, {
           glyph: errGlyph,
-          hint: ansi.dim(SIDECAR_TEXTS.consentRequiredNonTtyHint)
+          verb: "sm sidecar",
+          hint: ansi.dim(CONSENT_TEXTS.consentRequiredNonTtyHint)
         })
       );
       return ExitCode.Error;
     }
     const ok = await confirm(
-      SIDECAR_TEXTS.consentPrompt,
+      tx(CONSENT_TEXTS.consentPrompt, { glyph: ansi.cyan("\u2139") }),
       { stdin: bag.stdin, stderr: bag.stderr },
       { defaultAnswer: "yes" }
     );
     if (!ok) {
       bag.printInfo(
-        tx(SIDECAR_TEXTS.consentAborted, { glyph: ansi.cyan("\u2139") })
+        tx(CONSENT_TEXTS.consentAborted, {
+          glyph: ansi.cyan("\u2139"),
+          verb: "sm sidecar"
+        })
       );
       return ExitCode.Error;
     }