From 8d548a5b2ec224aef4b40edf3ccb357b11283585 Mon Sep 17 00:00:00 2001 From: William Aaron Cheung Date: Fri, 31 Jul 2026 13:33:49 +0800 Subject: [PATCH] docs(pr-review): gate comment reviews on non-reviewer authors The comment-trigger example ignored all bots. Only the reviewer's own status comment needs excluding (it would retrigger the review); other bots' comments may answer a question or push back on a finding and should be reconciled like any reply. Skip the reviewer app's login instead of all bots. Co-Authored-By: Claude Opus 4.8 (1M context) Claude-Session: https://claude.ai/code/session_01GCc8MLBBAp5jAxc2x19swa --- .github/actions/README.md | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/.github/actions/README.md b/.github/actions/README.md index e587579..22c7838 100644 --- a/.github/actions/README.md +++ b/.github/actions/README.md @@ -166,11 +166,14 @@ on: jobs: pr-review: - # Only human comments on a PR; ignore issue comments and bot chatter. + # Any PR comment except the reviewer's own status comment, which is posted + # by the CI app and would otherwise retrigger the review. Other bots are + # allowed — their comments may answer a question or push back on a finding. + # Replace mega-maxwell[bot] with your reviewer app's login. if: >- github.event_name != 'issue_comment' || (github.event.issue.pull_request != null && - github.event.comment.user.type != 'Bot') + github.event.comment.user.login != 'mega-maxwell[bot]') concurrency: # issue_comment payloads carry issue.number, not pull_request.number. group: claude-pr-review-${{ github.event.pull_request.number || github.event.issue.number }}