From 64c289a9a2017a2b3f1678236c6f7413247b157b Mon Sep 17 00:00:00 2001 From: Richard Feldman Date: Thu, 2 Oct 2025 20:26:22 -0400 Subject: [PATCH] Fix Claude Code login regression (#39413) This was added for Codex, but had undesirable consequences for Claude Code (on Nightly, never made it to Preview). We're going to address this in `codex-acp` instead. Release Notes: - N/A --- crates/agent_ui/src/acp/thread_view.rs | 25 ------------------------- 1 file changed, 25 deletions(-) diff --git a/crates/agent_ui/src/acp/thread_view.rs b/crates/agent_ui/src/acp/thread_view.rs index d025053bc5feb8dbdc1661dec356c20c72d98891..4c1e51a3f3d706b2fb48ffc437db0725976c5aab 100644 --- a/crates/agent_ui/src/acp/thread_view.rs +++ b/crates/agent_ui/src/acp/thread_view.rs @@ -577,31 +577,6 @@ impl AcpThreadView { AgentDiff::set_active_thread(&workspace, thread.clone(), window, cx); - // Proactively surface Authentication Required if the agent advertises auth methods. - if let Some(acp_conn) = thread - .read(cx) - .connection() - .clone() - .downcast::() - { - let methods = acp_conn.auth_methods(); - if !methods.is_empty() { - // Immediately transition to auth-required UI, but defer to avoid re-entrant update. - let err = AuthRequired { - description: None, - provider_id: None, - }; - let this_weak = cx.weak_entity(); - let agent = agent.clone(); - let connection = thread.read(cx).connection().clone(); - window.defer(cx, move |window, cx| { - Self::handle_auth_required( - this_weak, err, agent, connection, window, cx, - ); - }); - } - } - this.model_selector = thread .read(cx) .connection()