Commit Review

报告示例：

📊 CODE REVIEW REPORT
============================================================

🔴 Critical Issues: 3
🟡 Warnings: 7
🔵 Info: 12
✅ Total Issues: 22

检查内容（脚本实际实现）

安全性：

• 硬编码密码/密钥/Token
• SQL 注入拼接风险（简单模式）
• eval/exec 使用

代码质量：

• 非测试文件中的 print(...)
• 超长行（>120）
• 无 issue 号的 TODO/FIXME/HACK/XXX
• 注释掉的代码

最佳实践：

• 空 except 块
• 全局变量（大写常量赋值）
• 魔法数字（>=3 位数字）

工作流程

1. 解析参数并确定 commit 范围（无参数则检查未暂存变更）
2. 使用 git diff --name-only 获取变更文件
3. 对每个文本文件读取 diff，仅检查新增行
4. 输出按严重性分组的报告

参考资料

当需要更完整的人工审查标准时，查阅：

• references/review-standards.md：代码审查标准、严重性分级、语言实践与性能建议
• references/security-checklist.md：安全审查清单与 OWASP Top 10 覆盖

报告输出

审查报告包含：

📊 CODE REVIEW REPORT
============================================================

🔴 Critical Issues: 3
🟡 Warnings: 7
🔵 Info: 12
✅ Total Issues: 22

============================================================
🔴 CRITICAL ISSUES (Must Fix)
============================================================

🔴 [Security] src/auth.py:45
   Hardcoded password detected
   Code: password = "admin123"

🟡 [Code Quality] src/utils.js:123
   Remove print statement - use proper logging
   Code: console.log("Debug info")

使用限制与已知边界

• 仅分析新增行，不检查删除或上下文代码
• 规则为正则匹配，可能存在误报或漏报
• 仅检查脚本内定义的文本扩展名列表
• 不分析依赖、配置、构建脚本或运行时行为
• 不覆盖 XSS、空指针、资源泄漏等更深层问题（需人工复核）

常见问题

❌ Error: Not in a git repository：需要在 Git 仓库目录下运行
✅ No changes found to review.：指定范围或工作区当前无变更

人工复核建议

• 使用 references/review-standards.md 进行完整代码审查
• 使用 references/security-checklist.md 进行安全审查补充

注意事项

• 确保在 Git 仓库目录下执行
• 审查结果仅供参考，最终决策由开发者负责