converse-mcp-server 4.6.0 → 4.6.2

package/docs/API.md

@@ -474,7 +474,7 @@ A one-line summary per answer (options at 0.00 omitted), followed by the answers
 Decision · typesafe/jev-1.13-20260917 via OpenRouter · 394 input tokens · $0.000017
 - urgent (noul): 0.97
 - team (choice): billing · confidence 1.00 · billing 1.00
-- frustration (score): 1.24 on 0–2 · confidence 0.64 · 1 Frustrated 0.76, 2 Very angry 0.24
+- frustration (score): 1.24 on 0–2 · confidence 0.64 · 1 "Frustrated" 0.76, 2 "Very angry" 0.24
 
 ```json
 {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "converse-mcp-server",
-  "version": "4.6.0",
+  "version": "4.6.2",
   "description": "Converse MCP Server - Converse with other LLMs with chat and consensus tools",
   "type": "module",
   "main": "src/index.js",
@@ -93,22 +93,22 @@
     ".env.example"
   ],
   "dependencies": {
-    "@anthropic-ai/claude-agent-sdk": "^0.3.293",
-    "@anthropic-ai/sdk": "^0.132.0",
-    "@github/copilot-sdk": "^1.0.17",
-    "@google/genai": "^2.27.0",
+    "@anthropic-ai/claude-agent-sdk": "^0.3.296",
+    "@anthropic-ai/sdk": "^0.133.0",
+    "@github/copilot-sdk": "^1.0.19",
+    "@google/genai": "^2.28.0",
     "@lydell/node-pty": "1.2.0-beta.15",
     "@mistralai/mistralai": "^2.7.0",
     "@modelcontextprotocol/sdk": "^1.32.1",
-    "@openai/codex-sdk": "^0.161.0",
+    "@openai/codex-sdk": "^0.162.0",
     "cors": "^2.8.6",
-    "dotenv": "^18.0.6",
-    "express": "^5.2.1",
+    "dotenv": "^18.0.7",
+    "express": "^5.3.0",
     "lru-cache": "^11.5.3",
     "nanoid": "^6.0.2",
-    "openai": "^7.30.0",
+    "openai": "^7.31.0",
     "p-limit": "^7.3.3",
-    "vite": "^8.3.3"
+    "vite": "^8.3.4"
   },
   "devDependencies": {
     "@vitest/coverage-v8": "^5.0.3",

package/src/decisionProviders/formats.js

@@ -73,22 +73,26 @@ function decodedBytes(image) {
   return Math.floor((image.base64.length * 3) / 4);
 }
 
+/** Clef's limits hold on every host, so a breach is checked before sending. */
+function clefLimitError(message) {
+  return new DecisionError(message, { terminal: true, notSent: true });
+}
+
 function checkClefImages(images, state, questions) {
   const sizes = images.map(decodedBytes);
   const tooLarge = sizes.findIndex((bytes) => bytes > CLEF_MAX_IMAGE_BYTES);
   if (tooLarge !== -1) {
-    throw new DecisionError(`Clef accepts images up to 4 MiB each; image ${tooLarge + 1} is ${(sizes[tooLarge] / 1048576).toFixed(1)} MiB.`, { terminal: true });
+    throw clefLimitError(`Clef accepts images up to 4 MiB each; image ${tooLarge + 1} is ${(sizes[tooLarge] / 1048576).toFixed(1)} MiB.`);
   }
   if (sizes.reduce((a, b) => a + b, 0) > CLEF_MAX_TOTAL_IMAGE_BYTES) {
-    throw new DecisionError('Clef accepts at most 8 MiB of images per call.', { terminal: true });
+    throw clefLimitError('Clef accepts at most 8 MiB of images per call.');
   }
   const chars = images.reduce((n, image) => n + dataUrl(image).length, 0) + asText(state ?? '').length + JSON.stringify(questions).length;
   const estimate = Math.ceil(chars / CLEF_CHARS_PER_ESTIMATED_TOKEN);
   if (estimate > CLEF_CONTEXT_TOKENS) {
-    throw new DecisionError(
+    throw clefLimitError(
       `Cloudflare estimates this request at ~${estimate} tokens (4 base64 characters per token), over Clef's ${CLEF_CONTEXT_TOKENS}-token window. ` +
       'Downscale or recompress the images (about 180 KB in total fits), or use gpt-6-luna.',
-      { terminal: true },
     );
   }
 }
@@ -104,14 +108,11 @@ export const cloudflareFormat = {
   request(model, state, questions, images = []) {
     const ids = Object.keys(questions);
     if (ids.length > CLEF_MAX_QUESTIONS) {
-      throw new DecisionError(`Clef accepts at most ${CLEF_MAX_QUESTIONS} questions per call (got ${ids.length}).`, { terminal: true });
+      throw clefLimitError(`Clef accepts at most ${CLEF_MAX_QUESTIONS} questions per call (got ${ids.length}).`);
     }
     const bad = ids.filter((id) => !CLEF_QUESTION_ID.test(id));
     if (bad.length) {
-      throw new DecisionError(
-        `Clef question names may use only letters, digits, "_", ".", and "-" (max 100 characters): ${bad.join(', ')}.`,
-        { terminal: true },
-      );
+      throw clefLimitError(`Clef question names may use only letters, digits, "_", ".", and "-" (max 100 characters): ${bad.join(', ')}.`);
     }
     const body = { model, state: state ?? '', questions: toSystemOneQuestions(questions) };
     if (images.length === 0) return body;

package/src/decisionProviders/http.js

@@ -21,15 +21,18 @@ const MAX_RETRY_AFTER_MS = 30_000;
 /**
  * Error from a decision call. `retryable` marks failures worth repeating or
  * failing over (network, timeout, 408/429/5xx, auth); `terminal` marks request
- * faults that every host of the same model would reject the same way.
+ * faults that every host of the same model would reject the same way;
+ * `notSent` marks a request refused by a local limit check before any network
+ * call, so it is not reported as a failed request.
  */
 export class DecisionError extends Error {
-  constructor(message, { status = null, retryable = false, terminal = false, requestId = null, retryAfterMs = null } = {}) {
+  constructor(message, { status = null, retryable = false, terminal = false, notSent = false, requestId = null, retryAfterMs = null } = {}) {
     super(message);
     this.name = 'DecisionError';
     this.status = status;
     this.retryable = retryable;
     this.terminal = terminal;
+    this.notSent = notSent;
     this.requestId = requestId;
     this.retryAfterMs = retryAfterMs;
   }

package/src/tools/decide.js

@@ -250,7 +250,8 @@ function summarizeAnswer(key, answer) {
   if (type === 'score') {
     const levels = Object.keys(answer.legend || answer.probabilities || {});
     const range = levels.length ? ` on 0–${levels.length - 1}` : '';
-    const label = (k) => (answer.legend?.[k] ? `${k} ${answer.legend[k]}` : k);
+    // Quoted so a numeric label ("4") stays distinct from its level index (3).
+    const label = (k) => (answer.legend?.[k] ? `${k} "${answer.legend[k]}"` : k);
     return `- ${key} (score): ${fixed(answer.score)}${range} · confidence ${fixed(answer.confidence)} · ${byProbability(answer.probabilities, label)}`;
   }
   if (type === 'refusal') {
@@ -269,7 +270,7 @@ function formatResult(response, candidate, failures) {
 
   const lines = [header];
   for (const failure of failures) {
-    lines.push(`(${failure.provider} failed, fell back: ${failure.message})`);
+    lines.push(`(${failure.provider} ${failure.notSent ? 'skipped' : 'failed'}, fell back: ${failure.message})`);
   }
   lines.push(...Object.entries(response.answers).map(([key, answer]) => summarizeAnswer(key, answer)));
 
@@ -352,13 +353,14 @@ export async function decideTool(args, dependencies) {
     } catch (error) {
       if (signal?.aborted) return createToolError('Decision request cancelled.');
       logger.error('Decision request failed', { provider: candidate.providerName, model: candidate.model, error: error.message });
-      failures.push({ provider: candidate.providerName, message: error.message });
+      failures.push({ provider: candidate.providerName, message: error.message, notSent: Boolean(error.notSent) });
       if (error.terminal) rejectedFamilies.add(candidate.family);
     }
   }
 
-  const detail = failures.map((f) => `${f.provider}: ${f.message}`).join('; ');
-  return createToolError(`Decision request failed (${detail})`);
+  const noneSent = failures.every((f) => f.notSent);
+  const detail = failures.map((f) => `${f.provider}${f.notSent && !noneSent ? ', not sent' : ''}: ${f.message}`).join('; ');
+  return createToolError(`Decision request ${noneSent ? 'not sent' : 'failed'} (${detail})`);
 }
 
 decideTool.description =