converse-mcp-server 2.3.1 → 2.4.1

package/src/tools/consensus.js

@@ -5,9 +5,19 @@
  * Calls all available providers simultaneously and aggregates responses.
  */
 
-import { createToolResponse, createToolError, formatFailureDetails } from './index.js';
-import { processUnifiedContext, createFileContext } from '../utils/contextProcessor.js';
-import { generateContinuationId, addMessageToHistory } from '../continuationStore.js';
+import {
+  createToolResponse,
+  createToolError,
+  formatFailureDetails,
+} from './index.js';
+import {
+  processUnifiedContext,
+  createFileContext,
+} from '../utils/contextProcessor.js';
+import {
+  generateContinuationId,
+  addMessageToHistory,
+} from '../continuationStore.js';
 import { debugLog, debugError } from '../utils/console.js';
 import { createLogger } from '../utils/logger.js';
 import { CONSENSUS_PROMPT } from '../systemPrompts.js';
@@ -25,15 +35,28 @@ const logger = createLogger('consensus');
  */
 export async function consensusTool(args, dependencies) {
   try {
-    const { config, providers, continuationStore, contextProcessor, jobRunner, providerStreamNormalizer } = dependencies;
+    const {
+      config,
+      providers,
+      continuationStore,
+      contextProcessor,
+      jobRunner,
+      providerStreamNormalizer,
+    } = dependencies;
 
     // Validate required arguments
     if (!args.prompt || typeof args.prompt !== 'string') {
       return createToolError('Prompt is required and must be a string');
     }
 
-    if (!args.models || !Array.isArray(args.models) || args.models.length === 0) {
-      return createToolError('Models array is required and must contain at least one model');
+    if (
+      !args.models ||
+      !Array.isArray(args.models) ||
+      args.models.length === 0
+    ) {
+      return createToolError(
+        'Models array is required and must contain at least one model',
+      );
     }
 
     // Extract and validate arguments
@@ -48,14 +71,16 @@ export async function consensusTool(args, dependencies) {
       temperature = 0.2,
       reasoning_effort = 'medium',
       use_websearch = false,
-      async = false
+      async = false,
     } = args;
 
     // Handle async execution mode
     if (async) {
       // Validate async dependencies are available
       if (!jobRunner || !providerStreamNormalizer) {
-        return createToolError('Async execution not available - missing async dependencies');
+        return createToolError(
+          'Async execution not available - missing async dependencies',
+        );
       }
 
       // Generate continuation ID for background execution result
@@ -69,9 +94,14 @@ export async function consensusTool(args, dependencies) {
       let title = null;
       try {
         title = await summarizationService.generateTitle(prompt);
-        debugLog(`Consensus: Generated title for initial response - "${title}"`);
+        debugLog(
+          `Consensus: Generated title for initial response - "${title}"`,
+        );
       } catch (error) {
-        debugError('Consensus: Failed to generate title for initial response', error);
+        debugError(
+          'Consensus: Failed to generate title for initial response',
+          error,
+        );
         title = prompt.substring(0, 50);
       }
 
@@ -85,8 +115,8 @@ export async function consensusTool(args, dependencies) {
               ...args,
               jobId: bgContinuationId, // Use continuation ID as job ID
               models_list: modelsList, // Add models list for status display
-              title // Pass the generated title
-            }
+              title, // Pass the generated title
+            },
           },
           async (context) => {
             // Execute consensus in background using stream normalizer
@@ -95,23 +125,25 @@ export async function consensusTool(args, dependencies) {
               {
                 ...dependencies,
                 continuationId: bgContinuationId,
-                title // Pass title to execution context
+                title, // Pass title to execution context
               },
-              context
+              context,
             );
-          }
+          },
         );
 
         // Format initial response like check_status output
-        const startTime = new Date().toLocaleString('en-GB', {
-          day: '2-digit',
-          month: '2-digit',
-          year: 'numeric',
-          hour: '2-digit',
-          minute: '2-digit',
-          second: '2-digit',
-          hour12: false
-        }).replace(',', '');
+        const startTime = new Date()
+          .toLocaleString('en-GB', {
+            day: '2-digit',
+            month: '2-digit',
+            year: 'numeric',
+            hour: '2-digit',
+            minute: '2-digit',
+            second: '2-digit',
+            hour12: false,
+          })
+          .replace(',', '');
 
         const statusLine = `⏳ SUBMITTED | CONSENSUS | ${bgContinuationId} | 1/1 | Started: ${startTime} | "${title || 'Processing...'}" | ${modelsList}`;
 
@@ -119,12 +151,11 @@ export async function consensusTool(args, dependencies) {
         return createToolResponse({
           content: `${statusLine}\ncontinuation_id: ${bgContinuationId}`,
           continuation: {
-            id: bgContinuationId,  // Use continuation_id as the primary ID
-            status: 'processing'
+            id: bgContinuationId, // Use continuation_id as the primary ID
+            status: 'processing',
           },
-          async_execution: true
+          async_execution: true,
         });
-
       } catch (error) {
         logger.error('Failed to submit async consensus job', { error });
         return createToolError(`Async execution failed: ${error.message}`);
@@ -137,7 +168,8 @@ export async function consensusTool(args, dependencies) {
     // Load existing conversation if continuation_id provided
     if (continuationId) {
       try {
-        const existingState = await dependencies.continuationStore.get(continuationId);
+        const existingState =
+          await dependencies.continuationStore.get(continuationId);
         if (existingState) {
           conversationHistory = existingState.messages || [];
         } else {
@@ -156,10 +188,13 @@ export async function consensusTool(args, dependencies) {
 
     // Validate file paths before processing
     if (files.length > 0 || images.length > 0) {
-      const validation = await validateAllPaths({
-        files,
-        images
-      }, { clientCwd: config.server?.client_cwd });
+      const validation = await validateAllPaths(
+        {
+          files,
+          images,
+        },
+        { clientCwd: config.server?.client_cwd },
+      );
       if (!validation.valid) {
         logger.error('File validation failed', { errors: validation.errors });
         return validation.errorResponse;
@@ -172,24 +207,29 @@ export async function consensusTool(args, dependencies) {
       try {
         const contextRequest = {
           files: Array.isArray(files) ? files : [],
-          images: Array.isArray(images) ? images : []
+          images: Array.isArray(images) ? images : [],
         };
 
-        const contextResult = await contextProcessor.processUnifiedContext(contextRequest, {
-          enforceSecurityCheck: false,  // Allow files from any location
-          skipSecurityCheck: true,       // Legacy flag for backward compatibility
-          clientCwd: config.server?.client_cwd  // Use auto-detected client working directory
-        });
+        const contextResult = await contextProcessor.processUnifiedContext(
+          contextRequest,
+          {
+            enforceSecurityCheck: false, // Allow files from any location
+            skipSecurityCheck: true, // Legacy flag for backward compatibility
+            clientCwd: config.server?.client_cwd, // Use auto-detected client working directory
+          },
+        );
 
         // Create context message from files and images
-        const allProcessedFiles = [...contextResult.files, ...contextResult.images];
+        const allProcessedFiles = [
+          ...contextResult.files,
+          ...contextResult.images,
+        ];
         if (allProcessedFiles.length > 0) {
           contextMessage = createFileContext(allProcessedFiles, {
             includeMetadata: true,
-            includeErrors: true
+            includeErrors: true,
           });
         }
-
       } catch (error) {
         logger.error('Error processing context', { error });
         // Continue without context if processing fails
@@ -202,7 +242,7 @@ export async function consensusTool(args, dependencies) {
     // Add system prompt
     messages.push({
       role: 'system',
-      content: CONSENSUS_PROMPT
+      content: CONSENSUS_PROMPT,
     });
 
     // Add conversation history
@@ -211,7 +251,7 @@ export async function consensusTool(args, dependencies) {
     // Add user prompt with context
     const userMessage = {
       role: 'user',
-      content: prompt // default to simple string content
+      content: prompt, // default to simple string content
     };
 
     // If we have context (files/images), create complex content array
@@ -219,7 +259,7 @@ export async function consensusTool(args, dependencies) {
       // Create complex content array
       userMessage.content = [
         ...contextMessage.content, // Include all file/image parts
-        { type: 'text', text: prompt } // Add the user prompt as text
+        { type: 'text', text: prompt }, // Add the user prompt as text
       ];
     }
 
@@ -234,7 +274,15 @@ export async function consensusTool(args, dependencies) {
     if (models.length === 1 && models[0].toLowerCase() === 'auto') {
       // Find first 3 available providers
       const availableProviders = [];
-      const providerOrder = ['openai', 'google', 'xai', 'anthropic', 'mistral', 'deepseek', 'openrouter'];
+      const providerOrder = [
+        'openai',
+        'google',
+        'xai',
+        'anthropic',
+        'mistral',
+        'deepseek',
+        'openrouter',
+      ];
 
       for (const providerName of providerOrder) {
         if (availableProviders.length >= 3) break;
@@ -245,19 +293,21 @@ export async function consensusTool(args, dependencies) {
       }
 
       if (availableProviders.length === 0) {
-        return createToolError('No providers available. Please configure at least one API key.');
+        return createToolError(
+          'No providers available. Please configure at least one API key.',
+        );
       }
 
       // Create model names for each available provider with their default model
-      modelsToProcess = availableProviders.map(providerName =>
-        getDefaultModelForProvider(providerName)
+      modelsToProcess = availableProviders.map((providerName) =>
+        getDefaultModelForProvider(providerName),
       );
 
       logger.debug('Auto-expanded to providers', {
         data: {
           providers: availableProviders,
-          models: modelsToProcess
-        }
+          models: modelsToProcess,
+        },
       });
     }
 
@@ -266,7 +316,7 @@ export async function consensusTool(args, dependencies) {
         failedModels.push({
           model: modelName || 'unknown',
           error: 'Invalid model specification',
-          status: 'failed'
+          status: 'failed',
         });
         continue;
       }
@@ -279,7 +329,7 @@ export async function consensusTool(args, dependencies) {
           model: modelName,
           provider: providerName,
           error: `Provider not found: ${providerName}`,
-          status: 'failed'
+          status: 'failed',
         });
         continue;
       }
@@ -289,7 +339,7 @@ export async function consensusTool(args, dependencies) {
           model: modelName,
           provider: providerName,
           error: `Provider ${providerName} not available (check API key)`,
-          status: 'failed'
+          status: 'failed',
         });
         continue;
       }
@@ -303,30 +353,35 @@ export async function consensusTool(args, dependencies) {
           reasoning_effort,
           use_websearch,
           config,
-          model: resolvedModelName // Use resolved model name for API call
-        }
+          model: resolvedModelName, // Use resolved model name for API call
+        },
       });
     }
 
     if (providerCalls.length === 0) {
       return createToolError(
-        `No valid providers available for the specified models. Failed models: ${failedModels.map(f => f.model).join(', ')}`
+        `No valid providers available for the specified models. Failed models: ${failedModels.map((f) => f.model).join(', ')}`,
       );
     }
 
     // Phase 1: Initial parallel provider calls
-    logger.debug('Calling providers in parallel', { data: { providerCount: providerCalls.length } });
+    logger.debug('Calling providers in parallel', {
+      data: { providerCount: providerCalls.length },
+    });
     const consensusStartTime = Date.now();
     const initialResults = await Promise.allSettled(
       providerCalls.map(async (call) => {
         try {
-          const response = await call.providerInstance.invoke(messages, call.options);
+          const response = await call.providerInstance.invoke(
+            messages,
+            call.options,
+          );
           return {
             model: call.model,
             provider: call.provider,
             status: 'success',
             response: response.content,
-            metadata: response.metadata || {}
+            metadata: response.metadata || {},
           };
         } catch (error) {
           return {
@@ -334,16 +389,16 @@ export async function consensusTool(args, dependencies) {
             provider: call.provider,
             status: 'failed',
             error: error.message,
-            metadata: {}
+            metadata: {},
           };
         }
-      })
+      }),
     );
 
     // Process initial results
     const initialPhase = {
       successful: [],
-      failed: []
+      failed: [],
     };
 
     initialResults.forEach((result, index) => {
@@ -359,7 +414,7 @@ export async function consensusTool(args, dependencies) {
           provider: providerCalls[index].provider,
           status: 'failed',
           error: result.reason.message || 'Unknown error',
-          metadata: {}
+          metadata: {},
         });
       }
     });
@@ -371,10 +426,13 @@ export async function consensusTool(args, dependencies) {
 
     // Phase 2: Cross-feedback (if enabled and we have multiple successful responses)
     if (enable_cross_feedback && initialPhase.successful.length > 1) {
-      logger.debug('Running cross-feedback phase', { data: { responseCount: initialPhase.successful.length } });
+      logger.debug('Running cross-feedback phase', {
+        data: { responseCount: initialPhase.successful.length },
+      });
 
       // Create cross-feedback prompt
-      const feedbackPrompt = cross_feedback_prompt ||
+      const feedbackPrompt =
+        cross_feedback_prompt ||
         `Based on the other AI responses below, please refine your answer to the original question. Consider different perspectives and provide your final response:
 
 Original Question: ${prompt}
@@ -391,29 +449,34 @@ Please provide your refined response:`;
       const refinementResults = await Promise.allSettled(
         initialPhase.successful.map(async (initialResult) => {
           try {
-            const call = providerCalls.find(c => c.model === initialResult.model);
+            const call = providerCalls.find(
+              (c) => c.model === initialResult.model,
+            );
 
             // Build model-specific feedback messages with the assistant's initial response
             const modelFeedbackMessages = [...messages];
             // Add the assistant's initial response
             modelFeedbackMessages.push({
               role: 'assistant',
-              content: initialResult.response
+              content: initialResult.response,
             });
             // Now add the feedback prompt
             modelFeedbackMessages.push({
               role: 'user',
-              content: feedbackPrompt
+              content: feedbackPrompt,
             });
 
-            const response = await call.providerInstance.invoke(modelFeedbackMessages, call.options);
+            const response = await call.providerInstance.invoke(
+              modelFeedbackMessages,
+              call.options,
+            );
 
             return {
               ...initialResult,
               refined_response: response.content,
               refined_metadata: response.metadata || {},
               initial_response: initialResult.response,
-              status: 'success'
+              status: 'success',
             };
           } catch (error) {
             return {
@@ -421,10 +484,10 @@ Please provide your refined response:`;
               refined_response: null,
               refined_error: error.message,
               initial_response: initialResult.response,
-              status: 'partial' // Had initial success but refinement failed
+              status: 'partial', // Had initial success but refinement failed
             };
           }
-        })
+        }),
       );
 
       // Process refinement results
@@ -439,7 +502,7 @@ Please provide your refined response:`;
             ...originalResult,
             refined_response: null,
             refined_error: 'Refinement phase failed unexpectedly',
-            status: 'partial'
+            status: 'partial',
           });
         }
       });
@@ -449,8 +512,11 @@ Please provide your refined response:`;
     try {
       const consensusMessage = {
         role: 'assistant',
-        content: `Consensus completed with ${initialPhase.successful.length} successful responses` +
-                (refinedPhase ? ` and ${refinedPhase.filter(r => r.status === 'success').length} refined responses` : '')
+        content:
+          `Consensus completed with ${initialPhase.successful.length} successful responses` +
+          (refinedPhase
+            ? ` and ${refinedPhase.filter((r) => r.status === 'success').length} refined responses`
+            : ''),
       };
 
       const conversationState = {
@@ -461,11 +527,14 @@ Please provide your refined response:`;
           modelsRequested: models.length,
           providersSuccessful: initialPhase.successful.length,
           providersFailed: initialPhase.failed.length,
-          crossFeedbackEnabled: enable_cross_feedback
-        }
+          crossFeedbackEnabled: enable_cross_feedback,
+        },
       };
 
-      await dependencies.continuationStore.set(continuationId, conversationState);
+      await dependencies.continuationStore.set(
+        continuationId,
+        conversationState,
+      );
     } catch (error) {
       logger.error('Error saving consensus conversation', { error });
       // Continue even if save fails
@@ -479,17 +548,19 @@ Please provide your refined response:`;
 
     if (enable_cross_feedback && refinedPhase) {
       // When cross-feedback is enabled, count only models that succeeded in both phases
-      finalSuccessCount = refinedPhase.filter(r => r.status === 'success').length;
+      finalSuccessCount = refinedPhase.filter(
+        (r) => r.status === 'success',
+      ).length;
 
       // Collect detailed failure information
-      refinedPhase.forEach(result => {
+      refinedPhase.forEach((result) => {
         if (result.status === 'partial') {
           failureDetails.push(`${result.model} (refinement failed)`);
         }
       });
 
       // Add models that failed in initial phase
-      initialPhase.failed.forEach(failure => {
+      initialPhase.failed.forEach((failure) => {
         failureDetails.push(`${failure.model} (initial failed)`);
       });
     } else {
@@ -497,21 +568,23 @@ Please provide your refined response:`;
       finalSuccessCount = initialPhase.successful.length;
 
       // Collect initial failure information
-      initialPhase.failed.forEach(failure => {
+      initialPhase.failed.forEach((failure) => {
         failureDetails.push(`${failure.model} (${failure.error})`);
       });
     }
 
     // Create models list string for display
-    const modelsList = providerCalls.map(call => call.model).join(', ');
-
+    const modelsList = providerCalls.map((call) => call.model).join(', ');
 
     // Create unified status line (similar to async status display)
-    const finalCount = refinedPhase ? refinedPhase.filter(r => r.status === 'success').length : initialPhase.successful.length;
+    const finalCount = refinedPhase
+      ? refinedPhase.filter((r) => r.status === 'success').length
+      : initialPhase.successful.length;
     const totalCount = providerCalls.length;
-    const statusLine = config.environment?.nodeEnv !== 'test'
-      ? `✅ COMPLETED | CONSENSUS | ${continuationId} | ${consensusExecutionTime.toFixed(1)}s elapsed | ${finalCount}/${totalCount} succeeded | ${modelsList}\n`
-      : '';
+    const statusLine =
+      config.environment?.nodeEnv !== 'test'
+        ? `✅ COMPLETED | CONSENSUS | ${continuationId} | ${consensusExecutionTime.toFixed(1)}s elapsed | ${finalCount}/${totalCount} succeeded | ${modelsList}\n`
+        : '';
 
     // Always include continuation_id line for clarity
     const continuationIdLine = `continuation_id: ${continuationId}\n\n`;
@@ -522,21 +595,23 @@ Please provide your refined response:`;
       models_consulted: models.length,
       successful_initial_responses: initialPhase.successful.length,
       failed_responses: initialPhase.failed.length,
-      refined_responses: refinedPhase ? refinedPhase.filter(r => r.status === 'success').length : 0,
+      refined_responses: refinedPhase
+        ? refinedPhase.filter((r) => r.status === 'success').length
+        : 0,
       phases: {
         initial: initialPhase.successful,
         ...(refinedPhase !== null && { refined: refinedPhase }),
-        failed: initialPhase.failed
+        failed: initialPhase.failed,
       },
       continuation: {
         id: continuationId,
-        messageCount: messages.length + 1
+        messageCount: messages.length + 1,
       },
       settings: {
         enable_cross_feedback,
         temperature,
-        models_requested: models
-      }
+        models_requested: models,
+      },
     };
 
     // Apply token limiting to the final response
@@ -559,10 +634,9 @@ Please provide your refined response:`;
       content: finalContent,
       continuation: {
         id: continuationId,
-        messageCount: messages.length + 1
-      }
+        messageCount: messages.length + 1,
+      },
     });
-
   } catch (error) {
     logger.error('Consensus tool error', { error });
     return createToolError('Consensus tool failed', error);
@@ -579,13 +653,13 @@ Please provide your refined response:`;
  */
 function getDefaultModelForProvider(providerName) {
   const defaults = {
-    'openai': 'gpt-5',
-    'xai': 'grok-4-0709',
-    'google': 'gemini-2.5-pro',
-    'anthropic': 'claude-sonnet-4-20250514',
-    'mistral': 'magistral-medium-2506',
-    'deepseek': 'deepseek-reasoner',
-    'openrouter': 'qwen/qwen3-coder'
+    openai: 'gpt-5',
+    xai: 'grok-4-0709',
+    google: 'gemini-2.5-pro',
+    anthropic: 'claude-sonnet-4-20250514',
+    mistral: 'magistral-medium-2506',
+    deepseek: 'deepseek-reasoner',
+    openrouter: 'qwen/qwen3-coder',
   };
 
   return defaults[providerName] || 'gpt-5';
@@ -611,8 +685,12 @@ function mapModelToProvider(model, providers) {
   }
 
   // Check OpenRouter-specific patterns first
-  if (modelLower === 'openrouter auto' || modelLower === 'auto router' ||
-      modelLower === 'auto-router' || modelLower === 'openrouter-auto') {
+  if (
+    modelLower === 'openrouter auto' ||
+    modelLower === 'auto router' ||
+    modelLower === 'auto-router' ||
+    modelLower === 'openrouter-auto'
+  ) {
     return 'openrouter';
   }
 
@@ -622,7 +700,11 @@ function mapModelToProvider(model, providers) {
     for (const [providerName, provider] of Object.entries(providers)) {
       if (provider && provider.getModelConfig) {
         const modelConfig = provider.getModelConfig(model);
-        if (modelConfig && !modelConfig.isDynamic && !modelConfig.needsApiUpdate) {
+        if (
+          modelConfig &&
+          !modelConfig.isDynamic &&
+          !modelConfig.needsApiUpdate
+        ) {
           // Model exists in this provider's static list
           return providerName;
         }
@@ -635,8 +717,12 @@ function mapModelToProvider(model, providers) {
   // For non-slash models, use keyword matching as before
 
   // OpenAI models
-  if (modelLower.includes('gpt') || modelLower.includes('o1') ||
-      modelLower.includes('o3') || modelLower.includes('o4')) {
+  if (
+    modelLower.includes('gpt') ||
+    modelLower.includes('o1') ||
+    modelLower.includes('o3') ||
+    modelLower.includes('o4')
+  ) {
     return 'openai';
   }
 
@@ -646,14 +732,22 @@ function mapModelToProvider(model, providers) {
   }
 
   // Google models
-  if (modelLower.includes('gemini') || modelLower.includes('flash') ||
-      modelLower.includes('pro') || modelLower === 'google') {
+  if (
+    modelLower.includes('gemini') ||
+    modelLower.includes('flash') ||
+    modelLower.includes('pro') ||
+    modelLower === 'google'
+  ) {
     return 'google';
   }
 
   // Anthropic models
-  if (modelLower.includes('claude') || modelLower.includes('opus') ||
-      modelLower.includes('sonnet') || modelLower.includes('haiku')) {
+  if (
+    modelLower.includes('claude') ||
+    modelLower.includes('opus') ||
+    modelLower.includes('sonnet') ||
+    modelLower.includes('haiku')
+  ) {
     return 'anthropic';
   }
 
@@ -663,14 +757,22 @@ function mapModelToProvider(model, providers) {
   }
 
   // DeepSeek models
-  if (modelLower.includes('deepseek') || modelLower === 'reasoner' ||
-      modelLower === 'r1' || modelLower === 'chat') {
+  if (
+    modelLower.includes('deepseek') ||
+    modelLower === 'reasoner' ||
+    modelLower === 'r1' ||
+    modelLower === 'chat'
+  ) {
     return 'deepseek';
   }
 
   // OpenRouter models (specific model patterns)
-  if (modelLower.includes('qwen') || modelLower.includes('kimi') ||
-      modelLower.includes('moonshot') || modelLower === 'k2') {
+  if (
+    modelLower.includes('qwen') ||
+    modelLower.includes('kimi') ||
+    modelLower.includes('moonshot') ||
+    modelLower === 'k2'
+  ) {
     return 'openrouter';
   }
 
@@ -693,7 +795,7 @@ async function executeConsensusWithStreaming(args, dependencies, context) {
     contextProcessor,
     providerStreamNormalizer,
     continuationId,
-    title: passedTitle // Title passed from initial submission
+    title: passedTitle, // Title passed from initial submission
   } = dependencies;
 
   const {
@@ -705,7 +807,7 @@ async function executeConsensusWithStreaming(args, dependencies, context) {
     cross_feedback_prompt,
     temperature = 0.2,
     reasoning_effort = 'medium',
-    use_websearch = false
+    use_websearch = false,
   } = args;
 
   let conversationHistory = [];
@@ -725,13 +827,18 @@ async function executeConsensusWithStreaming(args, dependencies, context) {
 
   // Validate file paths before processing
   if (files.length > 0 || images.length > 0) {
-    const validation = await validateAllPaths({
-      files,
-      images
-    }, { clientCwd: config.server?.client_cwd });
+    const validation = await validateAllPaths(
+      {
+        files,
+        images,
+      },
+      { clientCwd: config.server?.client_cwd },
+    );
     if (!validation.valid) {
       logger.error('File validation failed', { errors: validation.errors });
-      throw new Error(`File validation failed: ${validation.errors.join(', ')}`);
+      throw new Error(
+        `File validation failed: ${validation.errors.join(', ')}`,
+      );
     }
   }
 
@@ -741,24 +848,29 @@ async function executeConsensusWithStreaming(args, dependencies, context) {
     try {
       const contextRequest = {
         files: Array.isArray(files) ? files : [],
-        images: Array.isArray(images) ? images : []
+        images: Array.isArray(images) ? images : [],
       };
 
-      const contextResult = await contextProcessor.processUnifiedContext(contextRequest, {
-        enforceSecurityCheck: false,
-        skipSecurityCheck: true,
-        clientCwd: config.server?.client_cwd
-      });
+      const contextResult = await contextProcessor.processUnifiedContext(
+        contextRequest,
+        {
+          enforceSecurityCheck: false,
+          skipSecurityCheck: true,
+          clientCwd: config.server?.client_cwd,
+        },
+      );
 
       // Create context message from files and images
-      const allProcessedFiles = [...contextResult.files, ...contextResult.images];
+      const allProcessedFiles = [
+        ...contextResult.files,
+        ...contextResult.images,
+      ];
       if (allProcessedFiles.length > 0) {
         contextMessage = createFileContext(allProcessedFiles, {
           includeMetadata: true,
-          includeErrors: true
+          includeErrors: true,
         });
       }
-
     } catch (error) {
       logger.error('Error processing context', { error });
       // Continue without context if processing fails
@@ -771,7 +883,7 @@ async function executeConsensusWithStreaming(args, dependencies, context) {
   // Add system prompt
   messages.push({
     role: 'system',
-    content: CONSENSUS_PROMPT
+    content: CONSENSUS_PROMPT,
   });
 
   // Add conversation history
@@ -780,14 +892,14 @@ async function executeConsensusWithStreaming(args, dependencies, context) {
   // Add user prompt with context
   const userMessage = {
     role: 'user',
-    content: prompt
+    content: prompt,
   };
 
   // If we have context (files/images), create complex content array
   if (contextMessage && contextMessage.content) {
     userMessage.content = [
       ...contextMessage.content,
-      { type: 'text', text: prompt }
+      { type: 'text', text: prompt },
     ];
   }
 
@@ -801,7 +913,15 @@ async function executeConsensusWithStreaming(args, dependencies, context) {
   let modelsToProcess = models;
   if (models.length === 1 && models[0].toLowerCase() === 'auto') {
     const availableProviders = [];
-    const providerOrder = ['openai', 'google', 'xai', 'anthropic', 'mistral', 'deepseek', 'openrouter'];
+    const providerOrder = [
+      'openai',
+      'google',
+      'xai',
+      'anthropic',
+      'mistral',
+      'deepseek',
+      'openrouter',
+    ];
 
     for (const providerName of providerOrder) {
       if (availableProviders.length >= 3) break;
@@ -812,18 +932,20 @@ async function executeConsensusWithStreaming(args, dependencies, context) {
     }
 
     if (availableProviders.length === 0) {
-      throw new Error('No providers available. Please configure at least one API key.');
+      throw new Error(
+        'No providers available. Please configure at least one API key.',
+      );
     }
 
-    modelsToProcess = availableProviders.map(providerName =>
-      getDefaultModelForProvider(providerName)
+    modelsToProcess = availableProviders.map((providerName) =>
+      getDefaultModelForProvider(providerName),
     );
 
     logger.debug('Auto-expanded to providers', {
       data: {
         providers: availableProviders,
-        models: modelsToProcess
-      }
+        models: modelsToProcess,
+      },
     });
   }
 
@@ -833,7 +955,7 @@ async function executeConsensusWithStreaming(args, dependencies, context) {
       failedModels.push({
         model: modelName || 'unknown',
         error: 'Invalid model specification',
-        status: 'failed'
+        status: 'failed',
       });
       continue;
     }
@@ -847,7 +969,7 @@ async function executeConsensusWithStreaming(args, dependencies, context) {
         model: modelName,
         provider: providerName,
         error: `Provider not found: ${providerName}`,
-        status: 'failed'
+        status: 'failed',
       });
       continue;
     }
@@ -857,7 +979,7 @@ async function executeConsensusWithStreaming(args, dependencies, context) {
         model: modelName,
         provider: providerName,
         error: `Provider ${providerName} not available (check API key)`,
-        status: 'failed'
+        status: 'failed',
       });
       continue;
     }
@@ -872,19 +994,19 @@ async function executeConsensusWithStreaming(args, dependencies, context) {
         use_websearch,
         signal: context?.signal, // Pass AbortSignal for cancellation support
         config,
-        model: resolvedModelName
-      }
+        model: resolvedModelName,
+      },
     });
   }
 
   if (providerCalls.length === 0) {
     throw new Error(
-      `No valid providers available for the specified models. Failed models: ${failedModels.map(f => f.model).join(', ')}`
+      `No valid providers available for the specified models. Failed models: ${failedModels.map((f) => f.model).join(', ')}`,
     );
   }
 
   // Create models list string for display
-  const modelsList = providerCalls.map(call => call.model).join(', ');
+  const modelsList = providerCalls.map((call) => call.model).join(', ');
 
   // Initialize SummarizationService
   const summarizationService = new SummarizationService(providers, config);
@@ -914,27 +1036,29 @@ async function executeConsensusWithStreaming(args, dependencies, context) {
       total_providers: providerCalls.length,
       completed_providers: 0,
       failed_providers: failedModels.length,
-      provider_status: {}
-    }
+      provider_status: {},
+    },
   });
 
   const consensusStartTime = Date.now();
 
   // Phase 1: Initial parallel provider calls with streaming
-  logger.debug('Calling providers in parallel with streaming', { data: { providerCount: providerCalls.length } });
+  logger.debug('Calling providers in parallel with streaming', {
+    data: { providerCount: providerCalls.length },
+  });
 
   const initialResults = await executeConsensusPhaseWithStreaming(
     providerCalls,
     messages,
     'initial',
     context,
-    providerStreamNormalizer
+    providerStreamNormalizer,
   );
 
   // Process initial results
   const initialPhase = {
     successful: [],
-    failed: []
+    failed: [],
   };
 
   initialResults.forEach((result) => {
@@ -952,7 +1076,9 @@ async function executeConsensusWithStreaming(args, dependencies, context) {
 
   // Phase 2: Cross-feedback (if enabled and we have multiple successful responses)
   if (enable_cross_feedback && initialPhase.successful.length > 1) {
-    logger.debug('Running cross-feedback phase with streaming', { data: { responseCount: initialPhase.successful.length } });
+    logger.debug('Running cross-feedback phase with streaming', {
+      data: { responseCount: initialPhase.successful.length },
+    });
 
     // Update job status for phase 2
     await context.updateJob({
@@ -960,12 +1086,13 @@ async function executeConsensusWithStreaming(args, dependencies, context) {
         phase: 'cross_feedback',
         total_providers: initialPhase.successful.length,
         completed_providers: 0,
-        provider_status: {}
-      }
+        provider_status: {},
+      },
     });
 
     // Create cross-feedback prompt
-    const feedbackPrompt = cross_feedback_prompt ||
+    const feedbackPrompt =
+      cross_feedback_prompt ||
       `Based on the other AI responses below, please refine your answer to the original question. Consider different perspectives and provide your final response:
 
 Original Question: ${prompt}
@@ -977,23 +1104,23 @@ Please provide your refined response:`;
 
     // Build model-specific feedback calls
     const feedbackCalls = initialPhase.successful.map((initialResult) => {
-      const call = providerCalls.find(c => c.model === initialResult.model);
+      const call = providerCalls.find((c) => c.model === initialResult.model);
 
       // Build model-specific feedback messages with the assistant's initial response
       const modelFeedbackMessages = [...messages];
       modelFeedbackMessages.push({
         role: 'assistant',
-        content: initialResult.response
+        content: initialResult.response,
       });
       modelFeedbackMessages.push({
         role: 'user',
-        content: feedbackPrompt
+        content: feedbackPrompt,
       });
 
       return {
         ...call,
         messages: modelFeedbackMessages,
-        initialResult
+        initialResult,
       };
     });
 
@@ -1003,7 +1130,7 @@ Please provide your refined response:`;
       null, // messages already embedded in feedbackCalls
       'refinement',
       context,
-      providerStreamNormalizer
+      providerStreamNormalizer,
     );
 
     // Process refinement results
@@ -1015,7 +1142,7 @@ Please provide your refined response:`;
         refined_metadata: result.status === 'success' ? result.metadata : {},
         refined_error: result.status === 'failed' ? result.error : null,
         initial_response: initialResult.response,
-        status: result.status === 'success' ? 'success' : 'partial'
+        status: result.status === 'success' ? 'success' : 'partial',
       };
     });
   }
@@ -1024,8 +1151,11 @@ Please provide your refined response:`;
   try {
     const consensusMessage = {
       role: 'assistant',
-      content: `Consensus completed with ${initialPhase.successful.length} successful responses` +
-              (refinedPhase ? ` and ${refinedPhase.filter(r => r.status === 'success').length} refined responses` : '')
+      content:
+        `Consensus completed with ${initialPhase.successful.length} successful responses` +
+        (refinedPhase
+          ? ` and ${refinedPhase.filter((r) => r.status === 'success').length} refined responses`
+          : ''),
     };
 
     const conversationState = {
@@ -1036,8 +1166,8 @@ Please provide your refined response:`;
         modelsRequested: models.length,
         providersSuccessful: initialPhase.successful.length,
         providersFailed: initialPhase.failed.length,
-        crossFeedbackEnabled: enable_cross_feedback
-      }
+        crossFeedbackEnabled: enable_cross_feedback,
+      },
     };
 
     await continuationStore.set(continuationId, conversationState);
@@ -1054,7 +1184,7 @@ Please provide your refined response:`;
   // Collect all successful responses for summary generation
   if (refinedPhase) {
     // Use refined responses when available
-    refinedPhase.forEach(result => {
+    refinedPhase.forEach((result) => {
       if (result.status === 'success' && result.refined_response) {
         combinedResponses.push(`${result.model}:\n${result.refined_response}`);
       } else if (result.initial_response) {
@@ -1064,7 +1194,7 @@ Please provide your refined response:`;
     });
   } else {
     // Use initial responses
-    initialPhase.successful.forEach(result => {
+    initialPhase.successful.forEach((result) => {
       if (result.response) {
         combinedResponses.push(`${result.model}:\n${result.response}`);
       }
@@ -1076,12 +1206,13 @@ Please provide your refined response:`;
     const combinedContent = combinedResponses.join('\n\n---\n\n');
     if (combinedContent.length > 100) {
       try {
-        finalSummary = await summarizationService.generateFinalSummary(combinedContent);
+        finalSummary =
+          await summarizationService.generateFinalSummary(combinedContent);
         debugLog(`Consensus: Generated final summary - "${finalSummary}"`);
 
         // Update job with final summary
         await context.updateJob({
-          final_summary: finalSummary
+          final_summary: finalSummary,
         });
       } catch (error) {
         debugError('Consensus: Error generating final summary', error);
@@ -1095,20 +1226,22 @@ Please provide your refined response:`;
   const failureDetails = [];
 
   if (enable_cross_feedback && refinedPhase) {
-    finalSuccessCount = refinedPhase.filter(r => r.status === 'success').length;
+    finalSuccessCount = refinedPhase.filter(
+      (r) => r.status === 'success',
+    ).length;
 
-    refinedPhase.forEach(result => {
+    refinedPhase.forEach((result) => {
       if (result.status === 'partial') {
         failureDetails.push(`${result.model} (refinement failed)`);
       }
     });
 
-    initialPhase.failed.forEach(failure => {
+    initialPhase.failed.forEach((failure) => {
       failureDetails.push(`${failure.model} (initial failed)`);
     });
   } else {
     finalSuccessCount = initialPhase.successful.length;
-    initialPhase.failed.forEach(failure => {
+    initialPhase.failed.forEach((failure) => {
       failureDetails.push(`${failure.model} (${failure.error})`);
     });
   }
@@ -1119,20 +1252,22 @@ Please provide your refined response:`;
     models_consulted: models.length,
     successful_initial_responses: initialPhase.successful.length,
     failed_responses: initialPhase.failed.length,
-    refined_responses: refinedPhase ? refinedPhase.filter(r => r.status === 'success').length : 0,
+    refined_responses: refinedPhase
+      ? refinedPhase.filter((r) => r.status === 'success').length
+      : 0,
     phases: {
       initial: initialPhase.successful,
       ...(refinedPhase !== null && { refined: refinedPhase }),
-      failed: initialPhase.failed
+      failed: initialPhase.failed,
     },
     continuation: {
       id: continuationId,
-      messageCount: messages.length + 1
+      messageCount: messages.length + 1,
     },
     settings: {
       enable_cross_feedback,
       temperature,
-      models_requested: models
+      models_requested: models,
     },
     metadata: {
       execution_time: consensusExecutionTime,
@@ -1141,8 +1276,8 @@ Please provide your refined response:`;
       total_models: models.length,
       failure_details: failureDetails,
       title,
-      final_summary: finalSummary
-    }
+      final_summary: finalSummary,
+    },
   };
 }
 
@@ -1155,7 +1290,13 @@ Please provide your refined response:`;
  * @param {object} streamNormalizer - Stream normalizer instance
  * @returns {Promise<Array>} Results from all providers
  */
-async function executeConsensusPhaseWithStreaming(providerCalls, messages, phase, context, streamNormalizer) {
+async function executeConsensusPhaseWithStreaming(
+  providerCalls,
+  messages,
+  phase,
+  context,
+  streamNormalizer,
+) {
   let completedCount = 0;
   const totalCount = providerCalls.length;
   const providerContents = {}; // Store accumulated content per provider
@@ -1172,21 +1313,31 @@ async function executeConsensusPhaseWithStreaming(providerCalls, messages, phase
         await context.updateJob({
           progress: {
             [`provider_${index}_status`]: 'prompting',
-            [`provider_${index}_model`]: call.model
-          }
+            [`provider_${index}_model`]: call.model,
+          },
         });
 
         const messagesToSend = call.messages || messages;
         let response;
 
         // Check if provider supports streaming
-        if (call.providerInstance.stream && typeof call.providerInstance.stream === 'function') {
+        if (
+          call.providerInstance.stream &&
+          typeof call.providerInstance.stream === 'function'
+        ) {
           // Use streaming with normalization
-          const stream = call.providerInstance.stream(messagesToSend, call.options);
-          const normalizedStream = streamNormalizer.normalize(call.provider, stream, {
-            model: call.options.model,
-            requestId: `${context.jobId}-${phase}-${index}`
-          });
+          const stream = call.providerInstance.stream(
+            messagesToSend,
+            call.options,
+          );
+          const normalizedStream = streamNormalizer.normalize(
+            call.provider,
+            stream,
+            {
+              model: call.options.model,
+              requestId: `${context.jobId}-${phase}-${index}`,
+            },
+          );
 
           // Process normalized stream
           let accumulatedContent = '';
@@ -1194,7 +1345,7 @@ async function executeConsensusPhaseWithStreaming(providerCalls, messages, phase
           let finalMetadata = {};
 
           await context.updateJob({
-            progress: { [`provider_${index}_status`]: 'streaming' }
+            progress: { [`provider_${index}_status`]: 'streaming' },
           });
 
           for await (const event of normalizedStream) {
@@ -1210,15 +1361,16 @@ async function executeConsensusPhaseWithStreaming(providerCalls, messages, phase
 
               // Combine all provider contents for unified accumulated_content
               const combinedContent = Object.values(providerContents)
-                .filter(content => content && content.length > 0)
+                .filter((content) => content && content.length > 0)
                 .join('\n\n---\n\n');
 
               // Update with both provider preview and combined accumulated content
               await context.updateJob({
-                [`provider_${index}_preview`]: accumulatedContent.length > 150
-                  ? accumulatedContent.substring(0, 150) + '...'
-                  : accumulatedContent,
-                accumulated_content: combinedContent // Full combined content from all providers
+                [`provider_${index}_preview`]:
+                    accumulatedContent.length > 150
+                      ? accumulatedContent.substring(0, 150) + '...'
+                      : accumulatedContent,
+                accumulated_content: combinedContent, // Full combined content from all providers
               });
               break;
             case 'usage':
@@ -1239,16 +1391,18 @@ async function executeConsensusPhaseWithStreaming(providerCalls, messages, phase
             metadata: {
               ...finalMetadata,
               usage: finalUsage,
-              streaming: true
-            }
+              streaming: true,
+            },
           };
 
           // Store final provider content
           providerContents[index] = accumulatedContent;
-
         } else {
           // Fall back to regular invoke
-          response = await call.providerInstance.invoke(messagesToSend, call.options);
+          response = await call.providerInstance.invoke(
+            messagesToSend,
+            call.options,
+          );
 
           // Store provider content for non-streaming response
           if (response && response.content) {
@@ -1256,29 +1410,30 @@ async function executeConsensusPhaseWithStreaming(providerCalls, messages, phase
 
             // Update accumulated content for non-streaming provider
             const combinedContent = Object.values(providerContents)
-              .filter(content => content && content.length > 0)
+              .filter((content) => content && content.length > 0)
               .join('\n\n---\n\n');
 
             await context.updateJob({
-              accumulated_content: combinedContent
+              accumulated_content: combinedContent,
             });
           }
         }
 
         // Update provider status to 'finished'
         completedCount++;
-        const progressText = phase === 'initial'
-          ? `${completedCount}/${totalCount} initial`
-          : phase === 'refinement'
-            ? `${completedCount}/${totalCount} refined`
-            : `${completedCount}/${totalCount} responded`;
+        const progressText =
+          phase === 'initial'
+            ? `${completedCount}/${totalCount} initial`
+            : phase === 'refinement'
+              ? `${completedCount}/${totalCount} refined`
+              : `${completedCount}/${totalCount} responded`;
 
         await context.updateJob({
           consensus_progress: progressText,
           progress: {
             [`provider_${index}_status`]: 'finished',
-            completed_providers: completedCount
-          }
+            completed_providers: completedCount,
+          },
         });
 
         return {
@@ -1286,16 +1441,15 @@ async function executeConsensusPhaseWithStreaming(providerCalls, messages, phase
           provider: call.provider,
           status: 'success',
           response: response.content,
-          metadata: response.metadata || {}
+          metadata: response.metadata || {},
         };
-
       } catch (error) {
         // Update provider status to 'failed'
         await context.updateJob({
           progress: {
             [`provider_${index}_status`]: 'failed',
-            [`provider_${index}_error`]: error.message
-          }
+            [`provider_${index}_error`]: error.message,
+          },
         });
 
         return {
@@ -1303,20 +1457,20 @@ async function executeConsensusPhaseWithStreaming(providerCalls, messages, phase
           provider: call.provider,
           status: 'failed',
           error: error.message,
-          metadata: {}
+          metadata: {},
         };
       }
-    })
+    }),
   );
 
   // After all providers complete, update with final combined content
   const finalCombinedContent = Object.values(providerContents)
-    .filter(content => content && content.length > 0)
+    .filter((content) => content && content.length > 0)
     .join('\n\n---\n\n');
 
   if (finalCombinedContent) {
     await context.updateJob({
-      accumulated_content: finalCombinedContent
+      accumulated_content: finalCombinedContent,
     });
   }
 
@@ -1329,14 +1483,15 @@ async function executeConsensusPhaseWithStreaming(providerCalls, messages, phase
         provider: providerCalls[index].provider,
         status: 'failed',
         error: result.reason.message || 'Unknown error',
-        metadata: {}
+        metadata: {},
       };
     }
   });
 }
 
 // Tool metadata
-consensusTool.description = 'PARALLEL CONSENSUS WITH CROSS-MODEL FEEDBACK - Query multiple models simultaneously, then optionally refine responses based on cross-feedback. For complex decisions, architectural choices, technical evaluations. Use models: ["auto"] for automatic selection.';
+consensusTool.description =
+  'PARALLEL CONSENSUS WITH CROSS-MODEL FEEDBACK - Query multiple models simultaneously, then optionally refine responses based on cross-feedback. For complex decisions, architectural choices, technical evaluations. Use models: ["auto"] for automatic selection.';
 consensusTool.inputSchema = {
   type: 'object',
   properties: {
@@ -1344,34 +1499,41 @@ consensusTool.inputSchema = {
       type: 'array',
       items: { type: 'string' },
       minItems: 1,
-      description: 'List of models to consult. Example: ["gpt-5", "gemini-2.5-pro", "grok-4-0709"]',
+      description:
+        'List of models to consult. Example: ["gpt-5", "gemini-2.5-pro", "grok-4-0709"]',
     },
     files: {
       type: 'array',
       items: { type: 'string' },
-      description: 'File paths for additional context (absolute or relative paths). Example: ["C:\\Users\\username\\project\\architecture.md", "./requirements.txt"]',
+      description:
+        'File paths for additional context (absolute or relative paths). Example: ["C:\\Users\\username\\project\\architecture.md", "./requirements.txt"]',
     },
     images: {
       type: 'array',
       items: { type: 'string' },
-      description: 'Image paths for visual context (absolute or relative paths, or base64). Example: ["C:\\Users\\username\\current_architecture.png", "./user_flow.jpg"]',
+      description:
+        'Image paths for visual context (absolute or relative paths, or base64). Example: ["C:\\Users\\username\\current_architecture.png", "./user_flow.jpg"]',
     },
     continuation_id: {
       type: 'string',
-      description: 'Thread continuation ID for multi-turn conversations. Example: "consensus_1703123456789_xyz789"',
+      description:
+        'Thread continuation ID for multi-turn conversations. Example: "consensus_1703123456789_xyz789"',
     },
     enable_cross_feedback: {
       type: 'boolean',
-      description: 'Enable refinement phase where models see others\' responses and can improve their answers. Example: true (recommended), false (faster single-phase only). Default: true',
+      description:
+        'Enable refinement phase where models see others\' responses and can improve their answers. Example: true (recommended), false (faster single-phase only). Default: true',
       default: true,
     },
     cross_feedback_prompt: {
       type: 'string',
-      description: 'Custom prompt for refinement phase. Example: "Focus on scalability trade-offs in your refinement" or leave empty for default cross-feedback prompt',
+      description:
+        'Custom prompt for refinement phase. Example: "Focus on scalability trade-offs in your refinement" or leave empty for default cross-feedback prompt',
     },
     temperature: {
       type: 'number',
-      description: 'Response randomness (0.0-1.0). Examples: 0.1 (very focused), 0.2 (analytical - default), 0.5 (balanced). Default: 0.2',
+      description:
+        'Response randomness (0.0-1.0). Examples: 0.1 (very focused), 0.2 (analytical - default), 0.5 (balanced). Default: 0.2',
       minimum: 0.0,
       maximum: 1.0,
       default: 0.2,
@@ -1379,22 +1541,26 @@ consensusTool.inputSchema = {
     reasoning_effort: {
       type: 'string',
       enum: ['none', 'minimal', 'low', 'medium', 'high', 'max'],
-      description: 'Reasoning depth for thinking models. Examples: "none" (no reasoning, fastest - GPT-5.1+ only), "low" (light analysis), "medium" (balanced), "high" (complex analysis). Default: "medium"',
-      default: 'medium'
+      description:
+        'Reasoning depth for thinking models. Examples: "none" (no reasoning, fastest - GPT-5.1+ only), "low" (light analysis), "medium" (balanced), "high" (complex analysis). Default: "medium"',
+      default: 'medium',
     },
     use_websearch: {
       type: 'boolean',
-      description: 'Enable web search for current information. Only works with models that support web search (OpenAI, XAI, Google). Example: true for recent developments or up to date documentation. Default: false',
-      default: false
+      description:
+        'Enable web search for current information. Only works with models that support web search (OpenAI, XAI, Google). Example: true for recent developments or up to date documentation. Default: false',
+      default: false,
     },
     async: {
       type: 'boolean',
-      description: 'Execute consensus in background with detailed progress tracking. When true, returns continuation_id immediately and processes request asynchronously with per-provider status updates. Default: false',
-      default: false
+      description:
+        'Execute consensus in background with detailed progress tracking. When true, returns continuation_id immediately and processes request asynchronously with per-provider status updates. Default: false',
+      default: false,
     },
     prompt: {
       type: 'string',
-      description: 'The problem or proposal to gather consensus on. Include context and specific questions. Example: "Should we use microservices or monolith architecture for our e-commerce platform with 100k users?"',
+      description:
+        'The problem or proposal to gather consensus on. Include context and specific questions. Example: "Should we use microservices or monolith architecture for our e-commerce platform with 100k users?"',
     },
   },
   required: ['prompt', 'models'],