reportdash-datastore-mcp-claude-desktop 1.0.4 → 1.0.6

package/index.js

@@ -41,7 +41,8 @@ rl.on('line', (line) => {
     const mcpRequest = JSON.parse(line);
     forwardToAPI(mcpRequest);
   } catch (error) {
-    console.error(JSON.stringify({
+    // Only output to stdout, never stderr for MCP protocol
+    console.log(JSON.stringify({
       jsonrpc: '2.0',
       error: {
         code: -32700,
@@ -78,14 +79,26 @@ function forwardToAPI(mcpRequest) {
     });
 
     res.on('end', () => {
-      if (res.statusCode === 200) {
-        console.log(data);
+      // Treat 200-299 as success (including 204 No Content)
+      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',
+            result: {},
+            id: mcpRequest.id || null
+          }));
+        } else {
+          // Forward the response as-is
+          console.log(data);
+        }
       } else {
-        console.error(JSON.stringify({
+        // 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}`,
+            message: `API error: ${res.statusCode}${data ? ' - ' + data : ''}`,
             data: { statusCode: res.statusCode, body: data }
           },
           id: mcpRequest.id || null
@@ -95,7 +108,8 @@ function forwardToAPI(mcpRequest) {
   });
 
   req.on('error', (error) => {
-    console.error(JSON.stringify({
+    // Send errors to stdout (not stderr!)
+    console.log(JSON.stringify({
       jsonrpc: '2.0',
       error: {
         code: -32603,
@@ -108,7 +122,8 @@ function forwardToAPI(mcpRequest) {
 
   req.on('timeout', () => {
     req.destroy();
-    console.error(JSON.stringify({
+    // Send errors to stdout (not stderr!)
+    console.log(JSON.stringify({
       jsonrpc: '2.0',
       error: {
         code: -32603,
@@ -162,28 +177,37 @@ function testConnection() {
     res.on('end', () => {
       console.log(`Response Status: ${res.statusCode}\n`);
 
-      if (res.statusCode === 200) {
-        try {
-          const response = JSON.parse(data);
+      if (res.statusCode >= 200 && res.statusCode < 300) {
+        if (res.statusCode === 204 || !data.trim()) {
           console.log('✅ Connection successful!');
-          console.log('✅ API key is valid\n');
-
-          if (response.result && response.result.tools) {
-            console.log(`📦 Available Tools: ${response.result.tools.length}\n`);
-            response.result.tools.forEach((tool, index) => {
-              console.log(`${index + 1}. ${tool.name}`);
-              console.log(`   ${tool.description}\n`);
-            });
-          } else {
-            console.log('Response:', JSON.stringify(response, null, 2));
+          console.log('✅ API key is valid');
+          console.log('ℹ️  Server returned 204 No Content\n');
+        } else {
+          try {
+            const response = JSON.parse(data);
+            console.log('✅ Connection successful!');
+            console.log('✅ API key is valid\n');
+
+            if (response.result && response.result.tools) {
+              console.log(`📦 Available Tools: ${response.result.tools.length}\n`);
+              response.result.tools.slice(0, 5).forEach((tool, index) => {
+                console.log(`${index + 1}. ${tool.name}`);
+                console.log(`   ${tool.description}\n`);
+              });
+              if (response.result.tools.length > 5) {
+                console.log(`... and ${response.result.tools.length - 5} more tools`);
+              }
+            } else {
+              console.log('Response:', JSON.stringify(response, null, 2));
+            }
+          } catch (e) {
+            console.log('✅ Connection successful but response parsing failed');
+            console.log('Raw response:', data);
           }
-
-          console.log('\n🎉 You can now use ReportDash DataStore in Claude Desktop!');
-          console.log('\nTry asking Claude: "list my reportdash datastore sources"');
-        } catch (e) {
-          console.log('✅ Connection successful but response parsing failed');
-          console.log('Raw response:', data);
         }
+
+        console.log('\n🎉 You can now use ReportDash DataStore in Claude Desktop!');
+        console.log('\nTry asking Claude: "list my reportdash datastore sources"');
       } else {
         console.log('❌ Connection failed');
         console.log(`Status: ${res.statusCode}`);

package/package.json

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