Skip to content

[codex] Use input items for Responses Lite tools - #27946

Merged
rka-oai merged 7 commits into
mainfrom
codex/responses-lite-additional-tools
Jun 23, 2026
Merged

[codex] Use input items for Responses Lite tools#27946
rka-oai merged 7 commits into
mainfrom
codex/responses-lite-additional-tools

Conversation

@rka-oai

@rka-oai rka-oai commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

When using Responses Lite, we should all use additional_tools and a developer item instead of the top level tools array & instructions field. This keeps things 1-to-1.

Forced namespacing for all tools will land in a following PR after some coordination & fixes in Responses API (around collisions & return items).

The goal is to eventually expand the scope of this to all requests from codex, but that will require larger coordination across providers & slower rollout.

Comment thread codex-rs/core/tests/suite/client_websockets.rs Outdated
@rka-oai
rka-oai marked this pull request as ready for review June 13, 2026 00:17
@rka-oai
rka-oai requested a review from a team as a code owner June 13, 2026 00:17

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: eb4deefaa6

ℹ️ About Codex in GitHub

Your team has set up Codex to 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 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread codex-rs/core/src/client.rs
Comment thread codex-rs/core/src/client.rs
@rka-oai
rka-oai force-pushed the codex/responses-lite-additional-tools branch 4 times, most recently from 47b01ad to 0bf5ab0 Compare June 17, 2026 05:13
@rka-oai
rka-oai force-pushed the codex/responses-lite-additional-tools branch from 0bf5ab0 to 0f97af0 Compare June 22, 2026 07:13
Comment thread codex-rs/core/tests/suite/responses_lite.rs
Comment thread codex-rs/codex-api/src/common.rs
@rka-oai
rka-oai force-pushed the codex/responses-lite-additional-tools branch from 1cf99c8 to 280e91e Compare June 22, 2026 23:59
@rka-oai
rka-oai merged commit 33cc928 into main Jun 23, 2026
31 checks passed
@rka-oai
rka-oai deleted the codex/responses-lite-additional-tools branch June 23, 2026 06:56
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 23, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants