clone/codex
1
0
Fork 0
mirror of https://github.com/openai/codex.git synced 2026-06-01 19:02:59 +00:00
Code Issues Packages Projects Releases Wiki Activity

protocol: remove legacy sandbox turn-context overrides

Browse Source
This commit is contained in:
Michael Bolin
2026-04-30 08:49:40 -07:00
parent dd07fb5a2a
commit d507eb9ce0
23 changed files with 70 additions and 127 deletions
Download Patch File Download Diff File Expand all files Collapse all files

26
codex-rs/core/tests/suite/collaboration_instructions.rs
View File

@@ -128,7 +128,7 @@ async fn user_input_includes_collaboration_instructions_after_override() -> Resu
cwd: None,
approval_policy: None,
approvals_reviewer: None,
sandbox_policy: None,
clear_active_permission_profile: false,
permission_profile: None,
windows_sandbox_level: None,
model: None,
@@ -234,7 +234,7 @@ async fn override_then_next_turn_uses_updated_collaboration_instructions() -> Re
cwd: None,
approval_policy: None,
approvals_reviewer: None,
sandbox_policy: None,
clear_active_permission_profile: false,
permission_profile: None,
windows_sandbox_level: None,
model: None,
@@ -293,7 +293,7 @@ async fn user_turn_overrides_collaboration_instructions_after_override() -> Resu
cwd: None,
approval_policy: None,
approvals_reviewer: None,
sandbox_policy: None,
clear_active_permission_profile: false,
permission_profile: None,
windows_sandbox_level: None,
model: None,
@@ -366,7 +366,7 @@ async fn collaboration_mode_update_emits_new_instruction_message() -> Result<()>
cwd: None,
approval_policy: None,
approvals_reviewer: None,
sandbox_policy: None,
clear_active_permission_profile: false,
permission_profile: None,
windows_sandbox_level: None,
model: None,
@@ -396,7 +396,7 @@ async fn collaboration_mode_update_emits_new_instruction_message() -> Result<()>
cwd: None,
approval_policy: None,
approvals_reviewer: None,
sandbox_policy: None,
clear_active_permission_profile: false,
permission_profile: None,
windows_sandbox_level: None,
model: None,
@@ -455,7 +455,7 @@ async fn collaboration_mode_update_noop_does_not_append() -> Result<()> {
cwd: None,
approval_policy: None,
approvals_reviewer: None,
sandbox_policy: None,
clear_active_permission_profile: false,
permission_profile: None,
windows_sandbox_level: None,
model: None,
@@ -485,7 +485,7 @@ async fn collaboration_mode_update_noop_does_not_append() -> Result<()> {
cwd: None,
approval_policy: None,
approvals_reviewer: None,
sandbox_policy: None,
clear_active_permission_profile: false,
permission_profile: None,
windows_sandbox_level: None,
model: None,
@@ -543,7 +543,7 @@ async fn collaboration_mode_update_emits_new_instruction_message_when_mode_chang
cwd: None,
approval_policy: None,
approvals_reviewer: None,
sandbox_policy: None,
clear_active_permission_profile: false,
permission_profile: None,
windows_sandbox_level: None,
model: None,
@@ -576,7 +576,7 @@ async fn collaboration_mode_update_emits_new_instruction_message_when_mode_chang
cwd: None,
approval_policy: None,
approvals_reviewer: None,
sandbox_policy: None,
clear_active_permission_profile: false,
permission_profile: None,
windows_sandbox_level: None,
model: None,
@@ -638,7 +638,7 @@ async fn collaboration_mode_update_noop_does_not_append_when_mode_is_unchanged()
cwd: None,
approval_policy: None,
approvals_reviewer: None,
sandbox_policy: None,
clear_active_permission_profile: false,
permission_profile: None,
windows_sandbox_level: None,
model: None,
@@ -671,7 +671,7 @@ async fn collaboration_mode_update_noop_does_not_append_when_mode_is_unchanged()
cwd: None,
approval_policy: None,
approvals_reviewer: None,
sandbox_policy: None,
clear_active_permission_profile: false,
permission_profile: None,
windows_sandbox_level: None,
model: None,
@@ -739,7 +739,7 @@ async fn resume_replays_collaboration_instructions() -> Result<()> {
cwd: None,
approval_policy: None,
approvals_reviewer: None,
sandbox_policy: None,
clear_active_permission_profile: false,
permission_profile: None,
windows_sandbox_level: None,
model: None,
@@ -807,7 +807,7 @@ async fn empty_collaboration_instructions_are_ignored() -> Result<()> {
cwd: None,
approval_policy: None,
approvals_reviewer: None,
sandbox_policy: None,
clear_active_permission_profile: false,
permission_profile: None,
windows_sandbox_level: None,
model: None,

2
codex-rs/core/tests/suite/compact.rs
View File

@@ -3037,7 +3037,7 @@ async fn snapshot_request_shape_pre_turn_compaction_including_incoming_user_mess
cwd: Some(PathBuf::from(PRETURN_CONTEXT_DIFF_CWD)),
approval_policy: None,
approvals_reviewer: None,
sandbox_policy: None,
clear_active_permission_profile: false,
permission_profile: None,
windows_sandbox_level: None,
model: None,

4
codex-rs/core/tests/suite/compact_remote.rs
View File

@@ -2243,7 +2243,7 @@ async fn snapshot_request_shape_remote_pre_turn_compaction_including_incoming_us
cwd: Some(PathBuf::from(PRETURN_CONTEXT_DIFF_CWD)),
approval_policy: None,
approvals_reviewer: None,
sandbox_policy: None,
clear_active_permission_profile: false,
permission_profile: None,
windows_sandbox_level: None,
model: None,
@@ -2359,7 +2359,7 @@ async fn snapshot_request_shape_remote_pre_turn_compaction_strips_incoming_model
cwd: None,
approval_policy: None,
approvals_reviewer: None,
sandbox_policy: None,
clear_active_permission_profile: false,
permission_profile: None,
windows_sandbox_level: None,
model: Some(next_model.to_string()),

2
codex-rs/core/tests/suite/compact_resume_fork.rs
View File

@@ -549,7 +549,7 @@ async fn snapshot_rollback_followup_turn_trims_context_updates() -> Result<()> {
cwd: Some(override_cwd.to_path_buf()),
approval_policy: None,
approvals_reviewer: None,
sandbox_policy: None,
clear_active_permission_profile: false,
permission_profile: None,
windows_sandbox_level: None,
model: None,

4
codex-rs/core/tests/suite/model_overrides.rs
View File

@@ -29,7 +29,7 @@ async fn override_turn_context_does_not_persist_when_config_exists() {
cwd: None,
approval_policy: None,
approvals_reviewer: None,
sandbox_policy: None,
clear_active_permission_profile: false,
permission_profile: None,
windows_sandbox_level: None,
model: Some("o3".to_string()),
@@ -68,7 +68,7 @@ async fn override_turn_context_does_not_create_config_file() {
cwd: None,
approval_policy: None,
approvals_reviewer: None,
sandbox_policy: None,
clear_active_permission_profile: false,
permission_profile: None,
windows_sandbox_level: None,
model: Some("o3".to_string()),

6
codex-rs/core/tests/suite/model_switching.rs
View File

@@ -158,7 +158,7 @@ async fn model_change_appends_model_instructions_developer_message() -> Result<(
cwd: None,
approval_policy: None,
approvals_reviewer: None,
sandbox_policy: None,
clear_active_permission_profile: false,
permission_profile: None,
windows_sandbox_level: None,
model: Some(next_model.to_string()),
@@ -238,7 +238,7 @@ async fn model_and_personality_change_only_appends_model_instructions() -> Resul
cwd: None,
approval_policy: None,
approvals_reviewer: None,
sandbox_policy: None,
clear_active_permission_profile: false,
permission_profile: None,
windows_sandbox_level: None,
model: Some(next_model.to_string()),
@@ -931,7 +931,7 @@ async fn model_switch_to_smaller_model_updates_token_context_window() -> Result<
cwd: None,
approval_policy: None,
approvals_reviewer: None,
sandbox_policy: None,
clear_active_permission_profile: false,
permission_profile: None,
windows_sandbox_level: None,
model: Some(smaller_model_slug.to_string()),

2
codex-rs/core/tests/suite/model_visible_layout.rs
View File

@@ -473,7 +473,7 @@ async fn snapshot_model_visible_layout_resume_override_matches_rollout_model() -
cwd: Some(resume_override_cwd),
approval_policy: None,
approvals_reviewer: None,
sandbox_policy: None,
clear_active_permission_profile: false,
permission_profile: None,
windows_sandbox_level: None,
model: Some("gpt-5.2".to_string()),

6
codex-rs/core/tests/suite/override_updates.rs
View File

@@ -117,7 +117,7 @@ async fn override_turn_context_without_user_turn_does_not_record_permissions_upd
cwd: None,
approval_policy: Some(AskForApproval::Never),
approvals_reviewer: None,
sandbox_policy: None,
clear_active_permission_profile: false,
permission_profile: None,
windows_sandbox_level: None,
model: None,
@@ -160,7 +160,7 @@ async fn override_turn_context_without_user_turn_does_not_record_environment_upd
cwd: Some(new_cwd.path().to_path_buf()),
approval_policy: None,
approvals_reviewer: None,
sandbox_policy: None,
clear_active_permission_profile: false,
permission_profile: None,
windows_sandbox_level: None,
model: None,
@@ -200,7 +200,7 @@ async fn override_turn_context_without_user_turn_does_not_record_collaboration_u
cwd: None,
approval_policy: None,
approvals_reviewer: None,
sandbox_policy: None,
clear_active_permission_profile: false,
permission_profile: None,
windows_sandbox_level: None,
model: None,

8
codex-rs/core/tests/suite/permissions_messages.rs
View File

@@ -106,7 +106,7 @@ async fn permissions_message_added_on_override_change() -> Result<()> {
cwd: None,
approval_policy: Some(AskForApproval::Never),
approvals_reviewer: None,
sandbox_policy: None,
clear_active_permission_profile: false,
permission_profile: None,
windows_sandbox_level: None,
model: None,
@@ -239,7 +239,7 @@ async fn permissions_message_omitted_when_disabled() -> Result<()> {
cwd: None,
approval_policy: Some(AskForApproval::Never),
approvals_reviewer: None,
sandbox_policy: None,
clear_active_permission_profile: false,
permission_profile: None,
windows_sandbox_level: None,
model: None,
@@ -328,7 +328,7 @@ async fn resume_replays_permissions_messages() -> Result<()> {
cwd: None,
approval_policy: Some(AskForApproval::Never),
approvals_reviewer: None,
sandbox_policy: None,
clear_active_permission_profile: false,
permission_profile: None,
windows_sandbox_level: None,
model: None,
@@ -434,7 +434,7 @@ async fn resume_and_fork_append_permissions_messages() -> Result<()> {
cwd: None,
approval_policy: Some(AskForApproval::Never),
approvals_reviewer: None,
sandbox_policy: None,
clear_active_permission_profile: false,
permission_profile: None,
windows_sandbox_level: None,
model: None,

8
codex-rs/core/tests/suite/personality.rs
View File

@@ -331,7 +331,7 @@ async fn user_turn_personality_some_adds_update_message() -> anyhow::Result<()>
cwd: None,
approval_policy: None,
approvals_reviewer: None,
sandbox_policy: None,
clear_active_permission_profile: false,
permission_profile: None,
windows_sandbox_level: None,
model: None,
@@ -415,7 +415,7 @@ async fn user_turn_personality_same_value_does_not_add_update_message() -> anyho
cwd: None,
approval_policy: None,
approvals_reviewer: None,
sandbox_policy: None,
clear_active_permission_profile: false,
permission_profile: None,
windows_sandbox_level: None,
model: None,
@@ -512,7 +512,7 @@ async fn user_turn_personality_skips_if_feature_disabled() -> anyhow::Result<()>
cwd: None,
approval_policy: None,
approvals_reviewer: None,
sandbox_policy: None,
clear_active_permission_profile: false,
permission_profile: None,
windows_sandbox_level: None,
model: None,
@@ -759,7 +759,7 @@ async fn user_turn_personality_remote_model_template_includes_update_message() -
cwd: None,
approval_policy: None,
approvals_reviewer: None,
sandbox_policy: None,
clear_active_permission_profile: false,
permission_profile: None,
windows_sandbox_level: None,
model: None,

4
codex-rs/core/tests/suite/prompt_caching.rs
View File

@@ -445,7 +445,7 @@ async fn overrides_turn_context_but_keeps_cached_prefix_and_key_constant() -> an
cwd: None,
approval_policy: Some(AskForApproval::Never),
approvals_reviewer: None,
sandbox_policy: None,
clear_active_permission_profile: false,
permission_profile: Some(permission_profile),
windows_sandbox_level: None,
model: None,
@@ -531,7 +531,7 @@ async fn override_before_first_turn_emits_environment_context() -> anyhow::Resul
cwd: None,
approval_policy: Some(AskForApproval::Never),
approvals_reviewer: None,
sandbox_policy: None,
clear_active_permission_profile: false,
permission_profile: None,
windows_sandbox_level: None,
model: Some("gpt-5.4".to_string()),

4
codex-rs/core/tests/suite/remote_models.rs
View File

@@ -602,7 +602,7 @@ async fn remote_models_remote_model_uses_unified_exec() -> Result<()> {
cwd: None,
approval_policy: None,
approvals_reviewer: None,
sandbox_policy: None,
clear_active_permission_profile: false,
permission_profile: None,
windows_sandbox_level: None,
model: Some(REMOTE_MODEL_SLUG.to_string()),
@@ -851,7 +851,7 @@ async fn remote_models_apply_remote_base_instructions() -> Result<()> {
cwd: None,
approval_policy: None,
approvals_reviewer: None,
sandbox_policy: None,
clear_active_permission_profile: false,
permission_profile: None,
windows_sandbox_level: None,
model: Some(model.to_string()),

2
codex-rs/core/tests/suite/resume.rs
View File

@@ -427,7 +427,7 @@ async fn resume_model_switch_is_not_duplicated_after_pre_turn_override() -> Resu
cwd: None,
approval_policy: None,
approvals_reviewer: None,
sandbox_policy: None,
clear_active_permission_profile: false,
permission_profile: None,
windows_sandbox_level: None,
model: Some("gpt-5.4".to_string()),

2
codex-rs/core/tests/suite/review.rs
View File

@@ -836,7 +836,7 @@ async fn review_uses_overridden_cwd_for_base_branch_merge_base() {
cwd: Some(repo_path.to_path_buf()),
approval_policy: None,
approvals_reviewer: None,
sandbox_policy: None,
clear_active_permission_profile: false,
permission_profile: None,
windows_sandbox_level: None,
model: None,