fix(refactor): refactor rename lookup manager listener to use custom rename lookup element #129

The rename lookup manager listener has been refactored to use a custom rename lookup element class, which improves the handling of insertions and the rendering of lookup elements with the correct icon. This change also includes the creation of a new file for the custom rename lookup element class, which is now located in the `cc.unitmesh.devti.practise` package.

Author: phodal

src/main/kotlin/cc/unitmesh/devti/practise/CustomRenameLookupElement.kt

@@ -0,0 +1,25 @@
+package cc.unitmesh.devti.practise
+
+import cc.unitmesh.devti.AutoDevIcons
+import com.intellij.codeInsight.completion.InsertionContext
+import com.intellij.codeInsight.lookup.LookupElement
+import com.intellij.codeInsight.lookup.LookupElementPresentation
+import com.intellij.codeInsight.template.impl.TemplateManagerImpl
+
+class CustomRenameLookupElement(val name: String) : LookupElement() {
+    override fun getLookupString(): String = name
+
+    override fun handleInsert(context: InsertionContext) {
+        val editor = context.editor
+        val templateState = TemplateManagerImpl.getTemplateState(editor)
+
+        if (templateState != null && !templateState.isFinished) {
+            templateState.considerNextTabOnLookupItemSelected(this)
+        }
+    }
+
+    override fun renderElement(presentation: LookupElementPresentation) {
+        presentation.icon = AutoDevIcons.Idea
+        super.renderElement(presentation)
+    }
+}

src/main/kotlin/cc/unitmesh/devti/practise/RenameLookupManagerListener.kt

@@ -1,16 +1,13 @@
 package cc.unitmesh.devti.practise
 
-import cc.unitmesh.devti.AutoDevIcons
 import cc.unitmesh.devti.llms.LlmFactory
 import cc.unitmesh.devti.settings.coder.coderSetting
 import cc.unitmesh.devti.statusbar.AutoDevStatus
 import cc.unitmesh.devti.statusbar.AutoDevStatusService
 import cc.unitmesh.devti.util.LLMCoroutineScope
-import com.intellij.codeInsight.completion.InsertionContext
 import com.intellij.codeInsight.completion.PrefixMatcher
 import com.intellij.codeInsight.lookup.*
 import com.intellij.codeInsight.lookup.impl.LookupImpl
-import com.intellij.codeInsight.template.impl.TemplateManagerImpl
 import com.intellij.openapi.application.runInEdt
 import com.intellij.openapi.application.runReadAction
 import com.intellij.openapi.diagnostic.logger
@@ -71,9 +68,11 @@ class RenameLookupManagerListener(val project: Project) : LookupManagerListener
                 }
                 val result = sb.toString()
                 logger.info("result: $result")
-                parseSuggestions(result).filter { it.isNotBlank() }.map {
+                extractSuggestionsFromString(result).filter { it.isNotBlank() }.map {
                         runReadAction {
-                            lookupImpl.addItem(CustomRenameLookupElement(it), PrefixMatcher.ALWAYS_TRUE)
+                            if (!lookupImpl.isLookupDisposed) {
+                                lookupImpl.addItem(CustomRenameLookupElement(it), PrefixMatcher.ALWAYS_TRUE)
+                            }
                         }
                     }
 
@@ -99,29 +98,10 @@ class RenameLookupManagerListener(val project: Project) : LookupManagerListener
         stringJob.start()
     }
 
-    private fun parseSuggestions(result: String) = result.split("\n").map {
+    private fun extractSuggestionsFromString(result: String) = result.split("\n").map {
         it.replace(Regex("^\\d+\\."), "")
             .trim()
             .removeSurrounding("`")
             .removeSuffix("()")
     }
-
-}
-
-class CustomRenameLookupElement(val name: String) : LookupElement() {
-    override fun getLookupString(): String = name
-
-    override fun handleInsert(context: InsertionContext) {
-        val editor = context.editor
-        val templateState = TemplateManagerImpl.getTemplateState(editor)
-
-        if (templateState != null && !templateState.isFinished) {
-            templateState.considerNextTabOnLookupItemSelected(this)
-        }
-    }
-
-    override fun renderElement(presentation: LookupElementPresentation) {
-        presentation.icon = AutoDevIcons.Idea
-        super.renderElement(presentation)
-    }
 }