diguike PR 上下文
!gh pr view $0 --json title,body,labels,changedFiles 2>/dev/null || echo "未指定 PR，将审查当前代码"
变更概览
!gh pr diff $0 --stat 2>/dev/null || git diff --stat
完整 diff
!gh pr diff $0 2>/dev/null || git diff
审查指令
基于以上变更内容，进行代码审查。
审查维度
- Bug 风险：空指针、未处理异常、边界条件、类型错误、竞态条件
- 安全问题：XSS、SQL 注入、敏感信息泄露、未授权访问、CSRF
- 可维护性：命名清晰度、函数长度、重复代码、过度耦合、注释质量
- 性能：不必要的渲染、内存泄漏、大循环中的重复计算、N+1 查询
严重度标准
- 🔴 Critical：必须修复才能合并。安全漏洞、数据丢失风险、线上必现 bug。
- 🟡 Warning：建议修复。影响可维护性、潜在的边界 bug、性能隐患。
- 🔵 Suggestion：可以考虑。更优雅的写法、微小的性能优化、代码风格。
不要把所有问题都标 Critical——只有真正阻塞合并的才算。过度警告会让开发者麻木。
规则加载
始终加载 rules/base.md。
根据代码内容，额外加载相关规则：
- 包含
.tsx/.jsx或 React 导入 → rules/react.md - 包含 CSS/SCSS/样式文件 → rules/css-layout.md
- 包含
platform/os判断或 React Native → rules/multi-platform.md - 包含主题/颜色变量/CSS 变量 → rules/theme-compat.md
- 包含
aria-/role=/tabIndex→ rules/accessibility.md - 包含文案字符串/i18n 函数 → rules/i18n.md
- 所有代码始终检查 → rules/security.md
参考知识
根据技术栈，按需读取：
- React 项目 → references/react.md
- Vue 项目 → references/vue.md
- TypeScript 严格模式相关 → references/typescript-strict.md
- API 接口代码 → references/api-design.md
团队知识（始终参考）：
- references/team-knowledge/naming-conventions.md
- references/team-knowledge/known-pitfalls.md
- references/team-knowledge/architecture-decisions.md
输出格式
严格按以下格式输出：
问题列表
| # | 文件 | 行号 | 严重度 | 问题描述 | 修复建议 |
|---|
总结
- 总体评分：X/10
- 主要风险：一句话概括最大的问题
- 亮点：值得肯定的做法
- 建议优先级：先修什么，后修什么
自我评分
审查完成后，对本次审查质量进行评分（诚实评估，不要虚高）：
- coverage（0-1）：你审查了变更文件的多大比例？
- accuracy（0-1）：你提出的问题中，有多少是真实存在的问题（而非过度警告）？
- depth（0-1）：你是否发现了需要理解业务逻辑才能看出的深层问题？
以 JSON 格式输出 selfScore。
审查后处理
审查完成后：
- 将审查结果保存到临时文件
/tmp/review-output.md - 运行
npx tsx ${CLAUDE_SKILL_DIR}/scripts/collect-metrics.ts /tmp/review-output.md生成统计 - 将统计结果和 selfScore 传给
npx tsx ${CLAUDE_SKILL_DIR}/scripts/append-log.ts data/review-metrics.jsonl '<统计JSON>'记录日志