From bcf2b559577285ea0fda54b142f30d1d718b9254 Mon Sep 17 00:00:00 2001 From: Michael Bolin Date: Thu, 28 May 2026 15:51:27 -0700 Subject: [PATCH] windows-sandbox: fix capture cancellation test roots (#24974) ## Why The Windows Bazel job on `main` started failing after #24108 because one Windows-only capture test still passed `cwd.as_path()` to `run_windows_sandbox_capture`. That helper now expects the explicit `workspace_roots` slice introduced by #24108, so the Windows test target no longer compiled. ## What Changed - Updates `legacy_capture_cancellation_is_not_reported_as_timeout` to pass `workspace_roots_for(cwd.as_path()).as_slice()`, matching the adjacent capture test and the new runner signature. ## Verification - GitHub Actions CI is the important validation for this Windows-only compile path. - Created quickly to get Windows CI running while the separate Ubuntu `compact_resume_fork` timeout is still under investigation. --- codex-rs/windows-sandbox-rs/src/unified_exec/tests.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/codex-rs/windows-sandbox-rs/src/unified_exec/tests.rs b/codex-rs/windows-sandbox-rs/src/unified_exec/tests.rs index 280faf3d28..96f5c39e92 100644 --- a/codex-rs/windows-sandbox-rs/src/unified_exec/tests.rs +++ b/codex-rs/windows-sandbox-rs/src/unified_exec/tests.rs @@ -477,7 +477,7 @@ fn legacy_capture_cancellation_is_not_reported_as_timeout() { let started_at = Instant::now(); let result = run_windows_sandbox_capture( &permission_profile, - cwd.as_path(), + workspace_roots_for(cwd.as_path()).as_slice(), codex_home.path(), vec![ pwsh.display().to_string(),