pipexerp/ai-proj-helper
6
0
Fork 0
Code Issues Pull Requests 2 Actions Packages Projects Releases Wiki Activity
Files
feat/devflow-integration-p1-part1
ai-proj-helper/skills-dev/review-checklist-plugin/checklists/general.md
John Qiu 3706d7f32d feat(skill): REQ-20260406-0004 技能三层分离重构（7主线+16插件） 
批次1: req-prd 瘦身 + req-design 重定位 + dev-coding 聚焦
批次2: dev-review 新建 + review-checklist 插件
批次3: dev-integration 新建 + req-compare 拆出
批次4: 插件完善 (req-research/db-migration/dev-scaffold/deploy-rollback)
批次5: 平台拆分 (dev-ios/dev-android/dev-mcp/dev-pda) + dev 分组更新
批次6: marketplace.json 32→44 plugins

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-04-06 17:44:08 +09:30

828 B
Raw Permalink Blame History

通用代码评审检查清单

适用于所有项目，补充五视角扫描法。

API 设计

  • RESTful 命名是否规范？（复数名词、无动词）
  • 分页参数是否有默认值和上限？
  • 响应格式是否统一？（code/message/data）

错误处理

  • 错误是否被正确传播？（不要吞掉错误）
  • 用户可见的错误消息是否友好？（不暴露技术细节）
  • 是否有 panic recover 兜底？

性能

  • 列表查询是否有分页？（不允许无限制查询）
  • N+1 查询问题？（循环内查数据库）
  • 是否有不必要的全表扫描？（缺少索引）

可维护性

  • 魔法数字是否提取为常量？
  • 复杂业务逻辑是否有注释说明？
  • 函数是否过长？（超过 100 行考虑拆分）
Reference in New Issue View Git Blame Copy Permalink