Skip to content

Commit 1b47376

Browse files
committed
Revert "Disable Routing by default (#11549)"
This reverts commit 8aace3a.
1 parent 14867c7 commit 1b47376

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
@@ -1328,7 +1328,7 @@ describe('loadCliConfig model selection', () => {
13281328
argv,
13291329
);
13301330

1331-
expect(config.getModel()).toBe(DEFAULT_GEMINI_MODEL);
1331+
expect(config.getModel()).toBe('auto');
13321332
});
13331333

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

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

16521652
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)