Skip to content

🐛 fix(review): 跨轮复用同一条 PR 总结评论 - #48

Draft
lishuceo wants to merge 1 commit into
mainfrom
fix/sticky-review-summary
Draft

lishuceo wants to merge 1 commit into
mainfrom
fix/sticky-review-summary

Conversation

@lishuceo

Copy link
Copy Markdown
Contributor

Maker 每次 push 都会按共享 workflow 的指令执行 gh pr comment,导致同一 PR 上累积相互过期的 review 总结。Maker #1123 已出现 4 条独立总结。

本改动沿用 urhox 的常驻评论机制:启用 track_progress 和 use_sticky_comment,总结通过 mcp__github_comment__update_claude_comment 覆盖原评论,展示当前 head SHA、结论和全部未解决问题;移除新增总结的工具许可,并拒绝常见替代发帖路径。

线程对账改为读取解决状态和回复:不重复处理已关闭线程,不重复确认已有修复说明,保留分页信息;已解决线程正文截断为 300 字,疑似回归时再读取全文。

验证:

  • actionlint 1.7.7 通过。
  • 修改文件的 Prettier 检查及 git diff --check 通过。
  • 将工作流中的 GraphQL 查询直接用于 Maker #1123(3 个已解决线程)和 urhox #2683(1 个已解决、1 个未解决线程),验证作者、回复、分页、正文保留与截断结果。
  • 核对当前锁定的 Claude Action 源码,确认支持这两个配置项及评论更新工具。尚未在 GitHub Actions 中完成连续两轮 review 验证。

Maker 已引用此仓库的 main,合并后后续运行自动使用新逻辑;其他引用该共享工作流的仓库也会生效。旧重复总结不删除。沿用上游 sticky 查找机制的限制:只查顶层评论第一页,复用第一条匹配的 Claude 评论,可能选中较早的问答评论。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants