@skill-map/cli 0.39.0 → 0.40.0

package/dist/kernel/index.d.ts

@@ -347,6 +347,14 @@ interface IAnnotationContribution {
  * Runtime extension descriptor as seen by the registry / orchestrator.
  * Authors writing a manifest on disk do NOT declare `id`, `kind`, or
  * `pluginId`; the loader injects all three from the filesystem layout.
+ *
+ * `version` is the runtime invariant: every loaded extension carries a
+ * string. External plugins MUST declare it in their manifest (enforced
+ * by AJV via `spec/schemas/extensions/base.schema.json#/required`).
+ * Built-in extensions inside this repo OMIT `version` from the manifest
+ * file (see `IBuiltInManifest` below) because the codegen at
+ * `scripts/generate-built-ins.js` stamps the CLI version onto every
+ * built-in at build time, alongside the `pluginId` stamp.
  */
 interface IExtensionBase {
     /**
@@ -442,7 +450,7 @@ interface IExtensionBase {
  *   5. **Internal type aliases**, anything declared as `type` (string-
  *      literal unions, function types, mapped/derived types) that lives
  *      only in TS: `TLogLevel`, `TLogMethodLevel`, `TProgressListener`,
- *      `TLogFormatter`, `TActionWrite`, `TExecutionMode`, `TGranularity`,
+ *      `TLogFormatter`, `TActionWrite`, `TExecutionMode`,
  *      `THookFilter`, `THookTrigger`, `TNodeChangeReason`,
  *      `TPluginLoadStatus`, `TPluginStorage`, `TWatchEventKind`. **`T`
  *      prefix.** Use this bucket when `interface` is the wrong shape
@@ -1059,24 +1067,6 @@ type TPluginStorage = {
     migrations: string[];
     schemas?: Record<string, string>;
 };
-/**
- * Toggle granularity for a plugin / built-in bundle.
- *
- * - `'bundle'` , the plugin id is the only enable/disable key. The whole
- *                 bundle of extensions follows the toggle; the user cannot
- *                 enable some extensions of the bundle and disable others.
- *                 Default for plugins (and for the built-in `claude`
- *                 bundle, where the provider and its kind-aware extractors
- *                 form a coherent provider).
- * - `'extension'`, each extension is independently toggle-able under its
- *                   qualified id `<plugin-id>/<extension-id>`. Used for
- *                   the built-in `core` bundle (every kernel built-in
- *                   rule / formatter is removable per spec
- *                   "no extension is privileged"). Plugin authors opt in
- *                   only when the plugin ships several orthogonal
- *                   capabilities a user might reasonably want piecemeal.
- */
-type TGranularity = 'bundle' | 'extension';
 /**
  * Raw `plugin.json` shape after successful AJV validation.
  *
@@ -1099,13 +1089,6 @@ interface IPluginManifest {
     /** Required short description shown in `sm plugins list` and the UI. */
     description: string;
     storage?: TPluginStorage;
-    /**
-     * Toggle granularity for this plugin. Optional with default
-     * `'extension'` since the structure-as-truth refactor (more
-     * permissive default: each extension is independently toggleable
-     * unless the author opts into bundle-level coupling).
-     */
-    granularity?: TGranularity;
     author?: string;
     license?: string;
     homepage?: string;
@@ -1186,12 +1169,6 @@ interface IDiscoveredPlugin {
     manifest?: IPluginManifest;
     /** Only present when status === 'enabled'. */
     extensions?: ILoadedExtension[];
-    /**
-     * Resolved granularity for this plugin. Always populated from
-     * `manifest.granularity` (default `'bundle'`) when the manifest parsed;
-     * absent for `invalid-manifest` paths where the manifest never validated.
-     */
-    granularity?: TGranularity;
     /**
      * Runtime-only, never persisted, never spec-modeled.
      *
@@ -2472,6 +2449,19 @@ interface IAnalyzerContext {
      * orphan files exist. Treat as read-only.
      */
     orphanJobFiles?: readonly string[];
+    /**
+     * Issues emitted by analyzers that already ran in the current pass.
+     * Lets a late-phase analyzer (`core/issue-counter`) compute
+     * cross-analyzer aggregates (per-node severity totals) without
+     * scanning the persisted DB. The orchestrator threads the live
+     * accumulator on every call so any analyzer can opt-in; only the
+     * aggregator reads it today, the rest treat it as inert.
+     *
+     * Treat as read-only, the accumulator is shared with downstream
+     * analyzers and a mutation here would corrupt their view of the
+     * scan. Absent (or empty) on legacy callers that never wired it.
+     */
+    accumulatedIssues?: readonly Issue[];
     /**
      * Set of absolute file paths the operator has opted into for
      * link-validation purposes via `scan.referencePaths`. The driving
@@ -2563,6 +2553,26 @@ interface IAnalyzer extends IExtensionBase {
      * "Resolve this issue" affordances.
      */
     precondition?: IExtensionPrecondition;
+    /**
+     * Execution phase. Drives the order the orchestrator schedules
+     * analyzers in:
+     *
+     *   - `'detect'` (default), the main pass. Walks nodes / links and
+     *     emits its own findings. Most analyzers live here.
+     *   - `'aggregate'`, runs strictly AFTER every `detect`-phase
+     *     analyzer has finished. The orchestrator passes the full
+     *     issue accumulator on `ctx.accumulatedIssues`, so an
+     *     aggregator can compute cross-analyzer summaries (per-node
+     *     severity totals, etc.) without re-reading the persisted DB.
+     *     Aggregators emit contributions; emitting issues is allowed
+     *     but uncommon.
+     *
+     * Two-phase scheduling is the clean alternative to ordering
+     * analyzers by hand in the built-ins registry: filesystem-sorted
+     * generators can keep their alphabetical output, the orchestrator
+     * applies the phase sort at run-time.
+     */
+    phase?: 'detect' | 'aggregate';
     evaluate(ctx: IAnalyzerContext): Issue[] | Promise<Issue[]>;
 }
 
@@ -3747,7 +3757,7 @@ interface IUpdateCheckCache {
  * any drift a compile error).
  *
  * Domain types (`IPluginManifest`, `ILoadedExtension`, `IDiscoveredPlugin`,
- * `TPluginStorage`, `TPluginLoadStatus`, `TGranularity`) live in
+ * `TPluginStorage`, `TPluginLoadStatus`) live in
  * `kernel/types/plugin.ts` because they are spec-mirroring DTOs, not
  * port-shape types. The port re-exports them for callers that import
  * from the ports barrel.
@@ -4307,4 +4317,4 @@ interface Kernel {
 }
 declare function createKernel(): Kernel;
 
-export { type Confidence, DuplicateExtensionError, EXTENSION_KINDS, type ExecutionFailureReason, type ExecutionKind, type ExecutionRecord, type ExecutionRunner, type ExecutionStatus, ExportQueryError, type ExtensionKind, type FilesystemPort, HOOK_TRIGGERS, type HistoryStats, type HistoryStatsErrorRates, type HistoryStatsExecutionsPerPeriod, type HistoryStatsPerActionRate, type HistoryStatsTokensPerAction, type HistoryStatsTopNode, type HistoryStatsTotals, type IAction, type IActionContext, type IActionPrecondition, type IActionResult, type IAnalyzer, type IAnalyzerContext, type IAnnotationContribution, type ICreateFsWatcherOptions, type IDedicatedStorePersist, type IDedicatedStoreWrapper, type IDiscoveredPlugin, type IEnrichmentRecord, type IExportQuery, type IExportSubset, type IExtension, type IExtensionBase, type IExternalRef, type IExtractor, type IExtractorCallbacks, type IExtractorContext, type IExtractorRunRecord, type IFormatter, type IFormatterContext, type IFsWatcher, type IHook, type IHookContext, type IHookDispatcher, type IIssueRow, type IKvStorePersist, type IKvStoreWrapper, type ILoadedExtension, type INodeBundle, type INodeChange, type INodeCounts, type INodeFilter, type IPersistOptions, type IPersistedEnrichment, type IPluginManifest, type IPluginStorageSchema, type IProvider, type IRawNode, type IRegisteredAnnotationKey, type IRegisteredViewContribution, type IRunOptions, type IRunResult, type IScanDelta, type ITransactionalStorage, type IViewContribution, type IWalkOptions, type IWatchBatch, type IWatchEvent, InMemoryProgressEmitter, type Issue, type IssueFix, KV_SCHEMA_KEY, type Kernel, LOG_LEVELS, type Link, type LinkKind, type LinkLocation, type LinkOccurrence, type LinkTrigger, type LogRecord, type LoggerPort, type Node, type NodeKind, type NodeStat, type PluginLoaderPort, type ProgressEmitterPort, type ProgressEvent, Registry, type RenameOp, type RunScanOptions, type RunnerPort, type ScanResult, type ScanScannedBy, type ScanStats, type Severity, SilentLogger, type Stability, type StoragePort, type TActionWrite, type TExecutionMode, type TGranularity, type THookFilter, type THookTrigger, type TInputTypeName, type TLogLevel, type TLogMethodLevel, type TNodeChangeReason, type TPluginLoadStatus, type TPluginStorage, type TPluginStore, type TProgressListener, type TSettingDeclaration, type TSettingValue, type TSeverity, type TSlotName, type TWatchEventKind, type TripleSplit, applyExportQuery, computeScanDelta, configureLogger, createChokidarWatcher, createKernel, detectRenamesAndOrphans, getActiveLogger, isEmptyDelta, isLogLevel, log, logLevelRank, makeDedicatedStoreWrapper, makeEvent, makeHookDispatcher, makeKvStoreWrapper, makePluginStore, mergeNodeWithEnrichments, parseExportQuery, parseLogLevel, qualifiedExtensionId, resetLogger, runExtractorsForNode, runScan, runScanWithRenames };
+export { type Confidence, DuplicateExtensionError, EXTENSION_KINDS, type ExecutionFailureReason, type ExecutionKind, type ExecutionRecord, type ExecutionRunner, type ExecutionStatus, ExportQueryError, type ExtensionKind, type FilesystemPort, HOOK_TRIGGERS, type HistoryStats, type HistoryStatsErrorRates, type HistoryStatsExecutionsPerPeriod, type HistoryStatsPerActionRate, type HistoryStatsTokensPerAction, type HistoryStatsTopNode, type HistoryStatsTotals, type IAction, type IActionContext, type IActionPrecondition, type IActionResult, type IAnalyzer, type IAnalyzerContext, type IAnnotationContribution, type ICreateFsWatcherOptions, type IDedicatedStorePersist, type IDedicatedStoreWrapper, type IDiscoveredPlugin, type IEnrichmentRecord, type IExportQuery, type IExportSubset, type IExtension, type IExtensionBase, type IExternalRef, type IExtractor, type IExtractorCallbacks, type IExtractorContext, type IExtractorRunRecord, type IFormatter, type IFormatterContext, type IFsWatcher, type IHook, type IHookContext, type IHookDispatcher, type IIssueRow, type IKvStorePersist, type IKvStoreWrapper, type ILoadedExtension, type INodeBundle, type INodeChange, type INodeCounts, type INodeFilter, type IPersistOptions, type IPersistedEnrichment, type IPluginManifest, type IPluginStorageSchema, type IProvider, type IRawNode, type IRegisteredAnnotationKey, type IRegisteredViewContribution, type IRunOptions, type IRunResult, type IScanDelta, type ITransactionalStorage, type IViewContribution, type IWalkOptions, type IWatchBatch, type IWatchEvent, InMemoryProgressEmitter, type Issue, type IssueFix, KV_SCHEMA_KEY, type Kernel, LOG_LEVELS, type Link, type LinkKind, type LinkLocation, type LinkOccurrence, type LinkTrigger, type LogRecord, type LoggerPort, type Node, type NodeKind, type NodeStat, type PluginLoaderPort, type ProgressEmitterPort, type ProgressEvent, Registry, type RenameOp, type RunScanOptions, type RunnerPort, type ScanResult, type ScanScannedBy, type ScanStats, type Severity, SilentLogger, type Stability, type StoragePort, type TActionWrite, type TExecutionMode, type THookFilter, type THookTrigger, type TInputTypeName, type TLogLevel, type TLogMethodLevel, type TNodeChangeReason, type TPluginLoadStatus, type TPluginStorage, type TPluginStore, type TProgressListener, type TSettingDeclaration, type TSettingValue, type TSeverity, type TSlotName, type TWatchEventKind, type TripleSplit, applyExportQuery, computeScanDelta, configureLogger, createChokidarWatcher, createKernel, detectRenamesAndOrphans, getActiveLogger, isEmptyDelta, isLogLevel, log, logLevelRank, makeDedicatedStoreWrapper, makeEvent, makeHookDispatcher, makeKvStoreWrapper, makePluginStore, mergeNodeWithEnrichments, parseExportQuery, parseLogLevel, qualifiedExtensionId, resetLogger, runExtractorsForNode, runScan, runScanWithRenames };

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.39.0",
+  version: "0.40.0",
   description: "skill-map reference implementation \u2014 kernel + CLI + adapters.",
   license: "MIT",
   type: "module",
@@ -1461,8 +1461,8 @@ function isExternalUrlLink(link) {
 }
 
 // kernel/orchestrator/analyzers.ts
-async function runAnalyzers(analyzers, nodes, internalLinks, orphanSidecars, sidecarRoots, annotationContributions, viewContributions, orphanJobFiles, referenceablePaths, cwd, registeredActionIds, emitter, hookDispatcher, reservedNodePaths, signals) {
-  const issues = [];
+async function runAnalyzers(analyzers, nodes, internalLinks, orphanSidecars, sidecarRoots, annotationContributions, viewContributions, orphanJobFiles, referenceablePaths, cwd, registeredActionIds, emitter, hookDispatcher, reservedNodePaths, signals, seedIssues = []) {
+  const issues = [...seedIssues];
   const contributions = [];
   const validators = loadSchemaValidators();
   void registeredActionIds;
@@ -1470,7 +1470,8 @@ async function runAnalyzers(analyzers, nodes, internalLinks, orphanSidecars, sid
     relativePath: o.relativePath,
     expectedMdPath: o.expectedMdPath
   }));
-  for (const analyzer of analyzers) {
+  const scheduled = orderAnalyzersByPhase(analyzers);
+  for (const analyzer of scheduled) {
     const qualifiedId = qualifiedExtensionId(analyzer.pluginId, analyzer.id);
     const declaredContributions = readDeclaredContributions(analyzer);
     const emitContribution = (nodePath, contributionId, payload) => {
@@ -1523,6 +1524,11 @@ async function runAnalyzers(analyzers, nodes, internalLinks, orphanSidecars, sid
       annotationContributions,
       viewContributions,
       orphanJobFiles,
+      // `issues` is the live accumulator, mutated by `issues.push(...)`
+      // below as each analyzer's emission lands. Late-phase analyzers
+      // (`core/issue-counter`) read it to compute cross-analyzer
+      // aggregates. Treat as read-only on the analyzer side.
+      accumulatedIssues: issues,
       ...referenceablePaths ? { referenceablePaths } : {},
       ...cwd ? { cwd } : {},
       ...reservedNodePaths ? { reservedNodePaths } : {},
@@ -1539,6 +1545,12 @@ async function runAnalyzers(analyzers, nodes, internalLinks, orphanSidecars, sid
   }
   return { issues, contributions };
 }
+function orderAnalyzersByPhase(analyzers) {
+  return analyzers.slice().sort((a, b) => phaseRank(a) - phaseRank(b));
+}
+function phaseRank(a) {
+  return a.phase === "aggregate" ? 1 : 0;
+}
 function validateIssue(analyzer, issue, emitter) {
   const severity = issue.severity;
   if (severity !== "error" && severity !== "warn" && severity !== "info") {
@@ -1835,10 +1847,9 @@ function resolveByName(link, indexes, ctx) {
   const winner = candidates.find((c) => allowedKinds.includes(c.kind));
   return winner ? winner.path : "none";
 }
-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 lookupAllowedKinds(link, _indexes, ctx) {
+  if (ctx.activeProvider === null) return void 0;
+  return ctx.providerResolution.get(ctx.activeProvider)?.[link.kind];
 }
 function stripTriggerSigil(normalized) {
   if (!normalized) return null;
@@ -3156,7 +3167,7 @@ async function runScanInternal(_kernel, options) {
     else walked.internalLinks.push(link);
   }
   walked.signals = resolved.resolvedSignals;
-  const postWalkCtx = buildPostWalkTransformCtx(exts.providers, walked.nodes);
+  const postWalkCtx = buildPostWalkTransformCtx(exts.providers, walked.nodes, activeProviderId);
   walked.internalLinks = applyPostWalkTransforms(walked.internalLinks, walked.nodes, postWalkCtx);
   recomputeLinkCounts(walked.nodes, walked.internalLinks);
   recomputeExternalRefsCount(walked.nodes, walked.externalLinks, walked.cachedPaths);
@@ -3179,11 +3190,15 @@ async function runScanInternal(_kernel, options) {
     emitter,
     hookDispatcher,
     postWalkCtx.reservedNodePaths,
-    walked.signals
+    walked.signals,
+    // Seed the accumulator with orchestrator-emitted frontmatter
+    // issues so the aggregate phase (`core/issue-counter`) counts
+    // them on the per-node chip. The seeds are echoed back on
+    // `analyzerResult.issues`, no explicit push is needed below.
+    walked.frontmatterIssues
   );
   mergeAnalyzerEmissions(walked, analyzerResult, exts.analyzers);
   const issues = analyzerResult.issues;
-  for (const issue of walked.frontmatterIssues) issues.push(issue);
   const silenced = options.ignoreFilter ? (path) => options.ignoreFilter.ignores(path) : void 0;
   const renameOps = prior ? detectRenamesAndOrphans(prior, walked.nodes, issues, silenced) : [];
   const stats = buildScanStats(walked, issues, start);
@@ -3192,14 +3207,14 @@ async function runScanInternal(_kernel, options) {
   await hookDispatcher.dispatch("scan.completed", scanCompletedEvent);
   return buildScanReturn(walked, issues, renameOps, stats, options, setup);
 }
-function buildPostWalkTransformCtx(providers, nodes) {
+function buildPostWalkTransformCtx(providers, nodes, activeProvider) {
   const { kindRegistry, providerResolution, reservedNamesByProviderKind } = buildProviderIndexes(providers);
   const reservedNodePaths = buildReservedNodePaths(
     nodes,
     kindRegistry,
     reservedNamesByProviderKind
   );
-  return { kindRegistry, providerResolution, reservedNodePaths };
+  return { kindRegistry, providerResolution, activeProvider, reservedNodePaths };
 }
 function buildProviderIndexes(providers) {
   const kindRegistry = /* @__PURE__ */ new Map();