Skip Claude code review for Renovate bot PRs

[samuelvkwong]

Summary

• Add if: github.event.pull_request.user.login != 'renovate[bot]' condition to skip the claude-review job for Renovate bot PRs
• Remove unused commented-out filter examples
• Prevents unnecessary review status checks on automated dependency update PRs

Uses github.event.pull_request.user.login for precise checking of the PR author.

Test plan

• Verify the workflow skips when a Renovate bot PR is opened
• Verify the workflow still runs for regular PRs

🤖 Generated with Claude Code

Summary by CodeRabbit

• Chores
  • Updated CI/CD workflow to prevent automated dependency update pull requests from triggering code review runs.

[coderabbitai]

📝 Walkthrough

Walkthrough

Modified the GitHub Actions workflow configuration to bypass execution for Renovate bot pull requests. Replaced previously commented multi-actor author filters with a conditional guard that checks whether the PR author is the Renovate bot.

Changes

Cohort / File(s)	Summary
Workflow Configuration .github/workflows/claude-code-review.yml	Replaced commented PR author filters with a direct conditional check to skip workflow runs when the pull request author is renovate[bot]. This adds a simple guard clause at the workflow level.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰✨ A bot comes knocking, but not this way,
Renovate's changes won't trigger today,
With one simple guard, we filter with grace,
Keeping our workflows in their rightful place! 🎯

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and concisely summarizes the main change: adding a condition to skip the Claude code review workflow for Renovate bot PRs.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch skip-claude-review-for-renovate

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[gemini-code-assist]

Note

Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported.

[coderabbitai]

🧹 Nitpick comments (1)

.github/workflows/claude-code-review.yml (1)

15-16: LGTM — condition is correctly expressed and precisely targets Renovate bot PRs.

github.event.pull_request.user.login is always populated on pull_request events, the string literal 'renovate[bot]' matches the canonical GitHub App login for Renovate, and a job-level if is the right placement (avoids consuming a runner at all).

If the repo also uses Dependabot for any ecosystem, you may want to extend the guard:

-    if: github.event.pull_request.user.login != 'renovate[bot]'
+    if: >-
+      github.event.pull_request.user.login != 'renovate[bot]' &&
+      github.event.pull_request.user.login != 'dependabot[bot]'

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In @.github/workflows/claude-code-review.yml around lines 15 - 16, Update the
job-level if condition that currently checks
github.event.pull_request.user.login != 'renovate[bot]' to also exclude
Dependabot PRs; modify the condition referenced (the job-level "if" that
inspects github.event.pull_request.user.login) so it checks both 'renovate[bot]'
and 'dependabot[bot]' (e.g., require the login to be not equal to both) to skip
runners for Dependabot as well.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In @.github/workflows/claude-code-review.yml:
- Around line 15-16: Update the job-level if condition that currently checks
github.event.pull_request.user.login != 'renovate[bot]' to also exclude
Dependabot PRs; modify the condition referenced (the job-level "if" that
inspects github.event.pull_request.user.login) so it checks both 'renovate[bot]'
and 'dependabot[bot]' (e.g., require the login to be not equal to both) to skip
runners for Dependabot as well.