MediaMetz/gemini-cli
1
0
Fork 0
mirror of https://github.com/google-gemini/gemini-cli.git synced 2026-03-18 01:51:20 -07:00
Code Issues Packages Projects Releases Wiki Activity

fix(ui): reuse UserIdentity component in AppHeader per TL feedback

Browse Source
This commit is contained in:
Keith Guerin
2026-02-10 11:14:36 -08:00
parent f307ff00ac
commit 39754c5093
1 changed files with 5 additions and 33 deletions
Download Patch File Download Diff File Expand all files Collapse all files

38
packages/cli/src/ui/components/AppHeader.tsx
View File

@@ -5,8 +5,7 @@
*/
import { Box, Text } from 'ink';
import { useEffect, useMemo, useState } from 'react';
import { UserAccountManager, AuthType } from '@google/gemini-cli-core';
import { UserIdentity } from './UserIdentity.js';
import { Tips } from './Tips.js';
import { useSettings } from '../contexts/SettingsContext.js';
import { useConfig } from '../contexts/ConfigContext.js';
@@ -36,20 +35,6 @@ export const AppHeader = ({ version, showDetails = true }: AppHeaderProps) => {
const { bannerText } = useBanner(bannerData);
const { showTips } = useTips();
const authType = config.getContentGeneratorConfig()?.authType;
const [email, setEmail] = useState<string | undefined>();
useEffect(() => {
if (authType) {
const userAccountManager = new UserAccountManager();
// Even though the current implementation of getCachedGoogleAccount is sync,
// it performs file I/O. Moving it to useEffect ensures it doesn't block the render cycle.
setEmail(userAccountManager.getCachedGoogleAccount() ?? undefined);
}
}, [authType]);
const tierName = useMemo(() => config.getUserTierName(), [config]);
const showHeader = !(
settings.merged.ui.hideBanner || config.getScreenReader()
);
@@ -107,24 +92,11 @@ export const AppHeader = ({ version, showDetails = true }: AppHeaderProps) => {
{/* Line 2: Blank */}
<Box height={1} />
{/* Line 3: User Email /auth */}
{/* Lines 3 & 4: User Identity info (Email /auth and Plan /upgrade) */}
<Box>
<Text color={theme.text.primary}>
{authType === AuthType.LOGIN_WITH_GOOGLE ? (
<Text>{email ?? 'Logged in with Google'}</Text>
) : (
`Authenticated with ${authType}`
)}
</Text>
<Text color={theme.text.secondary}> /auth</Text>
</Box>
{/* Line 4: Tier Name /upgrade */}
<Box>
<Text color={theme.text.primary}>
{tierName ?? 'Gemini Code Assist for individuals'}
</Text>
<Text color={theme.text.secondary}> /upgrade</Text>
{settings.merged.ui.showUserIdentity !== false && (
<UserIdentity config={config} />
)}
</Box>
</Box>
</Box>