Skip to content

review-gate: 严重度表头解析失灵(正文 P0=0 却渲染 P0×2/「解析失败」) #16

Description

@tizerluo

现象

review-gate 发的 PR 评论里,表头「严重度分布」与实际审查正文多次不一致:

  1. feat(api): stateless POST /mcp transport + RFC 9728 metadata (MCP 2026-07-28 P1+P2) openagentemail/openagentemail#17 首轮评论:表头「严重度分布解析失败,请人工核对」
  2. 同 PR 终 head(031e5a1)评论:表头「P0 × 2 · P1 × 2 · P2 × 3 · ⚠️ concerns」,但正文明确写「P0 × 0,✅ 可以合并」
  3. feat(site): MoR review pages — privacy-policy, terms-of-service, pricing, support email openagentemail/website#6docs: 同步 skill 到 App 3.3.0 (补 prompt/enhance) #7 多个 head 同样「解析失败」

影响

表头是读者第一眼看到的结论。误报为 P0 concerns 导致审查流程误判闸门卡死(真实案例:指挥 agent 看到表头停工等待,实际正文判定可以合并)。

疑似原因

表头解析逻辑可能把正文里的 P0/P1/P2 汇总表格行或文字描述误抓为计数(例如「P0 (阻断合并) | 0」这类表格行被解析成非零,或解析器在正文格式变化时整体 fallback 成「解析失败」)。

建议

  • 表头解析改为以正文的最终结论字段为准(或让审查输出带结构化 verdict 字段,表头只读该字段,不做正文正则猜测)
  • 解析失败时表头降级为「需人工核对正文」且 verdict 不显示 concerns(避免误导为红灯)

(由 MBP·Kimi 代舰队记录,复现样本见 openagentemail 主仓 PR #17/#19/#20 与 website PR #6/#7 的 gate 评论)

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions