From 17d03b90b4c0dd6b0a1077dee7d697d904b01f60 Mon Sep 17 00:00:00 2001
From: tris203 <admin@snappeh.com>
Date: Thu, 11 Dec 2025 22:33:58 +0000
Subject: [PATCH] fix(lsp): handle dynamic registration for didSave

Update server capabilities to set `save` to `None` when the client supports
dynamic registration for `didSaveTextDocument`. This prevents redundant
static registration and aligns with LSP specification.
---
 crates/rust-analyzer/src/lsp/capabilities.rs | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/crates/rust-analyzer/src/lsp/capabilities.rs b/crates/rust-analyzer/src/lsp/capabilities.rs
index f94e7486ff..d6a694be91 100644
--- a/crates/rust-analyzer/src/lsp/capabilities.rs
+++ b/crates/rust-analyzer/src/lsp/capabilities.rs
@@ -37,7 +37,11 @@ pub fn server_capabilities(config: &Config) -> ServerCapabilities {
             change: Some(TextDocumentSyncKind::INCREMENTAL),
             will_save: None,
             will_save_wait_until: None,
-            save: Some(SaveOptions::default().into()),
+            save: if config.caps().did_save_text_document_dynamic_registration() {
+                None
+            } else {
+                Some(SaveOptions::default().into())
+            },
         })),
         hover_provider: Some(HoverProviderCapability::Simple(true)),
         completion_provider: Some(CompletionOptions {