关于 code-review-analysis
供开发者在合并前评估代码变更的系统化审查技能,涵盖质量、安全、性能、测试和可维护性,并指导如何提供建设性反馈。
主要功能
- 01初步评估变更
- 02代码质量分析
- 03安全与性能审查
- 04测试审查
- 05建设性反馈实践
适用场景
- 01合并前审查拉取请求
- 02检查变更中的安全问题
- 03评估性能与可维护性
- 04向开发者提供可执行的反馈
安装
复制下面的命令,在终端中运行。
$
npx skills add https:/ / github.com/ aj-geddes/ useful-ai-prompts --skill code-review-analysis安装后在 Claude Code 中输入/code-review-analysis确认已加载。
包含的文件
除 SKILL.md 外,安装时一起下载的参考资料和脚本。
references/best-practices.mdcode-quality-analysis.mdinitial-assessment.mdperformance-review.mdsecurity-review.mdtesting-review.mdscripts/validate-schema.shtemplates/migration-template.sql详细介绍
代码审查分析
目录
概述
按照行业标准,系统地审查代码质量、安全性、性能、可维护性和最佳实践。
适用场景
- 审查拉取请求和合并请求
- 合并前分析代码质量
- 发现安全漏洞
- 为开发者提供建设性反馈
- 确保符合编码规范
- 通过代码审查指导开发者
快速开始
最简可运行示例:
# Check the changes
git diff main...feature-branch
# Review file changes
git diff --stat main...feature-branch
# Check commit history
git log main...feature-branch --oneline
参考指南
references/ 目录中的详细实现:
| 指南 | 内容 |
|---|---|
| 初步评估 | 初步评估 |
| 代码质量分析 | 代码质量分析 |
| 安全审查 | 安全审查 |
| 性能审查 | 性能审查 |
| 测试审查 | 测试审查 |
| 最佳实践 | 最佳实践 |
最佳实践
✅ 应该
- 保持建设性和尊重
- 解释建议背后的“原因”
- 提供代码示例
- 不清楚时提出问题
- 肯定良好实践
- 聚焦重要问题
- 考虑上下文
- 对于复杂问题,提议结对编程
❌ 不应该
- 过度批评或针对个人
- 吹毛求疵地指出细微的风格问题(使用自动化工具)
- 因主观偏好而阻止合并
- 一次审查过多更改(超过 400 行)
- 忘记检查测试
- 忽视安全影响
- 仓促审查
评论
支持 Markdown