@miphamai/cli 0.28.0 → 0.29.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@miphamai/cli",
-  "version": "0.28.0",
+  "version": "0.29.0",
   "description": "Mipham Code — Multi-model open-core intelligent coding terminal by MiphamAI",
   "keywords": [
     "ai",

package/skills/mipham/self-audit.mipham-skill.md

@@ -24,58 +24,58 @@ Mipham Code 审计 Mipham Code — 这是 CRSI 递归自我改进的第一个闭
 
 ### 1. 代码质量
 
-| 检查项 | 方法 |
-|--------|------|
-| Dead code | Grep 搜索未被引用的 export、未使用的 import |
-| 不一致模式 | 对比同一目录下多个文件的代码风格/模式差异 |
-| 类型安全 | 搜索 `as any`、`@ts-ignore`、`unknown` 未收窄 |
-| 错误处理 | 搜索裸 `catch`、无 `try/catch` 的 async 调用 |
-| Deep nesting | 搜索嵌套超过 4 层的 if/for/switch |
+| 检查项       | 方法                                          |
+| ------------ | --------------------------------------------- |
+| Dead code    | Grep 搜索未被引用的 export、未使用的 import   |
+| 不一致模式   | 对比同一目录下多个文件的代码风格/模式差异     |
+| 类型安全     | 搜索 `as any`、`@ts-ignore`、`unknown` 未收窄 |
+| 错误处理     | 搜索裸 `catch`、无 `try/catch` 的 async 调用  |
+| Deep nesting | 搜索嵌套超过 4 层的 if/for/switch             |
 
 ### 2. 架构完整性
 
-| 检查项 | 方法 |
-|--------|------|
-| 循环依赖 | 分析 import 图，检测 A→B→A |
-| 接口契约 | 对比 `shared/types.ts` 中的类型定义与实际使用 |
-| 模块边界 | 检查是否有跨层级直接访问（ui/ 直接 import core/ 内部实现） |
-| God objects | 搜索超过 500 行的单个类/函数 |
+| 检查项      | 方法                                                       |
+| ----------- | ---------------------------------------------------------- |
+| 循环依赖    | 分析 import 图，检测 A→B→A                                 |
+| 接口契约    | 对比 `shared/types.ts` 中的类型定义与实际使用              |
+| 模块边界    | 检查是否有跨层级直接访问（ui/ 直接 import core/ 内部实现） |
+| God objects | 搜索超过 500 行的单个类/函数                               |
 
 ### 3. 性能
 
-| 检查项 | 方法 |
-|--------|------|
-| 同步阻塞 | 搜索 `readFileSync`、`execSync` 在主线程中 |
+| 检查项       | 方法                                            |
+| ------------ | ----------------------------------------------- |
+| 同步阻塞     | 搜索 `readFileSync`、`execSync` 在主线程中      |
 | 内存泄漏风险 | 搜索未清理的 setInterval、EventEmitter listener |
-| 渲染性能 | 检查 React memo/callback 使用是否完整 |
-| N+1 模式 | 搜索在循环内的 I/O 操作 |
+| 渲染性能     | 检查 React memo/callback 使用是否完整           |
+| N+1 模式     | 搜索在循环内的 I/O 操作                         |
 
 ### 4. 安全
 
-| 检查项 | 方法 |
-|--------|------|
-| 硬编码凭据 | 搜索 API key、token、password 字符串 |
-| 路径遍历 | 搜索使用用户输入的 `join`/`resolve` 路径 |
-| 命令注入 | 搜索字符串拼接的 shell 命令 |
-| 许可合规 | 检查 package.json 中的 copyleft 依赖 |
+| 检查项     | 方法                                     |
+| ---------- | ---------------------------------------- |
+| 硬编码凭据 | 搜索 API key、token、password 字符串     |
+| 路径遍历   | 搜索使用用户输入的 `join`/`resolve` 路径 |
+| 命令注入   | 搜索字符串拼接的 shell 命令              |
+| 许可合规   | 检查 package.json 中的 copyleft 依赖     |
 
 ### 5. 测试覆盖
 
-| 检查项 | 方法 |
-|--------|------|
-| 未测试模块 | Glob 所有 `src/**/*.ts`，对比 `test/` 目录 |
-| 关键路径覆盖 | 识别 engine、permission、tools 层，检查测试 |
+| 检查项          | 方法                                              |
+| --------------- | ------------------------------------------------- |
+| 未测试模块      | Glob 所有 `src/**/*.ts`，对比 `test/` 目录        |
+| 关键路径覆盖    | 识别 engine、permission、tools 层，检查测试       |
 | Flaky test 风险 | 搜索 `setTimeout`、`Math.random`、Date 依赖的测试 |
-| 边界测试缺失 | 检查主要函数的 null/undefined/empty 参数测试 |
+| 边界测试缺失    | 检查主要函数的 null/undefined/empty 参数测试      |
 
 ### 6. CRSI 集成健康
 
-| 检查项 | 方法 |
-|--------|------|
-| Rule 引擎状态 | 检查活跃规则数、禁用规则数、builtin vs auto-generated |
-| 效果追踪 | 从 EffectivenessTracker 读取规则成功率 |
-| 模式分析器 | 检查累积的 Agent 失败模式 |
-| AutoMemory 状态 | 检查复盘文件数量、CRSI 洞察统计 |
+| 检查项          | 方法                                                  |
+| --------------- | ----------------------------------------------------- |
+| Rule 引擎状态   | 检查活跃规则数、禁用规则数、builtin vs auto-generated |
+| 效果追踪        | 从 EffectivenessTracker 读取规则成功率                |
+| 模式分析器      | 检查累积的 Agent 失败模式                             |
+| AutoMemory 状态 | 检查复盘文件数量、CRSI 洞察统计                       |
 
 ## 执行流程
 
@@ -125,14 +125,14 @@ Mipham Code 审计 Mipham Code — 这是 CRSI 递归自我改进的第一个闭
 
 ## 摘要
 
-| 维度 | 评分 | 发现数 | 严重 |
-|------|------|--------|------|
-| 代码质量 | 7/10 | 12 | 2 |
-| 架构完整性 | 8/10 | 3 | 0 |
-| 性能 | 7/10 | 5 | 1 |
-| 安全 | 8/10 | 2 | 0 |
-| 测试覆盖 | 7/10 | 8 | 1 |
-| CRSI 健康 | 9/10 | 0 | 0 |
+| 维度       | 评分 | 发现数 | 严重 |
+| ---------- | ---- | ------ | ---- |
+| 代码质量   | 7/10 | 12     | 2    |
+| 架构完整性 | 8/10 | 3      | 0    |
+| 性能       | 7/10 | 5      | 1    |
+| 安全       | 8/10 | 2      | 0    |
+| 测试覆盖   | 7/10 | 8      | 1    |
+| CRSI 健康  | 9/10 | 0      | 0    |
 
 ## 🔴 严重 (需要立即处理)
 
@@ -148,9 +148,9 @@ Mipham Code 审计 Mipham Code — 这是 CRSI 递归自我改进的第一个闭
 
 ## CRSI 规则更新
 
-| 规则 ID | 类型 | 状态 | 上次评估 |
-|---------|------|------|---------|
-| ... | ... | active/degraded/disabled | ... |
+| 规则 ID | 类型 | 状态                     | 上次评估 |
+| ------- | ---- | ------------------------ | -------- |
+| ...     | ...  | active/degraded/disabled | ...      |
 ```
 
 ## 安全约束

package/src/core/auto-memory.ts

@@ -419,10 +419,7 @@ export class AutoMemoryEngine {
     return decisions.slice(0, 5)
   }
 
-  private extractActionItems(
-    assistantContent: string,
-    toolCalls: ToolCallRecord[],
-  ): string[] {
+  private extractActionItems(assistantContent: string, toolCalls: ToolCallRecord[]): string[] {
     const items: string[] = []
 
     // Check for TODO markers in assistant content
@@ -594,7 +591,13 @@ export class AutoMemoryEngine {
       `| 成功率 | ${totalSuccesses + totalFailures > 0 ? Math.round((totalSuccesses / (totalSuccesses + totalFailures)) * 100) : 100}% |`,
       '',
       ...(criticalInsights.length > 0
-        ? ['## ⚠️ 关键洞察', '', ...criticalInsights.map((i) => `- **${i.category}**: ${i.description}\n  → ${i.suggestion}`)]
+        ? [
+            '## ⚠️ 关键洞察',
+            '',
+            ...criticalInsights.map(
+              (i) => `- **${i.category}**: ${i.description}\n  → ${i.suggestion}`,
+            ),
+          ]
         : []),
       '',
       '---',

package/src/core/crsi-sandbox.ts

@@ -52,7 +52,8 @@ export interface CrsiModificationResult {
   /** Error message if something failed */
   error?: string
   /** Current phase of the sandbox pipeline */
-  phase: 'pending' | 'applied' | 'testing' | 'passed' | 'failed' | 'approved' | 'merged' | 'rolled-back'
+  phase:
+    'pending' | 'applied' | 'testing' | 'passed' | 'failed' | 'approved' | 'merged' | 'rolled-back'
 }
 
 export interface CrsiTestResult {
@@ -154,7 +155,12 @@ export class CrsiSandbox {
    */
   applyModification(mod: CrsiModification): CrsiModificationResult {
     if (!this.worktreePath) {
-      return { modification: mod, applied: false, phase: 'pending', error: 'No worktree created. Call createWorktree() first.' }
+      return {
+        modification: mod,
+        applied: false,
+        phase: 'pending',
+        error: 'No worktree created. Call createWorktree() first.',
+      }
     }
 
     this.sessionReport.summary.total++
@@ -180,7 +186,8 @@ export class CrsiSandbox {
     try {
       const currentContent = readFileSync(targetPath, 'utf-8')
       if (currentContent !== mod.originalContent && mod.originalContent) {
-        result.error = 'Original content mismatch — file may have been modified since the audit. Aborting.'
+        result.error =
+          'Original content mismatch — file may have been modified since the audit. Aborting.'
         result.phase = 'failed'
         this.sessionReport.modifications.push(result)
         return result
@@ -313,9 +320,7 @@ export class CrsiSandbox {
     }
 
     // Check that all modifications passed tests
-    const failedCount = this.sessionReport.modifications.filter(
-      (m) => m.phase === 'failed',
-    ).length
+    const failedCount = this.sessionReport.modifications.filter((m) => m.phase === 'failed').length
     if (failedCount > 0) {
       return {
         success: false,

package/src/ui/app.tsx

@@ -44,6 +44,8 @@ export interface ToolMeta {
   input: string
   output?: string
   collapsed: boolean
+  /** Original tool name before Claude Code display renaming (e.g. Write→Update). */
+  originalName?: string
 }
 
 export interface ChatMessage {
@@ -87,7 +89,8 @@ const PERMISSION_COLORS: Record<PermissionMode, string> = {
   bypassPermissions: 'red',
 }
 
-/** Format a tool's input parameters into a compact one-line detail string. */
+/** Format a tool's input parameters into a compact one-line detail string.
+ *  Tool display names follow Claude Code convention: Write/Edit → Update. */
 function formatToolDetail(name: string, input: Record<string, unknown>): string {
   switch (name) {
     case 'Bash':
@@ -95,9 +98,9 @@ function formatToolDetail(name: string, input: Record<string, unknown>): string
     case 'Read':
       return (input.file_path as string) || ''
     case 'Write':
-      return (input.file_path as string) || ''
+      return (input.file_path as string) || '' // displayed as "Update" in chat
     case 'Edit':
-      return (input.file_path as string) || ''
+      return `${(input.file_path as string) || ''}: ${((input.old_string as string) || '').slice(0, 60)}` // displayed as "Update" in chat
     case 'Grep':
       return (input.pattern as string) || ''
     case 'Glob':
@@ -115,6 +118,12 @@ function formatToolDetail(name: string, input: Record<string, unknown>): string
   }
 }
 
+/** Claude Code display name: maps Write/Edit → Update for parity. */
+function toolDisplayName(name: string): string {
+  if (name === 'Write' || name === 'Edit') return 'Update'
+  return name
+}
+
 export function App({
   engine,
   config,
@@ -462,7 +471,12 @@ export function App({
               {
                 role: 'system' as const,
                 content: detail,
-                toolMeta: { name: toolName, input: detail, collapsed: true },
+                toolMeta: {
+                  name: toolDisplayName(toolName),
+                  input: detail,
+                  collapsed: true,
+                  originalName: toolName,
+                },
               },
             ])
             // Flush stream buffer before showing tool card
@@ -475,18 +489,35 @@ export function App({
             setAgentProgress(null)
             // Sync background agents (e.g. Agent tool may have spawned them)
             syncBgAgents()
-            // Only show tool results with meaningful content.
-            // Skip empty results, "(no matches)", and other noise.
             const output = chunk.content ? String(chunk.content).trim() : ''
             if (output && output !== '(no matches)' && output.length > 20) {
-              const firstLine = output.split('\n')[0]!.slice(0, 200)
-              const preview = firstLine.length < output.length ? `${firstLine}...` : firstLine
+              // Generate diff-style summary for file operations (Claude Code parity)
+              let preview: string
+              const isFileOp = chunk.toolUse?.name === 'Write' || chunk.toolUse?.name === 'Edit'
+              if (isFileOp && chunk.toolUse?.input?.file_path) {
+                const filePath = String(chunk.toolUse.input.file_path)
+                const newContent = String(
+                  chunk.toolUse.input.content || chunk.toolUse.input.new_string || '',
+                )
+                // Count lines added/removed for Write operations
+                const lineCount = newContent.split('\n').length
+                preview = `Updated ${filePath} · ${lineCount} lines`
+              } else {
+                const firstLine = output.split('\n')[0]!.slice(0, 200)
+                preview = firstLine.length < output.length ? `${firstLine}...` : firstLine
+              }
               setMessages((prev) => [
                 ...prev,
                 {
                   role: 'system' as const,
-                  content: `  ⎿  ${preview}`,
-                  toolMeta: { name: '', input: '', output: preview, collapsed: true },
+                  content: preview,
+                  toolMeta: {
+                    name: '',
+                    input: '',
+                    output: preview,
+                    collapsed: true,
+                    originalName: chunk.toolUse?.name,
+                  },
                 },
               ])
             }

package/src/ui/chat.tsx

@@ -28,6 +28,7 @@ function toolColor(name: string): string {
       return 'white'
     case 'Write':
     case 'Edit':
+    case 'Update':
       return 'red'
     case 'Agent':
     case 'Task':
@@ -59,12 +60,19 @@ const MessageRow = React.memo(
       >
         {msg.toolMeta ? (
           <Box flexDirection="column">
-            <Text color={toolColor(msg.toolMeta.name)}>
-              {msg.toolMeta.collapsed ? '⏺' : '⏺ ▼'} {msg.toolMeta.name}
-              {msg.toolMeta.input ? `(${msg.toolMeta.input.slice(0, 120)})` : ''}
-            </Text>
+            {/* Tool call line: ⏺ ToolName(detail) — Claude Code parity */}
+            {msg.toolMeta.name ? (
+              <Text color={toolColor(msg.toolMeta.name)}>
+                {msg.toolMeta.collapsed ? '⏺' : '⏺ ▼'} {msg.toolMeta.name}
+                {msg.toolMeta.input ? ` (${msg.toolMeta.input.slice(0, 120)})` : ''}
+              </Text>
+            ) : null}
+            {/* Tool result line: ⎿  summary text */}
             {msg.toolMeta.output ? (
-              <Text dimColor> ⎿ {msg.toolMeta.output.slice(0, 300)}</Text>
+              <Text dimColor>
+                {'  ⎿  '}
+                {msg.toolMeta.output.slice(0, 300)}
+              </Text>
             ) : msg.content && !msg.toolMeta.collapsed ? (
               <Text dimColor>{msg.content}</Text>
             ) : null}