micro-models-agent 2.11.0 → 2.12.0

package/dist/tools/browser.js

@@ -1,36 +1,72 @@
 import { BrowserSession } from "../modules/browser/session";
 import { DEFAULT_BROWSER_CONFIG } from "../modules/browser/types";
+import { bufferToDataUrl } from "../llm/image-utils";
 import { t } from "../i18n/index";
+import { defaultOutputBus } from "../output";
+import { errMsg } from "../utils";
 import { join } from "path";
+import { mkdirSync } from "fs";
+import { projectMmaDir } from "../config/paths";
 let session = null;
+/**
+ * Тестовый seam: подмена фабрики сессии без `vi.mock` (в Bun он глобален на
+ * процесс). Используется только в тестах сериализации действий.
+ */
+let sessionFactory;
+/** @internal */
+export function setBrowserSessionFactoryForTest(factory) {
+    sessionFactory = factory;
+    session = null;
+}
+/**
+ * Сериализация действий. В MoE параллельные волны делят module-level сессию:
+ * одновременный `execute()` перетирает состояние страницы, а `close()` одного
+ * субагента убивает сессию остальных. Promise-chain гарантирует, что в браузере
+ * одновременно выполняется ровно одно действие.
+ */
+let actionChain = Promise.resolve();
+/** Сериализует действия: одновременно в браузере ровно одно. */
+function serialize(fn) {
+    const run = actionChain.then(fn, fn);
+    actionChain = run.then(() => undefined, () => undefined);
+    return run;
+}
+/** Ленивая module-level сессия (cookie-dir в project .mma). */
 function getSession(ctx) {
     if (!session) {
-        const cookieDir = join(ctx.baseDir, ".mma", "browser");
-        session = new BrowserSession({
-            ...DEFAULT_BROWSER_CONFIG,
-            headless: ctx.config.browser?.headless ?? true,
-            maxElements: ctx.config.browser?.maxElements ?? 30,
-            maxContentChars: ctx.config.browser?.maxContentChars ?? 2500,
-            maxConsoleEntries: ctx.config.browser?.maxConsoleEntries ?? 40,
-            maxConsoleLineChars: ctx.config.browser?.maxConsoleLineChars ?? 400,
-            navigationTimeout: ctx.config.browser?.navigationTimeout ?? 15000,
-            viewportWidth: ctx.config.browser?.viewportWidth ?? 1280,
-            viewportHeight: ctx.config.browser?.viewportHeight ?? 720,
-            cookieDir,
-        });
+        const cookieDir = join(projectMmaDir(ctx.baseDir), "browser");
+        mkdirSync(cookieDir, { recursive: true });
+        session = sessionFactory
+            ? sessionFactory(ctx)
+            : new BrowserSession({
+                ...DEFAULT_BROWSER_CONFIG,
+                headless: ctx.config.browser?.headless ?? true,
+                maxElements: ctx.config.browser?.maxElements ?? 30,
+                maxContentChars: ctx.config.browser?.maxContentChars ?? 2500,
+                maxConsoleEntries: ctx.config.browser?.maxConsoleEntries ?? 40,
+                maxConsoleLineChars: ctx.config.browser?.maxConsoleLineChars ?? 400,
+                navigationTimeout: ctx.config.browser?.navigationTimeout ?? 15000,
+                viewportWidth: ctx.config.browser?.viewportWidth ?? 1280,
+                viewportHeight: ctx.config.browser?.viewportHeight ?? 720,
+                cookieDir,
+            });
     }
     return session;
 }
+/** Скриншот для текстовой модели: пометка + текстовый снапшот. */
 export function formatScreenshotForTextModel(snapshot) {
     return `${t("tool.screenshot_unavailable")}\n\n${snapshot}`;
 }
-export function formatScreenshotResult(snapshot, screenshot, supportsVision) {
-    if (!supportsVision) {
-        return formatScreenshotForTextModel(snapshot);
-    }
-    const base64 = screenshot.toString("base64");
-    return `${snapshot}\n\n[Screenshot: data:image/png;base64,${base64}]`;
+/**
+ * Превращает PNG-буфер скриншота в image-part для доставки модели. Размер
+ * ограничивается `bufferToDataUrl` (ресайз ≤800px, JPEG) — иначе base64
+ * раздул бы контекст малыша (32K).
+ */
+export async function screenshotToImagePart(screenshot) {
+    const { dataUrl } = await bufferToDataUrl(screenshot);
+    return { type: "image_url", image_url: { url: dataUrl } };
 }
+/** Фабрика browser-тула (сессия ленивая, module-level). */
 export function createBrowserTool() {
     return {
         name: "browser",
@@ -93,23 +129,32 @@ export function createBrowserTool() {
                 return { success: false, output: t("tool.action_required") };
             }
             const s = getSession(ctx);
-            const result = await s.execute(action, args);
-            const supportsVision = ctx.config.model.includes("vision") ||
-                ctx.config.model.includes("gpt-4o") ||
-                ctx.config.model.includes("claude");
+            const result = await serialize(() => s.execute(action, args));
             if (result.screenshot) {
-                return {
-                    success: result.success,
-                    output: formatScreenshotResult(result.output, result.screenshot, supportsVision),
-                };
+                // vision берётся из резолвера возможностей активной модели
+                // (пресет + native-probe + конфиг), а не из эвристики по имени.
+                // Без vision отдаём текстовый snapshot — base64 в контекст не льём.
+                if (ctx.capabilities?.vision) {
+                    return {
+                        success: result.success,
+                        output: result.output,
+                        images: [await screenshotToImagePart(result.screenshot)],
+                    };
+                }
+                return { success: result.success, output: formatScreenshotForTextModel(result.output) };
             }
             return { success: result.success, output: result.output };
         },
     };
 }
+/** Закрывает сессию после текущего действия (не посреди навигации). */
 export function closeBrowserSession() {
-    if (session) {
-        session.close().catch(() => { });
-        session = null;
-    }
+    const s = session;
+    session = null;
+    if (!s)
+        return;
+    // Дожидаемся текущего действия, чтобы не закрыть браузер во время навигации.
+    actionChain = actionChain.then(() => s.close().catch((err) => {
+        defaultOutputBus.log("warn", "browser", `Failed to close browser session: ${errMsg(err)}`);
+    }));
 }

package/dist/tools/chunk-query.js

@@ -1,15 +1,17 @@
+/** Тул ответа по большому тексту: чанкинг + параллельные запросы + синтез. */
 import { readFileSync } from "node:fs";
 import { resolve } from "node:path";
 import { t } from "../i18n/index";
 import { isPathInScope } from "../modules/security/path-validator";
 import { DEFAULT_SECURITY_CONFIG } from "../config/security";
 import { runChunkQuery } from "../modules/context/chunk-query";
+/** Отвечает на вопрос по тексту вне контекстного окна (чанк-параллелизм). */
 export const chunkQueryTool = {
     name: "chunk_query",
     icon: "🧩",
     description: "Answer a query over a large text (file or inline) by splitting it into chunks and querying the model per chunk in parallel, then optionally synthesizing a final answer. Use when the input is too large for the context window.",
     tags: ["research"],
-    /** Multiple LLM calls (parallel chunk queries + synthesis) need a longer budget. */
+    /** Несколько LLM-вызовов (чанк-запросы + синтез) — нужен больший бюджет. */
     timeoutMs: 300000,
     parameters: {
         type: "object",
@@ -64,22 +66,28 @@ export const chunkQueryTool = {
             return { success: false, output: t("tool.chunk_query_no_input") };
         let content = text || "";
         if (inputPath) {
-            // Same policy as read_file: global security paths (denied patterns
-            // like .env/keys) + subagent scope.
+            // Та же политика, что у read_file: глобальные security-пути (.env/ключи)
+            // + скоуп субагента.
             const resolvedInput = resolve(ctx.baseDir, inputPath);
             const check = isPathInScope(ctx.baseDir, resolvedInput, ctx.scope, ctx.config?.security?.paths || DEFAULT_SECURITY_CONFIG.paths);
             if (!check.allowed) {
-                return { success: false, output: `[SCOPE] ${check.reason || "Path not allowed"}` };
+                return {
+                    success: false,
+                    output: t("tool.chunk_query_path_not_allowed", { reason: check.reason || "Path not allowed" }),
+                };
             }
             try {
                 content = readFileSync(resolvedInput, "utf8");
             }
             catch (e) {
-                return { success: false, output: `Cannot read ${inputPath}: ${e.message}` };
+                return {
+                    success: false,
+                    output: t("tool.chunk_query_read_failed", { path: inputPath, error: e.message }),
+                };
             }
         }
         if (!ctx.llmProvider) {
-            return { success: false, output: "chunk_query: missing llmProvider in context" };
+            return { success: false, output: t("tool.chunk_query_no_provider") };
         }
         const maxParallel = typeof args.max_parallel === "number"
             ? args.max_parallel

package/dist/tools/create-dir.js

@@ -1,9 +1,12 @@
+/** Тул создания каталога (рекурсивно, с security-проверками). */
 import { mkdirSync, existsSync } from "fs";
 import { t } from "../i18n/index";
 import { isPathWritable } from "../modules/security/path-validator";
 import { logFileWrite, logSecurityBlock } from "../modules/security/audit-log";
 import { getSessionSecurityConfig } from "../modules/security/session-isolation";
 import { safeResolvePath } from "./path-utils";
+import { protectedPathBlock } from "../modules/security/protected-paths";
+/** Создаёт каталог со всеми промежуточными. */
 export const createDirTool = {
     name: "create_dir",
     icon: "📁",
@@ -19,11 +22,10 @@ export const createDirTool = {
     handler: async (ctx, args) => {
         const path = String(args.path);
         const resolved = safeResolvePath(ctx.baseDir, path);
-        // Get session-specific security config
         const securityConfig = ctx.sessionContext
             ? getSessionSecurityConfig(ctx.config, ctx.sessionContext)
             : ctx.config.security;
-        // Check file operations limit
+        // Лимит файловых операций.
         const maxFileOps = securityConfig?.maxFileOperations ?? 100;
         const currentCount = ctx.fileOperationsCount ?? 0;
         if (currentCount >= maxFileOps) {
@@ -33,7 +35,7 @@ export const createDirTool = {
                 output: `[SECURITY BLOCKED] Maximum file operations (${maxFileOps}) exceeded`,
             };
         }
-        // Check path permissions
+        // Проверка прав на путь.
         const scopeCheck = isPathWritable(ctx.baseDir, resolved, ctx.scope, ctx.config.security?.paths);
         if (!scopeCheck.allowed) {
             logSecurityBlock(ctx.sessionId, "file_write", scopeCheck.reason || "Path not allowed", path);
@@ -44,12 +46,14 @@ export const createDirTool = {
                 }),
             };
         }
+        // Защищённые пути (.git/, .env-каталоги, …) — спросить юзера.
+        const protectedBlock = protectedPathBlock(ctx, resolved, path);
+        if (protectedBlock)
+            return protectedBlock;
         if (!existsSync(resolved)) {
             mkdirSync(resolved, { recursive: true });
         }
-        // Increment file operations counter
         ctx.fileOperationsCount = currentCount + 1;
-        // Log directory creation
         logFileWrite(ctx.sessionId, path, true, "Directory created");
         return { success: true, output: t("file.created", { path }) };
     },

package/dist/tools/delete-file.js

@@ -1,3 +1,4 @@
+/** Тул удаления файла (с diff удалённого, security-проверки). */
 import { unlinkSync, existsSync, statSync, readFileSync } from "fs";
 import { t } from "../i18n/index";
 import { isPathWritable } from "../modules/security/path-validator";
@@ -5,6 +6,8 @@ import { logFileDelete, logSecurityBlock } from "../modules/security/audit-log";
 import { getSessionSecurityConfig } from "../modules/security/session-isolation";
 import { generateDeleteDiff } from "../ui/diff";
 import { safeResolvePath } from "./path-utils";
+import { protectedPathBlock } from "../modules/security/protected-paths";
+/** Удаляет файл; каталоги отклоняет. Возвращает diff удалённого. */
 export const deleteFileTool = {
     name: "delete_file",
     icon: "🗑️",
@@ -20,11 +23,10 @@ export const deleteFileTool = {
     handler: async (ctx, args) => {
         const path = String(args.path);
         const resolved = safeResolvePath(ctx.baseDir, path);
-        // Get session-specific security config
         const securityConfig = ctx.sessionContext
             ? getSessionSecurityConfig(ctx.config, ctx.sessionContext)
             : ctx.config.security;
-        // Check file operations limit
+        // Лимит файловых операций.
         const maxFileOps = securityConfig?.maxFileOperations ?? 100;
         const currentCount = ctx.fileOperationsCount ?? 0;
         if (currentCount >= maxFileOps) {
@@ -34,7 +36,7 @@ export const deleteFileTool = {
                 output: `[SECURITY BLOCKED] Maximum file operations (${maxFileOps}) exceeded`,
             };
         }
-        // Check path permissions
+        // Проверка прав на путь.
         const scopeCheck = isPathWritable(ctx.baseDir, resolved, ctx.scope, ctx.config.security?.paths);
         if (!scopeCheck.allowed) {
             logSecurityBlock(ctx.sessionId, "file_delete", scopeCheck.reason || "Path not allowed", path);
@@ -45,6 +47,10 @@ export const deleteFileTool = {
                 }),
             };
         }
+        // Защищённые пути (.git/, .env, ключи) — спросить юзера.
+        const protectedBlock = protectedPathBlock(ctx, resolved, path, "file_delete");
+        if (protectedBlock)
+            return protectedBlock;
         if (!existsSync(resolved)) {
             return { success: false, output: t("file.notfound", { path }) };
         }
@@ -54,9 +60,7 @@ export const deleteFileTool = {
         const content = readFileSync(resolved, "utf-8");
         unlinkSync(resolved);
         const diff = generateDeleteDiff(content);
-        // Increment file operations counter
         ctx.fileOperationsCount = currentCount + 1;
-        // Log successful file deletion
         logFileDelete(ctx.sessionId, path, true);
         return { success: true, output: t("file.deleted", { path }), diff };
     },

package/dist/tools/download-file.js

@@ -1,15 +1,18 @@
+/** Тул скачивания URL в файл (байты, SSRF-защита, cap размера). */
 import { writeFileSync, mkdirSync } from "fs";
-import { dirname } from "path";
+import { dirname, resolve } from "path";
 import { t } from "../i18n/index";
-import { isUrlAllowed, sanitizeUrl } from "../modules/security/network-validator";
+import { BlockedRedirectError, fetchValidated, isUrlAllowed, sanitizeUrl, } from "../modules/security/network-validator";
 import { isPathWritable } from "../modules/security/path-validator";
+import { protectedPathBlock } from "../modules/security/protected-paths";
 import { logNetworkRequest, logFileWrite, logSecurityBlock } from "../modules/security/audit-log";
 import { getSessionSecurityConfig } from "../modules/security/session-isolation";
 import { DEFAULT_SECURITY_CONFIG } from "../config/security";
 import { safeResolvePath } from "./path-utils";
-/** Default cap for a single download (bytes). Chunked responses without a
- * content-length header are checked after the body is read. */
+/** Дефолтный cap скачивания (байты). Чанковые ответы без content-length
+ * проверяются после чтения тела. */
 export const MAX_DOWNLOAD_BYTES = 100 * 1024 * 1024;
+/** Качает бинарник в файл; возвращает путь/размер/тип. Параметры: url, path, max_bytes. */
 export const downloadFileTool = {
     name: "download_file",
     icon: "⬇️",
@@ -36,7 +39,7 @@ export const downloadFileTool = {
         if (!url || !path) {
             return { success: false, output: t("tool.invalid_params") };
         }
-        // Network security: same rules as web_fetch/web_browse.
+        // Сеть: те же правила, что у web_fetch/web_browse.
         const securityConfig = ctx.sessionContext
             ? getSessionSecurityConfig(ctx.config, ctx.sessionContext).network
             : ctx.config.security?.network || DEFAULT_SECURITY_CONFIG.network;
@@ -48,7 +51,7 @@ export const downloadFileTool = {
                 output: `[SECURITY BLOCKED] URL is not allowed: ${validation.reason}`,
             };
         }
-        // Path safety: resolve against baseDir and verify writability.
+        // Путь: резолв от baseDir + проверка записи.
         const resolved = safeResolvePath(ctx.baseDir, path);
         const scopeCheck = isPathWritable(ctx.baseDir, resolved, ctx.scope, ctx.config.security?.paths);
         if (!scopeCheck.allowed) {
@@ -60,9 +63,27 @@ export const downloadFileTool = {
                 }),
             };
         }
+        // Тот же guard записи, что у write/edit: онбординг-allowlist и always-on
+        // проверка защищённых путей (иначе скачиванием перезаписали бы .env/ключи,
+        // которые остальные тулы записи отклоняют).
+        if (ctx.writePathAllowlist) {
+            const ok = ctx.writePathAllowlist.some((p) => resolve(p).toLowerCase() === resolve(resolved).toLowerCase());
+            if (!ok) {
+                return {
+                    success: false,
+                    output: t("file.path_not_allowed", {
+                        path: `${path} — ${t("onboarding.write_restricted")}`,
+                    }),
+                };
+            }
+        }
+        const protectedBlock = protectedPathBlock(ctx, resolved, path);
+        if (protectedBlock)
+            return protectedBlock;
         const maxBytes = Number(args.max_bytes) > 0 ? Number(args.max_bytes) : MAX_DOWNLOAD_BYTES;
         try {
-            const response = await fetch(url, {
+            // Редиректы перевадидируются (SSRF) внутри хелпера.
+            const { response } = await fetchValidated(url, securityConfig, {
                 signal: AbortSignal.timeout(securityConfig?.requestTimeout || 15000),
             });
             if (!response.ok) {
@@ -107,6 +128,10 @@ export const downloadFileTool = {
             };
         }
         catch (e) {
+            if (e instanceof BlockedRedirectError) {
+                logSecurityBlock(ctx.sessionId, "network_request", e.reason, sanitizeUrl(url));
+                return { success: false, output: `[SECURITY BLOCKED] redirect: ${e.reason}` };
+            }
             logNetworkRequest(ctx.sessionId, sanitizeUrl(url), false, `Error: ${e.message}`);
             return {
                 success: false,

package/dist/tools/edit-file.js

@@ -1,4 +1,6 @@
+/** Тул точечной правки: exact-match замена, скан, валидация, diff. */
 import { readFileSync, writeFileSync } from "fs";
+import { resolve } from "path";
 import { t } from "../i18n/index";
 import { isPathWritable } from "../modules/security/path-validator";
 import { scanContent } from "../modules/security/content-scanner";
@@ -7,6 +9,9 @@ import { getSessionSecurityConfig } from "../modules/security/session-isolation"
 import { generateDiff } from "../ui/diff";
 import { safeResolvePath } from "./path-utils";
 import { preValidateSyntax, detectImportConflicts } from "./syntax-validator";
+import { errMsg } from "../utils";
+import { protectedPathBlock } from "../modules/security/protected-paths";
+/** Заменяет точный фрагмент в существующем файле (не regex). */
 export const editFileTool = {
     name: "edit_file",
     icon: "✏️",
@@ -24,11 +29,23 @@ export const editFileTool = {
     handler: async (ctx, args) => {
         const path = String(args.path);
         const resolved = safeResolvePath(ctx.baseDir, path);
-        // Get session-specific security config
+        if (ctx.writePathAllowlist) {
+            const ok = ctx.writePathAllowlist.some((p) => resolve(p).toLowerCase() === resolve(resolved).toLowerCase());
+            if (!ok) {
+                return {
+                    success: false,
+                    output: t("file.path_not_allowed", {
+                        path: `${path} — ${t("onboarding.write_restricted")}`,
+                        reason: t("onboarding.write_restricted"),
+                    }),
+                };
+            }
+        }
+        // Сессионный security-конфиг.
         const securityConfig = ctx.sessionContext
             ? getSessionSecurityConfig(ctx.config, ctx.sessionContext)
             : ctx.config.security;
-        // Check file operations limit
+        // Лимит файловых операций.
         const maxFileOps = securityConfig?.maxFileOperations ?? 100;
         const currentCount = ctx.fileOperationsCount ?? 0;
         if (currentCount >= maxFileOps) {
@@ -38,7 +55,7 @@ export const editFileTool = {
                 output: `[SECURITY BLOCKED] Maximum file operations (${maxFileOps}) exceeded`,
             };
         }
-        // Check path permissions
+        // Проверка прав на путь.
         const scopeCheck = isPathWritable(ctx.baseDir, resolved, ctx.scope, ctx.config.security?.paths);
         if (!scopeCheck.allowed) {
             logSecurityBlock(ctx.sessionId, "file_write", scopeCheck.reason || "Path not allowed", path);
@@ -49,19 +66,43 @@ export const editFileTool = {
                 }),
             };
         }
-        const content = readFileSync(resolved, "utf-8");
-        const oldStr = String(args.old);
-        const newStr = String(args.new);
+        // Защищённые пути (.git/, .env, ключи) — всегда спросить юзера.
+        const protectedBlock = protectedPathBlock(ctx, resolved, path);
+        if (protectedBlock)
+            return protectedBlock;
+        // Строковые аргументы обязательны: String(null) дало бы "null" и молча
+        // испортило файл. Явно отклоняем не-строки с понятной ошибкой.
+        if (typeof args.old !== "string" || typeof args.new !== "string") {
+            return {
+                success: false,
+                output: t("file.edit_args_invalid", {
+                    oldType: typeof args.old,
+                    newType: typeof args.new,
+                }),
+            };
+        }
+        const oldStr = args.old;
+        const newStr = args.new;
+        let content;
+        try {
+            content = readFileSync(resolved, "utf-8");
+        }
+        catch (err) {
+            return {
+                success: false,
+                output: t("file.read_failed", { path, error: errMsg(err) }),
+            };
+        }
         if (!content.includes(oldStr)) {
             return {
                 success: false,
                 output: t("file.string_not_found", { str: oldStr, path }),
             };
         }
-        // Replacement via function — otherwise newStr is treated as a template
-        // and `$&`, `$1` etc. would silently corrupt the file content.
+        // Замена через функцию — иначе newStr трактуется как шаблон и `$&`, `$1`
+        // молча испортят содержимое файла.
         const updated = content.replace(oldStr, () => newStr);
-        // Check new content for dangerous patterns — only when security is enabled
+        // Скан нового контента — только при включённой безопасности.
         if (securityConfig?.enabled && securityConfig?.contentScan?.enabled) {
             const scanResult = scanContent(updated, path, securityConfig.contentScan);
             if (!scanResult.allowed) {
@@ -72,7 +113,7 @@ export const editFileTool = {
                 };
             }
         }
-        // Pre-validate syntax BEFORE writing — prevents "file written" + syntax error loop
+        // Превалидация синтаксиса ДО записи — иначе петля "файл записан" + syntax error.
         const validation = await preValidateSyntax(resolved, updated, ctx.baseDir);
         if (!validation.valid) {
             return {
@@ -80,15 +121,12 @@ export const editFileTool = {
                 output: t("file.syntax_error", { path, error: validation.error || "Unknown syntax error" }),
             };
         }
-        // Detect import/export conflicts — warning only (code may be valid)
+        // Конфликты import/export — только warning.
         const conflicts = detectImportConflicts(updated);
         writeFileSync(resolved, updated, "utf-8");
         const diff = generateDiff(content, updated);
-        // Increment file operations counter
         ctx.fileOperationsCount = currentCount + 1;
-        // Log successful file edit
         logFileWrite(ctx.sessionId, path, true, "File edited");
-        // Add conflict warning to output if any
         let output = t("file.replaced_in", { path });
         if (conflicts.length > 0) {
             output += `\n[WARNING] Import/export conflicts: ${conflicts.join(", ")}`;

package/dist/tools/enable-tools.js

@@ -1,11 +1,11 @@
+/** On-demand тул: включает скрытые категории тулов на сессию. */
 import { t } from "../i18n/index";
 /**
- * On-demand tool activation. The main loop exposes only `config.tools.defaultTags`
- * (+ `alwaysOn` structural tools). Specialized tools (web, browser, memory,
- * process) are hidden from the schema to shrink the token footprint; the model
- * activates them by calling this tool with the desired tag. The handler pushes
- * the tags into the shared mutable `ctx.activeToolTags` array, which the agent
- * loop re-reads each iteration to recompute the LLM-visible tool set.
+ * On-demand активация тулов. Главный цикл показывает только
+ * `config.tools.defaultTags` (+ структурные `alwaysOn`); спецтулы (web,
+ * browser, memory, process) скрыты из схемы ради экономии токенов. Модель
+ * включает их этим тулом; хендлер дописывает теги в общий мутабельный
+ * `ctx.activeToolTags`, который цикл перечитывает каждую итерацию.
  */
 export const enableToolsTool = {
     name: "enable_tools",