lint fix

KKonstantinov · KKonstantinov · commit 893b4845b825 · 2026-03-12T19:51:02.000+02:00
diff --git a/packages/client/src/experimental/tasks/client.ts b/packages/client/src/experimental/tasks/client.ts
@@ -13,14 +13,14 @@ import type {
     CreateTaskResult,
     GetTaskResult,
     ListTasksResult,
+    Request,
     RequestMethod,
     RequestOptions,
     ResponseMessage,
     ResultTypeMap,
     SchemaOutput
 } from '@modelcontextprotocol/core';
 import { CallToolResultSchema, getResultSchema, ProtocolError, ProtocolErrorCode } from '@modelcontextprotocol/core';
-import type { Request } from '@modelcontextprotocol/core';
 
 import type { Client } from '../../client/client.js';
 
diff --git a/packages/core/src/experimental/tasks/helpers.ts b/packages/core/src/experimental/tasks/helpers.ts
@@ -17,7 +17,7 @@ interface TaskRequestsCapability {
 
 /**
  * Asserts that task creation is supported for `tools/call`.
- * Used by {@linkcode @modelcontextprotocol/client!client/client.Client.assertTaskCapability | Client.assertTaskCapability} and {@linkcode @modelcontextprotocol/server!server/server.Server.assertTaskHandlerCapability | Server.assertTaskHandlerCapability}.
+ * Used as the `assertTaskCapability` or `assertTaskHandlerCapability` callback in `TaskManagerOptions`.
  *
  * @param requests - The task requests capability object
  * @param method - The method being checked
@@ -52,7 +52,7 @@ export function assertToolsCallTaskCapability(
 
 /**
  * Asserts that task creation is supported for `sampling/createMessage` or `elicitation/create`.
- * Used by {@linkcode @modelcontextprotocol/server!server/server.Server.assertTaskCapability | Server.assertTaskCapability} and {@linkcode @modelcontextprotocol/client!client/client.Client.assertTaskHandlerCapability | Client.assertTaskHandlerCapability}.
+ * Used as the `assertTaskCapability` or `assertTaskHandlerCapability` callback in `TaskManagerOptions`.
  *
  * @param requests - The task requests capability object
  * @param method - The method being checked
diff --git a/packages/core/src/shared/protocol.ts b/packages/core/src/shared/protocol.ts
@@ -282,6 +282,10 @@ type TimeoutInfo = {
     onTimeout: () => void;
 };
 
+async function _noopRouteResponse(): Promise<boolean> {
+    return false;
+}
+
 /**
  * Implements MCP protocol framing on top of a pluggable transport, including
  * features like request/response linking, notifications, and progress.
@@ -514,10 +518,10 @@ export abstract class Protocol<ContextT extends BaseContext> {
         };
 
         // Compose results from all modules
-        let sendNotification: (notification: Notification) => Promise<void> =
-            (notification: Notification) => inboundCtx.sendNotification(notification);
+        let sendNotification: (notification: Notification) => Promise<void> = (notification: Notification) =>
+            inboundCtx.sendNotification(notification);
         let sendRequest = inboundCtx.sendRequest;
-        let routeResponse: (message: JSONRPCResponse | JSONRPCErrorResponse) => Promise<boolean> = async () => false;
+        let routeResponse: (message: JSONRPCResponse | JSONRPCErrorResponse) => Promise<boolean> = _noopRouteResponse;
         let taskContext: BaseContext['task'] | undefined;
         let hasTaskCreationParams = false;
 
diff --git a/packages/core/test/shared/protocol.test.ts b/packages/core/test/shared/protocol.test.ts
@@ -182,7 +182,9 @@ function assertQueuedRequest(o?: QueuedMessage): asserts o is QueuedRequest {
  */
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 function testRequest(proto: Protocol<BaseContext>, request: Request, resultSchema: ZodType, options?: any) {
-    return (proto as unknown as { _requestWithSchema: (request: Request, resultSchema: ZodType, options?: unknown) => Promise<unknown> })._requestWithSchema(request, resultSchema, options);
+    return (
+        proto as unknown as { _requestWithSchema: (request: Request, resultSchema: ZodType, options?: unknown) => Promise<unknown> }
+    )._requestWithSchema(request, resultSchema, options);
 }
 
 describe('protocol tests', () => {
@@ -5264,7 +5266,9 @@ describe('Error handling for missing resolvers', () => {
 });
 
 // -- Helper: create a no-op mock module --
-function createMockModule(overrides?: Partial<ProtocolModule>): ProtocolModule & { boundHost?: ProtocolModuleHost; onCloseCalled: boolean } {
+function createMockModule(
+    overrides?: Partial<ProtocolModule>
+): ProtocolModule & { boundHost?: ProtocolModuleHost; onCloseCalled: boolean } {
     const mock: ProtocolModule & { boundHost?: ProtocolModuleHost; onCloseCalled: boolean } = {
         boundHost: undefined,
         onCloseCalled: false,
diff --git a/packages/server/src/experimental/tasks/server.ts b/packages/server/src/experimental/tasks/server.ts
@@ -20,7 +20,6 @@ import type {
     RequestMethod,
     RequestOptions,
     ResponseMessage,
-    Result,
     ResultTypeMap,
     SchemaOutput
 } from '@modelcontextprotocol/core';
diff --git a/test/integration/test/client/client.test.ts b/test/integration/test/client/client.test.ts
@@ -2206,10 +2206,13 @@ describe('outputSchema validation', () => {
             throw new Error('Unknown tool');
         });
 
-        const client = new Client({
-            name: 'test-client',
-            version: '1.0.0'
-        }, { capabilities: { tasks: { requests: { tools: { call: {} } } } } });
+        const client = new Client(
+            {
+                name: 'test-client',
+                version: '1.0.0'
+            },
+            { capabilities: { tasks: { requests: { tools: { call: {} } } } } }
+        );
 
         const [clientTransport, serverTransport] = InMemoryTransport.createLinkedPair();
 
@@ -2253,10 +2256,8 @@ describe('Task-based execution', () => {
                             },
 
                             taskStore: serverTaskStore
-
                         }
                     }
-
                 }
             );
 
@@ -2293,10 +2294,13 @@ describe('Task-based execution', () => {
                 }
             );
 
-            const client = new Client({
-                name: 'test-client',
-                version: '1.0.0'
-            }, { capabilities: { tasks: { requests: { tools: { call: {} } } } } });
+            const client = new Client(
+                {
+                    name: 'test-client',
+                    version: '1.0.0'
+                },
+                { capabilities: { tasks: { requests: { tools: { call: {} } } } } }
+            );
 
             const [clientTransport, serverTransport] = InMemoryTransport.createLinkedPair();
 
@@ -2335,10 +2339,8 @@ describe('Task-based execution', () => {
                             },
 
                             taskStore: serverTaskStore
-
                         }
                     }
-
                 }
             );
 
@@ -2375,10 +2377,13 @@ describe('Task-based execution', () => {
                 }
             );
 
-            const client = new Client({
-                name: 'test-client',
-                version: '1.0.0'
-            }, { capabilities: { tasks: { requests: { tools: { call: {} } } } } });
+            const client = new Client(
+                {
+                    name: 'test-client',
+                    version: '1.0.0'
+                },
+                { capabilities: { tasks: { requests: { tools: { call: {} } } } } }
+            );
 
             const [clientTransport, serverTransport] = InMemoryTransport.createLinkedPair();
 
@@ -2418,10 +2423,8 @@ describe('Task-based execution', () => {
                             },
 
                             taskStore: serverTaskStore
-
                         }
                     }
-
                 }
             );
 
@@ -2458,10 +2461,13 @@ describe('Task-based execution', () => {
                 }
             );
 
-            const client = new Client({
-                name: 'test-client',
-                version: '1.0.0'
-            }, { capabilities: { tasks: { requests: { tools: { call: {} } } } } });
+            const client = new Client(
+                {
+                    name: 'test-client',
+                    version: '1.0.0'
+                },
+                { capabilities: { tasks: { requests: { tools: { call: {} } } } } }
+            );
 
             const [clientTransport, serverTransport] = InMemoryTransport.createLinkedPair();
 
@@ -2505,10 +2511,8 @@ describe('Task-based execution', () => {
                             },
 
                             taskStore: serverTaskStore
-
                         }
                     }
-
                 }
             );
 
@@ -2545,10 +2549,13 @@ describe('Task-based execution', () => {
                 }
             );
 
-            const client = new Client({
-                name: 'test-client',
-                version: '1.0.0'
-            }, { capabilities: { tasks: { requests: { tools: { call: {} } } } } });
+            const client = new Client(
+                {
+                    name: 'test-client',
+                    version: '1.0.0'
+                },
+                { capabilities: { tasks: { requests: { tools: { call: {} } } } } }
+            );
 
             const [clientTransport, serverTransport] = InMemoryTransport.createLinkedPair();
 
@@ -2615,10 +2622,8 @@ describe('Task-based execution', () => {
                             },
 
                             taskStore: clientTaskStore
-
                         }
                     }
-
                 }
             );
 
@@ -2710,10 +2715,8 @@ describe('Task-based execution', () => {
                             },
 
                             taskStore: clientTaskStore
-
                         }
                     }
-
                 }
             );
 
@@ -2804,10 +2807,8 @@ describe('Task-based execution', () => {
                             },
 
                             taskStore: clientTaskStore
-
                         }
                     }
-
                 }
             );
 
@@ -2897,10 +2898,8 @@ describe('Task-based execution', () => {
                             },
 
                             taskStore: clientTaskStore
-
                         }
                     }
-
                 }
             );
 
@@ -3002,10 +3001,8 @@ describe('Task-based execution', () => {
                         },
 
                         taskStore: serverTaskStore
-
                     }
                 }
-
             }
         );
 
@@ -3130,10 +3127,8 @@ describe('Task-based execution', () => {
                             },
 
                             taskStore: serverTaskStore
-
                         }
                     }
-
                 }
             );
 
@@ -3180,10 +3175,8 @@ describe('Task-based execution', () => {
                             },
 
                             taskStore: serverTaskStore
-
                         }
                     }
-
                 }
             );
 
@@ -3230,10 +3223,8 @@ describe('Task-based execution', () => {
                             },
 
                             taskStore: clientTaskStore
-
                         }
                     }
-
                 }
             );
 
@@ -3287,10 +3278,8 @@ test('should respect server task capabilities', async () => {
                     },
 
                     taskStore: serverTaskStore
-
                 }
             }
-
         }
     );
 
diff --git a/test/integration/test/server.test.ts b/test/integration/test/server.test.ts
diff --git a/test/integration/test/server/mcp.test.ts b/test/integration/test/server/mcp.test.ts
diff --git a/test/integration/test/taskLifecycle.test.ts b/test/integration/test/taskLifecycle.test.ts

Original file line number	Diff line number	Diff line change
`@@ -2206,10 +2206,13 @@ describe('outputSchema validation', () => {`
`2206`	`2206`	`throw new Error('Unknown tool');`
`2207`	`2207`	`});`
`2208`	`2208`
`2209`		`- const client = new Client({`
`2210`		`- name: 'test-client',`
`2211`		`- version: '1.0.0'`
`2212`		`- }, { capabilities: { tasks: { requests: { tools: { call: {} } } } } });`
	`2209`	`+ const client = new Client(`
	`2210`	`+ {`
	`2211`	`+ name: 'test-client',`
	`2212`	`+ version: '1.0.0'`
	`2213`	`+ },`
	`2214`	`+ { capabilities: { tasks: { requests: { tools: { call: {} } } } } }`
	`2215`	`+ );`
`2213`	`2216`
`2214`	`2217`	`const [clientTransport, serverTransport] = InMemoryTransport.createLinkedPair();`
`2215`	`2218`
`@@ -2253,10 +2256,8 @@ describe('Task-based execution', () => {`
`2253`	`2256`	`},`
`2254`	`2257`
`2255`	`2258`	`taskStore: serverTaskStore`
`2256`		`-`
`2257`	`2259`	`}`
`2258`	`2260`	`}`
`2259`		`-`
`2260`	`2261`	`}`
`2261`	`2262`	`);`
`2262`	`2263`
`@@ -2293,10 +2294,13 @@ describe('Task-based execution', () => {`
`2293`	`2294`	`}`
`2294`	`2295`	`);`
`2295`	`2296`
`2296`		`- const client = new Client({`
`2297`		`- name: 'test-client',`
`2298`		`- version: '1.0.0'`
`2299`		`- }, { capabilities: { tasks: { requests: { tools: { call: {} } } } } });`
	`2297`	`+ const client = new Client(`
	`2298`	`+ {`
	`2299`	`+ name: 'test-client',`
	`2300`	`+ version: '1.0.0'`
	`2301`	`+ },`
	`2302`	`+ { capabilities: { tasks: { requests: { tools: { call: {} } } } } }`
	`2303`	`+ );`
`2300`	`2304`
`2301`	`2305`	`const [clientTransport, serverTransport] = InMemoryTransport.createLinkedPair();`
`2302`	`2306`
`@@ -2335,10 +2339,8 @@ describe('Task-based execution', () => {`
`2335`	`2339`	`},`
`2336`	`2340`
`2337`	`2341`	`taskStore: serverTaskStore`
`2338`		`-`
`2339`	`2342`	`}`
`2340`	`2343`	`}`
`2341`		`-`
`2342`	`2344`	`}`
`2343`	`2345`	`);`
`2344`	`2346`
`@@ -2375,10 +2377,13 @@ describe('Task-based execution', () => {`
`2375`	`2377`	`}`
`2376`	`2378`	`);`
`2377`	`2379`
`2378`		`- const client = new Client({`
`2379`		`- name: 'test-client',`
`2380`		`- version: '1.0.0'`
`2381`		`- }, { capabilities: { tasks: { requests: { tools: { call: {} } } } } });`
	`2380`	`+ const client = new Client(`
	`2381`	`+ {`
	`2382`	`+ name: 'test-client',`
	`2383`	`+ version: '1.0.0'`
	`2384`	`+ },`
	`2385`	`+ { capabilities: { tasks: { requests: { tools: { call: {} } } } } }`
	`2386`	`+ );`
`2382`	`2387`
`2383`	`2388`	`const [clientTransport, serverTransport] = InMemoryTransport.createLinkedPair();`
`2384`	`2389`
`@@ -2418,10 +2423,8 @@ describe('Task-based execution', () => {`
`2418`	`2423`	`},`
`2419`	`2424`
`2420`	`2425`	`taskStore: serverTaskStore`
`2421`		`-`
`2422`	`2426`	`}`
`2423`	`2427`	`}`
`2424`		`-`
`2425`	`2428`	`}`
`2426`	`2429`	`);`
`2427`	`2430`
`@@ -2458,10 +2461,13 @@ describe('Task-based execution', () => {`
`2458`	`2461`	`}`
`2459`	`2462`	`);`
`2460`	`2463`
`2461`		`- const client = new Client({`
`2462`		`- name: 'test-client',`
`2463`		`- version: '1.0.0'`
`2464`		`- }, { capabilities: { tasks: { requests: { tools: { call: {} } } } } });`
	`2464`	`+ const client = new Client(`
	`2465`	`+ {`
	`2466`	`+ name: 'test-client',`
	`2467`	`+ version: '1.0.0'`
	`2468`	`+ },`
	`2469`	`+ { capabilities: { tasks: { requests: { tools: { call: {} } } } } }`
	`2470`	`+ );`
`2465`	`2471`
`2466`	`2472`	`const [clientTransport, serverTransport] = InMemoryTransport.createLinkedPair();`
`2467`	`2473`
`@@ -2505,10 +2511,8 @@ describe('Task-based execution', () => {`
`2505`	`2511`	`},`
`2506`	`2512`
`2507`	`2513`	`taskStore: serverTaskStore`
`2508`		`-`
`2509`	`2514`	`}`
`2510`	`2515`	`}`
`2511`		`-`
`2512`	`2516`	`}`
`2513`	`2517`	`);`
`2514`	`2518`
`@@ -2545,10 +2549,13 @@ describe('Task-based execution', () => {`
`2545`	`2549`	`}`
`2546`	`2550`	`);`
`2547`	`2551`
`2548`		`- const client = new Client({`
`2549`		`- name: 'test-client',`
`2550`		`- version: '1.0.0'`
`2551`		`- }, { capabilities: { tasks: { requests: { tools: { call: {} } } } } });`
	`2552`	`+ const client = new Client(`
	`2553`	`+ {`
	`2554`	`+ name: 'test-client',`
	`2555`	`+ version: '1.0.0'`
	`2556`	`+ },`
	`2557`	`+ { capabilities: { tasks: { requests: { tools: { call: {} } } } } }`
	`2558`	`+ );`
`2552`	`2559`
`2553`	`2560`	`const [clientTransport, serverTransport] = InMemoryTransport.createLinkedPair();`
`2554`	`2561`
`@@ -2615,10 +2622,8 @@ describe('Task-based execution', () => {`
`2615`	`2622`	`},`
`2616`	`2623`
`2617`	`2624`	`taskStore: clientTaskStore`
`2618`		`-`
`2619`	`2625`	`}`
`2620`	`2626`	`}`
`2621`		`-`
`2622`	`2627`	`}`
`2623`	`2628`	`);`
`2624`	`2629`
`@@ -2710,10 +2715,8 @@ describe('Task-based execution', () => {`
`2710`	`2715`	`},`
`2711`	`2716`
`2712`	`2717`	`taskStore: clientTaskStore`
`2713`		`-`
`2714`	`2718`	`}`
`2715`	`2719`	`}`
`2716`		`-`
`2717`	`2720`	`}`
`2718`	`2721`	`);`
`2719`	`2722`
`@@ -2804,10 +2807,8 @@ describe('Task-based execution', () => {`
`2804`	`2807`	`},`
`2805`	`2808`
`2806`	`2809`	`taskStore: clientTaskStore`
`2807`		`-`
`2808`	`2810`	`}`
`2809`	`2811`	`}`
`2810`		`-`
`2811`	`2812`	`}`
`2812`	`2813`	`);`
`2813`	`2814`
`@@ -2897,10 +2898,8 @@ describe('Task-based execution', () => {`
`2897`	`2898`	`},`
`2898`	`2899`
`2899`	`2900`	`taskStore: clientTaskStore`
`2900`		`-`
`2901`	`2901`	`}`
`2902`	`2902`	`}`
`2903`		`-`
`2904`	`2903`	`}`
`2905`	`2904`	`);`
`2906`	`2905`
`@@ -3002,10 +3001,8 @@ describe('Task-based execution', () => {`
`3002`	`3001`	`},`
`3003`	`3002`
`3004`	`3003`	`taskStore: serverTaskStore`
`3005`		`-`
`3006`	`3004`	`}`
`3007`	`3005`	`}`
`3008`		`-`
`3009`	`3006`	`}`
`3010`	`3007`	`);`
`3011`	`3008`
`@@ -3130,10 +3127,8 @@ describe('Task-based execution', () => {`
`3130`	`3127`	`},`
`3131`	`3128`
`3132`	`3129`	`taskStore: serverTaskStore`
`3133`		`-`
`3134`	`3130`	`}`
`3135`	`3131`	`}`
`3136`		`-`
`3137`	`3132`	`}`
`3138`	`3133`	`);`
`3139`	`3134`
`@@ -3180,10 +3175,8 @@ describe('Task-based execution', () => {`
`3180`	`3175`	`},`
`3181`	`3176`
`3182`	`3177`	`taskStore: serverTaskStore`
`3183`		`-`
`3184`	`3178`	`}`
`3185`	`3179`	`}`
`3186`		`-`
`3187`	`3180`	`}`
`3188`	`3181`	`);`
`3189`	`3182`
`@@ -3230,10 +3223,8 @@ describe('Task-based execution', () => {`
`3230`	`3223`	`},`
`3231`	`3224`
`3232`	`3225`	`taskStore: clientTaskStore`
`3233`		`-`
`3234`	`3226`	`}`
`3235`	`3227`	`}`
`3236`		`-`
`3237`	`3228`	`}`
`3238`	`3229`	`);`
`3239`	`3230`
`@@ -3287,10 +3278,8 @@ test('should respect server task capabilities', async () => {`
`3287`	`3278`	`},`
`3288`	`3279`
`3289`	`3280`	`taskStore: serverTaskStore`
`3290`		`-`
`3291`	`3281`	`}`
`3292`	`3282`	`}`
`3293`		`-`
`3294`	`3283`	`}`
`3295`	`3284`	`);`
`3296`	`3285`