@anyshift/mcp-proxy 0.3.5 → 0.3.6

package/dist/index.js

@@ -50,6 +50,13 @@ function injectProxyParams(tool) {
             modifiedTool.inputSchema.properties[key] = value;
         }
     }
+    // Ensure isRetryAttempt is required (not optional)
+    if (!modifiedTool.inputSchema.required) {
+        modifiedTool.inputSchema.required = [];
+    }
+    if (!modifiedTool.inputSchema.required.includes('isRetryAttempt')) {
+        modifiedTool.inputSchema.required = [...modifiedTool.inputSchema.required, 'isRetryAttempt'];
+    }
     return modifiedTool;
 }
 /**
@@ -64,6 +71,18 @@ function addRetryMetadata(response, toolArgs) {
     }
     return response;
 }
+/**
+ * Create an error response with consistent structure
+ */
+function createErrorResponse(tool_id, error, toolArgs) {
+    return addRetryMetadata({ tool_id, wroteToFile: false, error }, toolArgs);
+}
+/**
+ * Create a success response with content (not written to file)
+ */
+function createContentResponse(tool_id, outputContent, toolArgs) {
+    return addRetryMetadata({ tool_id, wroteToFile: false, outputContent }, toolArgs);
+}
 /**
  * ENVIRONMENT VARIABLE CONTRACT
  * =============================
@@ -351,11 +370,7 @@ async function main() {
                 });
                 // JQ tool returns directly, wrap in unified format
                 const tool_id = generateToolId(toolName, toolArgs, fileWriterConfig.toolAbbreviations);
-                const unifiedResponse = addRetryMetadata({
-                    tool_id,
-                    wroteToFile: false,
-                    outputContent: result.content?.[0]?.text
-                }, toolArgs);
+                const unifiedResponse = createContentResponse(tool_id, result.content?.[0]?.text, toolArgs);
                 return {
                     content: [{
                             type: 'text',
@@ -367,15 +382,10 @@ async function main() {
             // Forward all other tools to child MCP (if child exists)
             if (!childClient) {
                 const tool_id = generateToolId(toolName, toolArgs, fileWriterConfig.toolAbbreviations);
-                const errorResponse = addRetryMetadata({
-                    tool_id,
-                    wroteToFile: false,
-                    error: `Tool ${toolName} not available in standalone mode (no child MCP)`
-                }, toolArgs);
                 return {
                     content: [{
                             type: 'text',
-                            text: JSON.stringify(errorResponse, null, 2)
+                            text: JSON.stringify(createErrorResponse(tool_id, `Tool ${toolName} not available in standalone mode (no child MCP)`, toolArgs), null, 2)
                         }],
                     isError: true
                 };
@@ -397,18 +407,13 @@ async function main() {
                     // If child returned error, pass through directly without file writing
                     if (childReturnedError) {
                         const tool_id = generateToolId(toolName, toolArgs, fileWriterConfig.toolAbbreviations);
-                        const errorResponse = addRetryMetadata({
-                            tool_id,
-                            wroteToFile: false,
-                            error: item.text
-                        }, toolArgs);
                         if (ENABLE_LOGGING) {
                             console.debug(`[mcp-proxy] Child MCP returned error for ${toolName}: ${item.text.substring(0, 100)}...`);
                         }
                         return {
                             content: [{
                                     type: 'text',
-                                    text: JSON.stringify(errorResponse, null, 2)
+                                    text: JSON.stringify(createErrorResponse(tool_id, item.text, toolArgs), null, 2)
                                 }],
                             isError: true
                         };
@@ -457,15 +462,10 @@ async function main() {
             }
             // Fallback: return result with generated tool_id
             const tool_id = generateToolId(toolName, toolArgs, fileWriterConfig.toolAbbreviations);
-            const fallbackResponse = addRetryMetadata({
-                tool_id,
-                wroteToFile: false,
-                outputContent: result
-            }, toolArgs);
             return {
                 content: [{
                         type: 'text',
-                        text: JSON.stringify(fallbackResponse, null, 2)
+                        text: JSON.stringify(createContentResponse(tool_id, result, toolArgs), null, 2)
                     }],
                 isError: childReturnedError
             };
@@ -473,15 +473,10 @@ async function main() {
         catch (error) {
             console.error(`[mcp-proxy] Error executing tool ${toolName}:`, error);
             const tool_id = generateToolId(toolName, toolArgs, fileWriterConfig.toolAbbreviations);
-            const errorResponse = addRetryMetadata({
-                tool_id,
-                wroteToFile: false,
-                error: `Error executing ${toolName}: ${error.message || String(error)}`
-            }, toolArgs);
             return {
                 content: [{
                         type: 'text',
-                        text: JSON.stringify(errorResponse, null, 2)
+                        text: JSON.stringify(createErrorResponse(tool_id, `Error executing ${toolName}: ${error.message || String(error)}`, toolArgs), null, 2)
                     }],
                 isError: true
             };

package/dist/jq/tool.d.ts

@@ -1,25 +1,19 @@
 import { z } from 'zod';
 /**
  * Zod schema for JQ query execution
+ * Note: Proxy params (description, isRetryAttempt, originalToolId) are handled
+ * by the proxy layer, not validated here. They're defined in JQ_TOOL_DEFINITION's
+ * inputSchema for LLM visibility.
  */
 export declare const ExecuteJqQuerySchema: z.ZodObject<{
     jq_query: z.ZodString;
     file_path: z.ZodString;
-    description: z.ZodOptional<z.ZodString>;
-    isRetryAttempt: z.ZodOptional<z.ZodBoolean>;
-    originalToolId: z.ZodOptional<z.ZodString>;
 }, "strip", z.ZodTypeAny, {
     jq_query: string;
     file_path: string;
-    description?: string | undefined;
-    isRetryAttempt?: boolean | undefined;
-    originalToolId?: string | undefined;
 }, {
     jq_query: string;
     file_path: string;
-    description?: string | undefined;
-    isRetryAttempt?: boolean | undefined;
-    originalToolId?: string | undefined;
 }>;
 /**
  * Tool definition for JQ query execution with enhanced prompts

package/dist/jq/tool.js

@@ -2,6 +2,9 @@ import { z } from 'zod';
 import { PROXY_PARAMS } from '../types/index.js';
 /**
  * Zod schema for JQ query execution
+ * Note: Proxy params (description, isRetryAttempt, originalToolId) are handled
+ * by the proxy layer, not validated here. They're defined in JQ_TOOL_DEFINITION's
+ * inputSchema for LLM visibility.
  */
 export const ExecuteJqQuerySchema = z.object({
     jq_query: z
@@ -10,9 +13,6 @@ export const ExecuteJqQuerySchema = z.object({
     file_path: z
         .string()
         .describe('Absolute path starting with "/" pointing to the JSON or JSONL file to process. Must be a valid, existing file with .json or .jsonl extension. The file will be validated for existence and readability before processing.'),
-    description: z.string().optional().describe(PROXY_PARAMS.description.description),
-    isRetryAttempt: z.boolean().optional().describe(PROXY_PARAMS.isRetryAttempt.description),
-    originalToolId: z.string().optional().describe(PROXY_PARAMS.originalToolId.description),
 });
 /**
  * Tool definition for JQ query execution with enhanced prompts
@@ -113,6 +113,6 @@ export const JQ_TOOL_DEFINITION = {
             },
             ...PROXY_PARAMS,
         },
-        required: ['jq_query', 'file_path'],
+        required: ['jq_query', 'file_path', 'isRetryAttempt'],
     },
 };

package/dist/types/index.d.ts

@@ -53,27 +53,6 @@ export interface FileWriterResult {
         text: string;
     }>;
 }
-/**
- * Schema analysis result for a JSON structure
- */
-export interface JsonSchema {
-    type: string;
-    properties?: Record<string, unknown>;
-    items?: unknown;
-    length?: number;
-    _hasNulls?: boolean;
-    _keysAreNumeric?: boolean;
-    _accessPattern?: string;
-}
-/**
- * Nullable fields extracted from schema
- */
-export interface NullableFields {
-    /** Fields that are always null */
-    alwaysNull: string[];
-    /** Fields that can be null (mixed types) */
-    nullable: string[];
-}
 /**
  * Unified response format for all tool calls
  * Provides a consistent structure for LLM consumption

package/dist/utils/filename.d.ts

@@ -6,11 +6,3 @@
  * @returns Tool ID like "1697834567123_met_qry_a3b4c5"
  */
 export declare const generateToolId: (toolName: string, args: Record<string, unknown>, toolAbbreviations?: Record<string, string>) => string;
-/**
- * Generate LLM-friendly compact filename
- * @param toolName - Name of the tool that generated the data
- * @param args - Arguments passed to the tool
- * @param toolAbbreviations - Optional custom abbreviations for tool names
- * @returns Compact filename like "1697834567123_met_qry_a3b4c5.json"
- */
-export declare const generateCompactFilename: (toolName: string, args: Record<string, unknown>, toolAbbreviations?: Record<string, string>) => string;

package/dist/utils/filename.js

@@ -40,13 +40,3 @@ export const generateToolId = (toolName, args, toolAbbreviations) => {
     const argsHash = hashArgs(args);
     return `${timestamp}_${toolAbbrev}_${argsHash}`;
 };
-/**
- * Generate LLM-friendly compact filename
- * @param toolName - Name of the tool that generated the data
- * @param args - Arguments passed to the tool
- * @param toolAbbreviations - Optional custom abbreviations for tool names
- * @returns Compact filename like "1697834567123_met_qry_a3b4c5.json"
- */
-export const generateCompactFilename = (toolName, args, toolAbbreviations) => {
-    return `${generateToolId(toolName, args, toolAbbreviations)}.json`;
-};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@anyshift/mcp-proxy",
-  "version": "0.3.5",
+  "version": "0.3.6",
   "description": "Generic MCP proxy that adds truncation, file writing, and JQ capabilities to any MCP server",
   "type": "module",
   "main": "dist/index.js",