@pikku/inspector 0.12.37 → 0.12.39

package/CHANGELOG.md

@@ -1,3 +1,41 @@
+## 0.12.39
+
+### Patch Changes
+
+- 4f92e6f: `pikku db` schema-codegen warnings are now coded diagnostics routed through the CLI logger instead of raw `console.warn`, so they participate in the existing `--fail-on-warn` gate.
+
+  Each warning now carries a PKU code and `warn` severity: `PKU481` (JSON/JSONB column with no concrete `tsType`, degrading to `unknown`), `PKU480` (column named like a date/bool but whose DB type contradicts it), and `PKU482` (a `format` annotation ignored on a non-string column). Running `pikku db migrate --fail-on-warn` (e.g. in CI) now turns these into a hard failure, forcing the `db/annotations.ts` entry — closing the loophole where an untyped jsonb column silently degrades type-safety. Default behaviour is unchanged: the warnings still print, and only fail the build when `--fail-on-warn` is set.
+
+- daec082: Drop Node 22 support — the minimum supported runtime is now Node 24 (LTS).
+
+  Node 22 deadlocks `pikku dev` at `loadUserBootstrap` (tsx `register()` + `require(esm)` cycle handling on node 22.12+), and Node 20 is already below our floor. The `engines.node` requirement is raised to `>=24` across all packages, matching `.nvmrc` and the CI test matrix. Closes #751.
+
+- ad26273: Remove 16 dormant `ErrorCode` enum entries that were defined but never emitted anywhere in the framework. These were placeholder registrations that were never wired to a diagnostic, or codes whose emission sites were removed in later refactors (e.g. `PKU901`, `PKU431`). A whole-repo audit found zero emission sites — no user could ever see them — so they only cluttered the registry and demanded docs pages for errors that cannot occur.
+
+  Removed: `PKU300`, `PKU426`, `PKU427`, `PKU431`, `PKU488`, `PKU529`, `PKU568`, `PKU685`, `PKU715`, `PKU736`, `PKU787`, `PKU835`, `PKU836`, `PKU901`, `PKU937`, `PKU975`.
+
+  A new guard test (`error-codes-emitted.test.ts`) fails if any `ErrorCode` value has no `ErrorCode.<NAME>` or raw `PKU###` reference in the source, so dead entries can't silently accumulate again.
+
+- Updated dependencies [7b17b14]
+- Updated dependencies [daec082]
+- Updated dependencies [e0fd352]
+  - @pikku/core@0.12.58
+
+## 0.12.38
+
+### Patch Changes
+
+- 66f3dae: Move `@pikku/core` from `dependencies` to `peerDependencies` in the last packages that still declared it as a regular dependency.
+
+  `@pikku/core` holds a single `pikkuState` registry and must resolve to exactly one copy at runtime — every wiring (workflows, RPCs, queue workers, middleware) registers into the copy it imports, and the runner reads the copy it imports. 35 packages already declare core as a peer for this reason; these six were the stragglers. Because they carried a regular `@pikku/core` dependency, bumping any one of them could leave a second, older core locked in a consumer's tree, splitting the registry so wirings silently fail to resolve (surfaced as `[PKU717] Multiple @pikku/core versions installed`).
+
+  Making core a peer everywhere means the consuming app provides the one copy (the react/react-dom singleton pattern), so duplication is structurally impossible. `@pikku/core` is also kept as a devDependency in each package so it still builds/typechecks standalone.
+
+  Backward-compatible for consumers that already list `@pikku/core` directly (every template does). A consumer that only pulled core transitively now gets a loud install-time peer warning instead of a silent runtime split — strictly better.
+
+- Updated dependencies [ded4f90]
+  - @pikku/core@0.12.54
+
 ## 0.12.37
 
 ### Patch Changes

package/dist/error-codes.d.ts

@@ -14,37 +14,25 @@ export declare enum ErrorCode {
     MISSING_URI = "PKU220",
     MISSING_FUNC = "PKU236",
     INVALID_TAGS_TYPE = "PKU247",
-    INVALID_HANDLER = "PKU300",
     MISSING_TITLE = "PKU370",
     MISSING_QUEUE_NAME = "PKU384",
     MISSING_CHANNEL_NAME = "PKU400",
     CLI_CLIENTSIDE_RENDERER_HAS_SERVICES = "PKU672",
     SCENARIO_HAS_SERVICES = "PKU673",
     EXPECT_EVENTUALLY_SCENARIO_ONLY = "PKU675",
-    DYNAMIC_STEP_NAME = "PKU529",
     WORKFLOW_ORCHESTRATOR_NOT_CONFIGURED = "PKU600",
     INVALID_DSL_WORKFLOW = "PKU641",
-    CONFIG_TYPE_NOT_FOUND = "PKU426",
-    CONFIG_TYPE_UNDEFINED = "PKU427",
-    SCHEMA_NO_ROOT = "PKU431",
+    DB_COLUMN_NAME_TYPE_CONTRADICTION = "PKU480",
+    DB_JSON_COLUMN_UNTYPED = "PKU481",
+    DB_FORMAT_ON_NON_STRING = "PKU482",
     SCHEMA_GENERATION_ERROR = "PKU456",
-    SCHEMA_LOAD_ERROR = "PKU488",
     INLINE_SCHEMA = "PKU489",
     FUNCTION_METADATA_NOT_FOUND = "PKU559",
-    HANDLER_NOT_RESOLVED = "PKU568",
     DUPLICATE_AUTH_DEFINITION = "PKU581",
     AUTH_NOT_EXPORTED = "PKU582",
     ROUTE_PARAM_MISMATCH = "PKU571",
     ROUTE_QUERY_MISMATCH = "PKU572",
     AUTH_DISABLED_REQUIRES_SESSIONLESS = "PKU573",
-    MIDDLEWARE_HANDLER_INVALID = "PKU685",
-    MIDDLEWARE_TAG_INVALID = "PKU715",
-    MIDDLEWARE_EMPTY_ARRAY = "PKU736",
-    MIDDLEWARE_PATTERN_INVALID = "PKU787",
-    PERMISSION_HANDLER_INVALID = "PKU835",
-    PERMISSION_TAG_INVALID = "PKU836",
-    PERMISSION_EMPTY_ARRAY = "PKU937",
-    PERMISSION_PATTERN_INVALID = "PKU975",
     DUPLICATE_FUNCTION_VERSION = "PKU850",
     DUPLICATE_FUNCTION_NAME = "PKU851",
     MANIFEST_MISSING = "PKU860",
@@ -59,7 +47,6 @@ export declare enum ErrorCode {
     SERVICES_NOT_DESTRUCTURED = "PKU410",
     WIRES_NOT_DESTRUCTURED = "PKU411",
     DUPLICATE_CORE_VERSION = "PKU717",
-    WORKFLOW_MULTI_QUEUE_NOT_SUPPORTED = "PKU901",
     PII_IN_OUTPUT = "PKU910",
     ADDON_WIRING_NOT_ALLOWED = "PKU920",
     ADDON_CONTRACT_HANDLERS_NOT_ALLOWED = "PKU921",

package/dist/error-codes.js

@@ -16,26 +16,23 @@ export var ErrorCode;
     ErrorCode["MISSING_URI"] = "PKU220";
     ErrorCode["MISSING_FUNC"] = "PKU236";
     ErrorCode["INVALID_TAGS_TYPE"] = "PKU247";
-    ErrorCode["INVALID_HANDLER"] = "PKU300";
     ErrorCode["MISSING_TITLE"] = "PKU370";
     ErrorCode["MISSING_QUEUE_NAME"] = "PKU384";
     ErrorCode["MISSING_CHANNEL_NAME"] = "PKU400";
     ErrorCode["CLI_CLIENTSIDE_RENDERER_HAS_SERVICES"] = "PKU672";
     ErrorCode["SCENARIO_HAS_SERVICES"] = "PKU673";
     ErrorCode["EXPECT_EVENTUALLY_SCENARIO_ONLY"] = "PKU675";
-    ErrorCode["DYNAMIC_STEP_NAME"] = "PKU529";
     ErrorCode["WORKFLOW_ORCHESTRATOR_NOT_CONFIGURED"] = "PKU600";
     ErrorCode["INVALID_DSL_WORKFLOW"] = "PKU641";
+    // Database schema codegen warnings
+    ErrorCode["DB_COLUMN_NAME_TYPE_CONTRADICTION"] = "PKU480";
+    ErrorCode["DB_JSON_COLUMN_UNTYPED"] = "PKU481";
+    ErrorCode["DB_FORMAT_ON_NON_STRING"] = "PKU482";
     // Configuration errors
-    ErrorCode["CONFIG_TYPE_NOT_FOUND"] = "PKU426";
-    ErrorCode["CONFIG_TYPE_UNDEFINED"] = "PKU427";
-    ErrorCode["SCHEMA_NO_ROOT"] = "PKU431";
     ErrorCode["SCHEMA_GENERATION_ERROR"] = "PKU456";
-    ErrorCode["SCHEMA_LOAD_ERROR"] = "PKU488";
     ErrorCode["INLINE_SCHEMA"] = "PKU489";
     // Function errors
     ErrorCode["FUNCTION_METADATA_NOT_FOUND"] = "PKU559";
-    ErrorCode["HANDLER_NOT_RESOLVED"] = "PKU568";
     // Auth errors
     ErrorCode["DUPLICATE_AUTH_DEFINITION"] = "PKU581";
     ErrorCode["AUTH_NOT_EXPORTED"] = "PKU582";
@@ -43,15 +40,6 @@ export var ErrorCode;
     ErrorCode["ROUTE_PARAM_MISMATCH"] = "PKU571";
     ErrorCode["ROUTE_QUERY_MISMATCH"] = "PKU572";
     ErrorCode["AUTH_DISABLED_REQUIRES_SESSIONLESS"] = "PKU573";
-    // Middleware/Permission errors
-    ErrorCode["MIDDLEWARE_HANDLER_INVALID"] = "PKU685";
-    ErrorCode["MIDDLEWARE_TAG_INVALID"] = "PKU715";
-    ErrorCode["MIDDLEWARE_EMPTY_ARRAY"] = "PKU736";
-    ErrorCode["MIDDLEWARE_PATTERN_INVALID"] = "PKU787";
-    ErrorCode["PERMISSION_HANDLER_INVALID"] = "PKU835";
-    ErrorCode["PERMISSION_TAG_INVALID"] = "PKU836";
-    ErrorCode["PERMISSION_EMPTY_ARRAY"] = "PKU937";
-    ErrorCode["PERMISSION_PATTERN_INVALID"] = "PKU975";
     // Versioning errors
     ErrorCode["DUPLICATE_FUNCTION_VERSION"] = "PKU850";
     ErrorCode["DUPLICATE_FUNCTION_NAME"] = "PKU851";
@@ -72,8 +60,6 @@ export var ErrorCode;
     ErrorCode["WIRES_NOT_DESTRUCTURED"] = "PKU411";
     // Dependency integrity errors
     ErrorCode["DUPLICATE_CORE_VERSION"] = "PKU717";
-    // Feature Flag
-    ErrorCode["WORKFLOW_MULTI_QUEUE_NOT_SUPPORTED"] = "PKU901";
     // Data classification errors
     ErrorCode["PII_IN_OUTPUT"] = "PKU910";
     // Addon authoring errors

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pikku/inspector",
-  "version": "0.12.37",
+  "version": "0.12.39",
   "author": "yasser.fadl@gmail.com",
   "license": "BUSL-1.1",
   "type": "module",
@@ -35,7 +35,6 @@
   },
   "dependencies": {
     "@openapi-contrib/json-schema-to-openapi-schema": "^4.3.1",
-    "@pikku/core": "^0.12.53",
     "openapi-types": "^12.1.3",
     "path-to-regexp": "^8.3.0",
     "ts-json-schema-generator": "2.9.1-next.16",
@@ -44,10 +43,14 @@
     "zod": "^4.3.6",
     "zod-to-ts": "^2.1.0"
   },
+  "peerDependencies": {
+    "@pikku/core": "^0.12.58"
+  },
   "devDependencies": {
+    "@pikku/core": "^0.12.58",
     "@types/node": "^24.11.0"
   },
   "engines": {
-    "node": ">=22"
+    "node": ">=24"
   }
 }

package/src/error-codes-emitted.test.ts

@@ -0,0 +1,76 @@
+import { test } from 'node:test'
+import { strict as assert } from 'node:assert'
+import { readFileSync, readdirSync } from 'node:fs'
+import { fileURLToPath } from 'node:url'
+import { dirname, join } from 'node:path'
+
+/**
+ * Guards against dead `ErrorCode` registry entries.
+ *
+ * Every value in the `ErrorCode` enum must have at least one emission site in
+ * the framework source — a `ErrorCode.<NAME>` reference or the raw `PKU###`
+ * string in a non-test `.ts` file under `packages/*​/src`. A code that is
+ * defined but never thrown is a dead entry: it can never appear in a user's
+ * terminal, yet it still demands a docs page and clutters the registry. This
+ * test fails and names any such entries so they get wired up or removed rather
+ * than silently accumulating.
+ */
+
+const here = dirname(fileURLToPath(import.meta.url))
+const packagesDir = join(here, '..', '..')
+const errorCodesFile = join(here, 'error-codes.ts')
+
+const enumMembers = [
+  ...readFileSync(errorCodesFile, 'utf8').matchAll(
+    /^\s*([A-Z0-9_]+)\s*=\s*'(PKU\d+)'/gm
+  ),
+].map((m) => ({ name: m[1], code: m[2] }))
+
+const collectSource = () => {
+  const contents: string[] = []
+  for (const pkg of readdirSync(packagesDir, { withFileTypes: true })) {
+    if (!pkg.isDirectory()) continue
+    const srcDir = join(packagesDir, pkg.name, 'src')
+    let entries
+    try {
+      entries = readdirSync(srcDir, { recursive: true, withFileTypes: true })
+    } catch {
+      continue
+    }
+    for (const entry of entries) {
+      if (!entry.isFile()) continue
+      const name = entry.name
+      if (
+        !name.endsWith('.ts') ||
+        name.endsWith('.d.ts') ||
+        name.endsWith('.test.ts') ||
+        name === 'error-codes.ts'
+      ) {
+        continue
+      }
+      contents.push(readFileSync(join(entry.parentPath, name), 'utf8'))
+    }
+  }
+  return contents.join('\n')
+}
+
+test('every ErrorCode value has an emission site in the source', () => {
+  assert.ok(
+    enumMembers.length > 0,
+    'Failed to parse any ErrorCode members from error-codes.ts'
+  )
+
+  const source = collectSource()
+
+  const dead = enumMembers.filter(
+    ({ name, code }) =>
+      !source.includes(`ErrorCode.${name}`) && !source.includes(code)
+  )
+
+  assert.deepEqual(
+    dead.map(({ code, name }) => `${code} ${name}`),
+    [],
+    `Dead ErrorCode entries (defined but never emitted). Wire them to a diagnostic or delete them:\n` +
+      dead.map(({ code, name }) => `  - ${code} ${name}`).join('\n')
+  )
+})

package/src/error-codes.ts

@@ -16,28 +16,26 @@ export enum ErrorCode {
   MISSING_URI = 'PKU220',
   MISSING_FUNC = 'PKU236',
   INVALID_TAGS_TYPE = 'PKU247',
-  INVALID_HANDLER = 'PKU300',
   MISSING_TITLE = 'PKU370',
   MISSING_QUEUE_NAME = 'PKU384',
   MISSING_CHANNEL_NAME = 'PKU400',
   CLI_CLIENTSIDE_RENDERER_HAS_SERVICES = 'PKU672',
   SCENARIO_HAS_SERVICES = 'PKU673',
   EXPECT_EVENTUALLY_SCENARIO_ONLY = 'PKU675',
-  DYNAMIC_STEP_NAME = 'PKU529',
   WORKFLOW_ORCHESTRATOR_NOT_CONFIGURED = 'PKU600',
   INVALID_DSL_WORKFLOW = 'PKU641',
 
+  // Database schema codegen warnings
+  DB_COLUMN_NAME_TYPE_CONTRADICTION = 'PKU480',
+  DB_JSON_COLUMN_UNTYPED = 'PKU481',
+  DB_FORMAT_ON_NON_STRING = 'PKU482',
+
   // Configuration errors
-  CONFIG_TYPE_NOT_FOUND = 'PKU426',
-  CONFIG_TYPE_UNDEFINED = 'PKU427',
-  SCHEMA_NO_ROOT = 'PKU431',
   SCHEMA_GENERATION_ERROR = 'PKU456',
-  SCHEMA_LOAD_ERROR = 'PKU488',
   INLINE_SCHEMA = 'PKU489',
 
   // Function errors
   FUNCTION_METADATA_NOT_FOUND = 'PKU559',
-  HANDLER_NOT_RESOLVED = 'PKU568',
 
   // Auth errors
   DUPLICATE_AUTH_DEFINITION = 'PKU581',
@@ -48,16 +46,6 @@ export enum ErrorCode {
   ROUTE_QUERY_MISMATCH = 'PKU572',
   AUTH_DISABLED_REQUIRES_SESSIONLESS = 'PKU573',
 
-  // Middleware/Permission errors
-  MIDDLEWARE_HANDLER_INVALID = 'PKU685',
-  MIDDLEWARE_TAG_INVALID = 'PKU715',
-  MIDDLEWARE_EMPTY_ARRAY = 'PKU736',
-  MIDDLEWARE_PATTERN_INVALID = 'PKU787',
-  PERMISSION_HANDLER_INVALID = 'PKU835',
-  PERMISSION_TAG_INVALID = 'PKU836',
-  PERMISSION_EMPTY_ARRAY = 'PKU937',
-  PERMISSION_PATTERN_INVALID = 'PKU975',
-
   // Versioning errors
   DUPLICATE_FUNCTION_VERSION = 'PKU850',
   DUPLICATE_FUNCTION_NAME = 'PKU851',
@@ -84,9 +72,6 @@ export enum ErrorCode {
   // Dependency integrity errors
   DUPLICATE_CORE_VERSION = 'PKU717',
 
-  // Feature Flag
-  WORKFLOW_MULTI_QUEUE_NOT_SUPPORTED = 'PKU901',
-
   // Data classification errors
   PII_IN_OUTPUT = 'PKU910',
 

package/src/utils/filter-inspector-state.test.ts

@@ -1668,7 +1668,7 @@ describe('addon bootstrap tree-shake', () => {
       ['console', { package: '@pikku/addon-console' }],
     ])
     state.rpc.usedAddons = new Set(['console'])
-    state.rpc.invokedFunctions = new Set(['console:streamFunctionTests'])
+    state.rpc.invokedFunctions = new Set(['console:streamWorkflowRun'])
     mutate?.(state)
     return state
   }
@@ -1687,7 +1687,7 @@ describe('addon bootstrap tree-shake', () => {
   test('keeps an addon when a kept wiring targets one of its functions', () => {
     const state = withAddon((s) => {
       s.http.meta.get['/console/stream'] = {
-        pikkuFuncId: 'console:streamFunctionTests',
+        pikkuFuncId: 'console:streamWorkflowRun',
         route: '/console/stream',
         method: 'GET',
         tags: [],
@@ -1697,7 +1697,7 @@ describe('addon bootstrap tree-shake', () => {
     })
     const filtered = filterInspectorState(
       state,
-      { names: ['console:streamFunctionTests'] },
+      { names: ['console:streamWorkflowRun'] },
       mockLogger
     )
     assert.strictEqual(filtered.rpc.wireAddonDeclarations.size, 1)
@@ -1706,45 +1706,43 @@ describe('addon bootstrap tree-shake', () => {
 
   test('keeps an addon when a kept route ref()-targets one of its functions', () => {
     const state = withAddon((s) => {
-      s.http.meta.get['/function-tests/stream'] = {
-        pikkuFuncId: 'http:get:/function-tests/stream',
-        refTarget: 'console:streamFunctionTests',
-        route: '/function-tests/stream',
+      s.http.meta.get['/workflow-run/stream'] = {
+        pikkuFuncId: 'http:get:/workflow-run/stream',
+        refTarget: 'console:streamWorkflowRun',
+        route: '/workflow-run/stream',
         method: 'GET',
         tags: [],
         middleware: [],
         permissions: [],
       } as any
-      s.functions.meta['http:get:/function-tests/stream'] = {
+      s.functions.meta['http:get:/workflow-run/stream'] = {
         services: { optimized: false, services: [] },
       } as any
     })
     const filtered = filterInspectorState(
       state,
-      { names: ['http:get:/function-tests/stream'] },
+      { names: ['http:get:/workflow-run/stream'] },
       mockLogger
     )
     assert.strictEqual(filtered.rpc.wireAddonDeclarations.size, 1)
     assert.ok(filtered.rpc.wireAddonDeclarations.has('console'))
     assert.ok(
-      filtered.serviceAggregation.usedFunctions.has(
-        'console:streamFunctionTests'
-      )
+      filtered.serviceAggregation.usedFunctions.has('console:streamWorkflowRun')
     )
   })
 
   test('drops the addon when the ref()-wired route is filtered out', () => {
     const state = withAddon((s) => {
-      s.http.meta.get['/function-tests/stream'] = {
-        pikkuFuncId: 'http:get:/function-tests/stream',
-        refTarget: 'console:streamFunctionTests',
-        route: '/function-tests/stream',
+      s.http.meta.get['/workflow-run/stream'] = {
+        pikkuFuncId: 'http:get:/workflow-run/stream',
+        refTarget: 'console:streamWorkflowRun',
+        route: '/workflow-run/stream',
         method: 'GET',
         tags: [],
         middleware: [],
         permissions: [],
       } as any
-      s.functions.meta['http:get:/function-tests/stream'] = {
+      s.functions.meta['http:get:/workflow-run/stream'] = {
         services: { optimized: false, services: [] },
       } as any
     })
@@ -1756,7 +1754,7 @@ describe('addon bootstrap tree-shake', () => {
     assert.strictEqual(filtered.rpc.wireAddonDeclarations.size, 0)
     assert.ok(
       !filtered.serviceAggregation.usedFunctions.has(
-        'console:streamFunctionTests'
+        'console:streamWorkflowRun'
       )
     )
   })

package/src/utils/post-process.test.ts

@@ -206,17 +206,17 @@ describe('aggregateRequiredServices — per-function addon services', () => {
   test('a ref()-wired route (inline id + namespaced target) aggregates the target services', () => {
     const state = makeState({
       usedFunctions: [
-        'http:get:/function-tests/stream',
-        'console:streamFunctionTests',
+        'http:get:/workflow-run/stream',
+        'console:streamWorkflowRun',
       ],
       functionsMeta: {
-        'http:get:/function-tests/stream': {
+        'http:get:/workflow-run/stream': {
           services: { optimized: false, services: [] },
         },
       },
       addonFunctions: {
         console: {
-          streamFunctionTests: {
+          streamWorkflowRun: {
             services: { optimized: true, services: ['metaService'] },
           },
         },