Skip to content

Commit 93227c9

Browse files
authored
Merge pull request #1121 from guillaumegay13/codex/fix-openrouter-available-models
fix(backend): preserve compatible OpenRouter models
2 parents 544a8cd + 9acb4f3 commit 93227c9

File tree

6 files changed

+329
-57
lines changed

6 files changed

+329
-57
lines changed

.changeset/wise-pans-smash.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
---
2+
"manifest": patch
3+
---
4+
5+
Preserve compatible OpenRouter text models during pricing sync while filtering out non-chat OpenRouter models from the local model list.

packages/backend/src/database/local-bootstrap.service.spec.ts

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -365,6 +365,17 @@ describe('LocalBootstrapService', () => {
365365
expect(mockPricingRepo.delete).not.toHaveBeenCalled();
366366
});
367367

368+
it('preserves OpenRouter provider models even if not curated', async () => {
369+
mockPricingRepo.find.mockResolvedValue([
370+
{ model_name: 'qwen/qwen3-235b-a22b', provider: 'OpenRouter' },
371+
]);
372+
373+
await service.onModuleInit();
374+
await new Promise((r) => setTimeout(r, 10));
375+
376+
expect(mockPricingRepo.delete).not.toHaveBeenCalled();
377+
});
378+
368379
it('preserves custom: prefixed models even if not curated', async () => {
369380
mockPricingRepo.find.mockResolvedValue([
370381
{ model_name: 'custom:provider-uuid/my-model', provider: 'Custom' },
@@ -391,6 +402,7 @@ describe('LocalBootstrapService', () => {
391402
{ model_name: 'random-model-1', provider: 'Unknown' },
392403
{ model_name: 'random-model-2', provider: 'Unknown' },
393404
{ model_name: 'llama3:latest', provider: 'Ollama' }, // preserved
405+
{ model_name: 'qwen/qwen3-235b-a22b', provider: 'OpenRouter' }, // preserved
394406
{ model_name: 'custom:uuid/my-llm', provider: 'Custom' }, // preserved
395407
]);
396408

@@ -404,6 +416,7 @@ describe('LocalBootstrapService', () => {
404416
);
405417
expect(deleteArg.model_name._value).not.toContain('claude-opus-4-6');
406418
expect(deleteArg.model_name._value).not.toContain('llama3:latest');
419+
expect(deleteArg.model_name._value).not.toContain('qwen/qwen3-235b-a22b');
407420
expect(deleteArg.model_name._value).not.toContain('custom:uuid/my-llm');
408421
});
409422
});

packages/backend/src/database/local-bootstrap.service.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -193,6 +193,7 @@ export class LocalBootstrapService implements OnModuleInit {
193193
(row) =>
194194
!curatedNames.has(row.model_name) &&
195195
row.provider !== 'Ollama' &&
196+
row.provider !== 'OpenRouter' &&
196197
!row.model_name.startsWith('custom:'),
197198
)
198199
.map((row) => row.model_name);

0 commit comments

Comments
 (0)