Skip to content

Commit b03d12c

Browse files
authored
Revert "Disable Routing by default (#11549)" (#11594)
1 parent 86c2f43 commit b03d12c

4 files changed

Lines changed: 6 additions & 6 deletions

File tree

packages/cli/src/config/config.test.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1327,7 +1327,7 @@ describe('loadCliConfig model selection', () => {
13271327
argv,
13281328
);
13291329

1330-
expect(config.getModel()).toBe(DEFAULT_GEMINI_MODEL);
1330+
expect(config.getModel()).toBe('auto');
13311331
});
13321332

13331333
it('always prefers model from argv', async () => {
@@ -1640,12 +1640,12 @@ describe('loadCliConfig useRipgrep', () => {
16401640
});
16411641

16421642
describe('loadCliConfig useModelRouter', () => {
1643-
it('should be false by default when useModelRouter is not set in settings', async () => {
1643+
it('should be true by default when useModelRouter is not set in settings', async () => {
16441644
process.argv = ['node', 'script.js'];
16451645
const argv = await parseArguments({} as Settings);
16461646
const settings: Settings = {};
16471647
const config = await loadCliConfig(settings, [], 'test-session', argv);
1648-
expect(config.getUseModelRouter()).toBe(false);
1648+
expect(config.getUseModelRouter()).toBe(true);
16491649
});
16501650

16511651
it('should be true when useModelRouter is set to true in settings', async () => {

packages/cli/src/config/config.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -542,7 +542,7 @@ export async function loadCliConfig(
542542
);
543543
}
544544

545-
const useModelRouter = settings.experimental?.useModelRouter ?? false;
545+
const useModelRouter = settings.experimental?.useModelRouter ?? true;
546546
const defaultModel = useModelRouter
547547
? DEFAULT_GEMINI_MODEL_AUTO
548548
: DEFAULT_GEMINI_MODEL;

packages/cli/src/config/settingsSchema.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -328,7 +328,7 @@ describe('SettingsSchema', () => {
328328
).toBe('Experimental');
329329
expect(
330330
getSettingsSchema().experimental.properties.useModelRouter.default,
331-
).toBe(false);
331+
).toBe(true);
332332
});
333333
});
334334
});

packages/cli/src/config/settingsSchema.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1071,7 +1071,7 @@ const SETTINGS_SCHEMA = {
10711071
label: 'Use Model Router',
10721072
category: 'Experimental',
10731073
requiresRestart: true,
1074-
default: false,
1074+
default: true,
10751075
description:
10761076
'Enable model routing to route requests to the best model based on complexity.',
10771077
showInDialog: true,

0 commit comments

Comments
 (0)