Code Review

审查代码的质量、安全性和可维护性。当用户说'review 这段代码'、'帮我看看这个 PR'、'检查一下代码质量'、'这段代码有没有问题'时使用。

diguike d4e2661 3 files · 4.3 KB Updated

File contents

PR 上下文

!gh pr view $0 --json title,body,labels,changedFiles

变更概览

!gh pr diff $0 --stat

完整 diff

!gh pr diff $0

审查指令

基于以上 PR 信息,进行代码审查。

审查维度

  1. Bug 风险:空指针、未处理异常、边界条件、类型错误
  2. 安全问题:XSS、SQL 注入、敏感信息泄露、未授权访问
  3. 可维护性:命名清晰度、函数长度、重复代码、过度耦合
  4. 性能:不必要的渲染、内存泄漏、大循环中的重复计算

严重度标准

  • 🔴 Critical:必须修复才能合并。安全漏洞、数据丢失风险、线上必现 bug。
  • 🟡 Warning:建议修复。影响可维护性、潜在的边界 bug、性能隐患。
  • 🔵 Suggestion:可以考虑。更优雅的写法、微小的性能优化、代码风格。

不要把所有问题都标 Critical——只有真正阻塞合并的才算。

参考知识

根据技术栈,按需读取:

  • React 项目 → references/react.md
  • Go 项目 → references/go.md

这些参考文档包含该技术栈的常见陷阱和团队最佳实践,审查时作为判断依据。

输出格式

严格按以下格式输出:

问题列表
# 文件 行号 严重度 问题描述 修复建议
总结
  • 总体评分:X/10
  • 主要风险:一句话概括最大的问题
  • 亮点:值得肯定的做法
  • 建议优先级:先修什么,后修什么

diguike/book-claude-skill/tree/main/skills/code-review-snapshots/v4-external-knowledge commit d4e266177e

Frequently asked questions

npx skillmds@latest add diguike/code-review-9