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/classifications/universal/create-classifications-universal.ts

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

packages/mcp-server/src/tools/embeddings/create-embeddings.ts

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

packages/mcp-server/src/tools/extractions/qa/create-extractions-qa.ts

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

packages/mcp-server/src/tools/rerankings/create-rerankings.ts

@@ -1,7 +1,7 @@
 // File generated from our OpenAPI spec by Stainless. See CONTRIBUTING.md for details.
 
-import { maybeFilter } from 'isaacus-mcp/filtering';
-import { Metadata, asTextContentResult } from 'isaacus-mcp/tools/types';
+import { isJqError, maybeFilter } from 'isaacus-mcp/filtering';
+import { Metadata, asErrorResult, asTextContentResult } from 'isaacus-mcp/tools/types';
 
 import { Tool } from '@modelcontextprotocol/sdk/types.js';
 import Isaacus from 'isaacus';
@@ -93,7 +93,14 @@ export const tool: Tool = {
 
 export const handler = async (client: Isaacus, args: Record<string, unknown> | undefined) => {
   const { jq_filter, ...body } = args as any;
-  return asTextContentResult(await maybeFilter(jq_filter, await client.rerankings.create(body)));
+  try {
+    return asTextContentResult(await maybeFilter(jq_filter, await client.rerankings.create(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';