converse-mcp-server 2.3.1 → 2.4.1

package/src/continuationStore.js

@@ -106,7 +106,7 @@ class MemoryContinuationStore extends ContinuationStoreInterface {
       if (!continuationId || typeof continuationId !== 'string') {
         throw new ContinuationStoreError(
           'Invalid continuation ID: must be a non-empty string',
-          'INVALID_CONTINUATION_ID'
+          'INVALID_CONTINUATION_ID',
         );
       }
 
@@ -114,7 +114,7 @@ class MemoryContinuationStore extends ContinuationStoreInterface {
       if (!state || typeof state !== 'object') {
         throw new ContinuationStoreError(
           'Invalid state: must be an object',
-          'INVALID_STATE'
+          'INVALID_STATE',
         );
       }
       // Cleanup old conversations if we hit the limit
@@ -125,8 +125,13 @@ class MemoryContinuationStore extends ContinuationStoreInterface {
 
       // Limit messages per conversation to prevent memory issues
       const sanitizedState = { ...state };
-      if (sanitizedState.messages && sanitizedState.messages.length > this.maxMessagesPerConversation) {
-        sanitizedState.messages = sanitizedState.messages.slice(-this.maxMessagesPerConversation);
+      if (
+        sanitizedState.messages &&
+        sanitizedState.messages.length > this.maxMessagesPerConversation
+      ) {
+        sanitizedState.messages = sanitizedState.messages.slice(
+          -this.maxMessagesPerConversation,
+        );
       }
 
       // Store with metadata
@@ -137,14 +142,13 @@ class MemoryContinuationStore extends ContinuationStoreInterface {
           ? this.conversations.get(continuationId).createdAt
           : Date.now(),
       });
-
     } catch (error) {
       if (error instanceof ContinuationStoreError) {
         throw error;
       }
       throw new ContinuationStoreError(
         `Failed to store continuation: ${error.message}`,
-        'STORAGE_ERROR'
+        'STORAGE_ERROR',
       );
     }
   }
@@ -161,7 +165,7 @@ class MemoryContinuationStore extends ContinuationStoreInterface {
       if (!continuationId || typeof continuationId !== 'string') {
         throw new ContinuationStoreError(
           'Invalid continuation ID: must be a non-empty string',
-          'INVALID_CONTINUATION_ID'
+          'INVALID_CONTINUATION_ID',
         );
       }
 
@@ -173,18 +177,17 @@ class MemoryContinuationStore extends ContinuationStoreInterface {
         const { createdAt, lastAccessed, ...cleanState } = state;
         return {
           ...cleanState,
-          _metadata: { createdAt, lastAccessed }
+          _metadata: { createdAt, lastAccessed },
         };
       }
       return null;
-
     } catch (error) {
       if (error instanceof ContinuationStoreError) {
         throw error;
       }
       throw new ContinuationStoreError(
         `Failed to retrieve continuation: ${error.message}`,
-        'RETRIEVAL_ERROR'
+        'RETRIEVAL_ERROR',
       );
     }
   }
@@ -201,21 +204,20 @@ class MemoryContinuationStore extends ContinuationStoreInterface {
       if (!continuationId || typeof continuationId !== 'string') {
         throw new ContinuationStoreError(
           'Invalid continuation ID: must be a non-empty string',
-          'INVALID_CONTINUATION_ID'
+          'INVALID_CONTINUATION_ID',
         );
       }
 
       const existed = this.conversations.has(continuationId);
       this.conversations.delete(continuationId);
       return existed;
-
     } catch (error) {
       if (error instanceof ContinuationStoreError) {
         throw error;
       }
       throw new ContinuationStoreError(
         `Failed to delete continuation: ${error.message}`,
-        'DELETION_ERROR'
+        'DELETION_ERROR',
       );
     }
   }
@@ -274,16 +276,21 @@ export function getContinuationStore() {
     continuationStore = new MemoryContinuationStore();
 
     // Set up periodic cleanup (runs every hour)
-    cleanupIntervalId = setInterval(async () => {
-      try {
-        const cleaned = await continuationStore.cleanup();
-        if (cleaned > 0) {
-          debugLog(`ContinuationStore: Cleaned up ${cleaned} old conversations`);
+    cleanupIntervalId = setInterval(
+      async () => {
+        try {
+          const cleaned = await continuationStore.cleanup();
+          if (cleaned > 0) {
+            debugLog(
+              `ContinuationStore: Cleaned up ${cleaned} old conversations`,
+            );
+          }
+        } catch (error) {
+          debugError('ContinuationStore cleanup failed:', error);
         }
-      } catch (error) {
-        debugError('ContinuationStore cleanup failed:', error);
-      }
-    }, 60 * 60 * 1000);
+      },
+      60 * 60 * 1000,
+    );
 
     // Unref the interval so it doesn't keep the process alive
     if (cleanupIntervalId && typeof cleanupIntervalId.unref === 'function') {
@@ -301,7 +308,7 @@ export function setContinuationStore(store) {
   if (!(store instanceof ContinuationStoreInterface)) {
     throw new ContinuationStoreError(
       'Store must extend ContinuationStoreInterface',
-      'INVALID_STORE'
+      'INVALID_STORE',
     );
   }
   continuationStore = store;
@@ -341,7 +348,8 @@ export function isValidContinuationId(continuationId) {
   const nanoidPattern = /^conv_[A-Za-z0-9_-]{10}$/;
 
   // Also accept legacy UUID format for backward compatibility
-  const uuidPattern = /^conv_[0-9a-f]{8}-[0-9a-f]{4}-[1-5][0-9a-f]{3}-[89ab][0-9a-f]{3}-[0-9a-f]{12}$/i;
+  const uuidPattern =
+    /^conv_[0-9a-f]{8}-[0-9a-f]{4}-[1-5][0-9a-f]{3}-[89ab][0-9a-f]{3}-[0-9a-f]{12}$/i;
 
   return nanoidPattern.test(continuationId) || uuidPattern.test(continuationId);
 }

package/src/index.js

@@ -10,7 +10,12 @@
 import { fileURLToPath, pathToFileURL } from 'url';
 import { Server } from '@modelcontextprotocol/sdk/server/index.js';
 import { StdioServerTransport } from '@modelcontextprotocol/sdk/server/stdio.js';
-import { loadConfig, validateRuntimeConfig, getMcpClientConfig, getHttpTransportConfig } from './config.js';
+import {
+  loadConfig,
+  validateRuntimeConfig,
+  getMcpClientConfig,
+  getHttpTransportConfig,
+} from './config.js';
 import { createRouter } from './router.js';
 import { createHTTPTransport } from './transport/httpTransport.js';
 import { createLogger, startTimer } from './utils/logger.js';
@@ -55,7 +60,7 @@ function getTransportType() {
   const args = process.argv.slice(2);
 
   // Support --transport=value format
-  const transportEqualArg = args.find(arg => arg.startsWith('--transport='));
+  const transportEqualArg = args.find((arg) => arg.startsWith('--transport='));
   if (transportEqualArg) {
     const transport = transportEqualArg.split('=')[1];
     if (transport && ['http', 'stdio'].includes(transport)) {
@@ -64,7 +69,7 @@ function getTransportType() {
   }
 
   // Support --transport value format
-  const transportIndex = args.findIndex(arg => arg === '--transport');
+  const transportIndex = args.findIndex((arg) => arg === '--transport');
   if (transportIndex >= 0 && transportIndex + 1 < args.length) {
     const transport = args[transportIndex + 1];
     if (transport && ['http', 'stdio'].includes(transport)) {
@@ -102,7 +107,7 @@ async function main() {
     const { configureLogger } = await import('./utils/logger.js');
     configureLogger({
       level: process.env.LOG_LEVEL || 'info',
-      isDevelopment: process.env.NODE_ENV === 'development'
+      isDevelopment: process.env.NODE_ENV === 'development',
     });
   }
 
@@ -121,7 +126,7 @@ async function main() {
     logger.info('Using transport type', { data: { transport: transportType } });
 
     logger.debug('Creating MCP server instance', {
-      data: { name: mcpConfig.name, version: mcpConfig.version }
+      data: { name: mcpConfig.name, version: mcpConfig.version },
     });
 
     // Create MCP server with configuration
@@ -130,7 +135,7 @@ async function main() {
         name: mcpConfig.name,
         version: mcpConfig.version,
       },
-      mcpConfig
+      mcpConfig,
     );
 
     // Set up router with server and config
@@ -146,14 +151,17 @@ async function main() {
       const status = httpTransport.getStatus();
 
       const startupTime = serverTimer('completed');
-      logger.info('Converse MCP Server started successfully with HTTP transport', {
-        data: {
-          startupTime: `${startupTime}ms`,
-          endpoint: `http://${status.host}:${status.port}/mcp`,
-          host: status.host,
-          port: status.port
-        }
-      });
+      logger.info(
+        'Converse MCP Server started successfully with HTTP transport',
+        {
+          data: {
+            startupTime: `${startupTime}ms`,
+            endpoint: `http://${status.host}:${status.port}/mcp`,
+            host: status.host,
+            port: status.port,
+          },
+        },
+      );
 
       // Store reference for shutdown
       process.httpTransport = httpTransport;
@@ -163,9 +171,12 @@ async function main() {
       await server.connect(transport);
 
       const startupTime = serverTimer('completed');
-      logger.info('Converse MCP Server started successfully with stdio transport', {
-        data: { startupTime: `${startupTime}ms` }
-      });
+      logger.info(
+        'Converse MCP Server started successfully with stdio transport',
+        {
+          data: { startupTime: `${startupTime}ms` },
+        },
+      );
     }
   } catch (error) {
     serverTimer('failed');
@@ -176,7 +187,7 @@ async function main() {
       debugError(error.message);
       if (error.details?.errors) {
         debugError('\nDetailed errors:');
-        error.details.errors.forEach(err => debugError(`  - ${err}`));
+        error.details.errors.forEach((err) => debugError(`  - ${err}`));
       }
       process.exit(1);
     } else {
@@ -194,8 +205,12 @@ async function gracefulShutdown(signal) {
 
   // Stop all cleanup intervals
   try {
-    const { stopContinuationStoreCleanup } = await import('./continuationStore.js');
-    const { stopAsyncJobStoreCleanup } = await import('./async/asyncJobStore.js');
+    const { stopContinuationStoreCleanup } = await import(
+      './continuationStore.js'
+    );
+    const { stopAsyncJobStoreCleanup } = await import(
+      './async/asyncJobStore.js'
+    );
     const { stopFileCacheCleanup } = await import('./async/fileCache.js');
 
     stopContinuationStoreCleanup();
@@ -226,8 +241,12 @@ process.on('beforeExit', async (code) => {
   logger.info('Process beforeExit event', { data: { code } });
   // Cleanup intervals if not already done
   try {
-    const { stopContinuationStoreCleanup } = await import('./continuationStore.js');
-    const { stopAsyncJobStoreCleanup } = await import('./async/asyncJobStore.js');
+    const { stopContinuationStoreCleanup } = await import(
+      './continuationStore.js'
+    );
+    const { stopAsyncJobStoreCleanup } = await import(
+      './async/asyncJobStore.js'
+    );
     const { stopFileCacheCleanup } = await import('./async/fileCache.js');
 
     stopContinuationStoreCleanup();
@@ -251,7 +270,7 @@ process.on('uncaughtException', (error) => {
 process.on('unhandledRejection', (reason, promise) => {
   logger.error('Unhandled promise rejection', {
     error: reason,
-    data: { promise: promise.toString() }
+    data: { promise: promise.toString() },
   });
   debugError('Unhandled promise rejection:', reason);
   process.exit(1);
@@ -262,8 +281,9 @@ export { main };
 
 // Check if this module is the main entry point
 // This works better across platforms and Node.js versions
-const isMainModule = import.meta.url === pathToFileURL(process.argv[1]).href ||
-                     process.argv[1] === fileURLToPath(import.meta.url);
+const isMainModule =
+  import.meta.url === pathToFileURL(process.argv[1]).href ||
+  process.argv[1] === fileURLToPath(import.meta.url);
 
 if (isMainModule) {
   main().catch((error) => {