[Bug]: reasoning_content to reasoning rename caused regression in chat templates expecting reasoning_content #33616
Description
Your current environment
The output of python collect_env.py
Your output of `python collect_env.py` here
🐛 Describe the bug
A regression was introduced by #33402
The change to removing reasoning_content was intended to be for the chat completion endpoint, but that downstream effect of also breaking chat templates that expect reasoning_content (like glm and Kimi K2).
It's unlikely many models will be updated to fix this change and will silently degrade in performance. And other inference engines are still providing/using "reasoning_content". So I think this is at a reasonable fix that ensures backwards compatibility with older models.
Before submitting a new issue...
- Make sure you already searched for relevant issues, and asked the chatbot living at the bottom right corner of the documentation page, which can answer lots of frequently asked questions.