Work around schema parsing issue in Cursor

Author: gmethvin

package.json

@@ -71,7 +71,7 @@
   },
   "dependencies": {
     "@alcyone-labs/zod-to-json-schema": "4.0.10",
-    "@iterable/api": "0.5.0",
+    "@iterable/api": "0.5.1",
     "@modelcontextprotocol/sdk": "1.18.1",
     "@primno/dpapi": "2.0.1",
     "@types/json-schema": "7.0.15",

pnpm-lock.yaml

@@ -15,6 +15,17 @@ import { z } from "zod";
 import { createTool } from "../schema-utils.js";
 
 export function createSnippetTools(client: IterableClient): Tool[] {
+  // The union type (string | number) causes Cursor to reject the (valid) JSON schema
+  const identifier: z.ZodString = z
+    .string()
+    .describe(
+      "Snippet ID or name (stringified). Provide either a snippet name (string) or snippet ID (as a string). The value will be converted to the appropriate type."
+    );
+
+  function maybeConvertNumericString(value: string): string | number {
+    return /^\d+$/.test(value) ? Number(value) : value;
+  }
+
   return [
     createTool({
       name: "get_snippets",
@@ -33,22 +44,32 @@ export function createSnippetTools(client: IterableClient): Tool[] {
     createTool({
       name: "get_snippet",
       description: "Get a snippet by ID (numeric) or name (string)",
-      schema: GetSnippetParamsSchema,
-      execute: (params) => client.getSnippet(params),
+      schema: GetSnippetParamsSchema.extend({ identifier }),
+      execute: (params) =>
+        client.getSnippet({
+          identifier: maybeConvertNumericString(params.identifier),
+        }),
     }),
 
     createTool({
       name: "update_snippet",
       description: "Update a snippet by ID (numeric) or name (string)",
-      schema: UpdateSnippetParamsSchema,
-      execute: (params) => client.updateSnippet(params),
+      schema: UpdateSnippetParamsSchema.extend({ identifier }),
+      execute: (params) =>
+        client.updateSnippet({
+          ...params,
+          identifier: maybeConvertNumericString(params.identifier),
+        }),
     }),
 
     createTool({
       name: "delete_snippet",
       description: "Delete a snippet by ID (numeric) or name (string)",
-      schema: DeleteSnippetParamsSchema,
-      execute: (params) => client.deleteSnippet(params),
+      schema: DeleteSnippetParamsSchema.extend({ identifier }),
+      execute: (params) =>
+        client.deleteSnippet({
+          identifier: maybeConvertNumericString(params.identifier),
+        }),
     }),
   ];
 }