mirror of
https://github.com/openai/codex.git
synced 2026-09-15 12:08:01 +00:00
- Copy latest tui sources into tui2
- Restore notifications, tests, and styles
- Keep codex-tui interop conversions and snapshots
The expected changes that are necessary to make this work are still in
place:
diff -ru codex-rs/tui codex-rs/tui2 --exclude='*.snap'
--exclude='*.snap.new'
```diff
diff -ru --ex codex-rs/tui/Cargo.toml codex-rs/tui2/Cargo.toml
--- codex-rs/tui/Cargo.toml 2025-12-12 16:39:12
+++ codex-rs/tui2/Cargo.toml 2025-12-12 17:31:01
@@ -1,15 +1,15 @@
[package]
-name = "codex-tui"
+name = "codex-tui2"
version.workspace = true
edition.workspace = true
license.workspace = true
[[bin]]
-name = "codex-tui"
+name = "codex-tui2"
path = "src/main.rs"
[lib]
-name = "codex_tui"
+name = "codex_tui2"
path = "src/lib.rs"
[features]
@@ -42,6 +42,7 @@
codex-login = { workspace = true }
codex-protocol = { workspace = true }
codex-utils-absolute-path = { workspace = true }
+codex-tui = { workspace = true }
color-eyre = { workspace = true }
crossterm = { workspace = true, features = ["bracketed-paste", "event-stream"] }
derive_more = { workspace = true, features = ["is_variant"] }
diff -ru --ex codex-rs/tui/src/app.rs codex-rs/tui2/src/app.rs
--- codex-rs/tui/src/app.rs 2025-12-12 16:39:05
+++ codex-rs/tui2/src/app.rs 2025-12-12 17:30:36
@@ -69,6 +69,16 @@
pub update_action: Option<UpdateAction>,
}
+impl From<AppExitInfo> for codex_tui::AppExitInfo {
+ fn from(info: AppExitInfo) -> Self {
+ codex_tui::AppExitInfo {
+ token_usage: info.token_usage,
+ conversation_id: info.conversation_id,
+ update_action: info.update_action.map(Into::into),
+ }
+ }
+}
+
fn session_summary(
token_usage: TokenUsage,
conversation_id: Option<ConversationId>,
Only in codex-rs/tui/src/bin: md-events.rs
Only in codex-rs/tui2/src/bin: md-events2.rs
diff -ru --ex codex-rs/tui/src/cli.rs codex-rs/tui2/src/cli.rs
--- codex-rs/tui/src/cli.rs 2025-11-19 13:40:42
+++ codex-rs/tui2/src/cli.rs 2025-12-12 17:30:43
@@ -88,3 +88,28 @@
#[clap(skip)]
pub config_overrides: CliConfigOverrides,
}
+
+impl From<codex_tui::Cli> for Cli {
+ fn from(cli: codex_tui::Cli) -> Self {
+ Self {
+ prompt: cli.prompt,
+ images: cli.images,
+ resume_picker: cli.resume_picker,
+ resume_last: cli.resume_last,
+ resume_session_id: cli.resume_session_id,
+ resume_show_all: cli.resume_show_all,
+ model: cli.model,
+ oss: cli.oss,
+ oss_provider: cli.oss_provider,
+ config_profile: cli.config_profile,
+ sandbox_mode: cli.sandbox_mode,
+ approval_policy: cli.approval_policy,
+ full_auto: cli.full_auto,
+ dangerously_bypass_approvals_and_sandbox: cli.dangerously_bypass_approvals_and_sandbox,
+ cwd: cli.cwd,
+ web_search: cli.web_search,
+ add_dir: cli.add_dir,
+ config_overrides: cli.config_overrides,
+ }
+ }
+}
diff -ru --ex codex-rs/tui/src/main.rs codex-rs/tui2/src/main.rs
--- codex-rs/tui/src/main.rs 2025-12-12 16:39:05
+++ codex-rs/tui2/src/main.rs 2025-12-12 16:39:06
@@ -1,8 +1,8 @@
use clap::Parser;
use codex_arg0::arg0_dispatch_or_else;
use codex_common::CliConfigOverrides;
-use codex_tui::Cli;
-use codex_tui::run_main;
+use codex_tui2::Cli;
+use codex_tui2::run_main;
#[derive(Parser, Debug)]
struct TopCli {
diff -ru --ex codex-rs/tui/src/update_action.rs codex-rs/tui2/src/update_action.rs
--- codex-rs/tui/src/update_action.rs 2025-11-19 11:11:47
+++ codex-rs/tui2/src/update_action.rs 2025-12-12 17:30:48
@@ -9,6 +9,20 @@
BrewUpgrade,
}
+impl From<UpdateAction> for codex_tui::update_action::UpdateAction {
+ fn from(action: UpdateAction) -> Self {
+ match action {
+ UpdateAction::NpmGlobalLatest => {
+ codex_tui::update_action::UpdateAction::NpmGlobalLatest
+ }
+ UpdateAction::BunGlobalLatest => {
+ codex_tui::update_action::UpdateAction::BunGlobalLatest
+ }
+ UpdateAction::BrewUpgrade => codex_tui::update_action::UpdateAction::BrewUpgrade,
+ }
+ }
+}
+
impl UpdateAction {
/// Returns the list of command-line arguments for invoking the update.
pub fn command_args(self) -> (&'static str, &'static [&'static str]) {
```
140 lines
3.8 KiB
Rust
140 lines
3.8 KiB
Rust
mod osc9;
|
|
mod windows_toast;
|
|
|
|
use std::env;
|
|
use std::io;
|
|
|
|
use codex_core::env::is_wsl;
|
|
use osc9::Osc9Backend;
|
|
use windows_toast::WindowsToastBackend;
|
|
|
|
#[derive(Debug, Clone, Copy, PartialEq, Eq)]
|
|
pub enum NotificationBackendKind {
|
|
Osc9,
|
|
WindowsToast,
|
|
}
|
|
|
|
#[derive(Debug)]
|
|
pub enum DesktopNotificationBackend {
|
|
Osc9(Osc9Backend),
|
|
WindowsToast(WindowsToastBackend),
|
|
}
|
|
|
|
impl DesktopNotificationBackend {
|
|
pub fn osc9() -> Self {
|
|
Self::Osc9(Osc9Backend)
|
|
}
|
|
|
|
pub fn windows_toast() -> Self {
|
|
Self::WindowsToast(WindowsToastBackend::default())
|
|
}
|
|
|
|
pub fn kind(&self) -> NotificationBackendKind {
|
|
match self {
|
|
DesktopNotificationBackend::Osc9(_) => NotificationBackendKind::Osc9,
|
|
DesktopNotificationBackend::WindowsToast(_) => NotificationBackendKind::WindowsToast,
|
|
}
|
|
}
|
|
|
|
pub fn notify(&mut self, message: &str) -> io::Result<()> {
|
|
match self {
|
|
DesktopNotificationBackend::Osc9(backend) => backend.notify(message),
|
|
DesktopNotificationBackend::WindowsToast(backend) => backend.notify(message),
|
|
}
|
|
}
|
|
}
|
|
|
|
pub fn detect_backend() -> DesktopNotificationBackend {
|
|
if should_use_windows_toasts() {
|
|
tracing::info!(
|
|
"Windows Terminal session detected under WSL; using Windows toast notifications"
|
|
);
|
|
DesktopNotificationBackend::windows_toast()
|
|
} else {
|
|
DesktopNotificationBackend::osc9()
|
|
}
|
|
}
|
|
|
|
fn should_use_windows_toasts() -> bool {
|
|
is_wsl() && env::var_os("WT_SESSION").is_some()
|
|
}
|
|
|
|
#[cfg(test)]
|
|
mod tests {
|
|
use super::NotificationBackendKind;
|
|
use super::detect_backend;
|
|
use serial_test::serial;
|
|
use std::ffi::OsString;
|
|
|
|
struct EnvVarGuard {
|
|
key: &'static str,
|
|
original: Option<OsString>,
|
|
}
|
|
|
|
impl EnvVarGuard {
|
|
fn set(key: &'static str, value: &str) -> Self {
|
|
let original = std::env::var_os(key);
|
|
unsafe {
|
|
std::env::set_var(key, value);
|
|
}
|
|
Self { key, original }
|
|
}
|
|
|
|
fn remove(key: &'static str) -> Self {
|
|
let original = std::env::var_os(key);
|
|
unsafe {
|
|
std::env::remove_var(key);
|
|
}
|
|
Self { key, original }
|
|
}
|
|
}
|
|
|
|
impl Drop for EnvVarGuard {
|
|
fn drop(&mut self) {
|
|
unsafe {
|
|
match &self.original {
|
|
Some(value) => std::env::set_var(self.key, value),
|
|
None => std::env::remove_var(self.key),
|
|
}
|
|
}
|
|
}
|
|
}
|
|
|
|
#[test]
|
|
#[serial]
|
|
fn defaults_to_osc9_outside_wsl() {
|
|
let _wsl_guard = EnvVarGuard::remove("WSL_DISTRO_NAME");
|
|
let _wt_guard = EnvVarGuard::remove("WT_SESSION");
|
|
assert_eq!(detect_backend().kind(), NotificationBackendKind::Osc9);
|
|
}
|
|
|
|
#[test]
|
|
#[serial]
|
|
fn waits_for_windows_terminal() {
|
|
let _wsl_guard = EnvVarGuard::set("WSL_DISTRO_NAME", "Ubuntu");
|
|
let _wt_guard = EnvVarGuard::remove("WT_SESSION");
|
|
assert_eq!(detect_backend().kind(), NotificationBackendKind::Osc9);
|
|
}
|
|
|
|
#[cfg(target_os = "linux")]
|
|
#[test]
|
|
#[serial]
|
|
fn selects_windows_toast_in_wsl_windows_terminal() {
|
|
let _wsl_guard = EnvVarGuard::set("WSL_DISTRO_NAME", "Ubuntu");
|
|
let _wt_guard = EnvVarGuard::set("WT_SESSION", "abc");
|
|
assert_eq!(
|
|
detect_backend().kind(),
|
|
NotificationBackendKind::WindowsToast
|
|
);
|
|
}
|
|
|
|
#[cfg(not(target_os = "linux"))]
|
|
#[test]
|
|
#[serial]
|
|
fn stays_on_osc9_outside_linux_even_with_wsl_env() {
|
|
let _wsl_guard = EnvVarGuard::set("WSL_DISTRO_NAME", "Ubuntu");
|
|
let _wt_guard = EnvVarGuard::set("WT_SESSION", "abc");
|
|
assert_eq!(detect_backend().kind(), NotificationBackendKind::Osc9);
|
|
}
|
|
}
|