beast-agent 2.30.6 → 2.32.0

package/package.json

@@ -1,7 +1,7 @@
 {
     "name": "beast-agent",
     "productName": "Beast Agent",
-    "version": "2.30.6",
+    "version": "2.32.0",
     "description": "Ultra-fast local agent shell for Windows.",
     "author": "algokodcom (AlgoKod)",
     "license": "MIT",

package/src/agent/fsguard.js

@@ -0,0 +1,46 @@
+'use strict';
+
+/* Dışa gönderim güvenlik kapısı: ajanın send_file aracı sır/anahtar
+   dosyalarını asla WhatsApp/sohbete göndermesin. Saf fonksiyon —
+   electron'a bağımlı değildir, test edilebilir. */
+
+const path = require('path');
+const os = require('os');
+
+function norm(p) {
+  return String(p || '').replace(/\//g, '\\').toLowerCase();
+}
+
+/* true → bu dosya hassas, dışa gönderilmez */
+function isSensitiveSendPath(abs, opts = {}) {
+  const appData = opts.appData || process.env.APPDATA || path.join(os.homedir(), 'AppData', 'Roaming');
+  const home = opts.home || os.homedir();
+  let full;
+  try {
+    full = norm(path.resolve(String(abs || '')));
+  } catch {
+    return false;
+  }
+  if (!full) return false;
+
+  /* Beast veri dizini: settings.json (API key'leri), wa-auth, mcp.json,
+     sessions, botlar, tools — tek komutla sızamaz */
+  const beastDir = norm(path.join(appData, 'beast'));
+  if (full === beastDir || full.startsWith(beastDir + '\\')) return true;
+
+  /* Kimlik/anahtar klasörleri */
+  for (const d of ['.ssh', '.aws', '.gnupg', '.docker', '.kube', '.config']) {
+    const p = norm(path.join(home, d));
+    if (full === p || full.startsWith(p + '\\')) return true;
+  }
+
+  const base = path.basename(full);
+  if (/^(id_rsa|id_ed25519|id_ecdsa|id_dsa|authorized_keys)(\.pub)?$/.test(base)) return true;
+  if (/^\.env(\.|$)/.test(base)) return true;
+  if (/\.(pem|key|pfx|p12|keystore|jks|ppk)$/.test(base)) return true;
+  if (/^(credentials|creds|secrets?)\.(json|ya?ml|txt)$/.test(base)) return true;
+  if (/^(\.npmrc|\.netrc|\.git-credentials|\.htpasswd)$/.test(base)) return true;
+  return false;
+}
+
+module.exports = { isSensitiveSendPath };

package/src/agent/mcp.js

@@ -42,6 +42,23 @@ function configPath() {
 
 let cfgCache = { mtime: -1, cfg: { servers: {} } };
 
+/* Sunucu kaydını normalize eder + token güvenliğini doğrular.
+   Komut yoksa null döner; satır sonu/NUL varsa fırlatır (kayıt atlanır/reddedilir). */
+function normalizeServer(s) {
+  if (!s || typeof s !== 'object' || !s.command) return null;
+  const command = String(s.command);
+  assertSafeToken(command, 'MCP komutu');
+  const args = (Array.isArray(s.args) ? s.args : []).map((a) => assertSafeToken(a, 'MCP argümanı'));
+  return {
+    command,
+    args,
+    env: s.env && typeof s.env === 'object' ? Object.fromEntries(Object.entries(s.env).map(([k, v]) => [String(k), String(v)])) : {},
+    enabled: s.enabled !== false,
+    tools: Array.isArray(s.tools) ? s.tools.map(String) : null,
+    timeoutMs: Number(s.timeoutMs) > 1000 ? Math.min(Number(s.timeoutMs), 600000) : CALL_TIMEOUT_MS,
+  };
+}
+
 function readConfig(force) {
   try {
     const p = configPath();
@@ -51,15 +68,12 @@ function readConfig(force) {
     const cfg = { servers: {} };
     if (parsed && typeof parsed.servers === 'object') {
       for (const [name, s] of Object.entries(parsed.servers)) {
-        if (!s || typeof s !== 'object' || !s.command) continue;
-        cfg.servers[sanitizeName(name)] = {
-          command: String(s.command),
-          args: Array.isArray(s.args) ? s.args.map(String) : [],
-          env: s.env && typeof s.env === 'object' ? Object.fromEntries(Object.entries(s.env).map(([k, v]) => [String(k), String(v)])) : {},
-          enabled: s.enabled !== false,
-          tools: Array.isArray(s.tools) ? s.tools.map(String) : null,
-          timeoutMs: Number(s.timeoutMs) > 1000 ? Math.min(Number(s.timeoutMs), 600000) : CALL_TIMEOUT_MS,
-        };
+        try {
+          const srv = normalizeServer(s);
+          if (srv) cfg.servers[sanitizeName(name)] = srv;
+        } catch (e) {
+          log.warn(`[mcp] '${name}' geçersiz, atlandı: ${String((e && e.message) || e)}`);
+        }
       }
     }
     cfgCache = { mtime: st.mtimeMs, cfg };
@@ -76,9 +90,15 @@ function readConfig(force) {
 
 function saveConfig(cfg) {
   const p = configPath();
+  /* DİSKE YAZMADAN doğrula: güvensiz/yapıbozuk kayıt asla mcp.json'a düşmez */
+  const clean = { servers: {} };
+  for (const [name, s] of Object.entries((cfg && cfg.servers) || {})) {
+    const srv = normalizeServer(s);
+    if (srv) clean.servers[sanitizeName(name)] = srv;
+  }
   fs.mkdirSync(path.dirname(p), { recursive: true });
-  fs.writeFileSync(p, JSON.stringify(cfg, null, 2) + '\n');
-  cfgCache = { mtime: -1, cfg }; /* sonraki okuma diskten gelsin */
+  fs.writeFileSync(p, JSON.stringify(clean, null, 2) + '\n');
+  cfgCache = { mtime: -1, cfg: clean }; /* sonraki okuma diskten gelsin */
 }
 
 function sanitizeName(name) {
@@ -89,20 +109,77 @@ function sanitizeName(name) {
     .slice(0, 32) || 'server';
 }
 
+/* UI'a maskeli ('***') giden env sırlarını diskteki mevcut değerlerle geri koyar.
+   Böylece kullanıcı formu kaydedince sırlar '***' ile ezilmez. */
+function restoreMaskedSecrets(cfg) {
+  let cur;
+  try { cur = readConfig(true); } catch { return cfg; }
+  for (const [name, s] of Object.entries((cfg && cfg.servers) || {})) {
+    if (!s || typeof s !== 'object' || !s.env || typeof s.env !== 'object') continue;
+    const old = cur.servers[sanitizeName(name)];
+    if (!old || !old.env) continue;
+    for (const [k, v] of Object.entries(s.env)) {
+      if (String(v) === '***' && old.env[k] !== undefined) s.env[k] = old.env[k];
+    }
+  }
+  return cfg;
+}
+
 /* ---------- süreç yaşam döngüsü ---------- */
 
+/* Token doğrulama: satır sonu/NUL her platformda enjeksiyon vektörüdür */
+function assertSafeToken(t, what) {
+  const s = String(t == null ? '' : t);
+  if (/[\0\r\n]/.test(s)) throw new Error(what + ' içinde satır sonu/NUL karakteri olamaz');
+  return s;
+}
+
+const CMD_META_RE = /[\s"&|<>^()]/;
+
+/* KOMUT token'ı: yalnız gerekiyorsa tırnaklanır — aksi halde npm/npx gibi
+   .cmd sarmalayıcılarının %~dp0 çözümü bozulur. %/! reddedilir. */
+function quoteCmdCommand(t) {
+  const s = assertSafeToken(t, 'MCP komutu');
+  if (/[%!]/.test(s)) throw new Error('MCP komutunda % veya ! kullanılamaz (güvenlik)');
+  if (!CMD_META_RE.test(s)) return s;
+  return '"' + s.replace(/"/g, '""') + '"';
+}
+
+/* ARGÜMAN token'ı: HER ZAMAN tırnak içinde → & | < > gibi metakarakterler
+   komut ayıramaz. %/! genişletilebildiği için reddedilir; kapanış tırnağı
+   öncesi ters bölü CRT kuralı gereği çiftlenir. */
+function quoteCmdArg(t) {
+  const s = assertSafeToken(t, 'MCP argümanı');
+  if (/[%!]/.test(s)) throw new Error('MCP argümanında % veya ! kullanılamaz (güvenlik)');
+  const out = s.replace(/"/g, '""').replace(/(\\+)$/, '$1$1');
+  return '"' + out + '"';
+}
+
 function spawnServer(cfg) {
-  /* Windows: npx/uvx gibi .cmd sarmalayıcıları düz spawn ile başlamaz —
-     cmd /d /s /c üzerinden, argümanlar elle tırnaklanarak geç. */
-  const quote = (t) => (/[\s"]/u.test(t) ? '"' + t.replace(/"/g, '\\"') + '"' : t);
-  const line = [cfg.command, ...cfg.args].map(quote).join(' ');
-  const child = spawn(line, {
-    shell: true,
+  const cmd = assertSafeToken(cfg.command, 'MCP komutu');
+  if (!cmd.trim()) throw new Error('MCP komutu boş');
+  const args = (Array.isArray(cfg.args) ? cfg.args : []).map((a) => assertSafeToken(a, 'MCP argümanı'));
+  const opts = {
     windowsHide: true,
     stdio: ['pipe', 'pipe', 'pipe'],
     env: { ...process.env, ...(cfg.env || {}) },
+  };
+  if (process.platform !== 'win32') {
+    /* POSIX: shell YOK — & | ; $() gibi metakarakterler enjekte edilemez */
+    return spawn(cmd, args, { ...opts, shell: false });
+  }
+  /* Windows: npx/uvx .cmd sarmalayıcıları düz spawn edilemez — cmd.exe'ye
+     tırnaklı/tırnaksız DOĞRU komut satırı elle kurulur; shell KAPALI,
+     windowsVerbatimArguments ile libuv müdahalesi kapatılır. */
+  const first = quoteCmdCommand(cmd);
+  const line = [first, ...args.map(quoteCmdArg)].join(' ');
+  /* İlk token tırnaklıysa cmd /s ilk ve son tırnağı KIRPAR — dış sarmal şart */
+  const full = first.startsWith('"') ? '"' + line + '"' : line;
+  return spawn(process.env.ComSpec || 'cmd.exe', ['/d', '/s', '/c', full], {
+    ...opts,
+    shell: false,
+    windowsVerbatimArguments: true,
   });
-  return child;
 }
 
 function connFor(name) {
@@ -414,6 +491,9 @@ module.exports = {
   configPath,
   readConfig,
   saveConfig,
+  restoreMaskedSecrets,
+  normalizeServer,
+  _spawnServer: spawnServer,
   mergeTools,
   toolSchemas,
   call,

package/src/agent/tools.js

@@ -4,6 +4,8 @@ const fs = require('fs');
 const os = require('os');
 const path = require('path');
 const https = require('https');
+const dns = require('dns');
+const net = require('net');
 const { execFile } = require('child_process');
 const { spawn } = require('child_process');
 const research = require('./research');
@@ -877,6 +879,111 @@ function assertPublicHttpUrl(raw) {
   return u.toString();
 }
 
+/* Özel/ayrılmış IP aralığı mı? (IPv4 + IPv6 + v4-mapped) */
+function isPrivateIp(ip) {
+  const v = String(ip || '').toLowerCase().replace(/^\[|\]$/g, '');
+  if (v.includes(':')) {
+    if (v === '::1' || v === '::') return true;
+    if (/^f[cd]/.test(v)) return true; /* fc00::/7 unique-local */
+    if (/^fe[89ab]/.test(v)) return true; /* fe80::/10 link-local */
+    if (/^ff/.test(v)) return true; /* multicast */
+    if (v.startsWith('2001:db8')) return true; /* dokümantasyon */
+    const m = v.match(/^::ffff:(\d+\.\d+\.\d+\.\d+)$/); /* v4-mapped */
+    if (m) return isPrivateIp(m[1]);
+    return false;
+  }
+  const p = v.split('.').map(Number);
+  if (p.length !== 4 || p.some((n) => !Number.isInteger(n) || n < 0 || n > 255)) return true; /* çözümlenemeyen → güvensiz */
+  if (p[0] === 0 || p[0] === 10 || p[0] === 127 || p[0] >= 224) return true;
+  if (p[0] === 100 && p[1] >= 64 && p[1] <= 127) return true; /* CGNAT */
+  if (p[0] === 169 && p[1] === 254) return true; /* link-local */
+  if (p[0] === 172 && p[1] >= 16 && p[1] <= 31) return true;
+  if (p[0] === 192 && (p[1] === 168 || (p[1] === 0 && (p[2] === 0 || p[2] === 2)) || (p[1] === 88 && p[2] === 99))) return true;
+  if (p[0] === 198 && (p[1] === 18 || p[1] === 19 || (p[1] === 51 && p[2] === 100))) return true;
+  if (p[0] === 203 && p[1] === 0 && p[2] === 113) return true;
+  return false;
+}
+
+/* DNS çözümlemesi: alan adı ÖZEL bir IP'ye çözülüyorsa (localtest.me vb.) engelle.
+   TOCTOU/DNS-rebinding için tam çözüm connection pinning gerektirir; bu katman
+   metin kontrolü + gerçek çözümleme + redirect doğrulamasıyla açığı kapatır. */
+async function assertPublicHostResolved(hostname) {
+  const h = String(hostname || '').replace(/^\[|\]$/g, '');
+  if (!h) throw new Error('geçersiz adres');
+  if (net.isIP(h)) {
+    if (isPrivateIp(h)) throw new Error('yerel/ağ içi adreslere erişim engellendi');
+    return;
+  }
+  let addrs;
+  try {
+    addrs = await dns.promises.lookup(h, { all: true, verbatim: true });
+  } catch {
+    throw new Error('alan adı çözümlenemedi: ' + h);
+  }
+  if (!addrs || !addrs.length) throw new Error('alan adı çözümlenemedi: ' + h);
+  for (const a of addrs) {
+    if (isPrivateIp(a.address)) throw new Error('yerel/ağ içi adreslere erişim engellendi (' + h + ')');
+  }
+}
+
+/* http_fetch için: yönlendirmeleri ELLE takip eder ve HER adımı doğrular
+   (302 → 127.0.0.1 gibi bypass'ları kapatır). Döner: { res, url } */
+async function fetchPublicFollow(rawUrl, opts = {}, timeoutMs = 30000, signal) {
+  let current = assertPublicHttpUrl(rawUrl);
+  for (let hop = 0; hop <= 5; hop++) {
+    const u = new URL(current);
+    await assertPublicHostResolved(u.hostname);
+    const ctrl = new AbortController();
+    const timer = setTimeout(() => ctrl.abort(new Error('zaman aşımı')), timeoutMs);
+    const onAbort = () => ctrl.abort(new Error('iptal'));
+    if (signal) {
+      if (signal.aborted) throw new Error('iptal');
+      signal.addEventListener('abort', onAbort, { once: true });
+    }
+    let res;
+    try {
+      res = await fetch(current, { ...opts, signal: ctrl.signal, redirect: 'manual' });
+    } finally {
+      clearTimeout(timer);
+      if (signal) signal.removeEventListener('abort', onAbort);
+    }
+    const loc = res.headers.get('location');
+    if (res.status >= 300 && res.status < 400 && loc) {
+      if (hop === 5) throw new Error('çok fazla yönlendirme');
+      current = assertPublicHttpUrl(new URL(loc, current).toString());
+      continue;
+    }
+    return { res, url: current };
+  }
+  throw new Error('çok fazla yönlendirme');
+}
+
+/* Gövdeyi SINIRLI oku — kötü niyetli sunucu sınırsız veri akıtıp belleği şişirmesin */
+async function readTextCapped(res, byteCap = 1024 * 1024) {
+  if (!res.body || typeof res.body.getReader !== 'function') {
+    const t = await res.text();
+    return t.length > byteCap ? t.slice(0, byteCap) : t;
+  }
+  const reader = res.body.getReader();
+  const chunks = [];
+  let size = 0;
+  try {
+    while (true) {
+      const { done, value } = await reader.read();
+      if (done) break;
+      if (!value || !value.length) continue;
+      const room = byteCap - size;
+      if (room <= 0) break;
+      chunks.push(value.length > room ? value.slice(0, room) : value);
+      size += Math.min(value.length, room);
+      if (size >= byteCap) break;
+    }
+  } finally {
+    try { await reader.cancel(); } catch {}
+  }
+  return Buffer.concat(chunks.map((c) => Buffer.from(c))).toString('utf8');
+}
+
 const ENTITIES = {
   amp: '&', lt: '<', gt: '>', quot: '"', apos: "'", nbsp: ' ', mdash: '—', ndash: '–',
   hellip: '…', rsquo: '\u2019', lsquo: '\u2018', ldquo: '\u201c', rdquo: '\u201d',
@@ -1004,10 +1111,10 @@ async function webSearch(query, { maxResults = 8, signal } = {}) {
 const MAX_FETCH_CHARS = 9000;
 
 async function httpFetch(url, { maxChars = MAX_FETCH_CHARS, format = 'text', timeoutMs = 30000, signal } = {}) {
-  const safe = assertPublicHttpUrl(url);
   const t = Math.min(Math.max(Number(timeoutMs) || 30000, 1000), 120000); // opencode: default 30s, max 120s
-  const res = await fetchWithTimeout(
-    safe,
+  /* SSRF: metin + DNS çözümlemesi + her redirect adımı ayrı doğrulanır */
+  const { res, url: safe } = await fetchPublicFollow(
+    url,
     { headers: { 'User-Agent': UA, Accept: 'text/html,text/plain,application/json;q=0.9,*/*;q=0.5' } },
     t,
     signal
@@ -1020,7 +1127,7 @@ async function httpFetch(url, { maxChars = MAX_FETCH_CHARS, format = 'text', tim
     return { ok: true, url: safe, status: res.status, contentType: ctype, note: 'ikili/metin olmayan içerik indirilmedi' };
   }
   const cap = 400000;
-  let body = await res.text();
+  let body = await readTextCapped(res, 1024 * 1024);
   if (body.length > cap) body = body.slice(0, cap);
   let content;
   if (/html/i.test(ctype)) {
@@ -2357,6 +2464,9 @@ module.exports = {
   globToRegExp,
   walkFiles,
   assertPublicHttpUrl,
+  isPrivateIp,
+  assertPublicHostResolved,
+  fetchPublicFollow,
   parseDdgResults,
   htmlToText,
   webSearch,