@link-assistant/hive-mind 2.0.4 → 2.0.5

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # @link-assistant/hive-mind
 
+## 2.0.5
+
+### Patch Changes
+
+- d815c7d: Treat a Claude Code `pending` Playwright MCP `system.init` status as a normal
+  still-connecting state instead of a failure (#1901). Claude Code enables Tool
+  Search by default, so the deferred `mcp__playwright__*` browser tools load on
+  demand and Claude waits for the connecting server before using them. Hive Mind
+  no longer aborts the working session on a `pending` status; only a terminal
+  `failed`/`error` status surfaces a non-blocking diagnostic in the session-start
+  comment. See `docs/case-studies/issue-1901`.
+
 ## 2.0.4
 
 ### Patch Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@link-assistant/hive-mind",
-  "version": "2.0.4",
+  "version": "2.0.5",
   "description": "AI-powered issue solver and hive mind for collaborative problem solving",
   "main": "src/hive.mjs",
   "type": "module",

package/src/interactive-mcp-status.lib.mjs

@@ -1,21 +1,36 @@
 const PLAYWRIGHT_TOOL_PREFIX = 'mcp__playwright__';
 
-export const isUnavailableMcpStatus = status => {
+// A `pending` (or `connecting`) MCP server is still being connected/reconnected
+// in the background. It is NOT a failure: Claude Code enables Tool Search by
+// default, so MCP tools are deferred and load on demand, and Claude waits for a
+// still-connecting server before it uses one of that server's tools. See
+// https://code.claude.com/docs/en/mcp and issue #1901.
+export const isConnectingMcpStatus = status => /\b(pending|connecting)\b/i.test(String(status || ''));
+
+// Terminal/unhealthy states where the MCP client has given up (or the server is
+// turned off). Claude Code reconnects an HTTP/SSE server with exponential
+// backoff and only marks it `failed` after the attempts are exhausted; at that
+// point the server's tools never load.
+export const isFailedMcpStatus = status => {
   const normalized = String(status || '').toLowerCase();
-  return /\b(pending|disabled|failed|error|disconnected|not[-_\s]+connected|unavailable|timed[-_\s]+out)\b|(?:^|[^a-z0-9_-])timeout(?:$|[^a-z0-9_-])/.test(normalized);
+  return /\b(disabled|failed|error|disconnected|not[-_\s]+connected|unavailable|timed[-_\s]+out)\b|(?:^|[^a-z0-9_-])timeout(?:$|[^a-z0-9_-])/.test(normalized);
 };
 
+// Backwards-compatible umbrella: any non-connected status (still connecting OR
+// failed). Prefer the narrower helpers above when the connecting/failed
+// distinction matters (e.g. whether to warn a human reviewer).
+export const isUnavailableMcpStatus = status => isConnectingMcpStatus(status) || isFailedMcpStatus(status);
+
 export const hasPlaywrightMcpTools = tools => (Array.isArray(tools) ? tools : []).some(tool => String(tool || '').startsWith(PLAYWRIGHT_TOOL_PREFIX));
 
 export const formatInteractiveMcpServerStatus = server => {
   const name = server?.name || 'unknown';
   const status = String(server?.status || 'unknown').trim() || 'unknown';
-  const normalizedStatus = status.toLowerCase();
   let displayStatus = status;
 
-  if (normalizedStatus === 'pending') {
-    displayStatus = 'pending - not connected; MCP tools unavailable';
-  } else if (isUnavailableMcpStatus(status)) {
+  if (isConnectingMcpStatus(status)) {
+    displayStatus = `${status} - connecting; tools load on demand via Tool Search`;
+  } else if (isFailedMcpStatus(status)) {
     displayStatus = `${status} - MCP tools unavailable`;
   }
 
@@ -29,10 +44,16 @@ export const getInteractiveMcpDiagnostics = (mcpServers = [], tools = []) => {
   for (const server of servers) {
     const name = String(server?.name || '').toLowerCase();
     if (!name.includes('playwright')) continue;
-    if (!isUnavailableMcpStatus(server?.status)) continue;
+    // With Tool Search the deferred `mcp__playwright__*` tools are intentionally
+    // absent from system.init `tools`, so their absence is not a problem by
+    // itself. If they are already present the server is fully connected.
     if (hasPlaywrightMcpTools(tools)) continue;
+    // `pending`/`connecting` is the normal startup state — Claude waits for the
+    // server before using a browser tool — so only warn when the MCP client has
+    // actually failed to connect.
+    if (!isFailedMcpStatus(server?.status)) continue;
 
-    diagnostics.push(`⚠️ Playwright MCP server is ${server?.status || 'unknown'}, but no \`${PLAYWRIGHT_TOOL_PREFIX}*\` browser tools were exposed. Browser automation hints are disabled until the MCP client reports the server as connected.`);
+    diagnostics.push(`⚠️ Playwright MCP server is ${server?.status || 'unknown'} (failed to connect), so no \`${PLAYWRIGHT_TOOL_PREFIX}*\` browser tools are available. Browser automation stays disabled until the MCP server connects.`);
   }
 
   return diagnostics;