task_plan.md 1.2 KB
reviewer - 审查计划
角色: 代码审查 状态: pending 分配的任务: 关键优化项的代码审查
审查范围
- 大功能/新模块完成后必须审查
- 小修改/Bug 修复/配置变更不需要审查
待审查项
- P0-1: OPT-02 WebSocket 推送（backend-dev）
- P0-2: OPT-01 按钮合并（frontend-dev）
- P1-5: OPT-08 一键完整生成（frontend-dev + backend-dev）
- P2-4: OPT-12 反馈页面（frontend-dev + backend-dev）
- P3-3: OPT-17 歌词时间轴（frontend-dev + backend-dev）
审查维度
| 维度 | 权重 | STRONG 表现 | WEAK 表现 |
|---|---|---|---|
| 产品深度 | 高 | 功能涵盖真实用户边界情况（空状态、错误恢复） | 仅开心路径，错误状态显示原始异常 |
| 代码质量 | 高 | 函数<50行，文件<800行，错误处理明确 | 大函数，深层嵌套，吞异常 |
| 前端性能 | 中 | 无不必要重渲染，有 memoization | 频繁重渲染，缺少优化 |
| 文档同步 | 高 | API/架构变更同步更新 docs/ | 文档漂移 |
备注
- 审查请求由 dev 直接发送
- 审查维度评分必须附理由
- [WEAK] 维度 → 判决不能是 [OK]