Skip to content

fix(agent): normalize empty assistant content to None when tool_calls present to fix DeepSeek HTTP 400 (#63200) - #63254

Closed
webtecnica wants to merge 1 commit into
NousResearch:mainfrom
webtecnica:fix/63200-deepseek-empty-content
Closed

fix(agent): normalize empty assistant content to None when tool_calls present to fix DeepSeek HTTP 400 (#63200)#63254
webtecnica wants to merge 1 commit into
NousResearch:mainfrom
webtecnica:fix/63200-deepseek-empty-content

Conversation

@webtecnica

@webtecnica webtecnica commented Jul 12, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Closing as duplicate — see comment above.

@alt-glitch alt-glitch added type/bug Something isn't working comp/agent Core agent runtime: loop, agent_init, prompt builder, context-compression, responses endpoint provider/deepseek DeepSeek API P2 Medium — degraded but workaround exists duplicate This issue or pull request already exists labels Jul 12, 2026
@alt-glitch

alt-glitch commented Jul 12, 2026

Copy link
Copy Markdown
Collaborator

This was generated by AI during triage.

Duplicate of #31582 — both normalize empty-string assistant content to None when tool_calls are present, at the same sanitize_api_messages() chokepoint (verified against both diffs). #31582 is the earlier open PR for the same #11906/#63200 empty-content family. Note this PR also bundles an unrelated "Repair pass 5" concatenated-JSON change in agent/message_sanitization.py that isn't in the description — worth splitting out if that fix is wanted on its own.

@webtecnica

webtecnica commented Jul 12, 2026

Copy link
Copy Markdown
Contributor Author

Closing as duplicate — the sweeper identified this as already covered by #58768 (#59110, merged Jul 6) which added empty tool_calls normalization in sanitize_api_messages(). Our fix added similar logic at the same chokepoint. 👍

@webtecnica webtecnica closed this Jul 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

comp/agent Core agent runtime: loop, agent_init, prompt builder, context-compression, responses endpoint duplicate This issue or pull request already exists P2 Medium — degraded but workaround exists provider/deepseek DeepSeek API type/bug Something isn't working

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@webtecnica @alt-glitch