用途:在审查涉及 Reader 社交/atmosphere/mastodon 区域的 PR 时使用,尤其是协议适配器模式的切片工作、乐观变更(optimistic mutations)以及共享的 post-card 组件。评审要点来自 CM-658/CM-660 周期,相同问题在两个 PR 中重复出现,可当作检查清单。
评审格式:每条发现写成表格,包含严重级别(blocker 阻断合并、important 真实缺陷、nit 清理或加固)、文件与行号(绝对路径或仓库相对路径,便于在 GitHub 内联评论)、1-2 句问题描述和建议修复。评论用 gh api repos/Automattic/wp-calypso/pulls/{n}/comments 发布内联回复,而非顶层 PR 评论。需明确区分本次 PR 内修复与后续 issue。给出结论行(Approve / Request changes / Needs more discussion),评审控制在约 700 词。
分派 code-reviewer:当 PR 超过 20 个文件或 500 行、形状与已审过的同类 PR 结构相似,或作者想获得独立视角时,分派 superpowers:code-reviewer。需提供 PR 编号、分支路径、PR 内容,以及 common-smells.md 中要重点检查的常见问题列表;代理回复即评审正文。
回复约定:不要写感谢评审,直接说明修复或反驳。内联评论的回复用 gh api repos/{owner}/{repo}/pulls/{pr}/comments/{id}/replies,保持线程上下文;顶层评论用于评审总结确认,列出哪些发现已落在提交 X、哪些推迟为后续。不要编造 wp.me 短链。
常见问题与文件检查:先走 references/common-smells.md 的 9 项以上清单。重点检查 post-card 的 like/repost context(接口无死字段、null-action 默认值与实际形状一致)、button(!action.supported 时渲染静态计数回退而非 null,回退位于按钮内)、post-card-counts(始终渲染 LikeButton/RepostButton,不用 connectionId 三元判断)、use-{protocol}-{like,repost}-action(errorMessageFor* 有 default 分支、trackError 含 logToLogstash、mutate 前有 cid/rkey 守卫)、api-queries(乐观补丁按连接作用域、onMutate 中 cancelQueries 包 try/catch、mutation 工厂接收 connectionId: number 而非 QueryClient)、fetchers(路径插值 wire ID 用 encodeURIComponent 包裹)。
跨 PR 同步修复:两个切片 PR 同时打开时,一个 PR 的反馈通常适用于另一个。可用 git -C /Users/pfefferle/Code/wp-calypso show <fix-commit-on-A> -- <relevant-paths> 查看修复,用 git -C /Users/pfefferle/Code/wp-calypso show <branch-of-B>:<relevant-path> | grep <smell> 检查另一分支,若存在则一并修复。
限制:资料未提供该技能的可安装来源、运行环境要求或收费标准。