refactor(devins-lang): refactor reorg conversation #100

Refactored `MessageCodeBlockCharProcessor` to improve readability and maintainability by moving constants to instance fields and adding additional checks for context changes.

In `DevInsProcessProcessor`, removed the `createTempTaskFile` function as it was not being used and replaced it with a TODO comment to indicate that functionality needs to be implemented in the future.

Author: phodal

exts/devins-lang/src/main/kotlin/cc/unitmesh/devti/language/run/flow/DevInsProcessProcessor.kt

@@ -103,12 +103,4 @@ class DevInsProcessProcessor(val project: Project) {
     fun createAgentTasks(): List<DevInFile> {
         TODO()
     }
-
-    /**
-     * Generate DevIns Task file by LLM
-     */
-    fun createTempTaskFile(): DevInFile {
-        // TODO
-        return DevInFile.fromString(project, "")
-    }
 }

src/main/kotlin/com/intellij/temporary/gui/block/MessageCodeBlockCharProcessor.kt

@@ -10,10 +10,8 @@ class Parameters(val char: Char, val charIndex: Int, val fullMessage: String)
 class ContextChange(@JvmField val contextType: MessageBlockType, @JvmField val borderType: BorderType)
 
 class MessageCodeBlockCharProcessor {
-    companion object {
-        const val triggerChar: Char = '`'
-        const val borderBlock: String = "```"
-    }
+    private val triggerChar: Char = '`'
+    private val borderBlock: String = "```"
 
     fun suggestTypeChange(
         parameters: Parameters,
@@ -52,10 +50,12 @@ class MessageCodeBlockCharProcessor {
                 val subSequence = fullMessage.subSequence(charIndex - 3, charIndex + 1)
                 subSequence == "\n$borderBlock"
             }
+
             parameters.char == '\n' && (charIndex - 3) - 1 >= 0 -> {
                 val subSequence = fullMessage.subSequence(charIndex - 4, charIndex)
                 subSequence == "\n$borderBlock"
             }
+
             else -> false
         }
     }