关于 reviewing-code
reviewing-code 是面向代码审查者的技能,用于提供按优先级排列且可执行的反馈。它围绕安全性、正确性、性能和可维护性组织检查,并提供统一的报告格式。
主要功能
- 01按优先级排列发现项
- 02检查安全与正确性风险
- 03审视性能和逻辑问题
- 04指出可维护性问题
- 05提供结构化审查格式
适用场景
- 01在合并前审查代码变更
- 02检查常见的安全与正确性问题
- 03生成格式统一的审查反馈
安装
复制下面的命令,在终端中运行。
$
npx skills add https:/ / github.com/ augmnt/ webdev-skills --skill reviewing-code安装后在 Claude Code 中输入/reviewing-code确认已加载。
附带文件REACT-PATTERNS.md
详细介绍
代码审查
优先提供可执行的反馈。默认对方出发点是好的。
优先级
| 级别 | 前缀 | 含义 |
|---|---|---|
| 阻断项 | 🚨 | 合并前必须修复 |
| 建议 | ⚠️ | 应考虑 |
| 小问题 | 💭 | 可选的润色 |
| 问题 | ❓ | 需要澄清 |
| 表扬 | ✨ | 做得很好 |
检查清单
阻断项(安全与正确性)
- 代码中没有密钥或凭据
- 已验证/清理用户输入
- 在需要的地方进行了身份验证检查
- 已处理边界情况(空值、空内容、错误)
- 异步代码中没有竞争条件
高优先级(性能与逻辑)
- 没有 N+1 查询
- 大型列表已分页
- 渲染循环中没有执行耗时操作
- 错误处理恰当
中优先级(可维护性)
- 函数各自只负责一项职责
- 没有复制粘贴造成的重复
- 命名清晰
输出格式
## Summary
[Approve / Request Changes / Discuss]
## 🚨 Blockers
- `file:line` - Issue and fix
## ⚠️ Suggestions
- `file:line` - Suggestion and why
## 💭 Nits
- `file:line` - Minor improvement
## ✨ Good Stuff
- What was done well
常见问题
React:缺少 useEffect 依赖项、应派生却未派生的状态、属性逐层传递
TypeScript:any 类型、缺少空值检查
API:缺少加载/错误状态、查询没有结果数量上限
有关详细的 React/hooks 模式和无障碍检查,请参阅 REACT-PATTERNS.md。
评论
支持 Markdown