@cloudstreamsoftware/knowledge-mcp-client 1.0.0 → 1.0.2

package/README.md

@@ -1,4 +1,4 @@
-# @cloudstream/knowledge-mcp-client
+# @cloudstreamsoftware/knowledge-mcp-client
 
 MCP client bridge for CloudStream Knowledge Server. Translates Claude Code's stdio-based MCP protocol to HTTP requests against the Cloudflare Knowledge Server.
 
@@ -26,7 +26,7 @@ Add to `~/.claude/settings.json`:
   "mcpServers": {
     "cloudstream-knowledge": {
       "command": "npx",
-      "args": ["@cloudstream/knowledge-mcp-client"],
+      "args": ["@cloudstreamsoftware/knowledge-mcp-client"],
       "env": {
         "CLOUDSTREAM_LICENSE": "your-license-key",
         "CLOUDSTREAM_API": "https://cloudstream-knowledge.broken-rain-0984.workers.dev"
@@ -64,7 +64,7 @@ When the Knowledge Server is unavailable, the bridge returns helpful error messa
 Enable debug logging:
 
 ```bash
-CLOUDSTREAM_DEBUG=true npx @cloudstream/knowledge-mcp-client
+CLOUDSTREAM_DEBUG=true npx @cloudstreamsoftware/knowledge-mcp-client
 ```
 
 Debug output goes to stderr (doesn't interfere with JSON-RPC protocol on stdout).
@@ -75,7 +75,7 @@ Debug output goes to stderr (doesn't interfere with JSON-RPC protocol on stdout)
 # Test initialize (no network required)
 echo '{"jsonrpc":"2.0","id":1,"method":"initialize"}' | \
   CLOUDSTREAM_LICENSE=test CLOUDSTREAM_API=http://localhost \
-  npx @cloudstream/knowledge-mcp-client
+  npx @cloudstreamsoftware/knowledge-mcp-client
 
 # Expected response:
 # {"jsonrpc":"2.0","id":1,"result":{"protocolVersion":"2024-11-05",...}}

package/dist/bridge.d.ts

@@ -31,7 +31,8 @@ export declare class HttpBridge {
      */
     private handleError;
     /**
-     * Generate offline mode response with helpful message
+     * Generate offline mode response
+     * Per JSON-RPC 2.0, failures must return error, not result
      */
     private offlineModeResponse;
     /**

package/dist/bridge.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"bridge.d.ts","sourceRoot":"","sources":["../src/bridge.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,WAAW,EAAE,YAAY,EAAE,eAAe,EAAE,MAAM,SAAS,CAAC;AAGjF;;;;;;;;GAQG;AACH,qBAAa,UAAU;IACrB,OAAO,CAAC,MAAM,CAAe;IAC7B,OAAO,CAAC,eAAe,CAAiC;IACxD,OAAO,CAAC,SAAS,CAAsB;gBAE3B,MAAM,EAAE,YAAY;IAIhC;;OAEG;IACG,OAAO,CAAC,OAAO,EAAE,UAAU,GAAG,OAAO,CAAC,WAAW,CAAC;IAgBxD;;;OAGG;IACH,OAAO,CAAC,gBAAgB;IAiBxB;;OAEG;YACW,WAAW;IAwDzB;;OAEG;IACH,OAAO,CAAC,WAAW;IAoBnB;;OAEG;IACH,OAAO,CAAC,mBAAmB;IA0C3B;;OAEG;IACH,OAAO,CAAC,aAAa;IAYrB;;OAEG;IACH,kBAAkB,IAAI,eAAe;IAIrC;;OAEG;IACH,YAAY,IAAI,KAAK,GAAG,IAAI;CAG7B"}
+{"version":3,"file":"bridge.d.ts","sourceRoot":"","sources":["../src/bridge.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,WAAW,EAAE,YAAY,EAAE,eAAe,EAAE,MAAM,SAAS,CAAC;AAGjF;;;;;;;;GAQG;AACH,qBAAa,UAAU;IACrB,OAAO,CAAC,MAAM,CAAe;IAC7B,OAAO,CAAC,eAAe,CAAiC;IACxD,OAAO,CAAC,SAAS,CAAsB;gBAE3B,MAAM,EAAE,YAAY;IAIhC;;OAEG;IACG,OAAO,CAAC,OAAO,EAAE,UAAU,GAAG,OAAO,CAAC,WAAW,CAAC;IAgBxD;;;OAGG;IACH,OAAO,CAAC,gBAAgB;IAiBxB;;OAEG;YACW,WAAW;IAwDzB;;OAEG;IACH,OAAO,CAAC,WAAW;IAoBnB;;;OAGG;IACH,OAAO,CAAC,mBAAmB;IA8B3B;;OAEG;IACH,OAAO,CAAC,aAAa;IAYrB;;OAEG;IACH,kBAAkB,IAAI,eAAe;IAIrC;;OAEG;IACH,YAAY,IAAI,KAAK,GAAG,IAAI;CAG7B"}

package/dist/bridge.js

@@ -115,10 +115,11 @@ class HttpBridge {
         `Bridge error: ${error.message}`);
     }
     /**
-     * Generate offline mode response with helpful message
+     * Generate offline mode response
+     * Per JSON-RPC 2.0, failures must return error, not result
      */
     offlineModeResponse(request) {
-        // For tools/list, return empty list with offline notice
+        // For tools/list, return empty list (graceful degradation)
         if (request.method === 'tools/list') {
             return {
                 jsonrpc: '2.0',
@@ -128,27 +129,11 @@ class HttpBridge {
                 },
             };
         }
-        // For tool calls, return friendly error
+        // For tool calls, return proper JSON-RPC error
         if (request.method === 'tools/call') {
             const toolName = request.params?.name || 'unknown tool';
-            return {
-                jsonrpc: '2.0',
-                id: request.id,
-                result: {
-                    content: [
-                        {
-                            type: 'text',
-                            text: `[OFFLINE] The CloudStream Knowledge Server is currently unavailable.\n\n` +
-                                `Unable to process: ${toolName}\n\n` +
-                                `Please check:\n` +
-                                `1. Your internet connection\n` +
-                                `2. Knowledge Server status at ${this.config.apiUrl}/health\n` +
-                                `3. Your license key validity\n\n` +
-                                `Local skills and web search remain available.`,
-                        },
-                    ],
-                },
-            };
+            return this.errorResponse(request.id, -32003, `Knowledge Server unavailable. Unable to process: ${toolName}. ` +
+                `Check your internet connection and try again.`);
         }
         return this.errorResponse(request.id, -32003, 'Knowledge Server unavailable');
     }

package/dist/index.js

@@ -99,56 +99,78 @@ async function main() {
     }
     // Create HTTP bridge
     const bridge = new bridge_1.HttpBridge(config);
+    // Track pending async operations to prevent premature exit
+    const pendingRequests = new Set();
+    let stdinClosed = false;
     // Set up readline interface for stdin
     const rl = readline.createInterface({
         input: process.stdin,
         terminal: false,
     });
     // Process JSON-RPC requests line by line
-    rl.on('line', async (line) => {
+    rl.on('line', (line) => {
         debug(`Received: ${line.slice(0, 100)}...`);
         if (!line.trim()) {
             return; // Ignore empty lines
         }
-        let request;
-        try {
-            request = JSON.parse(line);
-        }
-        catch {
-            // Parse error
-            sendResponse({
-                jsonrpc: '2.0',
-                id: null,
-                error: {
-                    code: -32700,
-                    message: 'Parse error: Invalid JSON',
-                },
-            });
-            return;
-        }
-        // Forward to bridge and send response
-        try {
-            const response = await bridge.forward(request);
-            debug(`Response for ${request.method}: ${bridge.getConnectionState()}`);
-            sendResponse(response);
-        }
-        catch (error) {
-            // Unexpected error - should be handled by bridge
-            debug(`Unexpected error: ${error.message}`);
-            sendResponse({
-                jsonrpc: '2.0',
-                id: request.id,
-                error: {
-                    code: -32603,
-                    message: `Internal error: ${error.message}`,
-                },
-            });
-        }
+        // Handle request asynchronously but track the promise
+        const requestPromise = (async () => {
+            let request;
+            try {
+                request = JSON.parse(line);
+            }
+            catch {
+                // Parse error
+                sendResponse({
+                    jsonrpc: '2.0',
+                    id: null,
+                    error: {
+                        code: -32700,
+                        message: 'Parse error: Invalid JSON',
+                    },
+                });
+                return;
+            }
+            // Forward to bridge and send response
+            try {
+                const response = await bridge.forward(request);
+                debug(`Response for ${request.method}: ${bridge.getConnectionState()}`);
+                sendResponse(response);
+            }
+            catch (error) {
+                // Unexpected error - should be handled by bridge
+                debug(`Unexpected error: ${error.message}`);
+                sendResponse({
+                    jsonrpc: '2.0',
+                    id: request.id,
+                    error: {
+                        code: -32603,
+                        message: `Internal error: ${error.message}`,
+                    },
+                });
+            }
+        })();
+        // Track this request
+        pendingRequests.add(requestPromise);
+        requestPromise.finally(() => {
+            pendingRequests.delete(requestPromise);
+            // If stdin is closed and no pending requests, exit
+            if (stdinClosed && pendingRequests.size === 0) {
+                debug('All requests completed, exiting');
+                process.exit(0);
+            }
+        });
     });
-    // Handle stdin close
-    rl.on('close', () => {
-        debug('stdin closed, exiting');
-        process.exit(0);
+    // Handle stdin close - wait for pending requests before exiting
+    rl.on('close', async () => {
+        debug(`stdin closed, ${pendingRequests.size} pending requests`);
+        stdinClosed = true;
+        // If no pending requests, exit immediately
+        if (pendingRequests.size === 0) {
+            debug('No pending requests, exiting');
+            process.exit(0);
+        }
+        // Otherwise, requests will exit when they complete (see finally block above)
     });
     // Handle errors
     process.on('uncaughtException', (error) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cloudstreamsoftware/knowledge-mcp-client",
-  "version": "1.0.0",
+  "version": "1.0.2",
   "description": "MCP client bridge for CloudStream Knowledge Server - translates stdio to HTTP",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",