Skip to content

chore: bump dependabot-rebase reusable to current main SHA#111

Open
don-petry wants to merge 2 commits intomainfrom
chore/bump-dependabot-rebase-sha
Open

chore: bump dependabot-rebase reusable to current main SHA#111
don-petry wants to merge 2 commits intomainfrom
chore/bump-dependabot-rebase-sha

Conversation

@don-petry
Copy link
Copy Markdown
Contributor

Bumps the dependabot-rebase-reusable.yml caller from the stale @v1 tag to the current main SHA of petry-projects/.github.

The @v1 tag points to an old commit that lacks:

  • Fallback @dependabot rebase comment for branches with workflow files (403 workaround)
  • Re-approval after update-branch to satisfy require_last_push_approval
  • Correct if CMD=$(cmd); then ... else ... fi pattern (bash -e bug fix)
  • contents: write + pull-requests: write job-level permissions (required for reusable to write)

This is a routine maintenance bump — no behaviour changes needed in this repo.

Copilot AI review requested due to automatic review settings April 17, 2026 16:39
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 17, 2026

Warning

Rate limit exceeded

@github-actions[bot] has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 51 minutes and 50 seconds before requesting another review.

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 51 minutes and 50 seconds.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: d4571524-52e2-4849-9c41-a0fae22c8c82

📥 Commits

Reviewing files that changed from the base of the PR and between 051f051 and 7843e2e.

📒 Files selected for processing (1)
  • .github/workflows/dependabot-rebase.yml
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/bump-dependabot-rebase-sha

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 and usage tips.

@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Dependabot rebase workflow caller to use a pinned commit SHA of the org reusable workflow, aligning the repo with the current upstream implementation and its required permissions/secrets plumbing.

Changes:

  • Switch petry-projects/.github reusable reference from @v1 to a specific commit SHA.
  • Add workflow_dispatch to allow manual runs.
  • Update job-level permissions and switch from secrets: inherit to an explicit secrets mapping.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +45 to +47
secrets:
APP_ID: ${{ secrets.APP_ID }}
APP_PRIVATE_KEY: ${{ secrets.APP_PRIVATE_KEY }}
push:
branches:
- main
workflow_dispatch: # allow manual trigger to flush Dependabot PR queue
@don-petry
Copy link
Copy Markdown
Contributor Author

Automated review — APPROVED

Risk: MEDIUM
Reviewed commit: 62a399aa369610c2ac53470f292beda117fb89ed
Cascade: triage → deep (see triage: haiku 4.5 → deep: sonnet 4.6 + duck: gpt-5.4 → audit: opus 4.6 for models)

Summary

This PR bumps the dependabot-rebase reusable workflow from a mutable @v1 tag to a pinned SHA, which is a net security improvement. All CI checks pass (CodeQL, SonarCloud, AgentShield all green). The permission escalation (read→write) is necessary for the reusable to perform branch updates and re-approvals, and is clearly documented; the switch from secrets: inherit to an explicit secrets block is strictly more secure than the prior state.

Findings

Minor

  • [minor] .github/workflows/dependabot-rebase.yml:43 — Job permissions escalated from read to write for contents and pull-requests. This is required for the reusable to call update-branch and re-approve PRs (explained in PR body), but reviewers should confirm the GitHub App scopes are similarly scoped.

Info

  • [info] .github/workflows/dependabot-rebase.yml:47 — SHA pinning (replacing @v1 mutable tag with commit SHA 9a694e5) is a best-practice security improvement — the exact code that runs is now immutable.
  • [info] .github/workflows/dependabot-rebase.yml:49 — Switching from secrets: inherit (passes ALL secrets) to explicit APP_ID + APP_PRIVATE_KEY mapping is a least-privilege improvement.
  • [info] .github/workflows/dependabot-rebase.yml:50 — APP_PRIVATE_KEY is forwarded to the petry-projects/.github reusable. This is the same org, pinned to a verified SHA, and is no regression from the prior secrets: inherit which already passed it. The App's actual permission scope is determined by its GitHub App installation — out of scope for this diff.

CI status

All CI checks pass: CodeQL (actions+python), SonarCloud (0 issues, 0 security hotspots), AgentShield, dependency-audit. No failures or warnings.


Reviewed by the don-petry PR-review cascade (triage: haiku 4.5 → deep: sonnet 4.6 + duck: gpt-5.4 → audit: opus 4.6). Reply with @don-petry if you need a human.

@don-petry don-petry enabled auto-merge (squash) April 17, 2026 17:40
@don-petry don-petry disabled auto-merge April 18, 2026 18:31
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