@anyshift/mcp-proxy 0.6.1 → 0.6.3

package/dist/index.js

@@ -417,6 +417,51 @@ async function main() {
         console.debug('[mcp-proxy] Standalone mode - no child MCP');
     }
     // ------------------------------------------------------------------------
+    // 2.5. STORE ORIGINAL TOOL SCHEMAS FOR SMART PARAMETER SANITIZATION
+    // ------------------------------------------------------------------------
+    // Create a map of original tool schemas (before proxy param injection)
+    // This allows us to detect if description/isRetryAttempt/originalToolId were
+    // originally part of the child tool's schema (to handle parameter name collisions)
+    const originalToolSchemas = new Map();
+    if (childToolsResponse.tools) {
+        for (const tool of childToolsResponse.tools) {
+            const originalParams = new Set();
+            if (tool.inputSchema?.properties) {
+                for (const paramName of Object.keys(tool.inputSchema.properties)) {
+                    originalParams.add(paramName);
+                }
+            }
+            originalToolSchemas.set(tool.name, originalParams);
+        }
+    }
+    /**
+     * Sanitize tool arguments before forwarding to child MCP.
+     * Removes proxy parameters (description, isRetryAttempt, originalToolId)
+     * ONLY if they weren't originally part of the child tool's schema.
+     * This handles parameter name collisions gracefully.
+     */
+    function sanitizeToolArgs(toolName, args) {
+        const originalParams = originalToolSchemas.get(toolName);
+        if (!originalParams) {
+            // If we don't have schema info, pass through all args (safer than stripping)
+            return args;
+        }
+        const sanitized = {};
+        const proxyParamNames = Object.keys(PROXY_PARAMS);
+        for (const [key, value] of Object.entries(args)) {
+            // Keep the parameter if:
+            // 1. It's NOT a proxy parameter, OR
+            // 2. It WAS in the original tool's schema (collision case)
+            if (!proxyParamNames.includes(key) || originalParams.has(key)) {
+                sanitized[key] = value;
+            }
+            else if (ENABLE_LOGGING) {
+                console.debug(`[mcp-proxy] Stripping proxy parameter '${key}' from ${toolName} call`);
+            }
+        }
+        return sanitized;
+    }
+    // ------------------------------------------------------------------------
     // 3. CREATE PROXY SERVER
     // ------------------------------------------------------------------------
     const server = new Server({
@@ -641,12 +686,18 @@ async function main() {
             if (ENABLE_LOGGING) {
                 console.debug(`[mcp-proxy] Forwarding to child MCP: ${toolName}`);
             }
+            // Sanitize arguments: remove proxy parameters that weren't in original tool schema
+            const sanitizedArgs = sanitizeToolArgs(toolName, toolArgs);
             result = await childClient.callTool({
                 name: toolName,
-                arguments: toolArgs
+                arguments: sanitizedArgs
             });
             // Check if child MCP returned an error
             const childReturnedError = !!result.isError;
+            // Preserve _meta from child response (e.g. parsed_commands) for passthrough
+            const childMeta = result._meta && typeof result._meta === 'object' && Object.keys(result._meta).length > 0
+                ? result._meta
+                : undefined;
             // Process result through file writer to get unified response
             if (result.content && Array.isArray(result.content) && result.content.length > 0) {
                 // Extract text content and embedded resources from all content items
@@ -709,7 +760,8 @@ async function main() {
                                     type: 'text',
                                     text: JSON.stringify(createErrorResponse(tool_id, contentStr, toolArgs), null, 2)
                                 }],
-                            isError: true
+                            isError: true,
+                            ...(childMeta ? { _meta: childMeta } : {})
                         };
                     }
                     // Get unified response from file writer
@@ -750,7 +802,8 @@ async function main() {
                                 type: 'text',
                                 text: JSON.stringify(unifiedResponse, null, 2)
                             }],
-                        isError: !!unifiedResponse.error
+                        isError: !!unifiedResponse.error,
+                        ...(childMeta ? { _meta: childMeta } : {})
                     };
                 }
             }
@@ -761,7 +814,8 @@ async function main() {
                         type: 'text',
                         text: JSON.stringify(createContentResponse(tool_id, result, toolArgs), null, 2)
                     }],
-                isError: childReturnedError
+                isError: childReturnedError,
+                ...(childMeta ? { _meta: childMeta } : {})
             };
         }
         catch (error) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@anyshift/mcp-proxy",
-  "version": "0.6.1",
+  "version": "0.6.3",
   "description": "Generic MCP proxy that adds truncation, file writing, and JQ capabilities to any MCP server",
   "type": "module",
   "main": "dist/index.js",
@@ -19,6 +19,7 @@
   },
   "devDependencies": {
     "@jest/globals": "^30.2.0",
+    "@types/glob": "^9.0.0",
     "@types/jest": "^30.0.0",
     "@types/node": "^22.0.0",
     "jest": "^30.2.0",