@visorcraft/idlehands 1.1.11 → 1.1.14

package/dist/agent.js

@@ -1068,6 +1068,10 @@ export async function createSession(opts) {
             noConfirm: cfg.no_confirm || cfg.approval_mode === 'yolo',
             dryRun: cfg.dry_run,
             mode: cfg.mode ?? 'code',
+            allowedWriteRoots: cfg.allowed_write_roots,
+            requireDirPinForMutations: cfg.require_dir_pin_for_mutations,
+            dirPinned: cfg.dir_pinned,
+            repoCandidates: cfg.repo_candidates,
             confirm: overrides?.confirmBridge ?? defaultConfirmBridge,
             replay,
             vault,
@@ -1213,6 +1217,21 @@ export async function createSession(opts) {
         if (!opts?.dry) {
             if (dropped.length && vault) {
                 try {
+                    // Store the original/current user prompt before compaction so it survives context loss.
+                    let userPromptToPreserve = null;
+                    for (let i = messages.length - 1; i >= 0; i--) {
+                        const m = messages[i];
+                        if (m.role === 'user') {
+                            const text = userContentToText((m.content ?? '')).trim();
+                            if (text && !text.startsWith('[Trifecta Vault') && !text.startsWith('[Vault context') && text.length > 20) {
+                                userPromptToPreserve = text;
+                                break;
+                            }
+                        }
+                    }
+                    if (userPromptToPreserve) {
+                        await vault.upsertNote('current_task', userPromptToPreserve.slice(0, 2000), 'system');
+                    }
                     await vault.archiveToolMessages(dropped, new Map());
                     await vault.note('compaction_summary', `Dropped ${dropped.length} messages (${freedTokens} tokens).`);
                 }
@@ -1855,6 +1874,23 @@ export async function createSession(opts) {
                 const dropped = beforeMsgs.filter((m) => !compactedByRefs.has(m));
                 if (dropped.length && vault) {
                     try {
+                        // Store the original/current user prompt before compaction so it survives context loss.
+                        // Find the last substantive user message that looks like a task/instruction.
+                        let userPromptToPreserve = null;
+                        for (let i = beforeMsgs.length - 1; i >= 0; i--) {
+                            const m = beforeMsgs[i];
+                            if (m.role === 'user') {
+                                const text = userContentToText((m.content ?? '')).trim();
+                                // Skip vault injection messages and short prompts
+                                if (text && !text.startsWith('[Trifecta Vault') && !text.startsWith('[Vault context') && text.length > 20) {
+                                    userPromptToPreserve = text;
+                                    break;
+                                }
+                            }
+                        }
+                        if (userPromptToPreserve) {
+                            await vault.upsertNote('current_task', userPromptToPreserve.slice(0, 2000), 'system');
+                        }
                         const toArchive = lens
                             ? await Promise.all(dropped.map((m) => archiveToolOutputForVault(m)))
                             : dropped;
@@ -2163,7 +2199,13 @@ export async function createSession(opts) {
                             mutationVersionBySig.set(sig, mutationVersion);
                             if (!hasMutatedSince) {
                                 const count = sigCounts.get(sig) ?? 0;
-                                const loopThreshold = harness.quirks.loopsOnToolError ? 3 : 6;
+                                let loopThreshold = harness.quirks.loopsOnToolError ? 3 : 6;
+                                // If the cached observation already tells the model "no matches found",
+                                // break much earlier — the model is ignoring the hint.
+                                const cachedObs = execObservationCacheBySig.get(sig) ?? '';
+                                if (cachedObs.includes('Do NOT retry')) {
+                                    loopThreshold = Math.min(loopThreshold, 3);
+                                }
                                 // At 3x, inject vault context so the model gets the data it needs
                                 if (count >= 3 && count < loopThreshold) {
                                     await injectVaultContext().catch(() => { });
@@ -2190,7 +2232,7 @@ export async function createSession(opts) {
                         }
                         // Read-only tools: only count consecutive identical calls (back-to-back turns
                         // with no other tool calls in between). A read → edit → read cycle is normal
-                        // and resets the counter. After 4 consecutive identical reads, inject a hint.
+                        // and resets the counter.
                         if (isReadOnlyTool(toolName)) {
                             // Check if this sig was also in the previous turn's set
                             if (lastTurnSigs.has(sig)) {
@@ -2203,8 +2245,8 @@ export async function createSession(opts) {
                             if (consec >= 3) {
                                 await injectVaultContext().catch(() => { });
                             }
-                            // Hard-break: after 6 consecutive identical reads, stop the session
-                            if (consec >= 6) {
+                            // Hard-break: after 4 consecutive identical reads, stop the session
+                            if (consec >= 4) {
                                 throw new Error(`tool ${toolName}: identical read repeated ${consec}x consecutively; breaking loop. ` +
                                     `The resource content has not changed between reads.`);
                             }
@@ -2442,6 +2484,14 @@ export async function createSession(opts) {
                                 content = await mcpManager.callTool(name, callArgs);
                             }
                         }
+                        // Append a hint when a read-only tool is called consecutively with
+                        // identical arguments — the model may not realize the content hasn't changed.
+                        if (isReadOnlyToolDynamic(name)) {
+                            const consec = consecutiveCounts.get(sig) ?? 0;
+                            if (consec >= 2) {
+                                content += `\n\n[WARNING: You have read this exact same resource ${consec}x consecutively with identical arguments. The content has NOT changed. Do NOT read it again. Use the information above and move on to the next step.]`;
+                            }
+                        }
                         // Hook: onToolResult (Phase 8.5 + Phase 7 rich display)
                         let toolSuccess = true;
                         let summary = reusedCachedReadOnlyExec