> 四维代码审查系统:安全 / 性能 / 质量 / 可维护性
当用户要求以下操作时使用此 Skill:
触发关键词:代码审查、code review、review、审查、审计、安全审查、质量评估
1. 获取代码 → 2. 四维分析 → 3. 评分定级 → 4. 生成报告 → 5. 输出建议
根据场景选择合适的代码获取方式:
| 场景 | 方法 |
|---|---|
| ------ | ------ |
| 用户直接粘贴代码 | 直接使用,进入 Step 2 |
| 本地文件 | read 工具读取文件内容 |
| GitHub PR | exec + git diff 或 gh CLI |
| Git 仓库 | exec + git log / git show |
| 飞书文档中的代码 | feishu-fetch-doc 技能获取内容 |
| 网页代码 | web_fetch 抓取页面内容 |
按以下四个维度逐一审查,每个维度使用对应的检查清单(见 templates/ 目录):
| 维度 | 关注点 | 检查清单 |
|---|---|---|
| ------ | -------- | ---------- |
| 🔒 安全 | 注入、认证、加密、敏感数据 | templates/checklist-security.md |
| ⚡ 性能 | 查询优化、缓存、并发、内存 | templates/checklist-performance.md |
| 🏆 质量 | 命名、错误处理、类型、SOLID | templates/checklist-quality.md |
| 🔧 可维护 | 复杂度、文档、测试、可扩展性 | templates/checklist-maintainability.md |
对每个维度进行 1-10 分评分,并计算总分:
| 分数区间 | 等级 | 含义 |
|---|---|---|
| ---------- | ------ | ------ |
| 9-10 | ⭐ 优秀 | 生产级代码,几乎无问题 |
| 7-8 | ✅ 良好 | 少量改进点,可直接合入 |
| 5-6 | ⚠️ 一般 | 存在较多改进点,建议修改后合入 |
| 3-4 | 🔶 较差 | 存在严重问题,需要大幅修改 |
| 1-2 | 🔴 危险 | 存在安全隐患或致命缺陷,禁止合入 |
总分计算:(安全 + 质量 + 性能 + 可维护) / 4,四舍五入到小数点后 1 位
使用 templates/finding-template.md 格式记录每个发现的问题,然后生成综合报告。
报告末尾给出:
重点关注:
重点关注:
重点关注:
重点关注:
当需要深度分析代码复杂度时,可使用提供的辅助脚本:
# 分析单个文件的基本指标
python3 <skill_dir>/scripts/analyze-metrics.py <file>
# 比较修改前后的复杂度变化
python3 <skill_dir>/scripts/compare-complexity.py <before_file> <after_file>
> 替换为本技能的实际路径(/Users/narain/.openclaw/workspace/skills/code-review)
# 代码审查报告
**文件**:`path/to/file.py`
**审查时间**:YYYY-MM-DD HH:MM
**审查范围**:安全 / 性能 / 质量 / 可维护性
## 总评
| 维度 | 评分 | 等级 |
|------|------|------|
| 🔒 安全 | X/10 | ⭐/✅/⚠️/🔶/🔴 |
| ⚡ 性能 | X/10 | ... |
| 🏆 质量 | X/10 | ... |
| 🔧 可维护 | X/10 | ... |
| **综合** | **X/10** | ... |
## 发现的问题
### [严重程度] 问题标题
- **位置**:文件名:行号
- **维度**:安全/性能/质量/可维护
- **描述**:问题说明
- **影响**:可能导致的后果
- **建议**:修复方案 + 代码示例
## 亮点
- 值得推广的好实践
## 建议
- **推荐操作**:✅ Approve / ⚠️ Request Changes
- **优先修复**:Top 3 改进建议
# PR 审查报告
**PR**:#123 - PR 标题
**分支**:feature/xxx → main
**变更统计**:+X / -Y 行,Z 个文件
## 总评
(同上格式,评分基于所有变更文件的综合评估)
## 按文件审查
### `src/module/file.py` (+10/-5)
(逐文件列出问题)
## 依赖影响分析
- 受影响的上下游模块
## 建议
(同上格式)
共 1 个版本