refactor: Extract RunOutputDock to reusable composable to avoid duplication

Author: phodal

mpp-ui/src/commonMain/kotlin/cc/unitmesh/devins/ui/compose/agent/CodingAgentPage.kt

@@ -147,7 +147,7 @@ fun CodingAgentPage(
             llmService = llmService
         )
     }
-    
+
     val runConfigState by runConfigViewModel.state.collectAsState()
     val runConfigs by runConfigViewModel.configs.collectAsState()
     val defaultRunConfig by runConfigViewModel.defaultConfig.collectAsState()
@@ -157,7 +157,7 @@ fun CodingAgentPage(
     val runAnalysisLog by runConfigViewModel.analysisLog.collectAsState()
 
     var showRunOutputDock by remember { mutableStateOf(false) }
-    
+
     // Track which mode the dock is in (analysis or run output)
     val isAnalyzing = runConfigState == cc.unitmesh.agent.runconfig.RunConfigState.ANALYZING
     val dockTitle = if (isAnalyzing) "AI Analysis" else "Run Output"
@@ -170,6 +170,27 @@ fun CodingAgentPage(
         }
     }
 
+    // Extracted RunOutputDock composable to avoid duplication
+    val runOutputDockContent: @Composable () -> Unit = {
+        RunOutputDock(
+            isVisible = showRunOutputDock,
+            title = dockTitle,
+            output = dockOutput,
+            isRunning = isRunning,
+            onClear = {
+                runConfigViewModel.clearOutput()
+                runConfigViewModel.clearAnalysisLog()
+            },
+            onClose = { showRunOutputDock = false },
+            onStop = if (!isAnalyzing) {
+                { runConfigViewModel.stopRunning() }
+            } else null,
+            modifier = Modifier
+                .fillMaxWidth()
+                .padding(horizontal = 12.dp)
+        )
+    }
+
     // Main content
     Box(modifier = modifier.fillMaxSize()) {
     if (isTreeViewVisibleState) {
@@ -260,23 +281,7 @@ fun CodingAgentPage(
                     when (selectedAgentType) {
                         AgentType.LOCAL_CHAT,
                         AgentType.CODING -> {
-                            RunOutputDock(
-                                isVisible = showRunOutputDock,
-                                title = dockTitle,
-                                output = dockOutput,
-                                isRunning = isRunning,
-                                onClear = { 
-                                    runConfigViewModel.clearOutput()
-                                    runConfigViewModel.clearAnalysisLog()
-                                },
-                                onClose = { showRunOutputDock = false },
-                                onStop = if (!isAnalyzing) { 
-                                    { runConfigViewModel.stopRunning() }
-                                } else null,
-                                modifier = Modifier
-                                    .fillMaxWidth()
-                                    .padding(horizontal = 12.dp)
-                            )
+                            runOutputDockContent()
                             DevInEditorInput(
                                 initialText = "",
                                 placeholder = "Describe your coding task...",
@@ -484,23 +489,7 @@ fun CodingAgentPage(
                     )
                 }
 
-            RunOutputDock(
-                isVisible = showRunOutputDock,
-                title = dockTitle,
-                output = dockOutput,
-                isRunning = isRunning,
-                onClear = { 
-                    runConfigViewModel.clearOutput()
-                    runConfigViewModel.clearAnalysisLog()
-                },
-                onClose = { showRunOutputDock = false },
-                onStop = if (!isAnalyzing) { 
-                    { runConfigViewModel.stopRunning() }
-                } else null,
-                modifier = Modifier
-                    .fillMaxWidth()
-                    .padding(horizontal = 12.dp)
-            )
+            runOutputDockContent()
             DevInEditorInput(
                 initialText = "",
                 placeholder = "Describe your coding task...",

mpp-ui/src/commonMain/kotlin/cc/unitmesh/devins/ui/compose/agent/artifact/ArtifactMessageList.kt

@@ -22,7 +22,7 @@ import kotlinx.coroutines.launch
 
 /**
  * Custom message list optimized for artifact generation.
- * 
+ *
  * Unlike AgentMessageList which shows full code details, this component:
  * - Shows user messages compactly
  * - Displays streaming output in a height-limited scrollable code block
@@ -114,7 +114,7 @@ fun ArtifactMessageList(
             userScrolledAway = false
         }
     }
-    
+
     LazyColumn(
         state = listState,
         modifier = modifier.padding(horizontal = 12.dp),
@@ -138,7 +138,7 @@ fun ArtifactMessageList(
                 }
             }
         }
-        
+
         // Show thinking block if active
         if (renderer.isThinking && renderer.currentThinkingOutput.isNotEmpty()) {
             item(key = "thinking") {
@@ -149,7 +149,7 @@ fun ArtifactMessageList(
                 )
             }
         }
-        
+
         // Show streaming artifact content
         if (streamingArtifact != null) {
             item(key = "streaming-artifact") {
@@ -189,7 +189,7 @@ private fun ArtifactMessageItem(
     content: String
 ) {
     val isUser = role == MessageRole.USER
-    
+
     if (isUser) {
         // User messages - right aligned, limited width
         Row(
@@ -204,7 +204,7 @@ private fun ArtifactMessageItem(
                     bottomStart = 12.dp,
                     bottomEnd = 4.dp
                 ),
-                modifier = Modifier.widthIn(max = 400.dp)
+                modifier = Modifier.fillMaxWidth()
             ) {
                 Text(
                     text = content,
@@ -268,7 +268,7 @@ private fun StreamingOutputBlock(
 ) {
     // Extract artifact info if present in streaming output
     val hasArtifactTag = content.contains("<autodev-artifact")
-    
+
     Surface(
         color = MaterialTheme.colorScheme.surfaceVariant.copy(alpha = 0.3f),
         shape = RoundedCornerShape(8.dp),
@@ -291,7 +291,7 @@ private fun StreamingOutputBlock(
                     color = MaterialTheme.colorScheme.onSurface
                 )
             }
-            
+
             if (!hasArtifactTag && content.length < 500) {
                 Spacer(modifier = Modifier.height(8.dp))
                 Surface(