跳转至

代码审查

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

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

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

安全问题(CRITICAL): - 硬编码的凭据、API 密钥或令牌 - SQL 注入漏洞 - XSS 漏洞 - 缺少输入验证 - 不安全的依赖 - 路径遍历风险

代码质量(HIGH): - 函数超过 50 行 - 文件超过 800 行 - 嵌套深度超过 4 层 - 缺少错误处理 - console.log 语句 - TODO/FIXME 注释 - 公共 API 缺少 JSDoc

最佳实践(MEDIUM): - 可变模式(应改用不可变方式) - 在代码/注释中使用 emoji - 新代码缺少测试 - 可访问性问题(a11y)

  1. 生成报告,包含:
  2. 严重级别:CRITICAL、HIGH、MEDIUM、LOW
  3. 文件中的位置及行号
  4. 问题描述
  5. 建议修复

  6. 如果存在 CRITICAL 或 HIGH 问题,阻止 commit

切勿批准存在安全漏洞的代码!

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