From a67d0dc430bd22c51cd643d627727bb561906e7e Mon Sep 17 00:00:00 2001 From: Matthew Zeng Date: Fri, 1 May 2026 16:21:56 -0700 Subject: [PATCH] update --- codex-rs/core/src/mcp_tool_call.rs | 2 -- codex-rs/core/src/mcp_tool_call_tests.rs | 4 ---- 2 files changed, 6 deletions(-) diff --git a/codex-rs/core/src/mcp_tool_call.rs b/codex-rs/core/src/mcp_tool_call.rs index 4a3363bae9..44182265f8 100644 --- a/codex-rs/core/src/mcp_tool_call.rs +++ b/codex-rs/core/src/mcp_tool_call.rs @@ -1548,7 +1548,6 @@ fn build_mcp_tool_approval_elicitation_request( server_name: request.server.to_string(), request: McpServerElicitationRequest::Form { meta: build_mcp_tool_approval_elicitation_meta( - request.server, request.is_host_owned_codex_apps_server, request.metadata, request.tool_params, @@ -1567,7 +1566,6 @@ fn build_mcp_tool_approval_elicitation_request( } fn build_mcp_tool_approval_elicitation_meta( - _server: &str, is_host_owned_codex_apps_server: bool, metadata: Option<&McpToolApprovalMetadata>, tool_params: Option<&serde_json::Value>, diff --git a/codex-rs/core/src/mcp_tool_call_tests.rs b/codex-rs/core/src/mcp_tool_call_tests.rs index bb100ca528..bf137bb821 100644 --- a/codex-rs/core/src/mcp_tool_call_tests.rs +++ b/codex-rs/core/src/mcp_tool_call_tests.rs @@ -1135,7 +1135,6 @@ fn accepted_elicitation_content_converts_to_request_user_input_response() { fn approval_elicitation_meta_marks_tool_approvals() { assert_eq!( build_mcp_tool_approval_elicitation_meta( - "custom_server", /*is_host_owned_codex_apps_server*/ false, /*metadata*/ None, /*tool_params*/ None, @@ -1154,7 +1153,6 @@ fn approval_elicitation_meta_marks_tool_approvals() { fn approval_elicitation_meta_merges_session_and_always_persist_for_custom_servers() { assert_eq!( build_mcp_tool_approval_elicitation_meta( - "custom_server", /*is_host_owned_codex_apps_server*/ false, Some(&approval_metadata( /*connector_id*/ None, @@ -1367,7 +1365,6 @@ async fn guardian_review_decision_maps_to_mcp_tool_decision() { fn approval_elicitation_meta_includes_connector_source_for_codex_apps() { assert_eq!( build_mcp_tool_approval_elicitation_meta( - CODEX_APPS_MCP_SERVER_NAME, /*is_host_owned_codex_apps_server*/ true, Some(&approval_metadata( Some("calendar"), @@ -1403,7 +1400,6 @@ fn approval_elicitation_meta_includes_connector_source_for_codex_apps() { fn approval_elicitation_meta_merges_session_and_always_persist_with_connector_source() { assert_eq!( build_mcp_tool_approval_elicitation_meta( - CODEX_APPS_MCP_SERVER_NAME, /*is_host_owned_codex_apps_server*/ true, Some(&approval_metadata( Some("calendar"),