feat(diff): improve diff viewer layout and auto-command handling #352

- Wrap diff viewer in panel with controlled height (25% of parent)
- Refactor auto-commands into dynamic builder function
- Remove unused imports and simplify code structure

The changes improve UI consistency and make command handling more flexible.

Author: phodal

core/src/main/kotlin/cc/unitmesh/devti/sketch/AutoSketchMode.kt

@@ -5,16 +5,15 @@ import cc.unitmesh.devti.devin.dataprovider.BuiltinCommand.*
 import cc.unitmesh.devti.observer.agent.AgentStateService
 import cc.unitmesh.devti.provider.devins.LanguageProcessor
 import cc.unitmesh.devti.provider.toolchain.ToolchainFunctionProvider
+import cc.unitmesh.devti.settings.coder.coderSetting
 import cc.unitmesh.devti.util.parser.CodeFence
 import com.intellij.openapi.application.ApplicationManager
-import com.intellij.openapi.application.invokeLater
 import com.intellij.openapi.application.runReadAction
 import com.intellij.openapi.components.Service
 import com.intellij.openapi.components.service
 import com.intellij.openapi.diagnostic.logger
 import com.intellij.openapi.project.Project
 import com.intellij.psi.PsiFileFactory
-import kotlinx.coroutines.runBlocking
 
 @Service(Service.Level.PROJECT)
 class AutoSketchMode(val project: Project) {
@@ -37,7 +36,7 @@ class AutoSketchMode(val project: Project) {
 
         val language = CodeFence.findLanguage("DevIn")
         commands += devinCodeFence.mapNotNull {
-            val psiFile = runReadAction {  PsiFileFactory.getInstance(project).createFileFromText(language, it.text) }
+            val psiFile = runReadAction { PsiFileFactory.getInstance(project).createFileFromText(language, it.text) }
                 ?: return@mapNotNull null
 
             LanguageProcessor.devin()?.transpileCommand(project, psiFile) ?: emptyList()
@@ -71,27 +70,31 @@ class AutoSketchMode(val project: Project) {
         listener?.manualSend(text)
     }
 
-    private fun hasReadCommand(fence: CodeFence): Boolean = AUTOABLE_COMMANDS.any { command ->
+    private fun hasReadCommand(fence: CodeFence): Boolean = buildAutoCommands().any { command ->
         fence.text.contains("/" + command.commandName + ":")
     }
 
-    val AUTOABLE_COMMANDS =
-        setOf(
+    private fun buildAutoCommands(): Set<BuiltinCommand> {
+        val of = mutableSetOf(
             DIR,
             LOCAL_SEARCH,
             FILE,
             REV,
             STRUCTURE,
             SYMBOL,
-            DATABASE, // should be handle in run sql
             RELATED,
             RIPGREP_SEARCH,
             RULE,
             BROWSE,
-            PATCH,
-            WRITE
         )
 
+        of += setOf(
+            PATCH, DATABASE, WRITE
+        )
+
+        return of
+    }
+
     private suspend fun hasToolchainFunctionCommand(fence: CodeFence): Boolean {
         val toolchainCmds = ToolchainFunctionProvider.all().map { it.funcNames() }.flatten()
         return toolchainCmds.any {

core/src/main/kotlin/cc/unitmesh/devti/sketch/ui/patch/SingleFileDiffSketch.kt

@@ -180,7 +180,24 @@ class SingleFileDiffSketch(
             override fun requestFocusInWindow() = Unit
         }, diffRequest)
         diffViewer.init()
-        return diffViewer.component
+        
+        val wrapperPanel = JPanel(BorderLayout())
+        wrapperPanel.add(diffViewer.component, BorderLayout.CENTER)
+        
+        // Set preferred height to 25% of parent (will be determined when laid out)
+        // with a minimum of 200 pixels
+        wrapperPanel.preferredSize = java.awt.Dimension(
+            wrapperPanel.preferredSize.width,
+            maxOf(200, (mainPanel.height * 0.25).toInt())
+        )
+        
+        // Set maximum height to prevent excessive growth
+        wrapperPanel.maximumSize = java.awt.Dimension(
+            Int.MAX_VALUE,
+            maxOf(200, (mainPanel.height * 0.25).toInt())
+        )
+        
+        return wrapperPanel
     }
 
     private fun createActionButtons(
@@ -381,3 +398,4 @@ fun saveText(file: VirtualFile, text: String) {
         }
     }
 }
+

core/src/main/resources/messages/AutoDevBundle_en.properties

@@ -133,6 +133,7 @@ settings.autodev.coder.enableAutoRunTerminal=Enable auto run terminal
 settings.autodev.coder.enableAutoLintCode=Enable auto lint patch code
 settings.autodev.coder.enableRenderWebview=Enable render chat in WebView
 settings.autodev.coder.enableAutoScrollInSketch=Enable AutoScroll in Sketch
+settings.autodev.coder.enableDiffViewer=Enable Diff Viewer in Sketch
 shell.command.suggestion.action.default.text=How to check out a branch?
 batch.nothing.to.testing=Nothing to AutoTest
 intentions.chat.code.test.verify=Verify test

core/src/main/resources/messages/AutoDevBundle_zh.properties

@@ -125,6 +125,7 @@ settings.autodev.coder.enableAutoRunTerminal=启用自动运行终端（有风
 settings.autodev.coder.enableAutoLintCode=启用自动修复 Lint 代码
 settings.autodev.coder.enableRenderWebview=在聊天页面中启用渲染 WebView
 settings.autodev.coder.enableAutoScrollInSketch=在 Sketch 中启用自动滚动
+settings.autodev.coder.enableDiffViewer=在 Sketch 中启用 Diff 视图
 shell.command.suggestion.action.default.text=如何创建一个新的分支?
 batch.nothing.to.testing=没有要 AutoTest 的内容
 intentions.chat.code.test.verify=验证测试中