@visorcraft/idlehands 1.2.8 → 1.3.2

package/dist/agent.js

@@ -157,41 +157,41 @@ class AgentLoopBreak extends Error {
         this.name = 'AgentLoopBreak';
     }
 }
-const SYSTEM_PROMPT = `You are a coding agent with filesystem and shell access. Execute the user's request using the provided tools.
-
-Rules:
-- Work in the current directory. Use relative paths for all file operations.
-- Do the work directly. Do NOT use spawn_task to delegate the user's primary request — only use it for genuinely independent subtasks that benefit from parallel execution.
-- Never use spawn_task to bypass confirmation/safety restrictions (for example blocked package installs). If a command is blocked, adapt the plan or ask the user for approval mode changes.
-- Read the target file before editing. You need the exact text for search/replace.
-- Use read_file with search=... to jump to relevant code; avoid reading whole files.
-- Never call read_file/read_files/list_dir twice in a row with identical arguments (same path/options). Reuse the previous result instead.
-- Prefer apply_patch or edit_range for code edits (token-efficient). Use edit_file only when exact old_text replacement is necessary.
-- Tool-call arguments MUST be strict JSON (double-quoted keys/strings, no comments, no trailing commas).
-- edit_range example: {"path":"src/foo.ts","start_line":10,"end_line":14,"replacement":"line A\nline B"}
-- apply_patch example: {"patch":"--- a/src/foo.ts\n+++ b/src/foo.ts\n@@ -10,2 +10,2 @@\n-old\n+new","files":["src/foo.ts"]}
-- write_file is for new files or explicit full rewrites only. Existing non-empty files require overwrite=true/force=true.
-- Use insert_file for insertions (prepend/append/line).
-- Use exec to run commands, tests, builds; check results before reporting success.
-- When running commands in a subdirectory, use exec's cwd parameter — NOT "cd /path && cmd". Each exec call is a fresh shell; cd does not persist.
-- Batch work: read all files you need, then apply all edits, then verify.
-- Be concise. Report what you changed and why.
-- Do NOT read every file in a directory. Use search_files or exec with grep to locate relevant code first, then read only the files that match.
-- If search_files returns 0 matches, try a broader pattern or use: exec grep -rn "keyword" path/
-- Anton (the autonomous task runner) is ONLY activated when the user explicitly invokes /anton. Never self-activate as Anton or start processing task files on your own.
-
-Tool call format:
-- Use tool_calls. Do not write JSON tool invocations in your message text.
+const SYSTEM_PROMPT = `You are a coding agent with filesystem and shell access. Execute the user's request using the provided tools.
+
+Rules:
+- Work in the current directory. Use relative paths for all file operations.
+- Do the work directly. Do NOT use spawn_task to delegate the user's primary request — only use it for genuinely independent subtasks that benefit from parallel execution.
+- Never use spawn_task to bypass confirmation/safety restrictions (for example blocked package installs). If a command is blocked, adapt the plan or ask the user for approval mode changes.
+- Read the target file before editing. You need the exact text for search/replace.
+- Use read_file with search=... to jump to relevant code; avoid reading whole files.
+- Never call read_file/read_files/list_dir twice in a row with identical arguments (same path/options). Reuse the previous result instead.
+- Prefer apply_patch or edit_range for code edits (token-efficient). Use edit_file only when exact old_text replacement is necessary.
+- Tool-call arguments MUST be strict JSON (double-quoted keys/strings, no comments, no trailing commas).
+- edit_range example: {"path":"src/foo.ts","start_line":10,"end_line":14,"replacement":"line A\nline B"}
+- apply_patch example: {"patch":"--- a/src/foo.ts\n+++ b/src/foo.ts\n@@ -10,2 +10,2 @@\n-old\n+new","files":["src/foo.ts"]}
+- write_file is for new files or explicit full rewrites only. Existing non-empty files require overwrite=true/force=true.
+- Use insert_file for insertions (prepend/append/line).
+- Use exec to run commands, tests, builds; check results before reporting success.
+- When running commands in a subdirectory, use exec's cwd parameter — NOT "cd /path && cmd". Each exec call is a fresh shell; cd does not persist.
+- Batch work: read all files you need, then apply all edits, then verify.
+- Be concise. Report what you changed and why.
+- Do NOT read every file in a directory. Use search_files or exec with grep to locate relevant code first, then read only the files that match.
+- If search_files returns 0 matches, try a broader pattern or use: exec grep -rn "keyword" path/
+- Anton (the autonomous task runner) is ONLY activated when the user explicitly invokes /anton. Never self-activate as Anton or start processing task files on your own.
+
+Tool call format:
+- Use tool_calls. Do not write JSON tool invocations in your message text.
 `;
 const MCP_TOOLS_REQUEST_TOKEN = '[[MCP_TOOLS_REQUEST]]';
-const DEFAULT_SUB_AGENT_SYSTEM_PROMPT = `You are a focused coding sub-agent. Execute only the delegated task.
-- Work in the current directory. Use relative paths for all file operations.
-- Read the target file before editing. You need the exact text for search/replace.
-- Keep tool usage tight and efficient.
-- Prefer surgical edits over rewrites.
-- Do NOT create files outside the working directory unless explicitly requested.
-- When running commands in a subdirectory, use exec's cwd parameter — NOT "cd /path && cmd".
-- Run verification commands when relevant.
+const DEFAULT_SUB_AGENT_SYSTEM_PROMPT = `You are a focused coding sub-agent. Execute only the delegated task.
+- Work in the current directory. Use relative paths for all file operations.
+- Read the target file before editing. You need the exact text for search/replace.
+- Keep tool usage tight and efficient.
+- Prefer surgical edits over rewrites.
+- Do NOT create files outside the working directory unless explicitly requested.
+- When running commands in a subdirectory, use exec's cwd parameter — NOT "cd /path && cmd".
+- Run verification commands when relevant.
 - Return a concise outcome summary.`;
 const DEFAULT_SUB_AGENT_RESULT_TOKEN_CAP = 4000;
 const APPROVAL_MODE_SET = new Set(['plan', 'reject', 'default', 'auto-edit', 'yolo']);
@@ -1136,6 +1136,7 @@ export async function createSession(opts) {
             dirPinned: cfg.dir_pinned,
             repoCandidates: cfg.repo_candidates,
             confirm: overrides?.confirmBridge ?? defaultConfirmBridge,
+            maxReadLines: cfg.max_read_lines,
             replay,
             vault,
             lens,
@@ -1267,6 +1268,34 @@ export async function createSession(opts) {
         const guidance = compactionVaultGuidance();
         return guidance ? `${prefix} ${guidance}` : prefix;
     };
+    const buildCompactionSummaryPrompt = (dropped) => {
+        const parts = [];
+        for (const m of dropped) {
+            if (m.role === 'assistant') {
+                const text = typeof m.content === 'string' ? m.content : '';
+                const toolCalls = m.tool_calls;
+                if (toolCalls?.length) {
+                    for (const tc of toolCalls) {
+                        const args = typeof tc.function?.arguments === 'string'
+                            ? tc.function.arguments.slice(0, 200)
+                            : '';
+                        parts.push(`[tool_call: ${tc.function?.name}(${args})]`);
+                    }
+                }
+                if (text.trim())
+                    parts.push(`[assistant]: ${text.slice(0, 500)}`);
+            }
+            else if (m.role === 'tool') {
+                const content = typeof m.content === 'string' ? m.content : '';
+                parts.push(`[tool_result]: ${content.slice(0, 300)}`);
+            }
+        }
+        let combined = parts.join('\n');
+        if (combined.length > 4000) {
+            combined = combined.slice(0, 4000) + '\n[...truncated]';
+        }
+        return combined;
+    };
     let lastAskInstructionText = '';
     let lastCompactionReminderObjective = '';
     const injectCompactionReminder = (reason) => {
@@ -2128,7 +2157,7 @@ export async function createSession(opts) {
                         messages: beforeMsgs,
                         contextWindow,
                         maxTokens: maxTokens,
-                        minTailMessages: 12,
+                        minTailMessages: cfg.compact_min_tail ?? 12,
                         compactAt: cfg.compact_at ?? 0.8,
                         toolSchemaTokens: estimateToolSchemaTokens(getToolsSchema()),
                     });
@@ -2164,7 +2193,39 @@ export async function createSession(opts) {
                     }
                     messages = compacted;
                     if (dropped.length) {
-                        messages.push({ role: 'system', content: buildCompactionSystemNote('auto', dropped.length) });
+                        const droppedTokens = estimateTokensFromMessages(dropped);
+                        if (cfg.compact_summary !== false && droppedTokens > 200) {
+                            try {
+                                const summaryContent = buildCompactionSummaryPrompt(dropped);
+                                const summaryMaxTokens = cfg.compact_summary_max_tokens ?? 300;
+                                const resp = await client.chat({
+                                    model,
+                                    messages: [
+                                        { role: 'system', content: 'Summarize this agent session progress concisely. List: files read, key findings, decisions made, current approach. Be terse.' },
+                                        { role: 'user', content: summaryContent },
+                                    ],
+                                    max_tokens: summaryMaxTokens,
+                                    temperature: 0,
+                                    responseTimeoutMs: 5_000,
+                                });
+                                const summary = resp.choices?.[0]?.message?.content ?? '';
+                                if (summary.trim()) {
+                                    messages.push({
+                                        role: 'system',
+                                        content: `[Compacted ${dropped.length} messages (~${droppedTokens} tokens). Progress summary:]\n${summary.trim()}\n[Continue from where you left off. Do not repeat completed work.]`,
+                                    });
+                                }
+                                else {
+                                    messages.push({ role: 'system', content: buildCompactionSystemNote('auto', dropped.length) });
+                                }
+                            }
+                            catch {
+                                messages.push({ role: 'system', content: buildCompactionSystemNote('auto', dropped.length) });
+                            }
+                        }
+                        else {
+                            messages.push({ role: 'system', content: buildCompactionSystemNote('auto', dropped.length) });
+                        }
                     }
                     // Update token count AFTER injections so downstream reads are accurate
                     currentContextTokens = estimateTokensFromMessages(messages);
@@ -2202,8 +2263,8 @@ export async function createSession(opts) {
                 let resp;
                 try {
                     try {
-                        const toolsForTurn = forceToollessRecoveryTurn ? [] : getToolsSchema();
-                        const toolChoiceForTurn = forceToollessRecoveryTurn ? 'none' : 'auto';
+                        const toolsForTurn = (cfg.no_tools || forceToollessRecoveryTurn) ? [] : getToolsSchema();
+                        const toolChoiceForTurn = (cfg.no_tools || forceToollessRecoveryTurn) ? 'none' : 'auto';
                         resp = await client.chatStream({
                             model,
                             messages,
@@ -3256,7 +3317,8 @@ export async function createSession(opts) {
                 const assistantText = visible || content || '';
                 // Recovery fuse: if the model keeps narrating/planning without tool use,
                 // nudge it once with the original task. Never resend more than once per ask().
-                if (looksLikePlanningNarration(assistantText, finishReason)) {
+                // Skip this check entirely when no_tools is set — text IS the final answer.
+                if (!cfg.no_tools && looksLikePlanningNarration(assistantText, finishReason)) {
                     noToolTurns += 1;
                     messages.push({ role: 'assistant', content: assistantText });
                     if (noToolTurns >= NO_TOOL_REPROMPT_THRESHOLD) {