lint

Author: MQ37

src/utils/tools-loader.ts

@@ -107,7 +107,7 @@ export async function loadToolsFromInput(
     } // else: selectors provided but none are actors => do not load defaults
 
     // Compose final tool list
-    let result: ToolEntry[] = [];
+    const result: ToolEntry[] = [];
 
     // Internal tools
     if (selectorsProvided) {
@@ -144,13 +144,13 @@ export async function loadToolsFromInput(
     // TEMP: for now we disable this swapping logic as the add-actor tool was misbehaving in some clients
     // Handle client capabilities logic for 'actors' category to swap call-actor for add-actor
     // if client supports dynamic tools.
-    //const selectorContainsCallActor = selectors?.some((s) => s === HelperTools.ACTOR_CALL);
-    //if (doesMcpClientSupportDynamicTools(initializeRequestData) && hasCallActor && !selectorContainsCallActor) {
+    // const selectorContainsCallActor = selectors?.some((s) => s === HelperTools.ACTOR_CALL);
+    // if (doesMcpClientSupportDynamicTools(initializeRequestData) && hasCallActor && !selectorContainsCallActor) {
     //    // Remove call-actor
     //    result = result.filter((entry) => entry.tool.name !== HelperTools.ACTOR_CALL);
     //    // Replace with add-actor if not already present
     //    if (!hasAddActorTool) result.push(addTool);
-    //}
+    // }
 
     // De-duplicate by tool name for safety
     const seen = new Set<string>();

tests/integration/suite.ts

@@ -979,27 +979,28 @@ export function createIntegrationTestsSuite(
             expect(tools.tools.length).toBeGreaterThan(0);
         });
 
-        // TEMP: this logic is currently disabled, see src/utils/tools-loader.ts
-        //it.runIf(options.transport === 'streamable-http')('should swap call-actor for add-actor when client supports dynamic tools', async () => {
-        //    client = await createClientFn({ clientName: 'Visual Studio Code', tools: ['actors'] });
-        //    const names = getToolNames(await client.listTools());
-
-        //    // should not contain call-actor but should contain add-actor
-        //    expect(names).not.toContain('call-actor');
-        //    expect(names).toContain('add-actor');
-
-        //    await client.close();
-        //});
-        //it.runIf(options.transport === 'streamable-http')(`should swap call-actor for add-actor when client supports dynamic tools for default tools`, async () => {
-        //    client = await createClientFn({ clientName: 'Visual Studio Code' });
-        //    const names = getToolNames(await client.listTools());
-
-        //    // should not contain call-actor but should contain add-actor
-        //    expect(names).not.toContain('call-actor');
-        //    expect(names).toContain('add-actor');
-
-        //    await client.close();
-        //});
+        //  TEMP: this logic is currently disabled, see src/utils/tools-loader.ts
+        // it.runIf(options.transport === 'streamable-http')('should swap call-actor for add-actor when client supports dynamic tools', async () => {
+        //     client = await createClientFn({ clientName: 'Visual Studio Code', tools: ['actors'] });
+        //     const names = getToolNames(await client.listTools());
+
+        //     // should not contain call-actor but should contain add-actor
+        //     expect(names).not.toContain('call-actor');
+        //     expect(names).toContain('add-actor');
+
+        //     await client.close();
+        // });
+        // it.runIf(options.transport === 'streamable-http')(
+        // `should swap call-actor for add-actor when client supports dynamic tools for default tools`, async () => {
+        //     client = await createClientFn({ clientName: 'Visual Studio Code' });
+        //     const names = getToolNames(await client.listTools());
+
+        //     // should not contain call-actor but should contain add-actor
+        //     expect(names).not.toContain('call-actor');
+        //     expect(names).toContain('add-actor');
+
+        //     await client.close();
+        // });
         it.runIf(options.transport === 'streamable-http')('should NOT swap call-actor for add-actor even when client supports dynamic tools', async () => {
             client = await createClientFn({ clientName: 'Visual Studio Code', tools: ['actors'] });
             const names = getToolNames(await client.listTools());