@pikku/inspector 0.11.2 → 0.12.0

package/dist/add/add-mcp-resource.js

@@ -2,7 +2,7 @@ import * as ts from 'typescript';
 import { getPropertyValue, getCommonWireMetaData, } from '../utils/get-property-value.js';
 import { extractWireNames } from '../utils/post-process.js';
 import { ensureFunctionMetadata } from '../utils/ensure-function-metadata.js';
-import { extractFunctionName } from '../utils/extract-function-name.js';
+import { extractFunctionName, makeContextBasedId, } from '../utils/extract-function-name.js';
 import { getPropertyAssignmentInitializer } from '../utils/type-utils.js';
 import { resolveMiddleware } from '../utils/middleware.js';
 import { resolvePermissions } from '../utils/permissions.js';
@@ -25,7 +25,9 @@ export const addMCPResource = (logger, node, checker, state, options) => {
         const obj = firstArg;
         const uriValue = getPropertyValue(obj, 'uri');
         const titleValue = getPropertyValue(obj, 'title');
-        const { tags, summary, description, errors } = getCommonWireMetaData(obj, 'MCP resource', uriValue, logger);
+        const { disabled, tags, summary, description, errors } = getCommonWireMetaData(obj, 'MCP resource', uriValue, logger);
+        if (disabled)
+            return;
         const streamingValue = getPropertyValue(obj, 'streaming');
         if (streamingValue === true) {
             logger.warn(`MCP resource '${uriValue}' has streaming enabled, but streaming is not yet supported.`);
@@ -35,9 +37,12 @@ export const addMCPResource = (logger, node, checker, state, options) => {
             logger.critical(ErrorCode.MISSING_FUNC, `No valid 'func' property for MCP resource '${uriValue}'.`);
             return;
         }
-        const pikkuFuncName = extractFunctionName(funcInitializer, checker, state.rootDir).pikkuFuncName;
-        // Ensure function metadata exists (creates stub for inline functions)
-        ensureFunctionMetadata(state, pikkuFuncName, uriValue || undefined);
+        const extracted = extractFunctionName(funcInitializer, checker, state.rootDir);
+        let pikkuFuncId = extracted.pikkuFuncId;
+        if (pikkuFuncId.startsWith('__temp_') && uriValue) {
+            pikkuFuncId = makeContextBasedId('mcp', 'resource', uriValue);
+        }
+        ensureFunctionMetadata(state, pikkuFuncId, uriValue || undefined, funcInitializer, checker, extracted.isHelper);
         if (!uriValue) {
             logger.critical(ErrorCode.MISSING_URI, "MCP resource is missing the required 'uri' property.");
             return;
@@ -51,9 +56,9 @@ export const addMCPResource = (logger, node, checker, state, options) => {
             return;
         }
         // lookup existing function metadata
-        const fnMeta = state.functions.meta[pikkuFuncName];
+        const fnMeta = state.functions.meta[pikkuFuncId];
         if (!fnMeta) {
-            logger.critical(ErrorCode.FUNCTION_METADATA_NOT_FOUND, `No function metadata found for '${pikkuFuncName}'.`);
+            logger.critical(ErrorCode.FUNCTION_METADATA_NOT_FOUND, `No function metadata found for '${pikkuFuncId}'.`);
             return;
         }
         const inputSchema = fnMeta.inputs?.[0] || null;
@@ -63,12 +68,12 @@ export const addMCPResource = (logger, node, checker, state, options) => {
         // --- resolve permissions ---
         const permissions = resolvePermissions(state, obj, tags, checker);
         // --- track used functions/middleware/permissions for service aggregation ---
-        state.serviceAggregation.usedFunctions.add(pikkuFuncName);
+        state.serviceAggregation.usedFunctions.add(pikkuFuncId);
         extractWireNames(middleware).forEach((name) => state.serviceAggregation.usedMiddleware.add(name));
         extractWireNames(permissions).forEach((name) => state.serviceAggregation.usedPermissions.add(name));
         state.mcpEndpoints.files.add(node.getSourceFile().fileName);
         state.mcpEndpoints.resourcesMeta[uriValue] = {
-            pikkuFuncName,
+            pikkuFuncId,
             uri: uriValue,
             title: titleValue,
             description,

package/dist/add/add-middleware.d.ts

@@ -1,5 +1,2 @@
-import { AddWiring } from '../types.js';
-/**
- * Inspect pikkuMiddleware calls, addMiddleware calls, and addHTTPMiddleware calls
- */
+import type { AddWiring } from '../types.js';
 export declare const addMiddleware: AddWiring;