feat(devins-lang): add support for line info in commands

This commit adds support for line information in commands by introducing a new `LineInfo` class and refactoring the `fromString` method to use a regular expression for parsing input strings. The `LineInfo` class now includes start and end line numbers, as well as start and end column numbers, and the `fromString` method has been updated to handle these additional fields. Additionally, several new tests have been added to ensure that the `fromString` method correctly parses strings in various formats.

Author: phodal

exts/devins-lang/src/main/kotlin/cc/unitmesh/devti/language/compiler/model/LineInfo.kt

@@ -1,33 +1,30 @@
 package cc.unitmesh.devti.language.compiler.model
 
-data class LineInfo(val startLine: Int, val endLine: Int) {
+data class LineInfo(
+    val startLine: Int,
+    val endLine: Int,
+    val startColumn: Int = 0,
+    val endColumn: Int = 0
+) {
     companion object {
+        private val regex = Regex("""L(\d+)(?:C(\d+))?(?:-L(\d+)(?:C(\d+))?)?""")
+
         /**
-         * Convert a string to a TextRange, if possible.
-         * format: "filepath#L1-L12" means from line 1 to line 12
+         * Convert a string to a `TextRange`, if possible. The string should be in the format: "filepath#L1-L12",
+         * where "filepath" is the path to the file, "#" is a hash symbol, "L1-L12" is a range of lines from line 1 to line 12.
+         *
+         * @param string The string to convert, in the format "filepath#L1-L12".
+         * @return A `LineInfo` object representing the range of lines, or `null` if the string is not in the correct format.
          */
-        fun fromString(string: String): LineInfo? {
-            val lineRange = string.substringAfter('#').split('-')
-            if (lineRange.size != 2) {
-                return null
-            }
-
-            val start = lineRange[0].substringAfter('L')
-            val end = lineRange[1].substringAfter('L')
-
-            // use regex to check if the start is a number
-            if (!start.matches(Regex("\\d+"))) {
-                return null
-            }
-
-            if (!end.matches(Regex("\\d+"))) {
-                return null
-            }
+        fun fromString(input: String): LineInfo? {
+            val matchResult = regex.find(input) ?: return null
 
-            val startLine = start.toInt()
-            val endLine = end.toInt()
+            val startLine = matchResult.groupValues[1].toIntOrNull() ?: return null
+            val startColumn = matchResult.groupValues[2].toIntOrNull() ?: 0
+            val endLine = matchResult.groupValues[3].toIntOrNull() ?: return null
+            val endColumn = matchResult.groupValues[4].toIntOrNull() ?: 0
 
-            return LineInfo(startLine, endLine)
+            return LineInfo(startLine, endLine, startColumn, endColumn)
         }
     }
 

exts/devins-lang/src/test/kotlin/cc/unitmesh/devti/language/compiler/data/LineInfoTest.kt

@@ -54,4 +54,46 @@ class LineInfoTest {
         // then
         assertEquals(null, result)
     }
-}
+
+    // L1C0-L2C0
+    @Test
+    fun should_createLineInfo_when_validStringWithColumnGiven() {
+        // given
+        val validString = "filepath#L1C0-L2C0"
+
+        // when
+        val result = LineInfo.fromString(validString)
+
+        // then
+        val expected = LineInfo(1, 2, 0, 0)
+        assertEquals(expected, result)
+    }
+
+    // L1-L2C0
+    @Test
+    fun should_createLineInfo_when_validStringWithEndColumnGiven() {
+        // given
+        val validString = "filepath#L1-L2C0"
+
+        // when
+        val result = LineInfo.fromString(validString)
+
+        // then
+        val expected = LineInfo(1, 2, 0, 0)
+        assertEquals(expected, result)
+    }
+
+    // L2C0-L3
+    @Test
+    fun should_createLineInfo_when_validStringWithStartColumnGiven() {
+        // given
+        val validString = "filepath#L2C0-L3"
+
+        // when
+        val result = LineInfo.fromString(validString)
+
+        // then
+        val expected = LineInfo(2, 3, 0, 0)
+        assertEquals(expected, result)
+    }
+}