From 5a6e90599434137b677b7224d995932183844646 Mon Sep 17 00:00:00 2001 From: Eric Traut Date: Thu, 21 May 2026 16:52:36 -0700 Subject: [PATCH] Fix auto-review permission profile override (#23956) ## Summary The auto-review runtime sync path was assigning a raw `PermissionProfile` into `runtime_permission_profile_override`, whose field now expects `RuntimePermissionProfileOverride`. That broke the TUI Bazel build. This changes the assignment to store `RuntimePermissionProfileOverride::from_config(&self.config)`, matching the other runtime override paths and preserving the active profile and network metadata with the permission profile. --- codex-rs/tui/src/app/config_persistence.rs | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/codex-rs/tui/src/app/config_persistence.rs b/codex-rs/tui/src/app/config_persistence.rs index 4baea7fb0..1cba3c85a 100644 --- a/codex-rs/tui/src/app/config_persistence.rs +++ b/codex-rs/tui/src/app/config_persistence.rs @@ -910,7 +910,8 @@ impl App { return; } - self.runtime_permission_profile_override = Some(permission_profile); + self.runtime_permission_profile_override = + Some(RuntimePermissionProfileOverride::from_config(&self.config)); self.sync_active_thread_permission_settings_to_cached_session() .await;