@pikku/inspector 0.12.74 → 0.12.81

package/src/utils/compute-required-schemas.ts

@@ -1,48 +0,0 @@
-import type { FunctionsMeta } from '@pikku/core/services'
-import type { TypesMap } from '../types-map.js'
-import type { SchemaRef } from '../types.js'
-
-const PRIMITIVE_TYPES = new Set([
-  'boolean',
-  'string',
-  'number',
-  'null',
-  'undefined',
-  'void',
-  'any',
-  'unknown',
-  'never',
-])
-
-export function computeRequiredSchemas(
-  functionsMeta: FunctionsMeta,
-  typesMap: TypesMap,
-  additionalTypes?: string[],
-  schemaLookup?: Map<string, SchemaRef>
-): Set<string> {
-  return new Set<string>([
-    ...Object.values(functionsMeta)
-      .flatMap(({ inputs, outputs }) => {
-        const types: (string | undefined)[] = []
-        if (inputs?.[0]) {
-          try {
-            types.push(typesMap.getUniqueName(inputs[0]))
-          } catch {
-            types.push(inputs[0])
-          }
-        }
-        if (outputs?.[0]) {
-          try {
-            types.push(typesMap.getUniqueName(outputs[0]))
-          } catch {
-            types.push(outputs[0])
-          }
-        }
-        return types
-      })
-      .filter((s): s is string => !!s && !PRIMITIVE_TYPES.has(s)),
-    ...typesMap.customTypes.keys(),
-    ...(additionalTypes || []),
-    ...(schemaLookup ? Array.from(schemaLookup.keys()) : []),
-  ])
-}

package/src/utils/contract-hashes.test.ts

@@ -1,553 +0,0 @@
-import { strict as assert } from 'assert'
-import { describe, test } from 'node:test'
-import { canonicalJSON } from './hash.js'
-import {
-  computeContractHash,
-  createEmptyManifest,
-  serializeManifest,
-  validateContracts,
-  updateManifest,
-  extractContractsFromMeta,
-} from './contract-hashes.js'
-import type { ContractEntry, VersionManifest } from './contract-hashes.js'
-import type { FunctionsMeta } from '@pikku/core/services'
-import { ErrorCode } from '../error-codes.js'
-
-function makeContracts(
-  entries: Record<
-    string,
-    {
-      functionKey: string
-      version: number
-      contractHash: string
-      inputHash?: string
-      outputHash?: string
-    }
-  >
-): Map<string, ContractEntry> {
-  const map = new Map<string, ContractEntry>()
-  for (const [id, entry] of Object.entries(entries)) {
-    map.set(id, {
-      ...entry,
-      inputHash: entry.inputHash ?? entry.contractHash.slice(0, 8),
-      outputHash: entry.outputHash ?? entry.contractHash.slice(0, 8),
-    })
-  }
-  return map
-}
-
-describe('canonicalJSON', () => {
-  test('sorts object keys alphabetically', () => {
-    assert.strictEqual(canonicalJSON({ b: 2, a: 1 }), '{"a":1,"b":2}')
-  })
-
-  test('sorts nested objects recursively', () => {
-    const input = { z: { b: 2, a: 1 }, a: { d: 4, c: 3 } }
-    const expected = '{"a":{"c":3,"d":4},"z":{"a":1,"b":2}}'
-    assert.strictEqual(canonicalJSON(input), expected)
-  })
-
-  test('preserves array order', () => {
-    assert.strictEqual(canonicalJSON([3, 1, 2]), '[3,1,2]')
-    assert.strictEqual(
-      canonicalJSON([
-        { b: 1, a: 2 },
-        { d: 3, c: 4 },
-      ]),
-      '[{"a":2,"b":1},{"c":4,"d":3}]'
-    )
-  })
-
-  test('handles null/undefined/primitives', () => {
-    assert.strictEqual(canonicalJSON(null), 'null')
-    assert.strictEqual(canonicalJSON(undefined), undefined)
-    assert.strictEqual(canonicalJSON(42), '42')
-    assert.strictEqual(canonicalJSON('hello'), '"hello"')
-    assert.strictEqual(canonicalJSON(true), 'true')
-  })
-
-  test('deterministic regardless of insertion order', () => {
-    const a: Record<string, number> = {}
-    a['x'] = 1
-    a['a'] = 2
-    a['m'] = 3
-
-    const b: Record<string, number> = {}
-    b['m'] = 3
-    b['a'] = 2
-    b['x'] = 1
-
-    assert.strictEqual(canonicalJSON(a), canonicalJSON(b))
-  })
-})
-
-describe('computeContractHash', () => {
-  test('returns 16-char hex string', () => {
-    const hash = computeContractHash({
-      functionKey: 'createUser',
-      inputSchema: { type: 'object' },
-      outputSchema: { type: 'object' },
-    })
-    assert.match(hash, /^[0-9a-f]{16}$/)
-  })
-
-  test('stable across calls', () => {
-    const data = {
-      functionKey: 'getUser',
-      inputSchema: { id: 'string' },
-      outputSchema: { name: 'string' },
-    }
-    assert.strictEqual(computeContractHash(data), computeContractHash(data))
-  })
-
-  test('different functionKey produces different hash', () => {
-    const schema = { type: 'object' }
-    const hash1 = computeContractHash({
-      functionKey: 'createUser',
-      inputSchema: schema,
-      outputSchema: schema,
-    })
-    const hash2 = computeContractHash({
-      functionKey: 'deleteUser',
-      inputSchema: schema,
-      outputSchema: schema,
-    })
-    assert.notStrictEqual(hash1, hash2)
-  })
-
-  test('different schema produces different hash', () => {
-    const hash1 = computeContractHash({
-      functionKey: 'createUser',
-      inputSchema: { type: 'string' },
-      outputSchema: null,
-    })
-    const hash2 = computeContractHash({
-      functionKey: 'createUser',
-      inputSchema: { type: 'number' },
-      outputSchema: null,
-    })
-    assert.notStrictEqual(hash1, hash2)
-  })
-})
-
-describe('createEmptyManifest', () => {
-  test('returns empty manifest with version 1', () => {
-    assert.deepStrictEqual(createEmptyManifest(), {
-      manifestVersion: 1,
-      contracts: {},
-    })
-  })
-})
-
-describe('serializeManifest', () => {
-  test('sorts contract keys alphabetically', () => {
-    const manifest: VersionManifest = {
-      manifestVersion: 1,
-      contracts: {
-        zeta: { latest: 1, versions: { '1': 'abc' } },
-        alpha: { latest: 1, versions: { '1': 'def' } },
-      },
-    }
-    const result = serializeManifest(manifest)
-    const keys = Object.keys(JSON.parse(result).contracts)
-    assert.deepStrictEqual(keys, ['alpha', 'zeta'])
-  })
-
-  test('sorts version keys numerically', () => {
-    const manifest: VersionManifest = {
-      manifestVersion: 1,
-      contracts: {
-        func: { latest: 10, versions: { '10': 'c', '2': 'b', '1': 'a' } },
-      },
-    }
-    const result = serializeManifest(manifest)
-    const versionKeys = Object.keys(JSON.parse(result).contracts.func.versions)
-    assert.deepStrictEqual(versionKeys, ['1', '2', '10'])
-  })
-
-  test('ends with trailing newline', () => {
-    const manifest: VersionManifest = {
-      manifestVersion: 1,
-      contracts: {},
-    }
-    const result = serializeManifest(manifest)
-    assert.ok(result.endsWith('\n'))
-  })
-
-  test('idempotent', () => {
-    const manifest: VersionManifest = {
-      manifestVersion: 1,
-      contracts: {
-        beta: { latest: 2, versions: { '2': 'xyz', '1': 'abc' } },
-        alpha: { latest: 1, versions: { '1': 'def' } },
-      },
-    }
-    const first = serializeManifest(manifest)
-    const second = serializeManifest(JSON.parse(first))
-    assert.strictEqual(first, second)
-  })
-})
-
-describe('validateContracts', () => {
-  describe('PKU861 — FUNCTION_VERSION_MODIFIED', () => {
-    test('hash changed for existing version produces error', () => {
-      const manifest: VersionManifest = {
-        manifestVersion: 1,
-        contracts: {
-          createUser: { latest: 1, versions: { '1': 'oldhash1234567890' } },
-        },
-      }
-      const contracts = makeContracts({
-        createUser: {
-          functionKey: 'createUser',
-          version: 1,
-          contractHash: 'newhash1234567890',
-        },
-      })
-      const result = validateContracts(manifest, contracts)
-      assert.strictEqual(result.valid, false)
-      assert.strictEqual(result.errors.length, 1)
-      assert.strictEqual(
-        result.errors[0].code,
-        ErrorCode.FUNCTION_VERSION_MODIFIED
-      )
-    })
-
-    test('hash unchanged for existing version produces no error', () => {
-      const manifest: VersionManifest = {
-        manifestVersion: 1,
-        contracts: {
-          createUser: { latest: 1, versions: { '1': 'samehash123456789' } },
-        },
-      }
-      const contracts = makeContracts({
-        createUser: {
-          functionKey: 'createUser',
-          version: 1,
-          contractHash: 'samehash123456789',
-        },
-      })
-      const result = validateContracts(manifest, contracts)
-      assert.strictEqual(result.valid, true)
-      assert.strictEqual(result.errors.length, 0)
-    })
-  })
-
-  describe('PKU862 — CONTRACT_CHANGED_REQUIRES_BUMP', () => {
-    test('latest hash changed without recorded version produces bump error', () => {
-      const manifest: VersionManifest = {
-        manifestVersion: 1,
-        contracts: {
-          getUser: { latest: 1, versions: {} },
-        },
-      }
-      const contracts = makeContracts({
-        getUser: {
-          functionKey: 'getUser',
-          version: 1,
-          contractHash: 'newhash1234567890',
-        },
-      })
-      const result = validateContracts(manifest, contracts)
-      const bump = result.errors.find(
-        (e) => e.code === ErrorCode.CONTRACT_CHANGED_REQUIRES_BUMP
-      )
-      assert.ok(bump)
-    })
-
-    test('function removed from code but still in manifest produces no error', () => {
-      const manifest: VersionManifest = {
-        manifestVersion: 1,
-        contracts: {
-          deletedFunc: { latest: 1, versions: { '1': 'abc1234567890abc' } },
-        },
-      }
-      const contracts = makeContracts({})
-      const result = validateContracts(manifest, contracts)
-      assert.strictEqual(result.valid, true)
-      assert.strictEqual(result.errors.length, 0)
-    })
-  })
-
-  describe('PKU863 — VERSION_REGRESSION_OR_CONFLICT', () => {
-    test('new version <= latest but not recorded produces error', () => {
-      const manifest: VersionManifest = {
-        manifestVersion: 1,
-        contracts: {
-          createUser: {
-            latest: 3,
-            versions: { '1': 'aaa', '2': 'bbb', '3': 'ccc' },
-          },
-        },
-      }
-      const contracts = makeContracts({
-        'createUser@v2': {
-          functionKey: 'createUser',
-          version: 2,
-          contractHash: 'bbb',
-        },
-        'createUser@v3': {
-          functionKey: 'createUser',
-          version: 3,
-          contractHash: 'ccc',
-        },
-        'createUser@v1': {
-          functionKey: 'createUser',
-          version: 1,
-          contractHash: 'different_hash!',
-        },
-      })
-      const result = validateContracts(manifest, contracts)
-      const regression = result.errors.find(
-        (e) => e.code === ErrorCode.VERSION_REGRESSION_OR_CONFLICT
-      )
-      assert.strictEqual(regression, undefined)
-
-      const manifest2: VersionManifest = {
-        manifestVersion: 1,
-        contracts: {
-          myFunc: { latest: 3, versions: { '1': 'aaa', '3': 'ccc' } },
-        },
-      }
-      const contracts2 = makeContracts({
-        'myFunc@v2': { functionKey: 'myFunc', version: 2, contractHash: 'xxx' },
-        'myFunc@v3': { functionKey: 'myFunc', version: 3, contractHash: 'ccc' },
-      })
-      const result2 = validateContracts(manifest2, contracts2)
-      const regression2 = result2.errors.find(
-        (e) => e.code === ErrorCode.VERSION_REGRESSION_OR_CONFLICT
-      )
-      assert.ok(regression2)
-    })
-  })
-
-  describe('PKU864 — VERSION_GAP_NOT_ALLOWED', () => {
-    test('new version > latest + 1 produces error', () => {
-      const manifest: VersionManifest = {
-        manifestVersion: 1,
-        contracts: {
-          createUser: { latest: 1, versions: { '1': 'aaa' } },
-        },
-      }
-      const contracts = makeContracts({
-        'createUser@v3': {
-          functionKey: 'createUser',
-          version: 3,
-          contractHash: 'xxx',
-        },
-      })
-      const result = validateContracts(manifest, contracts)
-      const gap = result.errors.find(
-        (e) => e.code === ErrorCode.VERSION_GAP_NOT_ALLOWED
-      )
-      assert.ok(gap)
-    })
-  })
-
-  describe('PKU865 — MANIFEST_INTEGRITY_ERROR', () => {
-    test('latest field not matching max version key produces error', () => {
-      const manifest: VersionManifest = {
-        manifestVersion: 1,
-        contracts: {
-          createUser: { latest: 1, versions: { '1': 'aaa', '2': 'bbb' } },
-        },
-      }
-      const contracts = makeContracts({})
-      const result = validateContracts(manifest, contracts)
-      const integrity = result.errors.find(
-        (e) => e.code === ErrorCode.MANIFEST_INTEGRITY_ERROR
-      )
-      assert.ok(integrity)
-    })
-  })
-
-  describe('happy paths', () => {
-    test('new function not in manifest is valid', () => {
-      const manifest: VersionManifest = {
-        manifestVersion: 1,
-        contracts: {},
-      }
-      const contracts = makeContracts({
-        createUser: {
-          functionKey: 'createUser',
-          version: 1,
-          contractHash: 'abc123',
-        },
-      })
-      const result = validateContracts(manifest, contracts)
-      assert.strictEqual(result.valid, true)
-    })
-
-    test('correct version bump is valid', () => {
-      const manifest: VersionManifest = {
-        manifestVersion: 1,
-        contracts: {
-          createUser: { latest: 1, versions: { '1': 'aaa' } },
-        },
-      }
-      const contracts = makeContracts({
-        'createUser@v1': {
-          functionKey: 'createUser',
-          version: 1,
-          contractHash: 'aaa',
-        },
-        'createUser@v2': {
-          functionKey: 'createUser',
-          version: 2,
-          contractHash: 'bbb',
-        },
-      })
-      const result = validateContracts(manifest, contracts)
-      assert.strictEqual(result.valid, true)
-    })
-
-    test('empty manifest with new contracts is valid', () => {
-      const manifest: VersionManifest = {
-        manifestVersion: 1,
-        contracts: {},
-      }
-      const contracts = makeContracts({
-        funcA: { functionKey: 'funcA', version: 1, contractHash: 'aaa' },
-        funcB: { functionKey: 'funcB', version: 1, contractHash: 'bbb' },
-      })
-      const result = validateContracts(manifest, contracts)
-      assert.strictEqual(result.valid, true)
-    })
-  })
-})
-
-describe('updateManifest', () => {
-  test('adds new function to empty manifest', () => {
-    const manifest: VersionManifest = { manifestVersion: 1, contracts: {} }
-    const contracts = makeContracts({
-      createUser: {
-        functionKey: 'createUser',
-        version: 1,
-        contractHash: 'abc',
-        inputHash: 'in123456',
-        outputHash: 'out1234',
-      },
-    })
-    const result = updateManifest(manifest, contracts)
-    assert.strictEqual(result.contracts.createUser.latest, 1)
-    assert.deepStrictEqual(result.contracts.createUser.versions['1'], {
-      inputHash: 'in123456',
-      outputHash: 'out1234',
-    })
-  })
-
-  test('adds new version to existing function and updates latest', () => {
-    const manifest: VersionManifest = {
-      manifestVersion: 1,
-      contracts: {
-        createUser: { latest: 1, versions: { '1': 'aaa' } },
-      },
-    }
-    const contracts = makeContracts({
-      'createUser@v1': {
-        functionKey: 'createUser',
-        version: 1,
-        contractHash: 'aaa',
-        inputHash: 'inaaa123',
-        outputHash: 'outaaa12',
-      },
-      'createUser@v2': {
-        functionKey: 'createUser',
-        version: 2,
-        contractHash: 'bbb',
-        inputHash: 'inbbb123',
-        outputHash: 'outbbb12',
-      },
-    })
-    const result = updateManifest(manifest, contracts)
-    assert.strictEqual(result.contracts.createUser.latest, 2)
-    assert.deepStrictEqual(result.contracts.createUser.versions['2'], {
-      inputHash: 'inbbb123',
-      outputHash: 'outbbb12',
-    })
-    assert.deepStrictEqual(result.contracts.createUser.versions['1'], {
-      inputHash: 'inaaa123',
-      outputHash: 'outaaa12',
-    })
-  })
-
-  test('preserves deleted functions', () => {
-    const manifest: VersionManifest = {
-      manifestVersion: 1,
-      contracts: {
-        removedFunc: { latest: 1, versions: { '1': 'old' } },
-      },
-    }
-    const contracts = makeContracts({
-      newFunc: { functionKey: 'newFunc', version: 1, contractHash: 'new' },
-    })
-    const result = updateManifest(manifest, contracts)
-    assert.ok(result.contracts.removedFunc)
-    assert.strictEqual(result.contracts.removedFunc.latest, 1)
-    assert.ok(result.contracts.newFunc)
-  })
-
-  test('does not mutate input manifest', () => {
-    const manifest: VersionManifest = {
-      manifestVersion: 1,
-      contracts: {
-        createUser: { latest: 1, versions: { '1': 'aaa' } },
-      },
-    }
-    const before = JSON.stringify(manifest)
-    const contracts = makeContracts({
-      'createUser@v2': {
-        functionKey: 'createUser',
-        version: 2,
-        contractHash: 'bbb',
-      },
-    })
-    updateManifest(manifest, contracts)
-    assert.strictEqual(JSON.stringify(manifest), before)
-  })
-})
-
-describe('extractContractsFromMeta', () => {
-  test('extracts entries where contractHash is set', () => {
-    const meta: FunctionsMeta = {
-      createUser: {
-        pikkuFuncId: 'createUser',
-        inputSchemaName: 'CreateUserInput',
-        outputSchemaName: 'CreateUserOutput',
-        contractHash: 'abc123',
-      },
-    }
-    const result = extractContractsFromMeta(meta)
-    assert.strictEqual(result.size, 1)
-    assert.ok(result.has('createUser'))
-    assert.strictEqual(result.get('createUser')!.contractHash, 'abc123')
-    assert.strictEqual(result.get('createUser')!.functionKey, 'createUser')
-    assert.strictEqual(result.get('createUser')!.version, 1)
-  })
-
-  test('skips remote functions', () => {
-    const meta: FunctionsMeta = {
-      remoteFunc: {
-        pikkuFuncId: 'remoteFunc',
-        inputSchemaName: null,
-        outputSchemaName: null,
-        remote: true,
-        contractHash: 'shouldskip',
-      },
-    }
-    const result = extractContractsFromMeta(meta)
-    assert.strictEqual(result.size, 0)
-  })
-
-  test('skips functions without contractHash', () => {
-    const meta: FunctionsMeta = {
-      noHash: {
-        pikkuFuncId: 'noHash',
-        inputSchemaName: null,
-        outputSchemaName: null,
-      },
-    }
-    const result = extractContractsFromMeta(meta)
-    assert.strictEqual(result.size, 0)
-  })
-})