mcpField
C
code-reviewer

code-reviewer

github.com/404kidwiz测试与质量107 次安装
代码仓库

关于 code-reviewer

面向团队和开发者的代码审查技能，用于评估代码变更。重点检查正确性和安全性，也涵盖性能、错误处理、测试与可维护性。

主要功能

  1. 01检查正确性与逻辑错误
  2. 02审查安全漏洞
  3. 03评估性能影响
  4. 04检查错误处理与测试覆盖
  5. 05提供分级且可执行的反馈

适用场景

  1. 01合并前审查拉取请求
  2. 02审计代码安全风险
  3. 03检查变更中的性能问题

安装

复制下面的命令，在终端中运行。

$npx skills add https://github.com/404kidwiz/claude-supercode-skills --skill code-reviewer

安装后在 Claude Code 中输入/code-reviewer确认已加载。

详细介绍

代码审查员

用途

提供全面的代码审查专业知识，重点关注正确性、安全性、性能和可维护性。发现 Bug、安全漏洞和代码质量问题，并提出改进建议。

适用场景

  • 审查拉取请求或代码变更
  • 对代码进行安全审计
  • 在合并前发现潜在 Bug
  • 确保代码遵循最佳实践
  • 检查性能问题
  • 验证错误处理
  • 审查代码中的架构决策

快速入门

在以下情况下调用此技能：

  • 审查拉取请求或代码变更
  • 对代码进行安全审计
  • 在合并前发现潜在 Bug
  • 确保代码遵循最佳实践
  • 检查性能问题

以下情况请勿调用：

  • 调试运行时问题（使用调试器）
  • 重构代码结构（使用 refactoring-specialist）
  • 编写新代码（使用特定语言的技能）
  • 审查系统架构（使用 architect-reviewer）

决策框架

Review Priority:
├── Security issues → Block merge, fix immediately
├── Correctness bugs → Block merge, require fix
├── Performance issues → Discuss, may block
├── Code style issues → Suggest, non-blocking
├── Documentation gaps → Suggest, non-blocking
└── Refactoring opportunities → Note for future

核心工作流

1. 拉取请求审查
  1. 根据 PR 描述了解其意图
  2. 检查正确性和逻辑错误
  3. 检查安全漏洞
  4. 评估对性能的影响
  5. 验证错误处理是否完整
  6. 检查测试覆盖率
  7. 提供可执行的反馈
2. 安全专项审查
  1. 检查输入验证和清理
  2. 审查身份验证和授权
  3. 查找注入漏洞
  4. 验证敏感数据处理
  5. 检查是否存在硬编码密钥
  6. 审查依赖项安全性
  7. 评估加密技术的使用
3. 性能审查
  1. 识别 N+1 查询模式
  2. 检查不必要的内存分配
  3. 审查算法复杂度
  4. 评估缓存机会
  5. 检查阻塞操作
  6. 审查数据库查询效率

最佳实践

  • 审查代码，而非作者
  • 明确指出问题和修复方法
  • 解释建议背后的原因
  • 按严重程度排列评论优先级
  • 也要肯定良好的模式
  • 优先使用自动化工具（代码检查器、SAST）

反模式

反模式问题正确做法
吹毛求疵地挑剔风格浪费时间，让作者感到沮丧使用自动格式化工具
缺少上下文审查者不了解变更内容阅读 PR 描述和关联的问题
因个人观点阻止合并不必要地延迟交付区分必须修复的问题和可选改进
随手式审查留下评论后不跟进解决跟进讨论并推动解决
没有正面反馈打击作者的积极性肯定良好的模式

评论

支持 Markdown