メニュー
PDD多轮审查技能，提供分层次代码审查能力。当功能点代码实现完成、用户请求代码审查review、检查代码、代码有问题吗、验收、质量检查、代码审核、代码质量验证等场景时自动触发。通过规则检查、AI审查、交叉验证三轮审查，弥补国产模型在复杂逻辑推理上的能力短板，将问题发现率从60%提升至90%+。即使用户只说"审查一下"、"检查代码"、"代码有问题吗"等简短表述，也应触发此Skill。
Codex または Claude でインストール この Prompt をコピーして Codex、Claude、または他のアシスタントに貼り付けると、Skill ページを確認してインストールできます。
SOC 職業分類に基づく
| 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多轮审查技能提供分层次的代码审查能力，通过规则检查、AI审查和交叉验证三轮审查，弥补国产模型在复杂逻辑推理上的能力短板，确保代码质量和业务逻辑正确性。
审查流程:
第一轮: 规则检查 (Rule Check)
类型: 自动化静态分析
速度: 快速
覆盖: 代码风格、安全漏洞、常见错误模式
第二轮: AI审查 (AI Review)
类型: 智能分析
速度: 中等
覆盖: 业务逻辑、代码质量、最佳实践
第三轮: 交叉验证 (Cross Validation)
类型: 规格对照
速度: 较慢
覆盖: 接口完整性、数据模型一致性、业务规则正确性
审查维度:
接口实现:
- 所有接口是否都已实现
- 接口参数是否与规格一致
- 返回值是否与规格一致
- 异常处理是否完整
数据模型:
- 实体字段是否与数据库表一致
- 字段类型是否正确
- 必填字段是否有验证
- 关联关系是否正确
业务逻辑:
- 业务规则是否正确实现
- 状态转换是否符合定义
- 边界条件是否处理
- 并发场景是否考虑
安全性:
- 是否有SQL注入风险
- 是否有XSS风险
- 权限控制是否完整
- 敏感数据是否加密
性能:
- 是否有N+1查询问题
- 是否有性能瓶颈
- 缓存使用是否合理
- 分页是否正确
代码质量:
- 代码是否清晰易读
- 命名是否规范
- 注释是否充分
- 是否有重复代码
问题级别:
Critical (严重):
定义: 阻塞功能正常运行的问题
示例:
- 接口实现缺失
- 业务逻辑错误
- 安全漏洞
- 数据丢失风险
处理: 必须立即修复，阻塞后续流程
Warning (警告):
定义: 影响代码质量或性能的问题
示例:
- 代码风格不一致
- 缺少异常处理
- 性能隐患
- 注释缺失
处理: 建议修复，不阻塞流程
Suggestion (建议):
定义: 改进建议
示例:
- 代码可读性改进
- 重构建议
- 最佳实践建议
处理: 可选修复，记录备查
反馈流程:
发现问题 → 记录问题 → 分配修复 → 验证修复 → 关闭问题
问题记录格式:
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-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("转让底价不得低于评估价值");
}
| 属性 | 内容 |
|---|---|
| 来源 | 第一轮：规则检查 |
| 类别 | 安全性 |
| 文件 | EquityTransferApplyController.java |
| 行号 | 35 |
| 描述 | 删除接口缺少权限注解 |
| 建议 | 添加 @PreAuthorize("@ss.hasPermi('equity:transfer:remove')") |
| 属性 | 内容 |
|---|---|
| 来源 | 第三轮：交叉验证 |
| 类别 | 接口完整性 |
| 文件 | EquityTransferApplyController.java |
| 行号 | 50 |
| 描述 | 详情接口返回值缺少附件列表 |
| 规格 | 返回 { entity, attachments } |
| 实际 | 只返回 { entity } |
| 建议 | 添加附件列表查询并返回 |
| ID | 来源 | 类别 | 文件 | 描述 |
|---|---|---|---|---|
| W-001 | 第一轮 | 代码风格 | Entity.java | 变量名不符合驼峰命名规范 |
| W-002 | 第二轮 | 代码质量 | ServiceImpl.java | 方法过长，建议拆分 |
| W-003 | 第三轮 | 数据模型 | Entity.java | 缺少remark字段 |
| ... | ... | ... | ... | ... |
| 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问题后重新审查 |
## 反馈循环流程
审查完成 │ ▼ ┌─────────────────────────────────┐ │ 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: 审查规则库
| 协作技能 | 协作方式 | 传入数据 | 期望输出 |
|---|---|---|---|
| pdd-implement-feature | 被调用 | 代码文件 | 审查报告 |
| expert-ruoyi | 咨询 | 若依相关问题 | 解决方案 |
| expert-activiti | 咨询 | 工作流相关问题 | 解决方案 |
| expert-code-quality | 咨询 | 代码质量问题 | 重构建议 |
| pdd-feedback-learner | 调用 | 问题记录 | 学习结果 |
"PDD框架下的业务分析Skill，运用专业方法论进行需求分析和业务建模。当用户输入/analyze、/audit、/doc等命令，或需要对业务流程、管理制度、Excel表单进行专业分析时触发。支持中文触发：业务分析、需求分析、需求建模、5W1H分析、MECE、流程分析。"
"PDD熵减智能体，持续监控和偿还技术债务，防止系统腐化。当用户需要代码清理、文档更新、技术债务管理、架构对齐、熵减、垃圾回收、清理技术债务时自动触发。即使用户只说'熵减'、'清理技术债务'或'垃圾回收'，也应触发此Skill。支持中文触发：熵减、技术债务、代码清理、文档更新、架构对齐、垃圾回收。"
根据开发规格实现功能点代码的核心Skill。当用户想要开始编码实现、根据规格文档生成代码、实现功能点时调用此Skill。此Skill会自动调用pdd-template-engine生成基础代码框架，然后由software-engineer补充业务逻辑。即使只有规格文档没有明确说'实现'，只要涉及代码生成、功能开发，都应触发此Skill。支持中文触发：实现功能点、编码实现、开始编码、功能开发、代码实现、PDD实现。
PRD驱动开发的主入口Skill，协调整个开发流程。当用户想要基于PRD文档进行功能开发、从需求文档生成代码、执行PDD方法论流程、开发业务模块、实现完整功能、'搞个功能'、'资产转让'、'国有产权转让'、'帮我搞个资产转让的功能'时必须调用此Skill。即使用户没有明确说'使用PDD'，只要涉及PRD文档、需求文档、功能点开发、规格文档、模块开发、根据文档开发、业务功能实现、'开发ZCCZ'、'我想开发'、'搞个功能'等场景，都应触发此Skill。此Skill会自动协调pdd-ba、pdd-extract-features、pdd-generate-spec、pdd-implement-feature等子Skill完成从需求分析到代码交付的完整流程。注意：单一接口设计、调试问题、文档查询等场景不应触发此Skill。支持中文触发：PRD驱动开发、PDD开发、功能开发、启动PDD。
"交通事故责任评估与判定专业技能。当用户需要交通事故责任分析、事故现场照片评估、交通法规咨询、事故责任划分、法律依据查询时触发此技能。适用于车辆碰撞事故、行人事故、非机动车事故等各类道路交通事故的责任认定场景。无论用户使用'交通事故'、'车祸'、'责任判定'、'交通法规'、'事故定责'等何种表述，只要涉及交通事故评估或责任认定，均应调用此技能。支持中文触发：交通事故、车祸、责任判定、交通法规、事故定责、责任划分、事故评估、追尾、碰撞、违章、赔偿。"
自动化重构专家技能，将收集到的质量改进任务转化为具体的代码操作。当用户需要代码重构、消除重复、简化复杂度时自动触发。即使用户只说'重构代码'、'消除重复'或'简化代码'，也应触发此Skill。支持中文触发：重构代码、消除重复、简化复杂度、自动重构、代码重构、PDD重构。