@google/gemini-cli-core 0.3.0-nightly.20250822.15c62bad → 0.3.0-preview.1

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

@@ -4,7 +4,7 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
-import { GeminiChat } from './geminiChat.js';
+import { GeminiChat, EmptyStreamError } from './geminiChat.js';
 import { setSimulate429 } from '../utils/testUtils.js';
 // Mocks
 const mockModelsModule = {
@@ -14,6 +14,16 @@ const mockModelsModule = {
     embedContent: vi.fn(),
     batchEmbedContents: vi.fn(),
 };
+const { mockLogInvalidChunk, mockLogContentRetry, mockLogContentRetryFailure } = vi.hoisted(() => ({
+    mockLogInvalidChunk: vi.fn(),
+    mockLogContentRetry: vi.fn(),
+    mockLogContentRetryFailure: vi.fn(),
+}));
+vi.mock('../telemetry/loggers.js', () => ({
+    logInvalidChunk: mockLogInvalidChunk,
+    logContentRetry: mockLogContentRetry,
+    logContentRetryFailure: mockLogContentRetryFailure,
+}));
 describe('GeminiChat', () => {
     let chat;
     let mockConfig;
@@ -45,6 +55,110 @@ describe('GeminiChat', () => {
         vi.resetAllMocks();
     });
     describe('sendMessage', () => {
+        it('should throw an error when attempting to add a user turn after another user turn', async () => {
+            // 1. Setup: Create a history that already ends with a user turn (a functionResponse).
+            const initialHistory = [
+                { role: 'user', parts: [{ text: 'Initial prompt' }] },
+                {
+                    role: 'model',
+                    parts: [{ functionCall: { name: 'test_tool', args: {} } }],
+                },
+                {
+                    role: 'user',
+                    parts: [{ functionResponse: { name: 'test_tool', response: {} } }],
+                },
+            ];
+            chat.setHistory(initialHistory);
+            // 2. Mock a valid model response so the call doesn't fail for other reasons.
+            const mockResponse = {
+                candidates: [
+                    { content: { role: 'model', parts: [{ text: 'some response' }] } },
+                ],
+            };
+            vi.mocked(mockModelsModule.generateContent).mockResolvedValue(mockResponse);
+            // 3. Action & Assert: Expect that sending another user message immediately
+            //    after a user-role turn throws the specific error.
+            await expect(chat.sendMessage({ message: 'This is an invalid consecutive user message' }, 'prompt-id-1')).rejects.toThrow('Cannot add a user turn after another user turn.');
+        });
+        it('should preserve text parts that are in the same response as a thought', async () => {
+            // 1. Mock the API to return a single response containing both a thought and visible text.
+            const mixedContentResponse = {
+                candidates: [
+                    {
+                        content: {
+                            role: 'model',
+                            parts: [
+                                { thought: 'This is a thought.' },
+                                { text: 'This is the visible text that should not be lost.' },
+                            ],
+                        },
+                    },
+                ],
+            };
+            vi.mocked(mockModelsModule.generateContent).mockResolvedValue(mixedContentResponse);
+            // 2. Action: Send a standard, non-streaming message.
+            await chat.sendMessage({ message: 'test message' }, 'prompt-id-mixed-response');
+            // 3. Assert: Check the final state of the history.
+            const history = chat.getHistory();
+            // The history should contain two turns: the user's message and the model's response.
+            expect(history.length).toBe(2);
+            const modelTurn = history[1];
+            expect(modelTurn.role).toBe('model');
+            // CRUCIAL ASSERTION:
+            // Buggy code would discard the entire response because a "thought" was present,
+            // resulting in an empty placeholder turn with 0 parts.
+            // The corrected code will pass, preserving the single visible text part.
+            expect(modelTurn?.parts?.length).toBe(1);
+            expect(modelTurn?.parts[0].text).toBe('This is the visible text that should not be lost.');
+        });
+        it('should add a placeholder model turn when a tool call is followed by an empty model response', async () => {
+            // 1. Setup: A history where the model has just made a function call.
+            const initialHistory = [
+                {
+                    role: 'user',
+                    parts: [{ text: 'Find a good Italian restaurant for me.' }],
+                },
+                {
+                    role: 'model',
+                    parts: [
+                        {
+                            functionCall: {
+                                name: 'find_restaurant',
+                                args: { cuisine: 'Italian' },
+                            },
+                        },
+                    ],
+                },
+            ];
+            chat.setHistory(initialHistory);
+            // 2. Mock the API to return an empty/thought-only response.
+            const emptyModelResponse = {
+                candidates: [
+                    { content: { role: 'model', parts: [{ thought: true }] } },
+                ],
+            };
+            vi.mocked(mockModelsModule.generateContent).mockResolvedValue(emptyModelResponse);
+            // 3. Action: Send the function response back to the model.
+            await chat.sendMessage({
+                message: {
+                    functionResponse: {
+                        name: 'find_restaurant',
+                        response: { name: 'Vesuvio' },
+                    },
+                },
+            }, 'prompt-id-1');
+            // 4. Assert: The history should now have four valid, alternating turns.
+            const history = chat.getHistory();
+            expect(history.length).toBe(4);
+            // The final turn must be the empty model placeholder.
+            const lastTurn = history[3];
+            expect(lastTurn.role).toBe('model');
+            expect(lastTurn?.parts?.length).toBe(0);
+            // The second-to-last turn must be the function response we sent.
+            const secondToLastTurn = history[2];
+            expect(secondToLastTurn.role).toBe('user');
+            expect(secondToLastTurn?.parts[0].functionResponse).toBeDefined();
+        });
         it('should call generateContent with the correct parameters', async () => {
             const response = {
                 candidates: [
@@ -70,6 +184,226 @@ describe('GeminiChat', () => {
         });
     });
     describe('sendMessageStream', () => {
+        it('should not consolidate text into a part that also contains a functionCall', async () => {
+            // 1. Mock the API to stream a malformed part followed by a valid text part.
+            const multiChunkStream = (async function* () {
+                // This malformed part has both text and a functionCall.
+                yield {
+                    candidates: [
+                        {
+                            content: {
+                                role: 'model',
+                                parts: [
+                                    {
+                                        text: 'Some text',
+                                        functionCall: { name: 'do_stuff', args: {} },
+                                    },
+                                ],
+                            },
+                        },
+                    ],
+                };
+                // This valid text part should NOT be merged into the malformed one.
+                yield {
+                    candidates: [
+                        {
+                            content: {
+                                role: 'model',
+                                parts: [{ text: ' that should not be merged.' }],
+                            },
+                        },
+                    ],
+                };
+            })();
+            vi.mocked(mockModelsModule.generateContentStream).mockResolvedValue(multiChunkStream);
+            // 2. Action: Send a message and consume the stream.
+            const stream = await chat.sendMessageStream({ message: 'test message' }, 'prompt-id-malformed-chunk');
+            for await (const _ of stream) {
+                // Consume the stream to trigger history recording.
+            }
+            // 3. Assert: Check that the final history was not incorrectly consolidated.
+            const history = chat.getHistory();
+            expect(history.length).toBe(2);
+            const modelTurn = history[1];
+            // CRUCIAL ASSERTION: There should be two separate parts.
+            // The old, non-strict logic would incorrectly merge them, resulting in one part.
+            expect(modelTurn?.parts?.length).toBe(2);
+            // Verify the contents of each part.
+            expect(modelTurn?.parts[0].text).toBe('Some text');
+            expect(modelTurn?.parts[0].functionCall).toBeDefined();
+            expect(modelTurn?.parts[1].text).toBe(' that should not be merged.');
+        });
+        it('should consolidate subsequent text chunks after receiving an empty text chunk', async () => {
+            // 1. Mock the API to return a stream where one chunk is just an empty text part.
+            const multiChunkStream = (async function* () {
+                yield {
+                    candidates: [
+                        { content: { role: 'model', parts: [{ text: 'Hello' }] } },
+                    ],
+                };
+                // FIX: The original test used { text: '' }, which is invalid.
+                // A chunk can be empty but still valid. This chunk is now removed
+                // as the important part is consolidating what comes after.
+                yield {
+                    candidates: [
+                        { content: { role: 'model', parts: [{ text: ' World!' }] } },
+                    ],
+                };
+            })();
+            vi.mocked(mockModelsModule.generateContentStream).mockResolvedValue(multiChunkStream);
+            // 2. Action: Send a message and consume the stream.
+            const stream = await chat.sendMessageStream({ message: 'test message' }, 'prompt-id-empty-chunk-consolidation');
+            for await (const _ of stream) {
+                // Consume the stream
+            }
+            // 3. Assert: Check that the final history was correctly consolidated.
+            const history = chat.getHistory();
+            expect(history.length).toBe(2);
+            const modelTurn = history[1];
+            expect(modelTurn?.parts?.length).toBe(1);
+            expect(modelTurn?.parts[0].text).toBe('Hello World!');
+        });
+        it('should consolidate adjacent text parts that arrive in separate stream chunks', async () => {
+            // 1. Mock the API to return a stream of multiple, adjacent text chunks.
+            const multiChunkStream = (async function* () {
+                yield {
+                    candidates: [
+                        { content: { role: 'model', parts: [{ text: 'This is the ' }] } },
+                    ],
+                };
+                yield {
+                    candidates: [
+                        { content: { role: 'model', parts: [{ text: 'first part.' }] } },
+                    ],
+                };
+                // This function call should break the consolidation.
+                yield {
+                    candidates: [
+                        {
+                            content: {
+                                role: 'model',
+                                parts: [{ functionCall: { name: 'do_stuff', args: {} } }],
+                            },
+                        },
+                    ],
+                };
+                yield {
+                    candidates: [
+                        {
+                            content: {
+                                role: 'model',
+                                parts: [{ text: 'This is the second part.' }],
+                            },
+                        },
+                    ],
+                };
+            })();
+            vi.mocked(mockModelsModule.generateContentStream).mockResolvedValue(multiChunkStream);
+            // 2. Action: Send a message and consume the stream.
+            const stream = await chat.sendMessageStream({ message: 'test message' }, 'prompt-id-multi-chunk');
+            for await (const _ of stream) {
+                // Consume the stream to trigger history recording.
+            }
+            // 3. Assert: Check that the final history was correctly consolidated.
+            const history = chat.getHistory();
+            // The history should contain the user's turn and ONE consolidated model turn.
+            expect(history.length).toBe(2);
+            const modelTurn = history[1];
+            expect(modelTurn.role).toBe('model');
+            // The model turn should have 3 distinct parts: the merged text, the function call, and the final text.
+            expect(modelTurn?.parts?.length).toBe(3);
+            expect(modelTurn?.parts[0].text).toBe('This is the first part.');
+            expect(modelTurn.parts[1].functionCall).toBeDefined();
+            expect(modelTurn.parts[2].text).toBe('This is the second part.');
+        });
+        it('should preserve text parts that stream in the same chunk as a thought', async () => {
+            // 1. Mock the API to return a single chunk containing both a thought and visible text.
+            const mixedContentStream = (async function* () {
+                yield {
+                    candidates: [
+                        {
+                            content: {
+                                role: 'model',
+                                parts: [
+                                    { thought: 'This is a thought.' },
+                                    { text: 'This is the visible text that should not be lost.' },
+                                ],
+                            },
+                        },
+                    ],
+                };
+            })();
+            vi.mocked(mockModelsModule.generateContentStream).mockResolvedValue(mixedContentStream);
+            // 2. Action: Send a message and fully consume the stream to trigger history recording.
+            const stream = await chat.sendMessageStream({ message: 'test message' }, 'prompt-id-mixed-chunk');
+            for await (const _ of stream) {
+                // This loop consumes the stream.
+            }
+            // 3. Assert: Check the final state of the history.
+            const history = chat.getHistory();
+            // The history should contain two turns: the user's message and the model's response.
+            expect(history.length).toBe(2);
+            const modelTurn = history[1];
+            expect(modelTurn.role).toBe('model');
+            // CRUCIAL ASSERTION:
+            // The buggy code would fail here, resulting in parts.length being 0.
+            // The corrected code will pass, preserving the single visible text part.
+            expect(modelTurn?.parts?.length).toBe(1);
+            expect(modelTurn?.parts[0].text).toBe('This is the visible text that should not be lost.');
+        });
+        it('should add a placeholder model turn when a tool call is followed by an empty stream response', async () => {
+            // 1. Setup: A history where the model has just made a function call.
+            const initialHistory = [
+                {
+                    role: 'user',
+                    parts: [{ text: 'Find a good Italian restaurant for me.' }],
+                },
+                {
+                    role: 'model',
+                    parts: [
+                        {
+                            functionCall: {
+                                name: 'find_restaurant',
+                                args: { cuisine: 'Italian' },
+                            },
+                        },
+                    ],
+                },
+            ];
+            chat.setHistory(initialHistory);
+            // 2. Mock the API to return an empty/thought-only stream.
+            const emptyStreamResponse = (async function* () {
+                yield {
+                    candidates: [
+                        { content: { role: 'model', parts: [{ thought: true }] } },
+                    ],
+                };
+            })();
+            vi.mocked(mockModelsModule.generateContentStream).mockResolvedValue(emptyStreamResponse);
+            // 3. Action: Send the function response back to the model and consume the stream.
+            const stream = await chat.sendMessageStream({
+                message: {
+                    functionResponse: {
+                        name: 'find_restaurant',
+                        response: { name: 'Vesuvio' },
+                    },
+                },
+            }, 'prompt-id-stream-1');
+            for await (const _ of stream) {
+                // This loop consumes the stream to trigger the internal logic.
+            }
+            // 4. Assert: The history should now have four valid, alternating turns.
+            const history = chat.getHistory();
+            expect(history.length).toBe(4);
+            // The final turn must be the empty model placeholder.
+            const lastTurn = history[3];
+            expect(lastTurn.role).toBe('model');
+            expect(lastTurn?.parts?.length).toBe(0);
+            // The second-to-last turn must be the function response we sent.
+            const secondToLastTurn = history[2];
+            expect(secondToLastTurn.role).toBe('user');
+            expect(secondToLastTurn?.parts[0].functionResponse).toBeDefined();
+        });
         it('should call generateContentStream with the correct parameters', async () => {
             const response = (async function* () {
                 yield {
@@ -88,7 +422,10 @@ describe('GeminiChat', () => {
                 };
             })();
             vi.mocked(mockModelsModule.generateContentStream).mockResolvedValue(response);
-            await chat.sendMessageStream({ message: 'hello' }, 'prompt-id-1');
+            const stream = await chat.sendMessageStream({ message: 'hello' }, 'prompt-id-1');
+            for await (const _ of stream) {
+                // consume stream to trigger internal logic
+            }
             expect(mockModelsModule.generateContentStream).toHaveBeenCalledWith({
                 model: 'gemini-pro',
                 contents: [{ role: 'user', parts: [{ text: 'hello' }] }],
@@ -101,295 +438,128 @@ describe('GeminiChat', () => {
             role: 'user',
             parts: [{ text: 'User input' }],
         };
-        it('should add user input and a single model output to history', () => {
+        it('should consolidate all consecutive model turns into a single turn', () => {
+            const userInput = {
+                role: 'user',
+                parts: [{ text: 'User input' }],
+            };
+            // This simulates a multi-part model response with different part types.
             const modelOutput = [
-                { role: 'model', parts: [{ text: 'Model output' }] },
+                { role: 'model', parts: [{ text: 'Thinking...' }] },
+                {
+                    role: 'model',
+                    parts: [{ functionCall: { name: 'do_stuff', args: {} } }],
+                },
             ];
-            // @ts-expect-error Accessing private method for testing purposes
+            // @ts-expect-error Accessing private method for testing
             chat.recordHistory(userInput, modelOutput);
             const history = chat.getHistory();
-            expect(history).toEqual([userInput, modelOutput[0]]);
-        });
-        it('should consolidate adjacent model outputs', () => {
-            const modelOutputParts = [
-                { role: 'model', parts: [{ text: 'Model part 1' }] },
-                { role: 'model', parts: [{ text: 'Model part 2' }] },
-            ];
-            // @ts-expect-error Accessing private method for testing purposes
-            chat.recordHistory(userInput, modelOutputParts);
-            const history = chat.getHistory();
+            // The history should contain the user's turn and ONE consolidated model turn.
+            // The old code would fail here, resulting in a length of 3.
+            //expect(history).toBe([]);
             expect(history.length).toBe(2);
-            expect(history[0]).toEqual(userInput);
-            expect(history[1].role).toBe('model');
-            expect(history[1].parts).toEqual([{ text: 'Model part 1Model part 2' }]);
+            const modelTurn = history[1];
+            expect(modelTurn.role).toBe('model');
+            // The consolidated turn should contain both the text part and the functionCall part.
+            expect(modelTurn?.parts?.length).toBe(2);
+            expect(modelTurn?.parts[0].text).toBe('Thinking...');
+            expect(modelTurn?.parts[1].functionCall).toBeDefined();
         });
-        it('should handle a mix of user and model roles in outputContents (though unusual)', () => {
-            const mixedOutput = [
-                { role: 'model', parts: [{ text: 'Model 1' }] },
-                { role: 'user', parts: [{ text: 'Unexpected User' }] }, // This should be pushed as is
-                { role: 'model', parts: [{ text: 'Model 2' }] },
-            ];
-            // @ts-expect-error Accessing private method for testing purposes
-            chat.recordHistory(userInput, mixedOutput);
-            const history = chat.getHistory();
-            expect(history.length).toBe(4); // user, model1, user_unexpected, model2
-            expect(history[0]).toEqual(userInput);
-            expect(history[1]).toEqual(mixedOutput[0]);
-            expect(history[2]).toEqual(mixedOutput[1]);
-            expect(history[3]).toEqual(mixedOutput[2]);
-        });
-        it('should consolidate multiple adjacent model outputs correctly', () => {
-            const modelOutputParts = [
-                { role: 'model', parts: [{ text: 'M1' }] },
-                { role: 'model', parts: [{ text: 'M2' }] },
-                { role: 'model', parts: [{ text: 'M3' }] },
-            ];
-            // @ts-expect-error Accessing private method for testing purposes
-            chat.recordHistory(userInput, modelOutputParts);
-            const history = chat.getHistory();
-            expect(history.length).toBe(2);
-            expect(history[1].parts).toEqual([{ text: 'M1M2M3' }]);
-        });
-        it('should not consolidate if roles are different between model outputs', () => {
-            const modelOutputParts = [
-                { role: 'model', parts: [{ text: 'M1' }] },
-                { role: 'user', parts: [{ text: 'Interjecting User' }] },
-                { role: 'model', parts: [{ text: 'M2' }] },
-            ];
-            // @ts-expect-error Accessing private method for testing purposes
-            chat.recordHistory(userInput, modelOutputParts);
-            const history = chat.getHistory();
-            expect(history.length).toBe(4); // user, M1, Interjecting User, M2
-            expect(history[1].parts).toEqual([{ text: 'M1' }]);
-            expect(history[3].parts).toEqual([{ text: 'M2' }]);
-        });
-        it('should merge with last history entry if it is also a model output', () => {
-            // @ts-expect-error Accessing private property for test setup
-            chat.history = [
-                userInput,
-                { role: 'model', parts: [{ text: 'Initial Model Output' }] },
-            ]; // Prime the history
-            const newModelOutput = [
-                { role: 'model', parts: [{ text: 'New Model Part 1' }] },
-                { role: 'model', parts: [{ text: 'New Model Part 2' }] },
-            ];
-            // @ts-expect-error Accessing private method for testing purposes
-            chat.recordHistory(userInput, newModelOutput); // userInput here is for the *next* turn, but history is already primed
-            // Reset and set up a more realistic scenario for merging with existing history
-            chat = new GeminiChat(mockConfig, mockModelsModule, config, []);
-            const firstUserInput = {
-                role: 'user',
-                parts: [{ text: 'First user input' }],
-            };
-            const firstModelOutput = [
-                { role: 'model', parts: [{ text: 'First model response' }] },
-            ];
-            // @ts-expect-error Accessing private method for testing purposes
-            chat.recordHistory(firstUserInput, firstModelOutput);
-            const secondUserInput = {
-                role: 'user',
-                parts: [{ text: 'Second user input' }],
-            };
-            const secondModelOutput = [
-                { role: 'model', parts: [{ text: 'Second model response part 1' }] },
-                { role: 'model', parts: [{ text: 'Second model response part 2' }] },
+        it('should add a placeholder model turn when a tool call is followed by an empty response', () => {
+            // 1. Setup: A history where the model has just made a function call.
+            const initialHistory = [
+                { role: 'user', parts: [{ text: 'Initial prompt' }] },
+                {
+                    role: 'model',
+                    parts: [{ functionCall: { name: 'test_tool', args: {} } }],
+                },
             ];
-            // @ts-expect-error Accessing private method for testing purposes
-            chat.recordHistory(secondUserInput, secondModelOutput);
-            const finalHistory = chat.getHistory();
-            expect(finalHistory.length).toBe(4); // user1, model1, user2, model2(consolidated)
-            expect(finalHistory[0]).toEqual(firstUserInput);
-            expect(finalHistory[1]).toEqual(firstModelOutput[0]);
-            expect(finalHistory[2]).toEqual(secondUserInput);
-            expect(finalHistory[3].role).toBe('model');
-            expect(finalHistory[3].parts).toEqual([
-                { text: 'Second model response part 1Second model response part 2' },
-            ]);
-        });
-        it('should correctly merge consolidated new output with existing model history', () => {
-            // Setup: history ends with a model turn
-            const initialUser = {
+            chat.setHistory(initialHistory);
+            // 2. Action: The user provides the tool's response, and the model's
+            // final output is empty (e.g., just a thought, which gets filtered out).
+            const functionResponse = {
                 role: 'user',
-                parts: [{ text: 'Initial user query' }],
-            };
-            const initialModel = {
-                role: 'model',
-                parts: [{ text: 'Initial model answer.' }],
+                parts: [{ functionResponse: { name: 'test_tool', response: {} } }],
             };
-            chat = new GeminiChat(mockConfig, mockModelsModule, config, [
-                initialUser,
-                initialModel,
+            const emptyModelOutput = [];
+            // @ts-expect-error Accessing private method for testing
+            chat.recordHistory(functionResponse, emptyModelOutput, [
+                functionResponse,
             ]);
-            // New interaction
-            const currentUserInput = {
-                role: 'user',
-                parts: [{ text: 'Follow-up question' }],
-            };
-            const newModelParts = [
-                { role: 'model', parts: [{ text: 'Part A of new answer.' }] },
-                { role: 'model', parts: [{ text: 'Part B of new answer.' }] },
-            ];
-            // @ts-expect-error Accessing private method for testing purposes
-            chat.recordHistory(currentUserInput, newModelParts);
+            // 3. Assert: The history should now have four valid, alternating turns.
             const history = chat.getHistory();
-            // Expected: initialUser, initialModel, currentUserInput, consolidatedNewModelParts
             expect(history.length).toBe(4);
-            expect(history[0]).toEqual(initialUser);
-            expect(history[1]).toEqual(initialModel);
-            expect(history[2]).toEqual(currentUserInput);
-            expect(history[3].role).toBe('model');
-            expect(history[3].parts).toEqual([
-                { text: 'Part A of new answer.Part B of new answer.' },
-            ]);
+            // The final turn must be the empty model placeholder.
+            const lastTurn = history[3];
+            expect(lastTurn.role).toBe('model');
+            expect(lastTurn?.parts?.length).toBe(0);
+            // The second-to-last turn must be the function response we provided.
+            const secondToLastTurn = history[2];
+            expect(secondToLastTurn.role).toBe('user');
+            expect(secondToLastTurn?.parts[0].functionResponse).toBeDefined();
         });
-        it('should handle empty modelOutput array', () => {
-            // @ts-expect-error Accessing private method for testing purposes
-            chat.recordHistory(userInput, []);
-            const history = chat.getHistory();
-            // If modelOutput is empty, it might push a default empty model part depending on isFunctionResponse
-            // Assuming isFunctionResponse(userInput) is false for this simple text input
-            expect(history.length).toBe(2);
-            expect(history[0]).toEqual(userInput);
-            expect(history[1].role).toBe('model');
-            expect(history[1].parts).toEqual([]);
-        });
-        it('should handle aggregating modelOutput', () => {
-            const modelOutputUndefinedParts = [
-                { role: 'model', parts: [{ text: 'First model part' }] },
-                { role: 'model', parts: [{ text: 'Second model part' }] },
-                { role: 'model', parts: undefined }, // Test undefined parts
-                { role: 'model', parts: [{ text: 'Third model part' }] },
-                { role: 'model', parts: [] }, // Test empty parts array
-            ];
-            // @ts-expect-error Accessing private method for testing purposes
-            chat.recordHistory(userInput, modelOutputUndefinedParts);
-            const history = chat.getHistory();
-            expect(history.length).toBe(5);
-            expect(history[0]).toEqual(userInput);
-            expect(history[1].role).toBe('model');
-            expect(history[1].parts).toEqual([
-                { text: 'First model partSecond model part' },
-            ]);
-            expect(history[2].role).toBe('model');
-            expect(history[2].parts).toBeUndefined();
-            expect(history[3].role).toBe('model');
-            expect(history[3].parts).toEqual([{ text: 'Third model part' }]);
-            expect(history[4].role).toBe('model');
-            expect(history[4].parts).toEqual([]);
-        });
-        it('should handle modelOutput with parts being undefined or empty (if they pass initial every check)', () => {
-            const modelOutputUndefinedParts = [
-                { role: 'model', parts: [{ text: 'Text part' }] },
-                { role: 'model', parts: undefined }, // Test undefined parts
-                { role: 'model', parts: [] }, // Test empty parts array
-            ];
-            // @ts-expect-error Accessing private method for testing purposes
-            chat.recordHistory(userInput, modelOutputUndefinedParts);
-            const history = chat.getHistory();
-            expect(history.length).toBe(4); // userInput, model1 (text), model2 (undefined parts), model3 (empty parts)
-            expect(history[0]).toEqual(userInput);
-            expect(history[1].role).toBe('model');
-            expect(history[1].parts).toEqual([{ text: 'Text part' }]);
-            expect(history[2].role).toBe('model');
-            expect(history[2].parts).toBeUndefined();
-            expect(history[3].role).toBe('model');
-            expect(history[3].parts).toEqual([]);
-        });
-        it('should correctly handle automaticFunctionCallingHistory', () => {
-            const afcHistory = [
-                { role: 'user', parts: [{ text: 'AFC User' }] },
-                { role: 'model', parts: [{ text: 'AFC Model' }] },
-            ];
-            const modelOutput = [
-                { role: 'model', parts: [{ text: 'Regular Model Output' }] },
-            ];
-            // @ts-expect-error Accessing private method for testing purposes
-            chat.recordHistory(userInput, modelOutput, afcHistory);
-            const history = chat.getHistory();
-            expect(history.length).toBe(3);
-            expect(history[0]).toEqual(afcHistory[0]);
-            expect(history[1]).toEqual(afcHistory[1]);
-            expect(history[2]).toEqual(modelOutput[0]);
-        });
-        it('should add userInput if AFC history is present but empty', () => {
+        it('should add user input and a single model output to history', () => {
             const modelOutput = [
-                { role: 'model', parts: [{ text: 'Model Output' }] },
+                { role: 'model', parts: [{ text: 'Model output' }] },
             ];
-            // @ts-expect-error Accessing private method for testing purposes
-            chat.recordHistory(userInput, modelOutput, []); // Empty AFC history
+            // @ts-expect-error Accessing private method for testing
+            chat.recordHistory(userInput, modelOutput);
             const history = chat.getHistory();
             expect(history.length).toBe(2);
             expect(history[0]).toEqual(userInput);
             expect(history[1]).toEqual(modelOutput[0]);
         });
-        it('should skip "thought" content from modelOutput', () => {
-            const modelOutputWithThought = [
-                { role: 'model', parts: [{ thought: true }, { text: 'Visible text' }] },
-                { role: 'model', parts: [{ text: 'Another visible text' }] },
+        it('should consolidate adjacent text parts from multiple content objects', () => {
+            const modelOutput = [
+                { role: 'model', parts: [{ text: 'Part 1.' }] },
+                { role: 'model', parts: [{ text: ' Part 2.' }] },
+                { role: 'model', parts: [{ text: ' Part 3.' }] },
             ];
-            // @ts-expect-error Accessing private method for testing purposes
-            chat.recordHistory(userInput, modelOutputWithThought);
+            // @ts-expect-error Accessing private method for testing
+            chat.recordHistory(userInput, modelOutput);
             const history = chat.getHistory();
-            expect(history.length).toBe(2); // User input + consolidated model output
-            expect(history[0]).toEqual(userInput);
+            expect(history.length).toBe(2);
             expect(history[1].role).toBe('model');
-            // The 'thought' part is skipped, 'Another visible text' becomes the first part.
-            expect(history[1].parts).toEqual([{ text: 'Another visible text' }]);
+            expect(history[1].parts).toEqual([{ text: 'Part 1. Part 2. Part 3.' }]);
         });
-        it('should skip "thought" content even if it is the only content', () => {
-            const modelOutputOnlyThought = [
-                { role: 'model', parts: [{ thought: true }] },
-            ];
-            // @ts-expect-error Accessing private method for testing purposes
-            chat.recordHistory(userInput, modelOutputOnlyThought);
-            const history = chat.getHistory();
-            expect(history.length).toBe(1); // User input + default empty model part
-            expect(history[0]).toEqual(userInput);
-        });
-        it('should correctly consolidate text parts when a thought part is in between', () => {
-            const modelOutputMixed = [
-                { role: 'model', parts: [{ text: 'Part 1.' }] },
-                {
-                    role: 'model',
-                    parts: [{ thought: true }, { text: 'Should be skipped' }],
-                },
-                { role: 'model', parts: [{ text: 'Part 2.' }] },
-            ];
-            // @ts-expect-error Accessing private method for testing purposes
-            chat.recordHistory(userInput, modelOutputMixed);
+        it('should add an empty placeholder turn if modelOutput is empty', () => {
+            // This simulates receiving a pre-filtered, thought-only response.
+            const emptyModelOutput = [];
+            // @ts-expect-error Accessing private method for testing
+            chat.recordHistory(userInput, emptyModelOutput);
             const history = chat.getHistory();
             expect(history.length).toBe(2);
             expect(history[0]).toEqual(userInput);
             expect(history[1].role).toBe('model');
-            expect(history[1].parts).toEqual([{ text: 'Part 1.Part 2.' }]);
+            expect(history[1].parts).toEqual([]);
         });
-        it('should handle multiple thought parts correctly', () => {
-            const modelOutputMultipleThoughts = [
-                { role: 'model', parts: [{ thought: true }] },
-                { role: 'model', parts: [{ text: 'Visible 1' }] },
-                { role: 'model', parts: [{ thought: true }] },
-                { role: 'model', parts: [{ text: 'Visible 2' }] },
+        it('should preserve model outputs with undefined or empty parts arrays', () => {
+            const malformedOutput = [
+                { role: 'model', parts: [{ text: 'Text part' }] },
+                { role: 'model', parts: undefined },
+                { role: 'model', parts: [] },
             ];
-            // @ts-expect-error Accessing private method for testing purposes
-            chat.recordHistory(userInput, modelOutputMultipleThoughts);
+            // @ts-expect-error Accessing private method for testing
+            chat.recordHistory(userInput, malformedOutput);
             const history = chat.getHistory();
-            expect(history.length).toBe(2);
-            expect(history[0]).toEqual(userInput);
-            expect(history[1].role).toBe('model');
-            expect(history[1].parts).toEqual([{ text: 'Visible 1Visible 2' }]);
+            expect(history.length).toBe(4); // userInput + 3 model turns
+            expect(history[1].parts).toEqual([{ text: 'Text part' }]);
+            expect(history[2].parts).toBeUndefined();
+            expect(history[3].parts).toEqual([]);
         });
-        it('should handle thought part at the end of outputContents', () => {
-            const modelOutputThoughtAtEnd = [
-                { role: 'model', parts: [{ text: 'Visible text' }] },
-                { role: 'model', parts: [{ thought: true }] },
+        it('should not consolidate content with different roles', () => {
+            const mixedOutput = [
+                { role: 'model', parts: [{ text: 'Model 1' }] },
+                { role: 'user', parts: [{ text: 'Unexpected User' }] },
+                { role: 'model', parts: [{ text: 'Model 2' }] },
             ];
-            // @ts-expect-error Accessing private method for testing purposes
-            chat.recordHistory(userInput, modelOutputThoughtAtEnd);
+            // @ts-expect-error Accessing private method for testing
+            chat.recordHistory(userInput, mixedOutput);
             const history = chat.getHistory();
-            expect(history.length).toBe(2);
-            expect(history[0]).toEqual(userInput);
-            expect(history[1].role).toBe('model');
-            expect(history[1].parts).toEqual([{ text: 'Visible text' }]);
+            expect(history.length).toBe(4); // userInput, model1, unexpected_user, model2
+            expect(history[1]).toEqual(mixedOutput[0]);
+            expect(history[2]).toEqual(mixedOutput[1]);
+            expect(history[3]).toEqual(mixedOutput[2]);
         });
     });
     describe('addHistory', () => {
@@ -420,5 +590,286 @@ describe('GeminiChat', () => {
             expect(history[1]).toEqual(content2);
         });
     });
+    describe('sendMessageStream with retries', () => {
+        it('should retry on invalid content, succeed, and report metrics', async () => {
+            // Use mockImplementationOnce to provide a fresh, promise-wrapped generator for each attempt.
+            vi.mocked(mockModelsModule.generateContentStream)
+                .mockImplementationOnce(async () => 
+            // First call returns an invalid stream
+            (async function* () {
+                yield {
+                    candidates: [{ content: { parts: [{ text: '' }] } }], // Invalid empty text part
+                };
+            })())
+                .mockImplementationOnce(async () => 
+            // Second call returns a valid stream
+            (async function* () {
+                yield {
+                    candidates: [
+                        { content: { parts: [{ text: 'Successful response' }] } },
+                    ],
+                };
+            })());
+            const stream = await chat.sendMessageStream({ message: 'test' }, 'prompt-id-retry-success');
+            const chunks = [];
+            for await (const chunk of stream) {
+                chunks.push(chunk);
+            }
+            // Assertions
+            expect(mockLogInvalidChunk).toHaveBeenCalledTimes(1);
+            expect(mockLogContentRetry).toHaveBeenCalledTimes(1);
+            expect(mockLogContentRetryFailure).not.toHaveBeenCalled();
+            expect(mockModelsModule.generateContentStream).toHaveBeenCalledTimes(2);
+            expect(chunks.some((c) => c.candidates?.[0]?.content?.parts?.[0]?.text ===
+                'Successful response')).toBe(true);
+            // Check that history was recorded correctly once, with no duplicates.
+            const history = chat.getHistory();
+            expect(history.length).toBe(2);
+            expect(history[0]).toEqual({
+                role: 'user',
+                parts: [{ text: 'test' }],
+            });
+            expect(history[1]).toEqual({
+                role: 'model',
+                parts: [{ text: 'Successful response' }],
+            });
+        });
+        it('should fail after all retries on persistent invalid content and report metrics', async () => {
+            vi.mocked(mockModelsModule.generateContentStream).mockImplementation(async () => (async function* () {
+                yield {
+                    candidates: [
+                        {
+                            content: {
+                                parts: [{ text: '' }],
+                                role: 'model',
+                            },
+                        },
+                    ],
+                };
+            })());
+            // This helper function consumes the stream and allows us to test for rejection.
+            async function consumeStreamAndExpectError() {
+                const stream = await chat.sendMessageStream({ message: 'test' }, 'prompt-id-retry-fail');
+                for await (const _ of stream) {
+                    // Must loop to trigger the internal logic that throws.
+                }
+            }
+            await expect(consumeStreamAndExpectError()).rejects.toThrow(EmptyStreamError);
+            // Should be called 3 times (initial + 2 retries)
+            expect(mockModelsModule.generateContentStream).toHaveBeenCalledTimes(3);
+            expect(mockLogInvalidChunk).toHaveBeenCalledTimes(3);
+            expect(mockLogContentRetry).toHaveBeenCalledTimes(2);
+            expect(mockLogContentRetryFailure).toHaveBeenCalledTimes(1);
+            // History should be clean, as if the failed turn never happened.
+            const history = chat.getHistory();
+            expect(history.length).toBe(0);
+        });
+    });
+    it('should correctly retry and append to an existing history mid-conversation', async () => {
+        // 1. Setup
+        const initialHistory = [
+            { role: 'user', parts: [{ text: 'First question' }] },
+            { role: 'model', parts: [{ text: 'First answer' }] },
+        ];
+        chat.setHistory(initialHistory);
+        // 2. Mock the API to fail once with an empty stream, then succeed.
+        vi.mocked(mockModelsModule.generateContentStream)
+            .mockImplementationOnce(async () => (async function* () {
+            yield {
+                candidates: [{ content: { parts: [{ text: '' }] } }],
+            };
+        })())
+            .mockImplementationOnce(async () => 
+        // Second attempt succeeds
+        (async function* () {
+            yield {
+                candidates: [{ content: { parts: [{ text: 'Second answer' }] } }],
+            };
+        })());
+        // 3. Send a new message
+        const stream = await chat.sendMessageStream({ message: 'Second question' }, 'prompt-id-retry-existing');
+        for await (const _ of stream) {
+            // consume stream
+        }
+        // 4. Assert the final history and metrics
+        const history = chat.getHistory();
+        expect(history.length).toBe(4);
+        // Assert that the correct metrics were reported for one empty-stream retry
+        expect(mockLogContentRetry).toHaveBeenCalledTimes(1);
+        // Explicitly verify the structure of each part to satisfy TypeScript
+        const turn1 = history[0];
+        if (!turn1?.parts?.[0] || !('text' in turn1.parts[0])) {
+            throw new Error('Test setup error: First turn is not a valid text part.');
+        }
+        expect(turn1.parts[0].text).toBe('First question');
+        const turn2 = history[1];
+        if (!turn2?.parts?.[0] || !('text' in turn2.parts[0])) {
+            throw new Error('Test setup error: Second turn is not a valid text part.');
+        }
+        expect(turn2.parts[0].text).toBe('First answer');
+        const turn3 = history[2];
+        if (!turn3?.parts?.[0] || !('text' in turn3.parts[0])) {
+            throw new Error('Test setup error: Third turn is not a valid text part.');
+        }
+        expect(turn3.parts[0].text).toBe('Second question');
+        const turn4 = history[3];
+        if (!turn4?.parts?.[0] || !('text' in turn4.parts[0])) {
+            throw new Error('Test setup error: Fourth turn is not a valid text part.');
+        }
+        expect(turn4.parts[0].text).toBe('Second answer');
+    });
+    describe('concurrency control', () => {
+        it('should queue a subsequent sendMessage call until the first one completes', async () => {
+            // 1. Create promises to manually control when the API calls resolve
+            let firstCallResolver;
+            const firstCallPromise = new Promise((resolve) => {
+                firstCallResolver = resolve;
+            });
+            let secondCallResolver;
+            const secondCallPromise = new Promise((resolve) => {
+                secondCallResolver = resolve;
+            });
+            // A standard response body for the mock
+            const mockResponse = {
+                candidates: [
+                    {
+                        content: { parts: [{ text: 'response' }], role: 'model' },
+                    },
+                ],
+            };
+            // 2. Mock the API to return our controllable promises in order
+            vi.mocked(mockModelsModule.generateContent)
+                .mockReturnValueOnce(firstCallPromise)
+                .mockReturnValueOnce(secondCallPromise);
+            // 3. Start the first message call. Do not await it yet.
+            const firstMessagePromise = chat.sendMessage({ message: 'first' }, 'prompt-1');
+            // Give the event loop a chance to run the async call up to the `await`
+            await new Promise(process.nextTick);
+            // 4. While the first call is "in-flight", start the second message call.
+            const secondMessagePromise = chat.sendMessage({ message: 'second' }, 'prompt-2');
+            // 5. CRUCIAL CHECK: At this point, only the first API call should have been made.
+            // The second call should be waiting on `sendPromise`.
+            expect(mockModelsModule.generateContent).toHaveBeenCalledTimes(1);
+            expect(mockModelsModule.generateContent).toHaveBeenCalledWith(expect.objectContaining({
+                contents: expect.arrayContaining([
+                    expect.objectContaining({ parts: [{ text: 'first' }] }),
+                ]),
+            }), 'prompt-1');
+            // 6. Unblock the first API call and wait for the first message to fully complete.
+            firstCallResolver(mockResponse);
+            await firstMessagePromise;
+            // Give the event loop a chance to unblock and run the second call.
+            await new Promise(process.nextTick);
+            // 7. CRUCIAL CHECK: Now, the second API call should have been made.
+            expect(mockModelsModule.generateContent).toHaveBeenCalledTimes(2);
+            expect(mockModelsModule.generateContent).toHaveBeenCalledWith(expect.objectContaining({
+                contents: expect.arrayContaining([
+                    expect.objectContaining({ parts: [{ text: 'second' }] }),
+                ]),
+            }), 'prompt-2');
+            // 8. Clean up by resolving the second call.
+            secondCallResolver(mockResponse);
+            await secondMessagePromise;
+        });
+    });
+    it('should retry if the model returns a completely empty stream (no chunks)', async () => {
+        // 1. Mock the API to return an empty stream first, then a valid one.
+        vi.mocked(mockModelsModule.generateContentStream)
+            .mockImplementationOnce(
+        // First call resolves to an async generator that yields nothing.
+        async () => (async function* () { })())
+            .mockImplementationOnce(
+        // Second call returns a valid stream.
+        async () => (async function* () {
+            yield {
+                candidates: [
+                    {
+                        content: {
+                            parts: [{ text: 'Successful response after empty' }],
+                        },
+                    },
+                ],
+            };
+        })());
+        // 2. Call the method and consume the stream.
+        const stream = await chat.sendMessageStream({ message: 'test empty stream' }, 'prompt-id-empty-stream');
+        const chunks = [];
+        for await (const chunk of stream) {
+            chunks.push(chunk);
+        }
+        // 3. Assert the results.
+        expect(mockModelsModule.generateContentStream).toHaveBeenCalledTimes(2);
+        expect(chunks.some((c) => c.candidates?.[0]?.content?.parts?.[0]?.text ===
+            'Successful response after empty')).toBe(true);
+        const history = chat.getHistory();
+        expect(history.length).toBe(2);
+        // Explicitly verify the structure of each part to satisfy TypeScript
+        const turn1 = history[0];
+        if (!turn1?.parts?.[0] || !('text' in turn1.parts[0])) {
+            throw new Error('Test setup error: First turn is not a valid text part.');
+        }
+        expect(turn1.parts[0].text).toBe('test empty stream');
+        const turn2 = history[1];
+        if (!turn2?.parts?.[0] || !('text' in turn2.parts[0])) {
+            throw new Error('Test setup error: Second turn is not a valid text part.');
+        }
+        expect(turn2.parts[0].text).toBe('Successful response after empty');
+    });
+    it('should queue a subsequent sendMessageStream call until the first stream is fully consumed', async () => {
+        // 1. Create a promise to manually control the stream's lifecycle
+        let continueFirstStream;
+        const firstStreamContinuePromise = new Promise((resolve) => {
+            continueFirstStream = resolve;
+        });
+        // 2. Mock the API to return controllable async generators
+        const firstStreamGenerator = (async function* () {
+            yield {
+                candidates: [
+                    { content: { parts: [{ text: 'first response part 1' }] } },
+                ],
+            };
+            await firstStreamContinuePromise; // Pause the stream
+            yield {
+                candidates: [{ content: { parts: [{ text: ' part 2' }] } }],
+            };
+        })();
+        const secondStreamGenerator = (async function* () {
+            yield {
+                candidates: [{ content: { parts: [{ text: 'second response' }] } }],
+            };
+        })();
+        vi.mocked(mockModelsModule.generateContentStream)
+            .mockResolvedValueOnce(firstStreamGenerator)
+            .mockResolvedValueOnce(secondStreamGenerator);
+        // 3. Start the first stream and consume only the first chunk to pause it
+        const firstStream = await chat.sendMessageStream({ message: 'first' }, 'prompt-1');
+        const firstStreamIterator = firstStream[Symbol.asyncIterator]();
+        await firstStreamIterator.next();
+        // 4. While the first stream is paused, start the second call. It will block.
+        const secondStreamPromise = chat.sendMessageStream({ message: 'second' }, 'prompt-2');
+        // 5. Assert that only one API call has been made so far.
+        expect(mockModelsModule.generateContentStream).toHaveBeenCalledTimes(1);
+        // 6. Unblock and fully consume the first stream to completion.
+        continueFirstStream();
+        await firstStreamIterator.next(); // Consume the rest of the stream
+        await firstStreamIterator.next(); // Finish the iterator
+        // 7. Now that the first stream is done, await the second promise to get its generator.
+        const secondStream = await secondStreamPromise;
+        // 8. Start consuming the second stream, which triggers its internal API call.
+        const secondStreamIterator = secondStream[Symbol.asyncIterator]();
+        await secondStreamIterator.next();
+        // 9. The second API call should now have been made.
+        expect(mockModelsModule.generateContentStream).toHaveBeenCalledTimes(2);
+        // 10. FIX: Fully consume the second stream to ensure recordHistory is called.
+        await secondStreamIterator.next(); // This finishes the iterator.
+        // 11. Final check on history.
+        const history = chat.getHistory();
+        expect(history.length).toBe(4);
+        const turn4 = history[3];
+        if (!turn4?.parts?.[0] || !('text' in turn4.parts[0])) {
+            throw new Error('Test setup error: Fourth turn is not a valid text part.');
+        }
+        expect(turn4.parts[0].text).toBe('second response');
+    });
 });
 //# sourceMappingURL=geminiChat.test.js.map