@office-ai/aioncli-core 0.2.3 → 0.18.4

package/dist/src/agents/executor.test.js

@@ -0,0 +1,1362 @@
+/**
+ * @license
+ * Copyright 2025 Google LLC
+ * SPDX-License-Identifier: Apache-2.0
+ */
+import { describe, it, expect, vi, beforeEach, afterEach, } from 'vitest';
+import { debugLogger } from '../utils/debugLogger.js';
+import { AgentExecutor } from './executor.js';
+import { makeFakeConfig } from '../test-utils/config.js';
+import { ToolRegistry } from '../tools/tool-registry.js';
+import { LSTool } from '../tools/ls.js';
+import { LS_TOOL_NAME, READ_FILE_TOOL_NAME } from '../tools/tool-names.js';
+import { GeminiChat, StreamEventType, } from '../core/geminiChat.js';
+import {} from '@google/genai';
+import { MockTool } from '../test-utils/mock-tool.js';
+import { getDirectoryContextString } from '../utils/environmentContext.js';
+import { z } from 'zod';
+import { promptIdContext } from '../utils/promptIdContext.js';
+import { logAgentStart, logAgentFinish, logRecoveryAttempt, } from '../telemetry/loggers.js';
+import { AgentStartEvent, AgentFinishEvent, RecoveryAttemptEvent, } from '../telemetry/types.js';
+import { AgentTerminateMode } from './types.js';
+import { CompressionStatus } from '../core/turn.js';
+import { ChatCompressionService } from '../services/chatCompressionService.js';
+import { getModelConfigAlias } from './registry.js';
+const { mockSendMessageStream, mockExecuteToolCall, mockSetSystemInstruction, mockCompress, mockSetTools, } = vi.hoisted(() => ({
+    mockSendMessageStream: vi.fn(),
+    mockExecuteToolCall: vi.fn(),
+    mockSetSystemInstruction: vi.fn(),
+    mockCompress: vi.fn(),
+    mockSetTools: vi.fn(),
+}));
+let mockChatHistory = [];
+const mockSetHistory = vi.fn((newHistory) => {
+    mockChatHistory = newHistory;
+});
+vi.mock('../services/chatCompressionService.js', () => ({
+    ChatCompressionService: vi.fn().mockImplementation(() => ({
+        compress: mockCompress,
+    })),
+}));
+vi.mock('../core/geminiChat.js', async (importOriginal) => {
+    const actual = await importOriginal();
+    return {
+        ...actual,
+        GeminiChat: vi.fn().mockImplementation(() => ({
+            sendMessageStream: mockSendMessageStream,
+            getHistory: vi.fn((_curated) => [...mockChatHistory]),
+            setHistory: mockSetHistory,
+            setSystemInstruction: mockSetSystemInstruction,
+            setTools: mockSetTools,
+        })),
+    };
+});
+vi.mock('../core/nonInteractiveToolExecutor.js', () => ({
+    executeToolCall: mockExecuteToolCall,
+}));
+vi.mock('../utils/environmentContext.js');
+vi.mock('../telemetry/loggers.js', () => ({
+    logAgentStart: vi.fn(),
+    logAgentFinish: vi.fn(),
+    logRecoveryAttempt: vi.fn(),
+}));
+vi.mock('../utils/promptIdContext.js', async (importOriginal) => {
+    const actual = await importOriginal();
+    return {
+        ...actual,
+        promptIdContext: {
+            ...actual.promptIdContext,
+            getStore: vi.fn(),
+            run: vi.fn((_id, fn) => fn()),
+        },
+    };
+});
+const MockedGeminiChat = vi.mocked(GeminiChat);
+const mockedGetDirectoryContextString = vi.mocked(getDirectoryContextString);
+const mockedPromptIdContext = vi.mocked(promptIdContext);
+const mockedLogAgentStart = vi.mocked(logAgentStart);
+const mockedLogAgentFinish = vi.mocked(logAgentFinish);
+const mockedLogRecoveryAttempt = vi.mocked(logRecoveryAttempt);
+// Constants for testing
+const TASK_COMPLETE_TOOL_NAME = 'complete_task';
+const MOCK_TOOL_NOT_ALLOWED = new MockTool({ name: 'write_file_interactive' });
+/**
+ * Helper to create a mock API response chunk.
+ * Uses conditional spread to handle readonly functionCalls property safely.
+ */
+const createMockResponseChunk = (parts, functionCalls) => ({
+    candidates: [{ index: 0, content: { role: 'model', parts } }],
+    ...(functionCalls && functionCalls.length > 0 ? { functionCalls } : {}),
+});
+/**
+ * Helper to mock a single turn of model response in the stream.
+ */
+const mockModelResponse = (functionCalls, thought, text) => {
+    const parts = [];
+    if (thought) {
+        parts.push({
+            text: `**${thought}** This is the reasoning part.`,
+            thought: true,
+        });
+    }
+    if (text)
+        parts.push({ text });
+    const responseChunk = createMockResponseChunk(parts, functionCalls);
+    mockSendMessageStream.mockImplementationOnce(async () => (async function* () {
+        yield {
+            type: StreamEventType.CHUNK,
+            value: responseChunk,
+        };
+    })());
+};
+/**
+ * Helper to extract the message parameters sent to sendMessageStream.
+ * Provides type safety for inspecting mock calls.
+ */
+const getMockMessageParams = (callIndex) => {
+    const call = mockSendMessageStream.mock.calls[callIndex];
+    expect(call).toBeDefined();
+    return {
+        modelConfigKey: call[0],
+        message: call[1],
+    };
+};
+let mockConfig;
+let parentToolRegistry;
+/**
+ * Type-safe helper to create agent definitions for tests.
+ */
+const createTestDefinition = (tools = [LS_TOOL_NAME], runConfigOverrides = {}, outputConfigMode = 'default', schema = z.string()) => {
+    let outputConfig;
+    if (outputConfigMode === 'default') {
+        outputConfig = {
+            outputName: 'finalResult',
+            description: 'The final result.',
+            schema,
+        };
+    }
+    return {
+        name: 'TestAgent',
+        description: 'An agent for testing.',
+        inputConfig: {
+            inputs: { goal: { type: 'string', required: true, description: 'goal' } },
+        },
+        modelConfig: { model: 'gemini-test-model', temp: 0, top_p: 1 },
+        runConfig: { max_time_minutes: 5, max_turns: 5, ...runConfigOverrides },
+        promptConfig: { systemPrompt: 'Achieve the goal: ${goal}.' },
+        toolConfig: { tools },
+        outputConfig,
+    };
+};
+describe('AgentExecutor', () => {
+    let activities;
+    let onActivity;
+    let abortController;
+    let signal;
+    beforeEach(async () => {
+        vi.resetAllMocks();
+        mockCompress.mockClear();
+        mockSetHistory.mockClear();
+        mockSendMessageStream.mockReset();
+        mockSetSystemInstruction.mockReset();
+        mockSetTools.mockReset();
+        mockExecuteToolCall.mockReset();
+        mockedLogAgentStart.mockReset();
+        mockedLogAgentFinish.mockReset();
+        mockedPromptIdContext.getStore.mockReset();
+        mockedPromptIdContext.run.mockImplementation((_id, fn) => fn());
+        ChatCompressionService.mockImplementation(() => ({
+            compress: mockCompress,
+        }));
+        mockCompress.mockResolvedValue({
+            newHistory: null,
+            info: { compressionStatus: CompressionStatus.NOOP },
+        });
+        MockedGeminiChat.mockImplementation(() => ({
+            sendMessageStream: mockSendMessageStream,
+            setSystemInstruction: mockSetSystemInstruction,
+            setTools: mockSetTools,
+            getHistory: vi.fn((_curated) => [...mockChatHistory]),
+            getLastPromptTokenCount: vi.fn(() => 100),
+            setHistory: mockSetHistory,
+        }));
+        vi.useFakeTimers();
+        mockConfig = makeFakeConfig();
+        parentToolRegistry = new ToolRegistry(mockConfig);
+        parentToolRegistry.registerTool(new LSTool(mockConfig));
+        parentToolRegistry.registerTool(new MockTool({ name: READ_FILE_TOOL_NAME }));
+        parentToolRegistry.registerTool(MOCK_TOOL_NOT_ALLOWED);
+        vi.spyOn(mockConfig, 'getToolRegistry').mockResolvedValue(parentToolRegistry);
+        mockedGetDirectoryContextString.mockResolvedValue('Mocked Environment Context');
+        activities = [];
+        onActivity = (activity) => activities.push(activity);
+        abortController = new AbortController();
+        signal = abortController.signal;
+    });
+    afterEach(() => {
+        vi.useRealTimers();
+    });
+    describe('create (Initialization and Validation)', () => {
+        it('should create successfully with allowed tools', async () => {
+            const definition = createTestDefinition([LS_TOOL_NAME]);
+            const executor = await AgentExecutor.create(definition, mockConfig, onActivity);
+            expect(executor).toBeInstanceOf(AgentExecutor);
+        });
+        it('SECURITY: should throw if a tool is not on the non-interactive allowlist', async () => {
+            const definition = createTestDefinition([MOCK_TOOL_NOT_ALLOWED.name]);
+            await expect(AgentExecutor.create(definition, mockConfig, onActivity)).rejects.toThrow(/not on the allow-list for non-interactive execution/);
+        });
+        it('should create an isolated ToolRegistry for the agent', async () => {
+            const definition = createTestDefinition([
+                LS_TOOL_NAME,
+                READ_FILE_TOOL_NAME,
+            ]);
+            const executor = await AgentExecutor.create(definition, mockConfig, onActivity);
+            const agentRegistry = executor['toolRegistry'];
+            expect(agentRegistry).not.toBe(parentToolRegistry);
+            expect(agentRegistry.getAllToolNames()).toEqual(expect.arrayContaining([LS_TOOL_NAME, READ_FILE_TOOL_NAME]));
+            expect(agentRegistry.getAllToolNames()).toHaveLength(2);
+            expect(agentRegistry.getTool(MOCK_TOOL_NOT_ALLOWED.name)).toBeUndefined();
+        });
+        it('should use parentPromptId from context to create agentId', async () => {
+            const parentId = 'parent-id';
+            mockedPromptIdContext.getStore.mockReturnValue(parentId);
+            const definition = createTestDefinition();
+            const executor = await AgentExecutor.create(definition, mockConfig, onActivity);
+            expect(executor['agentId']).toMatch(new RegExp(`^${parentId}-${definition.name}-`));
+        });
+        it('should correctly apply templates to initialMessages', async () => {
+            const definition = createTestDefinition();
+            // Override promptConfig to use initialMessages instead of systemPrompt
+            definition.promptConfig = {
+                initialMessages: [
+                    { role: 'user', parts: [{ text: 'Goal: ${goal}' }] },
+                    { role: 'model', parts: [{ text: 'OK, starting on ${goal}.' }] },
+                ],
+            };
+            const inputs = { goal: 'TestGoal' };
+            // Mock a response to prevent the loop from running forever
+            mockModelResponse([
+                {
+                    name: TASK_COMPLETE_TOOL_NAME,
+                    args: { finalResult: 'done' },
+                    id: 'call1',
+                },
+            ]);
+            const executor = await AgentExecutor.create(definition, mockConfig, onActivity);
+            await executor.run(inputs, signal);
+            const chatConstructorArgs = MockedGeminiChat.mock.calls[0];
+            const startHistory = chatConstructorArgs[3]; // history is the 4th arg
+            expect(startHistory).toBeDefined();
+            expect(startHistory).toHaveLength(2);
+            // Perform checks on defined objects to satisfy TS
+            const firstPart = startHistory?.[0]?.parts?.[0];
+            expect(firstPart?.text).toBe('Goal: TestGoal');
+            const secondPart = startHistory?.[1]?.parts?.[0];
+            expect(secondPart?.text).toBe('OK, starting on TestGoal.');
+        });
+    });
+    describe('run (Execution Loop and Logic)', () => {
+        it('should log AgentFinish with error if run throws', async () => {
+            const definition = createTestDefinition();
+            // Make the definition invalid to cause an error during run
+            definition.inputConfig.inputs = {
+                goal: { type: 'string', required: true, description: 'goal' },
+            };
+            const executor = await AgentExecutor.create(definition, mockConfig, onActivity);
+            // Run without inputs to trigger validation error
+            await expect(executor.run({}, signal)).rejects.toThrow(/Missing required input parameters/);
+            expect(mockedLogAgentStart).toHaveBeenCalledTimes(1);
+            expect(mockedLogAgentFinish).toHaveBeenCalledTimes(1);
+            expect(mockedLogAgentFinish).toHaveBeenCalledWith(mockConfig, expect.objectContaining({
+                terminate_reason: AgentTerminateMode.ERROR,
+            }));
+        });
+        it('should execute successfully when model calls complete_task with output (Happy Path with Output)', async () => {
+            const definition = createTestDefinition();
+            const executor = await AgentExecutor.create(definition, mockConfig, onActivity);
+            const inputs = { goal: 'Find files' };
+            // Turn 1: Model calls ls
+            mockModelResponse([{ name: LS_TOOL_NAME, args: { path: '.' }, id: 'call1' }], 'T1: Listing');
+            mockExecuteToolCall.mockResolvedValueOnce({
+                status: 'success',
+                request: {
+                    callId: 'call1',
+                    name: LS_TOOL_NAME,
+                    args: { path: '.' },
+                    isClientInitiated: false,
+                    prompt_id: 'test-prompt',
+                },
+                tool: {},
+                invocation: {},
+                response: {
+                    callId: 'call1',
+                    resultDisplay: 'file1.txt',
+                    responseParts: [
+                        {
+                            functionResponse: {
+                                name: LS_TOOL_NAME,
+                                response: { result: 'file1.txt' },
+                                id: 'call1',
+                            },
+                        },
+                    ],
+                    error: undefined,
+                    errorType: undefined,
+                    contentLength: undefined,
+                },
+            });
+            // Turn 2: Model calls complete_task with required output
+            mockModelResponse([
+                {
+                    name: TASK_COMPLETE_TOOL_NAME,
+                    args: { finalResult: 'Found file1.txt' },
+                    id: 'call2',
+                },
+            ], 'T2: Done');
+            const output = await executor.run(inputs, signal);
+            expect(mockSendMessageStream).toHaveBeenCalledTimes(2);
+            const systemInstruction = MockedGeminiChat.mock.calls[0][1];
+            expect(systemInstruction).toContain(`MUST call the \`${TASK_COMPLETE_TOOL_NAME}\` tool`);
+            expect(systemInstruction).toContain('Mocked Environment Context');
+            expect(systemInstruction).toContain('You are running in a non-interactive mode');
+            expect(systemInstruction).toContain('Always use absolute paths');
+            const { modelConfigKey } = getMockMessageParams(0);
+            expect(modelConfigKey.model).toBe(getModelConfigAlias(definition));
+            const call = mockSetTools.mock.calls[0];
+            const sentTools = call[0][0].functionDeclarations;
+            expect(sentTools).toBeDefined();
+            expect(sentTools).toEqual(expect.arrayContaining([
+                expect.objectContaining({ name: LS_TOOL_NAME }),
+                expect.objectContaining({ name: TASK_COMPLETE_TOOL_NAME }),
+            ]));
+            const completeToolDef = sentTools.find((t) => t.name === TASK_COMPLETE_TOOL_NAME);
+            expect(completeToolDef?.parameters?.required).toContain('finalResult');
+            expect(output.result).toBe('Found file1.txt');
+            expect(output.terminate_reason).toBe(AgentTerminateMode.GOAL);
+            // Telemetry checks
+            expect(mockedLogAgentStart).toHaveBeenCalledTimes(1);
+            expect(mockedLogAgentStart).toHaveBeenCalledWith(mockConfig, expect.any(AgentStartEvent));
+            expect(mockedLogAgentFinish).toHaveBeenCalledTimes(1);
+            expect(mockedLogAgentFinish).toHaveBeenCalledWith(mockConfig, expect.any(AgentFinishEvent));
+            const finishEvent = mockedLogAgentFinish.mock.calls[0][1];
+            expect(finishEvent.terminate_reason).toBe(AgentTerminateMode.GOAL);
+            // Context checks
+            expect(mockedPromptIdContext.run).toHaveBeenCalledTimes(2); // Two turns
+            const agentId = executor['agentId'];
+            expect(mockedPromptIdContext.run).toHaveBeenNthCalledWith(1, `${agentId}#0`, expect.any(Function));
+            expect(mockedPromptIdContext.run).toHaveBeenNthCalledWith(2, `${agentId}#1`, expect.any(Function));
+            expect(activities).toEqual(expect.arrayContaining([
+                expect.objectContaining({
+                    type: 'THOUGHT_CHUNK',
+                    data: { text: 'T1: Listing' },
+                }),
+                expect.objectContaining({
+                    type: 'TOOL_CALL_END',
+                    data: { name: LS_TOOL_NAME, output: 'file1.txt' },
+                }),
+                expect.objectContaining({
+                    type: 'TOOL_CALL_START',
+                    data: {
+                        name: TASK_COMPLETE_TOOL_NAME,
+                        args: { finalResult: 'Found file1.txt' },
+                    },
+                }),
+                expect.objectContaining({
+                    type: 'TOOL_CALL_END',
+                    data: {
+                        name: TASK_COMPLETE_TOOL_NAME,
+                        output: expect.stringContaining('Output submitted'),
+                    },
+                }),
+            ]));
+        });
+        it('should execute successfully when model calls complete_task without output (Happy Path No Output)', async () => {
+            const definition = createTestDefinition([LS_TOOL_NAME], {}, 'none');
+            const executor = await AgentExecutor.create(definition, mockConfig, onActivity);
+            mockModelResponse([
+                { name: LS_TOOL_NAME, args: { path: '.' }, id: 'call1' },
+            ]);
+            mockExecuteToolCall.mockResolvedValueOnce({
+                status: 'success',
+                request: {
+                    callId: 'call1',
+                    name: LS_TOOL_NAME,
+                    args: { path: '.' },
+                    isClientInitiated: false,
+                    prompt_id: 'test-prompt',
+                },
+                tool: {},
+                invocation: {},
+                response: {
+                    callId: 'call1',
+                    resultDisplay: 'ok',
+                    responseParts: [
+                        {
+                            functionResponse: {
+                                name: LS_TOOL_NAME,
+                                response: {},
+                                id: 'call1',
+                            },
+                        },
+                    ],
+                    error: undefined,
+                    errorType: undefined,
+                    contentLength: undefined,
+                },
+            });
+            mockModelResponse([{ name: TASK_COMPLETE_TOOL_NAME, args: {}, id: 'call2' }], 'Task finished.');
+            const output = await executor.run({ goal: 'Do work' }, signal);
+            const { modelConfigKey } = getMockMessageParams(0);
+            expect(modelConfigKey.model).toBe(getModelConfigAlias(definition));
+            const call = mockSetTools.mock.calls[0];
+            const sentTools = call[0][0].functionDeclarations;
+            expect(sentTools).toBeDefined();
+            const completeToolDef = sentTools.find((t) => t.name === TASK_COMPLETE_TOOL_NAME);
+            expect(completeToolDef?.parameters?.required).toEqual([]);
+            expect(completeToolDef?.description).toContain('signal that you have completed');
+            expect(output.result).toBe('Task completed successfully.');
+            expect(output.terminate_reason).toBe(AgentTerminateMode.GOAL);
+        });
+        it('should error immediately if the model stops tools without calling complete_task (Protocol Violation)', async () => {
+            const definition = createTestDefinition();
+            const executor = await AgentExecutor.create(definition, mockConfig, onActivity);
+            mockModelResponse([
+                { name: LS_TOOL_NAME, args: { path: '.' }, id: 'call1' },
+            ]);
+            mockExecuteToolCall.mockResolvedValueOnce({
+                status: 'success',
+                request: {
+                    callId: 'call1',
+                    name: LS_TOOL_NAME,
+                    args: { path: '.' },
+                    isClientInitiated: false,
+                    prompt_id: 'test-prompt',
+                },
+                tool: {},
+                invocation: {},
+                response: {
+                    callId: 'call1',
+                    resultDisplay: 'ok',
+                    responseParts: [
+                        {
+                            functionResponse: {
+                                name: LS_TOOL_NAME,
+                                response: {},
+                                id: 'call1',
+                            },
+                        },
+                    ],
+                    error: undefined,
+                    errorType: undefined,
+                    contentLength: undefined,
+                },
+            });
+            // Turn 2 (protocol violation)
+            mockModelResponse([], 'I think I am done.');
+            // Turn 3 (recovery turn - also fails)
+            mockModelResponse([], 'I still give up.');
+            const output = await executor.run({ goal: 'Strict test' }, signal);
+            expect(mockSendMessageStream).toHaveBeenCalledTimes(3);
+            const expectedError = `Agent stopped calling tools but did not call '${TASK_COMPLETE_TOOL_NAME}'.`;
+            expect(output.terminate_reason).toBe(AgentTerminateMode.ERROR_NO_COMPLETE_TASK_CALL);
+            expect(output.result).toBe(expectedError);
+            // Telemetry check for error
+            expect(mockedLogAgentFinish).toHaveBeenCalledWith(mockConfig, expect.objectContaining({
+                terminate_reason: AgentTerminateMode.ERROR_NO_COMPLETE_TASK_CALL,
+            }));
+            expect(activities).toContainEqual(expect.objectContaining({
+                type: 'ERROR',
+                data: expect.objectContaining({
+                    context: 'protocol_violation',
+                    error: expectedError,
+                }),
+            }));
+        });
+        it('should report an error if complete_task is called with missing required arguments', async () => {
+            const definition = createTestDefinition();
+            const executor = await AgentExecutor.create(definition, mockConfig, onActivity);
+            // Turn 1: Missing arg
+            mockModelResponse([
+                {
+                    name: TASK_COMPLETE_TOOL_NAME,
+                    args: { wrongArg: 'oops' },
+                    id: 'call1',
+                },
+            ]);
+            // Turn 2: Corrected
+            mockModelResponse([
+                {
+                    name: TASK_COMPLETE_TOOL_NAME,
+                    args: { finalResult: 'Corrected result' },
+                    id: 'call2',
+                },
+            ]);
+            const output = await executor.run({ goal: 'Error test' }, signal);
+            expect(mockSendMessageStream).toHaveBeenCalledTimes(2);
+            const expectedError = "Missing required argument 'finalResult' for completion.";
+            expect(activities).toContainEqual(expect.objectContaining({
+                type: 'ERROR',
+                data: {
+                    context: 'tool_call',
+                    name: TASK_COMPLETE_TOOL_NAME,
+                    error: expectedError,
+                },
+            }));
+            const turn2Params = getMockMessageParams(1);
+            const turn2Parts = turn2Params.message;
+            expect(turn2Parts).toBeDefined();
+            expect(turn2Parts).toHaveLength(1);
+            expect(turn2Parts[0]).toEqual(expect.objectContaining({
+                functionResponse: expect.objectContaining({
+                    name: TASK_COMPLETE_TOOL_NAME,
+                    response: { error: expectedError },
+                    id: 'call1',
+                }),
+            }));
+            expect(output.result).toBe('Corrected result');
+            expect(output.terminate_reason).toBe(AgentTerminateMode.GOAL);
+        });
+        it('should handle multiple calls to complete_task in the same turn (accept first, block rest)', async () => {
+            const definition = createTestDefinition([], {}, 'none');
+            const executor = await AgentExecutor.create(definition, mockConfig, onActivity);
+            // Turn 1: Duplicate calls
+            mockModelResponse([
+                { name: TASK_COMPLETE_TOOL_NAME, args: {}, id: 'call1' },
+                { name: TASK_COMPLETE_TOOL_NAME, args: {}, id: 'call2' },
+            ]);
+            const output = await executor.run({ goal: 'Dup test' }, signal);
+            expect(mockSendMessageStream).toHaveBeenCalledTimes(1);
+            expect(output.terminate_reason).toBe(AgentTerminateMode.GOAL);
+            const completions = activities.filter((a) => a.type === 'TOOL_CALL_END' &&
+                a.data['name'] === TASK_COMPLETE_TOOL_NAME);
+            const errors = activities.filter((a) => a.type === 'ERROR' && a.data['name'] === TASK_COMPLETE_TOOL_NAME);
+            expect(completions).toHaveLength(1);
+            expect(errors).toHaveLength(1);
+            expect(errors[0].data['error']).toContain('Task already marked complete in this turn');
+        });
+        it('should execute parallel tool calls and then complete', async () => {
+            const definition = createTestDefinition([LS_TOOL_NAME]);
+            const executor = await AgentExecutor.create(definition, mockConfig, onActivity);
+            const call1 = {
+                name: LS_TOOL_NAME,
+                args: { path: '/a' },
+                id: 'c1',
+            };
+            const call2 = {
+                name: LS_TOOL_NAME,
+                args: { path: '/b' },
+                id: 'c2',
+            };
+            // Turn 1: Parallel calls
+            mockModelResponse([call1, call2]);
+            // Concurrency mock
+            let callsStarted = 0;
+            let resolveCalls;
+            const bothStarted = new Promise((r) => {
+                resolveCalls = r;
+            });
+            mockExecuteToolCall.mockImplementation(async (_ctx, reqInfo) => {
+                callsStarted++;
+                if (callsStarted === 2)
+                    resolveCalls();
+                await vi.advanceTimersByTimeAsync(100);
+                return {
+                    status: 'success',
+                    request: reqInfo,
+                    tool: {},
+                    invocation: {},
+                    response: {
+                        callId: reqInfo.callId,
+                        resultDisplay: 'ok',
+                        responseParts: [
+                            {
+                                functionResponse: {
+                                    name: reqInfo.name,
+                                    response: {},
+                                    id: reqInfo.callId,
+                                },
+                            },
+                        ],
+                        error: undefined,
+                        errorType: undefined,
+                        contentLength: undefined,
+                    },
+                };
+            });
+            // Turn 2: Completion
+            mockModelResponse([
+                {
+                    name: TASK_COMPLETE_TOOL_NAME,
+                    args: { finalResult: 'done' },
+                    id: 'c3',
+                },
+            ]);
+            const runPromise = executor.run({ goal: 'Parallel' }, signal);
+            await vi.advanceTimersByTimeAsync(1);
+            await bothStarted;
+            await vi.advanceTimersByTimeAsync(150);
+            await vi.advanceTimersByTimeAsync(1);
+            const output = await runPromise;
+            expect(mockExecuteToolCall).toHaveBeenCalledTimes(2);
+            expect(output.terminate_reason).toBe(AgentTerminateMode.GOAL);
+            // Safe access to message parts
+            const turn2Params = getMockMessageParams(1);
+            const parts = turn2Params.message;
+            expect(parts).toBeDefined();
+            expect(parts).toHaveLength(2);
+            expect(parts).toEqual(expect.arrayContaining([
+                expect.objectContaining({
+                    functionResponse: expect.objectContaining({ id: 'c1' }),
+                }),
+                expect.objectContaining({
+                    functionResponse: expect.objectContaining({ id: 'c2' }),
+                }),
+            ]));
+        });
+        it('SECURITY: should block unauthorized tools and provide explicit failure to model', async () => {
+            const definition = createTestDefinition([LS_TOOL_NAME]);
+            const executor = await AgentExecutor.create(definition, mockConfig, onActivity);
+            // Turn 1: Model tries to use a tool not in its config
+            const badCallId = 'bad_call_1';
+            mockModelResponse([
+                {
+                    name: READ_FILE_TOOL_NAME,
+                    args: { path: 'secret.txt' },
+                    id: badCallId,
+                },
+            ]);
+            // Turn 2: Model gives up and completes
+            mockModelResponse([
+                {
+                    name: TASK_COMPLETE_TOOL_NAME,
+                    args: { finalResult: 'Could not read file.' },
+                    id: 'c2',
+                },
+            ]);
+            const consoleWarnSpy = vi
+                .spyOn(debugLogger, 'warn')
+                .mockImplementation(() => { });
+            await executor.run({ goal: 'Sec test' }, signal);
+            // Verify external executor was not called (Security held)
+            expect(mockExecuteToolCall).not.toHaveBeenCalled();
+            // 2. Verify console warning
+            expect(consoleWarnSpy).toHaveBeenCalledWith(expect.stringContaining(`[AgentExecutor] Blocked call:`));
+            consoleWarnSpy.mockRestore();
+            // Verify specific error was sent back to model
+            const turn2Params = getMockMessageParams(1);
+            const parts = turn2Params.message;
+            expect(parts).toBeDefined();
+            expect(parts[0]).toEqual(expect.objectContaining({
+                functionResponse: expect.objectContaining({
+                    id: badCallId,
+                    name: READ_FILE_TOOL_NAME,
+                    response: {
+                        error: expect.stringContaining('Unauthorized tool call'),
+                    },
+                }),
+            }));
+            // Verify Activity Stream reported the error
+            expect(activities).toContainEqual(expect.objectContaining({
+                type: 'ERROR',
+                data: expect.objectContaining({
+                    context: 'tool_call_unauthorized',
+                    name: READ_FILE_TOOL_NAME,
+                }),
+            }));
+        });
+    });
+    describe('Edge Cases and Error Handling', () => {
+        it('should report an error if complete_task output fails schema validation', async () => {
+            const definition = createTestDefinition([], {}, 'default', z.string().min(10));
+            const executor = await AgentExecutor.create(definition, mockConfig, onActivity);
+            // Turn 1: Invalid arg (too short)
+            mockModelResponse([
+                {
+                    name: TASK_COMPLETE_TOOL_NAME,
+                    args: { finalResult: 'short' },
+                    id: 'call1',
+                },
+            ]);
+            // Turn 2: Corrected
+            mockModelResponse([
+                {
+                    name: TASK_COMPLETE_TOOL_NAME,
+                    args: { finalResult: 'This is a much longer and valid result' },
+                    id: 'call2',
+                },
+            ]);
+            const output = await executor.run({ goal: 'Validation test' }, signal);
+            expect(mockSendMessageStream).toHaveBeenCalledTimes(2);
+            const expectedError = 'Output validation failed: {"formErrors":["String must contain at least 10 character(s)"],"fieldErrors":{}}';
+            // Check that the error was reported in the activity stream
+            expect(activities).toContainEqual(expect.objectContaining({
+                type: 'ERROR',
+                data: {
+                    context: 'tool_call',
+                    name: TASK_COMPLETE_TOOL_NAME,
+                    error: expect.stringContaining('Output validation failed'),
+                },
+            }));
+            // Check that the error was sent back to the model for the next turn
+            const turn2Params = getMockMessageParams(1);
+            const turn2Parts = turn2Params.message;
+            expect(turn2Parts).toEqual([
+                expect.objectContaining({
+                    functionResponse: expect.objectContaining({
+                        name: TASK_COMPLETE_TOOL_NAME,
+                        response: { error: expectedError },
+                        id: 'call1',
+                    }),
+                }),
+            ]);
+            // Check that the agent eventually succeeded
+            expect(output.result).toContain('This is a much longer and valid result');
+            expect(output.terminate_reason).toBe(AgentTerminateMode.GOAL);
+        });
+        it('should throw and log if GeminiChat creation fails', async () => {
+            const definition = createTestDefinition();
+            const initError = new Error('Chat creation failed');
+            MockedGeminiChat.mockImplementationOnce(() => {
+                throw initError;
+            });
+            // We expect the error to be thrown during the run, not creation
+            const executor = await AgentExecutor.create(definition, mockConfig, onActivity);
+            await expect(executor.run({ goal: 'test' }, signal)).rejects.toThrow(`Failed to create chat object: ${initError}`);
+            // Ensure the error was reported via the activity callback
+            expect(activities).toContainEqual(expect.objectContaining({
+                type: 'ERROR',
+                data: expect.objectContaining({
+                    error: `Error: Failed to create chat object: ${initError}`,
+                }),
+            }));
+            // Ensure the agent run was logged as a failure
+            expect(mockedLogAgentFinish).toHaveBeenCalledWith(mockConfig, expect.objectContaining({
+                terminate_reason: AgentTerminateMode.ERROR,
+            }));
+        });
+        it('should handle a failed tool call and feed the error to the model', async () => {
+            const definition = createTestDefinition([LS_TOOL_NAME]);
+            const executor = await AgentExecutor.create(definition, mockConfig, onActivity);
+            const toolErrorMessage = 'Tool failed spectacularly';
+            // Turn 1: Model calls a tool that will fail
+            mockModelResponse([
+                { name: LS_TOOL_NAME, args: { path: '/fake' }, id: 'call1' },
+            ]);
+            mockExecuteToolCall.mockResolvedValueOnce({
+                status: 'error',
+                request: {
+                    callId: 'call1',
+                    name: LS_TOOL_NAME,
+                    args: { path: '/fake' },
+                    isClientInitiated: false,
+                    prompt_id: 'test-prompt',
+                },
+                tool: {},
+                invocation: {},
+                response: {
+                    callId: 'call1',
+                    resultDisplay: '',
+                    responseParts: [
+                        {
+                            functionResponse: {
+                                name: LS_TOOL_NAME,
+                                response: { error: toolErrorMessage },
+                                id: 'call1',
+                            },
+                        },
+                    ],
+                    error: {
+                        type: 'ToolError',
+                        message: toolErrorMessage,
+                    },
+                    errorType: 'ToolError',
+                    contentLength: 0,
+                },
+            });
+            // Turn 2: Model sees the error and completes
+            mockModelResponse([
+                {
+                    name: TASK_COMPLETE_TOOL_NAME,
+                    args: { finalResult: 'Aborted due to tool failure.' },
+                    id: 'call2',
+                },
+            ]);
+            const output = await executor.run({ goal: 'Tool failure test' }, signal);
+            expect(mockExecuteToolCall).toHaveBeenCalledTimes(1);
+            expect(mockSendMessageStream).toHaveBeenCalledTimes(2);
+            // Verify the error was reported in the activity stream
+            expect(activities).toContainEqual(expect.objectContaining({
+                type: 'ERROR',
+                data: {
+                    context: 'tool_call',
+                    name: LS_TOOL_NAME,
+                    error: toolErrorMessage,
+                },
+            }));
+            // Verify the error was sent back to the model
+            const turn2Params = getMockMessageParams(1);
+            const parts = turn2Params.message;
+            expect(parts).toEqual([
+                expect.objectContaining({
+                    functionResponse: expect.objectContaining({
+                        name: LS_TOOL_NAME,
+                        id: 'call1',
+                        response: {
+                            error: toolErrorMessage,
+                        },
+                    }),
+                }),
+            ]);
+            expect(output.terminate_reason).toBe(AgentTerminateMode.GOAL);
+            expect(output.result).toBe('Aborted due to tool failure.');
+        });
+    });
+    describe('run (Termination Conditions)', () => {
+        const mockWorkResponse = (id) => {
+            mockModelResponse([{ name: LS_TOOL_NAME, args: { path: '.' }, id }]);
+            mockExecuteToolCall.mockResolvedValueOnce({
+                status: 'success',
+                request: {
+                    callId: id,
+                    name: LS_TOOL_NAME,
+                    args: { path: '.' },
+                    isClientInitiated: false,
+                    prompt_id: 'test-prompt',
+                },
+                tool: {},
+                invocation: {},
+                response: {
+                    callId: id,
+                    resultDisplay: 'ok',
+                    responseParts: [
+                        { functionResponse: { name: LS_TOOL_NAME, response: {}, id } },
+                    ],
+                    error: undefined,
+                    errorType: undefined,
+                    contentLength: undefined,
+                },
+            });
+        };
+        it('should terminate when max_turns is reached', async () => {
+            const MAX = 2;
+            const definition = createTestDefinition([LS_TOOL_NAME], {
+                max_turns: MAX,
+            });
+            const executor = await AgentExecutor.create(definition, mockConfig);
+            mockWorkResponse('t1');
+            mockWorkResponse('t2');
+            // Recovery turn
+            mockModelResponse([], 'I give up');
+            const output = await executor.run({ goal: 'Turns test' }, signal);
+            expect(output.terminate_reason).toBe(AgentTerminateMode.MAX_TURNS);
+            expect(mockSendMessageStream).toHaveBeenCalledTimes(MAX + 1);
+        });
+        it('should terminate with TIMEOUT if a model call takes too long', async () => {
+            const definition = createTestDefinition([LS_TOOL_NAME], {
+                max_time_minutes: 0.5, // 30 seconds
+            });
+            const executor = await AgentExecutor.create(definition, mockConfig, onActivity);
+            // Mock a model call that is interruptible by an abort signal.
+            mockSendMessageStream.mockImplementationOnce(async (_key, _message, _promptId, signal) => 
+            // eslint-disable-next-line require-yield
+            (async function* () {
+                await new Promise((resolve) => {
+                    // This promise resolves when aborted, ending the generator.
+                    signal?.addEventListener('abort', () => {
+                        resolve();
+                    });
+                });
+            })());
+            // Recovery turn
+            mockModelResponse([], 'I give up');
+            const runPromise = executor.run({ goal: 'Timeout test' }, signal);
+            // Advance time past the timeout to trigger the abort.
+            await vi.advanceTimersByTimeAsync(31 * 1000);
+            const output = await runPromise;
+            expect(output.terminate_reason).toBe(AgentTerminateMode.TIMEOUT);
+            expect(output.result).toContain('Agent timed out after 0.5 minutes.');
+            expect(mockSendMessageStream).toHaveBeenCalledTimes(2);
+            // Verify activity stream reported the timeout
+            expect(activities).toContainEqual(expect.objectContaining({
+                type: 'ERROR',
+                data: expect.objectContaining({
+                    context: 'timeout',
+                    error: 'Agent timed out after 0.5 minutes.',
+                }),
+            }));
+            // Verify telemetry
+            expect(mockedLogAgentFinish).toHaveBeenCalledWith(mockConfig, expect.objectContaining({
+                terminate_reason: AgentTerminateMode.TIMEOUT,
+            }));
+        });
+        it('should terminate with TIMEOUT if a tool call takes too long', async () => {
+            const definition = createTestDefinition([LS_TOOL_NAME], {
+                max_time_minutes: 1,
+            });
+            const executor = await AgentExecutor.create(definition, mockConfig);
+            mockModelResponse([
+                { name: LS_TOOL_NAME, args: { path: '.' }, id: 't1' },
+            ]);
+            // Long running tool
+            mockExecuteToolCall.mockImplementationOnce(async (_ctx, reqInfo) => {
+                await vi.advanceTimersByTimeAsync(61 * 1000);
+                return {
+                    status: 'success',
+                    request: reqInfo,
+                    tool: {},
+                    invocation: {},
+                    response: {
+                        callId: 't1',
+                        resultDisplay: 'ok',
+                        responseParts: [],
+                        error: undefined,
+                        errorType: undefined,
+                        contentLength: undefined,
+                    },
+                };
+            });
+            // Recovery turn
+            mockModelResponse([], 'I give up');
+            const output = await executor.run({ goal: 'Timeout test' }, signal);
+            expect(output.terminate_reason).toBe(AgentTerminateMode.TIMEOUT);
+            expect(mockSendMessageStream).toHaveBeenCalledTimes(2);
+        });
+        it('should terminate when AbortSignal is triggered', async () => {
+            const definition = createTestDefinition();
+            const executor = await AgentExecutor.create(definition, mockConfig);
+            mockSendMessageStream.mockImplementationOnce(async () => (async function* () {
+                yield {
+                    type: StreamEventType.CHUNK,
+                    value: createMockResponseChunk([
+                        { text: 'Thinking...', thought: true },
+                    ]),
+                };
+                abortController.abort();
+            })());
+            const output = await executor.run({ goal: 'Abort test' }, signal);
+            expect(output.terminate_reason).toBe(AgentTerminateMode.ABORTED);
+        });
+    });
+    describe('run (Recovery Turns)', () => {
+        const mockWorkResponse = (id) => {
+            mockModelResponse([{ name: LS_TOOL_NAME, args: { path: '.' }, id }]);
+            mockExecuteToolCall.mockResolvedValueOnce({
+                status: 'success',
+                request: {
+                    callId: id,
+                    name: LS_TOOL_NAME,
+                    args: { path: '.' },
+                    isClientInitiated: false,
+                    prompt_id: 'test-prompt',
+                },
+                tool: {},
+                invocation: {},
+                response: {
+                    callId: id,
+                    resultDisplay: 'ok',
+                    responseParts: [
+                        { functionResponse: { name: LS_TOOL_NAME, response: {}, id } },
+                    ],
+                    error: undefined,
+                    errorType: undefined,
+                    contentLength: undefined,
+                },
+            });
+        };
+        it('should recover successfully if complete_task is called during the grace turn after MAX_TURNS', async () => {
+            const MAX = 1;
+            const definition = createTestDefinition([LS_TOOL_NAME], {
+                max_turns: MAX,
+            });
+            const executor = await AgentExecutor.create(definition, mockConfig, onActivity);
+            // Turn 1 (hits max_turns)
+            mockWorkResponse('t1');
+            // Recovery Turn (succeeds)
+            mockModelResponse([
+                {
+                    name: TASK_COMPLETE_TOOL_NAME,
+                    args: { finalResult: 'Recovered!' },
+                    id: 't2',
+                },
+            ], 'Recovering from max turns');
+            const output = await executor.run({ goal: 'Turns recovery' }, signal);
+            expect(output.terminate_reason).toBe(AgentTerminateMode.GOAL);
+            expect(output.result).toBe('Recovered!');
+            expect(mockSendMessageStream).toHaveBeenCalledTimes(MAX + 1); // 1 regular + 1 recovery
+            expect(activities).toContainEqual(expect.objectContaining({
+                type: 'THOUGHT_CHUNK',
+                data: {
+                    text: 'Execution limit reached (MAX_TURNS). Attempting one final recovery turn with a grace period.',
+                },
+            }));
+            expect(activities).toContainEqual(expect.objectContaining({
+                type: 'THOUGHT_CHUNK',
+                data: { text: 'Graceful recovery succeeded.' },
+            }));
+        });
+        it('should fail if complete_task is NOT called during the grace turn after MAX_TURNS', async () => {
+            const MAX = 1;
+            const definition = createTestDefinition([LS_TOOL_NAME], {
+                max_turns: MAX,
+            });
+            const executor = await AgentExecutor.create(definition, mockConfig, onActivity);
+            // Turn 1 (hits max_turns)
+            mockWorkResponse('t1');
+            // Recovery Turn (fails by calling no tools)
+            mockModelResponse([], 'I give up again.');
+            const output = await executor.run({ goal: 'Turns recovery fail' }, signal);
+            expect(output.terminate_reason).toBe(AgentTerminateMode.MAX_TURNS);
+            expect(output.result).toContain('Agent reached max turns limit');
+            expect(mockSendMessageStream).toHaveBeenCalledTimes(MAX + 1);
+            expect(activities).toContainEqual(expect.objectContaining({
+                type: 'ERROR',
+                data: expect.objectContaining({
+                    context: 'recovery_turn',
+                    error: 'Graceful recovery attempt failed. Reason: stop',
+                }),
+            }));
+        });
+        it('should recover successfully from a protocol violation (no complete_task)', async () => {
+            const definition = createTestDefinition();
+            const executor = await AgentExecutor.create(definition, mockConfig, onActivity);
+            // Turn 1: Normal work
+            mockWorkResponse('t1');
+            // Turn 2: Protocol violation (no tool calls)
+            mockModelResponse([], 'I think I am done, but I forgot the right tool.');
+            // Turn 3: Recovery turn (succeeds)
+            mockModelResponse([
+                {
+                    name: TASK_COMPLETE_TOOL_NAME,
+                    args: { finalResult: 'Recovered from violation!' },
+                    id: 't3',
+                },
+            ], 'My mistake, here is the completion.');
+            const output = await executor.run({ goal: 'Violation recovery' }, signal);
+            expect(mockSendMessageStream).toHaveBeenCalledTimes(3);
+            expect(output.terminate_reason).toBe(AgentTerminateMode.GOAL);
+            expect(output.result).toBe('Recovered from violation!');
+            expect(activities).toContainEqual(expect.objectContaining({
+                type: 'THOUGHT_CHUNK',
+                data: {
+                    text: 'Execution limit reached (ERROR_NO_COMPLETE_TASK_CALL). Attempting one final recovery turn with a grace period.',
+                },
+            }));
+        });
+        it('should fail recovery from a protocol violation if it violates again', async () => {
+            const definition = createTestDefinition();
+            const executor = await AgentExecutor.create(definition, mockConfig, onActivity);
+            // Turn 1: Normal work
+            mockWorkResponse('t1');
+            // Turn 2: Protocol violation (no tool calls)
+            mockModelResponse([], 'I think I am done, but I forgot the right tool.');
+            // Turn 3: Recovery turn (fails again)
+            mockModelResponse([], 'I still dont know what to do.');
+            const output = await executor.run({ goal: 'Violation recovery fail' }, signal);
+            expect(mockSendMessageStream).toHaveBeenCalledTimes(3);
+            expect(output.terminate_reason).toBe(AgentTerminateMode.ERROR_NO_COMPLETE_TASK_CALL);
+            expect(output.result).toContain(`Agent stopped calling tools but did not call '${TASK_COMPLETE_TOOL_NAME}'`);
+            expect(activities).toContainEqual(expect.objectContaining({
+                type: 'ERROR',
+                data: expect.objectContaining({
+                    context: 'recovery_turn',
+                    error: 'Graceful recovery attempt failed. Reason: stop',
+                }),
+            }));
+        });
+        it('should recover successfully from a TIMEOUT', async () => {
+            const definition = createTestDefinition([LS_TOOL_NAME], {
+                max_time_minutes: 0.5, // 30 seconds
+            });
+            const executor = await AgentExecutor.create(definition, mockConfig, onActivity);
+            // Mock a model call that gets interrupted by the timeout.
+            mockSendMessageStream.mockImplementationOnce(async (_key, _message, _promptId, signal) => 
+            // eslint-disable-next-line require-yield
+            (async function* () {
+                // This promise never resolves, it waits for abort.
+                await new Promise((resolve) => {
+                    signal?.addEventListener('abort', () => resolve());
+                });
+            })());
+            // Recovery turn (succeeds)
+            mockModelResponse([
+                {
+                    name: TASK_COMPLETE_TOOL_NAME,
+                    args: { finalResult: 'Recovered from timeout!' },
+                    id: 't2',
+                },
+            ], 'Apologies for the delay, finishing up.');
+            const runPromise = executor.run({ goal: 'Timeout recovery' }, signal);
+            // Advance time past the timeout to trigger the abort and recovery.
+            await vi.advanceTimersByTimeAsync(31 * 1000);
+            const output = await runPromise;
+            expect(mockSendMessageStream).toHaveBeenCalledTimes(2); // 1 failed + 1 recovery
+            expect(output.terminate_reason).toBe(AgentTerminateMode.GOAL);
+            expect(output.result).toBe('Recovered from timeout!');
+            expect(activities).toContainEqual(expect.objectContaining({
+                type: 'THOUGHT_CHUNK',
+                data: {
+                    text: 'Execution limit reached (TIMEOUT). Attempting one final recovery turn with a grace period.',
+                },
+            }));
+        });
+        it('should fail recovery from a TIMEOUT if the grace period also times out', async () => {
+            const definition = createTestDefinition([LS_TOOL_NAME], {
+                max_time_minutes: 0.5, // 30 seconds
+            });
+            const executor = await AgentExecutor.create(definition, mockConfig, onActivity);
+            mockSendMessageStream.mockImplementationOnce(async (_key, _message, _promptId, signal) => 
+            // eslint-disable-next-line require-yield
+            (async function* () {
+                await new Promise((resolve) => signal?.addEventListener('abort', () => resolve()));
+            })());
+            // Mock the recovery call to also be long-running
+            mockSendMessageStream.mockImplementationOnce(async (_key, _message, _promptId, signal) => 
+            // eslint-disable-next-line require-yield
+            (async function* () {
+                await new Promise((resolve) => signal?.addEventListener('abort', () => resolve()));
+            })());
+            const runPromise = executor.run({ goal: 'Timeout recovery fail' }, signal);
+            // 1. Trigger the main timeout
+            await vi.advanceTimersByTimeAsync(31 * 1000);
+            // 2. Let microtasks run (start recovery turn)
+            await vi.advanceTimersByTimeAsync(1);
+            // 3. Trigger the grace period timeout (60s)
+            await vi.advanceTimersByTimeAsync(61 * 1000);
+            const output = await runPromise;
+            expect(mockSendMessageStream).toHaveBeenCalledTimes(2);
+            expect(output.terminate_reason).toBe(AgentTerminateMode.TIMEOUT);
+            expect(output.result).toContain('Agent timed out after 0.5 minutes.');
+            expect(activities).toContainEqual(expect.objectContaining({
+                type: 'ERROR',
+                data: expect.objectContaining({
+                    context: 'recovery_turn',
+                    error: 'Graceful recovery attempt failed. Reason: stop',
+                }),
+            }));
+        });
+    });
+    describe('Telemetry and Logging', () => {
+        const mockWorkResponse = (id) => {
+            mockModelResponse([{ name: LS_TOOL_NAME, args: { path: '.' }, id }]);
+            mockExecuteToolCall.mockResolvedValueOnce({
+                status: 'success',
+                request: {
+                    callId: id,
+                    name: LS_TOOL_NAME,
+                    args: { path: '.' },
+                    isClientInitiated: false,
+                    prompt_id: 'test-prompt',
+                },
+                tool: {},
+                invocation: {},
+                response: {
+                    callId: id,
+                    resultDisplay: 'ok',
+                    responseParts: [
+                        { functionResponse: { name: LS_TOOL_NAME, response: {}, id } },
+                    ],
+                    error: undefined,
+                    errorType: undefined,
+                    contentLength: undefined,
+                },
+            });
+        };
+        beforeEach(() => {
+            mockedLogRecoveryAttempt.mockClear();
+        });
+        it('should log a RecoveryAttemptEvent when a recoverable error occurs and recovery fails', async () => {
+            const MAX = 1;
+            const definition = createTestDefinition([LS_TOOL_NAME], {
+                max_turns: MAX,
+            });
+            const executor = await AgentExecutor.create(definition, mockConfig);
+            // Turn 1 (hits max_turns)
+            mockWorkResponse('t1');
+            // Recovery Turn (fails by calling no tools)
+            mockModelResponse([], 'I give up again.');
+            await executor.run({ goal: 'Turns recovery fail' }, signal);
+            expect(mockedLogRecoveryAttempt).toHaveBeenCalledTimes(1);
+            const recoveryEvent = mockedLogRecoveryAttempt.mock.calls[0][1];
+            expect(recoveryEvent).toBeInstanceOf(RecoveryAttemptEvent);
+            expect(recoveryEvent.agent_name).toBe(definition.name);
+            expect(recoveryEvent.reason).toBe(AgentTerminateMode.MAX_TURNS);
+            expect(recoveryEvent.success).toBe(false);
+            expect(recoveryEvent.turn_count).toBe(1);
+            expect(recoveryEvent.duration_ms).toBeGreaterThanOrEqual(0);
+        });
+        it('should log a successful RecoveryAttemptEvent when recovery succeeds', async () => {
+            const MAX = 1;
+            const definition = createTestDefinition([LS_TOOL_NAME], {
+                max_turns: MAX,
+            });
+            const executor = await AgentExecutor.create(definition, mockConfig);
+            // Turn 1 (hits max_turns)
+            mockWorkResponse('t1');
+            // Recovery Turn (succeeds)
+            mockModelResponse([
+                {
+                    name: TASK_COMPLETE_TOOL_NAME,
+                    args: { finalResult: 'Recovered!' },
+                    id: 't2',
+                },
+            ], 'Recovering from max turns');
+            await executor.run({ goal: 'Turns recovery success' }, signal);
+            expect(mockedLogRecoveryAttempt).toHaveBeenCalledTimes(1);
+            const recoveryEvent = mockedLogRecoveryAttempt.mock.calls[0][1];
+            expect(recoveryEvent).toBeInstanceOf(RecoveryAttemptEvent);
+            expect(recoveryEvent.success).toBe(true);
+            expect(recoveryEvent.reason).toBe(AgentTerminateMode.MAX_TURNS);
+        });
+    });
+    describe('Chat Compression', () => {
+        const mockWorkResponse = (id) => {
+            mockModelResponse([{ name: LS_TOOL_NAME, args: { path: '.' }, id }]);
+            mockExecuteToolCall.mockResolvedValueOnce({
+                status: 'success',
+                request: {
+                    callId: id,
+                    name: LS_TOOL_NAME,
+                    args: { path: '.' },
+                    isClientInitiated: false,
+                    prompt_id: 'test-prompt',
+                },
+                tool: {},
+                invocation: {},
+                response: {
+                    callId: id,
+                    resultDisplay: 'ok',
+                    responseParts: [
+                        { functionResponse: { name: LS_TOOL_NAME, response: {}, id } },
+                    ],
+                    error: undefined,
+                    errorType: undefined,
+                    contentLength: undefined,
+                },
+            });
+        };
+        it('should attempt to compress chat history on each turn', async () => {
+            const definition = createTestDefinition();
+            const executor = await AgentExecutor.create(definition, mockConfig, onActivity);
+            // Mock compression to do nothing
+            mockCompress.mockResolvedValue({
+                newHistory: null,
+                info: { compressionStatus: CompressionStatus.NOOP },
+            });
+            // Turn 1
+            mockWorkResponse('t1');
+            // Turn 2: Complete
+            mockModelResponse([
+                {
+                    name: TASK_COMPLETE_TOOL_NAME,
+                    args: { finalResult: 'Done' },
+                    id: 'call2',
+                },
+            ], 'T2');
+            await executor.run({ goal: 'Compress test' }, signal);
+            expect(mockCompress).toHaveBeenCalledTimes(2);
+        });
+        it('should update chat history when compression is successful', async () => {
+            const definition = createTestDefinition();
+            const executor = await AgentExecutor.create(definition, mockConfig, onActivity);
+            const compressedHistory = [
+                { role: 'user', parts: [{ text: 'compressed' }] },
+            ];
+            mockCompress.mockResolvedValue({
+                newHistory: compressedHistory,
+                info: { compressionStatus: CompressionStatus.COMPRESSED },
+            });
+            // Turn 1: Complete
+            mockModelResponse([
+                {
+                    name: TASK_COMPLETE_TOOL_NAME,
+                    args: { finalResult: 'Done' },
+                    id: 'call1',
+                },
+            ], 'T1');
+            await executor.run({ goal: 'Compress success' }, signal);
+            expect(mockCompress).toHaveBeenCalledTimes(1);
+            expect(mockSetHistory).toHaveBeenCalledTimes(1);
+            expect(mockSetHistory).toHaveBeenCalledWith(compressedHistory);
+        });
+        it('should pass hasFailedCompressionAttempt=true to compression after a failure', async () => {
+            const definition = createTestDefinition();
+            const executor = await AgentExecutor.create(definition, mockConfig, onActivity);
+            // First call fails
+            mockCompress.mockResolvedValueOnce({
+                newHistory: null,
+                info: {
+                    compressionStatus: CompressionStatus.COMPRESSION_FAILED_INFLATED_TOKEN_COUNT,
+                },
+            });
+            // Second call is neutral
+            mockCompress.mockResolvedValueOnce({
+                newHistory: null,
+                info: { compressionStatus: CompressionStatus.NOOP },
+            });
+            // Turn 1
+            mockWorkResponse('t1');
+            // Turn 2: Complete
+            mockModelResponse([
+                {
+                    name: TASK_COMPLETE_TOOL_NAME,
+                    args: { finalResult: 'Done' },
+                    id: 't2',
+                },
+            ], 'T2');
+            await executor.run({ goal: 'Compress fail' }, signal);
+            expect(mockCompress).toHaveBeenCalledTimes(2);
+            // First call, hasFailedCompressionAttempt is false
+            expect(mockCompress.mock.calls[0][5]).toBe(false);
+            // Second call, hasFailedCompressionAttempt is true
+            expect(mockCompress.mock.calls[1][5]).toBe(true);
+        });
+        it('should reset hasFailedCompressionAttempt flag after a successful compression', async () => {
+            const definition = createTestDefinition();
+            const executor = await AgentExecutor.create(definition, mockConfig, onActivity);
+            const compressedHistory = [
+                { role: 'user', parts: [{ text: 'compressed' }] },
+            ];
+            // Turn 1: Fails
+            mockCompress.mockResolvedValueOnce({
+                newHistory: null,
+                info: {
+                    compressionStatus: CompressionStatus.COMPRESSION_FAILED_INFLATED_TOKEN_COUNT,
+                },
+            });
+            // Turn 2: Succeeds
+            mockCompress.mockResolvedValueOnce({
+                newHistory: compressedHistory,
+                info: { compressionStatus: CompressionStatus.COMPRESSED },
+            });
+            // Turn 3: Neutral
+            mockCompress.mockResolvedValueOnce({
+                newHistory: null,
+                info: { compressionStatus: CompressionStatus.NOOP },
+            });
+            // Turn 1
+            mockWorkResponse('t1');
+            // Turn 2
+            mockWorkResponse('t2');
+            // Turn 3: Complete
+            mockModelResponse([
+                {
+                    name: TASK_COMPLETE_TOOL_NAME,
+                    args: { finalResult: 'Done' },
+                    id: 't3',
+                },
+            ], 'T3');
+            await executor.run({ goal: 'Compress reset' }, signal);
+            expect(mockCompress).toHaveBeenCalledTimes(3);
+            // Call 1: hasFailed... is false
+            expect(mockCompress.mock.calls[0][5]).toBe(false);
+            // Call 2: hasFailed... is true
+            expect(mockCompress.mock.calls[1][5]).toBe(true);
+            // Call 3: hasFailed... is false again
+            expect(mockCompress.mock.calls[2][5]).toBe(false);
+            expect(mockSetHistory).toHaveBeenCalledTimes(1);
+            expect(mockSetHistory).toHaveBeenCalledWith(compressedHistory);
+        });
+    });
+});
+//# sourceMappingURL=executor.test.js.map