You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Hermes already handles reasoning_content, reasoning, and streaming delta.reasoning fields from OpenAI-compatible providers — this was a documentation gap only. This PR adds a clear section under "Other Compatible Providers" explaining the support.
Changes
website/docs/integrations/providers.md:
Added new section ### Reasoning / Thinking Field Support after the "Other Compatible Providers" configuration example and before "Context Length Detection"
Documents that Hermes captures reasoning_content/reasoning into NormalizedResponse.provider_data
Explains gateway propagation via _ASSISTANT_REPLAY_FIELDS and the copy_reasoning_content_for_api helper
Covers specific use cases: vLLM --reasoning-parser qwen3, DeepSeek thinking, Codex Responses API, streaming
Documents how to disable reasoning output (chat_template_kwargs.enable_thinking=false)
References the relevant code locations for developers
agent/agent_runtime_helpers.py L1987: function definition
Correct
Issues
None. The documentation is accurate, well-placed, and follows the existing style.
Suggestions (Minor / Optional)
Line number volatility: The docs reference specific line numbers (e.g. "line 107", "lines 115-117", "line 1987"). These are accurate today but will drift as the codebase evolves. Consider whether the team wants to commit to keeping these in sync, or prefers general descriptions without line numbers. This is a pre-existing pattern in the docs, so it is consistent -- just worth being aware of.
Minimum Hermes version section: This is a useful addition for users, but no other section in this file has a version note. Consider moving this into the introductory paragraph (e.g. "This support is shipped and stable in the current Hermes release") to keep the section structure uniform.
Summary
This PR closes a genuine documentation gap identified in issue #38360. All code references are accurate, the documentation is clearly written, and it fits naturally between the Other Compatible Providers and Context Length Detection sections. No issues found -- ready to merge.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Closes #38360.
Hermes already handles
reasoning_content,reasoning, and streamingdelta.reasoningfields from OpenAI-compatible providers — this was a documentation gap only. This PR adds a clear section under "Other Compatible Providers" explaining the support.Changes
website/docs/integrations/providers.md:### Reasoning / Thinking Field Supportafter the "Other Compatible Providers" configuration example and before "Context Length Detection"reasoning_content/reasoningintoNormalizedResponse.provider_data_ASSISTANT_REPLAY_FIELDSand thecopy_reasoning_content_for_apihelper--reasoning-parser qwen3, DeepSeek thinking, Codex Responses API, streamingchat_template_kwargs.enable_thinking=false)Verification
NormalizedResponse.reasoning_contentproperty:agent/transports/types.pylines 115-117 ✅gateway/run.pylines 475-541 ✅gateway/session.pyline 1270 ✅copy_reasoning_content_for_api:agent/agent_runtime_helpers.pyline 1987 ✅