@pikku/inspector 0.8.0 → 0.8.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pikku/inspector",
-  "version": "0.8.0",
+  "version": "0.8.1",
   "author": "yasser.fadl@gmail.com",
   "license": "MIT",
   "type": "module",
@@ -17,7 +17,7 @@
     "test:coverage": "bash run-tests.sh --coverage"
   },
   "dependencies": {
-    "@pikku/core": "^0.8.0",
+    "@pikku/core": "^0.8.1",
     "path-to-regexp": "^8.2.0",
     "typescript": "^5.6"
   },

package/src/add-channel.ts

@@ -8,7 +8,11 @@ import {
   matchesFilters,
 } from './utils.js'
 import type { ChannelMessageMeta, ChannelMeta } from '@pikku/core/channel'
-import type { InspectorFilters, InspectorState } from './types.js'
+import type {
+  InspectorFilters,
+  InspectorState,
+  InspectorLogger,
+} from './types.js'
 
 /**
  * Safely get the “initializer” expression of a property-like AST node:
@@ -363,7 +367,8 @@ export function addChannel(
   node: ts.Node,
   checker: ts.TypeChecker,
   state: InspectorState,
-  filters: InspectorFilters
+  filters: InspectorFilters,
+  logger: InspectorLogger
 ) {
   if (!ts.isCallExpression(node)) return
   const { expression, arguments: args } = node
@@ -391,8 +396,15 @@ export function addChannel(
   const tags = getPropertyValue(obj, 'tags') as string[] | undefined
   const query = getPropertyValue(obj, 'query') as string[] | []
 
+  const filePath = node.getSourceFile().fileName
+
   if (
-    !matchesFilters(filters, { tags }, { type: PikkuEventTypes.channel, name })
+    !matchesFilters(
+      filters,
+      { tags },
+      { type: PikkuEventTypes.channel, name, filePath },
+      logger
+    )
   )
     return
 

package/src/add-functions.ts

@@ -1,5 +1,5 @@
 import * as ts from 'typescript'
-import { InspectorState, InspectorFilters } from './types.js'
+import { InspectorState, InspectorFilters, InspectorLogger } from './types.js'
 import { TypesMap } from './types-map.js'
 import {
   extractFunctionName,
@@ -231,7 +231,8 @@ export function addFunctions(
   node: ts.Node,
   checker: ts.TypeChecker,
   state: InspectorState,
-  filters: InspectorFilters
+  filters: InspectorFilters,
+  logger: InspectorLogger
 ) {
   if (!ts.isCallExpression(node)) return
 
@@ -305,10 +306,7 @@ export function addFunctions(
           services.services.push(original)
         }
       }
-    } else if (
-      ts.isIdentifier(firstParam.name) &&
-      !firstParam.name.text.startsWith('_')
-    ) {
+    } else if (ts.isIdentifier(firstParam.name)) {
       services.optimized = false
     }
   }

package/src/add-http-route.ts

@@ -8,7 +8,7 @@ import {
   getPropertyAssignmentInitializer,
   matchesFilters,
 } from './utils.js'
-import { InspectorState, InspectorFilters } from './types.js'
+import { InspectorState, InspectorFilters, InspectorLogger } from './types.js'
 
 /**
  * Populate metaInputTypes for a given route based on method, input type,
@@ -43,7 +43,8 @@ export const addHTTPRoute = (
   node: ts.Node,
   checker: ts.TypeChecker,
   state: InspectorState,
-  filters: InspectorFilters
+  filters: InspectorFilters,
+  logger: InspectorLogger
 ) => {
   // only look at calls
   if (!ts.isCallExpression(node)) return
@@ -69,11 +70,14 @@ export const addHTTPRoute = (
   const tags = (getPropertyValue(obj, 'tags') as string[]) || undefined
   const query = (getPropertyValue(obj, 'query') as string[]) || []
 
+  const filePath = node.getSourceFile().fileName
+
   if (
     !matchesFilters(
       filters,
       { tags },
-      { type: PikkuEventTypes.http, name: route }
+      { type: PikkuEventTypes.http, name: route, filePath },
+      logger
     )
   ) {
     return

package/src/add-mcp-prompt.ts

@@ -1,7 +1,7 @@
 import * as ts from 'typescript'
 import { getPropertyValue } from './get-property-value.js'
 import { PikkuEventTypes } from '@pikku/core'
-import { InspectorFilters, InspectorState } from './types.js'
+import { InspectorFilters, InspectorState, InspectorLogger } from './types.js'
 import {
   extractFunctionName,
   getPropertyAssignmentInitializer,
@@ -12,7 +12,8 @@ export const addMCPPrompt = (
   node: ts.Node,
   checker: ts.TypeChecker,
   state: InspectorState,
-  filters: InspectorFilters
+  filters: InspectorFilters,
+  logger: InspectorLogger
 ) => {
   if (!ts.isCallExpression(node)) {
     return
@@ -66,11 +67,14 @@ export const addMCPPrompt = (
       return
     }
 
+    const filePath = node.getSourceFile().fileName
+
     if (
       !matchesFilters(
         filters,
         { tags },
-        { type: PikkuEventTypes.mcp, name: nameValue }
+        { type: PikkuEventTypes.mcp, name: nameValue, filePath },
+        logger
       )
     ) {
       return

package/src/add-mcp-resource.ts

@@ -1,7 +1,7 @@
 import * as ts from 'typescript'
 import { getPropertyValue } from './get-property-value.js'
 import { PikkuEventTypes } from '@pikku/core'
-import { InspectorFilters, InspectorState } from './types.js'
+import { InspectorFilters, InspectorState, InspectorLogger } from './types.js'
 import {
   extractFunctionName,
   getPropertyAssignmentInitializer,
@@ -12,7 +12,8 @@ export const addMCPResource = (
   node: ts.Node,
   checker: ts.TypeChecker,
   state: InspectorState,
-  filters: InspectorFilters
+  filters: InspectorFilters,
+  logger: InspectorLogger
 ) => {
   if (!ts.isCallExpression(node)) {
     return
@@ -77,11 +78,14 @@ export const addMCPResource = (
       return
     }
 
+    const filePath = node.getSourceFile().fileName
+
     if (
       !matchesFilters(
         filters,
         { tags },
-        { type: PikkuEventTypes.mcp, name: uriValue }
+        { type: PikkuEventTypes.mcp, name: uriValue, filePath },
+        logger
       )
     ) {
       return

package/src/add-mcp-tool.ts

@@ -1,7 +1,7 @@
 import * as ts from 'typescript'
 import { getPropertyValue } from './get-property-value.js'
 import { PikkuEventTypes } from '@pikku/core'
-import { InspectorFilters, InspectorState } from './types.js'
+import { InspectorFilters, InspectorState, InspectorLogger } from './types.js'
 import {
   extractFunctionName,
   getPropertyAssignmentInitializer,
@@ -12,7 +12,8 @@ export const addMCPTool = (
   node: ts.Node,
   checker: ts.TypeChecker,
   state: InspectorState,
-  filters: InspectorFilters
+  filters: InspectorFilters,
+  logger: InspectorLogger
 ) => {
   if (!ts.isCallExpression(node)) {
     return
@@ -68,11 +69,14 @@ export const addMCPTool = (
       return
     }
 
+    const filePath = node.getSourceFile().fileName
+
     if (
       !matchesFilters(
         filters,
         { tags },
-        { type: PikkuEventTypes.mcp, name: nameValue }
+        { type: PikkuEventTypes.mcp, name: nameValue, filePath },
+        logger
       )
     ) {
       return

package/src/add-queue-worker.ts

@@ -1,7 +1,7 @@
 import * as ts from 'typescript'
 import { getPropertyValue } from './get-property-value.js'
 import { APIDocs, PikkuEventTypes } from '@pikku/core'
-import { InspectorFilters, InspectorState } from './types.js'
+import { InspectorFilters, InspectorState, InspectorLogger } from './types.js'
 import {
   extractFunctionName,
   getPropertyAssignmentInitializer,
@@ -12,7 +12,8 @@ export const addQueueWorker = (
   node: ts.Node,
   checker: ts.TypeChecker,
   state: InspectorState,
-  filters: InspectorFilters
+  filters: InspectorFilters,
+  logger: InspectorLogger
 ) => {
   if (!ts.isCallExpression(node)) {
     return
@@ -64,11 +65,14 @@ export const addQueueWorker = (
       return
     }
 
+    const filePath = node.getSourceFile().fileName
+
     if (
       !matchesFilters(
         filters,
         { tags },
-        { type: PikkuEventTypes.queue, name: queueName }
+        { type: PikkuEventTypes.queue, name: queueName, filePath },
+        logger
       )
     ) {
       console.info(

package/src/add-schedule.ts

@@ -1,7 +1,7 @@
 import * as ts from 'typescript'
 import { getPropertyValue } from './get-property-value.js'
 import { APIDocs, PikkuEventTypes } from '@pikku/core'
-import { InspectorFilters, InspectorState } from './types.js'
+import { InspectorFilters, InspectorState, InspectorLogger } from './types.js'
 import {
   extractFunctionName,
   getPropertyAssignmentInitializer,
@@ -12,7 +12,8 @@ export const addSchedule = (
   node: ts.Node,
   checker: ts.TypeChecker,
   state: InspectorState,
-  filters: InspectorFilters
+  filters: InspectorFilters,
+  logger: InspectorLogger
 ) => {
   if (!ts.isCallExpression(node)) {
     return
@@ -61,11 +62,14 @@ export const addSchedule = (
       return
     }
 
+    const filePath = node.getSourceFile().fileName
+
     if (
       !matchesFilters(
         filters,
         { tags },
-        { type: PikkuEventTypes.scheduled, name: nameValue }
+        { type: PikkuEventTypes.scheduler, name: nameValue, filePath },
+        logger
       )
     ) {
       return

package/src/inspector.ts

@@ -5,6 +5,7 @@ import {
   InspectorState,
   InspectorHTTPState,
   InspectorFilters,
+  InspectorLogger,
 } from './types.js'
 
 export const normalizeHTTPTypes = (
@@ -14,6 +15,7 @@ export const normalizeHTTPTypes = (
 }
 
 export const inspect = (
+  logger: InspectorLogger,
   routeFiles: string[],
   filters: InspectorFilters
 ): InspectorState => {
@@ -67,14 +69,14 @@ export const inspect = (
   // First sweep: add all functions
   for (const sourceFile of sourceFiles) {
     ts.forEachChild(sourceFile, (child) =>
-      visitSetup(checker, child, state, filters)
+      visitSetup(checker, child, state, filters, logger)
     )
   }
 
   // Second sweep: add all transports
   for (const sourceFile of sourceFiles) {
     ts.forEachChild(sourceFile, (child) =>
-      visitRoutes(checker, child, state, filters)
+      visitRoutes(checker, child, state, filters, logger)
     )
   }
 

package/src/types.ts

@@ -40,6 +40,15 @@ export interface InspectorChannelState {
 
 export type InspectorFilters = {
   tags?: string[]
+  types?: string[]
+  directories?: string[]
+}
+
+export interface InspectorLogger {
+  info: (message: string) => void
+  error: (message: string) => void
+  warn: (message: string) => void
+  debug: (message: string) => void
 }
 export interface InspectorState {
   singletonServicesTypeImportMap: PathToNameAndType