@pikku/inspector 0.12.2 → 0.12.4

package/src/utils/schema-generator.ts

@@ -63,24 +63,37 @@ function primitiveTypeToSchema(typeStr: string): JSONValue | null {
   return null
 }
 
+// Cached state for schema program reuse across inspect() calls
+let cachedSchemaProgram: ts.Program | undefined
+let cachedParsedConfig: ts.ParsedCommandLine | undefined
+let cachedTsconfigPath: string | undefined
+let cachedCustomTypesContent: string | undefined
+let cachedTSSchemas: Record<string, JSONValue> | undefined
+
 function createProgramWithVirtualFile(
   tsconfig: string,
   virtualFilePath: string,
   virtualFileContent: string
 ): ts.Program {
   const configPath = resolve(tsconfig)
-  const configFile = ts.readConfigFile(configPath, ts.sys.readFile)
-  const basePath = dirname(configPath)
-  const parsedConfig = ts.parseJsonConfigFileContent(
-    configFile.config,
-    ts.sys,
-    basePath
-  )
+
+  // Cache the parsed tsconfig — it doesn't change between runs
+  if (!cachedParsedConfig || cachedTsconfigPath !== configPath) {
+    const configFile = ts.readConfigFile(configPath, ts.sys.readFile)
+    const basePath = dirname(configPath)
+    cachedParsedConfig = ts.parseJsonConfigFileContent(
+      configFile.config,
+      ts.sys,
+      basePath
+    )
+    cachedTsconfigPath = configPath
+    cachedSchemaProgram = undefined
+  }
 
   const resolvedVirtualPath = resolve(virtualFilePath)
-  const fileNames = [...parsedConfig.fileNames, resolvedVirtualPath]
+  const fileNames = [...cachedParsedConfig.fileNames, resolvedVirtualPath]
 
-  const defaultHost = ts.createCompilerHost(parsedConfig.options)
+  const defaultHost = ts.createCompilerHost(cachedParsedConfig.options)
   const customHost: ts.CompilerHost = {
     ...defaultHost,
     getSourceFile(
@@ -113,7 +126,14 @@ function createProgramWithVirtualFile(
     },
   }
 
-  return ts.createProgram(fileNames, parsedConfig.options, customHost)
+  const program = ts.createProgram(
+    fileNames,
+    cachedParsedConfig.options,
+    customHost,
+    cachedSchemaProgram // reuse previous program for incremental compilation
+  )
+  cachedSchemaProgram = program
+  return program
 }
 
 function generateTSSchemas(
@@ -313,6 +333,11 @@ export async function generateAllSchemas(
     requiredTypes
   )
 
+  if (cachedTSSchemas && cachedCustomTypesContent === customTypesContent) {
+    logger.debug('Reusing cached TS schemas (types unchanged)')
+    return { ...cachedTSSchemas, ...zodSchemas }
+  }
+
   const tsSchemas = generateTSSchemas(
     logger,
     config.tsconfig,
@@ -325,5 +350,8 @@ export async function generateAllSchemas(
     state.schemaLookup
   )
 
+  cachedCustomTypesContent = customTypesContent
+  cachedTSSchemas = tsSchemas
+
   return { ...tsSchemas, ...zodSchemas }
 }

package/src/utils/serialize-inspector-state.ts

@@ -81,6 +81,7 @@ export interface SerializableInspectorState {
     }
     meta: InspectorState['functions']['meta']
     files: Array<[string, { path: string; exportedName: string }]>
+    approvalDescriptions: InspectorState['functions']['approvalDescriptions']
   }
   http: {
     metaInputTypes: Array<
@@ -249,6 +250,7 @@ export interface SerializableInspectorState {
   requiredSchemas: string[]
   openAPISpec: Record<string, any> | null
   diagnostics: InspectorDiagnostic[]
+  addonFunctions: InspectorState['addonFunctions']
 }
 
 /**
@@ -307,6 +309,7 @@ export function serializeInspectorState(
       typesMap: serializeTypesMap(state.functions.typesMap),
       meta: state.functions.meta,
       files: Array.from(state.functions.files.entries()),
+      approvalDescriptions: state.functions.approvalDescriptions,
     },
     http: {
       metaInputTypes: Array.from(state.http.metaInputTypes.entries()),
@@ -416,6 +419,7 @@ export function serializeInspectorState(
     requiredSchemas: Array.from(state.requiredSchemas),
     openAPISpec: state.openAPISpec,
     diagnostics: state.diagnostics,
+    addonFunctions: state.addonFunctions,
   }
 }
 
@@ -475,6 +479,7 @@ export function deserializeInspectorState(
       typesMap: deserializeTypesMap(data.functions.typesMap),
       meta: data.functions.meta,
       files: new Map(data.functions.files),
+      approvalDescriptions: (data.functions as any).approvalDescriptions || {},
     },
     http: {
       metaInputTypes: new Map(data.http.metaInputTypes),
@@ -590,5 +595,7 @@ export function deserializeInspectorState(
     requiredSchemas: new Set(data.requiredSchemas || []),
     openAPISpec: data.openAPISpec || null,
     diagnostics: data.diagnostics || [],
+    addonFunctions: data.addonFunctions || {},
+    program: null,
   }
 }

package/src/utils/serialize-mcp-json.ts

@@ -1,5 +1,6 @@
 import type { InspectorLogger, InspectorState } from '../types.js'
 import type { JSONValue } from '@pikku/core'
+import { resolveFunctionMeta } from './resolve-function-meta.js'
 
 interface MCPEndpoint {
   uri?: string
@@ -15,7 +16,7 @@ export const serializeMCPJson = (
   state: InspectorState
 ): string => {
   const { mcpEndpoints, functions, schemas } = state
-  const { meta: functionsMeta, typesMap } = functions
+  const { typesMap } = functions
   const { resourcesMeta, toolsMeta, promptsMeta } = mcpEndpoints
 
   const tools: MCPEndpoint[] = []
@@ -39,9 +40,13 @@ export const serializeMCPJson = (
       return undefined
     }
 
-    const uniqueName = typesMap.getUniqueName(typeName)
-    if (!uniqueName) {
-      return undefined
+    // Try local typesMap first, fall back to direct schema lookup (for addon types)
+    let uniqueName: string | undefined
+    try {
+      uniqueName = typesMap.getUniqueName(typeName)
+    } catch {
+      // Type not in local typesMap — try direct schema lookup (addon schemas)
+      uniqueName = typeName
     }
 
     const schema = schemas[uniqueName]
@@ -56,7 +61,7 @@ export const serializeMCPJson = (
   }
 
   for (const [name, endpointMeta] of Object.entries(resourcesMeta)) {
-    const functionMeta = functionsMeta[endpointMeta.pikkuFuncId]
+    const functionMeta = resolveFunctionMeta(state, endpointMeta.pikkuFuncId)
     if (!functionMeta) {
       logger.warn(
         `Function ${endpointMeta.pikkuFuncId} not found in functionsMeta. Skipping resource ${name}.`
@@ -81,7 +86,7 @@ export const serializeMCPJson = (
   }
 
   for (const [name, endpointMeta] of Object.entries(toolsMeta)) {
-    const functionMeta = functionsMeta[endpointMeta.pikkuFuncId]
+    const functionMeta = resolveFunctionMeta(state, endpointMeta.pikkuFuncId)
     if (!functionMeta) {
       logger.warn(
         `Function ${endpointMeta.pikkuFuncId} not found in functionsMeta. Skipping tool ${name}.`
@@ -105,7 +110,7 @@ export const serializeMCPJson = (
   }
 
   for (const [name, endpointMeta] of Object.entries(promptsMeta)) {
-    const functionMeta = functionsMeta[endpointMeta.pikkuFuncId]
+    const functionMeta = resolveFunctionMeta(state, endpointMeta.pikkuFuncId)
     if (!functionMeta) {
       logger.warn(
         `Function ${endpointMeta.pikkuFuncId} not found in functionsMeta. Skipping prompt ${name}.`

package/src/utils/workflow/graph/convert-dsl-to-graph.ts

@@ -400,6 +400,7 @@ export function convertDslToGraph(
     source,
     description: meta.description,
     tags: meta.tags,
+    inline: meta.inline,
     context: meta.context,
     nodes: nodesRecord,
     entryNodeIds,

package/src/utils/workflow/graph/workflow-graph.types.ts

@@ -192,6 +192,8 @@ export interface SerializedWorkflowGraph {
   description?: string
   /** Tags for organization */
   tags?: string[]
+  /** If true, workflow always executes inline without queues */
+  inline?: boolean
   /** Workflow context/state variables (from Zod schema) */
   context?: WorkflowContext
   /** Serialized nodes */

package/src/visit.ts

@@ -26,6 +26,7 @@ import { addSecret, addOAuth2Credential } from './add/add-secret.js'
 import { addVariable } from './add/add-variable.js'
 import { addWorkflowGraph } from './add/add-workflow-graph.js'
 import { addAIAgent } from './add/add-ai-agent.js'
+import { addApprovalDescription } from './add/add-approval-description.js'
 
 export const visitSetup = (
   logger: InspectorLogger,
@@ -87,6 +88,7 @@ export const visitSetup = (
   addWireAddon(node, state, logger)
   addMiddleware(logger, node, checker, state, options)
   addPermission(logger, node, checker, state, options)
+  addApprovalDescription(logger, node, checker, state, options)
   addWorkflow(logger, node, checker, state, options)
 
   ts.forEachChild(node, (child) =>