claude: Don't quote executable path in mcp configuration (#34805)

benbrandt · web-flow · commit 57ab09c2dabe · 2025-07-21T09:53:05.000Z
This was generating an invalid string for the configuration, removing
the extra quotes makes it work. This affected the versions of Zed that
have a space in their name.

Release Notes:

- N/A
diff --git a/crates/agent_servers/src/claude/mcp_server.rs b/crates/agent_servers/src/claude/mcp_server.rs
@@ -69,9 +69,6 @@ impl ClaudeMcpServer {
     }
 
     pub fn server_config(&self) -> Result<McpServerConfig> {
-        #[cfg(not(target_os = "windows"))]
-        let zed_path = util::get_shell_safe_zed_path()?;
-        #[cfg(target_os = "windows")]
         let zed_path = std::env::current_exe()
             .context("finding current executable path for use in mcp_server")?
             .to_string_lossy()

Original file line number	Diff line number	Diff line change
`@@ -69,9 +69,6 @@ impl ClaudeMcpServer {`
`69`	`69`	`}`
`70`	`70`
`71`	`71`	`pub fn server_config(&self) -> Result<McpServerConfig> {`
`72`		`- #[cfg(not(target_os = "windows"))]`
`73`		`- let zed_path = util::get_shell_safe_zed_path()?;`
`74`		`- #[cfg(target_os = "windows")]`
`75`	`72`	`let zed_path = std::env::current_exe()`
`76`	`73`	`.context("finding current executable path for use in mcp_server")?`
`77`	`74`	`.to_string_lossy()`