代码审查¶
对未提交的更改进行完整的安全和质量审查:
-
获取已更改的文件:
git diff --name-only HEAD -
对于每个已更改的文件,检查:
安全问题(CRITICAL): - 硬编码的凭据、API 密钥或令牌 - SQL 注入漏洞 - XSS 漏洞 - 缺少输入验证 - 不安全的依赖 - 路径遍历风险
代码质量(HIGH):
- 函数超过 50 行
- 文件超过 800 行
- 嵌套深度超过 4 层
- 缺少错误处理
- console.log 语句
- TODO/FIXME 注释
- 公共 API 缺少 JSDoc
最佳实践(MEDIUM): - 可变模式(应改用不可变方式) - 在代码/注释中使用 emoji - 新代码缺少测试 - 可访问性问题(a11y)
- 生成报告,包含:
- 严重级别:CRITICAL、HIGH、MEDIUM、LOW
- 文件中的位置及行号
- 问题描述
-
建议修复
-
如果存在 CRITICAL 或 HIGH 问题,阻止 commit
切勿批准存在安全漏洞的代码!
本页原文 Markdown:在 AtomGit 查看·内容源自开源项目 ev/ECC