@ecommaps/mcp 1.0.7 → 1.0.10

package/README.md

@@ -7,22 +7,12 @@ Streamable HTTP endpoint (`/api/v1/mcp`) while speaking stdio locally.
 
 ## Quick Start
 
-Run directly with your store key:
-
-```bash
-npx @ecommaps/mcp <YOUR_STORE_MCP_KEY>
-```
-
-Use a custom endpoint (production or local):
+Run with your store key and the official endpoint:
 
 ```bash
 npx @ecommaps/mcp <YOUR_STORE_MCP_KEY> --url https://api.ecommaps.com/api/v1/mcp
 ```
 
-```bash
-npx @ecommaps/mcp <YOUR_STORE_MCP_KEY> --url http://127.0.0.1:8001/api/v1/mcp
-```
-
 ## CLI Options
 
 - `--url <MCP_URL>`: Override the MCP endpoint URL.
@@ -43,6 +33,7 @@ Use the following shape in any editor that supports stdio MCP servers:
 {
   "command": "npx",
   "args": [
+    "-y",
     "@ecommaps/mcp",
     "sk_eco_xxxxxxxxxxxxxxxxxxxxxxxxxxxxx",
     "--url",
@@ -56,13 +47,16 @@ Or with env var:
 ```json
 {
   "command": "npx",
-  "args": ["@ecommaps/mcp", "--url", "https://api.ecommaps.com/api/v1/mcp"],
+  "args": ["-y", "@ecommaps/mcp", "--url", "https://api.ecommaps.com/api/v1/mcp"],
   "env": {
     "MCP_BEARER_TOKEN": "sk_eco_xxxxxxxxxxxxxxxxxxxxxxxxxxxxx"
   }
 }
 ```
 
+Important: keep `--url https://api.ecommaps.com/api/v1/mcp` in editor/server configs to avoid falling back to any outdated transport defaults.
+Also make sure `--url` comes **after** `@ecommaps/mcp` in `args`; otherwise `npx` treats it as an npm flag and fails.
+
 ## Transport Notes
 
 - Recommended endpoint: `POST /api/v1/mcp`

package/cli.js

@@ -11,17 +11,18 @@ function parseCliArgs(argv) {
   let showHelp = false;
 
   for (let i = 0; i < args.length; i += 1) {
-    const arg = args[i];
+    const rawArg = args[i];
+    const arg = typeof rawArg === 'string' ? rawArg.replace(/[–—−]/g, '-') : rawArg;
     if (arg === '--help' || arg === '-h') {
       showHelp = true;
       continue;
     }
-    if (arg === '--url' && args[i + 1]) {
+    if ((arg === '--url' || arg === '-url') && args[i + 1]) {
       mcpUrl = args[i + 1];
       i += 1;
       continue;
     }
-    if (arg === '--api-key' && args[i + 1]) {
+    if ((arg === '--api-key' || arg === '-api-key') && args[i + 1]) {
       apiKey = args[i + 1];
       i += 1;
       continue;
@@ -71,14 +72,14 @@ function normalizeResponsePayload(rawBody, contentType) {
   }
 }
 
-async function postRpc(mcpUrl, apiKey, rpcPayload) {
+async function postRpc(mcpUrl, apiKey, rpcPayload, protocolVersion) {
   const response = await fetch(mcpUrl, {
     method: 'POST',
     headers: {
       Authorization: `Bearer ${apiKey}`,
       'Content-Type': 'application/json',
       Accept: 'application/json, text/event-stream',
-      'MCP-Protocol-Version': '2025-06-18',
+      'MCP-Protocol-Version': protocolVersion || '2025-06-18',
     },
     body: JSON.stringify(rpcPayload),
   });
@@ -92,6 +93,11 @@ async function postRpc(mcpUrl, apiKey, rpcPayload) {
 
   const payload = normalizeResponsePayload(rawBody, contentType);
   if (payload === null) {
+    const isNotification = !Object.prototype.hasOwnProperty.call(rpcPayload ?? {}, 'id');
+    if (isNotification) {
+      // JSON-RPC notifications legitimately have no response body.
+      return null;
+    }
     throw new Error('MCP response was empty.');
   }
 
@@ -122,6 +128,7 @@ async function main() {
   });
 
   let queue = Promise.resolve();
+  let negotiatedProtocolVersion = '2025-06-18';
 
   rl.on('line', (line) => {
     const trimmed = line.trim();
@@ -137,16 +144,46 @@ async function main() {
       }
 
       try {
-        const result = await postRpc(mcpUrl, apiKey, payload);
-        process.stdout.write(`${JSON.stringify(result)}\n`);
+        // Some MCP clients may send params as [] or omit them for list methods.
+        // Ecommaps MCP expects params to be an object, so normalize empty variants.
+        if (payload && typeof payload === 'object') {
+          if (
+            payload.params == null ||
+            (Array.isArray(payload.params) && payload.params.length === 0)
+          ) {
+            payload.params = {};
+          }
+        }
+
+        const requestedVersion =
+          payload?.method === 'initialize' && typeof payload?.params?.protocolVersion === 'string'
+            ? payload.params.protocolVersion
+            : negotiatedProtocolVersion;
+
+        const result = await postRpc(mcpUrl, apiKey, payload, requestedVersion);
+        if (
+          payload?.method === 'initialize' &&
+          result &&
+          typeof result === 'object' &&
+          result.result &&
+          typeof result.result.protocolVersion === 'string'
+        ) {
+          negotiatedProtocolVersion = result.result.protocolVersion;
+        }
+
+        if (result !== null) {
+          process.stdout.write(`${JSON.stringify(result)}\n`);
+        }
       } catch (error) {
         const message = error && error.message ? error.message : String(error);
         console.error(message);
         // Best-effort JSON-RPC error response so MCP clients stay synchronized.
-        const id = payload && Object.prototype.hasOwnProperty.call(payload, 'id') ? payload.id : null;
-        process.stdout.write(
-          `${JSON.stringify({ jsonrpc: '2.0', id, error: { code: -32000, message } })}\n`
-        );
+        // Notifications do not expect responses.
+        if (payload && Object.prototype.hasOwnProperty.call(payload, 'id')) {
+          process.stdout.write(
+            `${JSON.stringify({ jsonrpc: '2.0', id: payload.id, error: { code: -32000, message } })}\n`
+          );
+        }
       }
     });
   });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ecommaps/mcp",
-  "version": "1.0.7",
+  "version": "1.0.10",
   "description": "Official Ecommaps MCP stdio bridge",
   "bin": {
     "mcp": "cli.js"