diff --git a/codex-rs/tui/src/app_server_session.rs b/codex-rs/tui/src/app_server_session.rs index 82c37d1610..93d671fbcf 100644 --- a/codex-rs/tui/src/app_server_session.rs +++ b/codex-rs/tui/src/app_server_session.rs @@ -1874,8 +1874,8 @@ mod tests { .to_legacy_sandbox_policy(cwd.as_path()) .expect("read-only profile must be legacy-compatible") .into(); - let split_profile = PermissionProfile::Managed { - file_system: ManagedFileSystemPermissions::Restricted { + let response_profile = AppServerPermissionProfile::Managed { + file_system: PermissionProfileFileSystemPermissions::Restricted { entries: vec![ FileSystemSandboxEntry { path: FileSystemPath::Special { @@ -1894,9 +1894,9 @@ mod tests { ], glob_scan_max_depth: None, }, - network: NetworkSandboxPolicy::Restricted, + network: PermissionProfileNetworkPermissions { enabled: false }, }; - let response_profile = split_profile.clone().into(); + let split_profile: PermissionProfile = response_profile.clone().into(); assert_eq!( permission_profile_from_thread_response( diff --git a/codex-rs/tui/src/status/tests.rs b/codex-rs/tui/src/status/tests.rs index a80b96669a..ae84315b4b 100644 --- a/codex-rs/tui/src/status/tests.rs +++ b/codex-rs/tui/src/status/tests.rs @@ -29,9 +29,9 @@ use codex_protocol::config_types::ApprovalsReviewer; use codex_protocol::config_types::ReasoningSummary; use codex_protocol::models::ActivePermissionProfile; use codex_protocol::models::ActivePermissionProfileModification; -use codex_protocol::models::ManagedFileSystemPermissions; use codex_protocol::models::PermissionProfile; use codex_protocol::openai_models::ReasoningEffort; +use codex_protocol::permissions::NetworkSandboxPolicy; use insta::assert_snapshot; use pretty_assertions::assert_eq; use ratatui::prelude::*; @@ -287,7 +287,7 @@ async fn status_permissions_named_read_only_profile_shows_builtin_label() { config .permissions .approval_policy - .set(AskForApproval::OnRequest) + .set(AskForApproval::OnRequest.to_core()) .expect("set approval policy"); config .permissions @@ -310,7 +310,7 @@ async fn status_permissions_read_only_profile_shows_additional_writable_roots() config .permissions .approval_policy - .set(AskForApproval::OnRequest) + .set(AskForApproval::OnRequest.to_core()) .expect("set approval policy"); let extra_root = test_path_buf("/workspace/extra").abs(); let file_system_policy = PermissionProfile::read_only() @@ -346,7 +346,7 @@ async fn status_permissions_named_workspace_profile_shows_builtin_label() { config .permissions .approval_policy - .set(AskForApproval::OnRequest) + .set(AskForApproval::OnRequest.to_core()) .expect("set approval policy"); config .permissions @@ -370,7 +370,7 @@ async fn status_permissions_workspace_auto_review_shows_reviewer_label() { config .permissions .approval_policy - .set(AskForApproval::OnRequest) + .set(AskForApproval::OnRequest.to_core()) .expect("set approval policy"); config .permissions @@ -393,7 +393,7 @@ async fn status_permissions_named_profile_shows_additional_writable_roots() { config .permissions .approval_policy - .set(AskForApproval::OnRequest) + .set(AskForApproval::OnRequest.to_core()) .expect("set approval policy"); let extra_root = test_path_buf("/workspace/extra").abs(); config @@ -428,7 +428,7 @@ async fn status_permissions_broadened_workspace_profile_shows_builtin_label() { config .permissions .approval_policy - .set(AskForApproval::OnRequest) + .set(AskForApproval::OnRequest.to_core()) .expect("set approval policy"); config .permissions