From f7b242b81dc956cc5f499e32ff53f4edd8d7e2a2 Mon Sep 17 00:00:00 2001 From: Eugene Brevdo Date: Wed, 4 Mar 2026 09:41:29 -0800 Subject: [PATCH] fixes --- codex-rs/core/src/approval_handler.rs | 2 +- codex-rs/core/tests/suite/approvals.rs | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/codex-rs/core/src/approval_handler.rs b/codex-rs/core/src/approval_handler.rs index ca98da1f28..778350ee14 100644 --- a/codex-rs/core/src/approval_handler.rs +++ b/codex-rs/core/src/approval_handler.rs @@ -346,7 +346,7 @@ mod tests { decision, } => { assert_eq!(id, event.effective_approval_id()); - assert_eq!(turn_id, Some(event.turn_id.clone())); + assert_eq!(turn_id.as_deref(), Some(event.turn_id.as_str())); decision } _ => unreachable!(), diff --git a/codex-rs/core/tests/suite/approvals.rs b/codex-rs/core/tests/suite/approvals.rs index d19863dae8..7896455123 100644 --- a/codex-rs/core/tests/suite/approvals.rs +++ b/codex-rs/core/tests/suite/approvals.rs @@ -1992,7 +1992,7 @@ async fn external_exec_approval_handler_approves_without_emitting_prompt() -> Re let mut builder = test_codex().with_home(Arc::clone(&home)).with_config({ move |config| { - config.features.enable(Feature::UnifiedExec); + let _ = config.features.enable(Feature::UnifiedExec); config.permissions.approval_policy = Constrained::allow_any(approval_policy); config.approval_handler = Some(ApprovalHandlerConfig { command: vec![