Fix code actions + remove unused imports

Author: rochala

presentation-compiler/src/main/dotty/tools/pc/DiagnosticProvider.scala

@@ -5,19 +5,14 @@ import org.eclipse.lsp4j.DiagnosticSeverity
 import dotty.tools.dotc.interactive.InteractiveDriver
 import dotty.tools.dotc.interfaces.Diagnostic as DiagnosticInterfaces
 import dotty.tools.dotc.reporting.Diagnostic
-import dotty.tools.dotc.semanticdb.DiagnosticOps.toSemanticDiagnostic
 import dotty.tools.pc.utils.InteractiveEnrichments.toLsp
 
 import scala.meta.pc.VirtualFileParams
-import com.google.gson.Gson
 import ch.epfl.scala.bsp4j
 import dotty.tools.dotc.reporting.CodeAction
 import dotty.tools.dotc.rewrites.Rewrites.ActionPatch
 import scala.jdk.CollectionConverters.*
 import dotty.tools.dotc.core.Contexts.Context
-import dotty.tools.dotc.reporting.Message
-import dotty.tools.dotc.interfaces.DiagnosticRelatedInformation
-import org.eclipse.lsp4j.Location
 import dotty.tools.dotc.reporting.ErrorMessageID
 import org.eclipse.lsp4j.DiagnosticTag
 
@@ -40,8 +35,9 @@ class DiagnosticProvider(driver: InteractiveDriver, params: VirtualFileParams):
 
       val scalaDiagnostic = new bsp4j.ScalaDiagnostic()
       val actions = diag.msg.actions.map(toBspScalaAction).asJava
+      scalaDiagnostic.setActions(actions)
       // lspDiag.setRelatedInformation(???) Currently not emitted by the compiler
-      lspDiag.setData(actions)
+      lspDiag.setData(scalaDiagnostic)
       if diag.msg.errorId == ErrorMessageID.UnusedSymbolID then
         lspDiag.setTags(List(DiagnosticTag.Unnecessary).asJava)
 

presentation-compiler/test/dotty/tools/pc/tests/DiagnosticProviderSuite.scala

@@ -6,20 +6,15 @@ import dotty.tools.pc.base.BasePCSuite
 import dotty.tools.pc.utils.RangeReplace
 
 import java.net.URI
-import org.eclipse.lsp4j.jsonrpc.messages.{Either => JEither}
 import scala.meta.internal.jdk.CollectionConverters.*
 import scala.meta.internal.metals.CompilerVirtualFileParams
 import scala.meta.internal.metals.EmptyCancelToken
-import scala.meta.internal.pc.PcReferencesRequest
 
 import org.junit.Test
-import scala.collection.mutable.ListBuffer
 import org.eclipse.lsp4j.DiagnosticSeverity
-import scala.concurrent.duration.*
 import dotty.tools.pc.utils.TestExtensions.getOffset
 
 class DiagnosticProviderSuite extends BasePCSuite with RangeReplace {
-  private val rangeRegex = "<<.*>>".r
   case class TestDiagnostic(startIndex: Int, endIndex: Int, msg: String, severity: DiagnosticSeverity)
 
   def check(