natureco-cli 5.37.0 → 5.38.0

package/CHANGELOG.md

@@ -2,6 +2,20 @@
 
 All notable changes to NatureCo CLI will be documented in this file.
 
+## [5.38.0] - 2026-07-08 — "FİNAL DOĞRULAMA: git/code_execution/http_request düzeltmeleri + git enjeksiyon açığı kapatıldı"
+
+Final doğrulama re-run'inde (agent yolu E2E) 3 fonksiyonel sorun + 1 güvenlik açığı bulundu ve düzeltildi.
+
+### 🔒 Güvenlik açığı (kapatıldı)
+- **git args komut enjeksiyonu + remote-yazma bypass (KRİTİK)**: `git` özel aracı `execSync('git log ' + args)` ile STRING komut kuruyordu → `args` içindeki `;`, `&&`, `$()`, backtick shell'de çalışıyordu (ör. `args: "-n1; rm -rf /"`). Ayrıca bu araç, agentic-runner'daki `git remote add`/`git push` bloklarını GÖRMÜYORDU (özel araç, bash guard'ından geçmez) → remote-yazma bypass. Artık `execFileSync` (shell:false) + tırnak-farkındalıklı tokenizer → metakarakterler işlem görmez; `remote add/set-url/remove/rename` engelli (salt-okunur remote serbest). POC: `-n1; echo PWNED` → git argüman sanıp reddetti, echo hiç çalışmadı.
+
+### 🐛 Düzeltmeler
+- **git "Unknown operation"**: araç yalnızca `operation` param'i kabul ediyordu; ajanlar `args:"log -n2"` gibi gönderince tanımıyor, bash'e sapıyordu. Artık esnek giriş (operation/args/command'dan parse), `cwd` param'i onurlandırma, env-tabanlı repo bulma (`NATURECO_PROJECT_DIR`/`INIT_CWD`/`PWD` — makineden bağımsız), +5 salt-okunur operasyon (show/remote/tag/describe/rev-parse). E2E: tek çağrıda `git:done`.
+- **code_execution Python bulamıyor**: sabit `python3` Windows'ta App-execution-alias tuzağına düşüyordu. Artık aday-deneme (`py`/`python`/`python3` — *nix'te `python3`/`python`), node için garantili `process.execPath`; yorumlayıcı yoksa net "Python kurulu değil" mesajı. E2E: `12! = 479001600` doğru.
+- **http_request `[object Object]`**: araç JSON gövdeyi PARSED NESNE döndürüyor, `buildFeedback` `String(obj)` yapınca `[object Object]` oluyordu → model değeri okuyamıyordu. Artık nesne alanlar JSON'a serileştirilir. E2E: nodejs/node stargazers_count doğru okundu.
+
+Doğrulama: grep/git/http/code_execution ajan üzerinden E2E (hepsi `done`); gerçek PTY 9/9; komut denetimi 23/23; güvenlik 20/20 yıkıcı + 7/7 hassas-dosya + git enjeksiyonu POC kapalı. **521 test yeşil** (12 yeni regresyon: git-tool + buildFeedback nesne-body), ESLint temiz.
+
 ## [5.37.0] - 2026-07-05 — "GÜVENLİK: 2 açık kapatıldı (inline-eval bypass + hassas dosya erişimi)"
 
 Sistematik guvenlik taramasi (POC'larla). 20 yikici komuttan 19'u zaten engelliydi; 2 gercek acik bulundu ve kapatildi.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "natureco-cli",
-  "version": "5.37.0",
+  "version": "5.38.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/tools/agentic-runner.js

@@ -278,7 +278,13 @@ function buildFeedback(norm, res) {
   let body = '';
   const first = ['content', 'output', 'stdout', 'text', 'transcript', 'body', 'reply', 'answer', 'summary', 'url', 'path'];
   const jsonish = ['results', 'items', 'matches', 'data', 'events', 'entries', 'files'];
-  for (const k of first) { if (res[k] != null && String(res[k]).trim()) { body = String(res[k]).slice(0, 2500); break; } }
+  // v5.38: body/data gibi alanlar nesne olabilir ( or. http_request JSON govdesi) —
+  // String(obj) "[object Object]" verir ve model degeri okuyamaz; nesneyse JSON'a cevir.
+  for (const k of first) {
+    if (res[k] == null) continue;
+    const v = typeof res[k] === 'object' ? JSON.stringify(res[k]) : String(res[k]);
+    if (v.trim() && v !== '[object Object]') { body = v.slice(0, 2500); break; }
+  }
   if (!body) for (const k of jsonish) { if (res[k] != null) { body = JSON.stringify(res[k]).slice(0, 2000); break; } }
   if (!body) {
     // hicbir bilinen alan yok → sonucu oldugu gibi ver (success/error disi anlamli alanlar)
@@ -436,4 +442,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, TOOL_ALIASES, DEFAULT_ALLOWED };
+module.exports = { parseAgenticCalls, stripProtocolTokens, executeCall, runAgentic, expandHome, makeStreamFilter, makeSanitizeStream, agentExecAllowed, buildFeedback, TOOL_ALIASES, DEFAULT_ALLOWED };

package/src/tools/code_execution.js

@@ -33,38 +33,50 @@ async function runCode({ code, language = 'auto', timeoutMs = 30000, cwd = null
     }
   }
 
-  if (language === 'python') { cmd = 'python3'; args = ['-c', code]; }
-  else if (language === 'node') { cmd = 'node'; args = ['-e', code]; }
-  else if (language === 'bash' || language === 'shell') { cmd = 'bash'; args = ['-c', code]; }
+  // v5.38: Yorumlayici adaylari — platformlar arasi saglam.
+  // node icin process.execPath her zaman mevcut; python icin Windows'ta py/python
+  // (python3 App-execution-alias tuzagina duser), *nix'te python3/python.
+  const isWin = process.platform === 'win32';
+  let candidates;
+  if (language === 'python') { candidates = isWin ? ['py', 'python', 'python3'] : ['python3', 'python']; args = ['-c', code]; }
+  else if (language === 'node') { candidates = [process.execPath]; args = ['-e', code]; }
+  else if (language === 'bash' || language === 'shell') { candidates = isWin ? ['bash'] : ['bash', 'sh']; args = ['-c', code]; }
   else return { success: false, error: `Desteklenmeyen dil: ${language}` };
 
-  return new Promise((resolve) => {
-    const proc = spawn(cmd, args, {
-      cwd: cwd || os.homedir(),
-      timeout: timeoutMs,
-      env: { ...process.env, FORCE_COLOR: '0' },
-    });
+  const truncated = (s) => s.length > 8000 ? s.slice(0, 8000) + '\n... (kesildi, ' + (s.length - 8000) + ' karakter daha)' : s;
 
-    let stdout = '';
-    let stderr = '';
+  const spawnOnce = (bin) => new Promise((resolve) => {
+    let stdout = '', stderr = '', proc;
+    try {
+      proc = spawn(bin, args, { cwd: cwd || os.homedir(), timeout: timeoutMs, env: { ...process.env, FORCE_COLOR: '0' } });
+    } catch (e) { return resolve({ notFound: true, err: e.message }); }
     proc.stdout.on('data', d => stdout += d.toString());
     proc.stderr.on('data', d => stderr += d.toString());
-
-    proc.on('close', (code) => {
-      const truncated = (s) => s.length > 8000 ? s.slice(0, 8000) + '\n... (kesildi, ' + (s.length - 8000) + ' karakter daha)' : s;
-      resolve({
-        success: code === 0,
-        language,
-        exitCode: code,
-        stdout: truncated(stdout).trim(),
-        stderr: truncated(stderr).trim(),
-        output: truncated(stdout + (stderr ? '\n[STDERR]: ' + stderr : '')).trim(),
-      });
-    });
-    proc.on('error', (e) => {
-      resolve({ success: false, error: e.message, language });
-    });
+    proc.on('error', (e) => resolve({ notFound: /ENOENT/i.test(e.message), err: e.message }));
+    proc.on('close', (exitCode) => resolve({
+      // 9009 (win) / 127 (*nix) veya alias mesaji = yorumlayici yok → sonraki adaya gec
+      notFound: exitCode === 9009 || exitCode === 127 || /not found|not recognized|install from the Microsoft Store/i.test(stderr),
+      exitCode, stdout, stderr,
+    }));
   });
+
+  let r = { notFound: true };
+  for (const bin of candidates) {
+    r = await spawnOnce(bin);
+    if (!r.notFound) break; // yorumlayici bulundu (basarili ya da kod hatasi) — bunu kullan
+  }
+  if (r.notFound) {
+    const nice = language === 'python' ? 'Python bu sistemde kurulu degil.' : `${language} yorumlayicisi bulunamadi.`;
+    return { success: false, language, error: `${nice} (denenen: ${candidates.join(', ')})` };
+  }
+  return {
+    success: r.exitCode === 0,
+    language,
+    exitCode: r.exitCode,
+    stdout: truncated(r.stdout).trim(),
+    stderr: truncated(r.stderr).trim(),
+    output: truncated(r.stdout + (r.stderr ? '\n[STDERR]: ' + r.stderr : '')).trim(),
+  };
 }
 
 module.exports = {

package/src/tools/git.js

@@ -1,4 +1,16 @@
-const { execSync } = require('child_process');
+const { execFileSync } = require('child_process');
+
+// v5.38 GUVENLIK: args'i shell'e sokmadan token'lara ayir. execFileSync shell:false ile
+// calisir; boylece ";", "&&", "|", "$()", backtick gibi metakarakterler ISLEM GORMEZ —
+// komut enjeksiyonu imkansiz. (Eski execSync `git log ${args}` enjeksiyona aciktir.)
+function tokenizeArgs(s) {
+  if (!s) return [];
+  const out = [];
+  const re = /"([^"]*)"|'([^']*)'|(\S+)/g;
+  let m;
+  while ((m = re.exec(String(s))) !== null) out.push(m[1] ?? m[2] ?? m[3]);
+  return out;
+}
 
 module.exports = {
   name: 'git',
@@ -17,27 +29,56 @@ module.exports = {
     required: ['operation']
   },
 
-  execute({ operation, args = '', message = '' }) {
+  execute({ operation, args = '', message = '', command = '', cwd: userCwd = '' }) {
     // v5.6.22: Git repo otomatik bul - this baglami kayboldugu icin module-level helper kullan
-    const cwd = findGitRepo();
+    // v5.38: ajan bir cwd verdiyse ve orada .git varsa onu kullan; yoksa oradan yukari tara.
+    const cwd = findGitRepo(userCwd);
+    // v5.38: Esnek giris — ajanlar farkli formatlarda gonderebilir (operation yerine args/command
+    // icinde "log -n 2" gibi). operation yoksa ilk token'i operation kabul et, gerisi args olur.
+    if (!operation && (command || args)) {
+      const raw = String(command || args).trim().replace(/^git\s+/i, '');
+      const m = raw.match(/^([a-z][a-z-]*)\s*([\s\S]*)$/i);
+      if (m) { operation = m[1].toLowerCase(); if (m[2].trim()) args = m[2].trim(); }
+    }
+    operation = String(operation || '').toLowerCase();
+    const tok = tokenizeArgs(args);
     try {
-      let cmd;
+      let gitArgs;
       switch (operation) {
-        case 'status':  cmd = 'git status --short'; break;
-        case 'diff':    cmd = `git diff ${args || 'HEAD'}`; break;
-        case 'log':     cmd = `git log --oneline ${args || '-10'}`; break;
-        case 'branches': cmd = 'git branch -a'; break;
-        case 'add':     cmd = `git add ${args || '.'}`; break;
-        case 'commit':  cmd = `git commit -m "${message.replace(/"/g, '\\"')}"`; break;
-        default: return { success: false, error: 'Unknown operation' };
+        case 'status':  gitArgs = ['status', '--short']; break;
+        case 'diff':    gitArgs = ['diff', ...(tok.length ? tok : ['HEAD'])]; break;
+        case 'log':     gitArgs = ['log', '--oneline', ...(tok.length ? tok : ['-10'])]; break;
+        case 'branch':
+        case 'branches': gitArgs = ['branch', '-a']; break;
+        case 'add':     gitArgs = ['add', ...(tok.length ? tok : ['.'])]; break;
+        case 'commit':  gitArgs = ['commit', '-m', String(message || '')]; break;
+        // v5.38: yaygin salt-okunur alt-komutlar (ajan bunlari da isteyebiliyor)
+        case 'show':      gitArgs = ['show', ...(tok.length ? tok : ['HEAD'])]; break;
+        case 'remote': {
+          // GUVENLIK: remote yazma islemleri (add/set-url/remove/rename) engelli — push
+          // zeminini ve exec-allowlist bypass'ini kapatir. Sadece salt-okunur.
+          const sub = (tok[0] || '').toLowerCase();
+          if (['add', 'set-url', 'remove', 'rm', 'rename', 'prune', 'set-head', 'set-branches'].includes(sub)) {
+            return { success: false, error: `git remote "${sub}" engellendi (guvenlik: remote yazma yasak)` };
+          }
+          gitArgs = ['remote', ...(tok.length ? tok : ['-v'])]; break;
+        }
+        case 'tag':       gitArgs = ['tag', ...tok]; break;
+        case 'describe':  gitArgs = ['describe', ...(tok.length ? tok : ['--tags', '--always'])]; break;
+        case 'rev-parse': gitArgs = ['rev-parse', ...(tok.length ? tok : ['HEAD'])]; break;
+        default: return { success: false, error: `Bilinmeyen git islemi: "${operation}". Gecerli: status, diff, log, branch, add, commit, show, remote, tag, describe, rev-parse` };
       }
-      const output = execSync(cmd, { cwd, stdio: 'pipe', encoding: 'utf8' });
-      return { success: true, output: output.trim() };
+      // shell:false (execFile default) → args ne olursa olsun shell yorumlamaz.
+      const output = execFileSync('git', gitArgs, { cwd, stdio: 'pipe', encoding: 'utf8' });
+      return { success: true, output: (output || '').trim() };
     } catch (err) {
       return { success: false, error: err.stderr?.toString() || err.message };
     }
   },
 
+  // v5.38: test icin — shell'e gitmeden token'lara ayirma davranisini dogrulamak icin
+  _tokenizeArgs: tokenizeArgs,
+
   /**
    * v5.6.22: Git repo bul - cwd'de yoksa ~/Projects ve parent dizinleri tara
    */
@@ -93,24 +134,30 @@ module.exports = {
 /**
  * Git repo bul - cwd'de yoksa ~/Projects ve parent dizinleri tara
  */
-function findGitRepo() {
+function findGitRepo(startDir = '') {
   const fs = require('fs');
   const path = require('path');
   const os = require('os');
 
-  if (fs.existsSync(path.join(process.cwd(), '.git'))) {
-    return process.cwd();
+  // v5.38: ajanin verdigi cwd oncelikli — orada .git varsa onu, yoksa oradan yukari tara.
+  const seed = startDir && fs.existsSync(startDir) ? startDir : process.cwd();
+  if (fs.existsSync(path.join(seed, '.git'))) {
+    return seed;
   }
 
   const home = os.homedir();
   const candidates = [
+    // v5.38: makineden bagimsiz — env ipuclari once (repo dizini disindan calisilsa bile bulunur)
+    process.env.NATURECO_PROJECT_DIR,
+    process.env.INIT_CWD,
+    process.env.PWD,
     path.join(home, 'Projects', 'natureco-cli'),
     path.join(home, 'Projects'),
     path.join(home, 'projects'),
     path.join(home, 'code'),
     path.join(home, 'dev'),
     path.join(home, 'src'),
-  ];
+  ].filter(Boolean);
 
   for (const dir of candidates) {
     try {
@@ -120,8 +167,8 @@ function findGitRepo() {
     } catch {}
   }
 
-  let current = process.cwd();
-  for (let i = 0; i < 5; i++) {
+  let current = seed;
+  for (let i = 0; i < 6; i++) {
     const parent = path.dirname(current);
     if (parent === current) break;
     try {
@@ -132,5 +179,5 @@ function findGitRepo() {
     current = parent;
   }
 
-  return process.cwd();
+  return seed;
 }