@office-ai/aioncli-core 0.2.2 → 0.8.1

package/dist/src/core/client.test.js

@@ -3,26 +3,46 @@
  * Copyright 2025 Google LLC
  * SPDX-License-Identifier: Apache-2.0
  */
-import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
-import { GoogleGenAI, } from '@google/genai';
-import { findIndexAfterFraction, GeminiClient } from './client.js';
-import { AuthType } from './contentGenerator.js';
-import { Config } from '../config/config.js';
-import { GeminiEventType, Turn } from './turn.js';
+import { describe, it, expect, vi, beforeEach, afterEach, } from 'vitest';
+import { findCompressSplitPoint, isThinkingDefault, isThinkingSupported, GeminiClient, } from './client.js';
+import { AuthType, } from './contentGenerator.js';
+import {} from './geminiChat.js';
+import { CompressionStatus, GeminiEventType, Turn, } from './turn.js';
 import { getCoreSystemPrompt } from './prompts.js';
 import { DEFAULT_GEMINI_FLASH_MODEL } from '../config/models.js';
 import { FileDiscoveryService } from '../services/fileDiscoveryService.js';
 import { setSimulate429 } from '../utils/testUtils.js';
 import { tokenLimit } from './tokenLimits.js';
-import { ideContext } from '../ide/ideContext.js';
+import { ideContextStore } from '../ide/ideContext.js';
 import { ClearcutLogger } from '../telemetry/clearcut-logger/clearcut-logger.js';
+import { uiTelemetryService } from '../telemetry/uiTelemetry.js';
+// Mock fs module to prevent actual file system operations during tests
+const mockFileSystem = new Map();
+vi.mock('node:fs', () => {
+    const fsModule = {
+        mkdirSync: vi.fn(),
+        writeFileSync: vi.fn((path, data) => {
+            mockFileSystem.set(path, data);
+        }),
+        readFileSync: vi.fn((path) => {
+            if (mockFileSystem.has(path)) {
+                return mockFileSystem.get(path);
+            }
+            throw Object.assign(new Error('ENOENT: no such file or directory'), {
+                code: 'ENOENT',
+            });
+        }),
+        existsSync: vi.fn((path) => mockFileSystem.has(path)),
+    };
+    return {
+        default: fsModule,
+        ...fsModule,
+    };
+});
 // --- Mocks ---
-const mockChatCreateFn = vi.fn();
-const mockGenerateContentFn = vi.fn();
-const mockEmbedContentFn = vi.fn();
 const mockTurnRunFn = vi.fn();
-vi.mock('@google/genai');
-vi.mock('./turn', () => {
+vi.mock('./turn', async (importOriginal) => {
+    const actual = await importOriginal();
     // Define a mock class that has the same shape as the real Turn
     class MockTurn {
         pendingToolCalls = [];
@@ -34,11 +54,8 @@ vi.mock('./turn', () => {
     }
     // Export the mock class as 'Turn'
     return {
+        ...actual,
         Turn: MockTurn,
-        GeminiEventType: {
-            MaxSessionTurns: 'MaxSessionTurns',
-            ChatCompressed: 'ChatCompressed',
-        },
     };
 });
 vi.mock('../config/config.js');
@@ -60,42 +77,78 @@ vi.mock('../telemetry/index.js', () => ({
     logApiError: vi.fn(),
 }));
 vi.mock('../ide/ideContext.js');
-describe('findIndexAfterFraction', () => {
-    const history = [
-        { role: 'user', parts: [{ text: 'This is the first message.' }] }, // JSON length: 66
-        { role: 'model', parts: [{ text: 'This is the second message.' }] }, // JSON length: 68
-        { role: 'user', parts: [{ text: 'This is the third message.' }] }, // JSON length: 66
-        { role: 'model', parts: [{ text: 'This is the fourth message.' }] }, // JSON length: 68
-        { role: 'user', parts: [{ text: 'This is the fifth message.' }] }, // JSON length: 65
-    ];
-    // Total length: 333
+vi.mock('../telemetry/uiTelemetry.js', () => ({
+    uiTelemetryService: {
+        setLastPromptTokenCount: vi.fn(),
+        getLastPromptTokenCount: vi.fn(),
+    },
+}));
+/**
+ * Array.fromAsync ponyfill, which will be available in es 2024.
+ *
+ * Buffers an async generator into an array and returns the result.
+ */
+async function fromAsync(promise) {
+    const results = [];
+    for await (const result of promise) {
+        results.push(result);
+    }
+    return results;
+}
+describe('findCompressSplitPoint', () => {
     it('should throw an error for non-positive numbers', () => {
-        expect(() => findIndexAfterFraction(history, 0)).toThrow('Fraction must be between 0 and 1');
+        expect(() => findCompressSplitPoint([], 0)).toThrow('Fraction must be between 0 and 1');
     });
     it('should throw an error for a fraction greater than or equal to 1', () => {
-        expect(() => findIndexAfterFraction(history, 1)).toThrow('Fraction must be between 0 and 1');
+        expect(() => findCompressSplitPoint([], 1)).toThrow('Fraction must be between 0 and 1');
+    });
+    it('should handle an empty history', () => {
+        expect(findCompressSplitPoint([], 0.5)).toBe(0);
     });
     it('should handle a fraction in the middle', () => {
-        // 333 * 0.5 = 166.5
-        // 0: 66
-        // 1: 66 + 68 = 134
-        // 2: 134 + 66 = 200
-        // 200 >= 166.5, so index is 2
-        expect(findIndexAfterFraction(history, 0.5)).toBe(2);
+        const history = [
+            { role: 'user', parts: [{ text: 'This is the first message.' }] }, // JSON length: 66 (19%)
+            { role: 'model', parts: [{ text: 'This is the second message.' }] }, // JSON length: 68 (40%)
+            { role: 'user', parts: [{ text: 'This is the third message.' }] }, // JSON length: 66 (60%)
+            { role: 'model', parts: [{ text: 'This is the fourth message.' }] }, // JSON length: 68 (80%)
+            { role: 'user', parts: [{ text: 'This is the fifth message.' }] }, // JSON length: 65 (100%)
+        ];
+        expect(findCompressSplitPoint(history, 0.5)).toBe(4);
     });
-    it('should handle a fraction that results in the last index', () => {
-        // 333 * 0.9 = 299.7
-        // ...
-        // 3: 200 + 68 = 268
-        // 4: 268 + 65 = 333
-        // 333 >= 299.7, so index is 4
-        expect(findIndexAfterFraction(history, 0.9)).toBe(4);
+    it('should handle a fraction of last index', () => {
+        const history = [
+            { role: 'user', parts: [{ text: 'This is the first message.' }] }, // JSON length: 66 (19%)
+            { role: 'model', parts: [{ text: 'This is the second message.' }] }, // JSON length: 68 (40%)
+            { role: 'user', parts: [{ text: 'This is the third message.' }] }, // JSON length: 66 (60%)
+            { role: 'model', parts: [{ text: 'This is the fourth message.' }] }, // JSON length: 68 (80%)
+            { role: 'user', parts: [{ text: 'This is the fifth message.' }] }, // JSON length: 65 (100%)
+        ];
+        expect(findCompressSplitPoint(history, 0.9)).toBe(4);
     });
-    it('should handle an empty history', () => {
-        expect(findIndexAfterFraction([], 0.5)).toBe(0);
+    it('should handle a fraction of after last index', () => {
+        const history = [
+            { role: 'user', parts: [{ text: 'This is the first message.' }] }, // JSON length: 66 (24%%)
+            { role: 'model', parts: [{ text: 'This is the second message.' }] }, // JSON length: 68 (50%)
+            { role: 'user', parts: [{ text: 'This is the third message.' }] }, // JSON length: 66 (74%)
+            { role: 'model', parts: [{ text: 'This is the fourth message.' }] }, // JSON length: 68 (100%)
+        ];
+        expect(findCompressSplitPoint(history, 0.8)).toBe(4);
+    });
+    it('should return earlier splitpoint if no valid ones are after threshhold', () => {
+        const history = [
+            { role: 'user', parts: [{ text: 'This is the first message.' }] },
+            { role: 'model', parts: [{ text: 'This is the second message.' }] },
+            { role: 'user', parts: [{ text: 'This is the third message.' }] },
+            { role: 'model', parts: [{ functionCall: {} }] },
+        ];
+        // Can't return 4 because the previous item has a function call.
+        expect(findCompressSplitPoint(history, 0.99)).toBe(2);
     });
     it('should handle a history with only one item', () => {
-        expect(findIndexAfterFraction(history.slice(0, 1), 0.5)).toBe(0);
+        const historyWithEmptyParts = [
+            { role: 'user', parts: [{ text: 'Message 1' }] },
+        ];
+        expect(findCompressSplitPoint(historyWithEmptyParts, 0.5)).toBe(0);
     });
     it('should handle history with weird parts', () => {
         const historyWithEmptyParts = [
@@ -103,37 +156,54 @@ describe('findIndexAfterFraction', () => {
             { role: 'model', parts: [{ fileData: { fileUri: 'derp' } }] },
             { role: 'user', parts: [{ text: 'Message 2' }] },
         ];
-        expect(findIndexAfterFraction(historyWithEmptyParts, 0.5)).toBe(1);
+        expect(findCompressSplitPoint(historyWithEmptyParts, 0.5)).toBe(2);
+    });
+});
+describe('isThinkingSupported', () => {
+    it('should return true for gemini-2.5', () => {
+        expect(isThinkingSupported('gemini-2.5')).toBe(true);
+    });
+    it('should return true for gemini-2.5-pro', () => {
+        expect(isThinkingSupported('gemini-2.5-pro')).toBe(true);
+    });
+    it('should return false for other models', () => {
+        expect(isThinkingSupported('gemini-1.5-flash')).toBe(false);
+        expect(isThinkingSupported('some-other-model')).toBe(false);
+    });
+});
+describe('isThinkingDefault', () => {
+    it('should return false for gemini-2.5-flash-lite', () => {
+        expect(isThinkingDefault('gemini-2.5-flash-lite')).toBe(false);
+    });
+    it('should return true for gemini-2.5', () => {
+        expect(isThinkingDefault('gemini-2.5')).toBe(true);
+    });
+    it('should return true for gemini-2.5-pro', () => {
+        expect(isThinkingDefault('gemini-2.5-pro')).toBe(true);
+    });
+    it('should return false for other models', () => {
+        expect(isThinkingDefault('gemini-1.5-flash')).toBe(false);
+        expect(isThinkingDefault('some-other-model')).toBe(false);
     });
 });
 describe('Gemini Client (client.ts)', () => {
+    let mockContentGenerator;
+    let mockConfig;
     let client;
+    let mockGenerateContentFn;
     beforeEach(async () => {
         vi.resetAllMocks();
+        vi.mocked(uiTelemetryService.setLastPromptTokenCount).mockClear();
+        mockGenerateContentFn = vi.fn().mockResolvedValue({
+            candidates: [{ content: { parts: [{ text: '{"key": "value"}' }] } }],
+        });
         // Disable 429 simulation for tests
         setSimulate429(false);
-        // Set up the mock for GoogleGenAI constructor and its methods
-        const MockedGoogleGenAI = vi.mocked(GoogleGenAI);
-        MockedGoogleGenAI.mockImplementation(() => {
-            const mock = {
-                chats: { create: mockChatCreateFn },
-                models: {
-                    generateContent: mockGenerateContentFn,
-                    embedContent: mockEmbedContentFn,
-                },
-            };
-            return mock;
-        });
-        mockChatCreateFn.mockResolvedValue({});
-        mockGenerateContentFn.mockResolvedValue({
-            candidates: [
-                {
-                    content: {
-                        parts: [{ text: '{"key": "value"}' }],
-                    },
-                },
-            ],
-        });
+        mockContentGenerator = {
+            generateContent: mockGenerateContentFn,
+            generateContentStream: vi.fn(),
+            batchEmbedContents: vi.fn(),
+        };
         // Because the GeminiClient constructor kicks off an async process (startChat)
         // that depends on a fully-formed Config object, we need to mock the
         // entire implementation of Config for these tests.
@@ -143,12 +213,11 @@ describe('Gemini Client (client.ts)', () => {
         };
         const fileService = new FileDiscoveryService('/test/dir');
         const contentGeneratorConfig = {
-            model: 'test-model',
             apiKey: 'test-key',
             vertexai: false,
             authType: AuthType.USE_GEMINI,
         };
-        const mockConfigObject = {
+        mockConfig = {
             getContentGeneratorConfig: vi
                 .fn()
                 .mockReturnValue(contentGeneratorConfig),
@@ -176,181 +245,34 @@ describe('Gemini Client (client.ts)', () => {
                 getDirectories: vi.fn().mockReturnValue(['/test/dir']),
             }),
             getGeminiClient: vi.fn(),
+            getModelRouterService: vi.fn().mockReturnValue({
+                route: vi.fn().mockResolvedValue({ model: 'default-routed-model' }),
+            }),
+            isInFallbackMode: vi.fn().mockReturnValue(false),
             setFallbackMode: vi.fn(),
             getChatCompression: vi.fn().mockReturnValue(undefined),
             getSkipNextSpeakerCheck: vi.fn().mockReturnValue(false),
+            getUseSmartEdit: vi.fn().mockReturnValue(false),
+            getUseModelRouter: vi.fn().mockReturnValue(false),
+            getProjectRoot: vi.fn().mockReturnValue('/test/project/root'),
+            storage: {
+                getProjectTempDir: vi.fn().mockReturnValue('/test/temp'),
+            },
+            getContentGenerator: vi.fn().mockReturnValue(mockContentGenerator),
+            getBaseLlmClient: vi.fn().mockReturnValue({
+                generateJson: vi.fn().mockResolvedValue({
+                    next_speaker: 'user',
+                    reasoning: 'test',
+                }),
+            }),
         };
-        const MockedConfig = vi.mocked(Config, true);
-        MockedConfig.mockImplementation(() => mockConfigObject);
-        // We can instantiate the client here since Config is mocked
-        // and the constructor will use the mocked GoogleGenAI
-        client = new GeminiClient(new Config({ sessionId: 'test-session-id' }));
-        mockConfigObject.getGeminiClient.mockReturnValue(client);
-        await client.initialize(contentGeneratorConfig);
+        client = new GeminiClient(mockConfig);
+        await client.initialize();
+        vi.mocked(mockConfig.getGeminiClient).mockReturnValue(client);
     });
     afterEach(() => {
         vi.restoreAllMocks();
     });
-    // NOTE: The following tests for startChat were removed due to persistent issues with
-    // the @google/genai mock. Specifically, the mockChatCreateFn (representing instance.chats.create)
-    // was not being detected as called by the GeminiClient instance.
-    // This likely points to a subtle issue in how the GoogleGenerativeAI class constructor
-    // and its instance methods are mocked and then used by the class under test.
-    // For future debugging, ensure that the `this.client` in `GeminiClient` (which is an
-    // instance of the mocked GoogleGenerativeAI) correctly has its `chats.create` method
-    // pointing to `mockChatCreateFn`.
-    // it('startChat should call getCoreSystemPrompt with userMemory and pass to chats.create', async () => { ... });
-    // it('startChat should call getCoreSystemPrompt with empty string if userMemory is empty', async () => { ... });
-    // NOTE: The following tests for generateJson were removed due to persistent issues with
-    // the @google/genai mock, similar to the startChat tests. The mockGenerateContentFn
-    // (representing instance.models.generateContent) was not being detected as called, or the mock
-    // was not preventing an actual API call (leading to API key errors).
-    // For future debugging, ensure `this.client.models.generateContent` in `GeminiClient` correctly
-    // uses the `mockGenerateContentFn`.
-    // it('generateJson should call getCoreSystemPrompt with userMemory and pass to generateContent', async () => { ... });
-    // it('generateJson should call getCoreSystemPrompt with empty string if userMemory is empty', async () => { ... });
-    describe('generateEmbedding', () => {
-        const texts = ['hello world', 'goodbye world'];
-        const testEmbeddingModel = 'test-embedding-model';
-        it('should call embedContent with correct parameters and return embeddings', async () => {
-            const mockEmbeddings = [
-                [0.1, 0.2, 0.3],
-                [0.4, 0.5, 0.6],
-            ];
-            const mockResponse = {
-                embeddings: [
-                    { values: mockEmbeddings[0] },
-                    { values: mockEmbeddings[1] },
-                ],
-            };
-            mockEmbedContentFn.mockResolvedValue(mockResponse);
-            const result = await client.generateEmbedding(texts);
-            expect(mockEmbedContentFn).toHaveBeenCalledTimes(1);
-            expect(mockEmbedContentFn).toHaveBeenCalledWith({
-                model: testEmbeddingModel,
-                contents: texts,
-            });
-            expect(result).toEqual(mockEmbeddings);
-        });
-        it('should return an empty array if an empty array is passed', async () => {
-            const result = await client.generateEmbedding([]);
-            expect(result).toEqual([]);
-            expect(mockEmbedContentFn).not.toHaveBeenCalled();
-        });
-        it('should throw an error if API response has no embeddings array', async () => {
-            mockEmbedContentFn.mockResolvedValue({}); // No `embeddings` key
-            await expect(client.generateEmbedding(texts)).rejects.toThrow('No embeddings found in API response.');
-        });
-        it('should throw an error if API response has an empty embeddings array', async () => {
-            const mockResponse = {
-                embeddings: [],
-            };
-            mockEmbedContentFn.mockResolvedValue(mockResponse);
-            await expect(client.generateEmbedding(texts)).rejects.toThrow('No embeddings found in API response.');
-        });
-        it('should throw an error if API returns a mismatched number of embeddings', async () => {
-            const mockResponse = {
-                embeddings: [{ values: [1, 2, 3] }], // Only one for two texts
-            };
-            mockEmbedContentFn.mockResolvedValue(mockResponse);
-            await expect(client.generateEmbedding(texts)).rejects.toThrow('API returned a mismatched number of embeddings. Expected 2, got 1.');
-        });
-        it('should throw an error if any embedding has nullish values', async () => {
-            const mockResponse = {
-                embeddings: [{ values: [1, 2, 3] }, { values: undefined }], // Second one is bad
-            };
-            mockEmbedContentFn.mockResolvedValue(mockResponse);
-            await expect(client.generateEmbedding(texts)).rejects.toThrow('API returned an empty embedding for input text at index 1: "goodbye world"');
-        });
-        it('should throw an error if any embedding has an empty values array', async () => {
-            const mockResponse = {
-                embeddings: [{ values: [] }, { values: [1, 2, 3] }], // First one is bad
-            };
-            mockEmbedContentFn.mockResolvedValue(mockResponse);
-            await expect(client.generateEmbedding(texts)).rejects.toThrow('API returned an empty embedding for input text at index 0: "hello world"');
-        });
-        it('should propagate errors from the API call', async () => {
-            const apiError = new Error('API Failure');
-            mockEmbedContentFn.mockRejectedValue(apiError);
-            await expect(client.generateEmbedding(texts)).rejects.toThrow('API Failure');
-        });
-    });
-    describe('generateContent', () => {
-        it('should call generateContent with the correct parameters', async () => {
-            const contents = [{ role: 'user', parts: [{ text: 'hello' }] }];
-            const generationConfig = { temperature: 0.5 };
-            const abortSignal = new AbortController().signal;
-            // Mock countTokens
-            const mockGenerator = {
-                countTokens: vi.fn().mockResolvedValue({ totalTokens: 1 }),
-                generateContent: mockGenerateContentFn,
-            };
-            client['contentGenerator'] = mockGenerator;
-            await client.generateContent(contents, generationConfig, abortSignal);
-            expect(mockGenerateContentFn).toHaveBeenCalledWith({
-                model: 'test-model',
-                config: {
-                    abortSignal,
-                    systemInstruction: getCoreSystemPrompt(''),
-                    temperature: 0.5,
-                    topP: 1,
-                },
-                contents,
-            }, 'test-session-id');
-        });
-    });
-    describe('generateJson', () => {
-        it('should call generateContent with the correct parameters', async () => {
-            const contents = [{ role: 'user', parts: [{ text: 'hello' }] }];
-            const schema = { type: 'string' };
-            const abortSignal = new AbortController().signal;
-            // Mock countTokens
-            const mockGenerator = {
-                countTokens: vi.fn().mockResolvedValue({ totalTokens: 1 }),
-                generateContent: mockGenerateContentFn,
-            };
-            client['contentGenerator'] = mockGenerator;
-            await client.generateJson(contents, schema, abortSignal);
-            expect(mockGenerateContentFn).toHaveBeenCalledWith({
-                model: 'test-model', // Should use current model from config
-                config: {
-                    abortSignal,
-                    systemInstruction: getCoreSystemPrompt(''),
-                    temperature: 0,
-                    topP: 1,
-                    responseJsonSchema: schema,
-                    responseMimeType: 'application/json',
-                },
-                contents,
-            }, 'test-session-id');
-        });
-        it('should allow overriding model and config', async () => {
-            const contents = [{ role: 'user', parts: [{ text: 'hello' }] }];
-            const schema = { type: 'string' };
-            const abortSignal = new AbortController().signal;
-            const customModel = 'custom-json-model';
-            const customConfig = { temperature: 0.9, topK: 20 };
-            const mockGenerator = {
-                countTokens: vi.fn().mockResolvedValue({ totalTokens: 1 }),
-                generateContent: mockGenerateContentFn,
-            };
-            client['contentGenerator'] = mockGenerator;
-            await client.generateJson(contents, schema, abortSignal, customModel, customConfig);
-            expect(mockGenerateContentFn).toHaveBeenCalledWith({
-                model: customModel,
-                config: {
-                    abortSignal,
-                    systemInstruction: getCoreSystemPrompt(''),
-                    temperature: 0.9,
-                    topP: 1, // from default
-                    topK: 20,
-                    responseJsonSchema: schema,
-                    responseMimeType: 'application/json',
-                },
-                contents,
-            }, 'test-session-id');
-        });
-    });
     describe('addHistory', () => {
         it('should call chat.addHistory with the provided content', async () => {
             const mockChat = {
@@ -388,37 +310,166 @@ describe('Gemini Client (client.ts)', () => {
         });
     });
     describe('tryCompressChat', () => {
-        const mockCountTokens = vi.fn();
-        const mockSendMessage = vi.fn();
         const mockGetHistory = vi.fn();
         beforeEach(() => {
             vi.mock('./tokenLimits', () => ({
                 tokenLimit: vi.fn(),
             }));
-            client['contentGenerator'] = {
-                countTokens: mockCountTokens,
-            };
             client['chat'] = {
                 getHistory: mockGetHistory,
                 addHistory: vi.fn(),
                 setHistory: vi.fn(),
-                sendMessage: mockSendMessage,
             };
         });
+        function setup({ chatHistory = [
+            { role: 'user', parts: [{ text: 'Long conversation' }] },
+            { role: 'model', parts: [{ text: 'Long response' }] },
+        ], originalTokenCount = 1000, summaryText = 'This is a summary.', } = {}) {
+            const mockOriginalChat = {
+                getHistory: vi.fn((_curated) => chatHistory),
+                setHistory: vi.fn(),
+            };
+            client['chat'] = mockOriginalChat;
+            vi.mocked(uiTelemetryService.getLastPromptTokenCount).mockReturnValue(originalTokenCount);
+            mockGenerateContentFn.mockResolvedValue({
+                candidates: [
+                    {
+                        content: {
+                            role: 'model',
+                            parts: [{ text: summaryText }],
+                        },
+                    },
+                ],
+            });
+            // Calculate what the new history will be
+            const splitPoint = findCompressSplitPoint(chatHistory, 0.7); // 1 - 0.3
+            const historyToKeep = chatHistory.slice(splitPoint);
+            // This is the history that the new chat will have.
+            // It includes the default startChat history + the extra history from tryCompressChat
+            const newCompressedHistory = [
+                // Mocked envParts + canned response from startChat
+                {
+                    role: 'user',
+                    parts: [{ text: 'Mocked env context' }],
+                },
+                {
+                    role: 'model',
+                    parts: [{ text: 'Got it. Thanks for the context!' }],
+                },
+                // extraHistory from tryCompressChat
+                {
+                    role: 'user',
+                    parts: [{ text: summaryText }],
+                },
+                {
+                    role: 'model',
+                    parts: [{ text: 'Got it. Thanks for the additional context!' }],
+                },
+                ...historyToKeep,
+            ];
+            const mockNewChat = {
+                getHistory: vi.fn().mockReturnValue(newCompressedHistory),
+                setHistory: vi.fn(),
+            };
+            client['startChat'] = vi
+                .fn()
+                .mockResolvedValue(mockNewChat);
+            const totalChars = newCompressedHistory.reduce((total, content) => total + JSON.stringify(content).length, 0);
+            const estimatedNewTokenCount = Math.floor(totalChars / 4);
+            return {
+                client,
+                mockOriginalChat,
+                mockNewChat,
+                estimatedNewTokenCount,
+            };
+        }
+        describe('when compression inflates the token count', () => {
+            it('allows compression to be forced/manual after a failure', async () => {
+                // Call 1 (Fails): Setup with a long summary to inflate tokens
+                const longSummary = 'long summary '.repeat(100);
+                const { client, estimatedNewTokenCount: inflatedTokenCount } = setup({
+                    originalTokenCount: 100,
+                    summaryText: longSummary,
+                });
+                expect(inflatedTokenCount).toBeGreaterThan(100); // Ensure setup is correct
+                await client.tryCompressChat('prompt-id-4', false); // Fails
+                // Call 2 (Forced): Re-setup with a short summary
+                const shortSummary = 'short';
+                const { estimatedNewTokenCount: compressedTokenCount } = setup({
+                    originalTokenCount: 100,
+                    summaryText: shortSummary,
+                });
+                expect(compressedTokenCount).toBeLessThanOrEqual(100); // Ensure setup is correct
+                const result = await client.tryCompressChat('prompt-id-4', true); // Forced
+                expect(result).toEqual({
+                    compressionStatus: CompressionStatus.COMPRESSED,
+                    newTokenCount: compressedTokenCount,
+                    originalTokenCount: 100,
+                });
+            });
+            it('yields the result even if the compression inflated the tokens', async () => {
+                const longSummary = 'long summary '.repeat(100);
+                const { client, estimatedNewTokenCount } = setup({
+                    originalTokenCount: 100,
+                    summaryText: longSummary,
+                });
+                expect(estimatedNewTokenCount).toBeGreaterThan(100); // Ensure setup is correct
+                const result = await client.tryCompressChat('prompt-id-4', false);
+                expect(result).toEqual({
+                    compressionStatus: CompressionStatus.COMPRESSION_FAILED_INFLATED_TOKEN_COUNT,
+                    newTokenCount: estimatedNewTokenCount,
+                    originalTokenCount: 100,
+                });
+                // IMPORTANT: The change in client.ts means setLastPromptTokenCount is NOT called on failure
+                expect(uiTelemetryService.setLastPromptTokenCount).not.toHaveBeenCalled();
+            });
+            it('does not manipulate the source chat', async () => {
+                const longSummary = 'long summary '.repeat(100);
+                const { client, mockOriginalChat, estimatedNewTokenCount } = setup({
+                    originalTokenCount: 100,
+                    summaryText: longSummary,
+                });
+                expect(estimatedNewTokenCount).toBeGreaterThan(100); // Ensure setup is correct
+                await client.tryCompressChat('prompt-id-4', false);
+                // On failure, the chat should NOT be replaced
+                expect(client['chat']).toBe(mockOriginalChat);
+            });
+            it('will not attempt to compress context after a failure', async () => {
+                const longSummary = 'long summary '.repeat(100);
+                const { client, estimatedNewTokenCount } = setup({
+                    originalTokenCount: 100,
+                    summaryText: longSummary,
+                });
+                expect(estimatedNewTokenCount).toBeGreaterThan(100); // Ensure setup is correct
+                await client.tryCompressChat('prompt-id-4', false); // This fails and sets hasFailedCompressionAttempt = true
+                // This call should now be a NOOP
+                const result = await client.tryCompressChat('prompt-id-5', false);
+                // generateContent (for summary) should only have been called once
+                expect(mockGenerateContentFn).toHaveBeenCalledTimes(1);
+                expect(result).toEqual({
+                    compressionStatus: CompressionStatus.NOOP,
+                    newTokenCount: 0,
+                    originalTokenCount: 0,
+                });
+            });
+        });
         it('should not trigger summarization if token count is below threshold', async () => {
             const MOCKED_TOKEN_LIMIT = 1000;
             vi.mocked(tokenLimit).mockReturnValue(MOCKED_TOKEN_LIMIT);
             mockGetHistory.mockReturnValue([
                 { role: 'user', parts: [{ text: '...history...' }] },
             ]);
-            mockCountTokens.mockResolvedValue({
-                totalTokens: MOCKED_TOKEN_LIMIT * 0.699, // TOKEN_THRESHOLD_FOR_SUMMARIZATION = 0.7
-            });
+            const originalTokenCount = MOCKED_TOKEN_LIMIT * 0.699;
+            vi.mocked(uiTelemetryService.getLastPromptTokenCount).mockReturnValue(originalTokenCount);
             const initialChat = client.getChat();
-            const result = await client.tryCompressChat('prompt-id-2');
+            const result = await client.tryCompressChat('prompt-id-2', false);
             const newChat = client.getChat();
             expect(tokenLimit).toHaveBeenCalled();
-            expect(result).toBeNull();
+            expect(result).toEqual({
+                compressionStatus: CompressionStatus.NOOP,
+                newTokenCount: originalTokenCount,
+                originalTokenCount,
+            });
             expect(newChat).toBe(initialChat);
         });
         it('logs a telemetry event when compressing', async () => {
@@ -429,24 +480,51 @@ describe('Gemini Client (client.ts)', () => {
             vi.spyOn(client['config'], 'getChatCompression').mockReturnValue({
                 contextPercentageThreshold: MOCKED_CONTEXT_PERCENTAGE_THRESHOLD,
             });
-            mockGetHistory.mockReturnValue([
-                { role: 'user', parts: [{ text: '...history...' }] },
-            ]);
+            const history = [{ role: 'user', parts: [{ text: '...history...' }] }];
+            mockGetHistory.mockReturnValue(history);
             const originalTokenCount = MOCKED_TOKEN_LIMIT * MOCKED_CONTEXT_PERCENTAGE_THRESHOLD;
-            const newTokenCount = 100;
-            mockCountTokens
-                .mockResolvedValueOnce({ totalTokens: originalTokenCount }) // First call for the check
-                .mockResolvedValueOnce({ totalTokens: newTokenCount }); // Second call for the new history
+            vi.mocked(uiTelemetryService.getLastPromptTokenCount).mockReturnValue(originalTokenCount);
+            // We need to control the estimated new token count.
+            // We mock startChat to return a chat with a known history.
+            const summaryText = 'This is a summary.';
+            const splitPoint = findCompressSplitPoint(history, 0.7);
+            const historyToKeep = history.slice(splitPoint);
+            const newCompressedHistory = [
+                { role: 'user', parts: [{ text: 'Mocked env context' }] },
+                { role: 'model', parts: [{ text: 'Got it. Thanks for the context!' }] },
+                { role: 'user', parts: [{ text: summaryText }] },
+                {
+                    role: 'model',
+                    parts: [{ text: 'Got it. Thanks for the additional context!' }],
+                },
+                ...historyToKeep,
+            ];
+            const mockNewChat = {
+                getHistory: vi.fn().mockReturnValue(newCompressedHistory),
+            };
+            client['startChat'] = vi
+                .fn()
+                .mockResolvedValue(mockNewChat);
+            const totalChars = newCompressedHistory.reduce((total, content) => total + JSON.stringify(content).length, 0);
+            const newTokenCount = Math.floor(totalChars / 4);
             // Mock the summary response from the chat
-            mockSendMessage.mockResolvedValue({
-                role: 'model',
-                parts: [{ text: 'This is a summary.' }],
+            mockGenerateContentFn.mockResolvedValue({
+                candidates: [
+                    {
+                        content: {
+                            role: 'model',
+                            parts: [{ text: summaryText }],
+                        },
+                    },
+                ],
             });
-            await client.tryCompressChat('prompt-id-3');
+            await client.tryCompressChat('prompt-id-3', false);
             expect(ClearcutLogger.prototype.logChatCompressionEvent).toHaveBeenCalledWith(expect.objectContaining({
                 tokens_before: originalTokenCount,
                 tokens_after: newTokenCount,
             }));
+            expect(uiTelemetryService.setLastPromptTokenCount).toHaveBeenCalledWith(newTokenCount);
+            expect(uiTelemetryService.setLastPromptTokenCount).toHaveBeenCalledTimes(1);
         });
         it('should trigger summarization if token count is at threshold with contextPercentageThreshold setting', async () => {
             const MOCKED_TOKEN_LIMIT = 1000;
@@ -455,26 +533,51 @@ describe('Gemini Client (client.ts)', () => {
             vi.spyOn(client['config'], 'getChatCompression').mockReturnValue({
                 contextPercentageThreshold: MOCKED_CONTEXT_PERCENTAGE_THRESHOLD,
             });
-            mockGetHistory.mockReturnValue([
-                { role: 'user', parts: [{ text: '...history...' }] },
-            ]);
+            const history = [{ role: 'user', parts: [{ text: '...history...' }] }];
+            mockGetHistory.mockReturnValue(history);
             const originalTokenCount = MOCKED_TOKEN_LIMIT * MOCKED_CONTEXT_PERCENTAGE_THRESHOLD;
-            const newTokenCount = 100;
-            mockCountTokens
-                .mockResolvedValueOnce({ totalTokens: originalTokenCount }) // First call for the check
-                .mockResolvedValueOnce({ totalTokens: newTokenCount }); // Second call for the new history
+            vi.mocked(uiTelemetryService.getLastPromptTokenCount).mockReturnValue(originalTokenCount);
+            // Mock summary and new chat
+            const summaryText = 'This is a summary.';
+            const splitPoint = findCompressSplitPoint(history, 0.7);
+            const historyToKeep = history.slice(splitPoint);
+            const newCompressedHistory = [
+                { role: 'user', parts: [{ text: 'Mocked env context' }] },
+                { role: 'model', parts: [{ text: 'Got it. Thanks for the context!' }] },
+                { role: 'user', parts: [{ text: summaryText }] },
+                {
+                    role: 'model',
+                    parts: [{ text: 'Got it. Thanks for the additional context!' }],
+                },
+                ...historyToKeep,
+            ];
+            const mockNewChat = {
+                getHistory: vi.fn().mockReturnValue(newCompressedHistory),
+            };
+            client['startChat'] = vi
+                .fn()
+                .mockResolvedValue(mockNewChat);
+            const totalChars = newCompressedHistory.reduce((total, content) => total + JSON.stringify(content).length, 0);
+            const newTokenCount = Math.floor(totalChars / 4);
             // Mock the summary response from the chat
-            mockSendMessage.mockResolvedValue({
-                role: 'model',
-                parts: [{ text: 'This is a summary.' }],
+            mockGenerateContentFn.mockResolvedValue({
+                candidates: [
+                    {
+                        content: {
+                            role: 'model',
+                            parts: [{ text: summaryText }],
+                        },
+                    },
+                ],
             });
             const initialChat = client.getChat();
-            const result = await client.tryCompressChat('prompt-id-3');
+            const result = await client.tryCompressChat('prompt-id-3', false);
             const newChat = client.getChat();
             expect(tokenLimit).toHaveBeenCalled();
-            expect(mockSendMessage).toHaveBeenCalled();
+            expect(mockGenerateContentFn).toHaveBeenCalled();
             // Assert that summarization happened and returned the correct stats
             expect(result).toEqual({
+                compressionStatus: CompressionStatus.COMPRESSED,
                 originalTokenCount,
                 newTokenCount,
             });
@@ -484,7 +587,7 @@ describe('Gemini Client (client.ts)', () => {
         it('should not compress across a function call response', async () => {
             const MOCKED_TOKEN_LIMIT = 1000;
             vi.mocked(tokenLimit).mockReturnValue(MOCKED_TOKEN_LIMIT);
-            mockGetHistory.mockReturnValue([
+            const history = [
                 { role: 'user', parts: [{ text: '...history 1...' }] },
                 { role: 'model', parts: [{ text: '...history 2...' }] },
                 { role: 'user', parts: [{ text: '...history 3...' }] },
@@ -501,55 +604,111 @@ describe('Gemini Client (client.ts)', () => {
                 { role: 'model', parts: [{ text: '...history 10...' }] },
                 // Instead we will break here.
                 { role: 'user', parts: [{ text: '...history 10...' }] },
-            ]);
+            ];
+            mockGetHistory.mockReturnValue(history);
             const originalTokenCount = 1000 * 0.7;
-            const newTokenCount = 100;
-            mockCountTokens
-                .mockResolvedValueOnce({ totalTokens: originalTokenCount }) // First call for the check
-                .mockResolvedValueOnce({ totalTokens: newTokenCount }); // Second call for the new history
+            vi.mocked(uiTelemetryService.getLastPromptTokenCount).mockReturnValue(originalTokenCount);
+            // Mock summary and new chat
+            const summaryText = 'This is a summary.';
+            const splitPoint = findCompressSplitPoint(history, 0.7); // This should be 10
+            expect(splitPoint).toBe(10); // Verify split point logic
+            const historyToKeep = history.slice(splitPoint); // Should keep last user message
+            expect(historyToKeep).toEqual([
+                { role: 'user', parts: [{ text: '...history 10...' }] },
+            ]);
+            const newCompressedHistory = [
+                { role: 'user', parts: [{ text: 'Mocked env context' }] },
+                { role: 'model', parts: [{ text: 'Got it. Thanks for the context!' }] },
+                { role: 'user', parts: [{ text: summaryText }] },
+                {
+                    role: 'model',
+                    parts: [{ text: 'Got it. Thanks for the additional context!' }],
+                },
+                ...historyToKeep,
+            ];
+            const mockNewChat = {
+                getHistory: vi.fn().mockReturnValue(newCompressedHistory),
+            };
+            client['startChat'] = vi
+                .fn()
+                .mockResolvedValue(mockNewChat);
+            const totalChars = newCompressedHistory.reduce((total, content) => total + JSON.stringify(content).length, 0);
+            const newTokenCount = Math.floor(totalChars / 4);
             // Mock the summary response from the chat
-            mockSendMessage.mockResolvedValue({
-                role: 'model',
-                parts: [{ text: 'This is a summary.' }],
+            mockGenerateContentFn.mockResolvedValue({
+                candidates: [
+                    {
+                        content: {
+                            role: 'model',
+                            parts: [{ text: summaryText }],
+                        },
+                    },
+                ],
             });
             const initialChat = client.getChat();
-            const result = await client.tryCompressChat('prompt-id-3');
+            const result = await client.tryCompressChat('prompt-id-3', false);
             const newChat = client.getChat();
             expect(tokenLimit).toHaveBeenCalled();
-            expect(mockSendMessage).toHaveBeenCalled();
+            expect(mockGenerateContentFn).toHaveBeenCalled();
             // Assert that summarization happened and returned the correct stats
             expect(result).toEqual({
+                compressionStatus: CompressionStatus.COMPRESSED,
                 originalTokenCount,
                 newTokenCount,
             });
             // Assert that the chat was reset
             expect(newChat).not.toBe(initialChat);
-            // 1. standard start context message
-            // 2. standard canned user start message
-            // 3. compressed summary message
-            // 4. standard canned user summary message
-            // 5. The last user message (not the last 3 because that would start with a function response)
+            // 1. standard start context message (env)
+            // 2. standard canned model response
+            // 3. compressed summary message (user)
+            // 4. standard canned model response
+            // 5. The last user message (historyToKeep)
             expect(newChat.getHistory().length).toEqual(5);
         });
         it('should always trigger summarization when force is true, regardless of token count', async () => {
-            mockGetHistory.mockReturnValue([
-                { role: 'user', parts: [{ text: '...history...' }] },
-            ]);
-            const originalTokenCount = 10; // Well below threshold
-            const newTokenCount = 5;
-            mockCountTokens
-                .mockResolvedValueOnce({ totalTokens: originalTokenCount })
-                .mockResolvedValueOnce({ totalTokens: newTokenCount });
+            const history = [{ role: 'user', parts: [{ text: '...history...' }] }];
+            mockGetHistory.mockReturnValue(history);
+            const originalTokenCount = 100; // Well below threshold, but > estimated new count
+            vi.mocked(uiTelemetryService.getLastPromptTokenCount).mockReturnValue(originalTokenCount);
+            // Mock summary and new chat
+            const summaryText = 'This is a summary.';
+            const splitPoint = findCompressSplitPoint(history, 0.7);
+            const historyToKeep = history.slice(splitPoint);
+            const newCompressedHistory = [
+                { role: 'user', parts: [{ text: 'Mocked env context' }] },
+                { role: 'model', parts: [{ text: 'Got it. Thanks for the context!' }] },
+                { role: 'user', parts: [{ text: summaryText }] },
+                {
+                    role: 'model',
+                    parts: [{ text: 'Got it. Thanks for the additional context!' }],
+                },
+                ...historyToKeep,
+            ];
+            const mockNewChat = {
+                getHistory: vi.fn().mockReturnValue(newCompressedHistory),
+            };
+            client['startChat'] = vi
+                .fn()
+                .mockResolvedValue(mockNewChat);
+            const totalChars = newCompressedHistory.reduce((total, content) => total + JSON.stringify(content).length, 0);
+            const newTokenCount = Math.floor(totalChars / 4);
             // Mock the summary response from the chat
-            mockSendMessage.mockResolvedValue({
-                role: 'model',
-                parts: [{ text: 'This is a summary.' }],
+            mockGenerateContentFn.mockResolvedValue({
+                candidates: [
+                    {
+                        content: {
+                            role: 'model',
+                            parts: [{ text: summaryText }],
+                        },
+                    },
+                ],
             });
             const initialChat = client.getChat();
             const result = await client.tryCompressChat('prompt-id-1', true); // force = true
             const newChat = client.getChat();
-            expect(mockSendMessage).toHaveBeenCalled();
+            expect(mockGenerateContentFn).toHaveBeenCalled();
             expect(result).toEqual({
+                compressionStatus: CompressionStatus.COMPRESSED,
                 originalTokenCount,
                 newTokenCount,
             });
@@ -558,9 +717,55 @@ describe('Gemini Client (client.ts)', () => {
         });
     });
     describe('sendMessageStream', () => {
+        it('emits a compression event when the context was automatically compressed', async () => {
+            // Arrange
+            mockTurnRunFn.mockReturnValue((async function* () {
+                yield { type: 'content', value: 'Hello' };
+            })());
+            const compressionInfo = {
+                compressionStatus: CompressionStatus.COMPRESSED,
+                originalTokenCount: 1000,
+                newTokenCount: 500,
+            };
+            vi.spyOn(client, 'tryCompressChat').mockResolvedValueOnce(compressionInfo);
+            // Act
+            const stream = client.sendMessageStream([{ text: 'Hi' }], new AbortController().signal, 'prompt-id-1');
+            const events = await fromAsync(stream);
+            // Assert
+            expect(events).toContainEqual({
+                type: GeminiEventType.ChatCompressed,
+                value: compressionInfo,
+            });
+        });
+        it.each([
+            {
+                compressionStatus: CompressionStatus.COMPRESSION_FAILED_INFLATED_TOKEN_COUNT,
+            },
+            { compressionStatus: CompressionStatus.NOOP },
+        ])('does not emit a compression event when the status is $compressionStatus', async ({ compressionStatus }) => {
+            // Arrange
+            const mockStream = (async function* () {
+                yield { type: 'content', value: 'Hello' };
+            })();
+            mockTurnRunFn.mockReturnValue(mockStream);
+            const compressionInfo = {
+                compressionStatus,
+                originalTokenCount: 1000,
+                newTokenCount: 500,
+            };
+            vi.spyOn(client, 'tryCompressChat').mockResolvedValueOnce(compressionInfo);
+            // Act
+            const stream = client.sendMessageStream([{ text: 'Hi' }], new AbortController().signal, 'prompt-id-1');
+            const events = await fromAsync(stream);
+            // Assert
+            expect(events).not.toContainEqual({
+                type: GeminiEventType.ChatCompressed,
+                value: expect.anything(),
+            });
+        });
         it('should include editor context when ideMode is enabled', async () => {
             // Arrange
-            vi.mocked(ideContext.getIdeContext).mockReturnValue({
+            vi.mocked(ideContextStore.get).mockReturnValue({
                 workspaceState: {
                     openFiles: [
                         {
@@ -581,21 +786,20 @@ describe('Gemini Client (client.ts)', () => {
                     ],
                 },
             });
-            vi.spyOn(client['config'], 'getIdeMode').mockReturnValue(true);
-            const mockStream = (async function* () {
+            vi.mocked(mockConfig.getIdeMode).mockReturnValue(true);
+            vi.spyOn(client, 'tryCompressChat').mockResolvedValue({
+                originalTokenCount: 0,
+                newTokenCount: 0,
+                compressionStatus: CompressionStatus.COMPRESSED,
+            });
+            mockTurnRunFn.mockReturnValue((async function* () {
                 yield { type: 'content', value: 'Hello' };
-            })();
-            mockTurnRunFn.mockReturnValue(mockStream);
+            })());
             const mockChat = {
                 addHistory: vi.fn(),
                 getHistory: vi.fn().mockReturnValue([]),
             };
             client['chat'] = mockChat;
-            const mockGenerator = {
-                countTokens: vi.fn().mockResolvedValue({ totalTokens: 0 }),
-                generateContent: mockGenerateContentFn,
-            };
-            client['contentGenerator'] = mockGenerator;
             const initialRequest = [{ text: 'Hi' }];
             // Act
             const stream = client.sendMessageStream(initialRequest, new AbortController().signal, 'prompt-id-ide');
@@ -603,7 +807,7 @@ describe('Gemini Client (client.ts)', () => {
                 // consume stream
             }
             // Assert
-            expect(ideContext.getIdeContext).toHaveBeenCalled();
+            expect(ideContextStore.get).toHaveBeenCalled();
             const expectedContext = `
 Here is the user's editor context as a JSON object. This is for your information only.
 \`\`\`json
@@ -628,7 +832,7 @@ ${JSON.stringify({
         });
         it('should not add context if ideMode is enabled but no open files', async () => {
             // Arrange
-            vi.mocked(ideContext.getIdeContext).mockReturnValue({
+            vi.mocked(ideContextStore.get).mockReturnValue({
                 workspaceState: {
                     openFiles: [],
                 },
@@ -643,11 +847,6 @@ ${JSON.stringify({
                 getHistory: vi.fn().mockReturnValue([]),
             };
             client['chat'] = mockChat;
-            const mockGenerator = {
-                countTokens: vi.fn().mockResolvedValue({ totalTokens: 0 }),
-                generateContent: mockGenerateContentFn,
-            };
-            client['contentGenerator'] = mockGenerator;
             const initialRequest = [{ text: 'Hi' }];
             // Act
             const stream = client.sendMessageStream(initialRequest, new AbortController().signal, 'prompt-id-ide');
@@ -655,12 +854,16 @@ ${JSON.stringify({
                 // consume stream
             }
             // Assert
-            expect(ideContext.getIdeContext).toHaveBeenCalled();
-            expect(mockTurnRunFn).toHaveBeenCalledWith(initialRequest, expect.any(Object));
+            expect(ideContextStore.get).toHaveBeenCalled();
+            // The `turn.run` method is now called with the model name as the first
+            // argument. We use `expect.any(String)` because this test is
+            // concerned with the IDE context logic, not the model routing,
+            // which is tested in its own dedicated suite.
+            expect(mockTurnRunFn).toHaveBeenCalledWith(expect.any(String), initialRequest, expect.any(Object));
         });
         it('should add context if ideMode is enabled and there is one active file', async () => {
             // Arrange
-            vi.mocked(ideContext.getIdeContext).mockReturnValue({
+            vi.mocked(ideContextStore.get).mockReturnValue({
                 workspaceState: {
                     openFiles: [
                         {
@@ -674,6 +877,11 @@ ${JSON.stringify({
                 },
             });
             vi.spyOn(client['config'], 'getIdeMode').mockReturnValue(true);
+            vi.spyOn(client, 'tryCompressChat').mockResolvedValue({
+                originalTokenCount: 0,
+                newTokenCount: 0,
+                compressionStatus: CompressionStatus.COMPRESSED,
+            });
             const mockStream = (async function* () {
                 yield { type: 'content', value: 'Hello' };
             })();
@@ -683,11 +891,6 @@ ${JSON.stringify({
                 getHistory: vi.fn().mockReturnValue([]),
             };
             client['chat'] = mockChat;
-            const mockGenerator = {
-                countTokens: vi.fn().mockResolvedValue({ totalTokens: 0 }),
-                generateContent: mockGenerateContentFn,
-            };
-            client['contentGenerator'] = mockGenerator;
             const initialRequest = [{ text: 'Hi' }];
             // Act
             const stream = client.sendMessageStream(initialRequest, new AbortController().signal, 'prompt-id-ide');
@@ -695,7 +898,7 @@ ${JSON.stringify({
                 // consume stream
             }
             // Assert
-            expect(ideContext.getIdeContext).toHaveBeenCalled();
+            expect(ideContextStore.get).toHaveBeenCalled();
             const expectedContext = `
 Here is the user's editor context as a JSON object. This is for your information only.
 \`\`\`json
@@ -719,7 +922,7 @@ ${JSON.stringify({
         });
         it('should add context if ideMode is enabled and there are open files but no active file', async () => {
             // Arrange
-            vi.mocked(ideContext.getIdeContext).mockReturnValue({
+            vi.mocked(ideContextStore.get).mockReturnValue({
                 workspaceState: {
                     openFiles: [
                         {
@@ -734,6 +937,11 @@ ${JSON.stringify({
                 },
             });
             vi.spyOn(client['config'], 'getIdeMode').mockReturnValue(true);
+            vi.spyOn(client, 'tryCompressChat').mockResolvedValue({
+                originalTokenCount: 0,
+                newTokenCount: 0,
+                compressionStatus: CompressionStatus.COMPRESSED,
+            });
             const mockStream = (async function* () {
                 yield { type: 'content', value: 'Hello' };
             })();
@@ -743,11 +951,6 @@ ${JSON.stringify({
                 getHistory: vi.fn().mockReturnValue([]),
             };
             client['chat'] = mockChat;
-            const mockGenerator = {
-                countTokens: vi.fn().mockResolvedValue({ totalTokens: 0 }),
-                generateContent: mockGenerateContentFn,
-            };
-            client['contentGenerator'] = mockGenerator;
             const initialRequest = [{ text: 'Hi' }];
             // Act
             const stream = client.sendMessageStream(initialRequest, new AbortController().signal, 'prompt-id-ide');
@@ -755,7 +958,7 @@ ${JSON.stringify({
                 // consume stream
             }
             // Assert
-            expect(ideContext.getIdeContext).toHaveBeenCalled();
+            expect(ideContextStore.get).toHaveBeenCalled();
             const expectedContext = `
 Here is the user's editor context as a JSON object. This is for your information only.
 \`\`\`json
@@ -781,11 +984,6 @@ ${JSON.stringify({
                 getHistory: vi.fn().mockReturnValue([]),
             };
             client['chat'] = mockChat;
-            const mockGenerator = {
-                countTokens: vi.fn().mockResolvedValue({ totalTokens: 0 }),
-                generateContent: mockGenerateContentFn,
-            };
-            client['contentGenerator'] = mockGenerator;
             // Act
             const stream = client.sendMessageStream([{ text: 'Hi' }], new AbortController().signal, 'prompt-id-1');
             // Consume the stream manually to get the final return value.
@@ -818,11 +1016,6 @@ ${JSON.stringify({
                 getHistory: vi.fn().mockReturnValue([]),
             };
             client['chat'] = mockChat;
-            const mockGenerator = {
-                countTokens: vi.fn().mockResolvedValue({ totalTokens: 0 }),
-                generateContent: mockGenerateContentFn,
-            };
-            client['contentGenerator'] = mockGenerator;
             // Use a signal that never gets aborted
             const abortController = new AbortController();
             const signal = abortController.signal;
@@ -885,11 +1078,6 @@ ${JSON.stringify({
                 getHistory: vi.fn().mockReturnValue([]),
             };
             client['chat'] = mockChat;
-            const mockGenerator = {
-                countTokens: vi.fn().mockResolvedValue({ totalTokens: 0 }),
-                generateContent: mockGenerateContentFn,
-            };
-            client['contentGenerator'] = mockGenerator;
             // Act & Assert
             // Run up to the limit
             for (let i = 0; i < MAX_SESSION_TURNS; i++) {
@@ -928,11 +1116,6 @@ ${JSON.stringify({
                 getHistory: vi.fn().mockReturnValue([]),
             };
             client['chat'] = mockChat;
-            const mockGenerator = {
-                countTokens: vi.fn().mockResolvedValue({ totalTokens: 0 }),
-                generateContent: mockGenerateContentFn,
-            };
-            client['contentGenerator'] = mockGenerator;
             // Use a signal that never gets aborted
             const abortController = new AbortController();
             const signal = abortController.signal;
@@ -971,19 +1154,107 @@ ${JSON.stringify({
             console.log(`Infinite loop protection working: checkNextSpeaker called ${callCount} times, ` +
                 `${eventCount} events generated (properly bounded by MAX_TURNS)`);
         });
+        describe('Model Routing', () => {
+            let mockRouterService;
+            beforeEach(() => {
+                mockRouterService = {
+                    route: vi
+                        .fn()
+                        .mockResolvedValue({ model: 'routed-model', reason: 'test' }),
+                };
+                vi.mocked(mockConfig.getModelRouterService).mockReturnValue(mockRouterService);
+                mockTurnRunFn.mockReturnValue((async function* () {
+                    yield { type: 'content', value: 'Hello' };
+                })());
+            });
+            it('should use the model router service to select a model on the first turn', async () => {
+                const stream = client.sendMessageStream([{ text: 'Hi' }], new AbortController().signal, 'prompt-1');
+                await fromAsync(stream); // consume stream
+                expect(mockConfig.getModelRouterService).toHaveBeenCalled();
+                expect(mockRouterService.route).toHaveBeenCalled();
+                expect(mockTurnRunFn).toHaveBeenCalledWith('routed-model', // The model from the router
+                [{ text: 'Hi' }], expect.any(Object));
+            });
+            it('should use the same model for subsequent turns in the same prompt (stickiness)', async () => {
+                // First turn
+                let stream = client.sendMessageStream([{ text: 'Hi' }], new AbortController().signal, 'prompt-1');
+                await fromAsync(stream);
+                expect(mockRouterService.route).toHaveBeenCalledTimes(1);
+                expect(mockTurnRunFn).toHaveBeenCalledWith('routed-model', [{ text: 'Hi' }], expect.any(Object));
+                // Second turn
+                stream = client.sendMessageStream([{ text: 'Continue' }], new AbortController().signal, 'prompt-1');
+                await fromAsync(stream);
+                // Router should not be called again
+                expect(mockRouterService.route).toHaveBeenCalledTimes(1);
+                // Should stick to the first model
+                expect(mockTurnRunFn).toHaveBeenCalledWith('routed-model', [{ text: 'Continue' }], expect.any(Object));
+            });
+            it('should reset the sticky model and re-route when the prompt_id changes', async () => {
+                // First prompt
+                let stream = client.sendMessageStream([{ text: 'Hi' }], new AbortController().signal, 'prompt-1');
+                await fromAsync(stream);
+                expect(mockRouterService.route).toHaveBeenCalledTimes(1);
+                expect(mockTurnRunFn).toHaveBeenCalledWith('routed-model', [{ text: 'Hi' }], expect.any(Object));
+                // New prompt
+                mockRouterService.route.mockResolvedValue({
+                    model: 'new-routed-model',
+                    reason: 'test',
+                });
+                stream = client.sendMessageStream([{ text: 'A new topic' }], new AbortController().signal, 'prompt-2');
+                await fromAsync(stream);
+                // Router should be called again for the new prompt
+                expect(mockRouterService.route).toHaveBeenCalledTimes(2);
+                // Should use the newly routed model
+                expect(mockTurnRunFn).toHaveBeenCalledWith('new-routed-model', [{ text: 'A new topic' }], expect.any(Object));
+            });
+            it('should use the fallback model and bypass routing when in fallback mode', async () => {
+                vi.mocked(mockConfig.isInFallbackMode).mockReturnValue(true);
+                mockRouterService.route.mockResolvedValue({
+                    model: DEFAULT_GEMINI_FLASH_MODEL,
+                    reason: 'fallback',
+                });
+                const stream = client.sendMessageStream([{ text: 'Hi' }], new AbortController().signal, 'prompt-1');
+                await fromAsync(stream);
+                expect(mockTurnRunFn).toHaveBeenCalledWith(DEFAULT_GEMINI_FLASH_MODEL, [{ text: 'Hi' }], expect.any(Object));
+            });
+            it('should stick to the fallback model for the entire sequence even if fallback mode ends', async () => {
+                // Start the sequence in fallback mode
+                vi.mocked(mockConfig.isInFallbackMode).mockReturnValue(true);
+                mockRouterService.route.mockResolvedValue({
+                    model: DEFAULT_GEMINI_FLASH_MODEL,
+                    reason: 'fallback',
+                });
+                let stream = client.sendMessageStream([{ text: 'Hi' }], new AbortController().signal, 'prompt-fallback-stickiness');
+                await fromAsync(stream);
+                // First call should use fallback model
+                expect(mockTurnRunFn).toHaveBeenCalledWith(DEFAULT_GEMINI_FLASH_MODEL, [{ text: 'Hi' }], expect.any(Object));
+                // End fallback mode
+                vi.mocked(mockConfig.isInFallbackMode).mockReturnValue(false);
+                // Second call in the same sequence
+                stream = client.sendMessageStream([{ text: 'Continue' }], new AbortController().signal, 'prompt-fallback-stickiness');
+                await fromAsync(stream);
+                // Router should still not be called, and it should stick to the fallback model
+                expect(mockTurnRunFn).toHaveBeenCalledTimes(2); // Ensure it was called again
+                expect(mockTurnRunFn).toHaveBeenLastCalledWith(DEFAULT_GEMINI_FLASH_MODEL, // Still the fallback model
+                [{ text: 'Continue' }], expect.any(Object));
+            });
+        });
         describe('Editor context delta', () => {
             const mockStream = (async function* () {
                 yield { type: 'content', value: 'Hello' };
             })();
             beforeEach(() => {
                 client['forceFullIdeContext'] = false; // Reset before each delta test
-                vi.spyOn(client, 'tryCompressChat').mockResolvedValue(null);
+                vi.spyOn(client, 'tryCompressChat').mockResolvedValue({
+                    originalTokenCount: 0,
+                    newTokenCount: 0,
+                    compressionStatus: CompressionStatus.COMPRESSED,
+                });
                 vi.spyOn(client['config'], 'getIdeMode').mockReturnValue(true);
                 mockTurnRunFn.mockReturnValue(mockStream);
                 const mockChat = {
                     addHistory: vi.fn(),
                     setHistory: vi.fn(),
-                    sendMessage: vi.fn().mockResolvedValue({ text: 'summary' }),
                     // Assume history is not empty for delta checks
                     getHistory: vi
                         .fn()
@@ -992,11 +1263,6 @@ ${JSON.stringify({
                     ]),
                 };
                 client['chat'] = mockChat;
-                const mockGenerator = {
-                    countTokens: vi.fn().mockResolvedValue({ totalTokens: 0 }),
-                    generateContent: mockGenerateContentFn,
-                };
-                client['contentGenerator'] = mockGenerator;
             });
             const testCases = [
                 {
@@ -1112,7 +1378,7 @@ ${JSON.stringify({
                     },
                 };
                 // Setup current context
-                vi.mocked(ideContext.getIdeContext).mockReturnValue({
+                vi.mocked(ideContextStore.get).mockReturnValue({
                     workspaceState: {
                         openFiles: [
                             { ...currentActiveFile, isActive: true, timestamp: Date.now() },
@@ -1158,7 +1424,7 @@ ${JSON.stringify({
                     },
                 };
                 // Setup current context (same as previous)
-                vi.mocked(ideContext.getIdeContext).mockReturnValue({
+                vi.mocked(ideContextStore.get).mockReturnValue({
                     workspaceState: {
                         openFiles: [
                             { ...activeFile, isActive: true, timestamp: Date.now() },
@@ -1190,7 +1456,11 @@ ${JSON.stringify({
         describe('IDE context with pending tool calls', () => {
             let mockChat;
             beforeEach(() => {
-                vi.spyOn(client, 'tryCompressChat').mockResolvedValue(null);
+                vi.spyOn(client, 'tryCompressChat').mockResolvedValue({
+                    originalTokenCount: 0,
+                    newTokenCount: 0,
+                    compressionStatus: CompressionStatus.COMPRESSED,
+                });
                 const mockStream = (async function* () {
                     yield { type: 'content', value: 'response' };
                 })();
@@ -1199,15 +1469,10 @@ ${JSON.stringify({
                     addHistory: vi.fn(),
                     getHistory: vi.fn().mockReturnValue([]), // Default empty history
                     setHistory: vi.fn(),
-                    sendMessage: vi.fn().mockResolvedValue({ text: 'summary' }),
                 };
                 client['chat'] = mockChat;
-                const mockGenerator = {
-                    countTokens: vi.fn().mockResolvedValue({ totalTokens: 0 }),
-                };
-                client['contentGenerator'] = mockGenerator;
                 vi.spyOn(client['config'], 'getIdeMode').mockReturnValue(true);
-                vi.mocked(ideContext.getIdeContext).mockReturnValue({
+                vi.mocked(ideContextStore.get).mockReturnValue({
                     workspaceState: {
                         openFiles: [{ path: '/path/to/file.ts', timestamp: Date.now() }],
                     },
@@ -1283,7 +1548,7 @@ ${JSON.stringify({
                         openFiles: [{ path: '/path/to/fileA.ts', timestamp: Date.now() }],
                     },
                 };
-                vi.mocked(ideContext.getIdeContext).mockReturnValue(initialIdeContext);
+                vi.mocked(ideContextStore.get).mockReturnValue(initialIdeContext);
                 // Act: Send the tool response
                 let stream = client.sendMessageStream([
                     {
@@ -1329,7 +1594,7 @@ ${JSON.stringify({
                         openFiles: [{ path: '/path/to/fileB.ts', timestamp: Date.now() }],
                     },
                 };
-                vi.mocked(ideContext.getIdeContext).mockReturnValue(newIdeContext);
+                vi.mocked(ideContextStore.get).mockReturnValue(newIdeContext);
                 // Act: Send a new, regular user message
                 stream = client.sendMessageStream([{ text: 'Thanks!' }], new AbortController().signal, 'prompt-id-final');
                 for await (const _ of stream) {
@@ -1359,7 +1624,7 @@ ${JSON.stringify({
                         ],
                     },
                 };
-                vi.mocked(ideContext.getIdeContext).mockReturnValue(contextA);
+                vi.mocked(ideContextStore.get).mockReturnValue(contextA);
                 // Act: Send a regular message to establish the initial context
                 let stream = client.sendMessageStream([{ text: 'Initial message' }], new AbortController().signal, 'prompt-id-initial');
                 for await (const _ of stream) {
@@ -1392,7 +1657,7 @@ ${JSON.stringify({
                         ],
                     },
                 };
-                vi.mocked(ideContext.getIdeContext).mockReturnValue(contextB);
+                vi.mocked(ideContextStore.get).mockReturnValue(contextB);
                 // Act: Send the tool response
                 stream = client.sendMessageStream([
                     {
@@ -1437,7 +1702,7 @@ ${JSON.stringify({
                         ],
                     },
                 };
-                vi.mocked(ideContext.getIdeContext).mockReturnValue(contextC);
+                vi.mocked(ideContextStore.get).mockReturnValue(contextC);
                 // Act: Send a new, regular user message
                 stream = client.sendMessageStream([{ text: 'Thanks!' }], new AbortController().signal, 'prompt-id-final');
                 for await (const _ of stream) {
@@ -1453,150 +1718,130 @@ ${JSON.stringify({
                 expect(JSON.stringify(finalCall)).toContain('fileC.ts');
             });
         });
+        it('should not call checkNextSpeaker when turn.run() yields an error', async () => {
+            // Arrange
+            const { checkNextSpeaker } = await import('../utils/nextSpeakerChecker.js');
+            const mockCheckNextSpeaker = vi.mocked(checkNextSpeaker);
+            const mockStream = (async function* () {
+                yield {
+                    type: GeminiEventType.Error,
+                    value: { error: { message: 'test error' } },
+                };
+            })();
+            mockTurnRunFn.mockReturnValue(mockStream);
+            const mockChat = {
+                addHistory: vi.fn(),
+                getHistory: vi.fn().mockReturnValue([]),
+            };
+            client['chat'] = mockChat;
+            // Act
+            const stream = client.sendMessageStream([{ text: 'Hi' }], new AbortController().signal, 'prompt-id-error');
+            for await (const _ of stream) {
+                // consume stream
+            }
+            // Assert
+            expect(mockCheckNextSpeaker).not.toHaveBeenCalled();
+        });
+        it('should not call checkNextSpeaker when turn.run() yields a value then an error', async () => {
+            // Arrange
+            const { checkNextSpeaker } = await import('../utils/nextSpeakerChecker.js');
+            const mockCheckNextSpeaker = vi.mocked(checkNextSpeaker);
+            const mockStream = (async function* () {
+                yield { type: GeminiEventType.Content, value: 'some content' };
+                yield {
+                    type: GeminiEventType.Error,
+                    value: { error: { message: 'test error' } },
+                };
+            })();
+            mockTurnRunFn.mockReturnValue(mockStream);
+            const mockChat = {
+                addHistory: vi.fn(),
+                getHistory: vi.fn().mockReturnValue([]),
+            };
+            client['chat'] = mockChat;
+            // Act
+            const stream = client.sendMessageStream([{ text: 'Hi' }], new AbortController().signal, 'prompt-id-error');
+            for await (const _ of stream) {
+                // consume stream
+            }
+            // Assert
+            expect(mockCheckNextSpeaker).not.toHaveBeenCalled();
+        });
+        it('should abort linked signal when loop is detected', async () => {
+            // Arrange
+            vi.spyOn(client['loopDetector'], 'turnStarted').mockResolvedValue(false);
+            vi.spyOn(client['loopDetector'], 'addAndCheck')
+                .mockReturnValueOnce(false)
+                .mockReturnValueOnce(true);
+            let capturedSignal;
+            mockTurnRunFn.mockImplementation((model, request, signal) => {
+                capturedSignal = signal;
+                return (async function* () {
+                    yield { type: 'content', value: 'First event' };
+                    yield { type: 'content', value: 'Second event' };
+                })();
+            });
+            const mockChat = {
+                addHistory: vi.fn(),
+                getHistory: vi.fn().mockReturnValue([]),
+            };
+            client['chat'] = mockChat;
+            // Act
+            const stream = client.sendMessageStream([{ text: 'Hi' }], new AbortController().signal, 'prompt-id-loop');
+            const events = [];
+            for await (const event of stream) {
+                events.push(event);
+            }
+            // Assert
+            expect(events).toContainEqual({ type: GeminiEventType.LoopDetected });
+            expect(capturedSignal.aborted).toBe(true);
+        });
     });
     describe('generateContent', () => {
+        it('should call generateContent with the correct parameters', async () => {
+            const contents = [{ role: 'user', parts: [{ text: 'hello' }] }];
+            const generationConfig = { temperature: 0.5 };
+            const abortSignal = new AbortController().signal;
+            await client.generateContent(contents, generationConfig, abortSignal, DEFAULT_GEMINI_FLASH_MODEL);
+            expect(mockContentGenerator.generateContent).toHaveBeenCalledWith({
+                model: DEFAULT_GEMINI_FLASH_MODEL,
+                config: {
+                    abortSignal,
+                    systemInstruction: getCoreSystemPrompt(''),
+                    temperature: 0.5,
+                    topP: 1,
+                },
+                contents,
+            }, 'test-session-id');
+        });
         it('should use current model from config for content generation', async () => {
             const initialModel = client['config'].getModel();
             const contents = [{ role: 'user', parts: [{ text: 'test' }] }];
             const currentModel = initialModel + '-changed';
             vi.spyOn(client['config'], 'getModel').mockReturnValueOnce(currentModel);
-            const mockGenerator = {
-                countTokens: vi.fn().mockResolvedValue({ totalTokens: 1 }),
-                generateContent: mockGenerateContentFn,
-            };
-            client['contentGenerator'] = mockGenerator;
-            await client.generateContent(contents, {}, new AbortController().signal);
-            expect(mockGenerateContentFn).not.toHaveBeenCalledWith({
+            await client.generateContent(contents, {}, new AbortController().signal, DEFAULT_GEMINI_FLASH_MODEL);
+            expect(mockContentGenerator.generateContent).not.toHaveBeenCalledWith({
                 model: initialModel,
                 config: expect.any(Object),
                 contents,
             });
-            expect(mockGenerateContentFn).toHaveBeenCalledWith({
-                model: currentModel,
+            expect(mockContentGenerator.generateContent).toHaveBeenCalledWith({
+                model: DEFAULT_GEMINI_FLASH_MODEL,
                 config: expect.any(Object),
                 contents,
             }, 'test-session-id');
         });
-    });
-    describe('tryCompressChat', () => {
-        it('should use current model from config for token counting after sendMessage', async () => {
-            const initialModel = client['config'].getModel();
-            const mockCountTokens = vi
-                .fn()
-                .mockResolvedValueOnce({ totalTokens: 100000 })
-                .mockResolvedValueOnce({ totalTokens: 5000 });
-            const mockSendMessage = vi.fn().mockResolvedValue({ text: 'Summary' });
-            const mockChatHistory = [
-                { role: 'user', parts: [{ text: 'Long conversation' }] },
-                { role: 'model', parts: [{ text: 'Long response' }] },
-            ];
-            const mockChat = {
-                getHistory: vi.fn().mockReturnValue(mockChatHistory),
-                setHistory: vi.fn(),
-                sendMessage: mockSendMessage,
-            };
-            const mockGenerator = {
-                countTokens: mockCountTokens,
-            };
-            // mock the model has been changed between calls of `countTokens`
-            const firstCurrentModel = initialModel + '-changed-1';
-            const secondCurrentModel = initialModel + '-changed-2';
-            vi.spyOn(client['config'], 'getModel')
-                .mockReturnValueOnce(firstCurrentModel)
-                .mockReturnValueOnce(secondCurrentModel);
-            client['chat'] = mockChat;
-            client['contentGenerator'] = mockGenerator;
-            client['startChat'] = vi.fn().mockResolvedValue(mockChat);
-            const result = await client.tryCompressChat('prompt-id-4', true);
-            expect(mockCountTokens).toHaveBeenCalledTimes(2);
-            expect(mockCountTokens).toHaveBeenNthCalledWith(1, {
-                model: firstCurrentModel,
-                contents: mockChatHistory,
-            });
-            expect(mockCountTokens).toHaveBeenNthCalledWith(2, {
-                model: secondCurrentModel,
-                contents: expect.any(Array),
-            });
-            expect(result).toEqual({
-                originalTokenCount: 100000,
-                newTokenCount: 5000,
-            });
-        });
-    });
-    describe('handleFlashFallback', () => {
-        it('should use current model from config when checking for fallback', async () => {
-            const initialModel = client['config'].getModel();
-            const fallbackModel = DEFAULT_GEMINI_FLASH_MODEL;
-            // mock config been changed
-            const currentModel = initialModel + '-changed';
-            const getModelSpy = vi.spyOn(client['config'], 'getModel');
-            getModelSpy.mockReturnValue(currentModel);
-            const mockFallbackHandler = vi.fn().mockResolvedValue(true);
-            client['config'].flashFallbackHandler = mockFallbackHandler;
-            client['config'].setModel = vi.fn();
-            const result = await client['handleFlashFallback'](AuthType.LOGIN_WITH_GOOGLE);
-            expect(result).toBe(fallbackModel);
-            expect(mockFallbackHandler).toHaveBeenCalledWith(currentModel, fallbackModel, undefined);
-        });
-    });
-    describe('setHistory', () => {
-        it('should strip thought signatures when stripThoughts is true', () => {
-            const mockChat = {
-                setHistory: vi.fn(),
-            };
-            client['chat'] = mockChat;
-            const historyWithThoughts = [
-                {
-                    role: 'user',
-                    parts: [{ text: 'hello' }],
-                },
-                {
-                    role: 'model',
-                    parts: [
-                        { text: 'thinking...', thoughtSignature: 'thought-123' },
-                        {
-                            functionCall: { name: 'test', args: {} },
-                            thoughtSignature: 'thought-456',
-                        },
-                    ],
-                },
-            ];
-            client.setHistory(historyWithThoughts, { stripThoughts: true });
-            const expectedHistory = [
-                {
-                    role: 'user',
-                    parts: [{ text: 'hello' }],
-                },
-                {
-                    role: 'model',
-                    parts: [
-                        { text: 'thinking...' },
-                        { functionCall: { name: 'test', args: {} } },
-                    ],
-                },
-            ];
-            expect(mockChat.setHistory).toHaveBeenCalledWith(expectedHistory);
-        });
-        it('should not strip thought signatures when stripThoughts is false', () => {
-            const mockChat = {
-                setHistory: vi.fn(),
-            };
-            client['chat'] = mockChat;
-            const historyWithThoughts = [
-                {
-                    role: 'user',
-                    parts: [{ text: 'hello' }],
-                },
-                {
-                    role: 'model',
-                    parts: [
-                        { text: 'thinking...', thoughtSignature: 'thought-123' },
-                        { text: 'ok', thoughtSignature: 'thought-456' },
-                    ],
-                },
-            ];
-            client.setHistory(historyWithThoughts, { stripThoughts: false });
-            expect(mockChat.setHistory).toHaveBeenCalledWith(historyWithThoughts);
+        it('should use the Flash model when fallback mode is active', async () => {
+            const contents = [{ role: 'user', parts: [{ text: 'hello' }] }];
+            const generationConfig = { temperature: 0.5 };
+            const abortSignal = new AbortController().signal;
+            const requestedModel = 'gemini-2.5-pro'; // A non-flash model
+            // Mock config to be in fallback mode
+            vi.spyOn(client['config'], 'isInFallbackMode').mockReturnValue(true);
+            await client.generateContent(contents, generationConfig, abortSignal, requestedModel);
+            expect(mockGenerateContentFn).toHaveBeenCalledWith(expect.objectContaining({
+                model: DEFAULT_GEMINI_FLASH_MODEL,
+            }), 'test-session-id');
         });
     });
 });