feat(parser): add GitHub-style TODO support in MarkdownPlanParser #331

- Added support for parsing GitHub-style TODO items with checkboxes.
- Skip recursive processing for list nodes to avoid double-processing.
- Updated task completion status check to include GitHub-style checkboxes.
- Added corresponding unit tests for the new functionality.

Author: phodal

core/src/main/kotlin/cc/unitmesh/devti/sketch/ui/plan/MarkdownPlanParser.kt

@@ -27,6 +27,8 @@ object MarkdownPlanParser {
     private val LOG = logger<MarkdownPlanParser>()
     private val ROOT_ELEMENT_TYPE = IElementType("ROOT")
     private val CHECKMARK = "✓"
+    private val GITHUB_TODO_PATTERN = Regex("^\\s*-\\s*\\[\\s*([xX]?)\\s*\\]\\s*(.*)")
+    private val GITHUB_TODO_CHECKED = listOf("x", "X")
 
     /**
      * 解析markdown文本为计划项列表
@@ -131,13 +133,19 @@ object MarkdownPlanParser {
                                 }
                             }
                         }
+                        // Skip recursive processing for ORDERED_LIST nodes since we've already processed them
+                        // Don't call super.visitNode for this type to avoid double-processing
                     }
                     MarkdownElementTypes.UNORDERED_LIST -> {
                         processTaskItems(node, content, currentSectionItems)
+                        // Skip recursive processing for UNORDERED_LIST nodes 
+                        // Don't call super.visitNode for this type to avoid double-processing
+                    }
+                    else -> {
+                        // Only continue recursion for other node types
+                        super.visitNode(node)
                     }
                 }
-
-                super.visitNode(node)
             }
         })
 
@@ -179,10 +187,28 @@ object MarkdownPlanParser {
                     taskText
                 }
 
-                // Process task text and retain the checkmark in the text
-                val cleanTaskText = taskFirstLine.replace(Regex("^[\\-\\*]\\s+"), "").trim()
-                if (cleanTaskText.isNotEmpty()) {
-                    itemsList.add(cleanTaskText)
+                // Check for GitHub style TODO
+                val githubTodoMatch = GITHUB_TODO_PATTERN.find(taskFirstLine)
+                if (githubTodoMatch != null) {
+                    // Extract the task text and preserve the checkbox status
+                    val checkState = githubTodoMatch.groupValues[1]
+                    val todoText = githubTodoMatch.groupValues[2].trim()
+                    val isCompleted = checkState in GITHUB_TODO_CHECKED
+                    
+                    // Add the task with the proper completion marker
+                    val formattedTask = if (isCompleted) {
+                        "[$CHECKMARK] $todoText"
+                    } else {
+                        "[ ] $todoText"
+                    }
+                    
+                    itemsList.add(formattedTask)
+                } else {
+                    // Process task text and retain the checkmark in the text (original behavior)
+                    val cleanTaskText = taskFirstLine.replace(Regex("^[\\-\\*]\\s+"), "").trim()
+                    if (cleanTaskText.isNotEmpty()) {
+                        itemsList.add(cleanTaskText)
+                    }
                 }
             }
         }
@@ -206,7 +232,7 @@ data class PlanItem(
     init {
         // Parse task completion status for each task
         tasks.forEachIndexed { index, task ->
-            taskCompleted[index] = task.contains("✓")
+            taskCompleted[index] = task.contains("✓") || Regex("\\[\\s*([xX])\\s*\\]").containsMatchIn(task)
         }
     }
 }

core/src/test/kotlin/cc/unitmesh/devti/sketch/ui/plan/MarkdownPlanParserTest.kt

@@ -143,4 +143,47 @@ class MarkdownPlanParserTest {
             "控制层：BlogController添加新端点 ✓"
         )
     }
+
+    @Test
+    fun should_return_correct_items() {
+        // Given
+        val markdownContent = """
+1. **分析现有代码结构**：
+   - BlogService 中的 `deleteBlog` 方法目前只支持按 ID 删除
+   - BlogPost 实体类中的 author 字段类型为 String，但 DTO 中的 author 是 Author 对象类型，存在映射不一致
+   - Repository 层使用 CrudRepository 需要扩展自定义删除方法
+
+2. **数据库字段确认**：
+   - 需要确认 BlogPost 表实际存储的 author 字段类型（当前代码显示为 String 类型）
+
+3. **功能实现步骤**：
+   - [ ] 在 BlogRepository 添加按作者删除的方法
+   - [ ] 扩展 BlogService 添加 deleteByAuthor 方法
+   - [ ] 在 BlogController 添加新的 DELETE 端点
+   - [ ] 修复 DTO 与实体类的 author 字段类型一致性
+   - [ ] 添加 Swagger 接口文档注解
+   - [ ] 补充单元测试
+
+4. **异常处理**：
+   - 处理不存在的作者删除请求
+   - 添加事务管理注解
+   - 统一返回结果格式
+        """.trimIndent()
+
+        // When
+        val planItems = MarkdownPlanParser.parse(markdownContent)
+
+        // Then
+        assertThat(planItems).hasSize(4)
+        assertThat(planItems[0].title).isEqualTo("**分析现有代码结构**：")
+        // test markdown
+        assertThat(planItems[2].tasks).containsExactly(
+            "[ ] 在 BlogRepository 添加按作者删除的方法",
+            "[ ] 扩展 BlogService 添加 deleteByAuthor 方法",
+            "[ ] 在 BlogController 添加新的 DELETE 端点",
+            "[ ] 修复 DTO 与实体类的 author 字段类型一致性",
+            "[ ] 添加 Swagger 接口文档注解",
+            "[ ] 补充单元测试"
+        )
+    }
 }