Skip to content

ci(scheduler): connect bounded review admission - #1863

Merged
seonghobae merged 2 commits into
mainfrom
codex/connect-bounded-admission-scheduler
Sep 4, 2026
Merged

ci(scheduler): connect bounded review admission#1863
seonghobae merged 2 commits into
mainfrom
codex/connect-bounded-admission-scheduler

Conversation

@seonghobae

Copy link
Copy Markdown
Contributor

Summary

  • connect the merged durable admission core to the real OpenCode and Strix scheduler dispatch paths
  • persist request/lease/completion state with a SHA-pinned Actions cache while retaining worker workflows and their permission, credential, concurrency, and exact-head publication boundaries
  • enforce one shared bounded worker-dispatch budget per scheduler execution, including all repository subprocesses in an organization sweep
  • requeue failed or superseded leases without adding a fan-out job or changing the live ruleset

Queue reduction evidence

  • RED: the new workflow/store contract and persisted-state test both failed before wiring
  • GREEN: two eligible PRs under a one-slot admission budget produce exactly one OpenCode repository dispatch; the second remains queued
  • restored leases count against the next run until exact-head terminal evidence completes them

Verification

  • uv run pytest -q tests/test_review_admission_controller.py tests/test_required_workflow_queue_contract.py tests/test_pr_review_merge_scheduler.py — 387 passed
  • uv run python scripts/ci/pr_review_merge_scheduler.py --self-test
  • actionlint .github/workflows/pr-review-merge-scheduler.yml
  • git diff --check

Boundaries

No worker workflow, permission, credential, concurrency namespace, publication CAS, or live ruleset changed. No new fan-out job was added.

Signed-off-by: Seongho Bae <me@seonghobae.me>
@coderabbitai

coderabbitai Bot commented Sep 4, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 33 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

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

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Team

Run ID: 9d793d26-b807-4f11-bf8d-7f0d97544f1d

📥 Commits

Reviewing files that changed from the base of the PR and between a23c666 and d1c5c31.

📒 Files selected for processing (6)
  • .github/workflows/pr-review-merge-scheduler.yml
  • scripts/ci/pr_review_merge_scheduler_core.py
  • scripts/ci/review_admission_controller.py
  • tests/test_pr_review_merge_scheduler.py
  • tests/test_required_workflow_queue_contract.py
  • tests/test_review_admission_controller.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.

@seonghobae
seonghobae enabled auto-merge (squash) September 4, 2026 17:39
Signed-off-by: Seongho Bae <me@seonghobae.me>
@seonghobae
seonghobae merged commit 1114a99 into main Sep 4, 2026
4 of 18 checks passed
@seonghobae
seonghobae deleted the codex/connect-bounded-admission-scheduler branch September 4, 2026 17:49
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