feat: Improve ApiDashboard lifecycle management and context handling

Author: tangcent

idea-plugin/src/main/kotlin/com/itangcent/idea/plugin/api/dashboard/ApiDashboardPanel.kt

@@ -1,5 +1,6 @@
 package com.itangcent.idea.plugin.api.dashboard
 
+import com.intellij.openapi.Disposable
 import com.intellij.openapi.application.ApplicationManager
 import com.intellij.openapi.fileChooser.FileChooserDescriptorFactory
 import com.intellij.openapi.project.Project
@@ -31,7 +32,6 @@ import java.awt.Dimension
 import java.awt.event.MouseAdapter
 import java.awt.event.MouseEvent
 import java.awt.event.MouseListener
-import java.util.concurrent.Executors
 import javax.swing.*
 import javax.swing.event.DocumentEvent
 import javax.swing.event.DocumentListener
@@ -45,7 +45,7 @@ import javax.swing.tree.TreePath
 import javax.swing.tree.TreeSelectionModel
 import kotlin.concurrent.thread
 
-class ApiDashboardPanel(private val project: Project) : JBPanel<ApiDashboardPanel>(BorderLayout()) {
+class ApiDashboardPanel(private val project: Project) : JBPanel<ApiDashboardPanel>(BorderLayout()), Disposable {
     companion object : Log()
 
     private val disabledFormTableBinder: ParamsTableBinder<FormParam> = DisabledFormTableBinder()
@@ -83,7 +83,6 @@ class ApiDashboardPanel(private val project: Project) : JBPanel<ApiDashboardPane
     private lateinit var service: ApiDashboardService
     private var apis: List<ProjectNodeData> = emptyList()
 
-    private val workThreadPool = Executors.newCachedThreadPool()
     private var currentResponse: HttpResponse? = null
     private var formTableBinder: ParamsTableBinder<FormParam> = disabledFormTableBinder
 
@@ -276,7 +275,7 @@ class ApiDashboardPanel(private val project: Project) : JBPanel<ApiDashboardPane
             val node = apiTree.lastSelectedPathComponent as? DefaultMutableTreeNode
             val userObject = node?.userObject
             if (userObject is ProjectNodeData) {
-                workThreadPool.submit {
+                actionContext.runAsync {
                     refreshProjectNodeData(userObject)
                 }
             }
@@ -1228,4 +1227,9 @@ class ApiDashboardPanel(private val project: Project) : JBPanel<ApiDashboardPane
         }
         return null
     }
+
+    override fun dispose() {
+        searchDebounceTimer.stop()
+        service.dispose()
+    }
 }

idea-plugin/src/main/kotlin/com/itangcent/idea/plugin/api/dashboard/ApiDashboardService.kt

@@ -5,6 +5,8 @@ import com.intellij.openapi.components.Service
 import com.intellij.openapi.components.service
 import com.intellij.openapi.module.ModuleManager
 import com.intellij.openapi.project.Project
+import com.intellij.openapi.project.ProjectManager
+import com.intellij.openapi.project.ProjectManagerListener
 import com.intellij.openapi.project.rootManager
 import com.intellij.openapi.wm.ToolWindowManager
 import com.intellij.psi.*
@@ -95,7 +97,7 @@ class ApiDashboardService(private val project: Project) {
     @Inject
     private lateinit var httpContextCacheHelper: HttpContextCacheHelper
 
-    var actionContext: ActionContext
+    lateinit var actionContext: ActionContext
         private set
 
     private val requestRawInfoBinderFactory: DbBeanBinderFactory<RequestRawInfo> by lazy {
@@ -140,6 +142,19 @@ class ApiDashboardService(private val project: Project) {
 
     init {
         Setup.load(ApiDashboardService::class.java.classLoader)
+        createNewActionContext()
+        
+        // Add project dispose listener
+        project.messageBus.connect().subscribe(ProjectManager.TOPIC, object : ProjectManagerListener {
+            override fun projectClosing(project: Project) {
+                if (project == this@ApiDashboardService.project) {
+                    actionContext.stop()
+                }
+            }
+        })
+    }
+
+    fun createNewActionContext() {
         val builder = ActionContext.builder()
         builder.bindInstance(Project::class, project)
         builder.bind(ClassExporter::class) { it.with(CachedRequestClassExporter::class).singleton() }
@@ -154,6 +169,12 @@ class ApiDashboardService(private val project: Project) {
         actionContext.init(this)
     }
 
+    fun ensureActionContextActive() {
+        if (!::actionContext.isInitialized || actionContext.isStopped()) {
+            createNewActionContext()
+        }
+    }
+
     fun setDashboardPanel(panel: ApiDashboardPanel) {
         this.dashboardPanel = panel
     }
@@ -747,6 +768,12 @@ class ApiDashboardService(private val project: Project) {
             httpClientExporter.export(listOf(request))
         }
     }
+
+    fun dispose() {
+        if (::actionContext.isInitialized && !actionContext.isStopped()) {
+            actionContext.stop()
+        }
+    }
 }
 
 interface DocContainer {

idea-plugin/src/main/kotlin/com/itangcent/idea/plugin/api/dashboard/ApiDashboardToolWindow.kt

@@ -3,11 +3,35 @@ package com.itangcent.idea.plugin.api.dashboard
 import com.intellij.openapi.project.Project
 import com.intellij.openapi.wm.ToolWindow
 import com.intellij.openapi.wm.ToolWindowFactory
+import com.intellij.openapi.wm.ToolWindowManager
+import com.intellij.openapi.wm.ex.ToolWindowManagerListener
+import com.intellij.ui.content.ContentManagerEvent
+import com.intellij.ui.content.ContentManagerListener
 
 class ApiDashboardToolWindow : ToolWindowFactory {
     override fun createToolWindowContent(project: Project, toolWindow: ToolWindow) {
         val dashboardPanel = ApiDashboardPanel(project)
         val content = toolWindow.contentManager.factory.createContent(dashboardPanel, "", false)
+
+        // Add content listener to handle disposal
+        toolWindow.contentManager.addContentManagerListener(object : ContentManagerListener {
+            override fun contentRemoved(event: ContentManagerEvent) {
+                if (event.content == content) {
+                    dashboardPanel.dispose()
+                }
+            }
+        })
+
+        // Add tool window visibility listener
+        project.messageBus.connect().subscribe(ToolWindowManagerListener.TOPIC, object : ToolWindowManagerListener {
+            override fun stateChanged(toolWindowManager: ToolWindowManager) {
+                if (toolWindow.isVisible) {
+                    // When tool window becomes visible, ensure ActionContext is active
+                    ApiDashboardService.getInstance(project).ensureActionContextActive()
+                }
+            }
+        })
+
         toolWindow.contentManager.addContent(content)
     }
 }