packages/cli/src/ui/hooks/useGeminiStream.test.tsx (1)

2070-2091: ⚠️ Potential issue | 🔴 Critical

Missing drainQueuedMessages argument will cause test failure.

This useGeminiStream call has only 18 arguments but the hook requires 19 parameters. The drainQueuedMessages: () => string[] argument is missing, unlike all other call sites that were updated in this PR.

🐛 Proposed fix to add the missing argument

     const { result } = renderHook(() =>
       useGeminiStream(
         mockConfig.getGeminiClient() as GeminiClient,
         [],
         mockAddItem,
         mockConfig,
         mockLoadedSettings,
         mockOnDebugMessage,
         mockHandleSlashCommand,
         false, // shellModeActive
         vi.fn(), // getPreferredEditor
         vi.fn(), // onAuthError
         vi.fn(), // performMemoryRefresh
         false, // modelSwitched
         vi.fn(), // setModelSwitched
         vi.fn(), // onEditorClose
         vi.fn(), // onCancelSubmit
         vi.fn(), // setShellInputFocused
         80, // terminalWidth
         24, // terminalHeight
+        () => [], // drainQueuedMessages
       ),
     );

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@packages/cli/src/ui/hooks/useGeminiStream.test.tsx` around lines 2070 - 2091,
The test call to useGeminiStream is missing the drainQueuedMessages parameter
(expects a () => string[]), causing the arity mismatch; update the renderHook
invocation to pass a drainQueuedMessages function (e.g., a mock like vi.fn(() =>
[]) or a simple () => []) as the missing argument so the call supplies all 19
parameters to useGeminiStream and aligns with other updated call sites.