chore: refactor lemminx extension structure

vespa-engine · Oct 25, 2024 · a6de126 · a6de126
1 parent f2c64f0
commit a6de126
Show file tree

Hide file tree

Showing 8 changed files with 45 additions and 21 deletions.
diff --git a/...age-server/lemminx-vespa/src/main/java/ai/vespa/lemminx/ServicesURIResolverExtension.java b/...age-server/lemminx-vespa/src/main/java/ai/vespa/lemminx/ServicesURIResolverExtension.java
@@ -1,7 +1,6 @@
 package ai.vespa.lemminx;
 
 import java.nio.file.Path;
-import java.util.logging.Logger;
 
 import org.eclipse.lemminx.uriresolver.URIResolverExtension;
 

diff --git a/...n/schema-language-server/lemminx-vespa/src/main/java/ai/vespa/lemminx/VespaExtension.java b/...n/schema-language-server/lemminx-vespa/src/main/java/ai/vespa/lemminx/VespaExtension.java
@@ -14,6 +14,12 @@
 import org.eclipse.lemminx.uriresolver.URIResolverExtension;
 import org.eclipse.lsp4j.InitializeParams;
 
+import ai.vespa.lemminx.command.SchemaLSCommands;
+import ai.vespa.lemminx.participants.DefinitionParticipant;
+import ai.vespa.lemminx.participants.DiagnosticsParticipant;
+import ai.vespa.lemminx.participants.DocumentLifecycleParticipant;
+import ai.vespa.lemminx.participants.HoverParticipant;
+
 public class VespaExtension implements IXMLExtension {
     private static final Logger logger = Logger.getLogger(VespaExtension.class.getName());
 

diff --git a/...va/ai/vespa/lemminx/SchemaLSCommands.java → ...spa/lemminx/command/SchemaLSCommands.java b/...va/ai/vespa/lemminx/SchemaLSCommands.java → ...spa/lemminx/command/SchemaLSCommands.java
@@ -1,4 +1,4 @@
-package ai.vespa.lemminx;
+package ai.vespa.lemminx.command;
 
 import java.lang.reflect.Type;
 import java.util.List;
@@ -35,12 +35,12 @@ public static SchemaLSCommands instance() {
     }
 
     public void sendSetupWorkspaceRequest(String fileURI) {
-        commandService.executeClientCommand(new ExecuteCommandParams("vespaSchemaLS.commands.setupWorkspace", List.of(fileURI)));
+        commandService.executeClientCommand(new ExecuteCommandParams("vespaSchemaLS.commands.schema.setupWorkspace", List.of(fileURI)));
     }
 
     public boolean hasSetupWorkspace() {
         Object result = commandService.executeClientCommand(
-                new ExecuteCommandParams("vespaSchemaLS.commands.hasSetupWorkspace", List.of())).join();
+                new ExecuteCommandParams("vespaSchemaLS.commands.schema.hasSetupWorkspace", List.of())).join();
         if (result == null) return false;
         try {
             String json = gson.toJson(result);
@@ -59,7 +59,7 @@ public boolean hasSetupWorkspace() {
     public List<Location> findSchemaDefinition(String schemaName) {
         // run sync
         Object findDocumentResult = commandService.executeClientCommand(
-            new ExecuteCommandParams("vespaSchemaLS.commands.findSchemaDefinition", List.of(schemaName))).join();
+            new ExecuteCommandParams("vespaSchemaLS.commands.schema.findSchemaDefinition", List.of(schemaName))).join();
 
         if (findDocumentResult == null) return List.of();
         try {

diff --git a/...rver/lemminx-vespa/src/main/java/ai/vespa/lemminx/participants/CodeActionParticipant.java b/...rver/lemminx-vespa/src/main/java/ai/vespa/lemminx/participants/CodeActionParticipant.java
@@ -0,0 +1,20 @@
+package ai.vespa.lemminx.participants;
+
+import java.util.List;
+import java.util.concurrent.CancellationException;
+import java.util.logging.Logger;
+
+import org.eclipse.lemminx.services.extensions.codeaction.ICodeActionParticipant;
+import org.eclipse.lemminx.services.extensions.codeaction.ICodeActionRequest;
+import org.eclipse.lsp4j.CodeAction;
+import org.eclipse.lsp4j.jsonrpc.CancelChecker;
+
+public class CodeActionParticipant implements ICodeActionParticipant {
+    private static final Logger logger = Logger.getLogger(CodeActionParticipant.class.getName());
+
+    @Override
+    public void doCodeAction(ICodeActionRequest request, List<CodeAction> codeActions, CancelChecker cancelChecker) throws CancellationException {
+        logger.info(request.getDiagnostic().toString());
+    }
+}
+
diff --git a/.../vespa/lemminx/DefinitionParticipant.java → ...x/participants/DefinitionParticipant.java b/.../vespa/lemminx/DefinitionParticipant.java → ...x/participants/DefinitionParticipant.java
@@ -1,4 +1,4 @@
-package ai.vespa.lemminx;
+package ai.vespa.lemminx.participants;
 
 import java.util.List;
 import java.util.logging.Logger;
@@ -10,6 +10,8 @@
 import org.eclipse.lsp4j.LocationLink;
 import org.eclipse.lsp4j.jsonrpc.CancelChecker;
 
+import ai.vespa.lemminx.command.SchemaLSCommands;
+
 public class DefinitionParticipant implements IDefinitionParticipant {
     private static final Logger logger = Logger.getLogger(DefinitionParticipant.class.getName());
 

diff --git a/...vespa/lemminx/DiagnosticsParticipant.java → .../participants/DiagnosticsParticipant.java b/...vespa/lemminx/DiagnosticsParticipant.java → .../participants/DiagnosticsParticipant.java
@@ -1,4 +1,4 @@
-package ai.vespa.lemminx;
+package ai.vespa.lemminx.participants;
 
 import java.util.List;
 import java.util.logging.Logger;
@@ -17,6 +17,8 @@
 import org.eclipse.lsp4j.jsonrpc.CancelChecker;
 import org.w3c.dom.Node;
 
+import ai.vespa.lemminx.command.SchemaLSCommands;
+
 public class DiagnosticsParticipant implements IDiagnosticsParticipant {
     private static final Logger logger = Logger.getLogger(DiagnosticsParticipant.class.getName());
 

diff --git a/...lemminx/DocumentLifecycleParticipant.java → ...cipants/DocumentLifecycleParticipant.java b/...lemminx/DocumentLifecycleParticipant.java → ...cipants/DocumentLifecycleParticipant.java
@@ -1,4 +1,4 @@
-package ai.vespa.lemminx;
+package ai.vespa.lemminx.participants;
 
 import java.util.List;
 import java.util.logging.Logger;
@@ -8,6 +8,8 @@
 import org.eclipse.lemminx.services.extensions.commands.IXMLCommandService;
 import org.eclipse.lsp4j.ExecuteCommandParams;
 
+import ai.vespa.lemminx.command.SchemaLSCommands;
+
 public class DocumentLifecycleParticipant implements IDocumentLifecycleParticipant {
     private static final Logger logger = Logger.getLogger(DocumentLifecycleParticipant.class.getName());
     private IXMLCommandService commandService;

diff --git a/...va/ai/vespa/lemminx/HoverParticipant.java → ...emminx/participants/HoverParticipant.java b/...va/ai/vespa/lemminx/HoverParticipant.java → ...emminx/participants/HoverParticipant.java
@@ -1,4 +1,4 @@
-package ai.vespa.lemminx;
+package ai.vespa.lemminx.participants;
 
 import java.io.IOException;
 import java.nio.file.Files;
@@ -9,14 +9,16 @@
 import java.util.logging.Logger;
 import java.util.stream.Stream;
 
-import org.eclipse.lemminx.dom.DOMNode;
 import org.eclipse.lemminx.services.extensions.hover.IHoverParticipant;
 import org.eclipse.lemminx.services.extensions.hover.IHoverRequest;
 import org.eclipse.lsp4j.Hover;
 import org.eclipse.lsp4j.MarkupContent;
 import org.eclipse.lsp4j.MarkupKind;
 import org.eclipse.lsp4j.jsonrpc.CancelChecker;
 
+/**
+ * TODO: refactor and handle tags with the same name based on context.
+ */
 public class HoverParticipant implements IHoverParticipant {
     private static final Logger logger = Logger.getLogger(HoverParticipant.class.getName());
     private Path serverPath;
@@ -29,15 +31,6 @@ public HoverParticipant(Path serverPath) {
     public Hover onTag(IHoverRequest request, CancelChecker cancelChecker) throws Exception {
         if (request.getCurrentTag() == null) return null;
 
-        DOMNode node = request.getNode();
-
-        String logMsg = "";
-        while (node != null) {
-            logMsg += node.getNodeName() + " -> ";
-            node = node.getParentNode();
-        }
-        logger.info(logMsg);
-
         Optional<MarkupContent> content = getFileHover(request.getCurrentTag());
 
         if (content.isEmpty()) return null;
@@ -85,8 +78,8 @@ private Optional<MarkupContent> getFileHover(String tagName) {
                                 markdownFiles.put(tag, innerPath);
                             }
                         });
-                    } catch (IOException ex) {
-                        logger.severe("Inner ioexception");
+                    } catch (IOException ex) { 
+                        // Ignore
                     }
                 } else {
                     String tag = path.getFileName().toString();