From 4dd777785e6ca044344fe9b8138ef530f3aec5a9 Mon Sep 17 00:00:00 2001 From: Neil <4138956+nwparker@users.noreply.github.com> Date: Fri, 29 May 2026 15:08:19 -0700 Subject: [PATCH] Fetch inactive account usage on expand --- .../src/components/status-bar/StatusBar.tsx | 18 ++++++++++++------ 1 file changed, 12 insertions(+), 6 deletions(-) diff --git a/src/renderer/src/components/status-bar/StatusBar.tsx b/src/renderer/src/components/status-bar/StatusBar.tsx index 1346eef8dfd..b9a236458ca 100644 --- a/src/renderer/src/components/status-bar/StatusBar.tsx +++ b/src/renderer/src/components/status-bar/StatusBar.tsx @@ -157,11 +157,17 @@ function ClaudeSwitcherMenu({ } }, []) - useEffect(() => { - if (accountsExpanded) { - void fetchInactiveClaudeAccountUsage() - } - }, [accountsExpanded, fetchInactiveClaudeAccountUsage]) + // Why: inactive-account usage is needed only for the explicit switcher + // expansion, so fetch it on that event instead of one render later. + const handleAccountsExpandedToggle = useCallback((): void => { + setAccountsExpanded((current) => { + const nextExpanded = !current + if (nextExpanded) { + void fetchInactiveClaudeAccountUsage() + } + return nextExpanded + }) + }, [fetchInactiveClaudeAccountUsage]) const handleSelectAccount = async (accountId: string | null): Promise => { if (isSwitching) { @@ -208,7 +214,7 @@ function ClaudeSwitcherMenu({ { event.preventDefault() - setAccountsExpanded((prev) => !prev) + handleAccountsExpandedToggle() }} >