From eb1c651c00955d31e4aafcda73a89c78c4c9e871 Mon Sep 17 00:00:00 2001 From: Lionel Cheng <60159831+lionelchg@users.noreply.github.com> Date: Wed, 12 Nov 2025 00:59:20 +0100 Subject: [PATCH] Update full-auto description with on-request (#6523) This PR fixes #6522 by correcting the comment for `full-auto` in both `codex-rs/exec/src/cli.rs` and `codex-rs/tui/src/cli.rs` from `-a on-failure` to `-a on-request` to make it coherent with `codex-rs/tui/src/lib.rs:97-105`: ```rust pub async fn run_main( mut cli: Cli, codex_linux_sandbox_exe: Option, ) -> std::io::Result { let (sandbox_mode, approval_policy) = if cli.full_auto { ( Some(SandboxMode::WorkspaceWrite), Some(AskForApproval::OnRequest), ) ``` Running `just codex --help` or `just codex exec --help` should now yield the correct description of `full-auto` CLI argument. Signed-off-by: lionelchg --- codex-rs/exec/src/cli.rs | 2 +- codex-rs/tui/src/cli.rs | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/codex-rs/exec/src/cli.rs b/codex-rs/exec/src/cli.rs index f56d07dc3..065f4826a 100644 --- a/codex-rs/exec/src/cli.rs +++ b/codex-rs/exec/src/cli.rs @@ -30,7 +30,7 @@ pub struct Cli { #[arg(long = "profile", short = 'p')] pub config_profile: Option, - /// Convenience alias for low-friction sandboxed automatic execution (-a on-failure, --sandbox workspace-write). + /// Convenience alias for low-friction sandboxed automatic execution (-a on-request, --sandbox workspace-write). #[arg(long = "full-auto", default_value_t = false)] pub full_auto: bool, diff --git a/codex-rs/tui/src/cli.rs b/codex-rs/tui/src/cli.rs index d86040b5c..5a7e40993 100644 --- a/codex-rs/tui/src/cli.rs +++ b/codex-rs/tui/src/cli.rs @@ -51,7 +51,7 @@ pub struct Cli { #[arg(long = "ask-for-approval", short = 'a')] pub approval_policy: Option, - /// Convenience alias for low-friction sandboxed automatic execution (-a on-failure, --sandbox workspace-write). + /// Convenience alias for low-friction sandboxed automatic execution (-a on-request, --sandbox workspace-write). #[arg(long = "full-auto", default_value_t = false)] pub full_auto: bool,