From 1c91c46326cbcf7fd4c2546183b5c6a495198a8d Mon Sep 17 00:00:00 2001 From: Chris Weller Date: Sun, 26 Apr 2026 14:53:25 -0600 Subject: [PATCH] fix(proxy): avoid settings override and nested reasoning for openai-compat --- src/ccs.ts | 6 +----- src/proxy/transformers/request-transformer.ts | 10 +--------- .../request-transformer-regressions.test.ts | 4 ++-- .../proxy/transformers/request-transformer.test.ts | 2 +- 4 files changed, 5 insertions(+), 17 deletions(-) diff --git a/src/ccs.ts b/src/ccs.ts index 23389316..24bb75be 100644 --- a/src/ccs.ts +++ b/src/ccs.ts @@ -1460,11 +1460,7 @@ async function main(): Promise { }; delete proxyEnv.ANTHROPIC_API_KEY; - const launchArgs = [ - '--settings', - expandedSettingsPath, - ...appendThirdPartyWebSearchToolArgs(browserArgs), - ]; + const launchArgs = [...appendThirdPartyWebSearchToolArgs(browserArgs)]; const traceEnv = createWebSearchTraceContext({ launcher: 'ccs.settings-profile.proxy', args: launchArgs, diff --git a/src/proxy/transformers/request-transformer.ts b/src/proxy/transformers/request-transformer.ts index 58e7969e..b5877bd6 100644 --- a/src/proxy/transformers/request-transformer.ts +++ b/src/proxy/transformers/request-transformer.ts @@ -362,7 +362,7 @@ function transformToolChoice( function mapThinkingToReasoning( thinking: AnthropicThinking | undefined, outputConfig: AnthropicOutputConfig | undefined -): Pick { +): Pick { if (!thinking || thinking.type === 'disabled') { return {}; } @@ -371,10 +371,6 @@ function mapThinkingToReasoning( const effort = toOpenAIEffort(resolveOutputConfigEffort(outputConfig) ?? 'high'); return { reasoning_effort: effort, - reasoning: { - enabled: true, - effort, - }, }; } @@ -389,10 +385,6 @@ function mapThinkingToReasoning( return { reasoning_effort: effort, - reasoning: { - enabled: true, - effort, - }, }; } diff --git a/tests/unit/proxy/transformers/request-transformer-regressions.test.ts b/tests/unit/proxy/transformers/request-transformer-regressions.test.ts index 4bb3192f..9e4bc5a8 100644 --- a/tests/unit/proxy/transformers/request-transformer-regressions.test.ts +++ b/tests/unit/proxy/transformers/request-transformer-regressions.test.ts @@ -27,7 +27,7 @@ describe('ProxyRequestTransformer regressions', () => { }); expect(result.reasoning_effort).toBe('high'); - expect(result.reasoning).toEqual({ enabled: true, effort: 'high' }); + expect(result.reasoning).toBeUndefined(); }); it('explicitly normalizes anthropic xhigh adaptive effort for OpenAI-compatible upstreams', () => { @@ -38,7 +38,7 @@ describe('ProxyRequestTransformer regressions', () => { }); expect(result.reasoning_effort).toBe('high'); - expect(result.reasoning).toEqual({ enabled: true, effort: 'high' }); + expect(result.reasoning).toBeUndefined(); }); it('rejects unsupported thinking types instead of silently dropping them', () => { diff --git a/tests/unit/proxy/transformers/request-transformer.test.ts b/tests/unit/proxy/transformers/request-transformer.test.ts index 25687c20..543361a1 100644 --- a/tests/unit/proxy/transformers/request-transformer.test.ts +++ b/tests/unit/proxy/transformers/request-transformer.test.ts @@ -28,7 +28,7 @@ describe('ProxyRequestTransformer', () => { expect(result.stream).toBe(true); expect(result.reasoning_effort).toBe('high'); - expect(result.reasoning).toEqual({ enabled: true, effort: 'high' }); + expect(result.reasoning).toBeUndefined(); expect(result.max_tokens).toBe(1024); expect(result.temperature).toBe(0.2); expect(result.top_p).toBe(0.9);