refactor: Rename sendCallTool/sendListTools to callTool/listTools

Avoid double-verb naming pattern for consistency with existing API.

Author: ochafik

src/app-bridge.test.ts

@@ -742,7 +742,7 @@ describe("App <-> AppBridge integration", () => {
       await bridge.connect(bridgeTransport);
     });
 
-    it("bridge.sendCallTool calls app.oncalltool handler", async () => {
+    it("bridge.callTool calls app.oncalltool handler", async () => {
       // App needs tool capabilities to handle tool calls
       const appCapabilities = { tools: {} };
       app = new App(testAppInfo, appCapabilities, { autoResize: false });
@@ -758,7 +758,7 @@ describe("App <-> AppBridge integration", () => {
 
       await app.connect(appTransport);
 
-      const result = await bridge.sendCallTool({
+      const result = await bridge.callTool({
         name: "test-tool",
         arguments: { foo: "bar" },
       });
@@ -773,7 +773,7 @@ describe("App <-> AppBridge integration", () => {
       ]);
     });
 
-    it("bridge.sendListTools calls app.onlisttools handler", async () => {
+    it("bridge.listTools calls app.onlisttools handler", async () => {
       // App needs tool capabilities to handle tool list requests
       const appCapabilities = { tools: {} };
       app = new App(testAppInfo, appCapabilities, { autoResize: false });
@@ -805,7 +805,7 @@ describe("App <-> AppBridge integration", () => {
 
       await app.connect(appTransport);
 
-      const result = await bridge.sendListTools({});
+      const result = await bridge.listTools({});
 
       expect(receivedCalls).toHaveLength(1);
       expect(result.tools).toHaveLength(3);
@@ -831,7 +831,7 @@ describe("App <-> AppBridge integration", () => {
       await bridge.connect(bridgeTransport);
       await app.connect(appTransport);
 
-      await bridge.sendCallTool({
+      await bridge.callTool({
         name: "test-tool",
         arguments: {},
       });
@@ -864,7 +864,7 @@ describe("App <-> AppBridge integration", () => {
       await bridge.connect(bridgeTransport);
       await app.connect(appTransport);
 
-      const result = await bridge.sendCallTool({
+      const result = await bridge.callTool({
         name: "greet",
         arguments: { name: "Alice" },
       });
@@ -898,7 +898,7 @@ describe("App <-> AppBridge integration", () => {
         await app.connect(appTransport);
 
         // Call the tool through bridge - should work automatically
-        const result = await bridge.sendCallTool({
+        const result = await bridge.callTool({
           name: "greet",
           arguments: { name: "Bob" },
         });
@@ -919,7 +919,7 @@ describe("App <-> AppBridge integration", () => {
 
         // Try to call a tool that doesn't exist
         await expect(
-          bridge.sendCallTool({
+          bridge.callTool({
             name: "nonexistent",
             arguments: {},
           }),
@@ -968,7 +968,7 @@ describe("App <-> AppBridge integration", () => {
         await app.connect(appTransport);
 
         // Call first tool
-        const addResult = await bridge.sendCallTool({
+        const addResult = await bridge.callTool({
           name: "add",
           arguments: { a: 5, b: 3 },
         });
@@ -977,7 +977,7 @@ describe("App <-> AppBridge integration", () => {
         ]);
 
         // Call second tool
-        const multiplyResult = await bridge.sendCallTool({
+        const multiplyResult = await bridge.callTool({
           name: "multiply",
           arguments: { a: 5, b: 3 },
         });
@@ -1004,15 +1004,15 @@ describe("App <-> AppBridge integration", () => {
 
         // Should work when enabled
         await expect(
-          bridge.sendCallTool({ name: "test-tool", arguments: {} }),
+          bridge.callTool({ name: "test-tool", arguments: {} }),
         ).resolves.toBeDefined();
 
         // Disable tool
         tool.disable();
 
         // Should throw when disabled
         await expect(
-          bridge.sendCallTool({ name: "test-tool", arguments: {} }),
+          bridge.callTool({ name: "test-tool", arguments: {} }),
         ).rejects.toThrow("Tool test-tool is disabled");
       });
 
@@ -1038,15 +1038,15 @@ describe("App <-> AppBridge integration", () => {
 
         // Valid input should work
         await expect(
-          bridge.sendCallTool({
+          bridge.callTool({
             name: "strict-tool",
             arguments: { required: "hello" },
           }),
         ).resolves.toBeDefined();
 
         // Invalid input should fail
         await expect(
-          bridge.sendCallTool({
+          bridge.callTool({
             name: "strict-tool",
             arguments: { wrong: "field" },
           }),
@@ -1074,7 +1074,7 @@ describe("App <-> AppBridge integration", () => {
         await app.connect(appTransport);
 
         // Valid output should work
-        const result = await bridge.sendCallTool({
+        const result = await bridge.callTool({
           name: "validated-output",
           arguments: {},
         });
@@ -1096,7 +1096,7 @@ describe("App <-> AppBridge integration", () => {
         await app.connect(appTransport);
 
         // First version
-        let result = await bridge.sendCallTool({
+        let result = await bridge.callTool({
           name: "dynamic-tool",
           arguments: {},
         });
@@ -1107,7 +1107,7 @@ describe("App <-> AppBridge integration", () => {
 
         // Should fail after removal
         await expect(
-          bridge.sendCallTool({ name: "dynamic-tool", arguments: {} }),
+          bridge.callTool({ name: "dynamic-tool", arguments: {} }),
         ).rejects.toThrow("Tool dynamic-tool not found");
 
         // Re-register with different behavior
@@ -1116,7 +1116,7 @@ describe("App <-> AppBridge integration", () => {
         }));
 
         // Should work with new version
-        result = await bridge.sendCallTool({
+        result = await bridge.callTool({
           name: "dynamic-tool",
           arguments: {},
         });
@@ -1142,7 +1142,7 @@ describe("App <-> AppBridge integration", () => {
 
         await app.connect(appTransport);
 
-        const result = await bridge.sendListTools({});
+        const result = await bridge.listTools({});
 
         expect(result.tools).toHaveLength(3);
         expect(result.tools.map((t) => t.name)).toContain("tool1");
@@ -1165,7 +1165,7 @@ describe("App <-> AppBridge integration", () => {
         // Remove the tool after connecting
         dummyTool.remove();
 
-        const result = await bridge.sendListTools({});
+        const result = await bridge.listTools({});
 
         expect(result.tools).toEqual([]);
       });
@@ -1184,7 +1184,7 @@ describe("App <-> AppBridge integration", () => {
         dummy.remove();
 
         // Initially no tools
-        let result = await bridge.sendListTools({});
+        let result = await bridge.listTools({});
         expect(result.tools).toEqual([]);
 
         // Add a tool
@@ -1193,7 +1193,7 @@ describe("App <-> AppBridge integration", () => {
         }));
 
         // Should now include the new tool
-        result = await bridge.sendListTools({});
+        result = await bridge.listTools({});
         expect(result.tools.map((t) => t.name)).toEqual(["new-tool"]);
 
         // Add another tool
@@ -1202,7 +1202,7 @@ describe("App <-> AppBridge integration", () => {
         }));
 
         // Should now include both tools
-        result = await bridge.sendListTools({});
+        result = await bridge.listTools({});
         expect(result.tools).toHaveLength(2);
         expect(result.tools.map((t) => t.name)).toContain("new-tool");
         expect(result.tools.map((t) => t.name)).toContain("another-tool");
@@ -1225,14 +1225,14 @@ describe("App <-> AppBridge integration", () => {
         await app.connect(appTransport);
 
         // Initially all three tools
-        let result = await bridge.sendListTools({});
+        let result = await bridge.listTools({});
         expect(result.tools).toHaveLength(3);
 
         // Remove one tool
         tool2.remove();
 
         // Should now have two tools
-        result = await bridge.sendListTools({});
+        result = await bridge.listTools({});
         expect(result.tools).toHaveLength(2);
         expect(result.tools.map((t) => t.name)).toContain("tool1");
         expect(result.tools.map((t) => t.name)).toContain("tool3");
@@ -1242,7 +1242,7 @@ describe("App <-> AppBridge integration", () => {
         tool1.remove();
 
         // Should now have one tool
-        result = await bridge.sendListTools({});
+        result = await bridge.listTools({});
         expect(result.tools.map((t) => t.name)).toEqual(["tool3"]);
       });
 
@@ -1270,7 +1270,7 @@ describe("App <-> AppBridge integration", () => {
         // Disable one tool after connecting
         tool2.disable();
 
-        const result = await bridge.sendListTools({});
+        const result = await bridge.listTools({});
 
         // Only enabled tool should be in the list
         expect(result.tools).toHaveLength(1);
@@ -1299,11 +1299,11 @@ describe("App <-> AppBridge integration", () => {
         );
 
         // List should include the tool
-        let listResult = await bridge.sendListTools({});
+        let listResult = await bridge.listTools({});
         expect(listResult.tools.map((t) => t.name)).toContain("counter");
 
         // Call the tool
-        let callResult = await bridge.sendCallTool({
+        let callResult = await bridge.callTool({
           name: "counter",
           arguments: {},
         });
@@ -1315,7 +1315,7 @@ describe("App <-> AppBridge integration", () => {
         tool.update({ description: "An updated counter tool" });
 
         // Should still be callable
-        callResult = await bridge.sendCallTool({
+        callResult = await bridge.callTool({
           name: "counter",
           arguments: {},
         });
@@ -1325,12 +1325,12 @@ describe("App <-> AppBridge integration", () => {
         tool.remove();
 
         // Should no longer be in list
-        listResult = await bridge.sendListTools({});
+        listResult = await bridge.listTools({});
         expect(listResult.tools.map((t) => t.name)).not.toContain("counter");
 
         // Should no longer be callable
         await expect(
-          bridge.sendCallTool({ name: "counter", arguments: {} }),
+          bridge.callTool({ name: "counter", arguments: {} }),
         ).rejects.toThrow("Tool counter not found");
       });
 
@@ -1381,19 +1381,19 @@ describe("App <-> AppBridge integration", () => {
         await app2.connect(app2Transport);
 
         // Each app should only see its own tools
-        const list1 = await bridge1.sendListTools({});
+        const list1 = await bridge1.listTools({});
         expect(list1.tools.map((t) => t.name)).toEqual(["app1-tool"]);
 
-        const list2 = await bridge2.sendListTools({});
+        const list2 = await bridge2.listTools({});
         expect(list2.tools.map((t) => t.name)).toEqual(["app2-tool"]);
 
         // Each app should only be able to call its own tools
         await expect(
-          bridge1.sendCallTool({ name: "app1-tool", arguments: {} }),
+          bridge1.callTool({ name: "app1-tool", arguments: {} }),
         ).resolves.toBeDefined();
 
         await expect(
-          bridge1.sendCallTool({ name: "app2-tool", arguments: {} }),
+          bridge1.callTool({ name: "app2-tool", arguments: {} }),
         ).rejects.toThrow("Tool app2-tool not found");
 
         // Clean up

src/app-bridge.ts

@@ -1247,15 +1247,15 @@ export class AppBridge extends Protocol<
   /** @deprecated Use {@link teardownResource} instead */
   sendResourceTeardown: AppBridge["teardownResource"] = this.teardownResource;
 
-  sendCallTool(params: CallToolRequest["params"], options?: RequestOptions) {
+  callTool(params: CallToolRequest["params"], options?: RequestOptions) {
     return this.request(
       { method: "tools/call", params },
       CallToolResultSchema,
       options,
     );
   }
 
-  sendListTools(params: ListToolsRequest["params"], options?: RequestOptions) {
+  listTools(params: ListToolsRequest["params"], options?: RequestOptions) {
     return this.request(
       { method: "tools/list", params },
       ListToolsResultSchema,