Skip to content

ci: rebuild hourly central PR maintenance on current develop - #326

Open
seonghobae wants to merge 6 commits into
developfrom
ci/hourly-pr-maintenance-d6c6665
Open

ci: rebuild hourly central PR maintenance on current develop#326
seonghobae wants to merge 6 commits into
developfrom
ci/hourly-pr-maintenance-d6c6665

Conversation

@seonghobae

@seonghobae seonghobae commented Aug 15, 2026

Copy link
Copy Markdown
Collaborator

Purpose

Rebuild the repository-local hourly pull-request maintenance caller from exact protected develop@d6c6665163eabe1b5eca80556c6963bafd6b2625, independently of stale multi-purpose PR #121. The caller reuses the centrally governed scheduler by immutable commit rather than copying review, branch-update, and merge logic into mightyETL.

TDD evidence

The test-only RED head 104f06fc576f216bf3dca2df00d247ed5c18d697 reached the intended contract boundary on macOS, Ubuntu, and Windows. Maven compiled production and test code, ran 325 tests, and failed only HourlyPrMaintenanceCallerTest because .github/workflows/hourly-pr-maintenance.yml did not exist.

Current GREEN head: f76b19bda972f1c826cd2d70c7c3b1dcb6ce08d0.

Bounded implementation

  • schedules one run at minute 17 of every hour plus workflow_dispatch;
  • serializes runs without canceling an in-progress queue evaluation;
  • immutably reuses ContextualWisdomLab/.github/.github/workflows/pr-review-merge-scheduler.yml@6eb06cdd08c79a06f7b390069d4ffa49e2eb7dba;
  • targets develop, scans at most 100 pull requests, requests missing current-head reviews, updates eligible branches, and uses policy-governed direct-or-auto merge;
  • grants only actions, checks, contents, id-token, and pull-requests permissions required by the reusable workflow;
  • does not inherit secrets or pass COPILOT_GITHUB_TOKEN, NVIDIA_NIM_API_KEY, or independent reviewer credentials;
  • keeps the previous repository-local disposition workflow as a manual-only fail-closed fallback, eliminating duplicate scheduled merge authorities;
  • updates operator documentation and CHANGELOG.md;
  • adds regression coverage for the immutable pin, schedule, permissions, secret boundary, single scheduled authority, documentation, and release-history contract.

Scope boundary

This PR replaces only the hourly PR-maintenance slice of #121. It does not supersede #121's distinct product-development, CI, SBOM, coverage, or supply-chain work.

Merge gate

Ready for independent review. Merge only after exact-head CI, SAST, dependency, SBOM, security checks, zero unresolved findings, and repository approval policy are satisfied. The automerge-workflow label authorizes review of the workflow-file change; because repository auto-merge is disabled, the centrally governed direct-or-auto scheduler may perform the final expected-head direct merge only after every policy gate passes.

@coderabbitai

coderabbitai Bot commented Aug 15, 2026

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

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

Next review available in: 4 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: 600e3c96-e91a-4734-8c14-3711c2d2514e

📥 Commits

Reviewing files that changed from the base of the PR and between d6c6665 and f76b19b.

📒 Files selected for processing (5)
  • .github/workflows/hourly-pr-disposition.yml
  • .github/workflows/hourly-pr-maintenance.yml
  • CHANGELOG.md
  • docs/hourly-pr-disposition.md
  • etl-service/src/test/java/com/xtrmetl/etl/documentation/HourlyPrMaintenanceCallerTest.java

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 added automerge-workflow documentation Improvements or additions to documentation labels Aug 15, 2026 — with ChatGPT Codex Connector
@seonghobae
seonghobae marked this pull request as ready for review August 15, 2026 09:48

Copy link
Copy Markdown
Collaborator Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Aug 15, 2026

Copy link
Copy Markdown
Contributor
⚠️ 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.

cancel-in-progress: false

permissions:
actions: write
permissions:
actions: write
checks: read
contents: write
@opencode-agent opencode-agent Bot added area: ci-cd CI, GitHub Actions, checks, release, or supply chain priority: medium Normal-priority or P2 work status: needs-review Open pull request requiring current-head review or checks type: maintenance Maintenance, build, dependency, or operational upkeep labels Aug 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: ci-cd CI, GitHub Actions, checks, release, or supply chain automerge-workflow documentation Improvements or additions to documentation priority: medium Normal-priority or P2 work status: needs-review Open pull request requiring current-head review or checks type: maintenance Maintenance, build, dependency, or operational upkeep

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants