opencode/picoclaw
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(openai_compat): preserve reasoning_content for all DeepSeek assistant turns

Browse source 
Remove the hasToolInteraction guard in filterDeepSeekReasoningTurn that
was stripping reasoning_content from non-tool interaction assistant
messages. DeepSeek V3/V4+ thinking mode rejects requests where any
assistant message is missing reasoning_content, returning 400:
"The reasoning_content in the thinking mode must be passed back to the API."

This is a follow-up to the streaming fix that captured reasoning_content
from SSE deltas. That fix alone wasn't sufficient because reasoning_content
was still being stripped on replay for plain assistant turns.
This commit is contained in:
cj 2026-05-02 03:28:19 +00:00
parent e0f5ccff2d
commit b4aa407601
1 changed files with 4 additions and 15 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

19
pkg/providers/openai_compat/provider.go
View file

@ -255,14 +255,6 @@ func filterDeepSeekReasoningMessages(messages []Message) []Message {
} }
func filterDeepSeekReasoningTurn(messages []Message) []Message { func filterDeepSeekReasoningTurn(messages []Message) []Message {
hasToolInteraction := false
for _, msg := range messages {
if msg.Role == "tool" || (msg.Role == "assistant" && len(msg.ToolCalls) > 0) {
hasToolInteraction = true
break
}
}
out := make([]Message, 0, len(messages)) out := make([]Message, 0, len(messages))
for _, msg := range messages { for _, msg := range messages {
if messageutil.IsTransientAssistantThoughtMessage(msg) { if messageutil.IsTransientAssistantThoughtMessage(msg) {
@ -270,13 +262,10 @@ func filterDeepSeekReasoningTurn(messages []Message) []Message {
} }
cloned := msg cloned := msg
// DeepSeek thinking-mode replay only requires reasoning_content for // DeepSeek thinking-mode requires reasoning_content to be echoed
// turns that participate in a tool interaction round. For plain // back for ALL assistant turns. If missing the API returns 400:
// assistant turns between two user messages, the docs say the API will // "The reasoning_content in the thinking mode must be passed back
// ignore reasoning_content on replay, so we strip it here. // to the API."
if cloned.Role == "assistant" && strings.TrimSpace(cloned.ReasoningContent) != "" && !hasToolInteraction {
cloned.ReasoningContent = ""
}
if assistantMessageEmpty(cloned) { if assistantMessageEmpty(cloned) {
continue continue
} }