PR堆积没人看?这10个Skill重做代码审查

PR 堆着没人看,多半不是没人负责,而是审查缺少一条能走完的路。

PR 堆着没人看,往往不是没人负责,而是审查没有一条能走完的路。比如你收到一个「把列表页首屏加载优化一下」的 PR,diff 不大,描述只有两行。直接点开看代码,你很难判断它到底改没改对。

所以先别急着读 diff。让 grilling 把需求追问到没有静默假设,再用 autoresearch 给「优化是否生效」定一个可测量的指标。目标站住了,后面的检查才有参照。

接着按改动波及面分头查:组件写法、数据库查询与权限、安全规则,各自对照对应规范,而不是凭印象。前端流程用 webapp-testing 和 playwright-cli 跑一遍,把截图和可回放路径当作评审证据。

最后回到部署后的真实指标,用 web-perf 和 vercel-optimize 判断这次改动是否拖慢了页面或抬高了成本,再决定放行。评审通过后,用 git-commit 把修改整理成可追溯的提交。

顺序不能反:先确认目标,再查实现,最后看线上结果。

grilling

在评审前连续追问 PR 作者,逼出需求描述里没想清楚的地方,确认改动目标是否成立

autoresearch

对 PR 中的代码改动跑自主迭代实验,用可量化的指标验证优化是否真的生效

webapp-testing

用 Playwright 跑一遍 PR 影响的前端流程,把截图和失败用例作为评审证据贴回 PR

playwright-cli

录制并复现 PR 中声称修好的交互路径,生成可回放的测试报告供评审核对

web-perf

用 Chrome DevTools 对比 PR 前后的 Core Web Vitals,判断这次改动是否拖慢了页面

vercel-optimize

结合 Vercel 指标分析 PR 部署后的性能与成本变化,给出是否放行的依据

git-commit

把评审通过后的修改整理成规范提交信息,让 PR 的变更历史可追溯