Merge pull request #2552 from digma-ai/fix-tool-window-flow

Change the flow of window opening

Author: kshmidt-digma

ide-common/src/main/kotlin/org/digma/intellij/plugin/scope/ScopeManager.kt

@@ -28,7 +28,7 @@ class ScopeManager(private val project: Project) {
     }
 
     fun changeToHome(
-        isCalledFromReact: Boolean = false,
+        openMainPanel: Boolean = false,
         scopeContext: ScopeContext? = null,
         environmentId: String? = null
     ) {
@@ -47,7 +47,7 @@ class ScopeManager(private val project: Project) {
             if (!PersistenceService.getInstance().isFirstWizardLaunch()) {
                 MainToolWindowCardsController.getInstance(project).closeCoveringViewsIfNecessary()
 
-                if (!isCalledFromReact) {
+                if (openMainPanel) {
                     // if react called changeToHome it's ok not to show the tool window, usually its on connection events.
                     ToolWindowShower.getInstance(project).showToolWindow()
                 }

src/main/kotlin/org/digma/intellij/plugin/ui/jcef/BaseMessageRouterHandler.kt

@@ -399,7 +399,6 @@ abstract class BaseMessageRouterHandler(protected val project: Project) : Common
         payload?.let { pl ->
 
             val changeScopeMessage = jsonToObject(pl, ChangeScopeMessage::class.java)
-
             changeScopeMessage.span?.spanCodeObjectId?.let { spanId ->
                 val spanScope = SpanScope(spanId)
                 ScopeManager.getInstance(project).changeScope(
@@ -408,7 +407,7 @@ abstract class BaseMessageRouterHandler(protected val project: Project) : Common
                     environmentId = changeScopeMessage.environmentId
                 )
             } ?: ScopeManager.getInstance(project).changeToHome(
-                isCalledFromReact = true,
+                openMainPanel = changeScopeMessage.openMainPanel ?: false,
                 scopeContext = changeScopeMessage.context,
                 environmentId = changeScopeMessage.environmentId
             )

src/main/kotlin/org/digma/intellij/plugin/ui/jcef/model/ScopeMessages.kt

@@ -20,12 +20,14 @@ data class ChangeScopeMessage
 @ConstructorProperties(
     "span",
     "environmentId",
-    "context"
+    "context",
+    "openMainPanel"
 )
 constructor(
     val span: ChangeScopeSpan? = null,
     val environmentId: String? = null,
-    val context: ScopeContext? = null
+    val context: ScopeContext? = null,
+    val openMainPanel: Boolean? = null
 )