@miphamai/cli 0.74.0 → 0.75.0

package/package.json

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

package/src/core/git-pr.ts

@@ -0,0 +1,51 @@
+import { exec } from 'node:child_process'
+
+export interface GitPr {
+  number: number
+  state: 'OPEN' | 'CLOSED' | 'MERGED'
+  isDraft: boolean
+  reviewDecision: string
+}
+
+export type PrColor = 'white' | 'green' | 'yellow' | 'magenta' | 'gray'
+
+/** Parse `gh pr list --json number,state,isDraft,reviewDecision` output into the first PR.
+ *  Bad input / empty list / missing number → null. */
+export function parseGitPr(json: string): GitPr | null {
+  try {
+    const arr = JSON.parse(json)
+    if (!Array.isArray(arr) || arr.length === 0) return null
+    const first = arr[0]
+    if (!first || typeof first.number !== 'number') return null
+    return {
+      number: first.number,
+      state: first.state ?? 'OPEN',
+      isDraft: Boolean(first.isDraft),
+      reviewDecision: first.reviewDecision ?? '',
+    }
+  } catch {
+    return null
+  }
+}
+
+/** Map PR state to an Ink color — mirrors Claude Code's status-line PR badge:
+ *  merged → magenta, closed/draft → gray, approved → green, changes-requested → yellow,
+ *  otherwise (open, no review) → white. */
+export function prColor(pr: GitPr): PrColor {
+  if (pr.state === 'MERGED') return 'magenta'
+  if (pr.state === 'CLOSED' || pr.isDraft) return 'gray'
+  if (pr.reviewDecision === 'APPROVED') return 'green'
+  if (pr.reviewDecision === 'CHANGES_REQUESTED') return 'yellow'
+  return 'white'
+}
+
+/** Detect the PR whose head is `branch`. gh unavailable / no PR / non-zero exit → null.
+ *  Async so the status line never blocks startup. */
+export function resolveGitPr(branch: string): Promise<GitPr | null> {
+  return new Promise((resolve) => {
+    const cmd = `gh pr list --head "${branch}" --state all --json number,state,isDraft,reviewDecision`
+    exec(cmd, { timeout: 3000 }, (err, stdout) => {
+      resolve(err ? null : parseGitPr(stdout))
+    })
+  })
+}

package/src/core/instructions.ts

@@ -275,6 +275,15 @@ say so and read it first, rather than answering hastily and retracting
 afterwards. This applies to every code question, not only research or
 borrow-analysis tasks.`)
 
+    // CRSI code-review 合并门 — 合并 PR 前必须先 review，勿靠「碰巧触发」
+    parts.push(`## Code-Review Merge Gate
+
+Before merging a PR or committing changes that have not been
+code-reviewed, run a code review first (the /code-review command or the
+code-review skill) and address its findings. Do not rely on "the user
+happened to ask" to trigger a review — review proactively as a fixed
+step before merge.`)
+
     // CRSI 教训召回 — 把 crsi-lessons.md 的教训精华注入，让模型「写后召回」而非只写不读
     const lessonsBlock = buildCrsiLessonsBlock(this.crsiLessonSummaries)
     if (lessonsBlock) parts.push(lessonsBlock)

package/src/i18n-core/locales/en-US.json

@@ -613,6 +613,16 @@
       "audit_clean": "  ✓ no code-derivable sections found",
       "audit_found": "Add the sections above to the `prompt-exclude` frontmatter to stop sending them to the model."
     },
+    "skillDoctor": {
+      "title": "── Skill Doctor ──",
+      "unavailable": "  (skills info unavailable)",
+      "summary": "Loaded {total} skills — ~{tokens} tokens in context. {unused} never used (~{unusedTokens} tokens reclaimable).",
+      "unused_header": "Never used:",
+      "used_header": "Recently used:",
+      "never": "never used",
+      "days_ago": "{days}d ago",
+      "today": "today"
+    },
     "fix": {
       "title": "── Fix ──",
       "dryrun_banner": "[dry-run] preview — no changes written",

package/src/i18n-core/locales/zh-CN.json

@@ -613,6 +613,16 @@
       "audit_clean": "  ✓ 未发现可从代码推断的冗余章节",
       "audit_found": "将上述章节加入 `prompt-exclude` frontmatter，可停止发送给模型、节省 token。"
     },
+    "skillDoctor": {
+      "title": "── 技能体检 ──",
+      "unavailable": "  （技能信息不可用）",
+      "summary": "已加载 {total} 个技能 — 占 context ~{tokens} tokens。{unused} 个从未使用（可省 ~{unusedTokens} tokens）。",
+      "unused_header": "从未使用：",
+      "used_header": "最近使用：",
+      "never": "从未使用",
+      "days_ago": "{days} 天前",
+      "today": "今天"
+    },
     "fix": {
       "title": "── 修复 ──",
       "dryrun_banner": "[dry-run] 预览模式 — 不落盘",

package/src/shared/package-info.ts

@@ -9,7 +9,7 @@
 export const PACKAGE_NAME = '@miphamai/cli' as const
 
 /** 当前发布版本 */
-export const PACKAGE_VERSION = '0.74.0' as const
+export const PACKAGE_VERSION = '0.75.0' as const
 
 /** npm install 全局安装命令 */
 export const NPM_INSTALL_COMMAND = `npm install -g ${PACKAGE_NAME}` as const

package/src/skills/usage.ts

@@ -0,0 +1,34 @@
+import { readFileSync, mkdirSync } from 'node:fs'
+import { join } from 'node:path'
+import { homedir } from 'node:os'
+import { atomicWriteFileSync } from '../shared/atomic-write'
+
+// Skill 使用记录：name → 最近调用时间戳（epoch ms）。持久化到 ~/.mipham/skill-usage.json，
+// 供 /skill-doctor 识别「从未被调用」的 skill（跨会话累积，按证据 prune）。
+const USAGE_DIR = join(homedir(), '.mipham')
+const USAGE_FILE = join(USAGE_DIR, 'skill-usage.json')
+
+/** 读 skill 使用记录。文件缺失或损坏（非法 JSON / 类型不符）→ 空 Map，绝不抛错。 */
+export function loadSkillUsage(): Map<string, number> {
+  try {
+    const raw = readFileSync(USAGE_FILE, 'utf-8')
+    const parsed = JSON.parse(raw) as Record<string, unknown>
+    const map = new Map<string, number>()
+    for (const [name, ts] of Object.entries(parsed)) {
+      if (typeof ts === 'number') map.set(name, ts)
+    }
+    return map
+  } catch {
+    return new Map()
+  }
+}
+
+/** 记录一次 skill 调用。原子写（temp + rename），读者要么见旧值要么见新值，绝不读到半截。 */
+export function recordSkillUsage(name: string, now = Date.now()): void {
+  const map = loadSkillUsage()
+  map.set(name, now)
+  const obj: Record<string, number> = {}
+  for (const [k, v] of map) obj[k] = v
+  mkdirSync(USAGE_DIR, { recursive: true })
+  atomicWriteFileSync(USAGE_FILE, JSON.stringify(obj))
+}

package/src/tools/agent/skill.ts

@@ -3,6 +3,7 @@ import { executeForkedSkill } from '../../skills/fork-executor'
 import { sanitizeSkillBody } from '../../skills/sanitizer'
 import { ensureSkillAssets } from '../../skills/skill-assets'
 import { checkRequiredBins } from '../../skills/bin-check'
+import { recordSkillUsage } from '../../skills/usage'
 
 export const skillTool: ToolDefinition = {
   name: 'Skill',
@@ -82,6 +83,7 @@ export const skillTool: ToolDefinition = {
             ctx.llm,
           )
           // Return to AI as internal context
+          recordSkillUsage(skillName)
           return { success: true, content: `[Forked skill "${skillName}" result]:\n${result}` }
         } catch (err) {
           return {
@@ -108,6 +110,7 @@ export const skillTool: ToolDefinition = {
         bodyText,
       ].filter(Boolean)
 
+      recordSkillUsage(skillName)
       return { success: true, content: lines.join('\n') }
     }
 

package/src/ui/app.tsx

@@ -9,6 +9,7 @@ import type { RemoteEngine } from '../daemon/remote-engine'
 import type { MiphamConfig } from '../shared/index.ts'
 import type { Llm } from '../providers/llm'
 import { AUTOCOMPLETE_MAX_CONTEXT, type RecentMessage } from '../core/autocomplete'
+import { resolveGitPr, prColor, type GitPr } from '../core/git-pr'
 import type { SkillsLoader } from '../skills/loader'
 import type { PluginManager } from '../plugin/plugin-manager'
 import { setPreference } from '../config/preferences'
@@ -200,6 +201,18 @@ export function App({
       return null
     }
   })
+  // Current branch's PR (head = branch) — async detect; gh unavailable / no PR → null.
+  const [gitPr, setGitPr] = useState<GitPr | null>(null)
+  useEffect(() => {
+    if (!gitBranch) return
+    let cancelled = false
+    resolveGitPr(gitBranch).then((pr) => {
+      if (!cancelled) setGitPr(pr)
+    })
+    return () => {
+      cancelled = true
+    }
+  }, [gitBranch])
 
   // 启动后台查新版（非阻塞；离线静默失败）
   useEffect(() => {
@@ -1249,7 +1262,12 @@ export function App({
             </Box>
 
             {/* Git branch — dim, bottom-most, mirrors Claude Code's "⏺ main" */}
-            {gitBranch && <Text dimColor>⏺ {gitBranch}</Text>}
+            {gitBranch && (
+              <Box>
+                <Text dimColor>⏺ {gitBranch}</Text>
+                {gitPr && <Text color={prColor(gitPr)}> · PR #{gitPr.number}</Text>}
+              </Box>
+            )}
           </>
         )}
       </Box>

package/src/ui/commands.ts

@@ -8,6 +8,7 @@ import type { QueryEngine } from '../core/engine'
 import type { MiphamConfig } from '../shared/index.ts'
 import { formatContextWindow } from '../shared/format'
 import type { SkillsLoader } from '../skills/loader'
+import { loadSkillUsage } from '../skills/usage'
 import type { PluginManager } from '../plugin/plugin-manager'
 import type { Message } from '../shared/types.js'
 import type { UpdateStatus } from '../shared/update'
@@ -3198,6 +3199,71 @@ const doctorCmd: CommandHandler = async (ctx) => {
   return { content: lines.join('\n') }
 }
 
+const skillDoctorCmd: CommandHandler = async (ctx) => {
+  const t = resolveT(ctx)
+  const lines: string[] = [t('commands.skillDoctor.title'), '']
+
+  const loader = ctx.skillsLoader
+  if (!loader) {
+    lines.push(t('commands.skillDoctor.unavailable'))
+    return { content: lines.join('\n') }
+  }
+
+  const skills = loader.list()
+  const usage = loadSkillUsage()
+
+  // Per-skill context cost = the reminder entry estimate (matches buildSystemReminder).
+  const rows = skills.map((s) => {
+    const entry = `- ${s.name}: ${s.description}`
+    return {
+      name: s.name,
+      type: s.type,
+      tokens: Math.ceil(entry.length / 4) + 1,
+      lastUsed: usage.get(s.name),
+    }
+  })
+
+  const unused = rows.filter((r) => r.lastUsed === undefined)
+  const used = rows
+    .filter((r) => r.lastUsed !== undefined)
+    .sort((a, b) => b.lastUsed! - a.lastUsed!)
+  const totalTokens = rows.reduce((sum, r) => sum + r.tokens, 0)
+  const unusedTokens = unused.reduce((sum, r) => sum + r.tokens, 0)
+
+  const age = (ts: number): string => {
+    const days = Math.floor((Date.now() - ts) / 86400000)
+    if (days <= 0) return t('commands.skillDoctor.today')
+    return t('commands.skillDoctor.days_ago', { days: String(days) })
+  }
+
+  lines.push(
+    t('commands.skillDoctor.summary', {
+      total: String(skills.length),
+      tokens: String(totalTokens),
+      unused: String(unused.length),
+      unusedTokens: String(unusedTokens),
+    }),
+  )
+  lines.push('')
+
+  if (unused.length > 0) {
+    lines.push(t('commands.skillDoctor.unused_header'))
+    for (const r of unused) {
+      lines.push(`  • ${r.name} (${r.type}) — ~${r.tokens} tokens`)
+    }
+    lines.push('')
+  }
+
+  if (used.length > 0) {
+    lines.push(t('commands.skillDoctor.used_header'))
+    for (const r of used) {
+      lines.push(`  • ${r.name} (${r.type}) — ${age(r.lastUsed!)}`)
+    }
+  }
+
+  return { content: lines.join('\n') }
+}
+
 const fixCmd: CommandHandler = async (ctx, args) => {
   const t = resolveT(ctx)
   const { readFileSync, writeFileSync } = await import('node:fs')
@@ -5143,6 +5209,7 @@ const commandsListCmd: CommandHandler = () => {
     '/save': 'Session & Identity',
     '/export': 'Session & Identity',
     '/doctor': 'Session & Identity',
+    '/skill-doctor': 'Session & Identity',
     '/dream': 'Session & Identity',
     '/constitution': 'Session & Identity',
     '/bug-report': 'Session & Identity',
@@ -5388,6 +5455,7 @@ registry.set('/pr-comments', prCommentsCmd)
 
 // Session Management
 registry.set('/doctor', doctorCmd)
+registry.set('/skill-doctor', skillDoctorCmd)
 registry.set('/fix', fixCmd)
 registry.set('/export', exportCmd)
 registry.set('/resume', resumeCmd)
@@ -5493,6 +5561,7 @@ const COMMAND_DESCRIPTIONS: Record<string, string> = {
   '/save': 'Save conversation to Obsidian wiki (skill: save-to-wiki)',
   '/export': 'Export conversation to file',
   '/doctor': 'System diagnostics',
+  '/skill-doctor': 'Show unused skills and their context cost',
   '/fix': 'Deterministic self-repair: doctor/config/cache',
   '/dream': 'Background memory consolidation',
   '/constitution': 'View or reload constitutional principles',