项目改动审查清单
将以下提示词粘贴到你的 AI 对话框中:
请根据 https://skillhub.cn/install/skillhub.md,安装 @user_8767e570/project-review-checklist。
技能介绍
解决的问题
前端仓库里的 diff 审查容易陷入泛泛而谈:只看到代码风格,却忽略 API 空字段、异步竞态、路由权限回归、响应式丢失等会导致线上问题。项目改动审查清单 把审查约束为一次面向合并动作的工程检查,重点覆盖 正确性、回归风险、类型安全、响应式安全、可维护性和发布准备度。
技能如何工作
它要求按固定顺序输出结论,而不是先写摘要再找问题:
- Critical:合并前必须修复。
- Major:高优先级修复。
- Minor:一致性与改进项。
- Test Gaps:缺少验证用例。
- Pass Summary:列完问题后再给通过说明。
执行时先收集 diff 上下文,优先关注 src/views、src/components、src/composables、src/router 中的变更。随后做三类扫描:正确性与回归检查未防护的 API 响应结构、异步 stale state、loading/error/empty 状态、未处理 promise rejection、try/catch 静默失败和路由权限回归;Vue 3 + TypeScript 规则检查 ref / reactive 使用、any、注入类型、响应式解构丢失和过度 watch;UI 与测试风险检查语义化 HTML、scoped 样式、BEM 命名、DOM 嵌套、共享组件状态一致性,以及边界用例和手动验证路径。
适用边界
这个清单更适合已经采用 Vue 3、TypeScript、组合式 API 的项目,尤其是包含路由、权限、API 状态和复杂 UI 交互的仓库。它不是泛用 lint,也不替代自动测试;当项目没有这些约定时,需要先把 Vue 规则、样式约定和测试策略映射成团队自己的检查项,否则会产出大量与项目无关的噪声。
使用场景
- 合并 Vue 前端 PR 前,检查 API 空字段、异步竞态与路由权限回归
- 审查 composable 变更,找出 any、响应式解构丢失和 watcher 滥用
- 发布前排查 loading、error、empty 状态缺失和关键用户路径阻塞
- 评审 UI 变更,确认 scoped 样式、BEM、语义化 HTML 与共享组件一致
适合人员
- 负责 Vue 3 + TypeScript 仓库合并把关的前端工程师,想在 PR 合入前堵住运行时与回归风险
- 需要把审查从风格转向线上风险的前端 Tech Lead,想快速定位 API、路由权限和响应式问题
- 维护复杂组件与路由权限的前端负责人,想统一 scoped 样式、语义化 HTML 和状态一致性
- 要在发版前补齐边界测试清单的资深前端,想生成缺失的 null、empty、权限拒绝验证项