Update tools/call input validation errors to be tool execution errors

src/server/mcp.test.ts

@@ -937,24 +937,6 @@ describe("tool()", () => {
       }),
     );
 
-    mcpServer.registerTool(
-      "test (new api)",
-      {
-        inputSchema: {
-          name: z.string(),
-          value: z.number(),
-        },
-      },
-      async ({ name, value }) => ({
-        content: [
-          {
-            type: "text",
-            text: `${name}: ${value}`,
-          },
-        ],
-      })
-    );
-
     const [clientTransport, serverTransport] =
       InMemoryTransport.createLinkedPair();
 
@@ -977,23 +959,15 @@ describe("tool()", () => {
         },
         CallToolResultSchema,
       ),
-    ).rejects.toThrow(/Invalid arguments/);
-
-    await expect(
-      client.request(
+    ).resolves.toStrictEqual({
+      content: [
         {
-          method: "tools/call",
-          params: {
-            name: "test (new api)",
-            arguments: {
-              name: "test",
-              value: "not a number",
-            },
-          },
+          type: "text",
+          text: expect.stringMatching(/Invalid arguments for tool test/),
         },
-        CallToolResultSchema,
-      ),
-    ).rejects.toThrow(/Invalid arguments/);
+      ],
+      isError: true,
+    });
   });
 
   /***

src/server/mcp.ts

@@ -162,10 +162,15 @@ export class McpServer {
             request.params.arguments,
           );
           if (!parseResult.success) {
-            throw new McpError(
-              ErrorCode.InvalidParams,
-              `Invalid arguments for tool ${request.params.name}: ${parseResult.error.message}`,
-            );
+            return {
+              content: [
+                {
+                  type: "text",
+                  text: `Invalid arguments for tool ${request.params.name}: ${parseResult.error.message}`,
+                },
+              ],
+              isError: true,
+            };
           }
 
           const args = parseResult.data;