代码审查器Skill code-reviewer

---

name: code-reviewer description: 使用此技能审查代码。它支持本地更改（暂存或工作树） 和远程Pull Requests（通过ID或URL）。它专注于正确性、可维护性、 和项目标准的遵守。

代码审查者

此技能指导代理进行专业且全面的代码审查，适用于本地开发和远程Pull Requests。

工作流程

1. 确定审查目标

• 远程PR: 如果用户提供了PR编号或URL（例如，“Review PR #123”），则针对该远程PR。
• 本地更改: 如果没有指定具体PR，或用户要求“审查我的更改”，则针对当前本地文件系统状态（暂存和非暂存更改）。

2. 准备

对于远程PRs:

1. 检出: 使用GitHub CLI检出PR。

   gh pr checkout <PR_NUMBER>
   

2. 预检: 执行项目的标准验证套件以早期捕获自动化失败。

   npm run preflight
   

3. 上下文: 阅读PR描述和任何现有评论，以理解目标和历史。

对于本地更改:

1. 识别更改:
   • 检查状态: git status
   • 阅读差异: git diff（工作树）和/或 git diff --staged（暂存）。
2. 预检（可选）: 如果更改很大，询问用户是否希望在审查前运行 npm run preflight。

3. 深度分析

基于以下支柱分析代码更改：

• 正确性: 代码是否实现了其既定目的，没有错误或逻辑错误？
• 可维护性: 代码是否干净、结构良好，并且易于将来理解和修改？考虑因素如代码清晰度、模块化和遵循既定设计模式。
• 可读性: 代码是否良好注释（必要时）并根据项目的编码风格指南一致格式化？
• 效率: 更改是否引入了明显的性能瓶颈或资源低效？
• 安全性: 是否有潜在的安全漏洞或不安全的编码实践？
• 边缘情况和错误处理: 代码是否适当处理边缘情况和潜在错误？
• 可测试性: 新的或修改的代码是否被测试充分覆盖（即使预检检查通过）？建议改进覆盖率或鲁棒性的额外测试案例。

4. 提供反馈

结构

• 总结: 审查的高层概述。
• 发现:
  • 关键: 错误、安全问题或破坏性更改。
  • 改进: 提高代码质量或性能的建议。
  • 小瑕疵: 格式化或次要风格问题（可选）。
• 结论: 明确推荐（批准 / 请求更改）。

语气

• 具有建设性、专业且友好。
• 解释 为什么 请求更改。
• 对于批准，承认贡献的具体价值。

5. 清理（仅限远程PRs）

• 审查后，询问用户是否想切换回默认分支（例如，main 或 master）。