refactor(terminal): move UIUpdatingWriter to core module #335

Moved UIUpdatingWriter from ext-terminal to core module for better code reuse and organization. Removed redundant errWriter in TerminalSketchProvider.

Author: phodal

exts/ext-terminal/src/main/kotlin/cc/unitmesh/terminal/sketch/UIUpdatingWriter.kt renamed to core/src/main/kotlin/cc/unitmesh/devti/sketch/run/UIUpdatingWriter.kt

@@ -1,4 +1,4 @@
-package cc.unitmesh.terminal.sketch
+package cc.unitmesh.devti.sketch.run
 
 import com.intellij.openapi.application.ApplicationManager
 import java.io.StringWriter
@@ -42,11 +42,11 @@ class UIUpdatingWriter(
         ApplicationManager.getApplication().invokeLater {
             val currentText = stringWriter.toString()
             onTextUpdate(currentText, !isExecuting)
-            
+
             if (checkCollapsed()) {
                 expandPanel()
             }
-            
+
             if (isExecuting) {
                 onPanelUpdate("Execution Results (Running...)", isExecuting)
             } else {
@@ -65,4 +65,4 @@ class UIUpdatingWriter(
     }
 
     fun getContent(): String = stringWriter.toString()
-}
+}

exts/ext-terminal/src/main/kotlin/cc/unitmesh/terminal/sketch/TerminalSketchProvider.kt

@@ -5,6 +5,7 @@ import cc.unitmesh.devti.AutoDevIcons
 import cc.unitmesh.devti.AutoDevNotifications
 import cc.unitmesh.devti.sketch.SketchToolWindow
 import cc.unitmesh.devti.sketch.run.ProcessExecutor
+import cc.unitmesh.devti.sketch.run.UIUpdatingWriter
 import cc.unitmesh.devti.sketch.ui.ExtensionLangSketch
 import cc.unitmesh.devti.sketch.ui.LanguageSketchProvider
 import cc.unitmesh.devti.sketch.ui.code.CodeHighlightSketch
@@ -232,29 +233,14 @@ class TerminalLangSketch(val project: Project, var content: String) : ExtensionL
                 }
             )
 
-            val errWriter = UIUpdatingWriter(
-                onTextUpdate = { text, complete ->
-                    resultSketch.updateViewText(text, complete)
-                },
-                onPanelUpdate = { title, _ ->
-                    collapsibleResultPanel.setTitle(title)
-                },
-                checkCollapsed = {
-                    collapsibleResultPanel.isCollapsed()
-                },
-                expandPanel = {
-                    collapsibleResultPanel.expand()
-                }
-            )
-
             resultSketch.updateViewText("", true)
             stdWriter.setExecuting(true)
 
             AutoDevCoroutineScope.scope(project).launch {
                 val executor = ProcessExecutor(project)
                 try {
                     val dispatcher = PooledThreadExecutor.INSTANCE.asCoroutineDispatcher()
-                    executor.exec(getViewText(), stdWriter, errWriter, dispatcher)
+                    executor.exec(getViewText(), stdWriter, stdWriter, dispatcher)
                     ApplicationManager.getApplication().invokeLater {
                         stdWriter.setExecuting(false)
                         if (collapsibleResultPanel.isCollapsed()) {
@@ -264,11 +250,7 @@ class TerminalLangSketch(val project: Project, var content: String) : ExtensionL
                 } catch (ex: Exception) {
                     ApplicationManager.getApplication().invokeLater {
                         stdWriter.setExecuting(false)
-                        resultSketch.updateViewText(
-                            "${stdWriter.getContent()}\nError: ${ex.message}",
-                            true
-                        )
-
+                        resultSketch.updateViewText("${stdWriter.getContent()}\nError: ${ex.message}", true)
                         collapsibleResultPanel.setTitle("Execution Results (Error)")
                     }
                 }