Skip to content

chore(auto-rebase): pin caller to @auto-rebase/stable channel (was frozen @v2) - #919

Merged
don-petry merged 2 commits into
mainfrom
chore/auto-rebase-pin-stable-channel
Jun 25, 2026
Merged

don-petry merged 2 commits into
mainfrom
chore/auto-rebase-pin-stable-channel

Conversation

@don-petry

Copy link
Copy Markdown
Collaborator

What

Re-syncs this repo's .github/workflows/auto-rebase.yml caller stub from the canonical standards/workflows/auto-rebase.yml. The only functional change is the reusable pin:

- uses: ...auto-rebase-reusable.yml@v2
+ uses: ...auto-rebase-reusable.yml@auto-rebase/stable

plus corrected header guidance and documentation of the eligibility / ready_label inputs.

Why

This caller was pinned to the frozen @v2 tag (376a4fcb, 2026-05-19), which:

Rollout interaction

Part of the auto-rebase review-ready rollout. Ordering is safe either way:

Depends on combsmith/.github#528 (keystone tooling_ref fix) being merged and the channel promoted before the restriction takes effect here.

🤖 Generated with Claude Code

This repo's auto-rebase caller pinned the frozen `@v2` tag (376a4fcb,
2026-05-19), which predates the review-ready eligibility gate (#465/#468)
and violates the org standard (callers MUST pin the centrally-advanced
`auto-rebase/stable` channel, never a frozen `@vX` — see
standards/ci-standards.md → Reusable workflow versioning).

Re-syncs the caller stub from standards/workflows/auto-rebase.yml:
  - uses: ...auto-rebase-reusable.yml@v2 → @auto-rebase/stable
  - corrected "MUST NOT change the uses ref" header guidance
  - documents the eligibility / ready_label inputs

Effect: once the `auto-rebase/stable` channel is promoted to a commit
containing #528 (the tooling_ref fix) + #468 (the gate), this repo
automatically adopts the review-ready restriction. Until then it runs the
current channel head (unchanged unrestricted behavior) — no broken state.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019LUUSUQHqwLWZ583SAs41K
@don-petry
don-petry requested a review from a team as a code owner June 24, 2026 22:39
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@gemini-code-assist

Copy link
Copy Markdown

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@coderabbitai

coderabbitai Bot commented Jun 24, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

@don-petry, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 53 minutes and 9 seconds. Learn how PR review limits work.

Your organization has used up its prepaid credits, and credit purchases are no longer available. Enable the review add-on in the billing tab to keep reviews running — you're only billed for reviews past your plan's rate limits ($0.25/file).

⌛ How to resolve this issue?

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 credits.

🚦 How do rate 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 see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 630982eb-007b-45dc-860e-53eb0753c7c6

📥 Commits

Reviewing files that changed from the base of the PR and between b612b91 and 89f9821.

📒 Files selected for processing (4)
  • .github/workflows/auto-rebase.yml
  • .github/workflows/test-dev-lead.yml
  • AGENTS.md
  • tests/dev-lead/integration/test_auto_rebase_stub.py
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/auto-rebase-pin-stable-channel

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.

@don-petry

Copy link
Copy Markdown
Collaborator Author

Dev-Lead — review-changes (applied)

Changes committed and pushed.

@don-petry
don-petry enabled auto-merge (squash) June 24, 2026 22:43
@don-petry
don-petry disabled auto-merge June 24, 2026 22:44
@sonarqubecloud

Copy link
Copy Markdown

@don-petry

Copy link
Copy Markdown
Collaborator Author

Dev-Lead — review-changes (no-changes)

No changes were needed for this PR.

@don-petry
don-petry enabled auto-merge (squash) June 24, 2026 22:44
@donpetry-bot

Copy link
Copy Markdown
Contributor

Advisory bots were rate-limited; auto-approval is withheld until they recover. pr-review-sweep will re-review this PR after 2026-06-24T23:45:35Z.

@donpetry-bot donpetry-bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Automated review — APPROVED ✓

Risk: MEDIUM
Reviewed commit: 89f98215986fc9182d0221639687265f8cb92c0d
Review mode: triage-approved (single reviewer)

Summary

Repins the auto-rebase.yml caller stub from the frozen @v2 tag to the centrally-advanced @auto-rebase/stable channel, aligning it with the canonical org stub and ci-standards.md. Matching, consistent updates to the regression test (test_auto_rebase_stub.py EXPECTED_REF), its CI step name, and the AGENTS.md exception note. One functional line; the rest are docs/test. Brings the caller into org-standard compliance rather than violating it.

Linked issue analysis

No issue is closed by this PR (closingIssuesReferences empty). It references #465/#468/#528 as rollout context and #139 as the test that enforces the pin; all are context, not closure targets. Not a blocker.

Findings

No blocking findings.

  • The single functional change (@v2 -> @auto-rebase/stable) targets a same-org first-party reusable. SHA-pinning is explicitly exempted for first-party reusables (AGENTS.md), and the moving channel tag is the mandated standard; this is by design, not a supply-chain smell.
  • All four files are mutually consistent (workflow ref, test EXPECTED_REF + endswith guard, step name, AGENTS.md note); the CI auto-rebase-stub job passed against the head SHA, confirming the test enforces the new ref.
  • GitHub Secret Protection MCP (run_secret_scanning) not exposed in this environment; gitleaks CI check passed and the diff contains no secret material.
  • Advisory reviewers (CodeRabbit, Codex, Gemini) were rate-limited or unsupported, but hard security gates (CodeQL actions+python, gitleaks, AgentShield, SonarCloud) all passed.

CI status

All checks green. SUCCESS: CodeQL (actions+python), Secret scan (gitleaks), AgentShield, SonarCloud, shellcheck/bats, unit-tests, validate-fixtures, auto-rebase-stub, and all Test Dev-Lead jobs. SKIPPED (n/a): dependabot-automerge, npm/pnpm/pip/cargo/govulncheck audits, dev-lead ci-relay. No failing checks. mergeStateStatus BLOCKED only because the org-leads team review is still required.


Reviewed automatically by the PR-review agent (single-reviewer mode: fable 5). Reply if you need a human review.

@don-petry
don-petry merged commit 680b304 into main Jun 25, 2026
38 checks passed
@don-petry
don-petry deleted the chore/auto-rebase-pin-stable-channel branch June 25, 2026 01:33
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.

2 participants