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

fix(cli): allow early stdout when config is undefined (#26453)

Browse Source
This commit is contained in:
Coco Sheng
2026-05-04 13:48:24 -04:00
committed by GitHub
parent 165efa8a38
commit 37edd1d4df
2 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
packages/cli/src/gemini.tsx
+1 -1
View File
@@ -830,7 +830,7 @@ export function initializeOutputListenersAndFlush(config?: Config) {
} }
const outputFormat = config?.getOutputFormat(); const outputFormat = config?.getOutputFormat();
const forceToStderr = outputFormat === 'json' || config === undefined; const forceToStderr = outputFormat === 'json';
coreEvents.drainBacklogs( coreEvents.drainBacklogs(
<K extends keyof CoreEvents>(event: K, args: CoreEvents[K]) => { <K extends keyof CoreEvents>(event: K, args: CoreEvents[K]) => {
packages/cli/src/output-redirection.test.ts
+4 -4
View File
@@ -69,16 +69,16 @@ describe('Output Redirection', () => {
expect(writeToStderr).not.toHaveBeenCalled(); expect(writeToStderr).not.toHaveBeenCalled();
}); });
it('should force stdout to stderr when config is undefined (early failure)', () => { it('should NOT force stdout to stderr when config is undefined (early init/version)', () => {
// Simulate buffered output during early init // Simulate buffered output during early init
coreEvents.emitOutput(false, 'early init message'); coreEvents.emitOutput(false, 'early init message');
// Initialize with undefined config // Initialize with undefined config
initializeOutputListenersAndFlush(undefined); initializeOutputListenersAndFlush(undefined);
// Verify it was forced to stderr // Verify it went to stdout (default behavior)
expect(writeToStderr).toHaveBeenCalledWith('early init message', undefined); expect(writeToStdout).toHaveBeenCalledWith('early init message', undefined);
expect(writeToStdout).not.toHaveBeenCalled(); expect(writeToStderr).not.toHaveBeenCalled();
}); });
it('should attach ConsoleLog and UserFeedback listeners even if Output already has one', () => { it('should attach ConsoleLog and UserFeedback listeners even if Output already has one', () => {