@sanity-labs/nuum 0.5.2 → 0.5.3

package/dist/index.js

@@ -45086,6 +45086,30 @@ var Mcp;
   function isHttpConfig(config2) {
     return "url" in config2;
   }
+  const TOOL_NAME_PATTERN = /^[a-zA-Z0-9_-]{1,64}$/;
+  function validateToolName(serverName, mcpToolName) {
+    const effectiveName = `${serverName}__${mcpToolName}`;
+    if (TOOL_NAME_PATTERN.test(effectiveName)) {
+      return null;
+    }
+    const invalidChars = effectiveName.split("").filter((c) => !/[a-zA-Z0-9_-]/.test(c));
+    const uniqueInvalid = Array.from(new Set(invalidChars));
+    let message;
+    if (effectiveName.length > 64) {
+      message = `Effective tool name "${effectiveName}" exceeds 64 character limit (${effectiveName.length} chars)`;
+    } else if (effectiveName.length === 0) {
+      message = `Tool name is empty`;
+    } else {
+      message = `Effective tool name "${effectiveName}" contains invalid character(s): ${uniqueInvalid.map((c) => `"${c}"`).join(", ")} (allowed: a-z, A-Z, 0-9, _, -)`;
+    }
+    return {
+      type: "invalid_tool_name",
+      tool: mcpToolName,
+      effectiveName,
+      message
+    };
+  }
+  Mcp.validateToolName = validateToolName;
 
   class Manager {
     servers = new Map;
@@ -45124,6 +45148,8 @@ var Mcp;
           config: config2,
           client: null,
           tools: [],
+          allToolCount: 0,
+          issues: [],
           status: "disabled"
         };
       }
@@ -45141,24 +45167,38 @@ var Mcp;
           new Promise((_, reject) => setTimeout(() => reject(new Error("Connection timeout")), timeoutMs))
         ]);
         const toolsResult = await client.listTools();
-        const tools = toolsResult.tools;
+        const allTools = toolsResult.tools;
+        const validTools = [];
+        const issues = [];
+        for (const mcpTool of allTools) {
+          const issue2 = validateToolName(name17, mcpTool.name);
+          if (issue2) {
+            issues.push(issue2);
+            console.error(`[mcp:${name17}] Skipping tool "${mcpTool.name}": ${issue2.message}`);
+          } else {
+            validTools.push(mcpTool);
+          }
+        }
         let sessionId;
         if (transport instanceof StreamableHTTPClientTransport) {
           sessionId = transport.sessionId;
           if (sessionId) {
-            console.error(`[mcp:${name17}] Connected with session ${sessionId.slice(0, 8)}..., ${tools.length} tools available`);
+            console.error(`[mcp:${name17}] Connected with session ${sessionId.slice(0, 8)}..., ${validTools.length}/${allTools.length} tools available`);
           } else {
-            console.error(`[mcp:${name17}] Connected (no session), ${tools.length} tools available`);
+            console.error(`[mcp:${name17}] Connected (no session), ${validTools.length}/${allTools.length} tools available`);
           }
         } else {
-          console.error(`[mcp:${name17}] Connected, ${tools.length} tools available`);
+          console.error(`[mcp:${name17}] Connected, ${validTools.length}/${allTools.length} tools available`);
         }
+        const status = issues.length > 0 ? "degraded" : "connected";
         return {
           name: name17,
           config: config2,
           client,
-          tools,
-          status: "connected",
+          tools: validTools,
+          allToolCount: allTools.length,
+          issues,
+          status,
           sessionId
         };
       } catch (e) {
@@ -45169,6 +45209,8 @@ var Mcp;
           config: config2,
           client,
           tools: [],
+          allToolCount: 0,
+          issues: [],
           status: "failed",
           error: error2
         };
@@ -45185,10 +45227,18 @@ var Mcp;
         this.servers.set(server.name, server);
       }
       const connected = results.filter((s) => s.status === "connected").length;
+      const degraded = results.filter((s) => s.status === "degraded").length;
       const failed = results.filter((s) => s.status === "failed").length;
       const disabled = results.filter((s) => s.status === "disabled").length;
       if (entries.length > 0) {
-        console.error(`[mcp] Initialized: ${connected} connected, ${failed} failed, ${disabled} disabled`);
+        const parts = [`${connected} connected`];
+        if (degraded > 0)
+          parts.push(`${degraded} degraded`);
+        if (failed > 0)
+          parts.push(`${failed} failed`);
+        if (disabled > 0)
+          parts.push(`${disabled} disabled`);
+        console.error(`[mcp] Initialized: ${parts.join(", ")}`);
       }
     }
     async shutdown() {
@@ -45206,14 +45256,16 @@ var Mcp;
       return Array.from(this.servers.values()).map((s) => ({
         name: s.name,
         status: s.status,
-        toolCount: s.tools.length,
+        toolCount: s.allToolCount,
+        activeToolCount: s.tools.length,
+        issues: s.issues,
         error: s.error
       }));
     }
     listTools() {
       const tools = [];
       for (const [serverName, server] of this.servers) {
-        if (server.status !== "connected")
+        if (server.status !== "connected" && server.status !== "degraded")
           continue;
         for (const mcpTool of server.tools) {
           tools.push(`${serverName}__${mcpTool.name}`);
@@ -45254,7 +45306,7 @@ var Mcp;
     getTools() {
       const tools = {};
       for (const [serverName, server] of this.servers) {
-        if (server.status !== "connected")
+        if (server.status !== "connected" && server.status !== "degraded")
           continue;
         for (const mcpTool of server.tools) {
           const toolName = `${serverName}__${mcpTool.name}`;
@@ -45319,8 +45371,8 @@ var Mcp;
 })(Mcp ||= {});
 
 // src/version.ts
-var VERSION = "0.5.2";
-var GIT_HASH = "91ca6b6";
+var VERSION = "0.5.3";
+var GIT_HASH = "a2a5ddc";
 var VERSION_STRING = `nuum v${VERSION} (${GIT_HASH})`;
 
 // src/tool/mcp-status.ts
@@ -45369,14 +45421,23 @@ Use this when:
     } else {
       let connectedCount = 0;
       for (const server of mcpStatus) {
-        const statusIcon = server.status === "connected" ? "\u2713" : server.status === "connecting" ? "\u22EF" : "\u2717";
+        const statusIcon = server.status === "connected" ? "\u2713" : server.status === "degraded" ? "\u26A0" : server.status === "connecting" ? "\u22EF" : "\u2717";
         output += `**${server.name}** ${statusIcon}
 `;
-        if (server.status === "connected") {
+        if (server.status === "connected" || server.status === "degraded") {
           connectedCount++;
           const serverTools = mcpToolNames.filter((t) => t.startsWith(`${server.name}__`));
-          output += `- ${server.toolCount} tools: ${serverTools.map((t) => t.replace(`${server.name}__`, "")).join(", ")}
+          if (server.status === "degraded") {
+            output += `- ${server.activeToolCount}/${server.toolCount} tools active: ${serverTools.map((t) => t.replace(`${server.name}__`, "")).join(", ")}
 `;
+            for (const issue2 of server.issues) {
+              output += `  - \u26A0 Skipped "${issue2.tool}": ${issue2.message}
+`;
+            }
+          } else {
+            output += `- ${server.toolCount} tools: ${serverTools.map((t) => t.replace(`${server.name}__`, "")).join(", ")}
+`;
+          }
         } else if (server.error) {
           output += `- Error: ${server.error}
 `;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sanity-labs/nuum",
-  "version": "0.5.2",
+  "version": "0.5.3",
   "description": "AI coding agent with continuous memory - infinite context across sessions",
   "type": "module",
   "bin": {