Set rangeHoverProvider under experimental capabilities

A workaround for microsoft/language-server-protocol#377 until it is resolved this would allow LSP clients to explicitly know that they can send a range with textDocument/hover instead of a position
ayoub-benali · Nov 16, 2021 · f3192aa · f3192aa
1 parent 07a66ff
commit f3192aa
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 0 deletions.
diff --git a/metals/src/main/scala/scala/meta/internal/metals/MetalsExperimental.scala b/metals/src/main/scala/scala/meta/internal/metals/MetalsExperimental.scala
@@ -0,0 +1,11 @@
+package scala.meta.internal.metals
+
+/**
+ * Language Server Protocol extensions that are declared as "server
+ * capabilities" in the initialize response.
+ */
+
+object MetalsExperimental {
+  // A workaround for https://github.com/microsoft/language-server-protocol/issues/377 until it is resolved
+  val rangeHoverProvider: java.lang.Boolean = true
+}
diff --git a/metals/src/main/scala/scala/meta/internal/metals/MetalsLanguageServer.scala b/metals/src/main/scala/scala/meta/internal/metals/MetalsLanguageServer.scala
@@ -839,6 +839,8 @@ class MetalsLanguageServer(
 
         capabilities.setTextDocumentSync(textDocumentSyncOptions)
 
+        capabilities.setExperimental(MetalsExperimental)
+
         val serverInfo = new ServerInfo("Metals", BuildInfo.metalsVersion)
         new InitializeResult(capabilities, serverInfo)
       })