feat(logging): add log summary and error recovery agents #453

Introduce LogSummaryAgent and ErrorRecoveryAgent services, update related UI and core logic, and add utilities for output formatting and testing.

Author: phodal

mpp-core/src/commonMain/kotlin/cc/unitmesh/devins/compiler/context/CompilerContext.kt

@@ -135,21 +135,36 @@ data class CompilerOptions(
 class CompilerLogger {
 
     private val logs = mutableListOf<LogEntry>()
+    var enableDebug: Boolean = false  // 默认关闭 debug 日志
+    var minLevel: LogLevel = LogLevel.ERROR  // 最小日志级别，默认只显示错误
+    
+    fun debug(message: String) {
+        if (enableDebug && minLevel <= LogLevel.DEBUG) {
+            logs.add(LogEntry(LogLevel.DEBUG, message))
+            println("[DEBUG] $message")
+        }
+    }
 
     fun info(message: String) {
-        logs.add(LogEntry(LogLevel.INFO, message))
-        println("[INFO] $message")
+        if (minLevel <= LogLevel.INFO) {
+            logs.add(LogEntry(LogLevel.INFO, message))
+            println("[INFO] $message")
+        }
     }
 
     fun warn(message: String) {
-        logs.add(LogEntry(LogLevel.WARN, message))
-        println("[WARN] $message")
+        if (minLevel <= LogLevel.WARN) {
+            logs.add(LogEntry(LogLevel.WARN, message))
+            println("[WARN] $message")
+        }
     }
 
     fun error(message: String, throwable: Throwable? = null) {
-        logs.add(LogEntry(LogLevel.ERROR, message, throwable))
-        println("[ERROR] $message")
-        throwable?.printStackTrace()
+        if (minLevel <= LogLevel.ERROR) {
+            logs.add(LogEntry(LogLevel.ERROR, message, throwable))
+            println("[ERROR] $message")
+            throwable?.printStackTrace()
+        }
     }
 
     fun getLogs(): List<LogEntry> = logs.toList()
@@ -173,7 +188,7 @@ data class LogEntry(
  * 日志级别
  */
 enum class LogLevel {
-    INFO, WARN, ERROR
+    DEBUG, INFO, WARN, ERROR
 }
 
 /**

mpp-core/src/commonMain/kotlin/cc/unitmesh/devins/compiler/processor/DevInsNodeProcessor.kt

@@ -145,9 +145,10 @@ abstract class BaseDevInsNodeProcessor : DevInsNodeProcessor {
 
     /**
      * 记录处理信息
+     * 默认使用 debug 级别，避免污染用户输出
      */
     protected fun logProcessing(node: DevInsNode, context: CompilerContext, message: String? = null) {
         val msg = message ?: "Processing ${node.nodeType}"
-        context.logger.info("[$name] $msg")
+        context.logger.debug("[$name] $msg")
     }
 }

mpp-ui/package.json

@@ -41,6 +41,7 @@
     "@js-joda/core": "^5.6.5",
     "chalk": "^5.3.0",
     "commander": "^12.1.0",
+    "diff": "^7.0.0",
     "dotenv": "^16.4.5",
     "ink": "^5.0.1",
     "ink-select-input": "^6.0.0",
@@ -50,6 +51,7 @@
     "yaml": "^2.6.1"
   },
   "devDependencies": {
+    "@types/diff": "^6.0.0",
     "@types/node": "^20.11.24",
     "@types/react": "^18.3.12",
     "ink-testing-library": "^4.0.0",

mpp-ui/src/jsMain/typescript/index.tsx

@@ -19,7 +19,7 @@ import * as fs from 'fs';
 /**
  * Run in coding agent mode
  */
-async function runCodingAgent(projectPath: string, task: string) {
+async function runCodingAgent(projectPath: string, task: string, quiet: boolean = false) {
   try {
     // Resolve project path
     const resolvedPath = path.resolve(projectPath);
@@ -40,24 +40,19 @@ async function runCodingAgent(projectPath: string, task: string) {
       process.exit(1);
     }
 
-    console.log(`\n🚀 AutoDev Coding Agent`);
-    console.log(`📦 Provider: ${activeConfig.provider}`);
-    console.log(`🤖 Model: ${activeConfig.model}\n`);
+    if (!quiet) {
+      console.log(`\n🚀 AutoDev Coding Agent`);
+      console.log(`📦 Provider: ${activeConfig.provider}`);
+      console.log(`🤖 Model: ${activeConfig.model}\n`);
+    }
 
     // Create and run coding agent
-    const agent = new CodingAgentService(resolvedPath, activeConfig);
+    const agent = new CodingAgentService(resolvedPath, activeConfig, quiet);
     const result = await agent.executeTask({
       requirement: task,
       projectPath: resolvedPath,
     });
 
-    // Print summary
-    console.log('\n' + '='.repeat(60));
-    console.log(result.message);
-    console.log(`Total steps: ${result.steps.length}`);
-    console.log(`Total edits: ${result.edits.length}`);
-    console.log('='.repeat(60) + '\n');
-
     process.exit(result.success ? 0 : 1);
 
   } catch (error) {
@@ -105,9 +100,11 @@ async function main() {
     .requiredOption('-p, --path <path>', 'Project path (e.g., /path/to/project or . for current directory)')
     .requiredOption('-t, --task <task>', 'Development task or requirement to complete')
     .option('-m, --max-iterations <number>', 'Maximum iterations', '10')
+    .option('-q, --quiet', 'Quiet mode - only show important messages', false)
+    .option('-v, --verbose', 'Verbose mode - show all debug information', false)
     .action(async (options) => {
       const projectPath = options.path === '.' ? process.cwd() : options.path;
-      await runCodingAgent(projectPath, options.task);
+      await runCodingAgent(projectPath, options.task, options.quiet && !options.verbose);
     });
 
   // Parse arguments