refactor(devti): implement shelf-based change management #352

- Replace `Change` objects with `ShelvedChange` for better change handling
- Create `PlannerResultSummary` class to summarize planner results
- Update `AgentStateService` to use `ShelvedChange` and `FilePatch`
- Remove unnecessary `ChangeListObserver` functionality
- Integrate change recording in `SingleFileDiffSketch`

Author: phodal

core/src/main/kotlin/cc/unitmesh/devti/gui/planner/PlannerResultSummary.kt

@@ -0,0 +1,15 @@
+package cc.unitmesh.devti.gui.planner
+
+import com.intellij.openapi.project.Project
+import com.intellij.openapi.vcs.changes.Change
+
+class PlannerResultSummary(
+    val project: Project,
+    val changes: List<Change>
+) {
+    fun getDiffContent(change: Change) {
+        change.afterRevision?.file?.name
+        change.afterRevision?.file?.path
+        change.afterRevision?.content
+    }
+}

core/src/main/kotlin/cc/unitmesh/devti/observer/ChangeListObserver.kt

@@ -1,37 +1,14 @@
 package cc.unitmesh.devti.observer
 
-import cc.unitmesh.devti.observer.agent.AgentStateService
 import cc.unitmesh.devti.provider.observer.AgentObserver
 import com.intellij.openapi.Disposable
 import com.intellij.openapi.project.Project
-import com.intellij.openapi.vcs.changes.Change
-import com.intellij.openapi.vcs.changes.ChangeList
-import com.intellij.openapi.vcs.changes.ChangeListAdapter
-import com.intellij.openapi.vcs.changes.ChangeListManager
-import com.intellij.util.messages.MessageBusConnection
 
 class ChangeListObserver : AgentObserver, Disposable {
-    private var connection: MessageBusConnection? = null
-
     override fun onRegister(project: Project) {
-        ChangeListManager.getInstance(project).addChangeListListener(object : ChangeListAdapter() {
-            override fun changesAdded(changes: Collection<Change?>?, toList: ChangeList?) {
-                super.changesAdded(changes, toList)
-                if (toList != null) {
-                    project.getService(AgentStateService::class.java).updateChanges(toList.changes)
-                }
-            }
 
-            override fun changesRemoved(changes: Collection<Change?>?, fromList: ChangeList?) {
-                super.changesRemoved(changes, fromList)
-                if (fromList != null) {
-                    project.getService(AgentStateService::class.java).updateChanges(fromList.changes)
-                }
-            }
-        }, this)
     }
 
     override fun dispose() {
-        connection?.disconnect()
     }
 }

core/src/main/kotlin/cc/unitmesh/devti/observer/agent/AgentStateService.kt

@@ -8,13 +8,15 @@ import cc.unitmesh.devti.observer.plan.AgentTaskEntry
 import cc.unitmesh.devti.observer.plan.MarkdownPlanParser
 import cc.unitmesh.devti.observer.plan.PlanUpdateListener
 import cc.unitmesh.devti.settings.AutoDevSettingsState
-import cc.unitmesh.devti.settings.customize.customizeSetting
 import cc.unitmesh.devti.util.parser.MarkdownCodeHelper
 import com.intellij.openapi.application.ApplicationManager
 import com.intellij.openapi.components.Service
 import com.intellij.openapi.diagnostic.logger
+import com.intellij.openapi.diff.impl.patch.FilePatch
 import com.intellij.openapi.project.Project
-import com.intellij.openapi.vcs.changes.Change
+import com.intellij.openapi.vcs.FileStatus
+import com.intellij.openapi.vcs.changes.shelf.ShelvedChange
+import java.nio.file.Path
 
 @Service(Service.Level.PROJECT)
 class AgentStateService(val project: Project) {
@@ -32,9 +34,30 @@ class AgentStateService(val project: Project) {
         logger<AgentStateService>().info("Called agent tools:\n ${state.usedTools.joinToString("\n")}")
     }
 
-    fun updateChanges(changes: Collection<Change?>?) {
-        val allChanges = changes?.filterNotNull()?.map { it } ?: emptyList()
-        state.changes = allChanges.toMutableList()
+    fun addToChange(path: Path, change: FilePatch) {
+        val shelvedChange = createShelvedChange(project, path, change)
+        if (shelvedChange != null) {
+            state.changes.add(shelvedChange.change)
+        }
+    }
+
+    fun createShelvedChange(project: Project, patchPath: Path, patch: FilePatch): ShelvedChange? {
+        val beforeName: String? = patch.beforeName
+        val afterName: String? = patch.afterName
+        if (beforeName == null || afterName == null) {
+            logger<AgentStateService>().warn("Failed to parse the file patch: [$patchPath]:$patch")
+            return null
+        }
+
+        val status = if (patch.isNewFile) {
+            FileStatus.ADDED
+        } else if (patch.isDeletedFile) {
+            FileStatus.DELETED
+        } else {
+            FileStatus.MODIFIED
+        }
+
+        return ShelvedChange.create(project, patchPath, beforeName, afterName, status)
     }
 
     fun buildOriginIntention(): String? {

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

@@ -2,6 +2,7 @@ package cc.unitmesh.devti.sketch.ui.patch
 
 import cc.unitmesh.devti.AutoDevBundle
 import cc.unitmesh.devti.AutoDevIcons
+import cc.unitmesh.devti.observer.agent.AgentStateService
 import cc.unitmesh.devti.settings.coder.coderSetting
 import cc.unitmesh.devti.sketch.AutoSketchMode
 import cc.unitmesh.devti.sketch.lint.SketchCodeInspection
@@ -159,6 +160,11 @@ class SingleFileDiffSketch(
         mainPanel.add(myHeaderPanel)
         mainPanel.add(contentPanel)
 
+        invokeLater {
+            myProject.getService<AgentStateService>(AgentStateService::class.java)
+                .addToChange(currentFile.toNioPath(), patch)
+        }
+
         if (myProject.coderSetting.state.enableDiffViewer && appliedPatch?.status == ApplyPatchStatus.SUCCESS) {
             invokeLater {
                 val diffPanel = createDiffViewer(oldCode, newCode)
@@ -182,23 +188,23 @@ class SingleFileDiffSketch(
             override fun requestFocusInWindow() = Unit
         }, diffRequest)
         diffViewer.init()
-        
+
         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
     }