fix(mcp): return tool execution error on jq failure

Author: stainless-app[bot]

packages/mcp-server/src/filtering.ts

@@ -12,3 +12,7 @@ export async function maybeFilter(jqFilter: unknown | undefined, response: any):
 async function jq(json: any, jqFilter: string) {
   return (await initJq).json(json, jqFilter);
 }
+
+export function isJqError(error: any): error is Error {
+  return error instanceof Error && 'stderr' in error;
+}

packages/mcp-server/src/tools/files/list-files.ts

@@ -1,7 +1,7 @@
 // File generated from our OpenAPI spec by Stainless. See CONTRIBUTING.md for details.
 
-import { maybeFilter } from 'scan-documents-mcp/filtering';
-import { Metadata, asTextContentResult } from 'scan-documents-mcp/tools/types';
+import { isJqError, maybeFilter } from 'scan-documents-mcp/filtering';
+import { Metadata, asErrorResult, asTextContentResult } from 'scan-documents-mcp/tools/types';
 
 import { Tool } from '@modelcontextprotocol/sdk/types.js';
 import ScanDocuments from 'scan-documents';
@@ -46,7 +46,14 @@ export const tool: Tool = {
 
 export const handler = async (client: ScanDocuments, args: Record<string, unknown> | undefined) => {
   const { jq_filter, ...body } = args as any;
-  return asTextContentResult(await maybeFilter(jq_filter, await client.files.list(body)));
+  try {
+    return asTextContentResult(await maybeFilter(jq_filter, await client.files.list(body)));
+  } catch (error) {
+    if (isJqError(error)) {
+      return asErrorResult(error.message);
+    }
+    throw error;
+  }
 };
 
 export default { metadata, tool, handler };

packages/mcp-server/src/tools/files/retrieve-files.ts

@@ -1,7 +1,7 @@
 // File generated from our OpenAPI spec by Stainless. See CONTRIBUTING.md for details.
 
-import { maybeFilter } from 'scan-documents-mcp/filtering';
-import { Metadata, asTextContentResult } from 'scan-documents-mcp/tools/types';
+import { isJqError, maybeFilter } from 'scan-documents-mcp/filtering';
+import { Metadata, asErrorResult, asTextContentResult } from 'scan-documents-mcp/tools/types';
 
 import { Tool } from '@modelcontextprotocol/sdk/types.js';
 import ScanDocuments from 'scan-documents';
@@ -41,7 +41,14 @@ export const tool: Tool = {
 
 export const handler = async (client: ScanDocuments, args: Record<string, unknown> | undefined) => {
   const { id, jq_filter, ...body } = args as any;
-  return asTextContentResult(await maybeFilter(jq_filter, await client.files.retrieve(id)));
+  try {
+    return asTextContentResult(await maybeFilter(jq_filter, await client.files.retrieve(id)));
+  } catch (error) {
+    if (isJqError(error)) {
+      return asErrorResult(error.message);
+    }
+    throw error;
+  }
 };
 
 export default { metadata, tool, handler };

packages/mcp-server/src/tools/files/upload-files.ts

@@ -1,7 +1,7 @@
 // File generated from our OpenAPI spec by Stainless. See CONTRIBUTING.md for details.
 
-import { maybeFilter } from 'scan-documents-mcp/filtering';
-import { Metadata, asTextContentResult } from 'scan-documents-mcp/tools/types';
+import { isJqError, maybeFilter } from 'scan-documents-mcp/filtering';
+import { Metadata, asErrorResult, asTextContentResult } from 'scan-documents-mcp/tools/types';
 
 import { Tool } from '@modelcontextprotocol/sdk/types.js';
 import ScanDocuments from 'scan-documents';
@@ -44,7 +44,14 @@ export const tool: Tool = {
 
 export const handler = async (client: ScanDocuments, args: Record<string, unknown> | undefined) => {
   const { jq_filter, ...body } = args as any;
-  return asTextContentResult(await maybeFilter(jq_filter, await client.files.upload(body)));
+  try {
+    return asTextContentResult(await maybeFilter(jq_filter, await client.files.upload(body)));
+  } catch (error) {
+    if (isJqError(error)) {
+      return asErrorResult(error.message);
+    }
+    throw error;
+  }
 };
 
 export default { metadata, tool, handler };

packages/mcp-server/src/tools/types.ts

@@ -87,6 +87,18 @@ export async function asBinaryContentResult(response: Response): Promise<ToolCal
   }
 }
 
+export function asErrorResult(message: string): ToolCallResult {
+  return {
+    content: [
+      {
+        type: 'text',
+        text: message,
+      },
+    ],
+    isError: true,
+  };
+}
+
 export type Metadata = {
   resource: string;
   operation: 'read' | 'write';