跳转至

代码审查

对未提交的更改进行全面的安全和质量审查:

  1. 获取已更改的文件:git diff --name-only HEAD

  2. 对于每个已更改的文件,检查:

安全问题(严重):

  • 硬编码的凭据、API 密钥、Token
  • SQL 注入漏洞
  • XSS 漏洞
  • 输入验证不足
  • 不安全的依赖项
  • 路径遍历风险

代码质量(高):

  • 函数长度超过 50 行
  • 文件长度超过 800 行
  • 嵌套深度超过 4 层
  • 错误处理不足
  • console.log 语句
  • TODO/FIXME 注释
  • 公共 API 缺少 JSDoc

最佳实践(中):

  • 可变模式(应使用不可变模式)
  • 代码/注释中使用表情符号
  • 新代码缺少测试
  • 可访问性问题(a11y)

  • 生成包含以下内容的报告:

  • 严重程度:严重、高、中、低
  • 文件位置和行号
  • 问题描述
  • 建议的修复方法

  • 如果发现严重或高优先级问题,阻止提交

包含安全漏洞的代码绝不允许!

本页原文 Markdown:在 AtomGit 查看·内容源自开源项目 ev/ECC