refactor(agenttool): simplify AgentTool structure and remove unused files #308

- Removed unused files: BrowseTool.kt, AgentToolContext.kt, AgentToolResult.kt, and BrowseToolTest.kt.
- Simplified AgentTool interface to a data class with commandName, description, and example.
- Updated SketchToolchainProvider to use the new AgentTool data class.
- Modified BridgeRunContext to use BridgeToolProvider for tool list collection.
- Added new Browse.kt class for parsing HTML.
- Updated XML and Kotlin files to reflect the new structure and dependencies.

Author: phodal

core/src/main/kotlin/cc/unitmesh/devti/agenttool/AgentTool.kt

@@ -1,18 +1,10 @@
 package cc.unitmesh.devti.agenttool
 
-import com.intellij.openapi.extensions.ExtensionPointName
-
-interface AgentTool {
-    val name: String
-    val description: String
-    fun execute(context: AgentToolContext): AgentToolResult
-
-    // extension point
-    companion object {
-        val EP_NAME = ExtensionPointName<AgentTool>("devins.agentTool")
-
-        fun allTools(): List<AgentTool> {
-            return EP_NAME.extensionList
-        }
-    }
-}
+data class AgentTool(val commandName: String, val description: String, val example: String) {
+    override fun toString(): String =
+        """<tool>name: ${commandName}, desc: $description, example:
+<devin>
+$example
+</devin>
+</tool>"""
+}

core/src/main/kotlin/cc/unitmesh/devti/agenttool/AgentToolContext.kt

@@ -0,0 +1,19 @@
+package cc.unitmesh.devti.agenttool.browse
+
+import org.jsoup.Jsoup
+import org.jsoup.nodes.Document
+
+class Browse {
+    companion object {
+        /**
+         * Doc for parseHtml
+         *
+         * Intellij API: [com.intellij.inspectopedia.extractor.utils.HtmlUtils.cleanupHtml]
+         */
+        fun parse(url: String): DocumentContent {
+            val doc: Document = Jsoup.connect(url).get()
+            return DocumentCleaner().cleanHtml(doc)
+        }
+    }
+}
+

core/src/main/kotlin/cc/unitmesh/devti/agenttool/AgentToolResult.kt

@@ -73,7 +73,7 @@ data class BridgeRunContext(
                 relatedFiles = emptyList(),
                 userInput = input,
                 workspace = workspace(project),
-                toolList = SketchToolchainProvider.collect(project).joinToString("\n"),
+                toolList = BridgeToolProvider.collect(project),
                 shell = ShellUtil.detectShells().firstOrNull() ?: "/bin/bash",
                 frameworkContext = runBlocking {
                     return@runBlocking ChatContextProvider.collectChatContextList(project, creationContext)

core/src/main/kotlin/cc/unitmesh/devti/agenttool/browse/Browse.kt

@@ -1,25 +1,17 @@
 package cc.unitmesh.devti.sketch
 
+import cc.unitmesh.devti.agenttool.AgentTool
 import com.intellij.openapi.extensions.ExtensionPointName
 import com.intellij.openapi.project.Project
 
-data class Toolchain(val commandName: String, val description: String, val example: String) {
-    override fun toString(): String =
-        """<tool>name: ${commandName}, desc: $description, example:
-<devin>
-$example
-</devin>
-</tool>"""
-}
-
 interface SketchToolchainProvider {
-    fun collect(): List<Toolchain>
+    fun collect(): List<AgentTool>
 
     companion object {
         private val EP_NAME: ExtensionPointName<SketchToolchainProvider> =
             ExtensionPointName.create("cc.unitmesh.sketchToolchainProvider")
 
-        fun collect(project: Project): List<Toolchain> {
+        fun collect(project: Project): List<AgentTool> {
             return EP_NAME.extensionList.flatMap {
                 it.collect()
             }

core/src/main/kotlin/cc/unitmesh/devti/agenttool/browse/BrowseTool.kt

@@ -2,18 +2,18 @@ package cc.unitmesh.devti.language.compiler
 
 import cc.unitmesh.devti.devin.dataprovider.BuiltinCommand
 import cc.unitmesh.devti.sketch.SketchToolchainProvider
-import cc.unitmesh.devti.sketch.Toolchain
+import cc.unitmesh.devti.agenttool.AgentTool
 
 class DevInsSketchToolchainProvider : SketchToolchainProvider {
-    override fun collect(): List<Toolchain> {
+    override fun collect(): List<AgentTool> {
         /// we need to ignore some bad case for llm
         return BuiltinCommand.all()
             .filter {
                 it.enableInSketch
             }
             .map {
             val example = BuiltinCommand.example(it)
-            Toolchain(it.commandName, it.description, example)
+            AgentTool(it.commandName, it.description, example)
         }
     }
 }

core/src/main/kotlin/cc/unitmesh/devti/bridge/BridgeRunContext.kt

@@ -1,6 +1,6 @@
 package cc.unitmesh.devti.language.compiler.exec
 
-import cc.unitmesh.devti.agenttool.browse.BrowseTool
+import cc.unitmesh.devti.agenttool.browse.Browse
 import cc.unitmesh.devti.devin.InsCommand
 import cc.unitmesh.devti.devin.dataprovider.BuiltinCommand
 import com.intellij.openapi.project.Project
@@ -9,7 +9,7 @@ class BrowseInsCommand(val myProject: Project, private val prop: String) : InsCo
     override val commandName: BuiltinCommand = BuiltinCommand.BROWSE
 
     override suspend fun execute(): String? {
-        val parse = BrowseTool.parse(prop)
+        val parse = Browse.parse(prop)
         return parse.body
     }
 }

core/src/main/kotlin/cc/unitmesh/devti/sketch/SketchToolchainProvider.kt

@@ -55,11 +55,6 @@
                          implementationClass="cc.unitmesh.devti.language.lints.DevInsDuplicateAgentInspection"/>
     </extensions>
 
-    <extensionPoints>
-        <extensionPoint qualifiedName="devins.agentTool"
-                        interface="cc.unitmesh.devti.agenttool.AgentTool"
-                        dynamic="true"/>
-    </extensionPoints>
 
     <actions>
         <action id="runDevInsFileAction"