From d1ad017629ea63eb2f6e4e43fdc75920cc5781dd Mon Sep 17 00:00:00 2001 From: colby-oai <228809017+colby-oai@users.noreply.github.com> Date: Sat, 28 Feb 2026 11:47:37 -0500 Subject: [PATCH] Remove tool name parenthesis --- codex-rs/core/src/mcp_tool_call.rs | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/codex-rs/core/src/mcp_tool_call.rs b/codex-rs/core/src/mcp_tool_call.rs index 0d729ca2c1..1c9cb3c9f7 100644 --- a/codex-rs/core/src/mcp_tool_call.rs +++ b/codex-rs/core/src/mcp_tool_call.rs @@ -558,9 +558,6 @@ fn build_mcp_tool_approval_header( fn format_mcp_tool_label(tool_name: &str, tool_title: Option<&str>) -> String { match tool_title { - Some(tool_title) if tool_title != tool_name => { - format!("\"{tool_title}\" ({tool_name})") - } Some(tool_title) => format!("\"{tool_title}\""), None => format!("\"{tool_name}\""), } @@ -811,7 +808,7 @@ mod tests { assert_eq!(question.header, "Approve custom_server: Run Action?"); assert_eq!( question.question, - "The custom_server MCP server wants to run the tool \"Run Action\" (run_action), which may modify or delete data.\n\nAllow this action?" + "The custom_server MCP server wants to run the tool \"Run Action\", which may modify or delete data.\n\nAllow this action?" ); assert!( question @@ -875,7 +872,7 @@ mod tests { assert_eq!(question.header, "Approve Linear: Create Issue?"); assert_eq!( question.question, - "The Linear app wants to run the tool \"Create Issue\" (create_issue), which may modify data and access external systems.\n\nTool call details:\n- projectId: \"proj_123\"\n- title: \"Approval prompt follow-up\"\n- body: \"Draft email body\"\n- description: \"Audit approval prompt copy\"\n\nAllow this action?" + "The Linear app wants to run the tool \"Create Issue\", which may modify data and access external systems.\n\nTool call details:\n- projectId: \"proj_123\"\n- title: \"Approval prompt follow-up\"\n- body: \"Draft email body\"\n- description: \"Audit approval prompt copy\"\n\nAllow this action?" ); }