chore: fix lint errors

Author: LucaButBoring

examples/client/src/simpleStreamableHttp.ts

@@ -145,9 +145,10 @@ function commandLoop(): void {
                     break;
                 }
 
-                case 'collect-info-task':
+                case 'collect-info-task': {
                     await callCollectInfoWithTask(args[1] || 'contact');
                     break;
+                }
 
                 case 'start-notifications': {
                     const interval = args[1] ? Number.parseInt(args[1], 10) : 2000;

packages/server/src/experimental/tasks/server.ts

@@ -130,21 +130,19 @@ export class ExperimentalServerTasks<
         const clientCapabilities = (this._server as unknown as ServerWithCapabilities).getClientCapabilities();
 
         // Capability check - only required when tools/toolChoice are provided
-        if (params.tools || params.toolChoice) {
-            if (!clientCapabilities?.sampling?.tools) {
-                throw new Error('Client does not support sampling tools capability.');
-            }
+        if ((params.tools || params.toolChoice) && !clientCapabilities?.sampling?.tools) {
+            throw new Error('Client does not support sampling tools capability.');
         }
 
         // Message structure validation - always validate tool_use/tool_result pairs.
         // These may appear even without tools/toolChoice in the current request when
         // a previous sampling request returned tool_use and this is a follow-up with results.
         if (params.messages.length > 0) {
-            const lastMessage = params.messages[params.messages.length - 1]!;
+            const lastMessage = params.messages.at(-1)!;
             const lastContent = Array.isArray(lastMessage.content) ? lastMessage.content : [lastMessage.content];
             const hasToolResults = lastContent.some(c => c.type === 'tool_result');
 
-            const previousMessage = params.messages.length > 1 ? params.messages[params.messages.length - 2] : undefined;
+            const previousMessage = params.messages.length > 1 ? params.messages.at(-2) : undefined;
             const previousContent = previousMessage
                 ? Array.isArray(previousMessage.content)
                     ? previousMessage.content
@@ -236,16 +234,18 @@ export class ExperimentalServerTasks<
 
         // Capability check based on mode
         switch (mode) {
-            case 'url':
+            case 'url': {
                 if (!clientCapabilities?.elicitation?.url) {
                     throw new Error('Client does not support url elicitation.');
                 }
                 break;
-            case 'form':
+            }
+            case 'form': {
                 if (!clientCapabilities?.elicitation?.form) {
                     throw new Error('Client does not support form elicitation.');
                 }
                 break;
+            }
         }
 
         // Normalize params to ensure mode is set

test/integration/test/server.test.ts

@@ -12,7 +12,6 @@ import type {
     Task,
     Transport
 } from '@modelcontextprotocol/core';
-import { toArrayAsync } from '@modelcontextprotocol/core';
 import {
     CallToolRequestSchema,
     CallToolResultSchema,
@@ -33,7 +32,8 @@ import {
     RequestSchema,
     ResultSchema,
     SetLevelRequestSchema,
-    SUPPORTED_PROTOCOL_VERSIONS
+    SUPPORTED_PROTOCOL_VERSIONS,
+    toArrayAsync
 } from '@modelcontextprotocol/core';
 import { createMcpExpressApp } from '@modelcontextprotocol/express';
 import { InMemoryTaskStore, McpServer, Server } from '@modelcontextprotocol/server';
@@ -2054,7 +2054,7 @@ describe('createMessageStream', () => {
 
             expect(terminalMessages.length).toBe(1);
 
-            const lastMessage = messages[messages.length - 1];
+            const lastMessage = messages.at(-1);
             expect(lastMessage.type === 'result' || lastMessage.type === 'error').toBe(true);
 
             if (lastMessage.type === 'result') {
@@ -2136,7 +2136,7 @@ describe('createMessageStream', () => {
                     messages: [{ role: 'user', content: { type: 'text', text: 'Task-augmented message' } }],
                     maxTokens: 100
                 },
-                { task: { ttl: 60000 } }
+                { task: { ttl: 60_000 } }
             );
 
             const messages = await toArrayAsync(stream);
@@ -2150,7 +2150,7 @@ describe('createMessageStream', () => {
             expect(taskCreated.task.taskId).toBeDefined();
 
             // Last message should be result
-            const lastMessage = messages[messages.length - 1];
+            const lastMessage = messages.at(-1);
             expect(lastMessage.type).toBe('result');
             if (lastMessage.type === 'result') {
                 expect((lastMessage.result as CreateMessageResult).model).toBe('test-model');
@@ -3688,7 +3688,7 @@ describe('elicitInputStream', () => {
             expect(terminalMessages.length).toBe(1);
 
             // Verify terminal message is the last message
-            const lastMessage = messages[messages.length - 1];
+            const lastMessage = messages.at(-1);
             expect(lastMessage.type === 'result' || lastMessage.type === 'error').toBe(true);
 
             // Verify result content matches expected action
@@ -3786,7 +3786,7 @@ describe('elicitInputStream', () => {
                         required: ['username']
                     }
                 },
-                { task: { ttl: 60000 } }
+                { task: { ttl: 60_000 } }
             );
 
             const messages = await toArrayAsync(stream);
@@ -3800,7 +3800,7 @@ describe('elicitInputStream', () => {
             expect(taskCreated.task.taskId).toBeDefined();
 
             // Last message should be result
-            const lastMessage = messages[messages.length - 1];
+            const lastMessage = messages.at(-1);
             expect(lastMessage.type).toBe('result');
             if (lastMessage.type === 'result') {
                 expect((lastMessage.result as ElicitResult).action).toBe('accept');