Greptile Summary

This PR fixes a bug where sending reasoning_effort: "none" to a DeepSeek channel would result in both "reasoning_effort":"none" and "thinking":{"type":"disabled"} being forwarded to the DeepSeek API simultaneously, causing an HTTP 400 because DeepSeek does not accept "none" as a valid reasoning effort value.

• Clears dsReq.Request.ReasoningEffort to "" in outbound.go when thinkingDisabled is true, so the omitempty JSON tag suppresses the field from the serialized body.
• The parallel Anthropic-format path (llm/transformer/anthropic/outbound_convert.go) was already fixed in fix(llm): missing anthropic thinking disabled transform #1552; this closes the same gap in the direct OpenAI-format DeepSeek path.

Confidence Score: 4/5

The change is a minimal, targeted one-liner that correctly exploits the existing omitempty tag to suppress the rejected field. Safe to merge.

The fix is correct and well-scoped. The only gap is that the existing test for the 'none' reasoning effort case doesn't assert that ReasoningEffort is absent from the serialized JSON, so a regression could slip through unnoticed in the future.

llm/transformer/deepseek/outbound_test.go — the test covering this path should assert that ReasoningEffort is empty in the output body.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Incoming llm.Request\nReasoningEffort = 'none'] --> B[Build dsReq from oaiReq\nReasoningEffort = 'none' copied in]
    B --> C{thinkingDisabled?\nReasoningEffort == 'none'}
    C -- Yes --> D[Set Thinking.Type = 'disabled']
    D --> E[Clear ReasoningEffort = ''\nomitempty omits from JSON]
    E --> F[JSON body sent to DeepSeek\nthinking: disabled ONLY]
    C -- No --> G[Set Thinking.Type = 'enabled'\nKeep ReasoningEffort as-is]
    G --> H[Fill empty reasoning_content\nin assistant messages]
    H --> I[JSON body sent to DeepSeek\nthinking: enabled + reasoning_effort]
    F --> J[DeepSeek API accepts]
    I --> J

_{Reviews (1): Last reviewed commit: "fix(llm): clear ReasoningEffort when Dee..." | Re-trigger Greptile}