From 75a2bd95b01c8386bda0b8a9b408c416e688d18e Mon Sep 17 00:00:00 2001 From: Eva Wong Date: Tue, 5 May 2026 11:14:23 -0700 Subject: [PATCH] fix(exec): use platform sandbox expectation --- codex-rs/exec/src/lib_tests.rs | 14 ++++++-------- 1 file changed, 6 insertions(+), 8 deletions(-) diff --git a/codex-rs/exec/src/lib_tests.rs b/codex-rs/exec/src/lib_tests.rs index bda14d762b..82314772d0 100644 --- a/codex-rs/exec/src/lib_tests.rs +++ b/codex-rs/exec/src/lib_tests.rs @@ -407,22 +407,20 @@ async fn thread_lifecycle_params_include_legacy_sandbox_when_no_active_profile() .await .expect("build config with legacy sandbox override"); let project_roots = vec![config.cwd.clone(), extra]; + let expected_sandbox = sandbox_mode_from_permission_profile( + &config.permissions.permission_profile(), + config.cwd.as_path(), + ); let start_params = thread_start_params_from_config(&config, Some(&project_roots)); let resume_params = thread_resume_params_from_config(&config, "thread-id".to_string(), Some(&project_roots)); assert_eq!(config.permissions.active_permission_profile(), None); - assert_eq!( - start_params.sandbox, - Some(codex_app_server_protocol::SandboxMode::WorkspaceWrite) - ); + assert_eq!(start_params.sandbox, expected_sandbox); assert_eq!(start_params.permissions, None); assert_eq!(start_params.project_roots, Some(project_roots.clone())); - assert_eq!( - resume_params.sandbox, - Some(codex_app_server_protocol::SandboxMode::WorkspaceWrite) - ); + assert_eq!(resume_params.sandbox, expected_sandbox); assert_eq!(resume_params.permissions, None); assert_eq!(resume_params.project_roots, Some(project_roots)); }