diff --git a/codex-rs/cli/src/mcp_cmd.rs b/codex-rs/cli/src/mcp_cmd.rs
index 2c2125baac..c53dd30f7a 100644
--- a/codex-rs/cli/src/mcp_cmd.rs
+++ b/codex-rs/cli/src/mcp_cmd.rs
@@ -89,7 +89,7 @@ pub struct AddArgs {
     group(
         ArgGroup::new("transport")
             .args(["command", "url"])
-            .required(true)
+            .required(/*value*/ true)
             .multiple(false)
     )
 )]
diff --git a/codex-rs/config/src/mcp_edit_tests.rs b/codex-rs/config/src/mcp_edit_tests.rs
index ff8047fbcd..a510a73267 100644
--- a/codex-rs/config/src/mcp_edit_tests.rs
+++ b/codex-rs/config/src/mcp_edit_tests.rs
@@ -23,7 +23,7 @@ async fn replace_mcp_servers_serializes_per_tool_approval_overrides() -> anyhow:
                 env_vars: Vec::new(),
                 cwd: None,
             })
-            .supports_parallel_tool_calls(true)
+            .supports_parallel_tool_calls(/*value*/ true)
             .default_tools_approval_mode(AppToolApproval::Auto)
             .tools(HashMap::from([
                 (
diff --git a/codex-rs/core/src/config/config_tests.rs b/codex-rs/core/src/config/config_tests.rs
index 0abd40c264..69bfed6cbe 100644
--- a/codex-rs/core/src/config/config_tests.rs
+++ b/codex-rs/core/src/config/config_tests.rs
@@ -5960,7 +5960,7 @@ async fn replace_mcp_servers_serializes_disabled_flag() -> anyhow::Result<()> {
                 env_vars: Vec::new(),
                 cwd: None,
             })
-            .enabled(false)
+            .enabled(/*value*/ false)
             .build(),
     )]);
 
@@ -5997,7 +5997,7 @@ async fn replace_mcp_servers_serializes_required_flag() -> anyhow::Result<()> {
                 env_vars: Vec::new(),
                 cwd: None,
             })
-            .required(true)
+            .required(/*value*/ true)
             .build(),
     )]);
 
diff --git a/codex-rs/core/src/config/edit_tests.rs b/codex-rs/core/src/config/edit_tests.rs
index d60f1645e9..043b9bfbc8 100644
--- a/codex-rs/core/src/config/edit_tests.rs
+++ b/codex-rs/core/src/config/edit_tests.rs
@@ -887,7 +887,7 @@ fn blocking_replace_mcp_servers_round_trips() {
                 env_vars: vec!["FOO".into()],
                 cwd: None,
             })
-            .supports_parallel_tool_calls(true)
+            .supports_parallel_tool_calls(/*value*/ true)
             .enabled_tools(vec!["one".to_string(), "two".to_string()])
             .build(),
     );
@@ -905,7 +905,7 @@ fn blocking_replace_mcp_servers_round_trips() {
                 ),
                 env_http_headers: None,
             })
-            .enabled(false)
+            .enabled(/*value*/ false)
             .startup_timeout_sec(std::time::Duration::from_secs(5))
             .disabled_tools(vec!["forbidden".to_string()])
             .oauth(McpServerOAuthConfig {
@@ -1051,7 +1051,7 @@ foo = { command = "cmd" } # keep me
                 env_vars: Vec::new(),
                 cwd: None,
             })
-            .enabled(false)
+            .enabled(/*value*/ false)
             .build(),
     );
 
@@ -1123,7 +1123,7 @@ foo = { command = "cmd" }
                 env_vars: Vec::new(),
                 cwd: None,
             })
-            .enabled(false)
+            .enabled(/*value*/ false)
             .build(),
     );