From 32b062d0e190943af7463f0f1aa257ce4977008a Mon Sep 17 00:00:00 2001 From: JBallin Date: Tue, 27 Jan 2026 02:21:09 +0200 Subject: [PATCH] fix: use `brew upgrade --cask codex` to avoid warnings and ambiguity (#9823) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Fixes #9822 ### Summary Make the Homebrew upgrade command explicit by using `brew upgrade --cask codex`. ### Motivation During the Codex self-update, Homebrew can emit an avoidable warning because the name `codex` resolves to a cask: ``` Warning: Formula codex was renamed to homebrew/cask/codex. ```` While the upgrade succeeds, this relies on implicit name resolution and produces unnecessary output during the update flow. ### Why `--cask` * Eliminates warning/noise for users * Explicitly matches how Codex is distributed via Homebrew * Avoids reliance on name resolution behavior * Makes the command more robust if a `codex` formula is ever introduced ### Context This restores the `--cask` flag that was removed in #6238 after being considered “not necessary” during review: [https://github.com/openai/codex/pull/6238#discussion_r2505947880](https://github.com/openai/codex/pull/6238#discussion_r2505947880). Co-authored-by: Eric Traut --- codex-rs/tui/src/update_action.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/codex-rs/tui/src/update_action.rs b/codex-rs/tui/src/update_action.rs index b5cf56a6b..0bb6fa2ca 100644 --- a/codex-rs/tui/src/update_action.rs +++ b/codex-rs/tui/src/update_action.rs @@ -15,7 +15,7 @@ impl UpdateAction { match self { UpdateAction::NpmGlobalLatest => ("npm", &["install", "-g", "@openai/codex"]), UpdateAction::BunGlobalLatest => ("bun", &["install", "-g", "@openai/codex"]), - UpdateAction::BrewUpgrade => ("brew", &["upgrade", "codex"]), + UpdateAction::BrewUpgrade => ("brew", &["upgrade", "--cask", "codex"]), } }