natureco-cli 2.11.2 → 2.11.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "natureco-cli",
-  "version": "2.11.2",
+  "version": "2.11.4",
   "description": "NatureCo AI Bot Terminal Interface",
   "main": "bin/natureco.js",
   "bin": {

package/src/commands/dashboard.js

@@ -211,7 +211,7 @@ body::before{
         <div class="header-bot-name" id="header-bot-name">Nature Bot</div>
         <div class="header-bot-model" id="header-bot-model">NatureCo</div>
       </div>
-      <div class="version-badge" id="version-badge">v2.11.2</div>
+      <div class="version-badge" id="version-badge">v2.11.4</div>
     </div>
     <div class="messages" id="messages"></div>
     <div class="input-area">
@@ -341,7 +341,7 @@ function dashboard(action) {
           apiKey: cfg.apiKey,
           defaultBot: cfg.defaultBot,
           defaultBotId: cfg.defaultBotId,
-          version: 'v2.11.2',
+          version: 'v2.11.4',
           bots: cfg.bots || [],
           telegramToken: cfg.telegramToken || null,
           whatsappConnected: cfg.whatsappConnected || false,

package/src/commands/gateway-server.js

@@ -138,7 +138,7 @@ async function startGateway() {
 
 async function runGatewayWorker() {
   // This runs in the background
-  log('gateway', 'Starting NatureCo Gateway v2.11.2...', 'green');
+  log('gateway', 'Starting NatureCo Gateway v2.11.3...', 'green');
   
   // Load config
   const { getConfig } = require('../utils/config');

package/src/utils/api.js

@@ -136,35 +136,67 @@ function getMcpTools() {
   return allTools;
 }
 
+/**
+ * Normalize MCP tool schema for AI consumption
+ * Adds hints to number/integer parameters to prevent string conversion
+ */
+function normalizeMcpToolSchema(tool) {
+  const schema = tool.inputSchema || tool.input_schema || {};
+  
+  // Ensure properties exist
+  if (!schema.properties) return tool;
+  
+  // Clone schema to avoid mutating original
+  const normalizedSchema = JSON.parse(JSON.stringify(schema));
+  
+  // Groq sometimes sends strings for number params
+  // Add coercion hint to description
+  for (const [key, prop] of Object.entries(normalizedSchema.properties)) {
+    if (prop.type === 'number' || prop.type === 'integer') {
+      prop.description = (prop.description || '') + ' (must be a number, not a string)';
+    }
+  }
+  
+  return { ...tool, inputSchema: normalizedSchema };
+}
+
 /**
  * Coerce MCP tool parameters to match schema types
  */
 function coerceMcpParams(tool, params) {
-  const schema = tool.inputSchema?.properties || tool.input_schema?.properties || {};
+  // GitHub MCP uses inputSchema, others may use input_schema
+  // Try all possible schema locations
+  const schema = tool.inputSchema?.properties || 
+                 tool.input_schema?.properties || 
+                 tool.parameters?.properties ||  // fallback
+                 {};
+  
+  // Debug: Show which schema format was found
+  console.log('[DEBUG] COERCE TOOL:', tool.name);
+  console.log('[DEBUG] COERCE SCHEMA SOURCE:', 
+    tool.inputSchema?.properties ? 'inputSchema.properties' :
+    tool.input_schema?.properties ? 'input_schema.properties' :
+    tool.parameters?.properties ? 'parameters.properties' : 'NONE');
+  console.log('[DEBUG] COERCE SCHEMA:', JSON.stringify(schema).slice(0, 200));
+  
   const coerced = { ...params };
   
   for (const [key, def] of Object.entries(schema)) {
     if (coerced[key] === undefined || coerced[key] === null) continue;
     
-    // Coerce number
-    if (def.type === 'number' && typeof coerced[key] === 'string') {
+    // Coerce number or integer
+    if ((def.type === 'number' || def.type === 'integer') && typeof coerced[key] === 'string') {
       const num = Number(coerced[key]);
       if (!isNaN(num)) {
         coerced[key] = num;
+        console.log('[DEBUG] COERCED', key, 'from string to number:', coerced[key]);
       }
     }
     
     // Coerce boolean
     if (def.type === 'boolean' && typeof coerced[key] === 'string') {
       coerced[key] = coerced[key] === 'true' || coerced[key] === '1';
-    }
-    
-    // Coerce integer
-    if (def.type === 'integer' && typeof coerced[key] === 'string') {
-      const num = parseInt(coerced[key], 10);
-      if (!isNaN(num)) {
-        coerced[key] = num;
-      }
+      console.log('[DEBUG] COERCED', key, 'from string to boolean:', coerced[key]);
     }
   }
   
@@ -328,8 +360,11 @@ function formatToolsForOpenAI() {
   const localTools = getToolDefinitions();
   const mcpTools = getMcpTools();
   
-  // Combine local and MCP tools
-  const allTools = [...localTools, ...mcpTools];
+  // Normalize MCP tools before combining
+  const normalizedMcpTools = mcpTools.map(tool => normalizeMcpToolSchema(tool));
+  
+  // Combine local and normalized MCP tools
+  const allTools = [...localTools, ...normalizedMcpTools];
   
   return allTools.map(tool => ({
     type: 'function',
@@ -348,8 +383,11 @@ function formatToolsForAnthropic() {
   const localTools = getToolDefinitions();
   const mcpTools = getMcpTools();
   
-  // Combine local and MCP tools
-  const allTools = [...localTools, ...mcpTools];
+  // Normalize MCP tools before combining
+  const normalizedMcpTools = mcpTools.map(tool => normalizeMcpToolSchema(tool));
+  
+  // Combine local and normalized MCP tools
+  const allTools = [...localTools, ...normalizedMcpTools];
   
   return allTools.map(tool => ({
     name: tool.name,
@@ -655,6 +693,10 @@ ENCODING: Tool results are base64 encoded with prefix [BASE64_ENCODED_RESULT]:.
 NEVER show the base64 string to the user. Decode it internally and present the data in a human-readable format.
 The encoding is for security - you must decode and interpret the results before responding.
 
+CRITICAL: For MCP tools, ALWAYS send number/integer parameters as numbers (not strings).
+Example: per_page must be 3 (not "3"), issue_number must be 5 (not "5"), limit must be 10 (not "10").
+When you see a parameter with type "number" or "integer", convert string values to actual numbers before calling the tool.
+
 TOOL SELECTION GUIDE:
 - read_file: Use for .txt, .md, .json, .log, .csv files
 - bash with "cat file | head -100": Use for .sh, .py, .js, .ts, .env, config files, or any file with special characters