Skip to content

Let the review gate skip a CodeRabbit check that was skipped at its rate limit - #454

Closed
bompus wants to merge 1 commit into
mainfrom
fix/gate-skipped-coderabbit-ratelimit
Closed

bompus wants to merge 1 commit into
mainfrom
fix/gate-skipped-coderabbit-ratelimit

Conversation

@bompus

@bompus bompus commented Oct 27, 2026 •

Copy link
Copy Markdown
Owner

When CodeRabbit is at its included review limit it answers a review command with a comment ("Review rate limited.") and leaves a check run on the head commit that concluded SKIPPED, titled "Review skipped", with no reason of its own. The gate accepted the rate limit only as a commit status, so it refused the check run as open feedback and the pull request could not land while the limit lasted (found on bompus/swarmail#206).

The gate now skips CodeRabbit, naming the reason, when its check run on the head concluded SKIPPED with that title, its latest reply to a review command said "Review rate limited.", and it has posted no review since. The reply may precede the head commit, since a push made while limited gets a skipped check without a new reply. A newer reply that says anything else (including one a person edited), a later CodeRabbit review, no reply, a hidden or non-CodeRabbit reply, another title or another conclusion still refuse.

Verified: bun run check exits 0; the 185 hr-pr-followup tests pass, and the new test fails without the change. Run against bompus/swarmail#206 the gate now prints "none open (2 review bot(s) skipped)". A GPT-6.1 Sol xhigh review found ways an old rate-limit reply could outlive newer evidence (a later review, a person-edited failure, a failure posted after the check, a newer file-count skip summary, a hidden newer failure); each is fixed and covered by tests.

@bompus

bompus commented Oct 27, 2026

Copy link
Copy Markdown
Owner Author

@coderabbitai full review

@coderabbitai

coderabbitai Bot commented Oct 27, 2026 •

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@coderabbitai

coderabbitai Bot commented Oct 27, 2026 •

Copy link
Copy Markdown
⚠️ Action not completed

Review rate limited.


Your included review limit is currently reached under our Fair Usage Limits Policy. This review may still proceed through usage-based billing if eligible. Your next included review will be available in 55 minutes.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Bompus AI Review (GPT-6.1 Sol, xhigh reasoning): 2 finding(s).

Comment thread skills/hr-pr-followup/scripts/check-pr-review.ts Outdated
Comment thread skills/hr-pr-followup/scripts/check-pr-review.ts

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Bompus AI Review (GPT-6.1 Sol, xhigh reasoning): 1 finding(s).

Comment thread skills/hr-pr-followup/scripts/check-pr-review.ts Outdated

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Bompus AI Review (GPT-6.1 Sol, xhigh reasoning): 1 finding(s).

Comment thread skills/hr-pr-followup/scripts/check-pr-review.ts Outdated

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Bompus AI Review (GPT-6.1 Sol, xhigh reasoning): 0 finding(s).

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Bompus AI Review (GPT-6.1 Sol, xhigh reasoning): 1 finding(s).

Comment thread skills/hr-pr-followup/scripts/check-pr-review.ts Outdated

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Bompus AI Review (GPT-6.1 Sol, xhigh reasoning): 1 finding(s).

Comment thread skills/hr-pr-followup/scripts/check-pr-review.ts Outdated

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Bompus AI Review (GPT-6.1 Sol, xhigh reasoning): 0 finding(s).

…ate limit

At its included review limit, CodeRabbit leaves a check run on the head concluded SKIPPED, titled "Review skipped", with no reason of its own; the reason is in its reply to a review command. The gate now skips that check when the latest visible CodeRabbit reply says "Review rate limited." and no CodeRabbit review or newer reply (hidden, edited or otherwise) follows it.
@bompus
bompus force-pushed the fix/gate-skipped-coderabbit-ratelimit branch from f21fe04 to 6376702 Compare October 27, 2026 05:35

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Bompus AI Review (GPT-6.1 Sol, xhigh reasoning): 0 finding(s).

@bompus bompus closed this Oct 27, 2026
@bompus
bompus deleted the fix/gate-skipped-coderabbit-ratelimit branch October 27, 2026 05:52
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