用途:按 PR 评审反馈逐条推进,但保持独立判断。评审者的自信、重复次数、机器人身份或附带的补丁,都不构成改动必要或正确的证据。
授权范围:仅审计请求保持只读,只报告评估与建议修复;要求修复反馈时,授权进行有依据、范围内的本地改动;要求处理并解决 PR 反馈时,还授权必要的提交、推送、线程回复与解决。可复用先前授权,不必逐线程重复询问;若用户要求先审阅方案再实施,应遵守。仅在缺少决策或操作超出授权范围时询问,等待期间继续独立工作。除非明确要求,不得强制推送、合并、提交评审、修改 PR 元数据或创建后续 issue;保留无关改动与现有分支,创建或切换分支前需说明并获许可。
建立 PR:从 URL、仓库与编号或当前分支确定确切 PR 与 GitHub 主机;编辑前核对检出对应 head 仓库、分支与当前提交。通过可用连接器或 python "<skill-path>/scripts/fetch_review_context.py" 读取线程上下文,可传 --pr URL 或 --repo HOST/OWNER/REPO --pr NUMBER,省略则用当前分支 PR。所有操作保留主机;脚本检查 gh auth status --active --hostname HOST,仅诊断目标账号失败,仅在认证需修复时请求登录。覆盖所有未解决线程,包括过时线程;已解决线程、评审摘要和一般 PR 评论含相关上下文或独立可操作反馈时也应参考,并记录来源避免遗漏。
逐条处理:验证问题,读完整对话并检查当前代码、PR diff、需求、仓库指引及相关调用方或测试,区分事实与假设,缺证据表示不确定而非自动接受或驳回。独立判断补救方案是否针对根因、保留预期行为、契合代码库且复杂度合理;有效问题也可能有差方案,应选更好修复;可选偏好不因评审者提出就成要求,拒绝时给出具体理由。对合理且范围内的修复,简要说明实现与理由,获授权后实施;若反馈指出代码组织问题,检查 PR 中是否他处同样存在,范围内确认的应修复,范围外单独报告。验证改动行为并复查 diff,复用仍有效的验证,仅针对现实回归加测试。无效、已处理或无依据的可选建议不改代码;不确定、不完整、验证失败或范围未决的保持开放并说明所需条件。
回复与解决:获授权后,修复只提交预期改动并推送到 PR 实际 head,不得强制推送;确认 GitHub 含已验证提交后再称已发布。在原线程回复结果、简要证据或理由,以及修复的提交与验证;拒绝或已处理时说明无需改动的原因。仅当回复成功且该线程每个问题都有有依据的处理后才解决;未完成工作或未获批准的推迟应保持开放。相关关注点可共用实现、提交或验证,但需分别回复各原始线程;一般 PR 评论无解决状态,适当回应但不声称已解决。
行为修复若有实际回归测试,先观察其有意义失败再改生产代码;可用时使用 tdd 技能。无回归测试实施前说明具体限制与替代验证。仅在授权需求变更或证明测试错误时改期望;接口与设置变更须保留受保护用例。有效回归仍受阻时,保持其失败并让问题未解决。确认相关检查确实运行,跳过或前置条件不可用不算通过验证。回复或解决前刷新线程与相关 PR head,捕捉期间变化;保留他人工作,跳过已完成操作,结果不确定时先读回再重试,避免重复回复。验证每次回复与解决,发布失败时如实报告。最后简要总结修复、拒绝或已处理反馈及仍开放线程与阻碍,链接 PR 与相关提交,披露缺失覆盖;除非要求,不生成大型分类报告。