tikazyq Code Review Skill
Scope: EXECSPEC_FULFILL — Fulfill ExecSpec（落实 ExecSpec）
版本: 0.1.0（占位）| 创建日期: 2025-11-27
概述
Code Review 是保证代码质量的关键门禁：
┌─────────────────────────────────────────────────────┐
│ 👀 Code Review Flow │
├─────────────────────────────────────────────────────┤
│ 提交 PR → 自查 → 同行审查 → 反馈修改 → 合并 │
│ (Submit) (Self) (Peer) (Fix) (Merge) │
└─────────────────────────────────────────────────────┘
核心目标：
- 发现缺陷和潜在问题
- 确保代码符合规范
- 知识共享和团队学习
- 保持代码库一致性
审查维度
1. 功能正确性
- 代码是否实现了预期功能
- 边界条件是否处理
- 错误处理是否完善
2. 代码质量
- 命名是否清晰有意义
- 函数是否单一职责
- 是否有重复代码
- 复杂度是否可接受
3. 测试覆盖
- 单元测试是否充分
- 测试是否有意义（不是为测而测）
- 边界用例是否覆盖
4. 安全性
- 是否有注入风险
- 敏感数据是否保护
- 权限检查是否到位
5. 性能
- 是否有明显性能问题
- 数据库查询是否合理
- 是否有 N+1 问题
L1-STREAMLINED 检查清单
- 代码可读性良好
- 测试通过
- 无明显安全问题
- GOAL 追溯完整
通过标准
- 4 项全部通过（100%）
L2-BALANCED 增加项
- 设计模式使用合理
- 性能考量充分
- 文档/注释适当
L3-RIGOROUS 增加项
- 架构符合性审查
- 安全深度审查
- 性能基准测试
>> 命令
>>review_checklist_l1 # L1 审查清单
>>review_checklist_l2 # L2 审查清单
>>review_feedback # 生成审查反馈
审查反馈模板
## Code Review Feedback
### 概要
- PR: #XXX
- 审查人: @reviewer
- 状态: ✅ Approved / 🔄 Changes Requested
### 发现
#### 🔴 必须修改
- [ ] 问题描述 (文件:行号)
#### 🟡 建议修改
- [ ] 改进建议
#### 🟢 良好实践
- 值得表扬的点
### 总结
整体代码质量良好/需要改进...
相关 Skills
- 前置: tdd-cycle（代码已完成）
- 并行: code-quality（质量门禁）
- 原则: principle-solid, principle-dry, principle-kiss
TODO: 待细化不同语言的审查重点和自动化检查集成