Skip to content

feat: expose multi-agent v2 as model-only tools#22514

Merged
jif-oai merged 1 commit into
mainfrom
jif/code-mode-ma-2
May 13, 2026
Merged

feat: expose multi-agent v2 as model-only tools#22514
jif-oai merged 1 commit into
mainfrom
jif/code-mode-ma-2

Conversation

@jif-oai

@jif-oai jif-oai commented May 13, 2026

Copy link
Copy Markdown
Collaborator

Why

code_mode_only filters code-mode nested tools out of the top-level tool list. For multi-agent v2, we need a rollout shape where the collaboration tools remain callable as normal model tools without also being embedded into the code-mode exec tool declaration.

Related to this: https://openai-corpws.slack.com/archives/C0AQLHB4U75/p1778660267922549

What Changed

  • Adds features.multi_agent_v2.non_code_mode_only, including config resolution, profile override handling, and generated schema coverage.
  • Introduces ToolExposure::DirectModelOnly so a tool can be included in the initial model-visible list while staying out of the nested code-mode tool surface.
  • Applies that exposure to the multi-agent v2 tools when the new flag is set: spawn_agent, send_message, followup_task, wait_agent, close_agent, and list_agents.
  • Updates code-mode-only filtering so direct-model-only tools remain visible while ordinary nested code-mode tools are still hidden.

Verification

  • Added config parsing/profile tests for non_code_mode_only.
  • Added tool spec coverage for the code-mode-only multi-agent v2 exposure behavior.

@jif-oai jif-oai requested a review from a team as a code owner May 13, 2026 16:37
@jif-oai

jif-oai commented May 13, 2026

Copy link
Copy Markdown
Collaborator Author

@codex review

@jif-oai jif-oai changed the title feat: multi_agent_v2_non_code_mode_only feat: expose multi-agent v2 as model-only tools May 13, 2026
@chatgpt-codex-connector

Copy link
Copy Markdown
Contributor

Codex Review: Didn't find any major issues. What shall we delve into next?

ℹ️ 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".

@jif-oai

jif-oai commented May 13, 2026

Copy link
Copy Markdown
Collaborator Author

@codex review

@jif-oai jif-oai force-pushed the jif/code-mode-ma-2 branch from 26fbdc1 to bdb3b1e Compare May 13, 2026 16:56

@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: bdb3b1ea66

ℹ️ 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/tools/spec_plan.rs
andmis
andmis previously requested changes May 13, 2026

@andmis andmis left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Left some comments!

Edit: oops, wrong PR, disregard.

@andmis andmis dismissed their stale review May 13, 2026 17:20

Oops, wrong PR

Comment thread codex-rs/core/src/tools/router.rs
@jif-oai jif-oai merged commit fc26af3 into main May 13, 2026
53 of 54 checks passed
@jif-oai jif-oai deleted the jif/code-mode-ma-2 branch May 13, 2026 17:49
@github-actions github-actions Bot locked and limited conversation to collaborators May 13, 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.

3 participants