Skip to content

feat: trigger Claude on CodeRabbit and Copilot review comments - #198

Merged
don-petry merged 1 commit into
mainfrom
claude/competent-mcnulty-d226bb
May 6, 2026
Merged

don-petry merged 1 commit into
mainfrom
claude/competent-mcnulty-d226bb

Conversation

@don-petry

Copy link
Copy Markdown
Contributor

Summary

  • Extends the pull_request_review_comment trigger condition in claude-code-reusable.yml to include coderabbitai[bot] and Copilot alongside the existing OWNER/MEMBER/COLLABORATOR check
  • Previously, bot review comments were silently skipped — the workflow event fired but the job if: evaluated false for author_association: NONE (CodeRabbit) and author_association: CONTRIBUTOR (Copilot)
  • Bot logins confirmed from PR #167 API response

Why

CodeRabbit and Copilot post actionable inline findings on PRs. Without this change, Claude never sees them and they accumulate unaddressed (as seen on PR #167).

Loop risk is low: both bots re-review only on pushes, not on comment replies, so Claude responding to a bot comment won't re-trigger those bots.

Test plan

  • CI passes (yamllint, dependency audit, SonarCloud)
  • After merge, verify Claude responds when CodeRabbit or Copilot post an inline review comment on a new PR

🤖 Generated with Claude Code

The pull_request_review_comment condition previously required OWNER/MEMBER/COLLABORATOR
author_association, which excluded both bots. Adds coderabbitai[bot] and Copilot as
allowed senders so Claude automatically addresses their inline findings.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings May 6, 2026 02:30
@don-petry
don-petry requested a review from a team as a code owner May 6, 2026 02:30
@coderabbitai

coderabbitai Bot commented May 6, 2026

Copy link
Copy Markdown

Warning

Rate limit exceeded

@don-petry has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 7 minutes and 11 seconds before requesting another review.

To continue reviewing without waiting, purchase usage credits in the billing tab.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: fad08e05-6567-4010-b8fa-a070661e0c4f

📥 Commits

Reviewing files that changed from the base of the PR and between 3f3ba15 and 9b6a0bd.

📒 Files selected for processing (1)
  • .github/workflows/claude-code-reusable.yml
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch claude/competent-mcnulty-d226bb

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@sonarqubecloud

sonarqubecloud Bot commented May 6, 2026

Copy link
Copy Markdown

@don-petry
don-petry enabled auto-merge (squash) May 6, 2026 02:32
@don-petry
don-petry merged commit 12802c8 into main May 6, 2026
24 checks passed
@don-petry
don-petry deleted the claude/competent-mcnulty-d226bb branch May 6, 2026 02:33

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the reusable Claude Code workflow gating so that inline PR review comments authored by CodeRabbit and GitHub Copilot can trigger the Claude job, in addition to comments from trusted human collaborators.

Changes:

  • Extends the pull_request_review_comment job if: condition to allow coderabbitai[bot] and Copilot logins to trigger runs.
  • Keeps the existing guard that prevents responding to claude[bot] comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@@ -25,7 +25,8 @@ jobs:
contains(fromJson('["OWNER","MEMBER","COLLABORATOR"]'), github.event.comment.author_association)) ||
(github.event_name == 'pull_request_review_comment' &&
don-petry added a commit that referenced this pull request Jun 8, 2026
The pull_request_review_comment condition previously required OWNER/MEMBER/COLLABORATOR
author_association, which excluded both bots. Adds coderabbitai[bot] and Copilot as
allowed senders so Claude automatically addresses their inline findings.

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
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