关于 code-reviewer
面向开发者的代码审查技能,按质量、安全、错误处理、性能和测试检查代码,并按严重程度整理反馈。
主要功能
- 01代码质量检查清单
- 02安全与输入检查
- 03错误和性能审查
- 04测试覆盖率审查
- 05按严重程度反馈
适用场景
- 01审查一组源文件
- 02查找安全或校验问题
- 03按严重程度安排修复
安装
复制下面的命令,在终端中运行。
$
npx skills add https:/ / github.com/ nodnarbnitram/ claude-code-extensions --skill code-reviewer安装后在 Claude Code 中输入/code-reviewer确认已加载。
详细介绍
代码审查员
全面审查代码,重点关注质量、安全性和可维护性。
说明
- 使用 Read 工具读取目标文件
- 使用 Grep 搜索模式和相关代码
- 使用 Glob 查找相关文件
- 根据审查清单分析代码
- 按严重程度提供结构化反馈
审查清单
代码质量
- 代码简洁易读
- 函数和变量命名恰当
- 没有重复代码(DRY 原则)
- 对复杂逻辑添加了适当注释
- 代码风格一致
安全性
- 没有暴露的密钥或 API 密钥
- 已实施输入验证
- 已防止 SQL 注入
- 已防止 Web 代码中的 XSS
- 已进行适当的身份验证和授权检查
错误处理
- 已捕获并妥善处理错误
- 错误消息有意义
- 没有静默失败
- 已添加适当的日志,便于调试
性能
- 没有明显的性能瓶颈
- 算法和数据结构高效
- 在需要时使用了适当的缓存
- 数据库查询经过优化
测试
- 测试覆盖充分
- 已测试边界情况
- 测试易于阅读和维护
输出格式
按严重程度整理反馈:
严重(必须修复)
可能导致安全漏洞、数据丢失或崩溃的问题。
警告(应当修复)
可能导致错误、性能不佳或维护问题的问题。
建议(可考虑)
针对可读性、一致性或最佳实践的改进建议。
反馈示例
### Critical
- **SQL Injection vulnerability** in `user_service.py:45`
- User input passed directly to query without sanitization
- Fix: Use parameterized queries
### Warning
- **Missing error handling** in `api_client.py:23`
- Network errors will crash the application
- Fix: Add try/catch with appropriate error response
### Suggestion
- Consider extracting the validation logic in `validators.py:78-95` into a separate function for reusability
评论
支持 Markdown