OpenAI Responses API: compatible with reasoning summary and encryption - #148
Sha1rholder wants to merge 3 commits into
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 193dab173a
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
|
Sorry, forgot to update after modifying the code. I’ll fix it in a bit. |
|
Resolved the Git conflict. @Neroued 请问能review一下吗?看一下第一个commit就行,只是给Responses API加了个占位符,很小的改动 |
…93dab1) PR Neroued#148 port onto our master-adopt tree: OpenAI Responses API now accepts 'include': ['reasoning.encrypted_content'] (sets the encrypted-reasoning flag and returns it in the response) instead of 400 include_not_supported. Also accepts a passed-in reasoning.summary string. Resolved conflicts: server logs use our newer record_request_rejected API (PR's log_compatibility_warning/log_request_start dropped); rest of the commit applied as-is. Needed by Inspect AI 0.3.260 GAIA runs.
… tooling') main is upstream 487f897 + 9 commits: own nvfp4full registration + graft tool, the Neroued#148/Neroued#97/Neroued#160/Neroued#61 PR ports (temp, drop as they merge), the registry doc, and curl for healthchecks. Registry table in PATCHES.md now covers all 9 (added curl + registry rows, date -> 2026-09-15).
…93dab1) PR Neroued#148 port onto our master-adopt tree: OpenAI Responses API now accepts 'include': ['reasoning.encrypted_content'] (sets the encrypted-reasoning flag and returns it in the response) instead of 400 include_not_supported. Also accepts a passed-in reasoning.summary string. Resolved conflicts: server logs use our newer record_request_rejected API (PR's log_compatibility_warning/log_request_start dropped); rest of the commit applied as-is. Needed by Inspect AI 0.3.260 GAIA runs.
… tooling') main is upstream 487f897 + 9 commits: own nvfp4full registration + graft tool, the Neroued#148/Neroued#97/Neroued#160/Neroued#61 PR ports (temp, drop as they merge), the registry doc, and curl for healthchecks. Registry table in PATCHES.md now covers all 9 (added curl + registry rows, date -> 2026-09-15).
|
@Sha1rholder thanks, I'm trying to use Codex too. Your branch is 151 commits behind master and has merge conflicts, so I rebased it and opened a PR as #295. The rebase was mostly routine. I did drop the Happy to close #295 if you're all over it. |
|
@Macasacker Please proceed with #295. I haven't been actively maintaining this PR recently, as I wasn't sure if it would get reviewed and merged. Thanks for taking over and cleaning this up. Feel free to continue with your PR. |
Please see the issue for details. Fixes #145
The code changes are actually minimal. Mostly are actually focused on adapting the test content.
before:

After:

The logs show that the compatibility improvements do not affect cache hits.