Fix to issue #736 resulting in GEMINI_CLI_AUTH_TYPE not getting populated most of the time in the start_session events (#8508)

Co-authored-by: owenofbrien <86964623+owenofbrien@users.noreply.github.com>
Co-authored-by: Arya Gummadi <aryagummadi@google.com>
This commit is contained in:
MRK
2025-09-16 23:08:23 -07:00
committed by GitHub
parent d2f87d15ed
commit 4df8dbaa0d
2 changed files with 4 additions and 2 deletions
+1 -1
View File
@@ -422,7 +422,7 @@ describe('Server Config (config.ts)', () => {
...baseParams, ...baseParams,
usageStatisticsEnabled: true, usageStatisticsEnabled: true,
}); });
await config.initialize(); await config.refreshAuth(AuthType.USE_GEMINI);
expect( expect(
ClearcutLogger.prototype.logStartSessionEvent, ClearcutLogger.prototype.logStartSessionEvent,
+3 -1
View File
@@ -459,7 +459,6 @@ export class Config {
} }
this.promptRegistry = new PromptRegistry(); this.promptRegistry = new PromptRegistry();
this.toolRegistry = await this.createToolRegistry(); this.toolRegistry = await this.createToolRegistry();
logCliConfiguration(this, new StartSessionEvent(this, this.toolRegistry));
await this.geminiClient.initialize(); await this.geminiClient.initialize();
} }
@@ -496,6 +495,9 @@ export class Config {
// Reset the session flag since we're explicitly changing auth and using default model // Reset the session flag since we're explicitly changing auth and using default model
this.inFallbackMode = false; this.inFallbackMode = false;
// Logging the cli configuration here as the auth related configuration params would have been loaded by this point
logCliConfiguration(this, new StartSessionEvent(this, this.toolRegistry));
} }
getUserTier(): UserTierId | undefined { getUserTier(): UserTierId | undefined {