workfullcircle-mcp-local 1.3.1 → 1.4.1

package/index.js

@@ -1,13 +1,13 @@
 #!/usr/bin/env node
 
 const fetch = require('node-fetch');
+const https = require('https');
+const { URL } = require('url');
 
 // Parse command line arguments
 const args = process.argv.slice(2);
 
 function printUsage() {
-  process.stderr.write('Usage: workfullcircle-mcp-local --token <api-token> [--url <api-url>]\n');
-  process.stderr.write('Environment variables: WFC_API_URL (default: https://workfullcircle.com)\n');
   process.exit(1);
 }
 
@@ -33,7 +33,6 @@ function parseArgs(argv) {
   }
 
   if (!parsed.token) {
-    process.stderr.write('Error: --token argument or WFC_TOKEN environment variable is required\n');
     printUsage();
   }
 
@@ -42,6 +41,58 @@ function parseArgs(argv) {
 
 const { token: API_TOKEN, url: API_URL } = parseArgs(args);
 
+// Create HTTPS agent with keepAlive disabled to prevent stale connections
+const httpsAgent = new https.Agent({
+  keepAlive: false, // Force fresh connection for each request
+  timeout: 10000   // 10 second timeout
+});
+
+// Helper function to make HTTP requests with retry logic
+async function makeRequest(url, payload, retryCount = 0) {
+  const controller = new AbortController();
+  const timeout = setTimeout(() => controller.abort(), 15000); // 15s timeout
+
+  try {
+    const response = await fetch(url, {
+      method: 'POST',
+      headers: {
+        'Content-Type': 'application/json',
+        'Authorization': `Bearer ${API_TOKEN}`,
+        'Connection': 'close' // Explicitly request connection close
+      },
+      body: JSON.stringify(payload),
+      agent: httpsAgent,
+      signal: controller.signal
+    });
+
+    clearTimeout(timeout);
+    
+    if (!response.ok) {
+      throw new Error(`HTTP ${response.status}: ${response.statusText}`);
+    }
+
+    const responseText = await response.text();
+    
+    // Validate response is valid JSON
+    try {
+      JSON.parse(responseText);
+      return responseText;
+    } catch (jsonError) {
+      throw new Error('Invalid JSON response from server');
+    }
+  } catch (error) {
+    clearTimeout(timeout);
+    
+    // If it's a timeout or connection error, retry once
+    if ((error.name === 'AbortError' || error.code === 'ECONNRESET' || error.message.includes('timeout')) && retryCount === 0) {
+      console.error(`Request failed, retrying... (${error.message})`);
+      return makeRequest(url, payload, retryCount + 1);
+    }
+    
+    throw error;
+  }
+}
+
 // Forward all MCP requests directly to the server's MCP endpoint
 async function handleStdio() {
   let inputBuffer = '';
@@ -63,46 +114,31 @@ async function handleStdio() {
       try {
         payload = JSON.parse(line);
       } catch (error) {
-        process.stderr.write(`[WFC-Local] Invalid JSON input: ${error.message}\n`);
         continue;
       }
 
       try {
-        // Forward the MCP request directly to the server
-        const response = await fetch(API_URL, {
-          method: 'POST',
-          headers: {
-            'Content-Type': 'application/json',
-            'Authorization': `Bearer ${API_TOKEN}`
-          },
-          body: JSON.stringify(payload)
-        });
-
-        const responseText = await response.text();
-
-        if (!response.ok) {
-          process.stderr.write(`[WFC-Local] Remote error ${response.status}: ${responseText}\n`);
-          continue;
-        }
-
-        // Validate response is valid JSON before outputting
-        try {
-          JSON.parse(responseText);
-          process.stdout.write(`${responseText}\n`);
-        } catch (jsonError) {
-          process.stderr.write(`[WFC-Local] Invalid JSON response: ${jsonError.message}\n`);
-          process.stderr.write(`[WFC-Local] Response was: ${responseText.substring(0, 200)}...\n`);
-        }
+        // Forward the MCP request directly to the server using the robust request function
+        const responseText = await makeRequest(API_URL, payload);
+        process.stdout.write(`${responseText}\n`);
       } catch (error) {
-        process.stderr.write(`[WFC-Local] Request failed: ${error.message}\n`);
+        console.error(`MCP request failed: ${error.message}`);
+        // Send error response back to MCP client
+        const errorResponse = JSON.stringify({
+          jsonrpc: "2.0",
+          id: payload.id || null,
+          error: {
+            code: -32603,
+            message: `Request failed: ${error.message}`
+          }
+        });
+        process.stdout.write(`${errorResponse}\n`);
       }
     }
   });
 }
 
 async function main() {
-  process.stderr.write(`[WFC-Local] Connected to MCP server at ${API_URL}\n`);
-  process.stderr.write(`[WFC-Local] Using token: ${API_TOKEN.substring(0, 10)}...\n`);
   await handleStdio();
 }
 

package/package.json

@@ -1,8 +1,8 @@
 {
   "$schema": "https://json.schemastore.org/package",
   "name": "workfullcircle-mcp-local",
-  "version": "1.3.1",
-  "description": "Local STDIO MCP server for WorkFullCircle API with Antigravity path fixes",
+  "version": "1.4.1",
+  "description": "Local STDIO MCP server for WorkFullCircle API with connection leak fixes",
   "main": "index.js",
   "bin": {
     "workfullcircle-mcp-local": "index.js"

package/test.js

@@ -1,98 +0,0 @@
-const { spawn } = require('child_process');
-const path = require('path');
-
-// Test configuration
-const TEST_API_KEY = process.env.TEST_API_KEY || 'uam_test_key';
-const TEST_API_URL = process.env.TEST_API_URL || process.env.WFC_API_URL || 'https://workfullcircle.com/api';
-
-console.log('Testing @workfullcircle/mcp-local...');
-console.log('API URL:', TEST_API_URL);
-console.log('API Key:', TEST_API_KEY.substring(0, 10) + '...');
-
-// Spawn the MCP server
-const mcpServer = spawn('node', [path.join(__dirname, 'index.js')], {
-  env: {
-    ...process.env,
-    WFC_API_URL: TEST_API_URL,
-    WFC_API_KEY: TEST_API_KEY
-  },
-  stdio: ['pipe', 'pipe', 'pipe']
-});
-
-let responseBuffer = '';
-
-mcpServer.stdout.on('data', (data) => {
-  responseBuffer += data.toString();
-  
-  // Try to parse complete JSON responses
-  const lines = responseBuffer.split('\n');
-  for (let i = 0; i < lines.length - 1; i++) {
-    const line = lines[i].trim();
-    if (line) {
-      try {
-        const response = JSON.parse(line);
-        console.log('Response:', JSON.stringify(response, null, 2));
-      } catch (e) {
-        console.log('Non-JSON output:', line);
-      }
-    }
-  }
-  responseBuffer = lines[lines.length - 1];
-});
-
-mcpServer.stderr.on('data', (data) => {
-  console.error('Error:', data.toString());
-});
-
-mcpServer.on('close', (code) => {
-  console.log('Server exited with code:', code);
-});
-
-// Send test requests
-setTimeout(() => {
-  console.log('Sending initialize request...');
-  mcpServer.stdin.write(JSON.stringify({
-    jsonrpc: '2.0',
-    id: 1,
-    method: 'initialize',
-    params: {
-      protocolVersion: '2024-11-05',
-      capabilities: {},
-      clientInfo: {
-        name: 'test-client',
-        version: '1.0.0'
-      }
-    }
-  }) + '\n');
-}, 1000);
-
-setTimeout(() => {
-  console.log('Sending tools/list request...');
-  mcpServer.stdin.write(JSON.stringify({
-    jsonrpc: '2.0',
-    id: 2,
-    method: 'tools/list'
-  }) + '\n');
-}, 2000);
-
-setTimeout(() => {
-  console.log('Sending tools/call request...');
-  mcpServer.stdin.write(JSON.stringify({
-    jsonrpc: '2.0',
-    id: 3,
-    method: 'tools/call',
-    params: {
-      name: 'save_memory',
-      arguments: {
-        content: 'Test memory from MCP local',
-        sector: 'Semantic',
-        tags: ['test', 'mcp-local']
-      }
-    }
-  }) + '\n');
-}, 3000);
-
-setTimeout(() => {
-  console.log('Terminating...');
-  mcpServer.kill();
-}, 5000);