Skip to content

Configure a subagent_models_manager#15048

Draft
surajs-oai wants to merge 1 commit intomainfrom
surajs/subagent_models
Draft

Configure a subagent_models_manager#15048
surajs-oai wants to merge 1 commit intomainfrom
surajs/subagent_models

Conversation

@surajs-oai
Copy link

Summary

  • add subagent_model_catalog_json / subagent_model_catalog config plumbing and regenerate the config schema
  • add a parallel subagent_models_manager alongside the main models_manager, defaulting to the same manager when no subagent catalog is configured
  • route spawn-agent tool description generation and runtime model validation through subagent models, and simplify ToolsConfig so it only carries available_subagent_models
  • update core tests to cover the new config and manager plumbing

Testing

  • just write-config-schema
  • CARGO_TARGET_DIR=/tmp/codex-rs-agent-target cargo test -p codex-core config::schema::tests::config_schema_matches_fixture -- --exact
  • CARGO_TARGET_DIR=/tmp/codex-rs-agent-target cargo test -p codex-core
    • fails only on tools::js_repl::tests::js_repl_imported_local_files_can_access_repl_globals because dotslash bootstrap hits mktemp ... Operation not permitted in this environment
  • CARGO_TARGET_DIR=/tmp/codex-rs-agent-target just fix -p codex-core
  • just fmt

Tickets

  • None

@github-actions
Copy link
Contributor


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@surajs-oai
Copy link
Author

I have read the CLA Document and I hereby sign the CLA

Copy link
Contributor

@andmis andmis left a comment

Choose a reason for hiding this comment

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

(Following up offline.)

@surajs-oai surajs-oai marked this pull request as draft March 18, 2026 17:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants