Support multiple projects simultaneously (#789)

* fix: sourceplusplus/sourceplusplus#475

* refactor: use intellij logging

* remove plugin debug console

* gracefully handle config with removed settings

* fix: reload compiled live plugins on project reset

* fix: already disposed IncorrectOperationException

* refactor: get live services from project
refactor: get service discovery backend from project

* fix: npe

* refactor: get live status manager from project

* fix check

* unused

Author: BFergerson

commander

@@ -2,7 +2,7 @@
 # https://github.com/detekt/detekt/blob/master/detekt-core/src/main/resources/default-detekt-config.yml
 
 build:
-  maxIssues: 26 #todo: zero out
+  maxIssues: 38 #todo: zero out
 
 formatting:
   Indentation:

detekt.yml

@@ -33,7 +33,6 @@ dependencies {
     compileOnly("org.jetbrains.kotlin:kotlin-stdlib-jdk8")
     compileOnly("com.google.guava:guava:31.1-jre")
     compileOnly("org.jetbrains:annotations:23.0.0")
-    compileOnly("org.slf4j:slf4j-api:$slf4jVersion")
     compileOnly("org.jetbrains.intellij.deps.jcef:jcef:97.2.22-g6779618-chromium-97.0.4692.45-api-1.6")
     compileOnly("com.jetbrains.intellij.platform:ide:$intellijVersion")
     compileOnly("com.jetbrains.intellij.platform:ide-impl:$intellijVersion")

marker/build.gradle.kts

@@ -32,7 +32,6 @@ dependencies {
     compileOnly("io.vertx:vertx-core:$vertxVersion")
     compileOnly("io.vertx:vertx-lang-kotlin:$vertxVersion")
     compileOnly("io.vertx:vertx-lang-kotlin-coroutines:$vertxVersion")
-    compileOnly("org.slf4j:slf4j-api:$slf4jVersion")
     compileOnly("org.jooq:joor:$joorVersion")
     compileOnly("org.apache.commons:commons-lang3:3.12.0")
     compileOnly("com.android.tools.external.org-jetbrains:uast:30.2.2")

marker/jvm-marker/build.gradle.kts

@@ -18,6 +18,7 @@ package spp.jetbrains.marker.jvm
 
 import com.intellij.ide.util.PsiNavigationSupport
 import com.intellij.openapi.application.ApplicationManager
+import com.intellij.openapi.diagnostic.logger
 import com.intellij.openapi.editor.Document
 import com.intellij.openapi.project.Project
 import com.intellij.psi.JavaPsiFacade
@@ -29,7 +30,6 @@ import io.vertx.core.*
 import io.vertx.kotlin.coroutines.await
 import org.jetbrains.uast.UMethod
 import org.jetbrains.uast.toUElement
-import org.slf4j.LoggerFactory
 import spp.jetbrains.marker.source.JVMMarkerUtils
 import spp.protocol.artifact.ArtifactQualifiedName
 import spp.protocol.artifact.exception.LiveStackTraceElement
@@ -44,7 +44,7 @@ import spp.protocol.artifact.exception.sourceAsLineNumber
  */
 object ArtifactNavigator {
 
-    private val log = LoggerFactory.getLogger(ArtifactNavigator::class.java)
+    private val log = logger<ArtifactNavigator>()
 
     //todo: remove method from method names and support navigating to classes?
 
@@ -77,7 +77,7 @@ object ArtifactNavigator {
                 handler.handle(Future.succeededFuture(true))
             }
         } else {
-            log.warn("Could not find artifact: {}", artifactQualifiedName)
+            log.warn("Could not find artifact: $artifactQualifiedName")
             handler.handle(Future.succeededFuture(false))
         }
     }

marker/jvm-marker/src/main/kotlin/spp/jetbrains/marker/jvm/ArtifactNavigator.kt

@@ -109,7 +109,7 @@ object ArtifactSearch {
     @Suppress("UnstableApiUsage")
     suspend fun findArtifact(vertx: Vertx, artifact: ArtifactQualifiedName): PsiElement? {
         val promise = Promise.promise<Optional<PsiElement>>()
-        val project = ProjectManager.getInstance().openProjects[0]
+        val project = ProjectManager.getInstance().openProjects[0] //todo: support multiple projects
 
         ApplicationManager.getApplication().runReadAction {
             if (artifact.type == ArtifactType.CLASS) {

marker/jvm-marker/src/main/kotlin/spp/jetbrains/marker/jvm/ArtifactSearch.kt

@@ -18,6 +18,7 @@ package spp.jetbrains.marker.jvm
 
 import com.intellij.codeInsight.daemon.GutterIconNavigationHandler
 import com.intellij.codeInsight.daemon.LineMarkerInfo
+import com.intellij.openapi.diagnostic.logger
 import com.intellij.openapi.editor.markup.GutterIconRenderer
 import com.intellij.psi.PsiClass
 import com.intellij.psi.PsiElement
@@ -29,7 +30,6 @@ import org.jetbrains.uast.UClass
 import org.jetbrains.uast.UMethod
 import org.jetbrains.uast.toUElement
 import org.jetbrains.uast.toUElementOfType
-import org.slf4j.LoggerFactory
 import spp.jetbrains.marker.SourceMarker
 import spp.jetbrains.marker.impl.ArtifactCreationService
 import spp.jetbrains.marker.plugin.SourceLineMarkerProvider
@@ -50,7 +50,7 @@ import spp.jetbrains.marker.source.mark.gutter.MethodGutterMark
 abstract class JVMLineMarkerProvider : SourceLineMarkerProvider() {
 
     companion object {
-        private val log = LoggerFactory.getLogger(JVMLineMarkerProvider::class.java)
+        private val log = logger<JVMLineMarkerProvider>()
     }
 
     override fun getLineMarkerInfo(
@@ -65,9 +65,9 @@ abstract class JVMLineMarkerProvider : SourceLineMarkerProvider() {
     }
 
     private fun getClassGutterMark(element: PsiIdentifier): LineMarkerInfo<PsiElement>? {
-        val fileMarker = SourceMarker.getSourceFileMarker(element.containingFile) ?: return null
+        val fileMarker = SourceMarker.getInstance(element.project).getSourceFileMarker(element.containingFile) ?: return null
         val artifactQualifiedName = JVMMarkerUtils.getFullyQualifiedName(element.parent.toUElement() as UClass)
-        if (!SourceMarker.configuration.createSourceMarkFilter.test(artifactQualifiedName)) return null
+        if (!SourceMarker.getInstance(element.project).configuration.createSourceMarkFilter.test(artifactQualifiedName)) return null
 
         //check by artifact name first due to user can erroneously name same class twice
         var gutterMark = fileMarker.getSourceMark(artifactQualifiedName, SourceMark.Type.GUTTER) as ClassGutterMark?
@@ -95,14 +95,14 @@ abstract class JVMLineMarkerProvider : SourceLineMarkerProvider() {
     }
 
     private fun getMethodGutterMark(element: PsiElement): LineMarkerInfo<PsiElement>? {
-        val fileMarker = SourceMarker.getSourceFileMarker(element.containingFile) ?: return null
+        val fileMarker = SourceMarker.getInstance(element.project).getSourceFileMarker(element.containingFile) ?: return null
         val uMethod = element.parent.toUElementOfType<UMethod>()
         if (uMethod == null) {
-            log.warn("Unable to transform to UMethod: {}", element.parent)
+            log.warn("Unable to transform to UMethod: ${element.parent}")
             return null
         }
         val artifactQualifiedName = JVMMarkerUtils.getFullyQualifiedName(uMethod)
-        if (!SourceMarker.configuration.createSourceMarkFilter.test(artifactQualifiedName)) return null
+        if (!SourceMarker.getInstance(element.project).configuration.createSourceMarkFilter.test(artifactQualifiedName)) return null
 
         //check by artifact name first due to user can erroneously name same method twice
         var gutterMark = fileMarker.getSourceMark(

marker/jvm-marker/src/main/kotlin/spp/jetbrains/marker/jvm/JVMLineMarkerProvider.kt

@@ -44,17 +44,17 @@ class JVMSourceInlayHintProvider : SourceInlayHintProvider() {
         if ((parent is PsiMethod && element === parent.nameIdentifier)
             || (JVMMarkerUtils.getNameIdentifier(parent) === element)
         ) {
-            val fileMarker = SourceMarker.getSourceFileMarker(element.containingFile)!!
+            val fileMarker = SourceMarker.getInstance(element.project).getSourceFileMarker(element.containingFile)!!
             val artifactQualifiedName = JVMMarkerUtils.getFullyQualifiedName(parent.toUElement() as UMethod)
-            return if (!SourceMarker.configuration.createSourceMarkFilter.test(artifactQualifiedName)) null else {
+            return if (!SourceMarker.getInstance(element.project).configuration.createSourceMarkFilter.test(artifactQualifiedName)) null else {
                 JVMMarkerUtils.getOrCreateMethodInlayMark(fileMarker, element)
             }
         } else if (element is PsiStatement) {
-            val fileMarker = SourceMarker.getSourceFileMarker(element.containingFile)!!
+            val fileMarker = SourceMarker.getInstance(element.project).getSourceFileMarker(element.containingFile)!!
             val artifactQualifiedName = JVMMarkerUtils.getFullyQualifiedName(
                 JVMMarkerUtils.getUniversalExpression(element).toUElement() as UExpression
             )
-            return if (!SourceMarker.configuration.createSourceMarkFilter.test(artifactQualifiedName)) null else {
+            return if (!SourceMarker.getInstance(element.project).configuration.createSourceMarkFilter.test(artifactQualifiedName)) null else {
                 JVMMarkerUtils.getOrCreateExpressionInlayMark(fileMarker, element)
             }
         }

marker/jvm-marker/src/main/kotlin/spp/jetbrains/marker/jvm/JVMSourceInlayHintProvider.kt

@@ -17,6 +17,7 @@
 package spp.jetbrains.marker.jvm.psi
 
 import com.intellij.openapi.application.ApplicationManager
+import com.intellij.openapi.diagnostic.logger
 import com.intellij.openapi.editor.Editor
 import com.intellij.openapi.editor.LogicalPosition
 import com.intellij.openapi.util.TextRange
@@ -33,7 +34,6 @@ import kotlinx.coroutines.launch
 import kotlinx.coroutines.runBlocking
 import org.jetbrains.uast.UMethod
 import org.jetbrains.uast.toUElementOfType
-import org.slf4j.LoggerFactory
 import spp.jetbrains.marker.source.SourceFileMarker
 import spp.jetbrains.marker.source.mark.api.MethodSourceMark
 import spp.jetbrains.marker.source.mark.api.key.SourceKey
@@ -48,7 +48,7 @@ import spp.jetbrains.marker.source.mark.inlay.InlayMark
 class LoggerDetector(val vertx: Vertx) {
 
     companion object {
-        private val log = LoggerFactory.getLogger(LoggerDetector::class.java)
+        private val log = logger<LoggerDetector>()
         val LOGGER_STATEMENTS = SourceKey<List<DetectedLogger>>("LOGGER_STATEMENTS")
 
         private val LOGGER_CLASSES = setOf(

marker/jvm-marker/src/main/kotlin/spp/jetbrains/marker/jvm/psi/LoggerDetector.kt

@@ -16,11 +16,11 @@
  */
 package spp.jetbrains.marker.source
 
+import com.intellij.openapi.diagnostic.logger
 import com.intellij.psi.*
 import com.intellij.psi.util.PsiUtil
 import org.jetbrains.uast.*
 import org.joor.Reflect
-import org.slf4j.LoggerFactory
 import spp.jetbrains.marker.SourceMarkerUtils
 import spp.jetbrains.marker.source.mark.api.SourceMark
 import spp.jetbrains.marker.source.mark.api.key.SourceKey
@@ -42,7 +42,7 @@ import java.util.*
 @Suppress("TooManyFunctions")
 object JVMMarkerUtils {
 
-    private val log = LoggerFactory.getLogger(JVMMarkerUtils::class.java)
+    private val log = logger<JVMMarkerUtils>()
 
     /**
      * todo: description.
@@ -445,7 +445,7 @@ object JVMMarkerUtils {
         if (gutterMark == null) {
             val uClass = element.parent.toUElement() as UClass
             if (uClass.qualifiedName == null) {
-                log.warn("Could not determine qualified name of class: {}", uClass)
+                log.warn("Could not determine qualified name of class: $uClass")
                 return null
             }
             gutterMark = fileMarker.createClassSourceMark(
@@ -611,7 +611,7 @@ object JVMMarkerUtils {
                     it.type.canonicalText
                 }
             } else {
-                log.warn("Unable to detect element type: {}", it)
+                log.warn("Unable to detect element type: $it")
             }
         }
         return "$methodName($methodParams)"