From 7c11c14efc693c19f59dadfccc7abcdc35bbaf89 Mon Sep 17 00:00:00 2001 From: starr-openai Date: Thu, 14 May 2026 15:54:18 -0700 Subject: [PATCH] Fix Windows sandbox clippy clones (#22687) ## Summary - remove two redundant `PathBuf` clones in Windows sandbox setup tests - fix current `rust-ci-full` Windows clippy failures on `main` ## Validation - `just fmt` - attempted on `dev`: `cargo clippy --target x86_64-pc-windows-msvc --tests --profile dev --timings -- -D warnings` - blocked by missing MSVC cross toolchain on the Linux devbox (`lib.exe` / MSVC C toolchain unavailable) - live failure evidence: main `rust-ci-full` runs 25880209898 and 25879137967 failed on `windows-sandbox-rs/src/bin/setup_main/win.rs` with `clippy::redundant_clone` at the two edited callsites --- codex-rs/windows-sandbox-rs/src/bin/setup_main/win.rs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/codex-rs/windows-sandbox-rs/src/bin/setup_main/win.rs b/codex-rs/windows-sandbox-rs/src/bin/setup_main/win.rs index 1f93d0a5ee..f0475bc39f 100644 --- a/codex-rs/windows-sandbox-rs/src/bin/setup_main/win.rs +++ b/codex-rs/windows-sandbox-rs/src/bin/setup_main/win.rs @@ -1034,7 +1034,7 @@ mod tests { let deny_sids = workspace_write_cap_sids_for_path( &codex_home, &workspace, - &[workspace.clone(), active_root.clone()], + &[workspace.clone(), active_root], &deny_path, ) .expect("deny sids"); @@ -1070,7 +1070,7 @@ mod tests { let deny_sids = workspace_write_cap_sids_for_path( &codex_home, &workspace, - &[workspace.clone(), active_root.clone()], + &[workspace.clone(), active_root], &deny_path, ) .expect("deny sids");