Update handler functions (#2340)

Co-authored-by: alisonlhart <[email protected]>
Co-authored-by: Shatakshi Mishra <[email protected]>

Author: 3 people

packages/ansible-mcp-server/package.json

@@ -1,7 +1,7 @@
 {
   "bin": "out/server/src/cli.js",
   "dependencies": {
-    "@modelcontextprotocol/sdk": "^1.17.3",
+    "@modelcontextprotocol/sdk": "1.24.2",
     "ajv": "^8.17.1",
     "ajv-formats": "^3.0.1",
     "yaml": "^2.8.1",

packages/ansible-mcp-server/src/server.ts

@@ -855,14 +855,14 @@ export function createAnsibleMcpServer(workspaceRoot: string) {
       // Execute the tool handler
       // eslint-disable-next-line @typescript-eslint/no-explicit-any
       const handlers = (server as any)._registeredTools;
-      if (handlers && handlers[toolName]?.callback) {
+      if (handlers && handlers[toolName]?.handler) {
         // Pass workspaceRoot to handlers that need it (like ansible_navigator)
         const handlerArgs = request.params.arguments || {};
 
         if (toolName === "ansible_navigator") {
-          return await handlers[toolName].callback(handlerArgs, workspaceRoot);
+          return await handlers[toolName].handler(handlerArgs, workspaceRoot);
         }
-        return await handlers[toolName].callback(handlerArgs);
+        return await handlers[toolName].handler(handlerArgs);
       }
 
       // Log available tools for debugging

packages/ansible-mcp-server/test/testWrapper.ts

@@ -20,7 +20,7 @@ export function createTestServer(workspaceRoot: string) {
       }
 
       // Call the tool handler
-      const handler = registeredTools[name].callback;
+      const handler = registeredTools[name].handler;
       if (handler) {
         return await handler(args);
       }

yarn.lock

@@ -51,7 +51,7 @@ __metadata:
   version: 0.0.0-use.local
   resolution: "@ansible/ansible-mcp-server@workspace:packages/ansible-mcp-server"
   dependencies:
-    "@modelcontextprotocol/sdk": "npm:^1.17.3"
+    "@modelcontextprotocol/sdk": "npm:1.24.2"
     "@types/node": "npm:^24.10.1"
     "@vitest/coverage-v8": "npm:^4.0.14"
     "@vitest/ui": "npm:^4.0.4"
@@ -1229,9 +1229,9 @@ __metadata:
   languageName: node
   linkType: hard
 
-"@modelcontextprotocol/sdk@npm:^1.17.3":
-  version: 1.22.0
-  resolution: "@modelcontextprotocol/sdk@npm:1.22.0"
+"@modelcontextprotocol/sdk@npm:1.24.2":
+  version: 1.24.2
+  resolution: "@modelcontextprotocol/sdk@npm:1.24.2"
   dependencies:
     ajv: "npm:^8.17.1"
     ajv-formats: "npm:^3.0.1"
@@ -1242,16 +1242,20 @@ __metadata:
     eventsource-parser: "npm:^3.0.0"
     express: "npm:^5.0.1"
     express-rate-limit: "npm:^7.5.0"
+    jose: "npm:^6.1.1"
     pkce-challenge: "npm:^5.0.0"
     raw-body: "npm:^3.0.0"
-    zod: "npm:^3.23.8"
-    zod-to-json-schema: "npm:^3.24.1"
+    zod: "npm:^3.25 || ^4.0"
+    zod-to-json-schema: "npm:^3.25.0"
   peerDependencies:
     "@cfworker/json-schema": ^4.1.1
+    zod: ^3.25 || ^4.0
   peerDependenciesMeta:
     "@cfworker/json-schema":
       optional: true
-  checksum: 10/938a50a0e39806edef797cfa0ea8ca89bb90a855ffc50dac03fec5dcf15b18dff8fa0eb89a3adde2135cae0b7277e74f57826f1a0a134a4ae444c8d6b5246e68
+    zod:
+      optional: false
+  checksum: 10/3411ade6e64188eeb7c68946a08f65040a5604f7bd7fe828647d73d037103214d8372f2e9f48c988bfec55dc273be8b4915498bc820bed0a61c0d53d0a5dc8a0
   languageName: node
   linkType: hard
 
@@ -7071,6 +7075,13 @@ __metadata:
   languageName: node
   linkType: hard
 
+"jose@npm:^6.1.1":
+  version: 6.1.3
+  resolution: "jose@npm:6.1.3"
+  checksum: 10/9626c51e8c3792b505e954f3094698c182208617b62dfb27269230f31e57560b083985ed8128b8a9753aa92daf18d3a2341cc826d149503f14569abe87d42389
+  languageName: node
+  linkType: hard
+
 "joycon@npm:^3.1.1":
   version: 3.1.1
   resolution: "joycon@npm:3.1.1"
@@ -11757,7 +11768,7 @@ __metadata:
   languageName: node
   linkType: hard
 
-"zod-to-json-schema@npm:^3.24.1":
+"zod-to-json-schema@npm:^3.25.0":
   version: 3.25.0
   resolution: "zod-to-json-schema@npm:3.25.0"
   peerDependencies:
@@ -11772,3 +11783,10 @@ __metadata:
   checksum: 10/f0c963ec40cd96858451d1690404d603d36507c1fc9682f2dae59ab38b578687d542708a7fdbf645f77926f78c9ed558f57c3d3aa226c285f798df0c4da16995
   languageName: node
   linkType: hard
+
+"zod@npm:^3.25 || ^4.0":
+  version: 4.1.13
+  resolution: "zod@npm:4.1.13"
+  checksum: 10/0679190318928f69fcb07751063719de232c663b13955fcdb55db59839569d39f3f29b955cb0cba7af0b724233f88c06b3e84c550397ad4e68f8088fa6799d88
+  languageName: node
+  linkType: hard