@sanity-labs/nuum 0.5.2 → 0.5.4

package/dist/index.js

@@ -35674,6 +35674,20 @@ function estimateSummaryTokens(input) {
 }
 // src/agent/loop.ts
 var log5 = Log.create({ service: "agent-loop" });
+var MODEL_MAX_OUTPUT_TOKENS = {
+  "claude-opus-4-6": 128000,
+  "claude-opus-4-6-20250918": 128000,
+  "claude-sonnet-4-5-20250929": 64000,
+  "claude-sonnet-4-5": 64000,
+  "claude-haiku-4-5-20251001": 64000,
+  "claude-haiku-4-5": 64000,
+  "claude-3-5-sonnet-20241022": 8192,
+  "claude-3-5-haiku-20241022": 8192
+};
+var DEFAULT_MAX_OUTPUT_TOKENS = 16384;
+function getMaxOutputTokens(modelId) {
+  return MODEL_MAX_OUTPUT_TOKENS[modelId] ?? DEFAULT_MAX_OUTPUT_TOKENS;
+}
 function addCacheMarkers(messages) {
   if (messages.length === 0)
     return messages;
@@ -35702,7 +35716,7 @@ async function runAgentLoop(options) {
     systemPrompt,
     initialMessages,
     tools,
-    maxTokens = 4096,
+    maxTokens: maxTokensOverride,
     temperature,
     maxTurns,
     abortSignal,
@@ -35713,6 +35727,12 @@ async function runAgentLoop(options) {
     onBeforeTurn,
     onThinking
   } = options;
+  const maxTokens = maxTokensOverride ?? getMaxOutputTokens(model.modelId);
+  log5.info("agent loop starting", {
+    model: model.modelId,
+    maxTokens,
+    maxTurns
+  });
   if (abortSignal?.aborted) {
     throw new AgentLoopCancelledError;
   }
@@ -35768,6 +35788,13 @@ async function runAgentLoop(options) {
         cacheHitRate: total > 0 ? `${Math.round(cacheRead / total * 100)}%` : "0%"
       });
     }
+    if (response.finishReason === "length") {
+      log5.warn("output truncated - model hit maxTokens limit", {
+        maxTokens,
+        outputTokens: response.usage.completionTokens,
+        hasToolCalls: (response.toolCalls?.length ?? 0) > 0
+      });
+    }
     if (response.text) {
       finalText = response.text;
       await onText?.(response.text);
@@ -35812,6 +35839,15 @@ async function runAgentLoop(options) {
         content: toolResultParts
       };
       messages.push(toolMsg);
+      if (response.finishReason === "length") {
+        const hadInvalidCalls = toolCallInfos.some((tc) => tc.toolName === "__invalid_tool_call__");
+        if (hadInvalidCalls) {
+          messages.push({
+            role: "user",
+            content: "[SYSTEM: Your previous output was truncated because it exceeded the output token limit. " + "Your tool call was incomplete \u2014 parameters were cut off mid-generation. " + "To fix this: break large content into smaller chunks, or use bash with echo/cat to write files incrementally. " + "Do NOT retry the same large tool call \u2014 it will truncate again.]"
+          });
+        }
+      }
       if (isDone(toolCallInfos)) {
         stopReason = "done";
         break;
@@ -36138,7 +36174,6 @@ async function runCompaction(storage, config) {
         systemPrompt: ctx.systemPrompt,
         initialMessages,
         tools,
-        maxTokens: 4096,
         temperature: 0,
         maxTurns: 5,
         isDone: stopOnTool("finish_distillation"),
@@ -36244,7 +36279,7 @@ async function runSubAgent(storage, config) {
     extractResult,
     tier = "workhorse",
     maxTurns = 20,
-    maxTokens = 4096,
+    maxTokens,
     temperature = 0,
     onToolResult
   } = config;
@@ -36431,8 +36466,7 @@ async function runReflection(storage, question) {
     finishToolName: "finish_reflection",
     extractResult: getAnswer,
     tier: "workhorse",
-    maxTurns: 20,
-    maxTokens: 4096
+    maxTurns: 20
   });
   const answer = result.result ?? "Unable to find relevant information.";
   activity.reflection.complete(`${result.turnsUsed} turns, ${answer.length} chars`);
@@ -36836,7 +36870,6 @@ async function runResearch(storage, topic) {
     },
     tier: "workhorse",
     maxTurns: MAX_RESEARCH_TURNS,
-    maxTokens: 8192,
     onToolResult: (toolCallId) => {
       const toolResult = getLastResult(toolCallId);
       if (!toolResult)
@@ -45086,6 +45119,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 +45181,8 @@ var Mcp;
           config: config2,
           client: null,
           tools: [],
+          allToolCount: 0,
+          issues: [],
           status: "disabled"
         };
       }
@@ -45141,24 +45200,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 +45242,8 @@ var Mcp;
           config: config2,
           client,
           tools: [],
+          allToolCount: 0,
+          issues: [],
           status: "failed",
           error: error2
         };
@@ -45185,10 +45260,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 +45289,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 +45339,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 +45404,8 @@ var Mcp;
 })(Mcp ||= {});
 
 // src/version.ts
-var VERSION = "0.5.2";
-var GIT_HASH = "91ca6b6";
+var VERSION = "0.5.4";
+var GIT_HASH = "e1f89a6";
 var VERSION_STRING = `nuum v${VERSION} (${GIT_HASH})`;
 
 // src/tool/mcp-status.ts
@@ -45369,14 +45454,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}
 `;
@@ -46829,7 +46923,6 @@ async function runConsolidation(storage, messages) {
     },
     tier: "workhorse",
     maxTurns: MAX_CONSOLIDATION_TURNS,
-    maxTokens: 2048,
     onToolResult: (toolCallId) => {
       const toolResult = getLastResult(toolCallId);
       if (!toolResult)
@@ -47371,7 +47464,6 @@ async function runAgent(prompt, options) {
     systemPrompt: ctx.systemPrompt,
     initialMessages,
     tools,
-    maxTokens: 8192,
     maxTurns: MAX_TURNS,
     abortSignal,
     onText: async (text3) => {

package/package.json

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