Selected node overview

TODO:
- Unsafe fallback option
- The Whole bundle must be from the same repository
- Better file watcher (create events, watch the whole local repo)
- Check how exceptions are handled while in checkers
- Better redraw algorithm for exceptions

Author: Mr3zee

src/main/kotlin/com/github/mr3zee/kotlinPlugins/KotlinPluginsActions.kt

@@ -1,7 +1,6 @@
 package com.github.mr3zee.kotlinPlugins
 
 import com.intellij.icons.AllIcons
-import com.intellij.openapi.actionSystem.ActionPlaces
 import com.intellij.openapi.actionSystem.ActionUpdateThread
 import com.intellij.openapi.actionSystem.AnAction
 import com.intellij.openapi.actionSystem.AnActionEvent
@@ -13,7 +12,7 @@ import javax.swing.JComponent
 open class KotlinPluginsClearCachesAction : AnAction(AllIcons.Actions.ClearCash) {
     override fun actionPerformed(e: AnActionEvent) {
         val project = e.project ?: return
-        val treeState = TreeState.getInstance(project)
+        val treeState = KotlinPluginsTreeState.getInstance(project)
 
         val clear = !treeState.showClearCachesDialog || run {
             val (clear, dontShowAgain) = ClearCachesDialog.show()

src/main/kotlin/com/github/mr3zee/kotlinPlugins/KotlinPluginsConfigurable.kt

@@ -6,6 +6,7 @@ import com.intellij.openapi.actionSystem.AnActionEvent
 import com.intellij.openapi.components.service
 import com.intellij.openapi.fileChooser.FileChooserDescriptorFactory
 import com.intellij.openapi.options.Configurable
+import com.intellij.openapi.options.ShowSettingsUtil
 import com.intellij.openapi.project.Project
 import com.intellij.openapi.ui.ComboBox
 import com.intellij.openapi.ui.DialogWrapper
@@ -51,7 +52,7 @@ private class LocalState {
 
     fun isModified(
         analyzer: KotlinPluginsExceptionAnalyzerState,
-        tree: TreeState,
+        tree: KotlinPluginsTreeState,
         settings: KotlinPluginsSettings.State,
     ): Boolean {
         return repositories != settings.repositories ||
@@ -64,7 +65,7 @@ private class LocalState {
 
     fun reset(
         analyzer: KotlinPluginsExceptionAnalyzerState,
-        tree: TreeState,
+        tree: KotlinPluginsTreeState,
         settings: KotlinPluginsSettings.State,
     ) {
         repositories.clear()
@@ -83,7 +84,7 @@ private class LocalState {
 
     fun applyTo(
         analyzer: KotlinPluginsExceptionAnalyzerService,
-        tree: TreeState,
+        tree: KotlinPluginsTreeState,
         settings: KotlinPluginsSettings,
     ) {
         val enabledPlugins = plugins.map {
@@ -101,6 +102,21 @@ private class LocalState {
 }
 
 class KotlinPluginsConfigurable(private val project: Project) : Configurable {
+    companion object {
+        @Volatile
+        private var selectArtifactsInitially: Boolean = false
+
+        fun showArtifacts(project: Project) {
+            selectArtifactsInitially = true
+            ShowSettingsUtil.getInstance().showSettingsDialog(project, KotlinPluginsConfigurable::class.java)
+        }
+
+        fun showGeneral(project: Project) {
+            selectArtifactsInitially = false
+            ShowSettingsUtil.getInstance().showSettingsDialog(project, KotlinPluginsConfigurable::class.java)
+        }
+    }
+
     private val local: LocalState = LocalState()
 
     private lateinit var repoTable: JBTable
@@ -303,6 +319,10 @@ class KotlinPluginsConfigurable(private val project: Project) : Configurable {
         val tabs = JBTabbedPane()
         tabs.addTab("General", generalContent)
         tabs.addTab("Artifacts", artifactsContent)
+        if (selectArtifactsInitially) {
+            tabs.selectedIndex = 1
+            selectArtifactsInitially = false
+        }
         rootPanel = JPanel(BorderLayout()).apply { add(tabs, BorderLayout.NORTH) }
 
         reset() // initialise from a persisted state

src/main/kotlin/com/github/mr3zee/kotlinPlugins/KotlinPluginsEditorNotificationProvider.kt

@@ -71,7 +71,14 @@ class KotlinPluginsEditorNotificationProvider : EditorNotificationProvider {
             }
 
             panel.createActionLabel("Open diagnostics") {
-                // todo update state to show error panel
+                val pluginOrNull = grouped.keys.singleOrNull()
+
+                val jarIdOrNull = pluginOrNull?.let { grouped[it]?.singleOrNull() }
+                val mavenIdOrNull = jarIdOrNull?.mavenId
+                val versionOrNull = jarIdOrNull?.version
+
+                KotlinPluginsTreeState.getInstance(project)
+                    .select(pluginOrNull, mavenIdOrNull, versionOrNull)
 
                 KotlinPluginsToolWindowFactory.show(project)
             }

src/main/kotlin/com/github/mr3zee/kotlinPlugins/KotlinPluginsExceptionAnalyzerService.kt

@@ -40,10 +40,11 @@ class KotlinPluginsExceptionAnalyzerService(
     Disposable {
     private val logger by lazy { thisLogger() }
     private val handler: AtomicReference<Handler?> = AtomicReference(null)
-    val supervisorJob = SupervisorJob(scope.coroutineContext.job)
+    private val supervisorJob = scope.coroutineContext + SupervisorJob(scope.coroutineContext.job)
+    private val publisherSync by lazy { project.messageBus.syncPublisher(KotlinPluginStatusUpdater.TOPIC) }
 
     override fun dispose() {
-        supervisorJob.cancel()
+        supervisorJob.job.cancel()
         val handler = handler.getAndSet(null)
         rootLogger().removeHandler(handler)
         handler?.close()
@@ -59,6 +60,7 @@ class KotlinPluginsExceptionAnalyzerService(
             }
 
             project.service<KotlinPluginsExceptionReporter>().start()
+            publisherSync.redraw()
             exceptionHandler.start()
             logger.debug("Exception analyzer started")
             rootLogger().addHandler(exceptionHandler)
@@ -79,15 +81,27 @@ class KotlinPluginsExceptionAnalyzerService(
                 val lookup = reporter.lookFor()
 
                 lookup.entries.find { (jarId, fqNames) ->
-                    exception.stackTrace.any { it.className in fqNames }
+                    var curr: Throwable? = exception
+                    while (curr != null) {
+                        if (curr.stackTrace.any { it.className in fqNames }) {
+                            return@find true
+                        }
+
+                        curr = curr.cause
+                    }
+                    false
                 } ?: continue
 
-                val ids = match(lookup, exception)
-                    .takeIf { it.isNotEmpty() }
-                    ?: return@launch
+                val ids = match(lookup, exception) ?: continue
 
                 logger.debug("Exception detected for $ids: ${exception.message}")
-                reporter.matched(ids.toList(), exception, autoDisable = state.autoDisable)
+
+                reporter.matched(
+                    ids = ids.toList(),
+                    exception = exception,
+                    autoDisable = state.autoDisable,
+                    isProbablyIncompatible = exception.isProbablyIncompatible(),
+                )
             }
         }
 
@@ -141,15 +155,26 @@ class KotlinPluginsExceptionAnalyzerService(
     }
 
     companion object {
-        internal fun match(lookup: Map<JarId, Set<String>>, exception: Throwable): Set<JarId> {
+        internal fun match(lookup: Map<JarId, Set<String>>, exception: Throwable): Set<JarId>? {
             return exception.stackTrace
                 .map { trace ->
                     lookup.entries
                         .filter { (_, fqNames) -> trace.className in fqNames }
                         .map { it.key }.toSet()
                 }
                 .filter { it.isNotEmpty() }
-                .reduce { acc, set -> acc.intersect(set) }
+                .takeIf { it.isNotEmpty() }
+                ?.reduce { acc, set -> acc.intersect(set) }
+                ?: match(lookup, exception.cause ?: return null)
+        }
+
+        private fun Throwable.isProbablyIncompatible(): Boolean {
+            return this is ClassNotFoundException ||
+                    this is NoClassDefFoundError ||
+                    this is IncompatibleClassChangeError ||
+                    this is LinkageError ||
+                    this is ClassCastException ||
+                    this.cause?.isProbablyIncompatible() == true
         }
     }
 }