| 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审查、交叉验证三轮审查,弥补国产模型在复杂逻辑推理上的能力短板,确保代码质量和业务逻辑正确性。
触发条件
- 功能点代码实现完成后
- 用户请求"审查代码"/"review"/"检查代码"/"代码有问题吗"
- pdd-implement-feature 完成代码生成后自动触发
核心能力
1. 三轮审查机制
| 轮次 | 类型 | 覆盖 |
|---|
| 第一轮 规则检查 | 自动化静态分析 | 代码风格、安全漏洞、常见错误模式 |
| 第二轮 AI审查 | 智能分析 | 业务逻辑、代码质量、最佳实践 |
| 第三轮 交叉验证 | 规格对照 | 接口完整性、数据模型一致性、业务规则正确性 |
2. 审查清单标准化
接口实现: 接口是否实现/参数一致/返回值一致/异常处理完整
数据模型: 实体字段与表一致/类型正确/必填验证/关联正确
业务逻辑: 规则正确/状态转换/边界条件/并发场景
安全性: SQL注入/XSS/权限控制/敏感数据加密
性能: N+1查询/瓶颈/缓存/分页
代码质量: 清晰可读/命名规范/注释充分/无重复
3. 问题分级标准
| 级别 | 定义 | 处理 |
|---|
| Critical 严重 | 阻塞功能正常运行 | 必须立即修复,阻塞流程 |
| Warning 警告 | 影响质量或性能 | 建议修复,不阻塞 |
| Suggestion 建议 | 改进建议 | 可选修复,记录备查 |
问题记录格式: id/round/level/category/location(file:line)/description/expected/actual/suggestion/status/history
4. 反馈循环机制
发现→记录→分配→验证修复→关闭。问题记录模板见 references/report-template.md
审查流程详解
第一轮:规则检查
检查: 代码风格(命名/格式/导入/注释) | 安全(SQL注入/XSS/敏感信息/权限注解) | 常见错误(空指针/资源泄露/异常处理/并发)
工具: Checkstyle/ESLint/PMD/SpotBugs
输出: rule-check-report.md
第二轮:AI审查
维度: 业务逻辑正确性(需求理解/规则完整/边界/异常) | 代码质量(结构/设计模式/重复/过度设计) | 最佳实践(框架规范/推荐方式/反模式)
方法: 逐文件审查/业务逻辑追踪/代码路径分析/依赖关系检查
输出: ai-review-report.md
第三轮:交叉验证
验证: 接口完整性(对照spec.md) | 数据模型一致性(对照数据库设计) | 业务规则正确性(对照规则定义)
方法: 规格对照表/字段映射/接口映射/业务规则追踪
输出: cross-validation-report.md
审查报告模板
完整报告模板见 references/report-template.md(概要/问题统计/Critical详情/Warning与Suggestion列表/验证结果/结论/修复建议)
反馈循环流程
- 问题分类与优先级排序(Critical优先) → 2. 问题分配(自动/AI/人工修复) → 3. 执行修复(记录过程/更新代码/标记状态) → 4. 验证修复(重跑检查/确认解决/查新问题) → 5. 关闭问题(更新状态/记录历史/生成报告)
问题修复记录模板
见 references/report-template.md(含修复前/后代码、验证结果、备注)
学习机制
- 问题模式学习: 记录常见问题→建模式库→优化规则
- 修复方案学习: 记录有效方案→建知识库→提高自动修复率
- 规则优化: 根据统计优化规则/添加新规则/调整级别
- 输出: 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 | 调用 | 问题记录 | 学习结果 |