From a74a04d12d19cf6c5c34d85f9ef98fe14e4021cb Mon Sep 17 00:00:00 2001 From: Abhi <43648792+abhipatel12@users.noreply.github.com> Date: Tue, 21 Oct 2025 14:36:28 -0400 Subject: [PATCH] Revert "Disable Routing by default (#11549)" (#11594) --- packages/cli/src/config/config.test.ts | 6 +++--- packages/cli/src/config/config.ts | 2 +- packages/cli/src/config/settingsSchema.test.ts | 2 +- packages/cli/src/config/settingsSchema.ts | 2 +- 4 files changed, 6 insertions(+), 6 deletions(-) diff --git a/packages/cli/src/config/config.test.ts b/packages/cli/src/config/config.test.ts index 6bc23ad7d6..95f3e6c778 100644 --- a/packages/cli/src/config/config.test.ts +++ b/packages/cli/src/config/config.test.ts @@ -1327,7 +1327,7 @@ describe('loadCliConfig model selection', () => { argv, ); - expect(config.getModel()).toBe(DEFAULT_GEMINI_MODEL); + expect(config.getModel()).toBe('auto'); }); it('always prefers model from argv', async () => { @@ -1640,12 +1640,12 @@ describe('loadCliConfig useRipgrep', () => { }); describe('loadCliConfig useModelRouter', () => { - it('should be false by default when useModelRouter is not set in settings', async () => { + it('should be true by default when useModelRouter is not set in settings', async () => { process.argv = ['node', 'script.js']; const argv = await parseArguments({} as Settings); const settings: Settings = {}; const config = await loadCliConfig(settings, [], 'test-session', argv); - expect(config.getUseModelRouter()).toBe(false); + expect(config.getUseModelRouter()).toBe(true); }); it('should be true when useModelRouter is set to true in settings', async () => { diff --git a/packages/cli/src/config/config.ts b/packages/cli/src/config/config.ts index 9dd1e32386..1bb1a3b70d 100755 --- a/packages/cli/src/config/config.ts +++ b/packages/cli/src/config/config.ts @@ -542,7 +542,7 @@ export async function loadCliConfig( ); } - const useModelRouter = settings.experimental?.useModelRouter ?? false; + const useModelRouter = settings.experimental?.useModelRouter ?? true; const defaultModel = useModelRouter ? DEFAULT_GEMINI_MODEL_AUTO : DEFAULT_GEMINI_MODEL; diff --git a/packages/cli/src/config/settingsSchema.test.ts b/packages/cli/src/config/settingsSchema.test.ts index 4088151049..67757637ec 100644 --- a/packages/cli/src/config/settingsSchema.test.ts +++ b/packages/cli/src/config/settingsSchema.test.ts @@ -328,7 +328,7 @@ describe('SettingsSchema', () => { ).toBe('Experimental'); expect( getSettingsSchema().experimental.properties.useModelRouter.default, - ).toBe(false); + ).toBe(true); }); }); }); diff --git a/packages/cli/src/config/settingsSchema.ts b/packages/cli/src/config/settingsSchema.ts index 3ad7e6c2dc..6716825812 100644 --- a/packages/cli/src/config/settingsSchema.ts +++ b/packages/cli/src/config/settingsSchema.ts @@ -1071,7 +1071,7 @@ const SETTINGS_SCHEMA = { label: 'Use Model Router', category: 'Experimental', requiresRestart: true, - default: false, + default: true, description: 'Enable model routing to route requests to the best model based on complexity.', showInDialog: true,