MediaMetz/gemini-cli
1
0
Fork 0
mirror of https://github.com/google-gemini/gemini-cli.git synced 2026-07-05 23:56:45 -07:00
Code Issues Packages Projects Releases Wiki Activity

feat(cli): disable tips by default

Browse Source
This commit is contained in:
Keith Guerin
2026-03-30 15:56:05 -07:00
parent 44cdb3e376
commit bf48644640
6 changed files with 51 additions and 21 deletions
Download Patch File Download Diff File Expand all files Collapse all files
packages/cli/src/config/settingsSchema.ts
+1 -1
View File
@@ -538,7 +538,7 @@ const SETTINGS_SCHEMA = {
label: 'Hide Tips',
category: 'UI',
requiresRestart: false,
default: false,
default: true,
description: 'Hide helpful tips in the UI',
showInDialog: true,
},
packages/cli/src/ui/App.test.tsx
+33 -11
View File
@@ -94,7 +94,9 @@ describe('App', () => {
it('should render main content and composer when not quitting', async () => {
const { lastFrame, unmount } = await renderWithProviders(<App />, {
uiState: mockUIState,
settings: createMockSettings({ ui: { useAlternateBuffer: false } }),
settings: createMockSettings({
ui: { useAlternateBuffer: false, hideTips: false },
}),
});
expect(lastFrame()).toContain('Tips for getting started');
@@ -111,7 +113,9 @@ describe('App', () => {
const { lastFrame, unmount } = await renderWithProviders(<App />, {
uiState: quittingUIState,
settings: createMockSettings({ ui: { useAlternateBuffer: false } }),
settings: createMockSettings({
ui: { useAlternateBuffer: false, hideTips: false },
}),
});
expect(lastFrame()).toContain('Quitting...');
@@ -128,7 +132,9 @@ describe('App', () => {
const { lastFrame, unmount } = await renderWithProviders(<App />, {
uiState: quittingUIState,
settings: createMockSettings({ ui: { useAlternateBuffer: true } }),
settings: createMockSettings({
ui: { useAlternateBuffer: true, hideTips: false },
}),
});
expect(lastFrame()).toContain('HistoryItemDisplay');
@@ -144,7 +150,9 @@ describe('App', () => {
const { lastFrame, unmount } = await renderWithProviders(<App />, {
uiState: dialogUIState,
settings: createMockSettings({ ui: { useAlternateBuffer: true } }),
settings: createMockSettings({
ui: { useAlternateBuffer: true, hideTips: false },
}),
});
expect(lastFrame()).toContain('Tips for getting started');
@@ -167,7 +175,9 @@ describe('App', () => {
const { lastFrame, unmount } = await renderWithProviders(<App />, {
uiState,
settings: createMockSettings({ ui: { useAlternateBuffer: true } }),
settings: createMockSettings({
ui: { useAlternateBuffer: true, hideTips: false },
}),
});
expect(lastFrame()).toContain(`Press Ctrl+${key} again to exit.`);
@@ -180,7 +190,9 @@ describe('App', () => {
const { lastFrame, unmount } = await renderWithProviders(<App />, {
uiState: mockUIState,
settings: createMockSettings({ ui: { useAlternateBuffer: true } }),
settings: createMockSettings({
ui: { useAlternateBuffer: true, hideTips: false },
}),
});
expect(lastFrame()).toContain('Notifications');
@@ -195,7 +207,9 @@ describe('App', () => {
const { lastFrame, unmount } = await renderWithProviders(<App />, {
uiState: mockUIState,
settings: createMockSettings({ ui: { useAlternateBuffer: true } }),
settings: createMockSettings({
ui: { useAlternateBuffer: true, hideTips: false },
}),
});
expect(lastFrame()).toContain('Tips for getting started');
@@ -247,7 +261,9 @@ describe('App', () => {
const { lastFrame, unmount } = await renderWithProviders(<App />, {
uiState: stateWithConfirmingTool,
config: configWithExperiment,
settings: createMockSettings({ ui: { useAlternateBuffer: true } }),
settings: createMockSettings({
ui: { useAlternateBuffer: true, hideTips: false },
}),
});
expect(lastFrame()).toContain('Tips for getting started');
@@ -263,7 +279,9 @@ describe('App', () => {
(useIsScreenReaderEnabled as Mock).mockReturnValue(false);
const { lastFrame, unmount } = await renderWithProviders(<App />, {
uiState: mockUIState,
settings: createMockSettings({ ui: { useAlternateBuffer: true } }),
settings: createMockSettings({
ui: { useAlternateBuffer: true, hideTips: false },
}),
});
expect(lastFrame()).toMatchSnapshot();
unmount();
@@ -273,7 +291,9 @@ describe('App', () => {
(useIsScreenReaderEnabled as Mock).mockReturnValue(true);
const { lastFrame, unmount } = await renderWithProviders(<App />, {
uiState: mockUIState,
settings: createMockSettings({ ui: { useAlternateBuffer: true } }),
settings: createMockSettings({
ui: { useAlternateBuffer: true, hideTips: false },
}),
});
expect(lastFrame()).toMatchSnapshot();
unmount();
@@ -286,7 +306,9 @@ describe('App', () => {
} as UIState;
const { lastFrame, unmount } = await renderWithProviders(<App />, {
uiState: dialogUIState,
settings: createMockSettings({ ui: { useAlternateBuffer: true } }),
settings: createMockSettings({
ui: { useAlternateBuffer: true, hideTips: false },
}),
});
expect(lastFrame()).toMatchSnapshot();
unmount();
packages/cli/src/ui/components/AppHeader.test.tsx
+12 -4
View File
@@ -8,6 +8,7 @@ import {
renderWithProviders,
persistentStateMock,
} from '../../test-utils/render.js';
import { createMockSettings } from '../../test-utils/settings.js';
import type { LoadedSettings } from '../../config/settings.js';
import { AppHeader } from './AppHeader.js';
import { describe, it, expect, vi } from 'vitest';
@@ -33,6 +34,7 @@ describe('<AppHeader />', () => {
<AppHeader version="1.0.0" />,
{
uiState,
settings: createMockSettings({ ui: { hideTips: false } }),
},
);
@@ -55,6 +57,7 @@ describe('<AppHeader />', () => {
<AppHeader version="1.0.0" />,
{
uiState,
settings: createMockSettings({ ui: { hideTips: false } }),
},
);
@@ -76,6 +79,7 @@ describe('<AppHeader />', () => {
<AppHeader version="1.0.0" />,
{
uiState,
settings: createMockSettings({ ui: { hideTips: false } }),
},
);
@@ -106,6 +110,7 @@ describe('<AppHeader />', () => {
<AppHeader version="1.0.0" />,
{
uiState,
settings: createMockSettings({ ui: { hideTips: false } }),
},
);
@@ -160,6 +165,7 @@ describe('<AppHeader />', () => {
<AppHeader version="1.0.0" />,
{
uiState,
settings: createMockSettings({ ui: { hideTips: false } }),
},
);
@@ -183,6 +189,7 @@ describe('<AppHeader />', () => {
<AppHeader version="1.0.0" />,
{
uiState,
settings: createMockSettings({ ui: { hideTips: false } }),
},
);
@@ -205,6 +212,7 @@ describe('<AppHeader />', () => {
<AppHeader version="1.0.0" />,
{
uiState,
settings: createMockSettings({ ui: { hideTips: false } }),
},
);
@@ -227,6 +235,7 @@ describe('<AppHeader />', () => {
// First session
const session1 = await renderWithProviders(<AppHeader version="1.0.0" />, {
uiState,
settings: createMockSettings({ ui: { hideTips: false } }),
});
expect(session1.lastFrame()).toContain('Tips');
@@ -234,10 +243,9 @@ describe('<AppHeader />', () => {
session1.unmount();
// Second session - state is persisted in the fake
const session2 = await renderWithProviders(
<AppHeader version="1.0.0" />,
{},
);
const session2 = await renderWithProviders(<AppHeader version="1.0.0" />, {
settings: createMockSettings({ ui: { hideTips: false } }),
});
expect(session2.lastFrame()).not.toContain('Tips');
session2.unmount();