Skip to content

common : fix gpt-oss Jinja error with content and thinking on tool-call messages#19704

Merged
aldehir merged 1 commit intoggml-org:masterfrom
abhijitb11:fix/gpt-oss-thinking-content-conflict
Feb 19, 2026
Merged

common : fix gpt-oss Jinja error with content and thinking on tool-call messages#19704
aldehir merged 1 commit intoggml-org:masterfrom
abhijitb11:fix/gpt-oss-thinking-content-conflict

Conversation

@abhijitb11
Copy link
Copy Markdown
Contributor

Erase the content from the adjusted message after copying reasoning_content to thinking.

Regression from #16937

Fixes #19703.

All four tests related to chat have passed in this fix:

  • test-chat
  • test-chat-parser
  • test-chat-peg-parser
  • test-chat-template

AI was used in an assistive capacity to help trace the regression source.

…ent and thinking with tool calls

Erase `content` from the adjusted message after copying `reasoning_content`
to `thinking`, so the gpt-oss template does not see both fields on
tool-call messages.

Regression from ggml-org#16937.
@abhijitb11 abhijitb11 requested a review from pwilkin as a code owner February 18, 2026 05:12
@aldehir aldehir merged commit 39e4b1d into ggml-org:master Feb 19, 2026
73 of 78 checks passed
liparetejas pushed a commit to liparetejas/llama.cpp that referenced this pull request Feb 23, 2026
bartowski1182 pushed a commit to bartowski1182/llama.cpp that referenced this pull request Mar 2, 2026
ArberSephirotheca pushed a commit to ArberSephirotheca/llama.cpp that referenced this pull request Mar 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Misc. bug: gpt-oss Jinja template error when assistant message has reasoning_content and tool_calls

3 participants