-
Notifications
You must be signed in to change notification settings - Fork 1.3k
feat: reasoning output responses api #5206
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
cdoern
merged 33 commits into
ogx-ai:main
from
robinnarsinghranabhat:feat/reasoning-output-responses-api
Apr 1, 2026
Merged
Changes from 32 commits
Commits
Show all changes
33 commits
Select commit
Hold shift + click to select a range
65583ae
feat: add reasoning output support for OpenAI Responses API
rranabha 929c5a8
chore: regenerate openai coverage after adding ReasoningItem variant
robinnarsinghranabhat 395e48f
test: add integration tests for reasoning output in Responses API
robinnarsinghranabhat c720a44
fix: use openai_client in reasoning integration tests
robinnarsinghranabhat 66df6d6
feat: provider-level reasoning support via openai_chat_completions_wi…
robinnarsinghranabhat d3ebff2
refactor: use internal AssistantMessageWithReasoning instead of modif…
robinnarsinghranabhat e401547
test: add reasoning effort param to integration tests
robinnarsinghranabhat 41ba0a0
feat: graceful fallback for reasoning unsupported providers
robinnarsinghranabhat ad81e02
fix: minor cleanups in reasoning tests
rranabha 15a3eaf
fix: make integration tests compatible with current LlamaStack client
robinnarsinghranabhat d911ba7
Recordings update from CI
github-actions[bot] fae02a5
test: re-record ollama-reasoning and vllm-reasoning integration test …
robinnarsinghranabhat ad15e40
fix: pass params copy to reasoning method to prevent router mutation …
robinnarsinghranabhat 86deda6
test: re-record gpt-reasoning integration test recordings
robinnarsinghranabhat f7cae37
chore: pre-commit formatting fixes
robinnarsinghranabhat b36f910
chore: regenerate docs after rebase
robinnarsinghranabhat 61b72fa
fix: clear reasoning_effort on fallback to prevent provider rejection
robinnarsinghranabhat d3477d9
fix: remove flawed unsupported-provider fallback test
robinnarsinghranabhat a06ff17
chore: restore recordings from main to clean state before re-recording
robinnarsinghranabhat f8a0c56
ollama-reasoning and gpt-reasoning integration test recordings
robinnarsinghranabhat 4a25d84
ci: add --reasoning-parser to vLLM CI setup for reasoning test support
robinnarsinghranabhat 85ca2a8
missing updates from rebase
robinnarsinghranabhat 4a0a0b7
updated recordings for ollama-reasoning, vllm-reasoning, gpt-reasoning
robinnarsinghranabhat ccf2756
spec changes after pre-commit
robinnarsinghranabhat 8433a63
fix: resolve mypy errors for reasoning method across providers
robinnarsinghranabhat 8c6bc42
fix: resolve mypy type errors in reasoning implementation
robinnarsinghranabhat 9154fae
unused recordings detected
robinnarsinghranabhat 570667e
updated after local pre-commit
robinnarsinghranabhat 5ff28f2
refactor: use wrapper dataclass for reasoning return types per mattf …
robinnarsinghranabhat e6b8493
record missing
robinnarsinghranabhat b4565e1
chore: regenerate conformance docs after rebase
robinnarsinghranabhat ef74218
recorded a failing integration test -- Integration Tests (server, oll…
robinnarsinghranabhat db5367e
Merge branch 'main' into feat/reasoning-output-responses-api
cdoern File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
Large diffs are not rendered by default.
Oops, something went wrong.
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
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
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
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
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
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
Oops, something went wrong.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.