feat(cli): Enable model router by default and add to settings dialog (#9262)

(cherry picked from commit ee36307495)
This commit is contained in:
Abhi
2025-09-24 17:33:14 -04:00
committed by Abhi
parent 89aba7cbcd
commit fd2bc71e39
5 changed files with 7 additions and 7 deletions
+2 -2
View File
@@ -1485,7 +1485,7 @@ describe('loadCliConfig model selection', () => {
argv, argv,
); );
expect(config.getModel()).toBe('gemini-2.5-pro'); expect(config.getModel()).toBe(DEFAULT_GEMINI_MODEL_AUTO);
}); });
it('always prefers model from argvs', async () => { it('always prefers model from argvs', async () => {
@@ -1803,7 +1803,7 @@ describe('loadCliConfig useRipgrep', () => {
const argv = await parseArguments({} as Settings); const argv = await parseArguments({} as Settings);
const settings: Settings = {}; const settings: Settings = {};
const config = await loadCliConfig(settings, [], 'test-session', argv); 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 () => { it('should be true when useModelRouter is set to true in settings', async () => {
+1 -1
View File
@@ -587,7 +587,7 @@ export async function loadCliConfig(
); );
} }
const useModelRouter = settings.experimental?.useModelRouter ?? false; const useModelRouter = settings.experimental?.useModelRouter ?? true;
const defaultModel = useModelRouter const defaultModel = useModelRouter
? DEFAULT_GEMINI_MODEL_AUTO ? DEFAULT_GEMINI_MODEL_AUTO
: DEFAULT_GEMINI_MODEL; : DEFAULT_GEMINI_MODEL;
@@ -328,7 +328,7 @@ describe('SettingsSchema', () => {
).toBe('Experimental'); ).toBe('Experimental');
expect( expect(
getSettingsSchema().experimental.properties.useModelRouter.default, getSettingsSchema().experimental.properties.useModelRouter.default,
).toBe(false); ).toBe(true);
}); });
}); });
}); });
+2 -2
View File
@@ -986,10 +986,10 @@ const SETTINGS_SCHEMA = {
label: 'Use Model Router', label: 'Use Model Router',
category: 'Experimental', category: 'Experimental',
requiresRestart: true, requiresRestart: true,
default: false, default: true,
description: description:
'Enable model routing to route requests to the best model based on complexity.', 'Enable model routing to route requests to the best model based on complexity.',
showInDialog: false, showInDialog: true,
}, },
}, },
}, },
@@ -342,7 +342,7 @@ describe('SettingsDialog', () => {
await wait(); await wait();
expect(lastFrame()).toContain('● Folder Trust'); expect(lastFrame()).toContain('● Use Model Router');
unmount(); unmount();
}); });