Смена апи MarkupContentBuilder в сторону SymbolKind

Author: nixel2007

src/main/java/com/github/_1c_syntax/bsl/languageserver/hover/MarkupContentBuilder.java

@@ -23,6 +23,7 @@
 
 import com.github._1c_syntax.bsl.languageserver.context.symbol.Symbol;
 import org.eclipse.lsp4j.MarkupContent;
+import org.eclipse.lsp4j.SymbolKind;
 
 /**
  * Интерфейс построителя контента для всплывающего окна на основе символа.
@@ -41,7 +42,7 @@ public interface MarkupContentBuilder<T extends Symbol> {
   /**
    * Тип символа, на основе которого работает данный построитель.
    *
-   * @return тип символа (см. {@link Symbol})
+   * @return тип символа.
    */
-  Class<T> getType();
+  SymbolKind getSymbolKind();
 }

src/main/java/com/github/_1c_syntax/bsl/languageserver/hover/MarkupContentBuilderConfiguration.java

@@ -22,6 +22,7 @@
 package com.github._1c_syntax.bsl.languageserver.hover;
 
 import com.github._1c_syntax.bsl.languageserver.context.symbol.Symbol;
+import org.eclipse.lsp4j.SymbolKind;
 import org.springframework.context.annotation.Bean;
 import org.springframework.context.annotation.Configuration;
 
@@ -41,10 +42,10 @@ public class MarkupContentBuilderConfiguration {
    * @return Карта построителей контента для всплывающего окна и типов символов, для которых они предназначены.
    */
   @Bean
-  public <T extends Symbol> Map<Class<T>, MarkupContentBuilder<T>> markupContentBuilders(
+  public <T extends Symbol> Map<SymbolKind, MarkupContentBuilder<T>> markupContentBuilders(
     Collection<MarkupContentBuilder<T>> builders
   ) {
-    return builders.stream().collect(Collectors.toMap(MarkupContentBuilder::getType, Function.identity()));
+    return builders.stream().collect(Collectors.toMap(MarkupContentBuilder::getSymbolKind, Function.identity()));
   }
 
 }

src/main/java/com/github/_1c_syntax/bsl/languageserver/hover/MethodSymbolMarkupContentBuilder.java

@@ -32,6 +32,7 @@
 import lombok.RequiredArgsConstructor;
 import org.eclipse.lsp4j.MarkupContent;
 import org.eclipse.lsp4j.MarkupKind;
+import org.eclipse.lsp4j.SymbolKind;
 import org.springframework.stereotype.Component;
 
 import java.util.Collections;
@@ -107,8 +108,8 @@ public MarkupContent getContent(MethodSymbol symbol) {
   }
 
   @Override
-  public Class<MethodSymbol> getType() {
-    return MethodSymbol.class;
+  public SymbolKind getSymbolKind() {
+    return SymbolKind.Method;
   }
 
   private static void addSectionIfNotEmpty(StringJoiner markupBuilder, String newContent) {

src/main/java/com/github/_1c_syntax/bsl/languageserver/hover/VariableSymbolMarkupContentBuilder.java

@@ -75,8 +75,8 @@ public MarkupContent getContent(VariableSymbol symbol) {
   }
 
   @Override
-  public Class<VariableSymbol> getType() {
-    return VariableSymbol.class;
+  public SymbolKind getSymbolKind() {
+    return SymbolKind.Variable;
   }
 
   private String getSignature(VariableSymbol symbol) {

src/main/java/com/github/_1c_syntax/bsl/languageserver/providers/HoverProvider.java

@@ -30,6 +30,7 @@
 import org.eclipse.lsp4j.Hover;
 import org.eclipse.lsp4j.HoverParams;
 import org.eclipse.lsp4j.Position;
+import org.eclipse.lsp4j.SymbolKind;
 import org.springframework.stereotype.Component;
 
 import java.util.Map;
@@ -40,7 +41,7 @@
 public final class HoverProvider {
 
   private final ReferenceResolver referenceResolver;
-  private final Map<Class<Symbol>, MarkupContentBuilder<Symbol>> markupContentBuilders;
+  private final Map<SymbolKind, MarkupContentBuilder<Symbol>> markupContentBuilders;
 
   public Optional<Hover> getHover(DocumentContext documentContext, HoverParams params) {
     Position position = params.getPosition();
@@ -50,7 +51,7 @@ public Optional<Hover> getHover(DocumentContext documentContext, HoverParams par
         var symbol = reference.getSymbol();
         var range = reference.getSelectionRange();
 
-        return Optional.ofNullable(markupContentBuilders.get(symbol.getClass()))
+        return Optional.ofNullable(markupContentBuilders.get(symbol.getSymbolKind()))
           .map(markupContentBuilder -> markupContentBuilder.getContent(symbol))
           .map(content -> new Hover(content, range));
       });