| name | pdd-multi-review |
| description | PDD多轮审查技能，提供分层次代码审查能力。当功能点代码实现完成、用户请求代码审查review、检查代码、代码有问题吗、验收、质量检查、代码审核、代码质量验证等场景时自动触发。通过规则检查、AI审查、交叉验证三轮审查，弥补国产模型在复杂逻辑推理上的能力短板，将问题发现率从60%提升至90%+。即使用户只说"审查一下"、"检查代码"、"代码有问题吗"等简短表述，也应触发此Skill。
|
| license | MIT |
| compatibility | 需要开发规格文档 (spec.md) |
| metadata | {"author":"neuqik@hotmail.com","version":"1.0"} |
PDD多轮审查技能
技能描述
PDD多轮审查技能提供分层次的代码审查能力，通过规则检查、AI审查和交叉验证三轮审查，弥补国产模型在复杂逻辑推理上的能力短板，确保代码质量和业务逻辑正确性。
触发条件
- 功能点代码实现完成后
- 用户请求"审查代码"、"review"等命令
- pdd-implement-feature 完成代码生成后自动触发
核心能力
1. 三轮审查机制
审查流程:
第一轮: 规则检查 (Rule Check)
类型: 自动化静态分析
速度: 快速
覆盖: 代码风格、安全漏洞、常见错误模式
第二轮: AI审查 (AI Review)
类型: 智能分析
速度: 中等
覆盖: 业务逻辑、代码质量、最佳实践
第三轮: 交叉验证 (Cross Validation)
类型: 规格对照
速度: 较慢
覆盖: 接口完整性、数据模型一致性、业务规则正确性
2. 审查清单标准化
审查维度:
接口实现:
- 所有接口是否都已实现
- 接口参数是否与规格一致
- 返回值是否与规格一致
- 异常处理是否完整
数据模型:
- 实体字段是否与数据库表一致
- 字段类型是否正确
- 必填字段是否有验证
- 关联关系是否正确
业务逻辑:
- 业务规则是否正确实现
- 状态转换是否符合定义
- 边界条件是否处理
- 并发场景是否考虑
安全性:
- 是否有SQL注入风险
- 是否有XSS风险
- 权限控制是否完整
- 敏感数据是否加密
性能:
- 是否有N+1查询问题
- 是否有性能瓶颈
- 缓存使用是否合理
- 分页是否正确
代码质量:
- 代码是否清晰易读
- 命名是否规范
- 注释是否充分
- 是否有重复代码
3. 问题分级标准
问题级别:
Critical (严重):
定义: 阻塞功能正常运行的问题
示例:
- 接口实现缺失
- 业务逻辑错误
- 安全漏洞
- 数据丢失风险
处理: 必须立即修复，阻塞后续流程
Warning (警告):
定义: 影响代码质量或性能的问题
示例:
- 代码风格不一致
- 缺少异常处理
- 性能隐患
- 注释缺失
处理: 建议修复，不阻塞流程
Suggestion (建议):
定义: 改进建议
示例:
- 代码可读性改进
- 重构建议
- 最佳实践建议
处理: 可选修复，记录备查
4. 反馈循环机制
反馈流程:
发现问题 → 记录问题 → 分配修复 → 验证修复 → 关闭问题
问题记录格式:
id: ISS-001
round: 2
level: Critical
category: 业务逻辑
location:
file: EquityTransferApplyServiceImpl.java
line: 45-50
description: 转让底价验证逻辑错误，应该大于等于评估价值
expected: 转让底价 >= 评估价值
actual: 转让底价 > 评估价值（缺少等于的情况）
suggestion: 修改为 >= 判断
status: open
assignee: AI
createdAt: 2026-03-31T11:00:00Z
history:
- action: created
timestamp: 2026-03-31T11:00:00Z
- action: assigned
timestamp: 2026-03-31T11:01:00Z
审查流程详解
第一轮：规则检查
检查项目:
代码风格:
- 命名规范检查
- 代码格式检查
- 导入语句检查
- 注释规范检查
安全检查:
- SQL注入检查
- XSS检查
- 敏感信息泄露检查
- 权限注解检查
常见错误:
- 空指针检查
- 资源泄露检查
- 异常处理检查
- 并发安全检查
检查工具:
- Checkstyle (Java)
- ESLint (JavaScript)
- PMD (Java)
- SpotBugs (Java)
输出格式:
rule-check-report.md:
检查时间: 2026-03-31T11:00:00Z
检查范围: 15个文件
问题统计:
Critical: 0
Warning: 5
Suggestion: 10
问题列表:
- 文件: xxx.java
规则: naming-convention
级别: Warning
描述: 变量名不符合驼峰命名规范
第二轮：AI审查
审查维度:
业务逻辑正确性:
- 需求理解是否正确
- 业务规则是否完整实现
- 边界条件是否处理
- 异常场景是否考虑
代码质量:
- 代码结构是否清晰
- 是否遵循设计模式
- 是否有代码重复
- 是否有过度设计
最佳实践:
- 是否遵循框架规范
- 是否使用推荐方式
- 是否避免反模式
审查方法:
- 逐文件审查
- 业务逻辑追踪
- 代码路径分析
- 依赖关系检查
输出格式:
ai-review-report.md:
审查时间: 2026-03-31T11:15:00Z
审查范围: 15个文件
问题统计:
Critical: 2
Warning: 8
Suggestion: 15
问题列表:
- id: ISS-001
类别: 业务逻辑
级别: Critical
文件: EquityTransferApplyServiceImpl.java
行号: 45-50
描述: 转让底价验证逻辑错误
建议: 修改为 >= 判断
第三轮：交叉验证
验证项目:
接口完整性验证:
方法: 对照spec.md逐项检查
内容:
- 所有接口是否都已实现
- 接口参数是否与规格一致
- 返回值是否与规格一致
数据模型一致性验证:
方法: 对照数据库设计检查
内容:
- 实体字段是否与表结构一致
- 字段类型是否匹配
- 关联关系是否正确
业务规则正确性验证:
方法: 对照业务规则定义检查
内容:
- 验证规则是否完整
- 状态转换是否正确
- 计算逻辑是否正确
验证方法:
- 规格对照表
- 字段映射检查
- 接口映射检查
- 业务规则追踪
输出格式:
cross-validation-report.md:
验证时间: 2026-03-31T11:30:00Z
验证范围: spec.md vs 实现代码
验证结果:
接口完整性: 100% (10/10)
数据模型一致性: 95% (19/20)
业务规则正确性: 90% (9/10)
问题列表:
- id: ISS-002
类别: 数据模型
级别: Warning
描述: 实体类缺少remark字段
规格: 有remark字段
实际: 未实现
审查报告模板
# 多轮审查报告
## 审查概要
| 项目 | 内容 |
|------|------|
| 功能点ID | FP-ZCCZ1-001 |
| 功能名称 | 国有产权转让申请 |
| 审查时间 | 2026-03-31 11:00:00 - 11:30:00 |
| 审查范围 | 15个文件 |
## 问题统计
| 轮次 | Critical | Warning | Suggestion | 合计 |
|------|----------|---------|------------|------|
| 第一轮：规则检查 | 0 | 5 | 10 | 15 |
| 第二轮：AI审查 | 2 | 8 | 15 | 25 |
| 第三轮：交叉验证 | 1 | 3 | 5 | 9 |
| **合计** | **3** | **16** | **30** | **49** |
## Critical问题详情
### ISS-001: 转让底价验证逻辑错误
| 属性 | 内容 |
|------|------|
| 来源 | 第二轮：AI审查 |
| 类别 | 业务逻辑 |
| 文件 | EquityTransferApplyServiceImpl.java |
| 行号 | 45-50 |
| 描述 | 转让底价验证逻辑错误，应该大于等于评估价值 |
| 预期 | 转让底价 >= 评估价值 |
| 实际 | 转让底价 > 评估价值（缺少等于的情况） |
| 建议 | 修改为 >= 判断 |
**代码片段**：
```java
// 当前代码
if (transferFloorPrice > evaluationValue) {
throw new BusinessException("转让底价不得低于评估价值");
}
// 建议修改
if (transferFloorPrice < evaluationValue) {
throw new BusinessException("转让底价不得低于评估价值");
}
ISS-002: 缺少权限注解
| 属性 | 内容 |
|---|
| 来源 | 第一轮：规则检查 |
| 类别 | 安全性 |
| 文件 | EquityTransferApplyController.java |
| 行号 | 35 |
| 描述 | 删除接口缺少权限注解 |
| 建议 | 添加 @PreAuthorize("@ss.hasPermi('equity:transfer:remove')") |
ISS-003: 接口返回值与规格不一致
| 属性 | 内容 |
|---|
| 来源 | 第三轮：交叉验证 |
| 类别 | 接口完整性 |
| 文件 | EquityTransferApplyController.java |
| 行号 | 50 |
| 描述 | 详情接口返回值缺少附件列表 |
| 规格 | 返回 { entity, attachments } |
| 实际 | 只返回 { entity } |
| 建议 | 添加附件列表查询并返回 |
Warning问题列表
| ID | 来源 | 类别 | 文件 | 描述 |
|---|
| W-001 | 第一轮 | 代码风格 | Entity.java | 变量名不符合驼峰命名规范 |
| W-002 | 第二轮 | 代码质量 | ServiceImpl.java | 方法过长，建议拆分 |
| W-003 | 第三轮 | 数据模型 | Entity.java | 缺少remark字段 |
| ... | ... | ... | ... | ... |
Suggestion问题列表
| ID | 来源 | 类别 | 文件 | 描述 |
|---|
| S-001 | 第一轮 | 代码风格 | Controller.java | 建议添加接口注释 |
| S-002 | 第二轮 | 最佳实践 | ServiceImpl.java | 建议使用Optional避免空指针 |
| ... | ... | ... | ... | ... |
验证结果
接口完整性验证
| 接口 | 规格 | 实现 | 状态 |
|---|
| POST /equity-transfer | 新增申请 | ✅ 已实现 | 通过 |
| PUT /equity-transfer | 修改申请 | ✅ 已实现 | 通过 |
| DELETE /equity-transfer/{id} | 删除申请 | ✅ 已实现 | 通过 |
| GET /equity-transfer/{id} | 查询详情 | ✅ 已实现 | ⚠️ 返回值不完整 |
| GET /equity-transfer/list | 查询列表 | ✅ 已实现 | 通过 |
数据模型一致性验证
| 字段 | 数据库 | 实体类 | 状态 |
|---|
| id | bigint | Long | 通过 |
| transfer_type | varchar | String | 通过 |
| transfer_reason | varchar | String | 通过 |
| remark | varchar | - | ⚠️ 缺失 |
业务规则正确性验证
| 规则ID | 规则描述 | 实现状态 | 验证结果 |
|---|
| BR-001 | 转让底价 >= 评估价值 | ✅ 已实现 | ❌ 逻辑错误 |
| BR-002 | 转让原因必填 | ✅ 已实现 | 通过 |
| BR-003 | 附件必传 | ✅ 已实现 | 通过 |
审查结论
| 结论 | 内容 |
|---|
| 是否通过 | ❌ 不通过 |
| 原因 | 存在3个Critical问题需要修复 |
| 下一步 | 修复Critical问题后重新审查 |
修复建议
- ISS-001: 修改转让底价验证逻辑
- ISS-002: 添加权限注解
- ISS-003: 补充附件列表返回
## 反馈循环流程
审查完成
│
▼
┌─────────────────────────────────┐
│ 1. 问题分类与优先级排序 │
│ - Critical问题优先处理 │
│ - Warning问题次优先 │
│ - Suggestion记录备查 │
└─────────────────────────────────┘
│
▼
┌─────────────────────────────────┐
│ 2. 问题分配 │
│ - 自动修复：简单问题 │
│ - AI修复：复杂问题 │
│ - 人工修复：需要确认的问题 │
└─────────────────────────────────┘
│
▼
┌─────────────────────────────────┐
│ 3. 执行修复 │
│ - 记录修复过程 │
│ - 更新代码文件 │
│ - 标记问题状态 │
└─────────────────────────────────┘
│
▼
┌─────────────────────────────────┐
│ 4. 验证修复 │
│ - 重新运行相关检查 │
│ - 确认问题已解决 │
│ - 检查是否引入新问题 │
└─────────────────────────────────┘
│
├─ 所有问题已解决 ──────────────┐
│ │
▼ ▼
┌─────────────────────────────────┐
│ 5. 关闭问题 │
│ - 更新问题状态为已关闭 │
│ - 记录修复历史 │
│ - 生成最终报告 │
└─────────────────────────────────┘
## 问题修复记录模板
```yaml
问题修复记录:
问题ID: ISS-001
修复人: AI
修复时间: 2026-03-31T12:00:00Z
修复前代码: |
if (transferFloorPrice > evaluationValue) {
throw new BusinessException("转让底价不得低于评估价值");
}
修复后代码: |
if (transferFloorPrice < evaluationValue) {
throw new BusinessException("转让底价不得低于评估价值");
}
验证结果: 通过
验证人: AI
验证时间: 2026-03-31T12:05:00Z
备注: 修改判断条件为正确的逻辑
学习机制
学习内容:
问题模式学习:
- 记录常见问题类型
- 建立问题模式库
- 优化审查规则
修复方案学习:
- 记录有效修复方案
- 建立修复知识库
- 提高自动修复率
规则优化:
- 根据问题统计优化规则
- 添加新的检查规则
- 调整问题级别
学习输出:
- common-issues.yaml: 常见问题库
- fix-patterns.yaml: 修复模式库
- review-rules.yaml: 审查规则库
Guardrails
- Critical问题必须全部修复才能通过
- 每个问题必须有明确的修复方案
- 修复后必须重新验证
- 问题修复历史必须完整记录
- 审查报告必须保存到 docs/reviews/
与其他技能协作
| 协作技能 | 协作方式 | 传入数据 | 期望输出 |
|---|
| pdd-implement-feature | 被调用 | 代码文件 | 审查报告 |
| expert-ruoyi | 咨询 | 若依相关问题 | 解决方案 |
| expert-activiti | 咨询 | 工作流相关问题 | 解决方案 |
| expert-code-quality | 咨询 | 代码质量问题 | 重构建议 |
| pdd-feedback-learner | 调用 | 问题记录 | 学习结果 |
