Skip to content

fix(deepseek-reasoner): add missing reasoning_content field#5595

Closed
hxj0x wants to merge 1 commit into
spring-projects:mainfrom
hxj0x:gh-5027
Closed

fix(deepseek-reasoner): add missing reasoning_content field#5595
hxj0x wants to merge 1 commit into
spring-projects:mainfrom
hxj0x:gh-5027

Conversation

@hxj0x
Copy link
Copy Markdown

@hxj0x hxj0x commented Mar 13, 2026

Resolves #5027

@hxj0x hxj0x changed the title fix: deepseek-reasoner Missing reasoning_content field in the assista… fix(deepseek-reasoner): add missing reasoning_content field Mar 13, 2026
@sdeleuze sdeleuze added this to the 2.0.0-M4 milestone Mar 16, 2026
@sdeleuze sdeleuze added bug Something isn't working deepseek labels Mar 16, 2026
…nt message at message

Resolves # 5027

Signed-off-by: huangxj <2063818987@qq.com>
@sdeleuze
Copy link
Copy Markdown
Contributor

sdeleuze commented May 3, 2026

@ilayaperumalg I am not able to run Deepseek integration tests, I think you are, so I am tentavely assigning this PR to you. Please unassigned if you prefer not handling it.

@ilayaperumalg
Copy link
Copy Markdown
Member

@sdeleuze sorry, I don't have the keys to test as well. Moving this to 2.0.x for now.

@ilayaperumalg ilayaperumalg modified the milestones: 2.0.0-M6, 2.0.x May 5, 2026
@intellijboy
Copy link
Copy Markdown

😄 The ball is rolling。。。

@hxj0x
Copy link
Copy Markdown
Author

hxj0x commented May 9, 2026

Hi, I found that PR #5968 provides a more reasonable and simpler fix, which I've tested locally.

Therefore, I'm closing this pull request. Please maintainers focus on reviewing PR #5968 or PR #5908 instead.

Thanks for your time.

@hxj0x hxj0x closed this May 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working deepseek

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Deepseek-reasoner tool calling fails: Missing reasoning_content in assistant message history

4 participants