Fix MCP lint regressions

Co-authored-by: Codex <noreply@openai.com>

Author: caseychow-oai

codex-rs/core/src/codex_delegate.rs

@@ -669,7 +669,7 @@ async fn maybe_auto_review_mcp_request_user_input(
         parent_session.as_ref(),
         parent_ctx.as_ref(),
         &invocation.tool,
-        None,
+        /*qualified_tool_namespace*/ None,
         &invocation.server,
         &invocation.tool,
     )

codex-rs/core/src/mcp_tool_call.rs

@@ -60,14 +60,18 @@ use std::path::Path;
 use std::sync::Arc;
 use toml_edit::value;
 
+pub(crate) struct QualifiedMcpTool<'a> {
+    pub(crate) name: &'a str,
+    pub(crate) namespace: Option<&'a str>,
+}
+
 /// Handles the specified tool call dispatches the appropriate
 /// `McpToolCallBegin` and `McpToolCallEnd` events to the `Session`.
 pub(crate) async fn handle_mcp_tool_call(
     sess: Arc<Session>,
     turn_context: &Arc<TurnContext>,
     call_id: String,
-    qualified_tool_name: &str,
-    qualified_tool_namespace: Option<&str>,
+    qualified_tool: QualifiedMcpTool<'_>,
     server: String,
     tool_name: String,
     arguments: String,
@@ -95,8 +99,8 @@ pub(crate) async fn handle_mcp_tool_call(
     let metadata = lookup_mcp_tool_metadata(
         sess.as_ref(),
         turn_context.as_ref(),
-        qualified_tool_name,
-        qualified_tool_namespace,
+        qualified_tool.name,
+        qualified_tool.namespace,
         &server,
         &tool_name,
     )

codex-rs/core/src/tools/handlers/mcp.rs

@@ -2,6 +2,7 @@ use async_trait::async_trait;
 use std::sync::Arc;
 
 use crate::function_tool::FunctionCallError;
+use crate::mcp_tool_call::QualifiedMcpTool;
 use crate::mcp_tool_call::handle_mcp_tool_call;
 use crate::tools::context::ToolInvocation;
 use crate::tools::context::ToolPayload;
@@ -49,8 +50,10 @@ impl ToolHandler for McpHandler {
             Arc::clone(&session),
             &turn,
             call_id.clone(),
-            &tool_name,
-            tool_namespace.as_deref(),
+            QualifiedMcpTool {
+                name: &tool_name,
+                namespace: tool_namespace.as_deref(),
+            },
             server,
             tool,
             arguments_str,