@anyshift/mcp-proxy 0.6.2 → 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,9 +686,11 @@ 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;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@anyshift/mcp-proxy",
-  "version": "0.6.2",
+  "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",