converse-mcp-server 2.19.2 → 2.20.0

package/bin/converse.js

@@ -1,4 +1,4 @@
-#!/usr/bin/env node
+#!/usr/bin/env node
 
 /**
  * Converse MCP Server - CLI Entry Point
@@ -10,6 +10,19 @@ import { fileURLToPath, pathToFileURL } from 'url';
 import { dirname, join } from 'path';
 import { createRequire } from 'module';
 
+// Capture the caller's working directory before we chdir to the package root.
+// This is critical for resolving relative file paths passed by MCP clients.
+// Parse --cwd <path> from CLI args as an explicit override.
+const cwdArgIndex = process.argv.indexOf('--cwd');
+const callerCwd = (cwdArgIndex !== -1 && process.argv[cwdArgIndex + 1])
+  ? process.argv[cwdArgIndex + 1]
+  : process.cwd();
+
+// Expose as env var so config.js can pick it up (only if not already set)
+if (!process.env.CLIENT_CWD) {
+  process.env.CLIENT_CWD = callerCwd;
+}
+
 // Get the directory of this script
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = dirname(__filename);
@@ -24,7 +37,7 @@ process.chdir(projectRoot);
 try {
   const indexPath = join(projectRoot, 'src/index.js');
   const { main } = await import(pathToFileURL(indexPath).href);
-  
+
   // The main function will handle all logging appropriately based on transport type
   await main();
 } catch (error) {

package/package.json

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

package/src/config.js

@@ -10,7 +10,7 @@ import dotenv from 'dotenv';
 import { createLogger, configureLogger } from './utils/logger.js';
 import { ConfigurationError } from './utils/errorHandler.js';
 import { fileURLToPath } from 'url';
-import { dirname, join } from 'path';
+import { dirname, join, resolve } from 'path';
 import { readFileSync } from 'fs';
 
 // Load environment variables from appropriate .env file
@@ -407,6 +407,23 @@ function validateApiKeyFormat(provider, apiKey) {
   }
 }
 
+/**
+ * Normalize Git Bash paths to Windows paths on Windows.
+ * Converts /c/Users/... to C:\Users\... so path.resolve() works correctly.
+ * On non-Windows platforms, returns the path unchanged.
+ */
+function normalizeGitBashPath(inputPath) {
+  if (
+    process.platform === 'win32' &&
+    /^\/[a-zA-Z]\//.test(inputPath)
+  ) {
+    return resolve(
+      inputPath[1].toUpperCase() + ':' + inputPath.slice(2).replace(/\//g, '\\'),
+    );
+  }
+  return inputPath;
+}
+
 /**
  * Loads and validates complete configuration from environment variables
  * @returns {Promise<object>} Validated configuration object
@@ -437,19 +454,17 @@ export async function loadConfig() {
     for (const [key, schema] of Object.entries(CONFIG_SCHEMA.server)) {
       try {
         // Special handling for CLIENT_CWD - auto-detect if not explicitly set
-        if (key === 'CLIENT_CWD' && !process.env[key]) {
-          // Try to detect the client's working directory from various sources
-          // When run via npx, INIT_CWD contains the directory where npx was invoked
-          // PWD is another common variable set to the working directory
-          // npm_config_local_prefix is set when run via npm/npx
-          const detectedCwd =
+        if (key === 'CLIENT_CWD') {
+          const explicitCwd = process.env[key];
+          const detectedCwd = explicitCwd ||
             process.env.INIT_CWD ||
             process.env.PWD ||
             process.env.npm_config_local_prefix ||
             process.cwd();
-          config.server.client_cwd = detectedCwd;
+          // Normalize Git Bash paths (/c/Users/... -> C:\Users\...)
+          config.server.client_cwd = normalizeGitBashPath(detectedCwd);
           configLogger.debug(
-            `Auto-detected client working directory: ${detectedCwd}`,
+            `Client working directory: ${config.server.client_cwd}${explicitCwd ? ' (from CLIENT_CWD)' : ' (auto-detected)'}`,
           );
         } else {
           config.server[key.toLowerCase()] = validateEnvVar(

package/src/providers/copilot.js

@@ -54,6 +54,7 @@ const SUPPORTED_MODELS = {
     supportsImages: false,
     supportsTemperature: false,
     supportsWebSearch: false,
+    supportsReasoningEffort: true,
     timeout: 120000,
     description: 'OpenAI GPT-5 Mini via Copilot subscription',
     aliases: [],
@@ -67,6 +68,7 @@ const SUPPORTED_MODELS = {
     supportsImages: false,
     supportsTemperature: false,
     supportsWebSearch: false,
+    supportsReasoningEffort: true,
     timeout: 120000,
     description: 'OpenAI GPT-5.1 via Copilot subscription',
     aliases: [],
@@ -80,6 +82,7 @@ const SUPPORTED_MODELS = {
     supportsImages: false,
     supportsTemperature: false,
     supportsWebSearch: false,
+    supportsReasoningEffort: true,
     timeout: 300000,
     description: 'OpenAI GPT-5.1 Codex via Copilot subscription',
     aliases: [],
@@ -93,6 +96,7 @@ const SUPPORTED_MODELS = {
     supportsImages: false,
     supportsTemperature: false,
     supportsWebSearch: false,
+    supportsReasoningEffort: true,
     timeout: 300000,
     description: 'OpenAI GPT-5.1 Codex Mini via Copilot subscription',
     aliases: [],
@@ -106,6 +110,7 @@ const SUPPORTED_MODELS = {
     supportsImages: false,
     supportsTemperature: false,
     supportsWebSearch: false,
+    supportsReasoningEffort: true,
     timeout: 600000,
     description: 'OpenAI GPT-5.1 Codex Max via Copilot subscription',
     aliases: [],
@@ -119,6 +124,7 @@ const SUPPORTED_MODELS = {
     supportsImages: false,
     supportsTemperature: false,
     supportsWebSearch: false,
+    supportsReasoningEffort: true,
     timeout: 120000,
     description: 'OpenAI GPT-5.2 via Copilot subscription',
     aliases: ['gpt-5'],
@@ -132,6 +138,7 @@ const SUPPORTED_MODELS = {
     supportsImages: false,
     supportsTemperature: false,
     supportsWebSearch: false,
+    supportsReasoningEffort: true,
     timeout: 300000,
     description: 'OpenAI GPT-5.2 Codex via Copilot subscription',
     aliases: [],
@@ -145,6 +152,7 @@ const SUPPORTED_MODELS = {
     supportsImages: false,
     supportsTemperature: false,
     supportsWebSearch: false,
+    supportsReasoningEffort: true,
     timeout: 300000,
     description: 'OpenAI GPT-5.3 Codex via Copilot subscription',
     aliases: ['codex'],
@@ -451,6 +459,37 @@ function resolveModelAlias(name) {
   return null;
 }
 
+/**
+ * Look up model config from SUPPORTED_MODELS by name or alias.
+ * Strips copilot: prefix and falls back to the base copilot config.
+ */
+function findModelConfig(modelName) {
+  if (typeof modelName !== 'string') return null;
+
+  let name = modelName;
+  if (name.toLowerCase().startsWith('copilot:')) {
+    name = name.slice('copilot:'.length).trim();
+  }
+  if (!name) return SUPPORTED_MODELS.copilot;
+
+  const nameLower = name.toLowerCase();
+
+  if (SUPPORTED_MODELS[nameLower]) {
+    return SUPPORTED_MODELS[nameLower];
+  }
+
+  for (const config of Object.values(SUPPORTED_MODELS)) {
+    if (
+      config.aliases &&
+      config.aliases.some((alias) => alias.toLowerCase() === nameLower)
+    ) {
+      return config;
+    }
+  }
+
+  return null;
+}
+
 /**
  * Resolve model to pass to SDK session
  * Precedence: explicit model param > config COPILOT_MODEL > omit (SDK default)
@@ -524,6 +563,25 @@ function mapReasoningEffort(effort) {
   return mapping[effort] || undefined;
 }
 
+/**
+ * Check if a model supports reasoning effort via the Copilot SDK's models.list API.
+ * Results are cached by the SDK internally.
+ * Returns true if supported, false if not, or undefined if the check fails.
+ */
+async function checkReasoningSupport(client, modelId) {
+  try {
+    const models = await client.listModels();
+    const match = models.find((m) => m.id === modelId);
+    if (match) {
+      return match.capabilities?.supports?.reasoningEffort === true;
+    }
+    return undefined;
+  } catch (err) {
+    debugLog('[Copilot SDK] Failed to query model capabilities: %s', err.message);
+    return undefined;
+  }
+}
+
 async function* createStreamingGenerator(client, prompt, options, signal, config) {
   const { model, timeout = 120000, reasoning_effort } = options;
 
@@ -542,12 +600,46 @@ async function* createStreamingGenerator(client, prompt, options, signal, config
   if (reasoning_effort) {
     const mapped = mapReasoningEffort(reasoning_effort);
     if (mapped) {
-      sessionConfig.reasoningEffort = mapped;
-      debugLog(`[Copilot SDK] Setting reasoningEffort: ${mapped} (from ${reasoning_effort})`);
+      const effectiveModel = sessionModel || model;
+      const modelDef = findModelConfig(effectiveModel);
+
+      let supported;
+      if (modelDef && modelDef.supportsReasoningEffort !== undefined) {
+        // Known model with explicit flag — use it directly (fast path)
+        supported = modelDef.supportsReasoningEffort;
+      } else {
+        // Unknown model or no static flag — query the SDK
+        supported = await checkReasoningSupport(client, effectiveModel);
+      }
+
+      if (supported === true) {
+        sessionConfig.reasoningEffort = mapped;
+        debugLog(`[Copilot SDK] Setting reasoningEffort: ${mapped} (from ${reasoning_effort})`);
+      } else if (supported === false) {
+        debugLog(`[Copilot SDK] Model "${effectiveModel}" does not support reasoningEffort (ignored)`);
+      } else {
+        // Could not determine — try optimistically, retry without on failure
+        sessionConfig.reasoningEffort = mapped;
+        debugLog(`[Copilot SDK] Model "${effectiveModel}" support unknown — trying reasoningEffort: ${mapped}`);
+      }
     }
   }
 
-  const session = await client.createSession(sessionConfig);
+  let session;
+  try {
+    session = await client.createSession(sessionConfig);
+  } catch (err) {
+    if (
+      sessionConfig.reasoningEffort &&
+      /does not support reasoning effort/i.test(err.message)
+    ) {
+      debugLog('[Copilot SDK] Model rejected reasoningEffort — retrying without it');
+      delete sessionConfig.reasoningEffort;
+      session = await client.createSession(sessionConfig);
+    } else {
+      throw err;
+    }
+  }
 
   try {
     yield {
@@ -716,7 +808,7 @@ export const copilotProvider = {
       const prompt = convertMessagesToPrompt(messages);
 
       const sessionModel = resolveSessionModel(model, config);
-      const modelConfig = SUPPORTED_MODELS.copilot;
+      const modelConfig = findModelConfig(sessionModel || model) || SUPPORTED_MODELS.copilot;
       const invokeOptions = {
         model,
         timeout: modelConfig.timeout,
@@ -831,29 +923,6 @@ export const copilotProvider = {
   },
 
   getModelConfig(modelName) {
-    if (typeof modelName !== 'string') return null;
-
-    let name = modelName;
-    if (name.toLowerCase().startsWith('copilot:')) {
-      name = name.slice('copilot:'.length).trim();
-    }
-    if (!name) return SUPPORTED_MODELS.copilot;
-
-    const nameLower = name.toLowerCase();
-
-    if (SUPPORTED_MODELS[nameLower]) {
-      return SUPPORTED_MODELS[nameLower];
-    }
-
-    for (const config of Object.values(SUPPORTED_MODELS)) {
-      if (
-        config.aliases &&
-				config.aliases.some((alias) => alias.toLowerCase() === nameLower)
-      ) {
-        return config;
-      }
-    }
-
-    return null;
+    return findModelConfig(modelName);
   },
 };