brainstorming
brainstorming
You MUST use this before any creative work - creating features, building components, adding functionality, modifying behavior, or planning anything new, in software or out of it (a talk, a business, a renovation).
code-reviewer-cn代码审查,生成中文报告,基于 anthropics/skills 和 obra/superpowers 项目
Independent GitHub snapshot · Automated classification · Original documentation governs
基于 anthropics/skills 和 obra/superpowers 的代码审查方法论。
当用户请求代码审查、PR Review、或代码质量检查时激活。
| 级别 | 说明 | 处理方式 |
|---|---|---|
| 🔴 Critical | 安全漏洞、数据丢失风险 | 必须修复 |
| 🟠 Major | 逻辑错误、性能问题 | 强烈建议修复 |
| 🟡 Minor | 代码风格、可读性 | 建议改进 |
| 🔵 Info | 最佳实践建议 | 可选优化 |
# 代码审查报告
## 📋 概览
- **审查范围**:[文件/模块列表]
- **代码行数**:[总行数]
- **审查时间**:[日期]
## 📊 评分
| 维度 | 评分 | 说明 |
|------|------|------|
| 正确性 | X/10 | |
| 安全性 | X/10 | |
| 性能 | X/10 | |
| 可维护性 | X/10 | |
| **综合** | **X/10** | |
## 🔴 Critical 问题 (X个)
### [问题标题]
- **位置**:`文件名:行号`
- **问题**:[描述]
- **风险**:[影响说明]
- **建议**:
```[语言]
// 修复代码
[同上格式]
[同上格式]
[最佳实践建议列表]
[代码中做得好的地方]
## 审查原则
1. **先理解,后评判** - 不要急于下结论
2. **对事不对人** - 关注代码,不评价作者
3. **给出解决方案** - 不只指出问题,还要提供修复建议
4. **保持建设性** - 语气友善,鼓励改进
## 常见检查项
### JavaScript/TypeScript
- [ ] 是否有未处理的 Promise rejection
- [ ] 是否有潜在的内存泄漏
- [ ] TypeScript 类型是否完整
- [ ] 是否有 any 类型滥用
### Python
- [ ] 是否有类型注解
- [ ] 异常处理是否完整
- [ ] 是否有资源泄漏(文件、连接)
- [ ] 是否遵循 PEP8
### 通用
- [ ] 是否有硬编码的敏感信息
- [ ] 是否有适当的日志记录
- [ ] 是否有单元测试覆盖
- [ ] 是否有文档说明