github/claude-code-best
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: update DeepSeek empty thinking test for v4 compatibility

Browse Source 
DeepSeek v4 requires reasoning_content to be echoed back even when
empty, otherwise the API returns HTTP 400. The implementation already
handles this correctly (see convertMessages.ts L253-257 comment),
but the test expected undefined instead of empty string.
This commit is contained in:
James Feng 2026-06-02 01:04:30 +08:00
parent 289fc9bf2e
commit af60071f6a
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/services/api/openai/__tests__/convertMessages.test.ts
View File

@ -432,7 +432,8 @@ describe('DeepSeek thinking mode (enableThinking)', () => {
{ enableThinking: true }, { enableThinking: true },
) )
const assistant = result.filter(m => m.role === 'assistant')[0] as any const assistant = result.filter(m => m.role === 'assistant')[0] as any
expect(assistant.reasoning_content).toBeUndefined() // DeepSeek v4 requires reasoning_content to be echoed back, even when empty
expect(assistant.reasoning_content).toBe('')
}) })
test('sets content to null when only thinking and tool_calls present', () => { test('sets content to null when only thinking and tool_calls present', () => {