tiennm99/ccs
1
0 Fork 0
mirror of https://github.com/tiennm99/ccs.git synced 2026-10-14 06:17:27 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #1731 from kaitranntt/kai/fix-1728-openrouter-base-url

Browse Source 
fix(openrouter): use /api endpoint root to prevent duplicated /v1 (#1728)
This commit is contained in:
Kai (Tam Nhu) Tran authored and GitHub committed 2026-09-09 16:00:56 -06:00
2 parents 54c6695508 283e8843ac
commit d9cf5c0796
3 files changed
+16 -7
Download Patch File Download Diff File Expand all files Collapse all files

No files matched your search

src/shared/provider-preset-catalog.ts
+1 -1
View File
@@ -53,7 +53,7 @@ export interface ProviderPresetDefinition {
icon?: string;
}
export const OPENROUTER_BASE_URL = 'https://openrouter.ai/api/v1';
export const OPENROUTER_BASE_URL = 'https://openrouter.ai/api';
export const ORCAROUTER_BASE_URL = 'https://api.orcarouter.ai/v1';
/**
tests/unit/api/profile-writer-anthropic.test.ts
+10 -4
View File
@@ -3,6 +3,7 @@ import * as fs from 'fs';
import * as os from 'os';
import * as path from 'path';
import { createApiProfile } from '../../../src/api/services/profile-writer';
import { getPresetById } from '../../../src/api/services/provider-presets';
describe('profile-writer Anthropic direct', () => {
let tempHome = '';
@@ -131,10 +132,14 @@ describe('profile-writer Anthropic direct', () => {
expect(config.profile_targets?.['hf-target-claude']).toBeUndefined();
});
it('preserves OpenRouter ANTHROPIC_API_KEY blank behavior', () => {
it('preserves OpenRouter ANTHROPIC_API_KEY blank behavior and resolves preset baseUrl without duplicating /v1 (#1728)', () => {
const preset = getPresetById('openrouter');
expect(preset).toBeDefined();
expect(preset?.baseUrl).toBe('https://openrouter.ai/api');
const result = createApiProfile(
'openrouter-test',
'https://openrouter.ai/api/v1',
preset!.baseUrl,
'sk-or-testkey',
{ default: 'anthropic/claude-opus-4.5', opus: 'anthropic/claude-opus-4.5', sonnet: 'anthropic/claude-opus-4.5', haiku: 'anthropic/claude-opus-4.5' }
);
@@ -144,8 +149,9 @@ describe('profile-writer Anthropic direct', () => {
const settingsPath = path.join(tempHome, '.ccs', 'openrouter-test.settings.json');
const settings = JSON.parse(fs.readFileSync(settingsPath, 'utf8'));
// OpenRouter: proxy mode with ANTHROPIC_API_KEY explicitly blank
expect(settings.env.ANTHROPIC_BASE_URL).toBe('https://openrouter.ai/api/v1');
// OpenRouter: proxy mode with ANTHROPIC_API_KEY explicitly blank and baseUrl without /v1
expect(settings.env.ANTHROPIC_BASE_URL).toBe('https://openrouter.ai/api');
expect(`${settings.env.ANTHROPIC_BASE_URL}/v1/messages`).toBe('https://openrouter.ai/api/v1/messages');
expect(settings.env.ANTHROPIC_AUTH_TOKEN).toBe('sk-or-testkey');
expect(settings.env.ANTHROPIC_API_KEY).toBe('');
});
tests/unit/api/provider-presets.test.ts
+5 -2
View File
@@ -113,9 +113,12 @@ describe('provider-presets', () => {
expect(isValidPresetId('te')).toBe(true);
});
it('uses OpenRouter v1 as the OpenAI-compatible API root', () => {
it('uses OpenRouter /api (without /v1) as the Anthropic-compatible API root to avoid double /v1 404 in Claude Code (#1728)', () => {
const preset = getPresetById('openrouter');
expect(preset?.baseUrl).toBe('https://openrouter.ai/api/v1');
expect(preset?.baseUrl).toBe('https://openrouter.ai/api');
const finalEndpoint = `${preset?.baseUrl}/v1/messages`;
expect(finalEndpoint).toBe('https://openrouter.ai/api/v1/messages');
expect(finalEndpoint).not.toContain('/v1/v1/');
});
it('keeps Anthropic direct last in the recommended preset order and reuses the Claude logo', () => {