fix: default stramable and backup SSE transport

MichalKalita · MichalKalita · commit 22f4c9d6a0c2 · 2025-09-18T12:03:16.000+02:00
diff --git a/src/mcp/client.ts b/src/mcp/client.ts
@@ -21,7 +21,9 @@ export async function connectMCPClient(
     let client: Client;
     try {
         client = await createMCPStreamableClient(url, token);
+        return client;
     } catch (error) {
+        // If streamable HTTP transport fails on not timeout error, continue with SSE transport
         if (error instanceof TimeoutError) {
             log.warning('Connection to MCP server using streamable HTTP transport timed out', { url });
             return null;
@@ -35,6 +37,7 @@ export async function connectMCPClient(
 
     try {
         client = await createMCPSSEClient(url, token);
+        return client;
     } catch (error) {
         if (error instanceof TimeoutError) {
             log.warning('Connection to MCP server using SSE transport timed out', { url });
@@ -44,8 +47,6 @@ export async function connectMCPClient(
         log.error('Failed to connect to MCP server using SSE transport', { cause: error });
         throw error;
     }
-
-    return client;
 }
 
 async function withTimeout<T>(millis: number, promise: Promise<T>): Promise<T> {
diff --git a/tests/integration/suite.ts b/tests/integration/suite.ts
@@ -940,5 +940,21 @@ export function createIntegrationTestsSuite(
 
             await client.close();
         });
+
+        // Skipping tests, MCP actors are not available in staging environment
+        // TODO: run this tests only for local environment
+        it.skip.for([
+            'mcp-servers/slidespeak-mcp-server',
+            'mcp-servers/brave-search-mcp-server',
+            'jiri.spilka/weather-mcp-server',
+            'apify/actors-mcp-server',
+            'jakub.kopecky/browserbase-mcp-server',
+            'jakub.kopecky/arxiv-mcp-server',
+            'jiri.spilka/playwright-mcp-server',
+        ])('should connect to "%s" MCP server and at least one tool is available', async (mcpServer) => {
+            client = await createClientFn({ tools: [mcpServer] });
+            const tools = await client.listTools();
+            expect(tools.tools.length).toBeGreaterThan(0);
+        });
     });
 }
