bugfix: Check if inlay hints refresh is enabled

Fixes #6937
scalameta · Nov 16, 2024 · 1104736 · 1104736
1 parent 165c266
commit 1104736
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/metals/src/main/scala/scala/meta/internal/metals/ClientConfiguration.scala b/metals/src/main/scala/scala/meta/internal/metals/ClientConfiguration.scala
@@ -162,6 +162,15 @@ final class ClientConfiguration(
     } yield true
   }.getOrElse(false)
 
+  def isInlayHintsRefreshEnabled(): Boolean = {
+    for {
+      capabilities <- clientCapabilities
+      workspace <- Option(capabilities.getWorkspace())
+      inlayHints <- Option(workspace.getInlayHint())
+      refreshSupport <- Option(inlayHints.getRefreshSupport())
+    } yield refreshSupport.booleanValue()
+  }.getOrElse(false)
+
   def hoverContentType(): ContentType =
     (for {
       capabilities <- clientCapabilities

diff --git a/...src/main/scala/scala/meta/internal/metals/clients/language/ConfiguredLanguageClient.scala b/...src/main/scala/scala/meta/internal/metals/clients/language/ConfiguredLanguageClient.scala
@@ -140,7 +140,10 @@ final class ConfiguredLanguageClient(
   }
 
   override def refreshInlayHints(): CompletableFuture[Void] = {
-    if (clientConfig.isInlayHintsEnabled()) {
+    if (
+      clientConfig.isInlayHintsEnabled() && clientConfig
+        .isInlayHintsRefreshEnabled()
+    ) {
       underlying
         .refreshInlayHints()
         .handle { (msg, ex) =>