feat(ui): add ACP agent configuration dialog and page #535

- Add AcpAgentConfigDialog for managing external ACP agents (Kimi CLI, Claude CLI, etc.)
- Add AcpAgentPage as main UI for interacting with external ACP agents
- Support agent configuration, connection status, and streaming response rendering
- Integrate with existing config system and timeline rendering

Author: phodal

mpp-core/src/commonMain/kotlin/cc/unitmesh/agent/AgentType.kt

@@ -10,6 +10,7 @@ package cc.unitmesh.agent
  * - KNOWLEDGE: Document reader mode for AI-native document reading
  * - CHAT_DB: Database chat mode for text-to-SQL interactions
  * - REMOTE: Remote agent connected to mpp-server
+ * - CUSTOM_AGENT: External ACP (Agent Client Protocol) agent (e.g., Kimi CLI, Claude CLI)
  */
 enum class AgentType {
     /**
@@ -51,7 +52,14 @@ enum class AgentType {
      * Artifact mode - generate reversible, executable artifacts (HTML/JS, Python scripts)
      * Similar to Claude's Artifacts system
      */
-    ARTIFACT;
+    ARTIFACT,
+
+    /**
+     * Custom ACP agent mode - connects to external ACP-compliant agents via stdio.
+     * Examples: Kimi CLI, Claude CLI (--acp), Gemini CLI (--acp), or any ACP agent.
+     * When active, all interaction goes through the external agent; local LLM is not used.
+     */
+    CUSTOM_AGENT;
 
     fun getDisplayName(): String = when (this) {
         LOCAL_CHAT -> "Chat"
@@ -62,6 +70,7 @@ enum class AgentType {
         REMOTE -> "Remote"
         WEB_EDIT -> "WebEdit"
         ARTIFACT -> "Artifact"
+        CUSTOM_AGENT -> "Custom Agent"
     }
 
     companion object {
@@ -75,6 +84,7 @@ enum class AgentType {
                 "chatdb", "database" -> CHAT_DB
                 "webedit", "web" -> WEB_EDIT
                 "artifact", "unit" -> ARTIFACT
+                "customagent", "custom_agent", "acp" -> CUSTOM_AGENT
                 else -> LOCAL_CHAT
             }
         }

mpp-core/src/commonMain/kotlin/cc/unitmesh/config/ConfigFile.kt

@@ -29,6 +29,18 @@ import kotlinx.serialization.Serializable
  *   url: "http://localhost:8080"
  *   enabled: false
  *   useServerConfig: false
+ * acpAgents:
+ *   kimi:
+ *     name: "Kimi CLI"
+ *     command: "kimi"
+ *     args: "--acp"
+ *     env: ""
+ *   claude:
+ *     name: "Claude CLI"
+ *     command: "claude"
+ *     args: "--acp"
+ *     env: ""
+ * activeAcpAgent: kimi
  * ```
  */
 @Serializable
@@ -41,7 +53,9 @@ public data class ConfigFile(
     val agentType: String? = "Local", // "Local" or "Remote" - which agent mode to use
     val lastWorkspace: WorkspaceInfo? = null, // Last opened workspace information
     val issueTracker: IssueTrackerConfig? = null, // Issue tracker configuration
-    val cloudStorage: CloudStorageConfig? = null // Cloud storage for multimodal image upload
+    val cloudStorage: CloudStorageConfig? = null, // Cloud storage for multimodal image upload
+    val acpAgents: Map<String, AcpAgentConfig>? = null, // ACP agent configurations
+    val activeAcpAgent: String? = null // Currently active ACP agent key
 )
 
 /**
@@ -118,6 +132,58 @@ data class CloudStorageConfig(
     }
 }
 
+/**
+ * ACP (Agent Client Protocol) agent configuration.
+ *
+ * Defines an external ACP-compliant agent that can be spawned as a child process
+ * and communicated with via JSON-RPC over stdio.
+ *
+ * Example config.yaml:
+ * ```yaml
+ * acpAgents:
+ *   kimi:
+ *     name: "Kimi CLI"
+ *     command: "kimi"
+ *     args: "--acp"
+ *     env: "KIMI_API_KEY=xxx"
+ *   claude:
+ *     name: "Claude CLI"
+ *     command: "claude"
+ *     args: "--acp"
+ *     env: ""
+ * activeAcpAgent: kimi
+ * ```
+ */
+@Serializable
+data class AcpAgentConfig(
+    val name: String = "",
+    val command: String = "",
+    val args: String = "", // Space-separated arguments (e.g., "--acp --verbose")
+    val env: String = "" // Environment variables, one per line: "KEY=VALUE"
+) {
+    fun isConfigured(): Boolean {
+        return command.isNotBlank()
+    }
+
+    fun getArgsList(): List<String> {
+        return args.trim().split("\\s+".toRegex()).filter { it.isNotBlank() }
+    }
+
+    fun getEnvMap(): Map<String, String> {
+        val result = mutableMapOf<String, String>()
+        env.lines().forEach { line ->
+            val trimmed = line.trim()
+            if (trimmed.isEmpty() || trimmed.startsWith("#")) return@forEach
+            val idx = trimmed.indexOf('=')
+            if (idx <= 0) return@forEach
+            val key = trimmed.substring(0, idx).trim()
+            val value = trimmed.substring(idx + 1).trim()
+            result[key] = value
+        }
+        return result
+    }
+}
+
 /**
  * Configuration file wrapper with utility methods
  */
@@ -210,6 +276,19 @@ class AutoDevConfigWrapper(val configFile: ConfigFile) {
         return configFile.cloudStorage?.isConfigured() == true
     }
 
+    fun getAcpAgents(): Map<String, AcpAgentConfig> {
+        return configFile.acpAgents ?: emptyMap()
+    }
+
+    fun getActiveAcpAgent(): AcpAgentConfig? {
+        val key = configFile.activeAcpAgent ?: return null
+        return configFile.acpAgents?.get(key)
+    }
+
+    fun getActiveAcpAgentKey(): String? {
+        return configFile.activeAcpAgent
+    }
+
     companion object {
         /**
          * Save agent type preference to config file
@@ -227,6 +306,45 @@ class AutoDevConfigWrapper(val configFile: ConfigFile) {
                 throw e
             }
         }
+
+        /**
+         * Save ACP agent configurations to config file.
+         *
+         * @param acpAgents Map of agent key to AcpAgentConfig
+         * @param activeKey The key of the currently active ACP agent (null to keep unchanged)
+         */
+        suspend fun saveAcpAgents(
+            acpAgents: Map<String, AcpAgentConfig>,
+            activeKey: String? = null
+        ) {
+            try {
+                val currentConfig = ConfigManager.load()
+                val updatedConfig = currentConfig.configFile.copy(
+                    acpAgents = acpAgents,
+                    activeAcpAgent = activeKey ?: currentConfig.configFile.activeAcpAgent
+                )
+                ConfigManager.save(updatedConfig)
+                println("ACP agent configurations saved: ${acpAgents.keys}")
+            } catch (e: Exception) {
+                println("Failed to save ACP agent configurations: ${e.message}")
+                throw e
+            }
+        }
+
+        /**
+         * Save the active ACP agent key.
+         */
+        suspend fun saveActiveAcpAgent(key: String) {
+            try {
+                val currentConfig = ConfigManager.load()
+                val updatedConfig = currentConfig.configFile.copy(activeAcpAgent = key)
+                ConfigManager.save(updatedConfig)
+                println("Active ACP agent saved: $key")
+            } catch (e: Exception) {
+                println("Failed to save active ACP agent: ${e.message}")
+                throw e
+            }
+        }
     }
 }
 

mpp-idea/src/main/kotlin/cc/unitmesh/devins/idea/components/header/IdeaAgentTabsHeader.kt

@@ -261,6 +261,7 @@ private fun getAgentTypeColor(type: AgentType): Color = when (type) {
     AgentType.LOCAL_CHAT -> JewelTheme.globalColors.text.normal
     AgentType.WEB_EDIT -> IdeaAutoDevColors.Blue.c400
     AgentType.ARTIFACT -> IdeaAutoDevColors.Indigo.c400
+    AgentType.CUSTOM_AGENT -> IdeaAutoDevColors.Amber.c400
 }
 
 /**
@@ -275,5 +276,6 @@ private fun getAgentTypeIcon(type: AgentType): ImageVector = when (type) {
     AgentType.LOCAL_CHAT -> IdeaComposeIcons.Chat
     AgentType.WEB_EDIT -> IdeaComposeIcons.Web
     AgentType.ARTIFACT -> IdeaComposeIcons.Description
+    AgentType.CUSTOM_AGENT -> IdeaComposeIcons.Code
 }
 

mpp-idea/src/main/kotlin/cc/unitmesh/devins/idea/toolwindow/IdeaAgentApp.kt

@@ -442,6 +442,13 @@ fun IdeaAgentApp(
                     IdeaEmptyStateMessage("ChatDB Agent coming soon...")
                 }
             }
+            AgentType.CUSTOM_AGENT -> {
+                // Custom ACP agents are handled via the ACP tab in Remote mode
+                // In the IDEA plugin, users configure ACP agents through the Remote panel
+                Box(modifier = Modifier.fillMaxWidth().weight(1f)) {
+                    IdeaEmptyStateMessage("Custom ACP Agent - Use Remote > ACP tab to configure")
+                }
+            }
         }
 
         // Tool loading status bar

mpp-ui/src/androidMain/kotlin/cc/unitmesh/devins/ui/compose/agent/acp/AcpConnectionProvider.android.kt

@@ -0,0 +1,5 @@
+package cc.unitmesh.devins.ui.compose.agent.acp
+
+actual fun createAcpConnection(): AcpConnection? = null
+
+actual fun isAcpSupported(): Boolean = false

mpp-ui/src/commonMain/kotlin/cc/unitmesh/devins/ui/compose/agent/AgentInterfaceRouter.kt

@@ -3,6 +3,7 @@ package cc.unitmesh.devins.ui.compose.agent
 import androidx.compose.runtime.Composable
 import androidx.compose.ui.Modifier
 import cc.unitmesh.agent.AgentType
+import cc.unitmesh.devins.ui.compose.agent.acp.AcpAgentPage
 import cc.unitmesh.devins.ui.compose.agent.artifact.ArtifactPage
 import cc.unitmesh.devins.ui.compose.agent.chatdb.ChatDBPage
 import cc.unitmesh.devins.ui.compose.agent.codereview.CodeReviewPage
@@ -20,6 +21,7 @@ import cc.unitmesh.agent.artifact.ArtifactBundle
  * - CODING: Full-featured coding agent with tools
  * - CODE_REVIEW: Dedicated code review interface
  * - REMOTE: Remote agent connected to mpp-server
+ * - CUSTOM_AGENT: External ACP agent (e.g., Kimi CLI, Claude CLI)
  */
 @Composable
 fun AgentInterfaceRouter(
@@ -158,6 +160,16 @@ fun AgentInterfaceRouter(
             )
         }
 
+        AgentType.CUSTOM_AGENT -> {
+            AcpAgentPage(
+                modifier = modifier,
+                onBack = {
+                    onAgentTypeChange(AgentType.CODING)
+                },
+                onNotification = onNotification
+            )
+        }
+
         AgentType.LOCAL_CHAT,
         AgentType.CODING -> {
             CodingAgentPage(

mpp-ui/src/commonMain/kotlin/cc/unitmesh/devins/ui/compose/agent/CodingAgentPage.kt

@@ -365,6 +365,11 @@ fun CodingAgentPage(
                             // ARTIFACT has its own full-page interface (ArtifactPage)
                             // It should not reach here - handled by AgentInterfaceRouter
                         }
+
+                        AgentType.CUSTOM_AGENT -> {
+                            // CUSTOM_AGENT has its own full-page interface (AcpAgentPage)
+                            // It should not reach here - handled by AgentInterfaceRouter
+                        }
                     }
 
                     ToolLoadingStatusBar(