converse-mcp-server 2.17.1 → 2.19.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "converse-mcp-server",
-  "version": "2.17.1",
+  "version": "2.19.0",
   "description": "Converse MCP Server - Converse with other LLMs with chat and consensus tools",
   "type": "module",
   "main": "src/index.js",

package/src/providers/copilot.js

@@ -507,8 +507,25 @@ function resolveSessionModel(requestModel, config) {
  * - session.idle → processing complete
  * - session.error → { data: { errorType, message } }
  */
+/**
+ * Map tool-level reasoning_effort values to Copilot SDK's ReasoningEffort.
+ * Tool enum:  'none' | 'minimal' | 'low' | 'medium' | 'high' | 'max'
+ * SDK enum:   'low' | 'medium' | 'high' | 'xhigh'
+ */
+function mapReasoningEffort(effort) {
+  const mapping = {
+    none: 'low',
+    minimal: 'low',
+    low: 'low',
+    medium: 'medium',
+    high: 'high',
+    max: 'xhigh',
+  };
+  return mapping[effort] || undefined;
+}
+
 async function* createStreamingGenerator(client, prompt, options, signal, config) {
-  const { model, timeout = 120000 } = options;
+  const { model, timeout = 120000, reasoning_effort } = options;
 
   const sessionModel = resolveSessionModel(model, config);
   const accessLevel = getToolAccessLevel(config);
@@ -522,6 +539,14 @@ async function* createStreamingGenerator(client, prompt, options, signal, config
     sessionConfig.model = sessionModel;
   }
 
+  if (reasoning_effort) {
+    const mapped = mapReasoningEffort(reasoning_effort);
+    if (mapped) {
+      sessionConfig.reasoningEffort = mapped;
+      debugLog(`[Copilot SDK] Setting reasoningEffort: ${mapped} (from ${reasoning_effort})`);
+    }
+  }
+
   const session = await client.createSession(sessionConfig);
 
   try {
@@ -685,12 +710,6 @@ export const copilotProvider = {
         '[Copilot SDK] Parameter "use_websearch" not supported by Copilot SDK (ignored)',
       );
     }
-    if (reasoning_effort !== undefined) {
-      debugLog(
-        '[Copilot SDK] Parameter "reasoning_effort" not supported by Copilot SDK (ignored)',
-      );
-    }
-
     try {
       const cwd = config.server?.client_cwd || process.cwd();
       const client = await getCopilotClient(cwd);
@@ -701,6 +720,7 @@ export const copilotProvider = {
       const invokeOptions = {
         model,
         timeout: modelConfig.timeout,
+        reasoning_effort,
       };
 
       if (stream) {

package/src/tools/chat.js

@@ -13,7 +13,9 @@ import {
 import {
   generateContinuationId,
   addMessageToHistory,
+  isValidContinuationId,
 } from '../continuationStore.js';
+import { isSafeIdSegment } from '../utils/idValidation.js';
 import { debugLog, debugError } from '../utils/console.js';
 import { createLogger } from '../utils/logger.js';
 import { CHAT_PROMPT } from '../systemPrompts.js';
@@ -72,10 +74,28 @@ export async function chatTool(args, dependencies) {
         );
       }
 
+      // Validate custom continuation ID for async safety (used as filesystem path segment)
+      if (continuation_id && !isSafeIdSegment(continuation_id)) {
+        return createToolError(
+          `Invalid continuation_id for async mode: "${continuation_id}". Async IDs must contain only letters, numbers, hyphens, and underscores (max 128 chars).`,
+        );
+      }
+
       // Generate or use existing continuation ID for the conversation
       const conversationContinuationId =
         continuation_id || generateContinuationId();
 
+      // Determine if this is a custom ID (non-standard format AND not found in store)
+      let isCustomId = false;
+      if (continuation_id && !isValidContinuationId(continuation_id)) {
+        try {
+          const existing = await continuationStore.get(continuation_id);
+          isCustomId = !existing;
+        } catch {
+          isCustomId = true;
+        }
+      }
+
       // Get provider and model info for the job
       const providerName = mapModelToProvider(args.model || 'auto', providers);
       const resolvedModel =
@@ -119,6 +139,7 @@ export async function chatTool(args, dependencies) {
               {
                 ...dependencies,
                 continuationId: conversationContinuationId,
+                isCustomId,
                 title, // Pass title to execution context
               },
               context,
@@ -147,6 +168,7 @@ export async function chatTool(args, dependencies) {
           continuation: {
             id: conversationContinuationId, // Use continuation_id as the primary ID
             status: 'processing',
+            ...(isCustomId && { custom_id: true }),
           },
           async_execution: true,
         });
@@ -158,6 +180,7 @@ export async function chatTool(args, dependencies) {
 
     let conversationHistory = [];
     let continuationId = continuation_id;
+    let isCustomId = false;
 
     // Load existing conversation if continuation_id provided
     if (continuationId) {
@@ -166,13 +189,13 @@ export async function chatTool(args, dependencies) {
         if (existingState) {
           conversationHistory = existingState.messages || [];
         } else {
-          // Invalid continuation ID - start fresh with new ID
-          continuationId = generateContinuationId();
+          // Preserve user-provided ID and start fresh conversation
+          isCustomId = !isValidContinuationId(continuationId);
         }
       } catch (error) {
         logger.error('Error loading conversation', { error });
-        // Continue with fresh conversation on error
-        continuationId = generateContinuationId();
+        // Preserve user-provided ID on error
+        isCustomId = !isValidContinuationId(continuationId);
       }
     } else {
       // Generate new continuation ID for new conversation
@@ -450,6 +473,7 @@ export async function chatTool(args, dependencies) {
         model,
         messageCount: updatedMessages.filter((msg) => msg.role !== 'system')
           .length,
+        ...(isCustomId && { custom_id: true }),
       },
     };
 
@@ -700,6 +724,7 @@ async function executeChatWithStreaming(args, dependencies, context) {
     contextProcessor,
     providerStreamNormalizer,
     continuationId,
+    isCustomId,
     title: passedTitle, // Title passed from initial submission
   } = dependencies;
 
@@ -1103,6 +1128,7 @@ async function executeChatWithStreaming(args, dependencies, context) {
       model,
       messageCount: updatedMessages.filter((msg) => msg.role !== 'system')
         .length,
+      ...(isCustomId && { custom_id: true }),
     },
     metadata: {
       provider: providerName,
@@ -1140,7 +1166,7 @@ chatTool.inputSchema = {
     continuation_id: {
       type: 'string',
       description:
-        'Continuation ID for persistent conversation. Example: "chat_1703123456789_abc123"',
+        'Continuation ID for persistent conversation. Example: "chat_1703123456789_abc123". Custom IDs are accepted — an unrecognized ID starts a new conversation under that ID. In async mode, IDs must contain only letters, numbers, hyphens, and underscores (max 128 chars).',
     },
     temperature: {
       type: 'number',

package/src/tools/consensus.js

@@ -17,7 +17,9 @@ import {
 import {
   generateContinuationId,
   addMessageToHistory,
+  isValidContinuationId,
 } from '../continuationStore.js';
+import { isSafeIdSegment } from '../utils/idValidation.js';
 import { debugLog, debugError } from '../utils/console.js';
 import { createLogger } from '../utils/logger.js';
 import { CONSENSUS_PROMPT } from '../systemPrompts.js';
@@ -86,9 +88,27 @@ export async function consensusTool(args, dependencies) {
         );
       }
 
+      // Validate custom continuation ID for async safety (used as filesystem path segment)
+      if (continuation_id && !isSafeIdSegment(continuation_id)) {
+        return createToolError(
+          `Invalid continuation_id for async mode: "${continuation_id}". Async IDs must contain only letters, numbers, hyphens, and underscores (max 128 chars).`,
+        );
+      }
+
       // Generate continuation ID for background execution result
       const bgContinuationId = continuation_id || generateContinuationId();
 
+      // Determine if this is a custom ID (non-standard format AND not found in store)
+      let isCustomId = false;
+      if (continuation_id && !isValidContinuationId(continuation_id)) {
+        try {
+          const existing = await continuationStore.get(continuation_id);
+          isCustomId = !existing;
+        } catch {
+          isCustomId = true;
+        }
+      }
+
       // Create models list for status display
       const modelsList = args.models.join(', ');
 
@@ -128,6 +148,7 @@ export async function consensusTool(args, dependencies) {
               {
                 ...dependencies,
                 continuationId: bgContinuationId,
+                isCustomId,
                 title, // Pass title to execution context
               },
               context,
@@ -156,6 +177,7 @@ export async function consensusTool(args, dependencies) {
           continuation: {
             id: bgContinuationId, // Use continuation_id as the primary ID
             status: 'processing',
+            ...(isCustomId && { custom_id: true }),
           },
           async_execution: true,
         });
@@ -167,6 +189,7 @@ export async function consensusTool(args, dependencies) {
 
     let conversationHistory = [];
     let continuationId = continuation_id;
+    let isCustomId = false;
 
     // Load existing conversation if continuation_id provided
     if (continuationId) {
@@ -176,13 +199,13 @@ export async function consensusTool(args, dependencies) {
         if (existingState) {
           conversationHistory = existingState.messages || [];
         } else {
-          // Invalid continuation ID - start fresh
-          continuationId = generateContinuationId();
+          // Preserve user-provided ID and start fresh conversation
+          isCustomId = !isValidContinuationId(continuationId);
         }
       } catch (error) {
         logger.error('Error loading conversation', { error });
-        // Continue with fresh conversation on error
-        continuationId = generateContinuationId();
+        // Preserve user-provided ID on error
+        isCustomId = !isValidContinuationId(continuationId);
       }
     } else {
       // Generate new continuation ID for new conversation
@@ -661,6 +684,7 @@ Please provide your refined response:`;
       continuation: {
         id: continuationId,
         messageCount: messages.length + 1,
+        ...(isCustomId && { custom_id: true }),
       },
       settings: {
         enable_cross_feedback,
@@ -690,6 +714,7 @@ Please provide your refined response:`;
       continuation: {
         id: continuationId,
         messageCount: messages.length + 1,
+        ...(isCustomId && { custom_id: true }),
       },
     });
   } catch (error) {
@@ -903,6 +928,7 @@ async function executeConsensusWithStreaming(args, dependencies, context) {
     contextProcessor,
     providerStreamNormalizer,
     continuationId,
+    isCustomId,
     title: passedTitle, // Title passed from initial submission
   } = dependencies;
 
@@ -1420,6 +1446,7 @@ Please provide your refined response:`;
     continuation: {
       id: continuationId,
       messageCount: messages.length + 1,
+      ...(isCustomId && { custom_id: true }),
     },
     settings: {
       enable_cross_feedback,
@@ -1695,7 +1722,7 @@ consensusTool.inputSchema = {
     continuation_id: {
       type: 'string',
       description:
-        'Thread continuation ID for multi-turn conversations. Example: "consensus_1703123456789_xyz789"',
+        'Thread continuation ID for multi-turn conversations. Example: "consensus_1703123456789_xyz789". Custom IDs are accepted — an unrecognized ID starts a new conversation under that ID. In async mode, IDs must contain only letters, numbers, hyphens, and underscores (max 128 chars).',
     },
     enable_cross_feedback: {
       type: 'boolean',