@skill-map/cli 0.52.0 → 0.53.0

package/migrations/001_initial.sql

@@ -431,6 +431,42 @@ CREATE TABLE scan_contributions (
 CREATE INDEX ix_scan_contributions_node_path ON scan_contributions(node_path);
 CREATE INDEX ix_scan_contributions_plugin_id ON scan_contributions(plugin_id);
 
+-- scan_contribution_errors: per-scan record of view contributions the
+-- orchestrator REJECTED at emit time (the "off-shape visible" follow-up).
+-- Each row is one `ctx.emitContribution(...)` call that did NOT survive
+-- validation, with the same diagnostic the ephemeral `extension.error`
+-- event (kind `contribution-rejected`) already carried. Two rejection
+-- shapes land here:
+--   1. `undeclared-contribution-ref` — the extension passed a `ref` that
+--      is not one of its declared `viewContributions` objects (a spread
+--      copy / inline literal). `contribution_id` and `slot` are NULL.
+--   2. AJV failure — the payload failed the slot's payload schema. `reason`
+--      carries the AJV error string; `contribution_id` and `slot` name the
+--      target contribution / slot.
+--
+-- Belongs to the `scan_*` family. Plain REPLACE-ALL per scan (delete all,
+-- then insert), the same posture as `scan_issues` — NOT the orphan/catalog/
+-- per-tuple sweep `scan_contributions` uses. A rejected emission is a
+-- transient scan finding, not durable state: every scan re-derives the
+-- full set, so there is no cached-node row to preserve and no compound PK
+-- (the nullable `contribution_id` / `slot` columns rule a compound PK out
+-- anyway). Indexes on `plugin_id` (the `sm plugins doctor` group-by) and
+-- `node_path` (the rename heuristic + per-node lookups).
+CREATE TABLE scan_contribution_errors (
+  plugin_id TEXT NOT NULL,
+  extension_id TEXT NOT NULL,
+  node_path TEXT NOT NULL,
+  reason TEXT NOT NULL,
+  message TEXT NOT NULL,
+  -- NULL for the `undeclared-contribution-ref` shape (the orchestrator
+  -- never resolved a contribution id / slot for the rejected ref).
+  contribution_id TEXT,
+  slot TEXT,
+  emitted_at INTEGER NOT NULL
+);
+CREATE INDEX ix_scan_contribution_errors_plugin_id ON scan_contribution_errors(plugin_id);
+CREATE INDEX ix_scan_contribution_errors_node_path ON scan_contribution_errors(node_path);
+
 -- scan_node_tags: tag system. One row per (node_path, tag) pair.
 -- Projected at persist time from `sidecar.annotations.tags`. Tags are
 -- a skill-map concept (no vendor carries `tags` in frontmatter), so the

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@skill-map/cli",
-  "version": "0.52.0",
+  "version": "0.53.0",
   "description": "skill-map reference implementation — kernel + CLI + adapters.",
   "license": "MIT",
   "type": "module",
@@ -61,7 +61,7 @@
     "smol-toml": "1.6.1",
     "typanion": "3.14.0",
     "ws": "8.21.0",
-    "@skill-map/spec": "0.46.0"
+    "@skill-map/spec": "0.47.0"
   },
   "devDependencies": {
     "@eslint/js": "10.0.1",
@@ -73,6 +73,7 @@
     "c8": "11.0.0",
     "eslint": "10.2.1",
     "eslint-plugin-import-x": "4.16.2",
+    "json-schema-to-typescript": "15.0.4",
     "tsup": "8.5.1",
     "tsx": "4.22.3",
     "typescript": "5.9.3",
@@ -98,15 +99,16 @@
     "prebuild": "pnpm build-built-ins",
     "validate": "pnpm validate:compile && pnpm validate:test",
     "validate:compile": "pnpm typecheck && pnpm lint && pnpm build && pnpm built-ins:check && pnpm view-catalog:check",
-    "validate:test": "pnpm test:ci",
+    "validate:test": "pnpm test:ci && pnpm conformance",
+    "conformance": "SKILL_MAP_TELEMETRY=0 SM_NO_UPDATE_CHECK=1 node --import tsx cli/entry.ts conformance run",
     "pretest": "tsup",
     "pretest:coverage": "tsup",
     "pretest:coverage:html": "tsup",
-    "test": "tsc --noEmit && SKILL_MAP_TELEMETRY=0 node --import tsx --test --test-reporter=./scripts/test-reporter.js --test-reporter-destination=stdout '__tests__/**/*.spec.ts' 'kernel/**/__tests__/**/*.spec.ts' 'cli/**/__tests__/**/*.spec.ts' 'server/**/__tests__/**/*.spec.ts' 'plugins/**/__tests__/**/*.spec.ts' 'core/**/__tests__/**/*.spec.ts' 'conformance/**/__tests__/**/*.spec.ts'",
-    "test:ci": "FORCE_COLOR=1 SKILL_MAP_TELEMETRY=0 node --import tsx --test --test-reporter=./scripts/test-reporter.js --test-reporter-destination=stdout '__tests__/**/*.spec.ts' 'kernel/**/__tests__/**/*.spec.ts' 'cli/**/__tests__/**/*.spec.ts' 'server/**/__tests__/**/*.spec.ts' 'plugins/**/__tests__/**/*.spec.ts' 'core/**/__tests__/**/*.spec.ts' 'conformance/**/__tests__/**/*.spec.ts'",
-    "test:spec": "SKILL_MAP_TELEMETRY=0 node --import tsx --test --test-reporter=spec '__tests__/**/*.spec.ts' 'kernel/**/__tests__/**/*.spec.ts' 'cli/**/__tests__/**/*.spec.ts' 'server/**/__tests__/**/*.spec.ts' 'plugins/**/__tests__/**/*.spec.ts' 'core/**/__tests__/**/*.spec.ts' 'conformance/**/__tests__/**/*.spec.ts'",
-    "test:coverage": "tsc --noEmit && SKILL_MAP_TELEMETRY=0 SKILL_MAP_SKIP_BENCHMARK=1 node --experimental-default-config-file --import tsx --test --experimental-test-coverage '__tests__/**/*.spec.ts' 'kernel/**/__tests__/**/*.spec.ts' 'cli/**/__tests__/**/*.spec.ts' 'server/**/__tests__/**/*.spec.ts' 'plugins/**/__tests__/**/*.spec.ts' 'core/**/__tests__/**/*.spec.ts' 'conformance/**/__tests__/**/*.spec.ts'",
-    "test:coverage:html": "tsc --noEmit && SKILL_MAP_TELEMETRY=0 SKILL_MAP_SKIP_BENCHMARK=1 c8 node --import tsx --test '__tests__/**/*.spec.ts' 'kernel/**/__tests__/**/*.spec.ts' 'cli/**/__tests__/**/*.spec.ts' 'server/**/__tests__/**/*.spec.ts' 'plugins/**/__tests__/**/*.spec.ts' 'core/**/__tests__/**/*.spec.ts' 'conformance/**/__tests__/**/*.spec.ts'",
+    "test": "tsc --noEmit && SKILL_MAP_TELEMETRY=0 SM_NO_UPDATE_CHECK=1 node --import tsx --test --test-reporter=./scripts/test-reporter.js --test-reporter-destination=stdout '__tests__/**/*.spec.ts' 'kernel/**/__tests__/**/*.spec.ts' 'cli/**/__tests__/**/*.spec.ts' 'server/**/__tests__/**/*.spec.ts' 'plugins/**/__tests__/**/*.spec.ts' 'core/**/__tests__/**/*.spec.ts' 'conformance/**/__tests__/**/*.spec.ts'",
+    "test:ci": "FORCE_COLOR=1 SKILL_MAP_TELEMETRY=0 SM_NO_UPDATE_CHECK=1 node --import tsx --test --test-reporter=./scripts/test-reporter.js --test-reporter-destination=stdout '__tests__/**/*.spec.ts' 'kernel/**/__tests__/**/*.spec.ts' 'cli/**/__tests__/**/*.spec.ts' 'server/**/__tests__/**/*.spec.ts' 'plugins/**/__tests__/**/*.spec.ts' 'core/**/__tests__/**/*.spec.ts' 'conformance/**/__tests__/**/*.spec.ts'",
+    "test:spec": "SKILL_MAP_TELEMETRY=0 SM_NO_UPDATE_CHECK=1 node --import tsx --test --test-reporter=spec '__tests__/**/*.spec.ts' 'kernel/**/__tests__/**/*.spec.ts' 'cli/**/__tests__/**/*.spec.ts' 'server/**/__tests__/**/*.spec.ts' 'plugins/**/__tests__/**/*.spec.ts' 'core/**/__tests__/**/*.spec.ts' 'conformance/**/__tests__/**/*.spec.ts'",
+    "test:coverage": "tsc --noEmit && SKILL_MAP_TELEMETRY=0 SM_NO_UPDATE_CHECK=1 SKILL_MAP_SKIP_BENCHMARK=1 node --experimental-default-config-file --import tsx --test --experimental-test-coverage '__tests__/**/*.spec.ts' 'kernel/**/__tests__/**/*.spec.ts' 'cli/**/__tests__/**/*.spec.ts' 'server/**/__tests__/**/*.spec.ts' 'plugins/**/__tests__/**/*.spec.ts' 'core/**/__tests__/**/*.spec.ts' 'conformance/**/__tests__/**/*.spec.ts'",
+    "test:coverage:html": "tsc --noEmit && SKILL_MAP_TELEMETRY=0 SM_NO_UPDATE_CHECK=1 SKILL_MAP_SKIP_BENCHMARK=1 c8 node --import tsx --test '__tests__/**/*.spec.ts' 'kernel/**/__tests__/**/*.spec.ts' 'cli/**/__tests__/**/*.spec.ts' 'server/**/__tests__/**/*.spec.ts' 'plugins/**/__tests__/**/*.spec.ts' 'core/**/__tests__/**/*.spec.ts' 'conformance/**/__tests__/**/*.spec.ts'",
     "clean": "rm -rf dist coverage"
   }
 }