Feature/disable hover support (#124)

* disable hover related tests

* updates version to 4.4.0

Author: raleigh-g-thompson

core/pom.xml

@@ -12,7 +12,7 @@
     <parent>
         <groupId>org.opencds.cqf.cql.ls</groupId>
         <artifactId>cql-ls</artifactId>
-        <version>4.4.0-SNAPSHOT</version>
+        <version>4.4.0</version>
         <relativePath>../pom.xml</relativePath>
     </parent>
 

debug/server/pom.xml

@@ -12,7 +12,7 @@
     <parent>
         <groupId>org.opencds.cqf.cql.ls</groupId>
         <artifactId>cql-ls</artifactId>
-        <version>4.4.0-SNAPSHOT</version>
+        <version>4.4.0</version>
         <relativePath>../../pom.xml</relativePath>
     </parent>
 

debug/service/pom.xml

@@ -12,7 +12,7 @@
     <parent>
         <groupId>org.opencds.cqf.cql.ls</groupId>
         <artifactId>cql-ls</artifactId>
-        <version>4.4.0-SNAPSHOT</version>
+        <version>4.4.0</version>
         <relativePath>../../pom.xml</relativePath>
     </parent>
 

ls/server/pom.xml

@@ -12,7 +12,7 @@
     <parent>
         <groupId>org.opencds.cqf.cql.ls</groupId>
         <artifactId>cql-ls</artifactId>
-        <version>4.4.0-SNAPSHOT</version>
+        <version>4.4.0</version>
         <relativePath>../../pom.xml</relativePath>
     </parent>
 

ls/server/src/main/java/org/opencds/cqf/cql/ls/server/provider/HoverProvider.kt

@@ -16,6 +16,12 @@ import org.opencds.cqf.cql.ls.server.manager.CqlCompilationManager
 class HoverProvider(private val cqlCompilationManager: CqlCompilationManager) {
 
     fun hover(position: HoverParams): Hover? {
+        // Returning null here effectively disables hover functionality from the server side.
+        // 2026-03-30 RGT: Disable hover functionality for now.
+        //                 The LibraryManager caching issues is causing the CQL Compiler to continutally fire.
+        //                 This is affect performance, revist hover functionality later.
+        return null
+
         val uri = Uris.parseOrNull(position.textDocument.uri) ?: return null
 
         // This translates on the fly. We may want to consider maintaining

ls/server/src/test/java/org/opencds/cqf/cql/ls/server/LanguageServerTest.kt

@@ -16,6 +16,7 @@ import org.junit.jupiter.api.Assertions.assertNotNull
 import org.junit.jupiter.api.Assertions.assertNull
 import org.junit.jupiter.api.Assertions.assertTrue
 import org.junit.jupiter.api.BeforeAll
+import org.junit.jupiter.api.Disabled
 import org.junit.jupiter.api.Test
 import org.junit.jupiter.api.assertDoesNotThrow
 import org.mockito.Mockito
@@ -71,6 +72,7 @@ class LanguageServerTest {
         assertNotNull(server)
     }
 
+    @Disabled("Disabled until LibraryManager caching issues are resolved")
     @Test
     fun hoverInt() {
         val hover = server.getTextDocumentService()
@@ -85,6 +87,7 @@ class LanguageServerTest {
         assertEquals("```cql\nSystem.Integer\n```", markup.value)
     }
 
+    @Disabled("Disabled until LibraryManager caching issues are resolved")
     @Test
     fun hoverNothing() {
         val hover = server.getTextDocumentService()
@@ -94,6 +97,7 @@ class LanguageServerTest {
         assertNull(hover)
     }
 
+    @Disabled("Disabled until LibraryManager caching issues are resolved")
     @Test
     fun hoverList() {
         val hover = server.getTextDocumentService()
@@ -124,6 +128,7 @@ class LanguageServerTest {
         assertNotNull(result.capabilities)
     }
 
+    @Disabled("Disabled until LibraryManager caching issues are resolved")
     @Test
     fun initialize_setsHoverProviderCapability() {
         val result = server.initialize(InitializeParams()).get()

ls/server/src/test/kotlin/org/opencds/cqf/cql/ls/server/provider/HoverProviderTest.kt

@@ -7,6 +7,7 @@ import org.junit.jupiter.api.Assertions.assertEquals
 import org.junit.jupiter.api.Assertions.assertNotNull
 import org.junit.jupiter.api.Assertions.assertNull
 import org.junit.jupiter.api.BeforeAll
+import org.junit.jupiter.api.Disabled
 import org.junit.jupiter.api.Test
 import org.opencds.cqf.cql.ls.server.manager.CompilerOptionsManager
 import org.opencds.cqf.cql.ls.server.manager.CqlCompilationManager
@@ -27,6 +28,7 @@ class HoverProviderTest {
         }
     }
 
+    @Disabled("Disabled until LibraryManager caching issues are resolved")
     @Test
     fun hoverInt() {
         val hover =
@@ -45,6 +47,7 @@ class HoverProviderTest {
         assertEquals("```cql\nSystem.Integer\n```", markup.value)
     }
 
+    @Disabled("Disabled until LibraryManager caching issues are resolved")
     @Test
     fun hoverNothing() {
         val hover =
@@ -58,6 +61,7 @@ class HoverProviderTest {
         assertNull(hover)
     }
 
+    @Disabled("Disabled until LibraryManager caching issues are resolved")
     @Test
     fun hoverList() {
         val hover =
@@ -76,6 +80,7 @@ class HoverProviderTest {
         assertEquals("```cql\nlist<System.Integer>\n```", markup.value)
     }
 
+    @Disabled("Disabled until LibraryManager caching issues are resolved")
     @Test
     fun hoverOnLibraryRef() {
         // Line 5 (0-indexed): "    1 + One."One"" — position (5, 8) is 'O' in 'One'
@@ -95,6 +100,7 @@ class HoverProviderTest {
         assertEquals("```cql\nSystem.Integer\n```", markup.value)
     }
 
+    @Disabled("Disabled until LibraryManager caching issues are resolved")
     @Test
     fun hoverOnDefineName() {
         // Line 4 (0-indexed): "define "Two":" — position (4, 8) is 'T' inside the define name

ls/service/pom.xml

@@ -11,7 +11,7 @@
     <parent>
         <groupId>org.opencds.cqf.cql.ls</groupId>
         <artifactId>cql-ls</artifactId>
-        <version>4.4.0-SNAPSHOT</version>
+        <version>4.4.0</version>
         <relativePath>../../pom.xml</relativePath>
     </parent>
 

plugin/debug/pom.xml

@@ -11,7 +11,7 @@
     <parent>
         <groupId>org.opencds.cqf.cql.ls</groupId>
         <artifactId>cql-ls</artifactId>
-        <version>4.4.0-SNAPSHOT</version>
+        <version>4.4.0</version>
         <relativePath>../../pom.xml</relativePath>
     </parent>
 

pom.xml

@@ -5,7 +5,7 @@
     <groupId>org.opencds.cqf.cql.ls</groupId>
     <artifactId>cql-ls</artifactId>
     <packaging>pom</packaging>
-    <version>4.4.0-SNAPSHOT</version>
+    <version>4.4.0</version>
 
     <name>CQL Language Server</name>
     <description>A Language Server for CQL implementing the LSP</description>