nyxora 26.7.4 → 26.7.5

package/dist/packages/core/src/agent/osAgent.js

@@ -6,151 +6,81 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.logger = void 0;
 exports.processOsIntent = processOsIntent;
 exports.processOsIntentStream = processOsIntentStream;
-const fs_1 = __importDefault(require("fs"));
 const parser_1 = require("../config/parser");
 const logger_1 = require("../memory/logger");
 const tracker_1 = require("../gateway/tracker");
-const episodic_1 = require("../memory/episodic");
 const skillManager_1 = require("../utils/skillManager");
-const cognitiveManager_1 = require("../cognitive/cognitiveManager");
 const reasoningScratchpad_1 = require("./reasoningScratchpad");
 const contextSummarizer_1 = require("../utils/contextSummarizer");
-const EXECUTION_DISCIPLINE = `
-<tool_persistence>
-Use tools whenever they can increase the accuracy, completeness, or factual correctness of your response.
-Do NOT stop early if another tool call would materially improve the result.
-Continue using tools until the task is completely finished and verified.
-</tool_persistence>
-
-<mandatory_tool_use>
-NEVER answer the following using only your internal memory — ALWAYS use the relevant tool:
-- Arithmetic, math, calculations
-- System State: OS version, RAM, processes
-- File contents, file sizes
-- Real-world current events
-</mandatory_tool_use>
-
-<web_search_accuracy>
-When using the search_web tool to look up news, current events, or factual data:
-1. NEVER pass casual, conversational, or highly localized queries directly to the tool (e.g. do not pass "hasil piala dunia tadi pagi").
-2. ALWAYS translate and optimize the query into an absolute, highly specific, and globally-understood English search query (e.g. "World Cup 2026 match results June 25 2026").
-3. Use depth: 2 (deep research) for anything that requires high factual accuracy, such as sports scores, news, or complex topics.
-</web_search_accuracy>
-
-<act_dont_ask>
-When a user's request has a clear, standard interpretation, take immediate ACTION instead of asking for clarification.
-NEVER show a command as a markdown code block and wait. CALL the tool directly.
-NEVER ask "do you want me to run this?" — just run it.
-NEVER say "you need to run this yourself" — you have direct shell access.
-If a command requires sudo and may need a password, just run it and report what happens.
-Only report failure AFTER actually attempting the tool call and receiving an error.
-</act_dont_ask>
-
-<anti_hallucination_execution>
-CRITICAL: It is STRICTLY FORBIDDEN to write a bash/shell command in a markdown code block (e.g. \`\`\`bash ... \`\`\`) as a substitute for calling the run_terminal_command tool.
-Writing a code block does NOT execute anything. It is a lie to the user.
-If you write \`\`\`bash\nsudo apt install steam\n\`\`\` instead of calling run_terminal_command, you are hallucinating execution.
-The ONLY way to run a command is to emit a proper tool_call for run_terminal_command.
-If the tool is available, USE IT. Do not simulate or describe running it.
-</anti_hallucination_execution>
-
-<task_completion>
-The deliverable must be a working artifact backed by real tool output — not just a description or a plan of how you would do it.
-NEVER fabricate, hallucinate, or forge tool outputs.
-</task_completion>
-
-<self_correction>
-Before providing a final answer to the user (especially regarding dates, events, news, or factual data), you MUST evaluate your tool results inside a <think> block.
-Ask yourself: "Is my answer based on absolute facts, or circumstantial evidence (e.g., guessing a registration date based on a video upload date)?"
-If the evidence is circumstantial or incomplete, you MUST NOT answer the user. Instead, call the search_web tool again with a highly optimized query and depth=2.
-</self_correction>
-`;
+const promptBuilder_1 = require("./promptBuilder");
 const registry_1 = require("../plugin/registry");
-const paths_1 = require("../config/paths");
 const picocolors_1 = __importDefault(require("picocolors"));
-exports.logger = new logger_1.Logger();
-const llmUtils_1 = require("../utils/llmUtils");
-async function getSystemPrompt(context = 'os', userInput = '') {
-    const config = (0, parser_1.loadConfig)();
-    const currentDateTime = new Date().toLocaleString('en-US');
-    let basePrompt = `You are Nyxora's OS Agent (System & Automation Specialist).
-The current real-world date and time is: ${currentDateTime}.
-
-You are running LOCALLY on the user's own computer — NOT on a remote cloud server. The 'run_terminal_command' tool executes shell commands directly on this machine, the same physical machine the user is sitting at. You have FULL local shell access. When asked to install software, manage files, or perform any OS task, you MUST use run_terminal_command immediately. NEVER claim you cannot access the user's system.
-
-Reason internally. Never reveal private reasoning. Provide only concise conclusions, assumptions, and actionable steps.
-
-[OS EXECUTION WORKFLOW]
-CRITICAL RULE 1: NEVER expose internal JSON tool calls. Explain the outcome naturally.
-CRITICAL RULE 2: STRICT LANGUAGE MATCHING. Reply in the exact same language as the user's LATEST prompt, UNLESS the Episodic Memories or Cognitive Skills specify a strict language preference.
-CRITICAL RULE 3: FILE SYSTEM SAFETY. You are STRICTLY FORBIDDEN from modifying config.yaml, rpc_key.yaml, or policy.yaml using terminal commands like sed or echo.
-CRITICAL RULE 4: CRON JOBS VS LIMIT ORDERS. Do NOT use schedule_task for price-based trading triggers. Use schedule_task for time-based recurring tasks.
-CRITICAL RULE 5: TOOL CONFIDENCE. NEVER fabricate file contents or command outputs.
-
-[SUDO & PACKAGE INSTALL STRATEGY]
-This tool runs in a NON-INTERACTIVE shell (no TTY). Therefore:
-- ALWAYS prefix apt/apt-get commands with: DEBIAN_FRONTEND=noninteractive
-- ALWAYS use the -y flag for package installations to auto-confirm.
-- If a command fails with "sudo: a password is required" or similar, DO NOT promise to retry without actually retrying. Instead:
-  1. First retry with: echo 'USER_PASSWORD' | sudo -S <command> — but since you don't know the password, skip this.
-  2. Instead, try running the command WITHOUT sudo if possible (e.g. for user-space tools).
-  3. If sudo is truly required and unavailable, clearly tell the user EXACTLY what command to run manually in their terminal, with a copy-paste ready command. Do not just say it failed without providing a solution.
-- NEVER promise to retry ("let me try again", "I'll run it again", etc.) without immediately making another tool_call in the same response.
-
-${EXECUTION_DISCIPLINE}
-`;
-    // Inject Active Cognitive Skills
-    const activeSOP = cognitiveManager_1.cognitiveManager.loadActiveCognitiveSkills(userInput);
-    if (activeSOP) {
-        basePrompt += `\n\n[ACTIVE COGNITIVE SKILLS]\n${activeSOP}\n`;
-    }
-    // Inject Episodic Memories via Python RAG
-    try {
-        const ragRes = await fetch('http://127.0.0.1:8000/memory/rag', {
-            method: 'POST',
-            headers: { 'Content-Type': 'application/json' },
-            body: JSON.stringify({ query: userInput, top_k: 5 })
-        });
-        if (ragRes.ok) {
-            const ragData = await ragRes.json();
-            if (ragData.memories && ragData.memories.length > 0) {
-                basePrompt += `\n\n--- EPISODIC MEMORIES (SMART SUGGESTIONS) ---\n`;
-                ragData.memories.forEach((mem) => {
-                    basePrompt += `- ${mem}\n`;
-                });
+// --- REGISTER LIFECYCLE HOOKS ---
+registry_1.pluginManager.registerHook({
+    name: 'ReasoningGate',
+    beforeToolCall: async (toolName, args, context) => {
+        const toolsRequiringReasoning = ['run_command', 'write_to_file', 'replace_file_content', 'multi_replace_file_content', 'execute_script'];
+        if (toolsRequiringReasoning.includes(toolName)) {
+            const responseMessage = context.responseMessage || {};
+            const hasThinkingTag = /<(think|thought|thinking|reasoning|analysis|reflection)>[\s\S]*?<\/\1>/i.test(responseMessage.content || '');
+            const hasNativeReasoning = !!responseMessage.reasoning_content;
+            if (!hasThinkingTag && !hasNativeReasoning) {
+                const msg = `[System Error] BLOCKED BY REASONING GATE: You MUST output a <think>...</think> block to plan your actions BEFORE calling the '${toolName}' tool. Please rethink and try again.`;
+                console.log(picocolors_1.default.red(`[❌ Blocked] Tool ${toolName} blocked by Reasoning Gate.`));
+                return { block: true, reason: msg };
             }
         }
     }
-    catch (e) {
-        // Fallback or ignore if Python ML engine is down
-    }
-    // Inject User Information & Personas from user.md
-    try {
-        const userMdPath = (0, paths_1.getPath)('user.md');
-        if (fs_1.default.existsSync(userMdPath)) {
-            const userInstructions = fs_1.default.readFileSync(userMdPath, 'utf8');
-            basePrompt += `\n\n--- USER INFORMATION & PREFERENCES ---\n${userInstructions}\n`;
+});
+registry_1.pluginManager.registerHook({
+    name: 'Web3FastReturn',
+    afterToolCall: async (toolName, args, result, context) => {
+        const fastReturnTools = ['transfer_token', 'transfer_native', 'swap_token', 'bridge_token', 'mint_nft', 'custom_tx', 'revoke_approval', 'supply_aave', 'deposit_yield_vault', 'provide_liquidity_v3', 'send_telegram_file'];
+        if (fastReturnTools.includes(toolName)) {
+            return { terminate: true };
         }
     }
-    catch (e) {
-        // Ignore error
-    }
-    // HIGHEST PRIORITY: Inject observed user communication style (NyxDaemon)
+});
+// --- END LIFECYCLE HOOKS ---
+exports.logger = new logger_1.Logger();
+// Helper to trigger background review async
+const triggerBackgroundReview = async (sessionId) => {
     try {
-        const strongPersonas = episodic_1.episodicDB.getStrongPersonas(0.5);
-        if (strongPersonas.length > 0) {
-            basePrompt += `\n\n--- ⚡ OVERRIDE: USER COMMUNICATION STYLE (HIGHEST PRIORITY — OVERRIDES ALL RULES ABOVE) ---\n`;
-            basePrompt += `CRITICAL: These are CONFIRMED user preferences observed across multiple conversations. You MUST apply them IMMEDIATELY and CONSISTENTLY, overriding any default tone or style instructions above.\n\n`;
-            strongPersonas.forEach(p => {
-                const label = p.category ? `[${p.category.toUpperCase()}]` : '[PREFERENCE]';
-                basePrompt += `${label} ${p.trait}\n`;
-            });
-        }
+        const history = exports.logger.getHistory(sessionId, 30);
+        fetch('http://127.0.0.1:8000/cognitive/review', {
+            method: 'POST',
+            headers: { 'Content-Type': 'application/json' },
+            body: JSON.stringify({
+                messages: history,
+                session_id: sessionId || 'default',
+                trigger: 'turn_end'
+            })
+        }).catch(() => { });
     }
     catch (e) {
-        // Ignore
+        // silently fail
     }
-    return basePrompt;
+};
+const llmUtils_1 = require("../utils/llmUtils");
+async function getSystemPrompt(context = 'os', userInput = '') {
+    const config = (0, parser_1.loadConfig)();
+    // Safely deduce model family from config
+    const provider = (config?.llm?.provider || '').toLowerCase();
+    let modelFamily = 'unknown';
+    if (provider.includes('openai'))
+        modelFamily = 'openai';
+    else if (provider.includes('gemini') || provider.includes('google'))
+        modelFamily = 'google';
+    else if (provider.includes('anthropic') || provider.includes('claude'))
+        modelFamily = 'anthropic';
+    else if (provider.includes('grok') || provider.includes('xai'))
+        modelFamily = 'grok';
+    return await promptBuilder_1.promptBuilder.buildSystemPrompt({
+        agentType: context,
+        userInput,
+        config,
+        modelFamily
+    });
 }
 async function processOsIntent(input, role = 'user', onProgress, sessionId) {
     const config = (0, parser_1.loadConfig)();
@@ -213,7 +143,8 @@ CRITICAL INSTRUCTIONS:
                     model: config.llm.model,
                     temperature: config.llm.temperature,
                     messages: messages,
-                    tools: activeTools
+                    tools: activeTools,
+                    reasoning_effort: (!config.llm.reasoning_effort || config.llm.reasoning_effort === 'none') ? undefined : config.llm.reasoning_effort
                 });
             });
             const responseMessage = response.message;
@@ -232,36 +163,10 @@ CRITICAL INSTRUCTIONS:
                 tool_calls: responseMessage.tool_calls,
             }, sessionId);
             if (!responseMessage.tool_calls || responseMessage.tool_calls.length === 0) {
-                // --- CRITIC PASS (Self-Improvement) ---
-                const isLongResponse = (cleanedContent?.length ?? 0) > 100;
-                if (isLongResponse && !criticHasFired) {
-                    criticHasFired = true;
-                    try {
-                        const criticRes = await fetch('http://127.0.0.1:8000/cognitive/critic', {
-                            method: 'POST',
-                            headers: { 'Content-Type': 'application/json' },
-                            body: JSON.stringify({ user_input: input, draft_answer: cleanedContent, current_utc_datetime: new Date().toISOString() })
-                        });
-                        if (criticRes.ok) {
-                            const evaluation = await criticRes.json();
-                            if (evaluation.needs_revision) {
-                                console.log(picocolors_1.default.cyan(`[🧠 Critic] Revision needed. Confidence: ${evaluation.factual_confidence?.toFixed(2)}. Completeness: ${evaluation.completeness?.toFixed(2)}. Re-generating...`));
-                                exports.logger.addEntry({
-                                    role: 'system',
-                                    content: `[SELF-CRITIQUE — MANDATORY REVISION] Your previous answer was REJECTED. Issues:\n${evaluation.revision_instructions}\n\nCRITICAL REVISION RULES:\n1. Look at the tool results (search_web, etc.) ALREADY in this conversation history above.\n2. Base your revised answer EXCLUSIVELY on those tool results — NEVER use training data memory for facts, dates, or events.\n3. If tool results contain the answer, state it directly and confidently. Do NOT say an event hasn't happened if the tool results show it has.\n4. Do NOT call any tools again — the results are already in your history. USE THEM NOW.`
-                                }, sessionId);
-                                continue; // Loop kembali ke Generator untuk revisi
-                            }
-                            else {
-                                console.log(picocolors_1.default.green(`[🧠 Critic] Passed. Confidence: ${evaluation.factual_confidence?.toFixed(2)}, Completeness: ${evaluation.completeness?.toFixed(2)}.`));
-                            }
-                        }
-                    }
-                    catch {
-                        // Python ML Engine tidak aktif — skip Critic, lanjut seperti biasa
-                    }
-                }
-                // --- END CRITIC PASS ---
+                // --- CRITIC PASS REMOVED ---
+                // The new PromptBuilder handles robust reasoning. Post-generation critic
+                // causes aggressive loops and UI artifacts.
+                triggerBackgroundReview(sessionId);
                 return cleanedContent || 'No response generated.';
             }
             let canFastReturnAll = true;
@@ -270,7 +175,7 @@ CRITICAL INSTRUCTIONS:
             const fastReturnTools = [
                 'transfer_token', 'transfer_native', 'swap_token', 'bridge_token',
                 'mint_nft', 'custom_tx', 'revoke_approval', 'supply_aave',
-                'deposit_yield_vault', 'provide_liquidity_v3'
+                'deposit_yield_vault', 'provide_liquidity_v3', 'send_telegram_file'
             ];
             for (const _toolCall of responseMessage.tool_calls) {
                 const toolCall = _toolCall;
@@ -335,7 +240,7 @@ CRITICAL INSTRUCTIONS:
                     content: result,
                 }, sessionId);
                 accumulatedResults.push(result);
-                if (!fastReturnTools.includes(toolName)) {
+                if (!['transfer_token', 'transfer_native', 'swap_token', 'bridge_token', 'mint_nft', 'custom_tx', 'revoke_approval', 'supply_aave', 'deposit_yield_vault', 'provide_liquidity_v3', 'send_telegram_file'].includes(toolName)) {
                     canFastReturnAll = false;
                 }
             }
@@ -352,6 +257,7 @@ CRITICAL INSTRUCTIONS:
                 if (consecutiveToolErrors >= 2) {
                     const errorSummary = `⚠️ Unable to complete this request after multiple attempts.\n${accumulatedResults.join('\n')}`;
                     exports.logger.addEntry({ role: 'assistant', content: errorSummary }, sessionId);
+                    triggerBackgroundReview(sessionId);
                     return errorSummary;
                 }
             }
@@ -362,12 +268,14 @@ CRITICAL INSTRUCTIONS:
             if (canFastReturnAll && accumulatedResults.length > 0) {
                 const finalContent = accumulatedResults.join('\n\n---\n\n');
                 exports.logger.addEntry({ role: 'assistant', content: finalContent }, sessionId);
+                triggerBackgroundReview(sessionId);
                 return finalContent;
             }
             // Loop continues, sending tool results in the next turn
         }
         const maxTurnMsg = "⚠️ Reached maximum interaction limit (10 turns). Please be more specific.";
         exports.logger.addEntry({ role: 'assistant', content: maxTurnMsg }, sessionId);
+        triggerBackgroundReview(sessionId);
         return maxTurnMsg;
     }
     catch (error) {
@@ -426,7 +334,9 @@ CRITICAL INSTRUCTIONS:
             ];
             let streamedContent = '';
             const response = await (0, llmUtils_1.executeWithRetry)(async (client) => {
-                return await client.stream({ model: config.llm.model, temperature: config.llm.temperature, messages, tools: activeTools }, (chunk) => {
+                streamedContent = '';
+                onChunk('[CLEAR_STREAM]');
+                return await client.stream({ model: config.llm.model, temperature: config.llm.temperature, messages, tools: activeTools, reasoning_effort: (!config.llm.reasoning_effort || config.llm.reasoning_effort === 'none') ? undefined : config.llm.reasoning_effort }, (chunk) => {
                     streamedContent += chunk;
                     onChunk(chunk);
                 });
@@ -440,53 +350,28 @@ CRITICAL INSTRUCTIONS:
             exports.logger.addEntry({
                 role: 'assistant',
                 content: responseMessage.content || '',
+                reasoning_content: responseMessage.reasoning_content,
                 tool_calls: responseMessage.tool_calls,
             }, sessionId);
             if (!responseMessage.tool_calls || responseMessage.tool_calls.length === 0) {
                 let finalContent = responseMessage.content || 'No response generated.';
                 finalContent = finalContent.replace(/<(think|thought|thinking|reasoning|analysis|reflection)[\s\S]*?<\/\1>\n?/gi, '').trim();
-                // --- CRITIC PASS (Self-Improvement) ---
-                const isLongResponseStream = finalContent.length > 100;
-                if (isLongResponseStream && !criticHasFiredStream) {
-                    criticHasFiredStream = true;
-                    try {
-                        const criticRes = await fetch('http://127.0.0.1:8000/cognitive/critic', {
-                            method: 'POST',
-                            headers: { 'Content-Type': 'application/json' },
-                            body: JSON.stringify({ user_input: input, draft_answer: finalContent, current_utc_datetime: new Date().toISOString() })
-                        });
-                        if (criticRes.ok) {
-                            const evaluation = await criticRes.json();
-                            if (evaluation.needs_revision) {
-                                console.log(picocolors_1.default.cyan(`[🧠 Critic] Revision needed. Confidence: ${evaluation.factual_confidence?.toFixed(2)}. Completeness: ${evaluation.completeness?.toFixed(2)}. Re-generating...`));
-                                exports.logger.addEntry({
-                                    role: 'system',
-                                    content: `[SELF-CRITIQUE — MANDATORY REVISION] Your previous answer was REJECTED. Issues:\n${evaluation.revision_instructions}\n\nCRITICAL REVISION RULES:\n1. Look at the tool results (search_web, etc.) ALREADY in this conversation history above.\n2. Base your revised answer EXCLUSIVELY on those tool results — NEVER use training data memory for facts, dates, or events.\n3. If tool results contain the answer, state it directly and confidently. Do NOT say an event hasn't happened if the tool results show it has.\n4. Do NOT call any tools again — the results are already in your history. USE THEM NOW.`
-                                }, sessionId);
-                                continue; // Loop kembali ke Generator untuk revisi
-                            }
-                            else {
-                                console.log(picocolors_1.default.green(`[🧠 Critic] Passed. Confidence: ${evaluation.factual_confidence?.toFixed(2)}, Completeness: ${evaluation.completeness?.toFixed(2)}.`));
-                            }
-                        }
-                    }
-                    catch {
-                        // Python ML Engine tidak aktif — skip Critic, lanjut seperti biasa
-                    }
-                }
-                // --- END CRITIC PASS ---
+                finalContent = finalContent.replace(/^\s*(?:\*\*)?(?:think|thought|thinking|reasoning|analysis|reflection)(?:\*\*)?\s*?\n[\s\S]*?\n\n/i, '').trim();
+                // --- CRITIC PASS REMOVED ---
+                // Prevents leaking internal reasoning into the stream and confusing the user.
                 fullResponse = finalContent;
+                triggerBackgroundReview(sessionId);
                 break;
             }
-            // Tool calls detected — pause stream visually and execute tools
-            const fastReturnTools = ['transfer_token', 'transfer_native', 'swap_token', 'bridge_token', 'mint_nft', 'custom_tx', 'revoke_approval', 'supply_aave', 'deposit_yield_vault', 'provide_liquidity_v3'];
-            let canFastReturnAll = true;
+            // Tool calls detected — pause stream visually and execute tools concurrently
+            let shouldFastReturn = false;
             const accumulatedResults = [];
-            for (const _toolCall of responseMessage.tool_calls) {
+            const promises = responseMessage.tool_calls.map(async (_toolCall) => {
                 const toolCall = _toolCall;
                 const toolName = toolCall.function.name;
                 let result = '';
                 let args = {};
+                const context = { sessionId, toolCallId: toolCall.id, responseMessage };
                 console.log(picocolors_1.default.yellow(`[⚡ Tool Execution] AI is calling ${toolName}...`));
                 if (onProgress)
                     onProgress(`_⚡ Running tool: ${toolName}..._`);
@@ -499,17 +384,28 @@ CRITICAL INSTRUCTIONS:
                 catch (parseError) {
                     result = `[System Error] Arguments for ${toolName} must be valid JSON. Error: ${parseError.message}`;
                     exports.logger.addEntry({ role: 'tool', tool_call_id: toolCall.id, content: result }, sessionId);
-                    continue;
+                    return { toolName, result };
                 }
                 if (!(0, skillManager_1.isSkillActive)(toolName)) {
                     result = `[System Error] Access denied: Skill '${toolName}' is currently disabled.`;
                     exports.logger.addEntry({ role: 'tool', tool_call_id: toolCall.id, content: result }, sessionId);
-                    continue;
+                    return { toolName, result };
+                }
+                // --- LIFECYCLE: BEFORE TOOL CALL ---
+                const beforeRes = await registry_1.pluginManager.triggerBeforeHooks(toolName, args, context);
+                if (beforeRes && beforeRes.block) {
+                    result = beforeRes.reason || `[System Error] Blocked by hook for ${toolName}`;
+                    exports.logger.addEntry({ role: 'tool', tool_call_id: toolCall.id, content: result }, sessionId);
+                    return { toolName, result };
                 }
                 try {
-                    const pluginResult = await registry_1.pluginManager.executeTool(toolName, args, { sessionId });
+                    const pluginResult = await registry_1.pluginManager.executeTool(toolName, args, context, (partialResult) => {
+                        // Partial Streaming callback
+                        if (onProgress)
+                            onProgress(`_⏳ [${toolName}] ${partialResult}..._`);
+                    });
                     result = pluginResult !== null ? pluginResult : `Error: Tool ${toolName} is not implemented.`;
-                    if (result.includes('[Security Blocked]') || result.startsWith('Error:')) {
+                    if (typeof result === 'string' && (result.includes('[Security Blocked]') || result.startsWith('Error:'))) {
                         console.log(picocolors_1.default.red(`[❌ Failed] Tool ${toolName} returned an error or was blocked.`));
                     }
                     else {
@@ -520,16 +416,23 @@ CRITICAL INSTRUCTIONS:
                     result = `Error executing ${toolName}: ${toolError.message}`;
                     console.error(picocolors_1.default.red(`[❌ Error Crash] Execution of ${toolName} failed: ${toolError.message}`));
                 }
+                // --- LIFECYCLE: AFTER TOOL CALL ---
+                const afterRes = await registry_1.pluginManager.triggerAfterHooks(toolName, args, result, context);
+                result = afterRes.content || result;
+                if (afterRes.terminate) {
+                    shouldFastReturn = true;
+                }
                 exports.logger.addEntry({ role: 'tool', tool_call_id: toolCall.id, name: toolName, content: result }, sessionId);
-                accumulatedResults.push(result);
-                if (!fastReturnTools.includes(toolName))
-                    canFastReturnAll = false;
-            }
-            if (canFastReturnAll && accumulatedResults.length > 0) {
+                return { toolName, result };
+            });
+            const results = await Promise.all(promises);
+            results.forEach(r => accumulatedResults.push(r.result));
+            if (shouldFastReturn && accumulatedResults.length > 0) {
                 const finalContent = accumulatedResults.join('\n\n---\n\n');
                 exports.logger.addEntry({ role: 'assistant', content: finalContent }, sessionId);
                 onChunk(finalContent);
                 fullResponse = finalContent;
+                triggerBackgroundReview(sessionId);
                 break;
             }
         }
@@ -537,6 +440,7 @@ CRITICAL INSTRUCTIONS:
             const maxTurnMsg = '⚠️ Reached maximum interaction limit (10 turns). Please be more specific.';
             exports.logger.addEntry({ role: 'assistant', content: maxTurnMsg }, sessionId);
             fullResponse = maxTurnMsg;
+            triggerBackgroundReview(sessionId);
         }
         return fullResponse;
     }