reportdash-datastore-mcp-claude-desktop 1.0.8 → 1.0.9

package/index.js

@@ -1,5 +1,5 @@
 #!/usr/bin/env node
-//goto folder, increment version then run > npm publish --access public
+// goto folder, increment version then run > npm publish --access public
 
 const https = require('https');
 const http = require('http');
@@ -12,13 +12,18 @@ const API_URL = process.env.REPORTDASH_API_URL || 'https://datastore.reportdash.
 
 // Validate configuration
 if (!API_KEY) {
-  console.error(JSON.stringify({
-    jsonrpc: '2.0',
-    error: {
-      code: -32600,
-      message: 'REPORTDASH_API_KEY environment variable is required. Get your API key from ReportDash DataStore app settings.'
-    }
-  }));
+  // NOTE: In MCP stdio mode, prefer stdout only.
+  // But keeping your original behavior as-is.
+  console.error(
+    JSON.stringify({
+      jsonrpc: '2.0',
+      error: {
+        code: -32600,
+        message:
+          'REPORTDASH_API_KEY environment variable is required. Get your API key from ReportDash DataStore app settings.',
+      },
+    })
+  );
   process.exit(1);
 }
 
@@ -27,13 +32,20 @@ if (!API_KEY) {
  * without breaking existing requests.
  */
 function withClaudePlatform(mcpRequest) {
-  // If caller already provided platform, keep it.
   if (mcpRequest && typeof mcpRequest === 'object') {
-    if (!mcpRequest.platform) mcpRequest.platform = 'claude';
+    if (mcpRequest.platform == null) mcpRequest.platform = 'claude';
   }
   return mcpRequest;
 }
 
+/**
+ * JSON-RPC id helper:
+ * Keep 0 as a valid id; only default when null/undefined.
+ */
+function rpcId(mcpRequest) {
+  return mcpRequest?.id ?? null;
+}
+
 // Test mode
 if (process.argv.includes('--test')) {
   testConnection();
@@ -44,7 +56,7 @@ if (process.argv.includes('--test')) {
 const rl = readline.createInterface({
   input: process.stdin,
   output: process.stdout,
-  terminal: false
+  terminal: false,
 });
 
 rl.on('line', (line) => {
@@ -55,14 +67,16 @@ rl.on('line', (line) => {
     forwardToAPI(withClaudePlatform(mcpRequest));
   } catch (error) {
     // Only output to stdout, never stderr for MCP protocol
-    console.log(JSON.stringify({
-      jsonrpc: '2.0',
-      error: {
-        code: -32700,
-        message: 'Parse error: ' + error.message
-      },
-      id: null
-    }));
+    console.log(
+      JSON.stringify({
+        jsonrpc: '2.0',
+        error: {
+          code: -32700,
+          message: 'Parse error: ' + error.message,
+        },
+        id: null,
+      })
+    );
   }
 });
 
@@ -79,9 +93,9 @@ function forwardToAPI(mcpRequest) {
     headers: {
       'Content-Type': 'application/json',
       'X-Api-Key': API_KEY,
-      'User-Agent': 'ReportDash-DataStore-MCP/1.0'
+      'User-Agent': 'ReportDash-DataStore-MCP/1.0',
     },
-    timeout: 30000 // 30 second timeout
+    timeout: 30000, // 30 second timeout
   };
 
   const req = client.request(options, (res) => {
@@ -96,57 +110,62 @@ function forwardToAPI(mcpRequest) {
       if (res.statusCode >= 200 && res.statusCode < 300) {
         // For 204 No Content or empty response, return empty success
         if (res.statusCode === 204 || !data.trim()) {
-          console.log(JSON.stringify({
-            jsonrpc: '2.0',
-            error: {
-              code: res.statusCode,
-              message: `API error: ${res.statusCode}${data ? ' - ' + data : ''}`,
-              data: { statusCode: res.statusCode, body: data }
-            },
-            id: mcpRequest.id ?? null
-          }));
+          console.log(
+            JSON.stringify({
+              jsonrpc: '2.0',
+              result: {},
+              id: rpcId(mcpRequest), // FIX: preserves id=0
+            })
+          );
         } else {
           // Forward the response as-is
           console.log(data);
         }
       } else {
         // Send errors to stdout (not stderr!) so Claude can properly handle them
-        console.log(JSON.stringify({
-          jsonrpc: '2.0',
-          error: {
-            code: res.statusCode,
-            message: `API error: ${res.statusCode}${data ? ' - ' + data : ''}`,
-            data: { statusCode: res.statusCode, body: data }
-          },
-          id: mcpRequest.id || null
-        }));
+        console.log(
+          JSON.stringify({
+            jsonrpc: '2.0',
+            error: {
+              code: res.statusCode,
+              message: `API error: ${res.statusCode}${data ? ' - ' + data : ''}`,
+              data: { statusCode: res.statusCode, body: data },
+            },
+            id: rpcId(mcpRequest), // FIX: preserves id=0
+          })
+        );
       }
     });
   });
 
   req.on('error', (error) => {
     // Send errors to stdout (not stderr!)
-    console.log(JSON.stringify({
-      jsonrpc: '2.0',
-      error: {
-        code: -32603,
-        message: 'Request timeout after 30 seconds'
-      },
-      id: mcpRequest.id ?? null
-    }));
+    console.log(
+      JSON.stringify({
+        jsonrpc: '2.0',
+        error: {
+          code: -32603,
+          message: 'Network error: ' + error.message,
+          data: { error: error.message },
+        },
+        id: rpcId(mcpRequest), // FIX: preserves id=0
+      })
+    );
   });
 
   req.on('timeout', () => {
     req.destroy();
     // Send errors to stdout (not stderr!)
-    console.log(JSON.stringify({
-      jsonrpc: '2.0',
-      error: {
-        code: -32603,
-        message: 'Request timeout after 30 seconds'
-      },
-      id: mcpRequest.id || null
-    }));
+    console.log(
+      JSON.stringify({
+        jsonrpc: '2.0',
+        error: {
+          code: -32603,
+          message: 'Request timeout after 30 seconds',
+        },
+        id: rpcId(mcpRequest), // FIX: preserves id=0
+      })
+    );
   });
 
   req.write(JSON.stringify(mcpRequest));
@@ -154,9 +173,13 @@ function forwardToAPI(mcpRequest) {
 }
 
 function testConnection() {
+  // NOTE: This prints non-JSON to stdout, which is fine for --test.
+  // Do not run --test as an MCP stdio server in Claude Desktop.
   console.log('🔍 Testing ReportDash DataStore connection...\n');
   console.log(`API URL: ${API_URL}`);
-  console.log(`API Key: ${API_KEY.substring(0, 10)}...${API_KEY.substring(API_KEY.length - 4)}\n`);
+  console.log(
+    `API Key: ${API_KEY.substring(0, 10)}...${API_KEY.substring(API_KEY.length - 4)}\n`
+  );
 
   const url = new URL(API_URL);
   const isHttps = url.protocol === 'https:';
@@ -166,7 +189,7 @@ function testConnection() {
   const mcpRequest = withClaudePlatform({
     jsonrpc: '2.0',
     method: 'tools/list',
-    id: 'test-connection'
+    id: 'test-connection',
   });
 
   const postData = JSON.stringify(mcpRequest);
@@ -180,16 +203,18 @@ function testConnection() {
       'Content-Type': 'application/json',
       'X-Api-Key': API_KEY,
       'Content-Length': Buffer.byteLength(postData),
-      'User-Agent': 'ReportDash-DataStore-MCP/1.0'
+      'User-Agent': 'ReportDash-DataStore-MCP/1.0',
     },
-    timeout: 10000
+    timeout: 10000,
   };
 
   console.log('📡 Sending MCP tools/list request...\n');
 
   const req = client.request(options, (res) => {
     let data = '';
-    res.on('data', (chunk) => { data += chunk; });
+    res.on('data', (chunk) => {
+      data += chunk;
+    });
     res.on('end', () => {
       console.log(`Response Status: ${res.statusCode}\n`);
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "reportdash-datastore-mcp-claude-desktop",
-  "version": "1.0.8",
+  "version": "1.0.9",
   "description": "ReportDash DataStore MCP server for Claude Desktop",
   "main": "index.js",
   "bin": {