fix: match dependabot author login format in automerge#110
Conversation
gh pr list returns author.login as "app/dependabot" not "dependabot[bot]". Use regex test instead of exact match. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
Note Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported. |
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the 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 configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Security Issue Found
The fix correctly addresses the root cause (API returning app/dependabot instead of dependabot[bot]), but the regex pattern is overly permissive and creates a security vulnerability. The unanchored regex test("dependabot") will match any username containing "dependabot" anywhere in it, potentially allowing unauthorized PRs to auto-merge.
Critical change required: Anchor the regex patterns to match only the legitimate bot formats to prevent unauthorized auto-merging.
You can now have the agent implement changes and create commits directly on your pull request's source branch. Simply comment with /q followed by your request in natural language to ask the agent to make changes.
Summary
gh pr list --json authorreturnsapp/dependabot, notdependabot[bot]test("dependabot")to catch both formatsrelease-pleaseRoot cause
The automerge workflow found 0 eligible PRs despite 8 open dependabot PRs because the jq filter was checking for
dependabot[bot]but the API returnsapp/dependabot.🤖 Generated with Claude Code