clojure: Don't cache user-installed `clojure-lsp` (#10636)

Marshall Bowers created 2 years ago

This PR updates the Clojure extension to not cache the binary when it is
using the one on the $PATH.

Release Notes:

- N/A

Change summary

extensions/clojure/src/clojure.rs | 9 ++++-----
1 file changed, 4 insertions(+), 5 deletions(-)

Detailed changes

extensions/clojure/src/clojure.rs

@@ -11,17 +11,16 @@ impl ClojureExtension {
         config: zed::LanguageServerConfig,
         worktree: &zed::Worktree,
     ) -> Result<String> {
+        if let Some(path) = worktree.which("clojure-lsp") {
+            return Ok(path);
+        }
+
         if let Some(path) = &self.cached_binary_path {
             if fs::metadata(path).map_or(false, |stat| stat.is_file()) {
                 return Ok(path.clone());
             }
         }
 
-        if let Some(path) = worktree.which("clojure-lsp") {
-            self.cached_binary_path = Some(path.clone());
-            return Ok(path);
-        }
-
         zed::set_language_server_installation_status(
             &config.name,
             &zed::LanguageServerInstallationStatus::CheckingForUpdate,