fa-mcp-sdk 0.4.76 → 0.4.77

package/src/tests/mcp/sse/mcp-sse-client-handling.md

@@ -1,111 +1,111 @@
-# MCP SSE Client - Preventing unhandledRejection
-
-## Problem
-
-When using `fa-mcp-sdk` as an npm package in tests where `McpSseClient` is imported, `unhandledRejection` errors may occur. This happens because MCP errors are handled asynchronously through SSE (Server-Sent Events) and may not be properly caught in the test context.
-
-## Why this happens
-
-1. **Local tests**: Run in the same process as the MCP server
-2. **NPM package**: The test runs in a separate process
-3. **SSE asynchronicity**: Errors may occur after the promise is created but before they are handled
-
-## Solutions
-
-### Solution 1: Use a static method (recommended)
-
-```javascript
-import { McpSseClient } from 'fa-mcp-sdk';
-
-// Create a client with automatic unhandledRejection handling
-const client = McpSseClient.createWithErrorHandler('http://localhost:3000');
-
-try {
-  const response = await client.callTool('execute_sql_query', { sql: test.sql });
-  console.log('✅ Success:', response);
-} catch (error) {
-  console.log('❌ Error:', error.message);
-}
-```
-
-### Solution 2: Global unhandledRejection handler
-
-Add at the beginning of the test file:
-
-```javascript
-// Global handling of MCP errors
-process.on('unhandledRejection', (reason) => {
-  if (typeof reason === 'object' && reason?.message?.includes('MCP Error:')) {
-    // Ignore MCP errors — they are handled in try-catch
-    return;
-  }
-});
-
-// Or using the built-in method
-import { McpSseClient } from 'fa-mcp-sdk';
-McpSseClient.setupGlobalErrorHandler();
-```
-
-### Solution 3: Promise with error handling
-
-```javascript
-import { McpSseClient } from 'fa-mcp-sdk';
-
-const client = new McpSseClient('http://localhost:3000');
-
-// Wrap all calls in an additional catch
-async function safeCallTool(toolName, args) {
-  try {
-    return await client.callTool(toolName, args);
-  } catch (error) {
-    // Additional handling to prevent unhandledRejection
-    if (error.message.includes('MCP Error:')) {
-      throw error;
-    }
-    throw new Error(`Unexpected error: ${error.message}`);
-  }
-}
-```
-
-## Full test example
-
-```javascript
-import { McpSseClient } from 'fa-mcp-sdk';
-
-async function testToolExecution() {
-  // Create a client with error handling
-  const client = McpSseClient.createWithErrorHandler('http://localhost:3000');
-
-  try {
-    // Test successful execution
-    const successResponse = await client.callTool('example_tool', { query: 'ping' });
-    console.log('✅ Tool executed successfully:', successResponse);
-
-    // Test error handling (as in your example)
-    try {
-      await client.callTool('example_tool', {}); // No query — should error
-      console.log('❌ Expected error but got success');
-    } catch (error) {
-      console.log('✅ Expected error caught:', error.message);
-    }
-
-  } catch (error) {
-    console.error('❌ Unexpected test error:', error);
-  } finally {
-    await client.close();
-  }
-}
-
-testToolExecution();
-```
-
-## Important notes
-
-1. **For npm packages only**: The issue usually does not occur in local tests
-2. **MCP errors**: The handler filters only errors containing 'MCP Error:'
-3. **Global handler**: Installed once per process
-4. **Resources**: Don’t forget to call `client.close()` for cleanup
-
-## Technical details
-
-The issue arises because SSE errors are handled in `McpSseClient.handleSseEvent()` and may cause a promise rejection that is not yet caught at the moment the error occurs. New methods provide an additional layer of handling to prevent unhandledRejection.
+# MCP SSE Client - Preventing unhandledRejection
+
+## Problem
+
+When using `fa-mcp-sdk` as an npm package in tests where `McpSseClient` is imported, `unhandledRejection` errors may occur. This happens because MCP errors are handled asynchronously through SSE (Server-Sent Events) and may not be properly caught in the test context.
+
+## Why this happens
+
+1. **Local tests**: Run in the same process as the MCP server
+2. **NPM package**: The test runs in a separate process
+3. **SSE asynchronicity**: Errors may occur after the promise is created but before they are handled
+
+## Solutions
+
+### Solution 1: Use a static method (recommended)
+
+```javascript
+import { McpSseClient } from 'fa-mcp-sdk';
+
+// Create a client with automatic unhandledRejection handling
+const client = McpSseClient.createWithErrorHandler('http://localhost:3000');
+
+try {
+  const response = await client.callTool('execute_sql_query', { sql: test.sql });
+  console.log('✅ Success:', response);
+} catch (error) {
+  console.log('❌ Error:', error.message);
+}
+```
+
+### Solution 2: Global unhandledRejection handler
+
+Add at the beginning of the test file:
+
+```javascript
+// Global handling of MCP errors
+process.on('unhandledRejection', (reason) => {
+  if (typeof reason === 'object' && reason?.message?.includes('MCP Error:')) {
+    // Ignore MCP errors — they are handled in try-catch
+    return;
+  }
+});
+
+// Or using the built-in method
+import { McpSseClient } from 'fa-mcp-sdk';
+McpSseClient.setupGlobalErrorHandler();
+```
+
+### Solution 3: Promise with error handling
+
+```javascript
+import { McpSseClient } from 'fa-mcp-sdk';
+
+const client = new McpSseClient('http://localhost:3000');
+
+// Wrap all calls in an additional catch
+async function safeCallTool(toolName, args) {
+  try {
+    return await client.callTool(toolName, args);
+  } catch (error) {
+    // Additional handling to prevent unhandledRejection
+    if (error.message.includes('MCP Error:')) {
+      throw error;
+    }
+    throw new Error(`Unexpected error: ${error.message}`);
+  }
+}
+```
+
+## Full test example
+
+```javascript
+import { McpSseClient } from 'fa-mcp-sdk';
+
+async function testToolExecution() {
+  // Create a client with error handling
+  const client = McpSseClient.createWithErrorHandler('http://localhost:3000');
+
+  try {
+    // Test successful execution
+    const successResponse = await client.callTool('example_tool', { query: 'ping' });
+    console.log('✅ Tool executed successfully:', successResponse);
+
+    // Test error handling (as in your example)
+    try {
+      await client.callTool('example_tool', {}); // No query — should error
+      console.log('❌ Expected error but got success');
+    } catch (error) {
+      console.log('✅ Expected error caught:', error.message);
+    }
+
+  } catch (error) {
+    console.error('❌ Unexpected test error:', error);
+  } finally {
+    await client.close();
+  }
+}
+
+testToolExecution();
+```
+
+## Important notes
+
+1. **For npm packages only**: The issue usually does not occur in local tests
+2. **MCP errors**: The handler filters only errors containing 'MCP Error:'
+3. **Global handler**: Installed once per process
+4. **Resources**: Don’t forget to call `client.close()` for cleanup
+
+## Technical details
+
+The issue arises because SSE errors are handled in `McpSseClient.handleSseEvent()` and may cause a promise rejection that is not yet caught at the moment the error occurs. New methods provide an additional layer of handling to prevent unhandledRejection.

package/src/tests/mcp/sse/test-sse-npm-package.js

@@ -13,7 +13,7 @@ import { McpSseClient } from 'fa-mcp-sdk';
 
 const SERVER_URL = 'http://localhost:9876';
 
-async function testMcpClient () {
+async function testMcpClient() {
   console.log('🧪 Testing MCP client as npm package');
   console.log('='.repeat(50));
 
@@ -42,7 +42,7 @@ async function testMcpClient () {
     // List tools
     console.log('3. Listing tools...');
     const tools = await client.listTools();
-    const toolNames = tools.tools?.map(t => t.name) || [];
+    const toolNames = tools.tools?.map((t) => t.name) || [];
     console.log('✅ Available tools:', toolNames.join(', '));
 
     // Test successful tool call
@@ -76,7 +76,6 @@ async function testMcpClient () {
 
     console.log('\n🎉 All tests completed successfully!');
     console.log('✨ No unhandledRejection errors occurred');
-
   } catch (error) {
     console.error('❌ Unexpected error:', error.message);
   } finally {

package/src/tests/mcp/test-cases.js

@@ -30,10 +30,10 @@ export const TEMPLATE_TESTS = {
       try {
         const list = await client.listPrompts();
         const prompts = list?.prompts || list;
-        const names = Array.isArray(prompts) ? prompts.map(p => p.name) : [];
+        const names = Array.isArray(prompts) ? prompts.map((p) => p.name) : [];
         const okBrief = names.includes('agent_brief');
         const okPrompt = names.includes('agent_prompt');
-        return (okBrief && okPrompt) ? ok(name, { names }) : fail(name, { names });
+        return okBrief && okPrompt ? ok(name, { names }) : fail(name, { names });
       } catch (e) {
         return fail(name, { error: e?.message });
       }
@@ -77,7 +77,7 @@ export const TEMPLATE_TESTS = {
       try {
         const list = await client.listResources();
         const resources = list?.resources || list;
-        const uris = Array.isArray(resources) ? resources.map(r => r.uri) : [];
+        const uris = Array.isArray(resources) ? resources.map((r) => r.uri) : [];
         const found = uris.includes('custom-resource://resource1');
         return found ? ok(name, { uris }) : fail(name, { uris });
       } catch (e) {
@@ -105,10 +105,10 @@ export const TEMPLATE_TESTS = {
       try {
         const list = await client.listTools();
         const tools = list?.tools || list;
-        const names = Array.isArray(tools) ? tools.map(t => t.name) : [];
+        const names = Array.isArray(tools) ? tools.map((t) => t.name) : [];
         const ok1 = names.includes('example_tool');
         const ok2 = names.includes('example_search');
-        return (ok1 && ok2) ? ok(name, { names }) : fail(name, { names });
+        return ok1 && ok2 ? ok(name, { names }) : fail(name, { names });
       } catch (e) {
         return fail(name, { error: e?.message });
       }
@@ -121,8 +121,7 @@ export const TEMPLATE_TESTS = {
         // Both structuredContent and text are acceptable; check message echo
         const structured = r?.structuredContent;
         const text = r?.content?.[0]?.text;
-        const hasProcessed = (structured && structured.message?.includes('Processed query'))
-          || (typeof text === 'string' && text.includes('Processed query'));
+        const hasProcessed = (structured && structured.message?.includes('Processed query')) || (typeof text === 'string' && text.includes('Processed query'));
         return hasProcessed ? ok(name, { response: r }) : fail(name, { response: r });
       } catch (e) {
         return fail(name, { error: e?.message });

package/src/tests/mcp/test-http.js

@@ -11,7 +11,7 @@ import TEMPLATE_TESTS from './test-cases.js';
 
 const baseURL = (process.env.TEST_MCP_SERVER_URL || `http://localhost:${appConfig.webServer.port}`).replace(/\/+$/, '');
 
-async function runTestGroup (title, tests, client) {
+async function runTestGroup(title, tests, client) {
   console.log(`\n${title}:`);
   let passed = 0;
   for (const test of tests) {
@@ -34,7 +34,7 @@ async function runTestGroup (title, tests, client) {
   return passed;
 }
 
-async function main () {
+async function main() {
   console.log('🧪 HTTP tests for template MCP server');
   console.log('='.repeat(60));
 

package/src/tests/mcp/test-sse.js

@@ -11,7 +11,7 @@ import TEMPLATE_TESTS from './test-cases.js';
 
 const baseURL = (process.env.TEST_MCP_SERVER_URL || `http://localhost:${appConfig.webServer.port}`).replace(/\/+$/, '');
 
-async function runTestGroup (title, tests, client) {
+async function runTestGroup(title, tests, client) {
   console.log(`\n${title}:`);
   let passed = 0;
   for (const test of tests) {
@@ -34,7 +34,7 @@ async function runTestGroup (title, tests, client) {
   return passed;
 }
 
-async function main () {
+async function main() {
   console.log('🧪 SSE tests for template MCP server');
   console.log('='.repeat(60));
 
@@ -52,11 +52,13 @@ async function main () {
     await client.health().catch(() => undefined);
 
     // Initialize over RPC channel used by SSE client (POST /rpc)
-    await client.sendRequest('initialize', {
-      protocolVersion: '2024-11-05',
-      capabilities: { tools: {} },
-      clientInfo: { name: 'sse-test', version: '1.0.0' },
-    }).catch(() => undefined);
+    await client
+      .sendRequest('initialize', {
+        protocolVersion: '2024-11-05',
+        capabilities: { tools: {} },
+        clientInfo: { name: 'sse-test', version: '1.0.0' },
+      })
+      .catch(() => undefined);
 
     const p1 = await runTestGroup('Prompts', TEMPLATE_TESTS.prompts, client);
     const p2 = await runTestGroup('Resources', TEMPLATE_TESTS.resources, client);

package/src/tests/mcp/test-stdio.js

@@ -18,7 +18,7 @@ const __dirname = dirname(__filename);
 
 const serverPath = join(__dirname, '../../../dist/template/start.js');
 
-async function runTestGroup (title, tests, client) {
+async function runTestGroup(title, tests, client) {
   console.log(`\n${title}:`);
   let passed = 0;
   for (const test of tests) {
@@ -42,7 +42,7 @@ async function runTestGroup (title, tests, client) {
   return passed;
 }
 
-async function main () {
+async function main() {
   console.log('🧪 STDIO tests for template MCP server');
   console.log('='.repeat(60));
 
@@ -55,11 +55,13 @@ async function main () {
 
   try {
     // Initialize handshake (optional for stdio server; safe to send)
-    await client.send('initialize', {
-      protocolVersion: '2024-11-05',
-      capabilities: { tools: {} },
-      clientInfo: { name: 'stdio-test', version: '1.0.0' },
-    }).catch(() => undefined);
+    await client
+      .send('initialize', {
+        protocolVersion: '2024-11-05',
+        capabilities: { tools: {} },
+        clientInfo: { name: 'stdio-test', version: '1.0.0' },
+      })
+      .catch(() => undefined);
 
     const p1 = await runTestGroup('Prompts', TEMPLATE_TESTS.prompts, client);
     const p2 = await runTestGroup('Resources', TEMPLATE_TESTS.resources, client);
@@ -69,7 +71,9 @@ async function main () {
     const sum = p1 + p2 + p3;
     console.log(`\nSummary: ${sum}/${total} tests passed`);
   } finally {
-    try { proc.kill(); } catch {}
+    try {
+      proc.kill();
+    } catch {}
   }
 }
 

package/src/tests/utils.ts

@@ -26,7 +26,7 @@ export interface ITestResult {
 
   // Temporal metadata
   timestamp: string; // ISO string
-  duration: number;  // milliseconds
+  duration: number; // milliseconds
 
   // Execution status
   status: 'pending' | 'passed' | 'failed' | 'skipped' | 'expected_failure';
@@ -85,7 +85,9 @@ export const formatResultAsMarkdown = (result: ITestResult) => {
 
   let requestHeaders = '';
   if (result.requestHeaders && Object.keys(result.requestHeaders).length > 0) {
-    requestHeaders = `\nHeaders:\n${Object.entries(result.requestHeaders).map(([k, v]) => `  ${k}: ${v}`).join('\n')}\n`;
+    requestHeaders = `\nHeaders:\n${Object.entries(result.requestHeaders)
+      .map(([k, v]) => `  ${k}: ${v}`)
+      .join('\n')}\n`;
   }
 
   // Format response section
@@ -121,7 +123,6 @@ export const formatResultAsMarkdown = (result: ITestResult) => {
         }
         responseText = `## Response\n\n${mdJson(text)}\n\n${addText}`;
       }
-
     } catch {
       // Fallback to text if any parsing errors
       responseText = `## Response\n\n${mdText(String(result.response))}\n\n`;

package/cli-template/eslint.config.js

@@ -1,27 +0,0 @@
-import baseConfig from 'eslint-config-at-26-2';
-
-export default [
-  ...baseConfig,
-
-  // Project-specific ignores not covered by base config
-  {
-    ignores: [
-      'cli-template/',
-    ],
-  },
-
-  // Rules that were not in old config — disable to match previous behavior
-  {
-    rules: {
-      'no-console': 'off',
-      'import/order': 'off',
-      'prefer-arrow/prefer-arrow-functions': 'off',
-    },
-  },
-
-  // JS-specific overrides
-  {
-    files: ['**/*.js', '**/*.mjs', '**/*.cjs'],
-    rules: {},
-  },
-];