converse-mcp-server 2.9.2 → 2.9.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "converse-mcp-server",
-  "version": "2.9.2",
+  "version": "2.9.4",
   "description": "Converse MCP Server - Converse with other LLMs with chat and consensus tools",
   "type": "module",
   "main": "src/index.js",
@@ -11,60 +11,6 @@
   "engines": {
     "node": ">=20.0.0"
   },
-  "scripts": {
-    "kill-server": "node scripts/kill-server.js",
-    "start": "npm run kill-server && node src/index.js",
-    "start:clean": "node src/index.js",
-    "start:port": "cross-env PORT=3001 npm run start:clean",
-    "dev": "npm run kill-server && cross-env NODE_ENV=development LOG_LEVEL=debug node --watch src/index.js",
-    "dev:clean": "cross-env NODE_ENV=development LOG_LEVEL=debug node --watch src/index.js",
-    "dev:port": "cross-env PORT=3001 npm run dev:clean",
-    "dev:quiet": "npm run kill-server && cross-env NODE_ENV=development LOG_LEVEL=info node --watch src/index.js",
-    "dev:verbose": "npm run kill-server && cross-env NODE_ENV=development LOG_LEVEL=trace node --watch src/index.js",
-    "test": "npm run test:all",
-    "test:all": "vitest run",
-    "test:watch": "vitest",
-    "test:unit": "vitest run --config vitest.unit.config.js",
-    "test:integration": "vitest run --config vitest.integration.config.js",
-    "test:integration:mcp": "vitest run tests/integration/mcp-protocol",
-    "test:integration:tools": "vitest run tests/integration/tools",
-    "test:integration:providers": "vitest run tests/integration/providers",
-    "test:integration:performance": "vitest run tests/integration/performance",
-    "test:integration:general": "vitest run tests/integration/general",
-    "test:e2e": "npm run test:real-api",
-    "test:providers": "vitest run --config vitest.providers.config.js",
-    "test:tools": "vitest run tests/tools",
-    "test:mcp-client": "vitest run --config vitest.mcp-client.config.js",
-    "test:real-api": "vitest run --config vitest.real-api.config.js",
-    "test:performance": "vitest run --config vitest.performance.config.js",
-    "test:ci": "vitest run --config vitest.ci.config.js",
-    "test:utils": "vitest run tests/utils",
-    "test:resources": "vitest run tests/resources",
-    "test:prompts": "vitest run tests/prompts",
-    "test:coverage": "vitest run --coverage",
-    "test:coverage:unit": "vitest run --config vitest.unit.config.js --coverage",
-    "test:coverage:integration": "vitest run --config vitest.integration.config.js --coverage",
-    "test:coverage:mcp-client": "vitest run --config vitest.mcp-client.config.js --coverage",
-    "test:ui": "vitest --ui",
-    "lint": "eslint src/ tests/",
-    "lint:fix": "eslint src/ tests/ --fix",
-    "lint:watch": "nodemon --exec \"npm run lint\" --watch src --watch tests",
-    "format": "prettier --write src/ tests/ *.md *.json",
-    "format:check": "prettier --check src/ tests/ *.md *.json",
-    "typecheck": "node scripts/typecheck.js",
-    "validate:simple": "npm run typecheck && npm run lint && npm run format:check && npm run test",
-    "clean": "rimraf node_modules package-lock.json && npm install",
-    "debug": "cross-env NODE_ENV=development LOG_LEVEL=trace node --inspect src/index.js",
-    "debug:break": "cross-env NODE_ENV=development LOG_LEVEL=trace node --inspect-brk src/index.js",
-    "check-deps": "npm outdated",
-    "update-deps": "npm update",
-    "security-audit": "npm audit",
-    "dev-server": "node dev-server.js",
-    "validate": "node scripts/validate.js",
-    "validate:fix": "node scripts/validate.js --fix",
-    "validate:fast": "node scripts/validate.js --skip-tests --skip-lint",
-    "precommit": "npm run validate"
-  },
   "keywords": [
     "mcp",
     "server",
@@ -118,5 +64,59 @@
     "prettier": "^3.6.2",
     "rimraf": "^6.1.0",
     "vitest": "^4.0.10"
+  },
+  "scripts": {
+    "kill-server": "node scripts/kill-server.js",
+    "start": "npm run kill-server && node src/index.js",
+    "start:clean": "node src/index.js",
+    "start:port": "cross-env PORT=3001 npm run start:clean",
+    "dev": "npm run kill-server && cross-env NODE_ENV=development LOG_LEVEL=debug node --watch src/index.js",
+    "dev:clean": "cross-env NODE_ENV=development LOG_LEVEL=debug node --watch src/index.js",
+    "dev:port": "cross-env PORT=3001 npm run dev:clean",
+    "dev:quiet": "npm run kill-server && cross-env NODE_ENV=development LOG_LEVEL=info node --watch src/index.js",
+    "dev:verbose": "npm run kill-server && cross-env NODE_ENV=development LOG_LEVEL=trace node --watch src/index.js",
+    "test": "npm run test:all",
+    "test:all": "vitest run",
+    "test:watch": "vitest",
+    "test:unit": "vitest run --config vitest.unit.config.js",
+    "test:integration": "vitest run --config vitest.integration.config.js",
+    "test:integration:mcp": "vitest run tests/integration/mcp-protocol",
+    "test:integration:tools": "vitest run tests/integration/tools",
+    "test:integration:providers": "vitest run tests/integration/providers",
+    "test:integration:performance": "vitest run tests/integration/performance",
+    "test:integration:general": "vitest run tests/integration/general",
+    "test:e2e": "npm run test:real-api",
+    "test:providers": "vitest run --config vitest.providers.config.js",
+    "test:tools": "vitest run tests/tools",
+    "test:mcp-client": "vitest run --config vitest.mcp-client.config.js",
+    "test:real-api": "vitest run --config vitest.real-api.config.js",
+    "test:performance": "vitest run --config vitest.performance.config.js",
+    "test:ci": "vitest run --config vitest.ci.config.js",
+    "test:utils": "vitest run tests/utils",
+    "test:resources": "vitest run tests/resources",
+    "test:prompts": "vitest run tests/prompts",
+    "test:coverage": "vitest run --coverage",
+    "test:coverage:unit": "vitest run --config vitest.unit.config.js --coverage",
+    "test:coverage:integration": "vitest run --config vitest.integration.config.js --coverage",
+    "test:coverage:mcp-client": "vitest run --config vitest.mcp-client.config.js --coverage",
+    "test:ui": "vitest --ui",
+    "lint": "eslint src/ tests/",
+    "lint:fix": "eslint src/ tests/ --fix",
+    "lint:watch": "nodemon --exec \"npm run lint\" --watch src --watch tests",
+    "format": "prettier --write src/ tests/ *.md *.json",
+    "format:check": "prettier --check src/ tests/ *.md *.json",
+    "typecheck": "node scripts/typecheck.js",
+    "validate:simple": "npm run typecheck && npm run lint && npm run format:check && npm run test",
+    "clean": "rimraf node_modules package-lock.json && npm install",
+    "debug": "cross-env NODE_ENV=development LOG_LEVEL=trace node --inspect src/index.js",
+    "debug:break": "cross-env NODE_ENV=development LOG_LEVEL=trace node --inspect-brk src/index.js",
+    "check-deps": "npm outdated",
+    "update-deps": "npm update",
+    "security-audit": "npm audit",
+    "dev-server": "node dev-server.js",
+    "validate": "node scripts/validate.js",
+    "validate:fix": "node scripts/validate.js --fix",
+    "validate:fast": "node scripts/validate.js --skip-tests --skip-lint",
+    "precommit": "npm run validate"
   }
-}
+}

package/src/prompts/helpPrompt.js

@@ -326,6 +326,10 @@ function generateConfigurationTips(tools) {
 
   // File Context
   output += '### File Context\n';
+  output +=
+    '- **IMPORTANT**: Always use the `files` parameter to share code/file content instead of copying into the prompt\n';
+  output +=
+    '- Using `files` provides better formatting, line numbers, and preserves full context\n';
   output +=
     '- Supports multiple file formats: code files, text, markdown, JSON, etc.\n';
   output +=
@@ -509,67 +513,19 @@ These providers use local CLI tools and don't require API keys:
 
 /**
  * Help prompt handler function
- * @param {object} args - Prompt arguments
+ * @param {object} args - Prompt arguments (unused)
  * @param {object} config - Configuration object (optional)
  */
 export async function helpPromptHandler(args = {}, config = null) {
-  const { topic } = args;
-
   const fullHelp = generateHelpContent(config);
 
-  // If no topic specified, return full help
-  if (!topic) {
-    return {
-      messages: [
-        {
-          role: 'user',
-          content: {
-            type: 'text',
-            text: `Please provide the following comprehensive help guide for the Converse MCP Server to the user. Share all of this information with them:\n\n${fullHelp}`,
-          },
-        },
-      ],
-    };
-  }
-
-  // Extract specific sections based on topic
-  const topicLower = topic.toLowerCase();
-  let sectionContent = '';
-
-  if (topicLower === 'tools') {
-    const toolsMatch = fullHelp.match(/## Available Tools[\s\S]*?(?=## Provider Models|$)/);
-    sectionContent = toolsMatch ? toolsMatch[0] : 'Tools section not found';
-  } else if (topicLower === 'models' || topicLower === 'providers') {
-    const modelsMatch = fullHelp.match(
-      /## Provider Models[\s\S]*?(?=## Model Categories|$)/,
-    );
-    sectionContent = modelsMatch ? modelsMatch[0] : 'Models section not found';
-  } else if (topicLower === 'parameters') {
-    const paramsMatch = fullHelp.match(
-      /## Configuration Tips[\s\S]*?(?=## Best Practices|$)/,
-    );
-    sectionContent = paramsMatch
-      ? paramsMatch[0]
-      : 'Parameters section not found';
-  } else if (topicLower === 'examples') {
-    // Extract example usage from all tools
-    const examples = fullHelp.match(
-      /\*\*Example Usage:\*\*[\s\S]*?```[\s\S]*?```/g,
-    );
-    sectionContent = examples
-      ? '## Examples\n\n' + examples.join('\n\n')
-      : 'Examples not found';
-  } else {
-    sectionContent = `Topic "${topic}" not found. Available topics: tools, models, providers, parameters, examples`;
-  }
-
   return {
     messages: [
       {
         role: 'user',
         content: {
           type: 'text',
-          text: `Please share this help information about "${topic}" for the Converse MCP Server with the user:\n\n${sectionContent}`,
+          text: `Please provide the following comprehensive help guide for the Converse MCP Server to the user. Share all of this information with them:\n\n${fullHelp}`,
         },
       },
     ],
@@ -582,13 +538,6 @@ export async function helpPromptHandler(args = {}, config = null) {
 export const helpPromptMetadata = {
   name: 'help',
   description:
-    'Comprehensive guide for using the Converse MCP Server with all tools, parameters, and models',
-  arguments: [
-    {
-      name: 'topic',
-      description:
-        'Specific topic to get help on (optional). Options: tools, models, providers, parameters, examples',
-      required: false,
-    },
-  ],
+    'Comprehensive guide for Converse MCP Server - tools, models, parameters, and configuration',
+  arguments: [],
 };

package/src/tools/chat.js

@@ -1016,7 +1016,7 @@ async function executeChatWithStreaming(args, dependencies, context) {
 
 // Tool metadata
 chatTool.description =
-  'GENERAL CHAT & COLLABORATIVE THINKING - Development assistance, brainstorming, code analysis. Supports files, images, continuation_id for multi-turn conversations. Use model: "auto" for automatic selection.';
+  'GENERAL CHAT & COLLABORATIVE THINKING - Development assistance, brainstorming, code analysis. Supports files, images, continuation_id for multi-turn conversations. Use model: "auto" for automatic selection. IMPORTANT: Use the "files" parameter to share code/file content instead of pasting into the prompt.';
 chatTool.inputSchema = {
   type: 'object',
   properties: {
@@ -1029,7 +1029,7 @@ chatTool.inputSchema = {
       type: 'array',
       items: { type: 'string' },
       description:
-        'File paths to include as context (absolute or relative paths). Supports line ranges: file.txt{10:50}, file.txt{100:}. Example: ["./src/utils/auth.js{50:100}", "./config.json"]',
+        'File paths to include as context (absolute or relative paths). Supports line ranges: file.txt{10:50}, file.txt{100:}. Example: ["./src/utils/auth.js{50:100}", "./config.json"]. IMPORTANT: Always use this parameter to share file content instead of copying code into the prompt - it provides better formatting, line numbers, and preserves context.',
     },
     images: {
       type: 'array',

package/src/tools/consensus.js

@@ -1616,7 +1616,7 @@ async function executeConsensusPhaseWithStreaming(
 
 // 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.';
+  '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. IMPORTANT: Use the "files" parameter to share code/file content instead of pasting into the prompt.';
 consensusTool.inputSchema = {
   type: 'object',
   properties: {
@@ -1631,7 +1631,7 @@ consensusTool.inputSchema = {
       type: 'array',
       items: { type: 'string' },
       description:
-        'File paths for additional context (absolute or relative paths). Supports line ranges: file.txt{10:50}, file.txt{100:}. Example: ["./docs/architecture.md{1:100}", "./requirements.txt"]',
+        'File paths for additional context (absolute or relative paths). Supports line ranges: file.txt{10:50}, file.txt{100:}. Example: ["./docs/architecture.md{1:100}", "./requirements.txt"]. IMPORTANT: Always use this parameter to share file content instead of copying code into the prompt - it provides better formatting, line numbers, and preserves context.',
     },
     images: {
       type: 'array',