go: Do not fill out root_uri in initialization params to prevent stale notifications (#25644)

Piotr Osiewicz created 10 months ago

Closes #25381

Release Notes:

- N/A

Change summary

crates/languages/src/go.rs | 10 +++++++++-
1 file changed, 9 insertions(+), 1 deletion(-)

Detailed changes

crates/languages/src/go.rs

@@ -5,7 +5,7 @@ use futures::StreamExt;
 use gpui::{App, AsyncApp, Task};
 use http_client::github::latest_github_release;
 pub use language::*;
-use lsp::{LanguageServerBinary, LanguageServerName};
+use lsp::{InitializeParams, LanguageServerBinary, LanguageServerName};
 use project::Fs;
 use regex::Regex;
 use serde_json::json;
@@ -373,6 +373,14 @@ impl super::LspAdapter for GoLspAdapter {
             filter_range,
         })
     }
+    fn prepare_initialize_params(
+        &self,
+        mut original: InitializeParams,
+    ) -> Result<InitializeParams> {
+        #[allow(deprecated)]
+        let _ = original.root_uri.take();
+        Ok(original)
+    }
 }
 
 fn parse_version_output(output: &Output) -> Result<&str> {