By name: 621-feature-dev-review description: 功能开发评审技能。当620-feature-dev开发完成后触发：(1)评审技术方案和代码规范, (2)检查安全性与影响范围, (3)输出评审结论。 alwaysApply: false author: "axeon(23231269@qq.com)" version: "1.0.0"
功能开发评审
项目环境检测
从当前目录向上查找 project-info.md，最多 3 层，找到后记为 PROJECT_ROOT。详见 检测方法与前置检查。未找到 → 提示用户先执行 0-init。
角色职责
| 角色 | 职责 | 智能体 |
|---|---|---|
| 主导 | 方案评审 + 代码审计 | system-architect + java-lead + js-lead |
| 协作 | 需求符合度 | product-manager |
| 协作 | 安全审计 | system-architect |
源技能引用
评审必须先读取源技能文件获取原始约定，再基于约定进行评审，禁止仅凭模型自身知识评审。
| 源技能文件 | 评审时读取的内容 |
|---|---|
| 620-feature-dev/SKILL.md | 必读全文：修改范围决策表、Phase 1-5 全部要求 |
| 210-java-uniweb-dev/references/uniweb/dev-standards.md | 必读"数据访问常见陷阱"：16 条 API 陷阱 + 外部集成陷阱 |
| 210-java-uniweb-dev/SKILL.md | 架构约定速查表（Controller/Helper/代码风格约定） |
输入
| 输入项 | 来源 | 说明 |
|---|---|---|
| 功能需求文档 | PROJECT_ROOT/issue/features/FEATURE-{YYMMDD}-{topic}.md |
610阶段输出 |
| 技术方案 | 各端 issue/FEATURE-DESIGN-*-tech-design.md |
620 Phase 1 输出 |
| DDL文件 | PROJECT_ROOT/database/migrations/FEATURE-*.sql |
620 Phase 1 输出 |
| 各端代码 | 各端 src/ |
620 Phase 4 输出 |
| 各端测试 | 各端 src/test/ 或 src/**/*.spec.ts |
620 Phase 4 输出 |
输出
| 输出项 | 位置 | 说明 |
|---|---|---|
| 评审报告 | PROJECT_ROOT/issue/reviews/REVIEW-DEV-{YYMMDDHHMM}.md |
评审结论和问题清单 |
报告格式详见 评审报告模板。
评审维度
详细检查清单见 checklist.md。
| 维度 | 检查要点 | 源技能章节 |
|---|---|---|
| 技术方案 | DB设计合理、接口规范、前端设计完整、测试策略覆盖、PRD覆盖100% | 620 Phase 1 |
| 方案一致性 | 代码实现与技术方案完全一致，接口/DTO/Helper无偏差 | 620 Phase 1 vs 4 |
| 代码规范 | 后端uw-base规范（禁Lombok）、前端Vue3/UniApp规范 | 620 Phase 4 |
| TDD覆盖 | 后端行≥80%/分支≥70%，前端核心业务≥70% | 620 Phase 4 |
| 安全性 | @Valid校验、SQL参数化、@MscPermDeclare、SaaS租户隔离（saas_id） | 620 安全性 |
| 影响范围 | 未修改无关模块，变更范围可控 | 620 修改范围决策表 |
通过标准
| 等级 | 评分 | 条件 |
|---|---|---|
| 通过 | ≥ 95 分 | 无 Critical，Major ≤ 2，方案一致性100%，TDD覆盖达标 |
| 不通过 | < 95 分 | 存在 Critical 或 Major > 2 或方案偏差或TDD覆盖不达标 |
评分 < 95 进入修复循环，无"有条件通过"中间态。
评审流程
开始评审前，先按"源技能引用"读取源技能，按"输入"读取所有评审对象。
1. 编译验证（前置步骤）
| 端 | 验证命令 |
|---|---|
| 后端 | mvn compile + mvn test -Dspring.profiles.active=debug |
| Admin Web | pnpm build + pnpm vue-tsc --noEmit |
| Guest Web | pnpm build + pnpm nuxi typecheck |
| UniApp | pnpm build:h5 + pnpm build:mp-weixin |
编译未通过不得进入评审。
2. 执行评审
按维度检查。详见 评审报告模板。详细的评审检查清单见 checklist.md。
3. 评审结论
评分 ≥ 95（通过）：
- 标记评审状态为「通过」
- 输出评审报告，任务结束
评分 < 95（不通过）→ 自动修复循环：
- 立即调用
620-feature-dev，传入问题清单 - 修复完成后立即重新执行本技能评审
- 若仍 < 95，回到步骤 1（最多 5 轮）
- 仅在通过或轮次耗尽时输出结果
此流程全自动执行：中间不暂停、不询问、不汇报。 未收到通过确认前，禁止结束本技能任务。
流转关系
输入: 620-feature-dev 全部输出（方案+代码+测试）
↓
621-feature-dev-review
↓
通过 → 等待 630-feature-test
不通过 → 自动调用 620 修复 → 重新评审