From bd34a42ec3520f1964c7b9a5a0fd3418c57e7462 Mon Sep 17 00:00:00 2001 From: adithya32 <163162210+KumarADITHYA123@users.noreply.github.com> Date: Wed, 18 Mar 2026 06:10:38 +0530 Subject: [PATCH] fix(cli): lazily load settings in onModelChange to prevent stale closure data loss (#20403) Co-authored-by: Spencer --- packages/cli/src/config/config.ts | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/packages/cli/src/config/config.ts b/packages/cli/src/config/config.ts index 010e6d8d99..80c1e19443 100755 --- a/packages/cli/src/config/config.ts +++ b/packages/cli/src/config/config.ts @@ -430,8 +430,6 @@ export async function loadCliConfig( const { cwd = process.cwd(), projectHooks } = options; const debugMode = isDebugMode(argv); - const loadedSettings = loadSettings(cwd); - if (argv.sandbox) { process.env['GEMINI_SANDBOX'] = 'true'; } @@ -886,7 +884,7 @@ export async function loadCliConfig( hooks: settings.hooks || {}, disabledHooks: settings.hooksConfig?.disabled || [], projectHooks: projectHooks || {}, - onModelChange: (model: string) => saveModelChange(loadedSettings, model), + onModelChange: (model: string) => saveModelChange(loadSettings(cwd), model), onReload: async () => { const refreshedSettings = loadSettings(cwd); return {