requesting-code-review

请求代码审查

核心原则：审查不是可选的。每个有意义的代码变更都需要经过审查。

宣告开始时说："I'm using the requesting-code-review skill to ensure proper review process."

触发时机

必须请求审查的情况

场景	审查类型	原因
完成计划中的任务步骤	三阶段审查	确保符合计划且质量达标
PR 准备合并	完整审查	合并前的最后质量门
涉及认证/授权	安全审查必需	安全风险最高的领域
涉及支付/敏感数据	安全审查必需	合规和数据保护要求
修改核心业务逻辑	规格+代码审查	确保正确性和可维护性

可以跳过审查的情况

场景	条件
纯文档更新	不涉及代码逻辑
依赖版本 bump	只有 lockfile 变更
格式化变更	仅空格/缩进/格式
测试修复	测试本身的修复（非逻辑变更）

⚠️ 注意：即使可以跳过，遇到不确定时宁可审查。

审查前检查清单

在请求审查之前，确保以下事项：

自检清单

• 代码可编译：npm run build 或 tsc --noEmit 通过
• 测试通过：npm test 全绿
• 格式化完成：npm run format 或 prettier --check
• Lint 通过：npm run lint 无错误
• 没有调试代码：无 console.log, debugger, TODO: remove

检测命令

# 快速自检脚本
echo "🔍 Running pre-review checks..."

# 1. TypeScript 编译
echo "Checking TypeScript..."
npx tsc --noEmit || { echo "❌ TypeScript errors"; exit 1; }

# 2. 测试
echo "Running tests..."
npm test || { echo "❌ Tests failed"; exit 1; }

# 3. Lint
echo "Running lint..."
npm run lint || { echo "❌ Lint errors"; exit 1; }

# 4. 调试代码检查
echo "Checking for debug code..."
grep -rn "console.log\|debugger\|TODO:\s*remove" --include="*.ts" src/ && {
  echo "⚠️ Found debug code"
}

echo "✅ Pre-review checks passed"

审查流程

质量审查采用多阶段流程，顺序不可颠倒：

┌─────────────────────────────────────────────────────────────┐
│                    审查流程                                  │
├─────────────────────────────────────────────────────────────┤
│                                                             │
│   阶段 0: 设计还原审查 (design-reviewer) [按需]              │
│   └── 检查：视觉还原度、布局结构、组件映射                    │
│   └── 触发：上下文有设计参考（Figma URL/截图/设计稿）时       │
│       │                                                     │
│       ▼ 通过后（或无设计参考时跳过）                          │
│                                                             │
│   阶段 1: 规格合规审查 (spec-reviewer)                       │
│   └── 检查：实现是否满足 PRD 和计划中的所有需求               │
│   └── 通过条件：需求覆盖 100%，无缺失实现                     │
│       │                                                     │
│       ▼ 通过后                                              │
│                                                             │
│   阶段 2: 代码质量审查 (code-reviewer)                       │
│   └── 检查：代码质量、最佳实践、可维护性                      │
│   └── 通过条件：无严重问题，重要问题已知晓                    │
│       │                                                     │
│       ▼ 通过后                                              │
│                                                             │
│   阶段 3: 安全审查 (security-reviewer) [按需]                │
│   └── 检查：OWASP Top 10，敏感数据处理                       │
│   └── 触发：涉及认证/授权/敏感数据时必需                      │
│       │                                                     │
│       ▼ 全部通过                                            │
│                                                             │
│   ✅ 审查完成，可以合并                                      │
│                                                             │
└─────────────────────────────────────────────────────────────┘