review-pr-holistic

review-pr-holistic

将一堆发现整理成谨慎的审查者真正会发布的评论。此技能负责编辑和整合；不会生成新发现，也不会自行发布评论。

接收。 Fleet 发现——来自各审查维度、按严重程度分级的缺陷发现，保留维度和严重程度；调用方选定的验收和发布策略；根据后果检查后仅按该策略筛选出的发现；以及判断评估及其发现——这是由 ../review-pr-judgment/SKILL.md 定义的独立非严重程度类别，已综合为每个根因一个问题；PR 历史（PR 上的所有评论和讨论串，包括我们之前任何一轮留下的评论、作者在该分支上近期的提交消息、PR 描述）；每个关联 PR 的上下文包（差异、描述、顶层对话——不包括关联 PR 中的行内审查评论）；若有则包括清单；本次调用的审查范围——Fleet 缺陷发现所限定的代码范围；调用方的语气配置——规定已发布评论的表达方式；以及调用方做过的任何截断，以便延续处理。

保留调用方选定策略所纳入的低严重程度发现；整合不会自行设定严重程度门槛。

按以下顺序执行：

• 剔除已由现有讨论串涵盖的重复问题、被证据驳斥的发现，以及异议完全针对明确的有意要求的缺陷发现。明确陈述的意图界定范围；证据则证明实际行为。除非 PR 已按照 ../review-pr-judgment/SKILL.md → What answers a concern 回答了该具体后果，否则保留判断问题。即使没有评论需要发布，也要在返回的评估中保留任何已解决的分歧。作者单方面的说法不能驳倒已证实的失败。
• **跨审查者去重：**当多个问题指向同一根本问题时，将近似重复的发现合并为一条评论。
• 前提涵盖。 当某项判断发现质疑某个界面的必要性或存在与否时，将 Fleet 对同一界面的发现置于该前提问题之下——删去价值较低的发现，并将任何保留的发现归并到其下（“如果保留 X：还需处理 a、b”），而不是将它们列为彼此独立的问题。即使某项发现的等级为低，若仍有未履行的实质性补救义务，也要保留，并根据证据调整其等级；非阻塞性的前提问题不能免除该义务。保留符合调用方策略的每项独立缺陷：判断问题不能仅凭质疑某个界面就免除该发现。
• **限定范围：**将提出的缺陷发现限定为由审查范围引入或暴露的问题，除非该范围是完整的 PR 差异——这既适用于首次审查，也适用于对未变更 head 的重复审查。范围限定取决于审查范围，而非我们是否审查过：若将重复审查限定为上次审查以来的差异，就会导致读取整个 PR 后却不提出任何问题。在审查范围内的修复会暴露其依赖边界和受影响位置，即使这些位置的代码行未变；无关的后续差异不会重新开启旧修复的问题。判断发现不受此限：它们按定义针对整个 PR。
• 锚定每项保留的发现，且只能锚定到以下一种位置：行内文件:行号（默认）、文件级（针对整个文件的问题）或 PR 级（横跨多个部分且没有特定锚点的问题）。
• 改写所有评论正文及任何起草的回复，使其符合所提供的语气配置。
• 默认省略摘要标题——仅当各条评论未能体现一个确实存在的整体结论时才添加（用一句简短、符合语气的表述，不写套话）。 返回内容。 要发布的评论（锚点加符合语气要求的正文）、摘要标题文本（如有）、截断说明、简要列出的已舍弃发现及主要原因，以及供操作员参考的保留判断评估。改写会保留其建议、证据和不确定性；明确的建议可以直接表述，无需使用该语气中禁用的弱化措辞。