Revert "Disable Routing by default (#11549)" (#11594)

This commit is contained in:
Abhi
2025-10-21 14:36:28 -04:00
committed by GitHub
parent 2c93542e5c
commit a74a04d12d
4 changed files with 6 additions and 6 deletions
+3 -3
View File
@@ -1327,7 +1327,7 @@ describe('loadCliConfig model selection', () => {
argv, argv,
); );
expect(config.getModel()).toBe(DEFAULT_GEMINI_MODEL); expect(config.getModel()).toBe('auto');
}); });
it('always prefers model from argv', async () => { it('always prefers model from argv', async () => {
@@ -1640,12 +1640,12 @@ describe('loadCliConfig useRipgrep', () => {
}); });
describe('loadCliConfig useModelRouter', () => { 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']; process.argv = ['node', 'script.js'];
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
@@ -542,7 +542,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);
}); });
}); });
}); });
+1 -1
View File
@@ -1071,7 +1071,7 @@ 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: true, showInDialog: true,