natureco-cli 5.64.5 → 5.64.6

package/CHANGELOG.md

@@ -2,6 +2,16 @@
 
 All notable changes to NatureCo CLI will be documented in this file.
 
+## [5.64.6] - 2026-07-13 — Typed MiniMax tool arguments
+
+### Fixed
+- MiniMax XML tool parameters are now coerced from text to their declared JSON Schema types before validation and execution.
+- `computer_use_loop` now accepts XML values such as `<parameter name="maxSteps">30</parameter>` as the number `30`, so visible GUI automation starts instead of failing with `expected number, got string`.
+- Conversion also covers integer, boolean, object, and array parameters while leaving invalid values untouched for normal schema validation.
+
+### Tests
+- Added regressions for valid type conversion, invalid-value preservation, and the exact `computer_use_loop` `maxSteps` failure reported on macOS.
+
 ## [5.64.5] - 2026-07-13 — Deterministic visible-browser recovery
 
 ### Fixed

package/README.md

@@ -51,6 +51,7 @@ natureco code
 
 | Version | Highlights |
 |---------|-----------|
+| **v5.64.6** | **Reliable MiniMax tool calls:** XML parameters are converted to their declared schema types, fixing `computer_use_loop` failures such as `maxSteps: expected number, got string`. |
 | **v5.64.5** | **Deterministic GUI recovery:** visible web tasks use one verified GUI loop; failures expose their concrete reason and stop blind browser/AppleScript fallback chains. |
 | **v5.64.4** | **Unified MiniMax media:** the existing MiniMax key now powers `MiniMax-VL-01` analysis, `image-01` generation, `MiniMax-Hailuo-2.3` video, and verified GUI vision—no second key required. |
 | **v5.64.3** | **Evidence-based GUI completion:** desktop actions require a changed screen plus independent visual evidence before success. MiniMax screenshots use the Token Plan VLM with the existing provider key. |

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "natureco-cli",
-  "version": "5.64.5",
+  "version": "5.64.6",
   "description": "Terminal-native AI agent CLI with bilingual TR/EN UI, multi-agent orchestration, persistent memory, secure tools and messaging integrations.",
   "bin": {
     "natureco": "bin/natureco.js"

package/src/tools/agentic-runner.js

@@ -288,6 +288,37 @@ function buildFeedback(norm, res) {
   return body ? `${head}:\n${body}` : head;
 }
 
+function coerceArgsForSchema(args, schema) {
+  if (!args || typeof args !== 'object' || !schema || typeof schema !== 'object') return { ...(args || {}) };
+  const properties = schema.properties || {};
+  const result = { ...args };
+
+  for (const [key, property] of Object.entries(properties)) {
+    const value = result[key];
+    if (typeof value !== 'string' || !property) continue;
+    const types = Array.isArray(property.type) ? property.type : [property.type];
+    const trimmed = value.trim();
+
+    if (types.includes('integer') && /^[-+]?\d+$/.test(trimmed)) {
+      const number = Number(trimmed);
+      if (Number.isSafeInteger(number)) result[key] = number;
+    } else if (types.includes('number') && trimmed !== '' && Number.isFinite(Number(trimmed))) {
+      result[key] = Number(trimmed);
+    } else if (types.includes('boolean') && /^(true|false)$/i.test(trimmed)) {
+      result[key] = trimmed.toLowerCase() === 'true';
+    } else if (types.includes('object') || types.includes('array')) {
+      try {
+        const parsed = JSON.parse(trimmed);
+        if ((types.includes('array') && Array.isArray(parsed)) ||
+            (types.includes('object') && parsed && typeof parsed === 'object' && !Array.isArray(parsed))) {
+          result[key] = parsed;
+        }
+      } catch {}
+    }
+  }
+  return result;
+}
+
 /**
  * Tek bir parse edilmis cagriyi calistir.
  * Donus: { records: [...], feedback: '...' }
@@ -389,9 +420,11 @@ async function executeCall(call, opts = {}) {
     records.push({ tool: norm, status: 'error', error: 'execute yok' });
     return { records, feedback: `${norm} HATA: execute fonksiyonu yok` };
   }
+  const schema = mod.inputSchema || mod.parameters || (mod.default && (mod.default.inputSchema || mod.default.parameters));
+  const typedArgs = coerceArgsForSchema(args, schema);
   const res = await executeThroughGateway({
     toolName: norm,
-    args,
+    args: typedArgs,
     resolveTool: () => mod,
     execute: fn,
     normalizeSuccess: value => value,
@@ -401,7 +434,7 @@ async function executeCall(call, opts = {}) {
   const ok = typeof res === 'string' ? true : (res && res.success !== false);
   const status = ok ? 'done' : 'error';
   const feedback = buildFeedback(norm, res);
-  records.push({ tool: norm, status, args: sanitizeArgs(args), result: res, error: ok ? undefined : res.error });
+  records.push({ tool: norm, status, args: sanitizeArgs(typedArgs), result: res, error: ok ? undefined : res.error });
   return { records, feedback };
 }
 
@@ -462,4 +495,4 @@ async function runAgentic({ callModel, systemPrompt, historyMessages, task, tool
   return { records: allRecords, reply: finalReply, iterations };
 }
 
-module.exports = { parseAgenticCalls, stripProtocolTokens, executeCall, runAgentic, expandHome, makeStreamFilter, makeSanitizeStream, agentExecAllowed, buildFeedback, TOOL_ALIASES, DEFAULT_ALLOWED };
+module.exports = { parseAgenticCalls, stripProtocolTokens, executeCall, runAgentic, expandHome, makeStreamFilter, makeSanitizeStream, agentExecAllowed, buildFeedback, coerceArgsForSchema, TOOL_ALIASES, DEFAULT_ALLOWED };