openai/codex
1
0
Fork 0
mirror of https://github.com/openai/codex.git synced 2026-11-02 19:43:24 +00:00
Code Issues Packages Projects Releases Wiki Activity

Rename show_nux_new to show_nux

Browse Source
This commit is contained in:
Ahmed Ibrahim
2026-02-26 17:43:08 -08:00
parent 4abd8e9b24
commit d204d68c91
10 changed files with 22 additions and 22 deletions
Download Patch File Download Diff File Expand all files Collapse all files
codex-rs/app-server/tests/common/models_cache.rs
+1 -1
View File
@@ -34,7 +34,7 @@ fn preset_to_info(preset: &ModelPreset, priority: i32) -> ModelInfo {
default_reasoning_summary: ReasoningSummary::Auto,
support_verbosity: false,
default_verbosity: None,
show_nux_new: false,
show_nux: false,
apply_patch_tool_type: None,
truncation_policy: TruncationPolicyConfig::bytes(10_000),
supports_parallel_tool_calls: false,
codex-rs/codex-api/tests/models_integration.rs
+1 -1
View File
@@ -82,7 +82,7 @@ async fn models_client_hits_models_endpoint() {
default_reasoning_summary: ReasoningSummary::Auto,
support_verbosity: false,
default_verbosity: None,
show_nux_new: false,
show_nux: false,
apply_patch_tool_type: None,
truncation_policy: TruncationPolicyConfig::bytes(10_000),
supports_parallel_tool_calls: false,
codex-rs/core/src/models_manager/model_info.rs
+1 -1
View File
@@ -69,7 +69,7 @@ pub(crate) fn model_info_from_slug(slug: &str) -> ModelInfo {
visibility: ModelVisibility::None,
supported_in_api: true,
priority: 99,
show_nux_new: false,
show_nux: false,
upgrade: None,
base_instructions: BASE_INSTRUCTIONS.to_string(),
model_messages: local_personality_messages_for_slug(slug),
codex-rs/core/tests/suite/model_switching.rs
+2 -2
View File
@@ -237,7 +237,7 @@ async fn model_change_from_image_to_text_strips_prior_image_content() -> Result<
default_reasoning_summary: ReasoningSummary::Auto,
support_verbosity: false,
default_verbosity: None,
show_nux_new: false,
show_nux: false,
apply_patch_tool_type: None,
truncation_policy: TruncationPolicyConfig::bytes(10_000),
supports_parallel_tool_calls: false,
@@ -396,7 +396,7 @@ async fn model_switch_to_smaller_model_updates_token_context_window() -> Result<
default_reasoning_summary: ReasoningSummary::Auto,
support_verbosity: false,
default_verbosity: None,
show_nux_new: false,
show_nux: false,
apply_patch_tool_type: None,
truncation_policy: TruncationPolicyConfig::bytes(10_000),
supports_parallel_tool_calls: false,
codex-rs/core/tests/suite/models_cache_ttl.rs
+1 -1
View File
@@ -339,7 +339,7 @@ fn test_remote_model(slug: &str, priority: i32) -> ModelInfo {
default_reasoning_summary: ReasoningSummary::Auto,
support_verbosity: false,
default_verbosity: None,
show_nux_new: false,
show_nux: false,
apply_patch_tool_type: None,
truncation_policy: TruncationPolicyConfig::bytes(10_000),
supports_parallel_tool_calls: false,
codex-rs/core/tests/suite/personality.rs
+2 -2
View File
@@ -602,7 +602,7 @@ async fn remote_model_friendly_personality_instructions_with_feature() -> anyhow
default_reasoning_summary: ReasoningSummary::Auto,
support_verbosity: false,
default_verbosity: None,
show_nux_new: false,
show_nux: false,
apply_patch_tool_type: None,
truncation_policy: TruncationPolicyConfig::bytes(10_000),
supports_parallel_tool_calls: false,
@@ -711,7 +711,7 @@ async fn user_turn_personality_remote_model_template_includes_update_message() -
default_reasoning_summary: ReasoningSummary::Auto,
support_verbosity: false,
default_verbosity: None,
show_nux_new: false,
show_nux: false,
apply_patch_tool_type: None,
truncation_policy: TruncationPolicyConfig::bytes(10_000),
supports_parallel_tool_calls: false,
codex-rs/core/tests/suite/remote_models.rs
+3 -3
View File
@@ -297,7 +297,7 @@ async fn remote_models_remote_model_uses_unified_exec() -> Result<()> {
default_reasoning_summary: ReasoningSummary::Auto,
support_verbosity: false,
default_verbosity: None,
show_nux_new: false,
show_nux: false,
apply_patch_tool_type: None,
truncation_policy: TruncationPolicyConfig::bytes(10_000),
supports_parallel_tool_calls: false,
@@ -535,7 +535,7 @@ async fn remote_models_apply_remote_base_instructions() -> Result<()> {
default_reasoning_summary: ReasoningSummary::Auto,
support_verbosity: false,
default_verbosity: None,
show_nux_new: false,
show_nux: false,
apply_patch_tool_type: None,
truncation_policy: TruncationPolicyConfig::bytes(10_000),
supports_parallel_tool_calls: false,
@@ -997,7 +997,7 @@ fn test_remote_model_with_policy(
default_reasoning_summary: ReasoningSummary::Auto,
support_verbosity: false,
default_verbosity: None,
show_nux_new: false,
show_nux: false,
apply_patch_tool_type: None,
truncation_policy,
supports_parallel_tool_calls: false,
codex-rs/core/tests/suite/rmcp_client.rs
+1 -1
View File
@@ -401,7 +401,7 @@ async fn stdio_image_responses_are_sanitized_for_text_only_model() -> anyhow::Re
default_reasoning_summary: ReasoningSummary::Auto,
support_verbosity: false,
default_verbosity: None,
show_nux_new: false,
show_nux: false,
apply_patch_tool_type: None,
truncation_policy: TruncationPolicyConfig::bytes(10_000),
supports_parallel_tool_calls: false,
codex-rs/core/tests/suite/view_image.rs
+1 -1
View File
@@ -678,7 +678,7 @@ async fn view_image_tool_returns_unsupported_message_for_text_only_model() -> an
default_reasoning_summary: ReasoningSummary::Auto,
support_verbosity: false,
default_verbosity: None,
show_nux_new: false,
show_nux: false,
apply_patch_tool_type: None,
truncation_policy: TruncationPolicyConfig::bytes(10_000),
supports_parallel_tool_calls: false,
codex-rs/protocol/src/openai_models.rs
+9 -9
View File
@@ -125,7 +125,7 @@ pub struct ModelPreset {
pub show_in_picker: bool,
/// Whether to show NUX for this preset.
#[serde(default)]
pub show_nux_new: bool,
pub show_nux: bool,
/// whether this model is supported in the api
pub supported_in_api: bool,
/// Input modalities accepted when composing user turns for this preset.
@@ -229,7 +229,7 @@ pub struct ModelInfo {
pub supported_in_api: bool,
pub priority: i32,
#[serde(default)]
pub show_nux_new: bool,
pub show_nux: bool,
pub upgrade: Option<ModelInfoUpgrade>,
pub base_instructions: String,
#[serde(default, skip_serializing_if = "Option::is_none")]
@@ -415,7 +415,7 @@ impl From<ModelInfo> for ModelPreset {
migration_markdown: Some(upgrade.migration_markdown.clone()),
}),
show_in_picker: info.visibility == ModelVisibility::List,
show_nux_new: info.show_nux_new,
show_nux: info.show_nux,
supported_in_api: info.supported_in_api,
input_modalities: info.input_modalities,
}
@@ -501,7 +501,7 @@ mod tests {
visibility: ModelVisibility::List,
supported_in_api: true,
priority: 1,
show_nux_new: false,
show_nux: false,
upgrade: None,
base_instructions: "base".to_string(),
model_messages: spec,
@@ -677,7 +677,7 @@ mod tests {
}
#[test]
fn model_info_defaults_show_nux_new_to_false_when_omitted() {
fn model_info_defaults_show_nux_to_false_when_omitted() {
let model: ModelInfo = serde_json::from_value(serde_json::json!({
"slug": "test-model",
"display_name": "Test Model",
@@ -709,16 +709,16 @@ mod tests {
}))
.expect("deserialize model info");
assert!(!model.show_nux_new);
assert!(!model.show_nux);
}
#[test]
fn model_preset_preserves_show_nux_new() {
fn model_preset_preserves_show_nux() {
let preset = ModelPreset::from(ModelInfo {
show_nux_new: true,
show_nux: true,
..test_model(None)
});
assert!(preset.show_nux_new);
assert!(preset.show_nux);
}
}