natureco-cli 5.21.0 → 5.23.0

package/CHANGELOG.md

@@ -2,6 +2,13 @@
 
 All notable changes to NatureCo CLI will be documented in this file.
 
+## [5.23.0] - 2026-07-02 — "NON-TOOL-CALLING FIX" (MiniMax dosya yazma)
+
+### 🐛 Düzeltme
+- **MiniMax (non-tool-calling) dosya oluşturamıyordu**: workflow non-tool-calling path'inde simple/complex ayrımı yoktu — LLM yanıtını doğrudan sohbete yazıp geçiyordu. Artık complex görevlerde LLM'den JSON plan istenir, dosyalar Node.js tarafında yazılır.
+
+## [5.22.0] - 2026-07-02
+
 ## [5.21.0] - 2026-07-02 — "GÜVENİLİRLİK SPRİNTİ" (gerçek API E2E denetimi)
 
 Gerçek MiniMax API anahtarıyla uçtan uca canlı test turu; bulunan her hata düzeltilip yine canlı doğrulandı. 461 test yeşil.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "natureco-cli",
-  "version": "5.21.0",
+  "version": "5.23.0",
   "description": "OpenClaw'dan daha güvenli, daha hızlı, daha ucuz AI agent CLI. Multi-agent, self-evolving skills, audit log, maliyet optimizasyonu ve NatureCo platform-native.",
   "bin": {
     "natureco": "bin/natureco.js"

package/src/commands/repl.js

@@ -1090,6 +1090,8 @@ async function startRepl(args) {
     memorySnapshotBlock, skillsIndexBlock, projectRules,
     crossSessionContext: crossSessionContext || '',
     userHome: cfg.userHome || '',
+    platform: process.platform,
+    desktopPath: cfg.userHome ? path.join(cfg.userHome, 'Desktop') : '',
     hasHistory: messages.length > 0,
     memoryFacts: memory.facts || [],
   };
@@ -1468,7 +1470,8 @@ async function startRepl(args) {
       // v5.13.0: Run workflow FIRST for every request
       process.stdout.write(tui.styled('\r  🔧 workflow...  ', { color: tui.PALETTE.muted }));
       const wfToolDefs = getToolDefs();
-      const wfResult = await executeTool('workflow', { action: 'run', task: line }, wfToolDefs);
+      const recentHistory = messages.length > 1 ? messages.slice(-10) : [];
+      const wfResult = await executeTool('workflow', { action: 'run', task: line, conversationHistory: recentHistory }, wfToolDefs);
       const wf = wfResult?.result || {};
       if (wf.success !== false) {
         const loaded = wf.skillsLoaded && wf.skillsLoaded.length > 0 ? ` [skill: ${wf.skillsLoaded.join(', ')}]` : '';

package/src/tools/workflow.js

@@ -83,7 +83,7 @@ function apiCall(providerUrl, apiKey, body) {
 }
 
 async function workflow(params) {
-  const { action, task, steps, name, workflowId, regenerateStep } = params;
+  const { action, task, steps, name, workflowId, regenerateStep, conversationHistory } = params;
   const cfg = loadConfig();
   const tools = allToolNames();
   ensureDir(WORKFLOW_DIR);
@@ -98,6 +98,19 @@ async function workflow(params) {
 
   const skillsIndexBlock = buildSkillIndex();
 
+  // Build chat messages with optional conversation history for context
+  function chatMessages(sysMsg, userTask) {
+    const msgs = [{ role: 'system', content: sysMsg }];
+    if (conversationHistory && Array.isArray(conversationHistory)) {
+      for (const m of conversationHistory) {
+        if (m._internal) continue;
+        msgs.push({ role: m.role, content: m.content || '' });
+      }
+    }
+    msgs.push({ role: 'user', content: userTask });
+    return msgs;
+  }
+
   // Non-tool-calling model tespiti
   function supportsToolCalls() {
     const url = (providerUrl || '').toLowerCase();
@@ -113,14 +126,88 @@ async function workflow(params) {
   if (action === 'run') {
     if (!task) return { success: false, error: 'task gerekli' };
 
-    // Non-tool-calling modellerde dogrudan passthrough (plan yok, direkt uretim)
+    // Non-tool-calling modellerde once simple/complex kontrolu yap
     if (!supportsToolCalls()) {
       const memCtx = memoryContext();
-      const sysMsg = 'Sen yardimci bir asistansin. Kullanici ne istediyse onu dogrudan yap. Dosya olusturulacaksa icerigi eksiksiz olarak yanitla.' + (memCtx ? '\n\nKullanici bilgisi:\n' + memCtx : '') + '\n\n' + skillsIndexBlock;
-      const chatBody = { model, stream: false, messages: [{ role: 'system', content: sysMsg }, { role: 'user', content: task }], temperature: 0.7, max_tokens: 4000 };
+      // Phase 0: Check if simple chat or complex task
+      const simpleCheckPrompt = {
+        role: 'system',
+        content: 'Gorevin basit bir selamlasma/sohbet mi yoksa arac gerektiren bir islem mi oldugunu belirle. Sadece "simple" veya "complex" yaz, kesinlikle baska bir sey yazma.\n\nSimple: selamlasma, nasilsin, bugun ne yaptin, havadan sudan, genel bilgi sorusu, ben kimim, adim ne, kullanici bilgisi sorgulama, hatirlatma talebi\nComplex: dosya islemleri, kod yazma, arastirma, karsilastirma, duzenleme, otomasyon'
+      };
+      let isComplex = true;
       try {
-        const chatResult = await apiCall(providerUrl, providerApiKey, chatBody);
-        const reply = chatResult.choices?.[0]?.message?.content || '';
+        const checkBody = { model, stream: false, messages: [simpleCheckPrompt, { role: 'user', content: task }], temperature: 0, max_tokens: 20 };
+        const checkResult = await apiCall(providerUrl, providerApiKey, checkBody);
+        const raw = (checkResult.choices?.[0]?.message?.content || '').trim().toLowerCase().replace(/[^a-z]/g, '');
+        isComplex = raw !== 'simple';
+      } catch {}
+
+      if (!isComplex) {
+        // Simple chat — passthrough
+        const sysMsg = 'Sen yardimci bir asistansin. Kisa ve oz yanit ver. Konusma gecmisi varsa onceki mesajlari dikkate al.' + (memCtx ? '\n\nKullanici bilgisi:\n' + memCtx : '') + '\n\n' + skillsIndexBlock;
+        const chatBody = { model, stream: false, messages: chatMessages(sysMsg, task), temperature: 0.7, max_tokens: 1000 };
+        try {
+          const chatResult = await apiCall(providerUrl, providerApiKey, chatBody);
+          const reply = chatResult.choices?.[0]?.message?.content || '';
+          return { success: true, workflowId: 'passthrough', name: 'Direct Chat', status: 'completed', totalSteps: 0, completedSteps: 0, results: [], passthrough: true, reply };
+        } catch (e) {
+          return { success: false, error: 'Sohbet yaniti alinamadi: ' + e.message };
+        }
+      }
+
+      // Complex task for non-tool-calling model: ask LLM for structured file ops
+      const execSysMsg = 'Gorevi tamamlamak icin hangi dosyalarin olusturulacagini JSON olarak belirt. SADECE JSON yaz.\n\n' +
+        'Kullanici bilgisi:\n' + (memCtx || '') +
+        '\n\nMasaustu yolu: ' + require('path').join(require('os').homedir(), 'Desktop') +
+        '\n\nKullanici home: ' + require('os').homedir() +
+        '\n\nJSON format:\n{"files": [{"path": "/tam/dosya/yolu/dosya.html", "content": "dosya icerigi buraya"}]}\n\nHer dosya icin path ve content zorunlu. path TAM yol olmali (~ veya goreceli degil). Birden fazla dosya olusturulabilir.' +
+        '\n\n' + skillsIndexBlock;
+      const execBody = { model, stream: false, messages: chatMessages(execSysMsg, task), temperature: 0.3, max_tokens: 8000 };
+      try {
+        const execResult = await apiCall(providerUrl, providerApiKey, execBody);
+        const reply = execResult.choices?.[0]?.message?.content || '';
+
+        // Try to extract JSON with file operations
+        const jsonStart = reply.indexOf('{');
+        const jsonEnd = reply.lastIndexOf('}');
+        let stepResults = [];
+        let fileWriteSuccess = false;
+
+        if (jsonStart !== -1 && jsonEnd > jsonStart) {
+          try {
+            const plan = JSON.parse(reply.slice(jsonStart, jsonEnd + 1));
+            if (plan.files && Array.isArray(plan.files)) {
+              // Execute file writes locally
+              for (const f of plan.files) {
+                try {
+                  const wfPath = f.path ? require('path').resolve(f.path.replace(/^~/, require('os').homedir())) : '';
+                  if (!wfPath) continue;
+                  require('fs').mkdirSync(require('path').dirname(wfPath), { recursive: true });
+                  require('fs').writeFileSync(wfPath, f.content || '', 'utf-8');
+                  const stats = require('fs').statSync(wfPath);
+                  stepResults.push({ step: 1, tool: 'write_file', status: 'done', args: { path: wfPath }, result: { success: true, path: wfPath, size: stats.size } });
+                  fileWriteSuccess = true;
+                } catch (wfErr) {
+                  stepResults.push({ step: 1, tool: 'write_file', status: 'error', error: wfErr.message });
+                }
+              }
+            }
+          } catch {}
+        }
+
+        if (fileWriteSuccess) {
+          const lines = stepResults.map(r => { const p = r.result?.path || ''; const s = r.result?.size || 0; const name = require('path').basename(p); return `  ✓ ${name} oluşturuldu (${s} bytes)`; }).join('\n');
+          const reply = `Dosya(lar) başarıyla oluşturuldu:\n${lines}`;
+          return {
+            success: true, workflowId: 'file_write_' + Date.now().toString(36),
+            name: 'File Operations', status: 'completed',
+            totalSteps: stepResults.length, completedSteps: stepResults.length,
+            results: stepResults,
+            passthrough: true, reply,
+          };
+        }
+
+        // If no structured file ops detected, return LLM reply as passthrough
         return { success: true, workflowId: 'passthrough', name: 'Direct Generation', status: 'completed', totalSteps: 0, completedSteps: 0, results: [{ step: 0, tool: 'chat', status: 'done', result: { reply } }], passthrough: true, reply };
       } catch (e) {
         return { success: false, error: 'Yanit alinamadi: ' + e.message };
@@ -141,10 +228,10 @@ async function workflow(params) {
     } catch {}
 
     if (isSimple) {
-      // Passthrough: just chat with LLM, no tools — include user memory
+      // Passthrough: just chat with LLM, no tools — include user memory + conversation history
       const memCtx = memoryContext();
-      const sysMsg = 'Sen yardimci bir asistansin. Kisa ve oz yanit ver.' + (memCtx ? '\n\nKullanici bilgisi:\n' + memCtx : '') + '\n\n' + skillsIndexBlock;
-      const chatBody = { model, stream: false, messages: [{ role: 'system', content: sysMsg }, { role: 'user', content: task }], temperature: 0.7, max_tokens: 1000 };
+      const sysMsg = 'Sen yardimci bir asistansin. Kisa ve oz yanit ver. Konusma gecmisi varsa onceki mesajlari dikkate al.' + (memCtx ? '\n\nKullanici bilgisi:\n' + memCtx : '') + '\n\n' + skillsIndexBlock;
+      const chatBody = { model, stream: false, messages: chatMessages(sysMsg, task), temperature: 0.7, max_tokens: 1000 };
       try {
         const chatResult = await apiCall(providerUrl, providerApiKey, chatBody);
         const reply = chatResult.choices?.[0]?.message?.content || '';
@@ -417,6 +504,7 @@ module.exports = {
       regenerateStep: { type: 'number', description: '(retry) Yeniden calistirilacak adim numarasi' },
       newParams: { type: 'object', description: '(retry) Yeni parametreler' },
       description: { type: 'string', description: 'Aciklama' },
+      conversationHistory: { type: 'array', description: '(internal) REPL konusma gecmisi', items: { type: 'object' } },
     },
     required: ['action'],
   },

package/src/utils/system-prompt.js

@@ -23,6 +23,8 @@ function buildTiers(opts) {
     skillsIndexBlock = '',
     crossSessionContext = '',
     userHome = '',
+    platform = '',
+    desktopPath = '',
     hasHistory = false,
     memoryFacts = [],
     projectRules = '',
@@ -58,7 +60,8 @@ function buildTiers(opts) {
     `ONEMLI: Tool cagirma SIMULE ETME. Sadece duz metin cevap ver. Islem yapmak gerekirse tool'u gercekten cagir.`,
     `TOOL KURALI: Selamlasma, sohbet, bilgi sorusu, fikir/aciklama isteklerinde TOOL CAGIRMA — dogrudan yanit ver. Bu en hizli ve en ucuz yoldur.`,
     `EYLEM gerektiren isteklerde (dosya okuma/yazma, komut calistirma, arama, hatirlatici, cok adimli gorev) workflow(action="run", task="<istek>") cagir — uygun tool'lari secip sirayla calistirir.`,
-    `Tek ve net bir tool yeterliyse (or. read_file, web_search) workflow yerine dogrudan o tool'u cagirabilirsin.`,
+    `Tek ve net bir tool yeterliyse (or. read_file, web_search, write_file) workflow yerine dogrudan o tool'u cagirabilirsin.`,
+    `DOSYA YAZMA: SADECE write_file tool'unu kullan. "bulk-file-operations", "create-file", "file-write" gibi tool'lar YOK. write_file(content, file_path) kullan. Dosya yolu olarak desktopPath veya userHome kullan.`,
     `COK KRITIK: Goreve baslamadan ONCE <available_skills> listesini tara. Ilgili skill varsa skill_view(name) ile yukle, SONRA goreve basla.`,
     `KRITIK: Skill yuklemeden islem yapma. Ilgili skill varsa once yukle.`,
     `KRITIK: Kullanici kisisel bilgi verdiginde memory(action=add, target=user) ile kaydet.`,
@@ -88,6 +91,8 @@ function buildTiers(opts) {
 
     // User environment (stable within session but changes on resume)
     userHome ? `Kullanicinin home: ${userHome}` : '',
+    platform ? `Isletim sistemi: ${platform}` : '',
+    desktopPath ? `Masaustu yolu: ${desktopPath}` : '',
     hasHistory ? 'Bu oturum daha onceki konusmalarin devami.' : '',
   ].filter(Boolean).join('\n');