From d7a57d85a39535e84bba7e65eb02dcb627b9cb81 Mon Sep 17 00:00:00 2001 From: Tommaso Sciortino Date: Mon, 21 Jul 2025 13:44:43 -0700 Subject: Simplify streaming code for code assist server (#4619) --- packages/cli/src/ui/App.test.tsx | 5 ++++- packages/cli/src/ui/App.tsx | 19 ++----------------- 2 files changed, 6 insertions(+), 18 deletions(-) (limited to 'packages/cli/src') diff --git a/packages/cli/src/ui/App.test.tsx b/packages/cli/src/ui/App.test.tsx index 4c98827e..ceab46b1 100644 --- a/packages/cli/src/ui/App.test.tsx +++ b/packages/cli/src/ui/App.test.tsx @@ -138,7 +138,9 @@ vi.mock('@google/gemini-cli-core', async (importOriginal) => { getShowMemoryUsage: vi.fn(() => opts.showMemoryUsage ?? false), getAccessibility: vi.fn(() => opts.accessibility ?? {}), getProjectRoot: vi.fn(() => opts.targetDir), - getGeminiClient: vi.fn(() => ({})), + getGeminiClient: vi.fn(() => ({ + getUserTier: vi.fn(), + })), getCheckpointingEnabled: vi.fn(() => opts.checkpointing ?? true), getAllGeminiMdFilenames: vi.fn(() => ['GEMINI.md']), setFlashFallbackHandler: vi.fn(), @@ -639,6 +641,7 @@ describe('App UI', () => { mockConfig.getGeminiClient.mockReturnValue({ isInitialized: vi.fn(() => true), + getUserTier: vi.fn(), } as unknown as GeminiClient); const { unmount, rerender } = render( diff --git a/packages/cli/src/ui/App.tsx b/packages/cli/src/ui/App.tsx index 027665f1..c6e6bd43 100644 --- a/packages/cli/src/ui/App.tsx +++ b/packages/cli/src/ui/App.tsx @@ -206,26 +206,11 @@ const App = ({ config, settings, startupWarnings = [], version }: AppProps) => { // Sync user tier from config when authentication changes useEffect(() => { - const syncUserTier = async () => { - try { - const configUserTier = await config.getUserTier(); - if (configUserTier !== userTier) { - setUserTier(configUserTier); - } - } catch (error) { - // Silently fail - this is not critical functionality - // Only log in debug mode to avoid cluttering the console - if (config.getDebugMode()) { - console.debug('Failed to sync user tier:', error); - } - } - }; - // Only sync when not currently authenticating if (!isAuthenticating) { - syncUserTier(); + setUserTier(config.getGeminiClient()?.getUserTier()); } - }, [config, userTier, isAuthenticating]); + }, [config, isAuthenticating]); const { isEditorDialogOpen, -- cgit v1.2.3