c7261394b0
* fix(i18n): re-point translations to code-review-specialist rename The English code-review skill was renamed to code-review-specialist in #127 to avoid shadowing the new built-in /code-review command. Mirror that rename across the uk, vi, ja, zh locales: rename each 03-skills/code-review/ directory to code-review-specialist/, update the i18n-source markers in the moved ja files, and fix path references in INDEX.md, QUICK_REFERENCE.md, and the vi TRANSLATION_QUEUE.md. * chore: ignore .asm-improver and .gitissue tooling state * fix(i18n): re-point remaining code-review install paths to code-review-specialist
1.3 KiB
1.3 KiB
代码审查检查清单
安全检查
- 没有硬编码的凭据或密钥
- 所有用户输入都做了校验
- 使用参数化查询，防止 SQL 注入
- 所有会修改状态的操作都有 CSRF 防护
- 使用正确的转义，防止 XSS
- 受保护的端点有身份验证检查
- 资源访问有授权检查
- 密码使用安全哈希算法（bcrypt、argon2）
- 日志中没有敏感数据
- 强制使用 HTTPS
性能检查
- 没有 N+1 查询
- 索引使用合理
- 在有价值的地方做了缓存
- 主线程上没有阻塞操作
- 正确使用 async/await
- 大数据集已经分页
- 数据库连接已做连接池
- 正则表达式已优化
- 没有不必要的对象创建
- 没有内存泄漏
质量检查
- 函数少于 50 行
- 变量命名清晰
- 没有重复代码
- 错误处理合理
- 注释解释的是 WHY，而不是 WHAT
- 生产环境中没有 console.log
- 有类型检查（TypeScript / JSDoc）
- 遵循 SOLID 原则
- 正确应用设计模式
- 代码具备自解释性
测试检查
- 已编写单元测试
- 覆盖了边界情况
- 测试了错误场景
- 有集成测试
- 覆盖率大于 80%
- 没有不稳定测试
- 外部依赖已做 mock
- 测试名称清晰