micro-models-agent 0.41.0 → 0.41.2

package/dist/tools/subagent.js

@@ -5,6 +5,20 @@ import { HallucinationDetector } from "../modules/hallucination/detector";
 import { PluginManager } from "../modules/plugins/manager";
 import { logSecurityBlock } from "../modules/security/audit-log";
 import { getSessionSecurityConfig } from "../modules/security/session-isolation";
+import { ArtifactStore } from "../modules/artifacts/store";
+/** Fixed system prompt (KV-cache friendly when stableSystemPrompt is on). */
+export const SUBAGENT_SYSTEM_PROMPT_TEMPLATE = `You are a sub-agent working on a specific task. You have your own context window and tool set.
+
+Workflow rules:
+- Prefer small, focused steps. Verify your own work (typecheck/tests) before finishing.
+- For large outputs, write the result to a file with write_file instead of reproducing it in your answer.
+- If the task is big and decomposable, delegate parts to the subagent tool instead of doing everything in one context.
+- When you finish, return a concise final answer; do not repeat file contents verbatim if you already wrote them to disk.`;
+function buildSubagentSystemPrompt(context, stable) {
+    if (stable)
+        return SUBAGENT_SYSTEM_PROMPT_TEMPLATE;
+    return `You are a sub-agent working on a specific task. ${context ? `Context: ${context}` : ""}`;
+}
 export const subagentTool = {
     name: "subagent",
     description: "Spawn an isolated sub-agent to work on a task independently. The sub-agent has its own context and executes autonomously. Use for parallel work or complex sub-tasks.",
@@ -47,6 +61,15 @@ export const subagentTool = {
                 items: { type: "string" },
                 description: "Tool tags to expose to the sub-agent",
             },
+            result_mode: {
+                type: "string",
+                enum: ["text", "file"],
+                description: '"file": persist the full result to an artifact and return a path + summary (pass-by-reference). "text" (default): return the full text into context.',
+            },
+            artifact_name: {
+                type: "string",
+                description: "Artifact file name (sanitized). Default: subagent-<timestamp>.",
+            },
         },
         required: ["task"],
     },
@@ -86,8 +109,10 @@ export const subagentTool = {
         // Defensive guard: if the caller requested specific tool tags but none of the
         // registered tools match, the sub-agent would have zero tools and would
         // hallucinate instead of acting. Warn the caller so it can correct the tags.
+        // Uses getByTags (not getAllForLLM) because alwaysOn tools always match the
+        // latter — the guard must test real tag coverage, not schema presence.
         if (toolTags && toolTags.length > 0) {
-            const matched = ctx.toolExecutor.getToolDefinitions(toolTags);
+            const matched = ctx.toolExecutor.getRegistry().getByTags(toolTags);
             if (matched.length === 0) {
                 return {
                     success: false,
@@ -104,10 +129,10 @@ export const subagentTool = {
             const parentScope = ctx.scope;
             ctx.toolExecutor.setScope(scope || { allowed_files: [], read_only_files: [] });
             const systemPrompt = {
-                content: `You are a sub-agent working on a specific task. ${context ? `Context: ${context}` : ""}`,
+                content: buildSubagentSystemPrompt(context, ctx.config.subagent?.stableSystemPrompt !== false),
                 priority: "critical",
                 essential: true,
-                estimatedTokens: 100,
+                estimatedTokens: 150,
             };
             const subDeps = {
                 config: ctx.config,
@@ -128,15 +153,35 @@ export const subagentTool = {
             let scopeRestored = false;
             try {
                 const result = await subAgent.run(fullTask);
-                return result.success
-                    ? {
-                        success: true,
-                        output: `Sub-agent completed:\n${result.text}\n\nIterations: ${result.iterationCount}`,
-                    }
-                    : {
+                if (!result.success) {
+                    return {
                         success: false,
                         output: `Sub-agent failed: ${result.error}\nPartial output: ${result.text}`,
                     };
+                }
+                const resultMode = args.result_mode || ctx.config.subagent?.resultMode || "text";
+                if (resultMode === "file") {
+                    const store = new ArtifactStore({
+                        baseDir: ctx.baseDir,
+                        sessionId: ctx.sessionId || "default",
+                        artifactsDir: ctx.config.subagent?.artifactsDir,
+                    });
+                    const abs = store.save(args.artifact_name || `subagent-${Date.now()}`, result.text, "md");
+                    const summary = store.summary(result.text, ctx.config.subagent?.maxSummaryChars ?? 2000);
+                    ctx.logger.debug(`subagent result persisted to artifact ${abs} (${result.text.length} chars, ${result.iterationCount} iterations)`);
+                    return {
+                        success: true,
+                        output: t("tool.subagent_artifact", {
+                            path: abs,
+                            iterations: String(result.iterationCount),
+                            summary,
+                        }),
+                    };
+                }
+                return {
+                    success: true,
+                    output: `Sub-agent completed:\n${result.text}\n\nIterations: ${result.iterationCount}`,
+                };
             }
             finally {
                 if (!scopeRestored) {

package/dist/tools/user-input.js

@@ -82,7 +82,7 @@ export async function askChoice(question, options, opts = {}) {
             const answer = await promptLine(rl, choicePrompt(entryCount, multiple));
             const parsed = parseSelection(answer, entryCount, multiple);
             if (parsed) {
-                return parsed.map((i) => opts.allowCustom && i === customEntry ? CUSTOM_INDEX : i);
+                return parsed.map((i) => (opts.allowCustom && i === customEntry ? CUSTOM_INDEX : i));
             }
             console.log(t("tool.user_input.invalid"));
         }

package/dist/tools/web-browse.js

@@ -1,12 +1,12 @@
 import { t } from "../i18n/index";
-import { isUrlAllowed, sanitizeUrl, } from "../modules/security/network-validator";
-import { logNetworkRequest, logSecurityBlock, } from "../modules/security/audit-log";
+import { isUrlAllowed, sanitizeUrl } from "../modules/security/network-validator";
+import { logNetworkRequest, logSecurityBlock } from "../modules/security/audit-log";
 import { getSessionSecurityConfig } from "../modules/security/session-isolation";
 import { MAX_PREVIEW_LINES } from "./preview";
 const MAX_CHARS = 3000;
 export const webBrowseTool = {
     name: "web_browse",
-    description: `Fetch and read a web page. Returns the page content as plain text (max ${MAX_PREVIEW_LINES} lines / ${MAX_CHARS} chars).`,
+    description: `Fetch and read a web page. Returns the page content as plain text (max ${MAX_PREVIEW_LINES} lines / ${MAX_CHARS} chars). This is a plain HTTP request — it does NOT execute JavaScript, so JS-rendered single-page apps (weather sites, dashboards, modern SPAs) return empty or near-empty content. For such pages use the "browser" tool instead.`,
     tags: ["research"],
     parameters: {
         type: "object",

package/dist/tools/web-fetch.js

@@ -1,6 +1,6 @@
 import { t } from "../i18n/index";
-import { isUrlAllowed, sanitizeUrl, } from "../modules/security/network-validator";
-import { logNetworkRequest, logSecurityBlock, } from "../modules/security/audit-log";
+import { isUrlAllowed, sanitizeUrl } from "../modules/security/network-validator";
+import { logNetworkRequest, logSecurityBlock } from "../modules/security/audit-log";
 import { getSessionSecurityConfig } from "../modules/security/session-isolation";
 import { DEFAULT_SECURITY_CONFIG } from "../config/security";
 import { MAX_PREVIEW_LINES } from "./preview";
@@ -16,7 +16,7 @@ function stripHtml(html) {
 }
 export const webFetchTool = {
     name: "web_fetch",
-    description: "Fetch a URL and convert its content to markdown. Use for reading documentation, APIs, web pages.",
+    description: 'Fetch a URL and convert its content to markdown. Use for reading documentation, APIs, web pages. This is a plain HTTP request without a browser engine — it does NOT run JavaScript, so JS-rendered single-page apps (weather sites, dashboards, modern SPAs) return empty or near-empty content; use the "browser" tool for those.',
     tags: ["research"],
     parameters: {
         type: "object",

package/dist/tools/web-search.js

@@ -1,10 +1,10 @@
 import { t } from "../i18n/index";
-import { isUrlAllowed, sanitizeUrl, } from "../modules/security/network-validator";
-import { logNetworkRequest, logSecurityBlock, } from "../modules/security/audit-log";
+import { isUrlAllowed, sanitizeUrl } from "../modules/security/network-validator";
+import { logNetworkRequest, logSecurityBlock } from "../modules/security/audit-log";
 import { getSessionSecurityConfig } from "../modules/security/session-isolation";
 export const webSearchTool = {
     name: "web_search",
-    description: "Search the web for information. Returns search results with titles and snippets.",
+    description: "Search the web for information. Returns search results with titles and snippets. Use this for search queries instead of opening search engines (Google/Yandex/Bing) in the browser — they block headless browsers.",
     tags: ["research"],
     parameters: {
         type: "object",

package/dist/tools/write-file.js

@@ -71,9 +71,7 @@ export const writeFileTool = {
             oldContent = readFileSync(resolved, "utf-8");
         }
         writeFileSync(resolved, content, "utf-8");
-        const diff = fileExists
-            ? generateDiff(oldContent, content)
-            : generateNewFileDiff(content);
+        const diff = fileExists ? generateDiff(oldContent, content) : generateNewFileDiff(content);
         // Increment file operations counter
         ctx.fileOperationsCount = currentCount + 1;
         // Log successful file write

package/dist/ui/box.js

@@ -65,11 +65,7 @@ export function box(lines, opts = {}) {
         out.push(pc.dim(`┌${"─".repeat(width - 2)}┐`));
     }
     for (const line of wrapped) {
-        out.push(pc.dim("│") +
-            " ".repeat(pad) +
-            padTo(line, inner) +
-            " ".repeat(pad) +
-            pc.dim("│"));
+        out.push(pc.dim("│") + " ".repeat(pad) + padTo(line, inner) + " ".repeat(pad) + pc.dim("│"));
     }
     out.push(pc.dim(`└${"─".repeat(width - 2)}┘`));
     return out;

package/dist/ui/index.js

@@ -1,6 +1,6 @@
-export { formatMarkdown, formatError, formatSuccess, formatWarning, FormattingStream } from './md-formatter';
-export { writeOutput, writeError, writeSuccess, writeWarning } from './output';
-export { formatTable, splitRow, isSeparatorRow, getTerminalWidth } from './table';
-export { box, divider, wrapText } from './box';
-export { Spinner, isSpinnerSupported } from './spinner';
-export { Renderer, isRichTerminal, summarizeArgs } from './renderer';
+export { formatMarkdown, formatError, formatSuccess, formatWarning, FormattingStream, } from "./md-formatter";
+export { writeOutput, writeError, writeSuccess, writeWarning } from "./output";
+export { formatTable, splitRow, isSeparatorRow, getTerminalWidth } from "./table";
+export { box, divider, wrapText } from "./box";
+export { Spinner, isSpinnerSupported } from "./spinner";
+export { Renderer, isRichTerminal, summarizeArgs } from "./renderer";