diff --git a/codex-rs/exec/src/lib.rs b/codex-rs/exec/src/lib.rs index 50196ea377..136f58bed9 100644 --- a/codex-rs/exec/src/lib.rs +++ b/codex-rs/exec/src/lib.rs @@ -959,12 +959,6 @@ fn thread_start_params_from_config(config: &Config) -> ThreadStartParams { fn thread_resume_params_from_config(config: &Config, thread_id: String) -> ThreadResumeParams { let permissions = permissions_selection_from_config(config); - let sandbox = permissions.is_none().then(|| { - sandbox_mode_from_permission_profile( - &config.permissions.permission_profile(), - config.cwd.as_path(), - ) - }); ThreadResumeParams { thread_id, model: config.model.clone(), @@ -973,7 +967,7 @@ fn thread_resume_params_from_config(config: &Config, thread_id: String) -> Threa workspace_roots: Some(config.workspace_roots.clone()), approval_policy: Some(config.permissions.approval_policy.value().into()), approvals_reviewer: approvals_reviewer_override_from_config(config), - sandbox: sandbox.flatten(), + sandbox: None, permissions, config: config_request_overrides_from_config(config), ..ThreadResumeParams::default() diff --git a/codex-rs/exec/src/lib_tests.rs b/codex-rs/exec/src/lib_tests.rs index 89c49e1bf9..9c7d36473e 100644 --- a/codex-rs/exec/src/lib_tests.rs +++ b/codex-rs/exec/src/lib_tests.rs @@ -456,7 +456,7 @@ async fn thread_start_params_include_review_policy_when_auto_review_is_enabled() } #[tokio::test] -async fn thread_lifecycle_params_include_legacy_sandbox_when_no_active_profile() { +async fn thread_lifecycle_params_handle_legacy_sandbox_when_no_active_profile() { let codex_home = tempdir().expect("create temp codex home"); let cwd = tempdir().expect("create temp cwd"); let config = ConfigBuilder::default() @@ -479,10 +479,7 @@ async fn thread_lifecycle_params_include_legacy_sandbox_when_no_active_profile() Some(codex_app_server_protocol::SandboxMode::DangerFullAccess) ); assert_eq!(start_params.permissions, None); - assert_eq!( - resume_params.sandbox, - Some(codex_app_server_protocol::SandboxMode::DangerFullAccess) - ); + assert_eq!(resume_params.sandbox, None); assert_eq!(resume_params.permissions, None); }