grove-mcp 1.0.6 → 1.0.7

package/index.js

@@ -96,91 +96,126 @@ let sessionPath = null;
 let affinityCookie = null; // sticky-session cookie from SSE response headers
 const pending = [];
 
-// ── SSE connection ──────────────────────────────────────────────────────────
-
-const sseReq = https.request(
-  {
-    hostname: HOSTNAME,
-    path: SSE_PATH,
-    method: 'GET',
-    headers: { Accept: 'text/event-stream' },
-  },
-  (res) => {
-    if (res.statusCode !== 200) {
-      process.stderr.write(
-        `grove-mcp-bridge: SSE connect failed (HTTP ${res.statusCode})\n`
-      );
-      process.exit(1);
-    }
+// ── SSE connection with reconnect ───────────────────────────────────────────
 
-    // Capture nginx sticky-session cookie so POSTs reach the same pod
-    const setCookie = res.headers['set-cookie'];
-    if (setCookie) {
-      const cookies = Array.isArray(setCookie) ? setCookie : [setCookie];
-      const parts = cookies.map((c) => c.split(';')[0].trim());
-      affinityCookie = parts.join('; ');
-    }
+let reconnectDelay = 1000; // ms; doubles on each failure, capped at 30s
 
-    let buf = '';
-    let eventType = '';
+function connect() {
+  // Reset session state so stdin messages are queued until the new endpoint arrives
+  sessionPath = null;
+  affinityCookie = null;
 
-    res.on('data', (chunk) => {
-      buf += chunk.toString('utf8');
+  const sseReq = https.request(
+    {
+      hostname: HOSTNAME,
+      path: SSE_PATH,
+      method: 'GET',
+      headers: { Accept: 'text/event-stream' },
+    },
+    (res) => {
+      if (res.statusCode !== 200) {
+        process.stderr.write(
+          `grove-mcp-bridge: SSE connect failed (HTTP ${res.statusCode}), retrying in ${reconnectDelay}ms\n`
+        );
+        res.on('error', () => {}); // prevent unhandled-error crash while draining
+        res.resume();
+        scheduleReconnect();
+        return;
+      }
 
-      let nl;
-      while ((nl = buf.indexOf('\n')) !== -1) {
-        const line = buf.slice(0, nl).replace(/\r$/, '');
-        buf = buf.slice(nl + 1);
+      // Successful connection — reset backoff
+      reconnectDelay = 1000;
 
-        if (line.startsWith('event:')) {
-          eventType = line.slice(6).trim();
-        } else if (line.startsWith('data:')) {
-          const raw = line.slice(5).trim();
+      // Capture nginx sticky-session cookie so POSTs reach the same pod
+      const setCookie = res.headers['set-cookie'];
+      if (setCookie) {
+        const cookies = Array.isArray(setCookie) ? setCookie : [setCookie];
+        const parts = cookies.map((c) => c.split(';')[0].trim());
+        affinityCookie = parts.join('; ');
+      }
 
-          if (eventType === 'endpoint') {
-            try {
-              sessionPath = JSON.parse(raw).uri;
-            } catch (_) {
-              // Fallback: server sent a plain path instead of JSON
-              sessionPath = raw;
+      let buf = '';
+      let eventType = '';
+
+      res.on('data', (chunk) => {
+        buf += chunk.toString('utf8');
+
+        let nl;
+        while ((nl = buf.indexOf('\n')) !== -1) {
+          const line = buf.slice(0, nl).replace(/\r$/, '');
+          buf = buf.slice(nl + 1);
+
+          if (line.startsWith('event:')) {
+            eventType = line.slice(6).trim();
+          } else if (line.startsWith('data:')) {
+            const raw = line.slice(5).trim();
+
+            if (eventType === 'endpoint') {
+              try {
+                sessionPath = JSON.parse(raw).uri;
+              } catch (_) {
+                // Fallback: server sent a plain path instead of JSON
+                sessionPath = raw;
+              }
+              // Flush any stdin lines that arrived before the endpoint was ready
+              for (const msg of pending) sendToServer(msg);
+              pending.length = 0;
+            } else if (eventType === 'message' && raw) {
+              // Strip modern MCP spec fields that cause Claude Desktop to silently
+              // drop all tools (anthropics/claude-code#25081): FastMCP 3.x emits
+              // outputSchema in tools/list which Claude Desktop doesn't understand.
+              process.stdout.write(stripMcpCompat(raw) + '\n');
             }
-            // Flush any stdin lines that arrived before the endpoint was ready
-            for (const msg of pending) sendToServer(msg);
-            pending.length = 0;
-          } else if (eventType === 'message' && raw) {
-            // Strip modern MCP spec fields that cause Claude Desktop to silently
-            // drop all tools (anthropics/claude-code#25081): FastMCP 3.x emits
-            // outputSchema in tools/list which Claude Desktop doesn't understand.
-            process.stdout.write(stripMcpCompat(raw) + '\n');
-          }
 
-          eventType = '';
-        } else if (line === '') {
-          eventType = '';
+            eventType = '';
+          } else if (line === '') {
+            eventType = '';
+          }
         }
-      }
-    });
-
-    res.on('end', () => {
-      process.stderr.write('grove-mcp-bridge: SSE stream ended\n');
-      process.exit(0);
-    });
+      });
 
-    res.on('error', (err) => {
-      process.stderr.write(`grove-mcp-bridge: SSE error: ${err.message}\n`);
-      process.exit(1);
-    });
-  }
-);
+      res.on('end', () => {
+        process.stderr.write(`grove-mcp-bridge: SSE stream ended, reconnecting in ${reconnectDelay}ms\n`);
+        scheduleReconnect();
+      });
 
-sseReq.on('error', (err) => {
-  process.stderr.write(
-    `grove-mcp-bridge: cannot reach ${HOSTNAME}: ${err.message}\n`
+      res.on('error', (err) => {
+        process.stderr.write(`grove-mcp-bridge: SSE error: ${err.message}, reconnecting in ${reconnectDelay}ms\n`);
+        scheduleReconnect();
+      });
+    }
   );
-  process.exit(1);
-});
 
-sseReq.end();
+  sseReq.on('error', (err) => {
+    process.stderr.write(
+      `grove-mcp-bridge: cannot reach ${HOSTNAME}: ${err.message}, retrying in ${reconnectDelay}ms\n`
+    );
+    scheduleReconnect();
+  });
+
+  // Server sends keepalive pings every 30s. If nothing arrives in 75s the
+  // connection is stale (server crash, network partition) — destroy and reconnect.
+  sseReq.setTimeout(75000);
+  sseReq.on('timeout', () => {
+    process.stderr.write(`grove-mcp-bridge: SSE idle timeout, reconnecting in ${reconnectDelay}ms\n`);
+    sseReq.destroy(); // triggers sseReq.on('error') or res.on('error') → scheduleReconnect()
+  });
+
+  sseReq.end();
+}
+
+function scheduleReconnect() {
+  // Null the session immediately so stdin messages are queued rather than
+  // fired at a dead endpoint during the reconnect delay window.
+  sessionPath = null;
+  affinityCookie = null;
+  setTimeout(() => {
+    connect();
+  }, reconnectDelay);
+  reconnectDelay = Math.min(reconnectDelay * 2, 30000);
+}
+
+connect();
 
 // ── stdin → POST ────────────────────────────────────────────────────────────
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "grove-mcp",
-  "version": "1.0.6",
+  "version": "1.0.7",
   "description": "Claude Desktop bridge for the Grove MCP Server (Mattermost + Outline)",
   "main": "index.js",
   "bin": {