Skip to content

fix(scheduler): run bounded review repairs every hour - #1006

Closed
seonghobae wants to merge 3 commits into
mainfrom
fix/hourly-pr-review-fixes
Closed

fix(scheduler): run bounded review repairs every hour#1006
seonghobae wants to merge 3 commits into
mainfrom
fix/hourly-pr-review-fixes

Conversation

@seonghobae

Copy link
Copy Markdown
Contributor

Outcome

Run the central review-feedback repair scheduler once per hour instead of once every two hours, while preserving its bounded and fail-closed operating contract.

  • change the scheduled cadence from 23 */2 * * * to 23 * * * *;
  • change the same-head redispatch default from 24 hours to 1 hour;
  • retain max_dispatches = 1, head-scoped repair markers, target-repository concurrency, and canonical central workflow ownership;
  • add a workflow-source regression contract for the hourly cron and one-hour retry default;
  • record the operating change in CHANGELOG.md.

The merge scheduler already performs organization sweeps every 15 minutes; this PR only raises the slower code-repair loop to the requested hourly cadence. It does not duplicate scheduler logic into application repositories or alter review-agent credentials.

Merge only after exact-head quality/security checks and an independent review. Auto-merge may be enabled after the repository validates this head.

@seonghobae
seonghobae enabled auto-merge (squash) August 14, 2026 13:24
@coderabbitai

coderabbitai Bot commented Aug 14, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@seonghobae, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 3 minutes

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 3781b785-d71c-4cd8-9736-a10dc7859666

📥 Commits

Reviewing files that changed from the base of the PR and between 6eb06cd and 16a9916.

📒 Files selected for processing (3)
  • .github/workflows/pr-review-fix-scheduler.yml
  • CHANGELOG.md
  • tests/test_pr_review_fix_scheduler_workflow_contract.py

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.

Copy link
Copy Markdown
Contributor Author

@opencode-agent @cwl-noema-review Please independently review exact head 16a9916af8e5e11182c837a1c1b950300fead99b. Verify the hourly repair cadence, one-hour same-head retry, max_dispatches = 1, immutable central ownership, target-repository concurrency, least privilege, and regression contracts before approval.

@seonghobae
seonghobae marked this pull request as draft August 15, 2026 11:02
auto-merge was automatically disabled August 15, 2026 11:02

Pull request was converted to draft

@seonghobae
seonghobae marked this pull request as ready for review August 15, 2026 11:02
@seonghobae
seonghobae enabled auto-merge (squash) August 15, 2026 11:03

Copy link
Copy Markdown
Contributor Author

@coderabbitai review

Please review the unchanged exact head 16a9916af8e5e11182c837a1c1b950300fead99b, focusing on the hourly cadence, one-hour same-head retry, bounded single-dispatch behavior, concurrency, least privilege, and regression contract.

@coderabbitai

coderabbitai Bot commented Aug 15, 2026

Copy link
Copy Markdown

@seonghobae I will review exact head 16a9916af8e5e11182c837a1c1b950300fead99b.

I will verify the hourly cron, one-hour same-head retry, max_dispatches = 1, target-repository concurrency, least-privilege permissions, immutable central ownership, and regression contracts.

⚠️ Action not completed

Review rate limited.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

Copy link
Copy Markdown
Contributor Author

Closing this narrower cadence-only PR as superseded by #782. PR #782 already carries the hourly review-repair cadence together with the required NVIDIA NIM credential separation, exact-head RCA/repair contract, bounded writer scope, fail-closed behavior, and repository-specific callers. Keeping both open would race on .github/workflows/pr-review-fix-scheduler.yml, duplicate checks, and obscure which implementation should become the protected central authority.

@seonghobae seonghobae closed this Aug 15, 2026
auto-merge was automatically disabled August 15, 2026 11:32

Pull request was closed

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