Skip to content

chore: tune .coderabbit.yaml to reduce review rate-limit hits - #309

Merged
getappz merged 1 commit into
masterfrom
task/315
Jul 22, 2026
Merged

chore: tune .coderabbit.yaml to reduce review rate-limit hits#309
getappz merged 1 commit into
masterfrom
task/315

Conversation

@getappz

@getappz getappz commented Jul 22, 2026

Copy link
Copy Markdown
Owner

Summary

Repeated "CodeRabbit: Review rate limited" seen this session on PR #306 and #308. Current .coderabbit.yaml only set drafts: true — the opposite of CodeRabbit's own default (false), so draft PRs were burning review budget too.

Per CodeRabbit's own auto-review docs (docs.coderabbit.ai/configuration/auto-review), the highest-leverage lever for our specific pattern (small PRs, several fast-follow commits pushed within one session — fmt/clippy fixes, review-response commits) is auto_pause_after_reviewed_commits, which defaults to 5 and counts every incremental auto-review toward the same per-developer rate limit as a fresh PR review.

Changes

  • drafts: false — reverted to CodeRabbit's default; draft PRs no longer auto-reviewed.
  • auto_pause_after_reviewed_commits: 2 — pauses incremental auto-review after 2 reviewed commits on an active branch instead of 5. A fresh pass is still one @coderabbitai review comment away when actually wanted.
  • ignore_title_keywords: ["WIP", "DO NOT MERGE", "[skip review]"] — no-cost opt-out for PRs explicitly marked not-ready.
  • ignore_usernames for dependabot[bot]/renovate[bot]/github-actions[bot] — no-cost, doesn't affect any real PR in this repo today.

Deliberately did not go all the way to enabled: false + label-based opt-in — that fits teams where most PRs don't want review by default; this repo's PRs are almost all real, mergeable work that should keep getting reviewed automatically. The chosen levers target the actual observed pattern (rate limit hit on active branches with repeated small pushes) without adding a manual labeling step to every PR.

Test plan

  • YAML parses correctly (yaml.safe_load)
  • N/A — config-only change, no code/tests affected

Summary by CodeRabbit

  • Chores
    • Updated automated review settings to skip draft pull requests.
    • Added automatic pausing after multiple reviewed commits.
    • Excluded pull requests with specified skip keywords and automated update accounts from review.

drafts was explicitly set to true (the opposite of CodeRabbit's default
false), so draft PRs burned review budget before they were even ready.
Reverted to false.

auto_pause_after_reviewed_commits defaults to 5. This repo's PRs commonly
get several small fast-follow commits pushed within one session (fmt/
clippy fixes, review-response commits) which each count toward the same
per-developer review-rate limit as a fresh PR. Lowered to 2 so incremental
auto-review pauses earlier on active branches; a fresh pass is still one
`@coderabbitai review` comment away when actually wanted.

Also added ignore_title_keywords (WIP/DO NOT MERGE/[skip review]) and
ignore_usernames for common bot accounts, both no-cost exclusions that
don't affect real PRs.

Agentflare-Agent: claude-code_2-1-217_agent
Agentflare-Branch: task/315
Agentflare-Item: 315
@coderabbitai

coderabbitai Bot commented Jul 22, 2026

Copy link
Copy Markdown

Review Change Stack

Warning

Ignoring CodeRabbit configuration file changes. For security, only the configuration from the base branch is applied for open source repositories.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: d39f44e5-9a16-4dc1-929d-c9d727eff3f5

📥 Commits

Reviewing files that changed from the base of the PR and between 41acba6 and 0fdfa54.

📒 Files selected for processing (1)
  • .coderabbit.yaml

📝 Walkthrough

Walkthrough

The auto-review configuration now skips draft pull requests, pauses after two reviewed commits, and excludes pull requests matching specified title keywords or automation usernames.

Changes

Auto-review configuration

Layer / File(s) Summary
Auto-review eligibility and pause rules
.coderabbit.yaml
Disables draft pull request reviews, adds a two-commit pause threshold, and configures title-keyword and automation-account ignore filters.

Estimated code review effort: 1 (Trivial) | ~5 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main change: tuning .coderabbit.yaml to reduce CodeRabbit review rate-limit hits.
Description check ✅ Passed The description covers the summary and test plan, and is mostly aligned with the template despite a missing Notes for reviewers section.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch task/315

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 Checkov (3.3.8)
.coderabbit.yaml

Traceback (most recent call last):
File "/usr/local/bin/checkov", line 2, in
from checkov.main import Checkov
ModuleNotFoundError: No module named 'checkov'


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.

@getappz
getappz merged commit 921b472 into master Jul 22, 2026
18 checks passed
@getappz
getappz deleted the task/315 branch July 22, 2026 15:53
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.

1 participant