Skip to content

feat(core): support custom model aliases in config.toml#13825

Open
friel-openai wants to merge 1 commit intomainfrom
dev/codex/add-custom-models-support-in-config.toml-r2
Open

feat(core): support custom model aliases in config.toml#13825
friel-openai wants to merge 1 commit intomainfrom
dev/codex/add-custom-models-support-in-config.toml-r2

Conversation

@friel-openai
Copy link
Contributor

@friel-openai friel-openai commented Mar 7, 2026

Summary

  • add config.toml-defined custom model aliases that map to provider model slugs
  • apply alias-specific model_context_window and model_auto_compact_token_limit overrides
  • list configured custom aliases before bundled and remote models in the picker
  • add config/schema/docs coverage plus a regression test for the [[custom_models]] array form

Testing

  • cargo test -p codex-core models_manager::manager::tests::get_model_info_uses_custom_alias_metadata_and_request_model -- --exact
  • cargo test -p codex-core models_manager::manager::tests::build_available_models_lists_custom_aliases_before_remote_models -- --exact
  • cargo test -p codex-core config::tests::config_toml_deserializes_custom_models -- --exact
  • just fix -p codex-core

friel-openai added a commit that referenced this pull request Mar 10, 2026
@friel-openai friel-openai force-pushed the dev/codex/add-custom-models-support-in-config.toml-r2 branch from 4459a79 to 2b8c565 Compare March 10, 2026 17:53
Add custom model aliases configured with [[custom_models]].\n\nThis keeps alias-specific context and auto-compact overrides, makes alias values take precedence over the global model overrides when an alias is selected, and updates the current thread-manager / test harness wiring to match main.
@friel-openai friel-openai force-pushed the dev/codex/add-custom-models-support-in-config.toml-r2 branch from 2b8c565 to 143ba10 Compare March 10, 2026 22:14
@friel-openai friel-openai requested a review from pakrym-oai March 10, 2026 23:16
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