@skill-map/cli 0.34.0 → 0.35.0

package/dist/kernel/index.d.ts

@@ -1742,7 +1742,42 @@ interface IProviderKind {
      * invent visuals for a Provider-declared kind.
      */
     ui: IProviderKindUi;
+    /**
+     * Priority-ordered list of identifier sources the post-walk resolver
+     * uses to derive this kind's canonical name(s). Each entry contributes
+     * one normalized name to the name index built by
+     * `liftResolvedLinkConfidence`; multiple sources accumulate (e.g. a
+     * skill with `name: foo` AND dirname `foo` yields one entry, a skill
+     * with `name: bar` and dirname `foo` yields two).
+     *
+     * Defaults to `[]` (no name-resolvable). Source semantics:
+     *
+     *   - `'frontmatter.name'`, read `node.frontmatter.name`. Required-name
+     *     kinds (`agent`, `command`, `skill` per their schemas) typically
+     *     declare this first.
+     *   - `'filename-basename'`, `basename(path)` without the extension.
+     *     For Claude/Gemini/OpenAI agents and commands the filename IS the
+     *     invocation handle when `name:` is absent.
+     *   - `'dirname'`, `basename(dirname(path))`. Anthropic skills + Gemini
+     *     skills + agent-skills resolve to the directory between the
+     *     skills root and the SKILL.md (e.g.
+     *     `.claude/skills/foo/SKILL.md` → `foo`).
+     *
+     * Compare with `IProvider.resolution` (which declares which target
+     * kinds resolve which link.kind): `identifiers` is a per-kind detail
+     * about WHERE the name lives; `resolution` is a per-provider strict
+     * matrix about WHICH kinds count as resolution for a given link.kind.
+     */
+    identifiers?: TIdentifierSource[];
 }
+/**
+ * Sources the post-walk confidence-lift transform consults to derive a
+ * node's canonical name. Closed set: extending it is a spec + kernel
+ * change. Order is meaningful inside `IProviderKind.identifiers`, the
+ * resolver visits sources in declaration order, but the resulting index
+ * is presence-based so multiple matches collapse.
+ */
+type TIdentifierSource = 'frontmatter.name' | 'filename-basename' | 'dirname';
 /**
  * Presentation contract for one Provider kind. The Provider declares
  * intent (label + base color, optional dark variant + emoji + icon);
@@ -1902,6 +1937,38 @@ interface IProvider extends IExtensionBase {
      * kind against `NodeKind`.
      */
     classify(path: string, frontmatter: Record<string, unknown>): string | null;
+    /**
+     * Strict resolution matrix consumed by the post-walk confidence-lift
+     * transform: maps a `link.kind` (emitted by an Extractor in this
+     * Provider's bundle, e.g. `'mentions'`, `'invokes'`) to the set of
+     * target `node.kind` values that count as a valid resolution.
+     *
+     * Used to decide whether to bump a link's confidence to 1.0 when its
+     * normalized trigger matches some node's identifier (see
+     * `IProviderKind.identifiers`). A link whose name resolves to a node
+     * whose kind is NOT in `resolution[link.kind]` stays at its
+     * extractor-emitted confidence, the name exists but does not resolve
+     * AS THIS link.kind. Example: in `claude`, `invokes` resolves to
+     * `['command', 'skill']`, so a `/foo` slash matching an `agent` named
+     * `foo` does not bump (agents are mentioned with `@`, not invoked
+     * with `/`).
+     *
+     * The lookup uses the Provider id attached to the link's SOURCE node
+     * (i.e. who wrote the trigger). A link sourced from a markdown body
+     * outside any Provider's territory falls under `core/markdown`'s
+     * empty rules, no bump applies via the name path (the path-match rule
+     * still does).
+     *
+     * Default `undefined` ≡ empty map ≡ no link.kind bumps under this
+     * Provider's name index. Path matches (`link.target === node.path`)
+     * are unaffected, those always bump regardless of `resolution`.
+     *
+     * Distinct from the spec's `IProvider.resolverRules` (referenced in
+     * §Resolver phase): `resolverRules` rank candidates inside the Signal
+     * IR (Phase 3+, not wired today); `resolution` is the post-walk
+     * confidence-lift contract, which runs against the merged Link graph.
+     */
+    resolution?: Record<string, string[]>;
 }
 /**
  * Declarative read config a Provider declares via `IProvider.read`.

package/dist/kernel/index.js

@@ -101,7 +101,7 @@ import cl100k_base from "js-tiktoken/ranks/cl100k_base";
 // package.json
 var package_default = {
   name: "@skill-map/cli",
-  version: "0.34.0",
+  version: "0.35.0",
   description: "skill-map reference implementation \u2014 kernel + CLI + adapters.",
   license: "MIT",
   type: "module",
@@ -1584,7 +1584,7 @@ function originatingNodeOf(link, priorNodePaths) {
 function computeCacheDecision(opts) {
   const applicableExtractors = opts.extractors.filter((ex) => {
     if (!matchesKindPrecondition(ex, opts.kind)) return false;
-    if (!matchesProviderPrecondition(ex, opts.provider, opts.activeProvider)) return false;
+    if (!matchesProviderPrecondition(ex, opts.activeProvider)) return false;
     return true;
   });
   const applicableQualifiedIds = new Set(
@@ -1608,10 +1608,9 @@ function matchesKindPrecondition(ex, kind) {
     return kindOnly === kind;
   });
 }
-function matchesProviderPrecondition(ex, nodeProvider, activeProvider) {
+function matchesProviderPrecondition(ex, activeProvider) {
   const providers = ex.precondition?.provider;
   if (!providers || providers.length === 0) return true;
-  if (!providers.includes(nodeProvider)) return false;
   if (activeProvider === null) return false;
   return providers.includes(activeProvider);
 }
@@ -1714,6 +1713,9 @@ function classifyLinkSource(source, shortIdToQualified, cachedQualifiedIds, appl
   return "obsolete";
 }
 
+// kernel/orchestrator/lift-resolved-link-confidence.ts
+import { posix as pathPosix } from "path";
+
 // kernel/trigger-normalize.ts
 function normalizeTrigger(source) {
   let out = source.normalize("NFD");
@@ -1724,37 +1726,92 @@ function normalizeTrigger(source) {
   return out.trim();
 }
 
-// kernel/orchestrator/lift-mention-confidence.ts
-function liftMentionConfidence(links, nodes) {
-  if (!links.some((l) => l.kind === "mentions")) return;
-  const byPath2 = /* @__PURE__ */ new Set();
-  for (const node of nodes) byPath2.add(node.path);
-  const byNormalizedName = indexByNormalizedName(nodes);
+// kernel/orchestrator/lift-resolved-link-confidence.ts
+function liftResolvedLinkConfidence(links, nodes, ctx) {
+  if (!links.some((l) => l.confidence < 1)) return;
+  const indexes = buildIndexes(nodes, ctx);
   for (const link of links) {
-    if (link.kind !== "mentions") continue;
-    if (isResolved(link, byPath2, byNormalizedName)) {
+    if (link.confidence < 1 && resolves(link, indexes, ctx)) {
       link.confidence = 1;
     }
   }
 }
-function isResolved(link, byPath2, byNormalizedName) {
-  const normalized = link.trigger?.normalizedTrigger;
-  if (normalized) {
-    const withoutSigil = normalized.replace(/^[/@]/, "").trim();
-    if (byNormalizedName.has(withoutSigil)) return true;
-  }
-  if (byPath2.has(link.target)) return true;
-  return false;
-}
-function indexByNormalizedName(nodes) {
-  const out = /* @__PURE__ */ new Map();
+function buildIndexes(nodes, ctx) {
+  const byPath2 = /* @__PURE__ */ new Set();
+  const byName = /* @__PURE__ */ new Map();
+  const nodeByPath = /* @__PURE__ */ new Map();
   for (const node of nodes) {
-    const raw = node.frontmatter?.["name"];
-    const name = typeof raw === "string" ? raw : "";
-    if (!name) continue;
-    out.set(normalizeTrigger(name), true);
+    byPath2.add(node.path);
+    nodeByPath.set(node.path, node);
+    indexNode(node, ctx, byName);
+  }
+  return { byPath: byPath2, byName, nodeByPath };
+}
+function resolves(link, indexes, ctx) {
+  if (indexes.byPath.has(link.target)) return true;
+  return resolvesByName(link, indexes, ctx);
+}
+function resolvesByName(link, indexes, ctx) {
+  const stripped = stripTriggerSigil(link.trigger?.normalizedTrigger);
+  if (stripped === null) return false;
+  const candidates = indexes.byName.get(stripped);
+  if (!candidates?.length) return false;
+  const allowedKinds = lookupAllowedKinds(link, indexes, ctx);
+  if (!allowedKinds?.length) return false;
+  return candidates.some((c) => allowedKinds.includes(c.kind));
+}
+function lookupAllowedKinds(link, indexes, ctx) {
+  const sourceNode = indexes.nodeByPath.get(link.source);
+  if (!sourceNode) return void 0;
+  return ctx.providerResolution.get(sourceNode.provider)?.[link.kind];
+}
+function stripTriggerSigil(normalized) {
+  if (!normalized) return null;
+  const trimmed = normalized.replace(/^[/@]/, "").trim();
+  return trimmed.length === 0 ? null : trimmed;
+}
+function indexNode(node, ctx, byName) {
+  const kindDescriptor = ctx.kindRegistry.get(kindKey(node));
+  const sources = kindDescriptor?.identifiers;
+  if (!sources || sources.length === 0) return;
+  for (const source of sources) {
+    const raw = deriveIdentifier(source, node);
+    if (!raw) continue;
+    const normalized = normalizeTrigger(raw);
+    if (!normalized) continue;
+    const bucket = byName.get(normalized);
+    if (bucket) {
+      bucket.push({ kind: node.kind });
+    } else {
+      byName.set(normalized, [{ kind: node.kind }]);
+    }
   }
-  return out;
+}
+function deriveIdentifier(source, node) {
+  if (source === "frontmatter.name") return readFrontmatterName(node);
+  if (source === "filename-basename") return readFilenameBasename(node);
+  return readDirname(node);
+}
+function readFrontmatterName(node) {
+  const raw = node.frontmatter?.["name"];
+  if (typeof raw !== "string") return null;
+  return raw.length > 0 ? raw : null;
+}
+function readFilenameBasename(node) {
+  const base = pathPosix.basename(node.path);
+  if (!base) return null;
+  const ext = pathPosix.extname(base);
+  const stem = ext ? base.slice(0, -ext.length) : base;
+  return stem.length > 0 ? stem : null;
+}
+function readDirname(node) {
+  const dir = pathPosix.dirname(node.path);
+  if (!dir || dir === "." || dir === "/") return null;
+  const base = pathPosix.basename(dir);
+  return base.length > 0 ? base : null;
+}
+function kindKey(node) {
+  return `${node.provider}/${node.kind}`;
 }
 
 // kernel/orchestrator/post-walk-transforms.ts
@@ -1767,17 +1824,17 @@ var POST_WALK_TRANSFORMS = [
     }
   },
   {
-    id: "lift-mention-confidence",
-    description: "Bump resolved `mentions` links to confidence 1.0 once the full node graph is known (post-merge polish).",
-    run(links, nodes) {
-      liftMentionConfidence(links, nodes);
+    id: "lift-resolved-link-confidence",
+    description: "Bump invocation links to confidence 1.0 when target / trigger resolves against the full node graph per the source Provider rules.",
+    run(links, nodes, ctx) {
+      liftResolvedLinkConfidence(links, nodes, ctx);
     }
   }
 ];
-function applyPostWalkTransforms(links, nodes, transforms = POST_WALK_TRANSFORMS) {
+function applyPostWalkTransforms(links, nodes, ctx, transforms = POST_WALK_TRANSFORMS) {
   let current = links;
   for (const transform of transforms) {
-    const next = transform.run(current, nodes);
+    const next = transform.run(current, nodes, ctx);
     if (next) current = next;
   }
   return current;
@@ -2630,7 +2687,6 @@ async function processRawNode(raw, provider, wctx, accum, claimedPaths, nextInde
   const cacheDecision = computeCacheDecision({
     extractors: wctx.opts.extractors,
     kind,
-    provider: provider.id,
     activeProvider: wctx.opts.activeProvider,
     nodePath: raw.path,
     bodyHash,
@@ -2848,7 +2904,8 @@ async function runScanInternal(_kernel, options) {
     pluginStores: options.pluginStores,
     activeProvider: resolveActiveProviderOption(options.activeProvider, options.roots)
   });
-  walked.internalLinks = applyPostWalkTransforms(walked.internalLinks, walked.nodes);
+  const postWalkCtx = buildPostWalkTransformCtx(_kernel);
+  walked.internalLinks = applyPostWalkTransforms(walked.internalLinks, walked.nodes, postWalkCtx);
   recomputeLinkCounts(walked.nodes, walked.internalLinks);
   recomputeExternalRefsCount(walked.nodes, walked.externalLinks, walked.cachedPaths);
   await dispatchExtractorCompleted(exts.extractors, emitter, hookDispatcher);
@@ -2881,6 +2938,22 @@ async function runScanInternal(_kernel, options) {
   await hookDispatcher.dispatch("scan.completed", scanCompletedEvent);
   return buildScanReturn(walked, issues, renameOps, stats, options, setup);
 }
+function buildPostWalkTransformCtx(kernel) {
+  const kindRegistry = /* @__PURE__ */ new Map();
+  const providerResolution = /* @__PURE__ */ new Map();
+  const providers = kernel.registry.all("provider");
+  for (const provider of providers) {
+    if (provider.kinds) {
+      for (const [kindName, descriptor] of Object.entries(provider.kinds)) {
+        kindRegistry.set(`${provider.id}/${kindName}`, descriptor);
+      }
+    }
+    if (provider.resolution) {
+      providerResolution.set(provider.id, provider.resolution);
+    }
+  }
+  return { kindRegistry, providerResolution };
+}
 function buildScanSetup(options) {
   const start = Date.now();
   const emitter = options.emitter ?? new InMemoryProgressEmitter();