fix(cli): lazily load settings in onModelChange to prevent stale closure data loss (#20403)

Co-authored-by: Spencer <spencertang@google.com>
This commit is contained in:
adithya32
2026-03-18 06:10:38 +05:30
committed by GitHub
parent e2658ccda8
commit bd34a42ec3

View File

@@ -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 {