代码审查器
产品开发全链路 Skill #7 · 上线前的最后一道防线
功能
审查代码质量，发现潜在 Bug、性能问题、安全漏洞和风格问题。
核心工作流
Step 1：代码结构分析
- 目录结构是否合理
- 模块划分是否清晰
- 依赖管理是否规范
- 配置文件是否安全（密钥硬编码检查）
Step 2：代码质量检查
| 检查项 | 说明 | 严重级别 |
|---|---|---|
| 空值处理 | 是否有 None/null 检查 | P0 |
| 异常处理 | try-catch 是否覆盖关键路径 | P0 |
| 硬编码 | 密钥/URL/配置是否硬编码 | P0 |
| 性能问题 | N+1查询、循环内数据库调用 | P1 |
| 代码重复 | 是否有可复用的重复代码 | P1 |
| 命名规范 | 变量/函数/类命名是否一致 | P2 |
| 注释质量 | 复杂逻辑是否有注释 | P2 |
| 类型标注 | Python 是否有 type hints | P2 |
Step 3：安全审查
- SQL 注入风险
- XSS 风险
- 认证/授权漏洞
- 敏感信息泄露
- 第三方依赖漏洞
Step 4：改进建议
按优先级输出改进建议：
- P0：必须修复（阻塞上线）
- P1：建议修复（影响体验）
- P2：后续优化（代码质量）
输出格式
- 最终交付物：Markdown 审查报告
- 命名规范：
{项目名}-代码审查报告.md
硬约束
- 必须区分 P0/P1/P2 级别
- P0 问题必须给出修复方案
- 不能只发现问题，必须给出改进建议
- 如果是大型项目，按模块分批审查