@pikku/inspector 0.12.75 → 0.12.81

package/src/add/add-agent-scorer.ts

@@ -1,83 +0,0 @@
-import * as ts from 'typescript'
-import { getPropertyValue } from '../utils/get-property-value.js'
-import type { AddWiring } from '../types.js'
-import { extractFunctionName } from '../utils/extract-function-name.js'
-import { ErrorCode } from '../error-codes.js'
-
-/**
- * `pikkuAgentScorer` grades in code and `pikkuAgentJudge` grades with a model, which
- * is the whole difference between the two lanes — so the lane is read off which
- * constructor was called rather than from a field an author could contradict.
- */
-const LANES: Record<string, 'fast' | 'slow'> = {
-  pikkuAgentScorer: 'fast',
-  pikkuAgentJudge: 'slow',
-}
-
-export const addAgentScorer: AddWiring = (logger, node, checker, state) => {
-  if (!ts.isCallExpression(node)) return
-
-  const expression = node.expression
-  if (!ts.isIdentifier(expression)) return
-
-  const lane = LANES[expression.text]
-  if (!lane) return
-
-  const firstArg = node.arguments[0]
-  if (!firstArg || !ts.isObjectLiteralExpression(firstArg)) return
-
-  const obj = firstArg
-  const { exportedName } = extractFunctionName(node, checker, state.rootDir)
-
-  const nameValue = getPropertyValue(obj, 'name') as string | null
-  if (!nameValue) {
-    logger.critical(
-      ErrorCode.MISSING_NAME,
-      "AI scorer is missing the required 'name' property."
-    )
-    return
-  }
-
-  const description = getPropertyValue(obj, 'description') as string | null
-  if (!description) {
-    logger.critical(
-      ErrorCode.MISSING_DESCRIPTION,
-      `AI scorer '${nameValue}' is missing the required 'description' property.`
-    )
-    return
-  }
-
-  const sampleRateValue = getPropertyValue(obj, 'sampleRate') as number | null
-  if (
-    sampleRateValue !== null &&
-    (sampleRateValue < 0 || sampleRateValue > 1)
-  ) {
-    logger.critical(
-      ErrorCode.INVALID_VALUE,
-      `AI scorer '${nameValue}' has a sampleRate of ${sampleRateValue} — it is the fraction of live runs to grade, so it must be between 0 and 1.`
-    )
-    return
-  }
-
-  const requiresReference =
-    (getPropertyValue(obj, 'requiresReference') as boolean | null) ?? false
-
-  const scorerKey = exportedName || nameValue
-
-  if (exportedName) {
-    state.scorers.files.set(scorerKey, {
-      path: node.getSourceFile().fileName,
-      exportedName,
-    })
-  }
-
-  state.scorers.scorersMeta[scorerKey] = {
-    name: nameValue,
-    description,
-    lane,
-    sampleRate: sampleRateValue ?? 1,
-    requiresReference,
-    sourceFile: node.getSourceFile().fileName,
-    exportedName: exportedName || undefined,
-  }
-}

package/src/add/add-agent.test.ts

@@ -1,87 +0,0 @@
-import { describe, test } from 'node:test'
-import assert from 'node:assert'
-import * as ts from 'typescript'
-
-import { resolveWorkflowReferences } from './add-agent.js'
-import type { InspectorLogger } from '../types.js'
-
-function createSourceFile(code: string): ts.SourceFile {
-  return ts.createSourceFile('test.ts', code, ts.ScriptTarget.ESNext, true)
-}
-
-function getObjectLiteral(
-  sourceFile: ts.SourceFile
-): ts.ObjectLiteralExpression {
-  let result: ts.ObjectLiteralExpression | undefined
-  ts.forEachChild(sourceFile, function visit(node) {
-    if (ts.isObjectLiteralExpression(node) && !result) {
-      result = node
-    }
-    ts.forEachChild(node, visit)
-  })
-  if (!result) throw new Error('No object literal found')
-  return result
-}
-
-function createLogger(): {
-  logger: InspectorLogger
-  criticals: Array<{ code: unknown; message: string }>
-} {
-  const criticals: Array<{ code: unknown; message: string }> = []
-  const logger = {
-    info: () => {},
-    error: () => {},
-    warn: () => {},
-    debug: () => {},
-    diagnostic: () => {},
-    critical: (code: unknown, message: string) =>
-      criticals.push({ code, message }),
-    hasCriticalErrors: () => criticals.length > 0,
-  } as unknown as InspectorLogger
-  return { logger, criticals }
-}
-
-describe('resolveWorkflowReferences', () => {
-  test('returns null when no workflows property exists', () => {
-    const obj = getObjectLiteral(createSourceFile('const x = { name: "a" }'))
-    const { logger } = createLogger()
-    assert.strictEqual(
-      resolveWorkflowReferences(obj, {} as any, 'a', logger),
-      null
-    )
-  })
-
-  test('returns null for an empty workflows array', () => {
-    const obj = getObjectLiteral(
-      createSourceFile('const x = { workflows: [] }')
-    )
-    const { logger } = createLogger()
-    assert.strictEqual(
-      resolveWorkflowReferences(obj, {} as any, 'a', logger),
-      null
-    )
-  })
-
-  test('resolves ref() and workflow() string-literal references', () => {
-    const obj = getObjectLiteral(
-      createSourceFile(
-        `const x = { workflows: [ref('buildReport'), workflow('country_capitals')] }`
-      )
-    )
-    const { logger, criticals } = createLogger()
-    const result = resolveWorkflowReferences(obj, {} as any, 'planner', logger)
-    assert.deepStrictEqual(result, ['buildReport', 'country_capitals'])
-    assert.strictEqual(criticals.length, 0)
-  })
-
-  test('flags a raw string literal as a critical error', () => {
-    const obj = getObjectLiteral(
-      createSourceFile(`const x = { workflows: ['buildReport'] }`)
-    )
-    const { logger, criticals } = createLogger()
-    const result = resolveWorkflowReferences(obj, {} as any, 'planner', logger)
-    assert.strictEqual(result, null)
-    assert.strictEqual(criticals.length, 1)
-    assert.match(criticals[0].message, /string literal/)
-  })
-})