chore: consolidate imports, remove unused (code-quality bot)

Author: felixweinberger

src/app-bridge.test.ts

@@ -1,7 +1,9 @@
 import { describe, it, expect, beforeEach, afterEach } from "bun:test";
-import { InMemoryTransport } from "@modelcontextprotocol/sdk/inMemory.js";
-import type { Client } from "@modelcontextprotocol/client";
-import type { ServerCapabilities } from "@modelcontextprotocol/client";
+import {
+  InMemoryTransport,
+  type Client,
+  type ServerCapabilities,
+} from "@modelcontextprotocol/client";
 import {
   EmptyResultSchema,
   ListPromptsResultSchema,
@@ -680,11 +682,7 @@ describe("App <-> AppBridge integration", () => {
       await app.connect(appTransport);
 
       // Bridge can send ping via the protocol's request method
-      const result = await bridge.request(
-        "ping",
-        {},
-        EmptyResultSchema,
-      );
+      const result = await bridge.request("ping", {}, EmptyResultSchema);
 
       expect(result).toEqual({});
     });

src/app-bridge.ts

@@ -1,8 +1,7 @@
-import { Client } from "@modelcontextprotocol/client";
-import { Transport } from "@modelcontextprotocol/client";
 import {
   CallToolRequest,
   CallToolResult,
+  Client,
   EmptyResult,
   Implementation,
   ListPromptsRequest,
@@ -15,35 +14,26 @@ import {
   LoggingMessageNotification,
   PingRequest,
   PromptListChangedNotification,
+  ProtocolOptions,
   ReadResourceRequest,
   ReadResourceResult,
+  RequestOptions,
   ResourceListChangedNotification,
   Tool,
   ToolListChangedNotification,
+  Transport,
 } from "@modelcontextprotocol/client";
 import {
   CallToolRequestSchema,
   CallToolResultSchema,
   ListPromptsRequestSchema,
-  ListPromptsResultSchema,
   ListResourcesRequestSchema,
-  ListResourcesResultSchema,
   ListResourceTemplatesRequestSchema,
-  ListResourceTemplatesResultSchema,
-  ListToolsRequestSchema,
   ListToolsResultSchema,
   LoggingMessageNotificationSchema,
   PingRequestSchema,
-  PromptListChangedNotificationSchema,
   ReadResourceRequestSchema,
-  ReadResourceResultSchema,
-  ResourceListChangedNotificationSchema,
-  ToolListChangedNotificationSchema,
 } from "./sdk-schemas";
-import {
-  ProtocolOptions,
-  RequestOptions,
-} from "@modelcontextprotocol/client";
 import { ProtocolWithEvents } from "./events";
 
 import {
@@ -374,7 +364,7 @@ export class AppBridge extends ProtocolWithEvents<
     // Hosts can override this by setting bridge.onrequestdisplaymode = ...
     this.replaceRequestHandler(
       McpUiRequestDisplayModeRequestSchema,
-      (request) => {
+      (_request) => {
         const currentMode = this._hostContext.displayMode ?? "inline";
         return { mode: currentMode };
       },
@@ -1586,7 +1576,10 @@ export class AppBridge extends ProtocolWithEvents<
     params: McpUiResourceTeardownRequest["params"],
     options?: RequestOptions,
   ) {
-    return this.request("ui/resource-teardown", params, McpUiResourceTeardownResultSchema,
+    return this.request(
+      "ui/resource-teardown",
+      params,
+      McpUiResourceTeardownResultSchema,
       options,
     );
   }
@@ -1604,9 +1597,7 @@ export class AppBridge extends ProtocolWithEvents<
    * @returns Promise resolving to the tool call result
    */
   callTool(params: CallToolRequest["params"], options?: RequestOptions) {
-    return this.request("tools/call", params, CallToolResultSchema,
-      options,
-    );
+    return this.request("tools/call", params, CallToolResultSchema, options);
   }
 
   /**
@@ -1619,9 +1610,7 @@ export class AppBridge extends ProtocolWithEvents<
    * @returns Promise resolving to the list of tools
    */
   listTools(params: ListToolsRequest["params"], options?: RequestOptions) {
-    return this.request("tools/list", params, ListToolsResultSchema,
-      options,
-    );
+    return this.request("tools/list", params, ListToolsResultSchema, options);
   }
 
   /**

src/app.ts

@@ -1,8 +1,3 @@
-import {
-  type RequestOptions,
-  ProtocolOptions,
-} from "@modelcontextprotocol/client";
-
 import {
   CallToolRequest,
   CallToolResult,
@@ -12,8 +7,11 @@ import {
   ListToolsRequest,
   ListToolsResult,
   LoggingMessageNotification,
+  ProtocolOptions,
   ReadResourceRequest,
   ReadResourceResult,
+  type RequestOptions,
+  Transport,
 } from "@modelcontextprotocol/client";
 import {
   CallToolRequestSchema,
@@ -62,7 +60,6 @@ import {
   McpUiRequestDisplayModeRequest,
   McpUiRequestDisplayModeResultSchema,
 } from "./types";
-import { Transport } from "@modelcontextprotocol/client";
 
 export { PostMessageTransport } from "./message-transport";
 export * from "./types";
@@ -825,16 +822,14 @@ export class App extends ProtocolWithEvents<
           `Did you mean: callServerTool({ name: "${params}", arguments: { ... } })?`,
       );
     }
-    return await this.request("tools/call", params, CallToolResultSchema,
-      {
-        // Hosts may interpose long-running or user-interactive steps before the
-        // tool result arrives. Opting in here lets a host heartbeat keep the
-        // request alive past the default timeout; callers can still override.
-        onprogress: () => {},
-        resetTimeoutOnProgress: true,
-        ...options,
-      },
-    );
+    return await this.request("tools/call", params, CallToolResultSchema, {
+      // Hosts may interpose long-running or user-interactive steps before the
+      // tool result arrives. Opting in here lets a host heartbeat keep the
+      // request alive past the default timeout; callers can still override.
+      onprogress: () => {},
+      resetTimeoutOnProgress: true,
+      ...options,
+    });
   }
 
   /**
@@ -881,7 +876,10 @@ export class App extends ProtocolWithEvents<
     params: ReadResourceRequest["params"],
     options?: RequestOptions,
   ): Promise<ReadResourceResult> {
-    return await this.request("resources/read", params, ReadResourceResultSchema,
+    return await this.request(
+      "resources/read",
+      params,
+      ReadResourceResultSchema,
       options,
     );
   }
@@ -926,7 +924,10 @@ export class App extends ProtocolWithEvents<
     params?: ListResourcesRequest["params"],
     options?: RequestOptions,
   ): Promise<ListResourcesResult> {
-    return await this.request("resources/list", params, ListResourcesResultSchema,
+    return await this.request(
+      "resources/list",
+      params,
+      ListResourcesResultSchema,
       options,
     );
   }