From 33a6cc66ab50a93cca6640f9525441b6fd086da2 Mon Sep 17 00:00:00 2001
From: Kyuheon Kim <65669461+kyuheon-kr@users.noreply.github.com>
Date: Fri, 21 Nov 2025 05:32:12 +0900
Subject: [PATCH] fix(cli): correct mcp add usage order (#6827)

## Summary
- add an explicit `override_usage` string to `AddArgs` so clap prints
`<NAME>` before the command/url choice, matching the actual parser and
docs

### Before

Usage: codex mcp add [OPTIONS] <COMMAND|--url <URL>> <NAME>


### After

Usage: codex mcp add [OPTIONS] <NAME> [--url <URL> | -- <COMMAND>...]

---------

Signed-off-by: kyuheon-kr <kyuheon.kr@gmail.com>
---
 codex-rs/cli/src/mcp_cmd.rs | 1 +
 1 file changed, 1 insertion(+)

diff --git a/codex-rs/cli/src/mcp_cmd.rs b/codex-rs/cli/src/mcp_cmd.rs
index ec37c3a6b..93f22e705 100644
--- a/codex-rs/cli/src/mcp_cmd.rs
+++ b/codex-rs/cli/src/mcp_cmd.rs
@@ -79,6 +79,7 @@ pub struct GetArgs {
 }
 
 #[derive(Debug, clap::Parser)]
+#[command(override_usage = "codex mcp add [OPTIONS] <NAME> (--url <URL> | -- <COMMAND>...)")]
 pub struct AddArgs {
     /// Name for the MCP server configuration.
     pub name: String,