fix(ci): sync dependabot-rebase.yml with canonical org stub - #136
fix(ci): sync dependabot-rebase.yml with canonical org stub#136don-petry wants to merge 22 commits into
Conversation
Replace outdated workflow stub with canonical version from petry-projects/.github/standards/workflows/dependabot-rebase.yml. Changes: - Add missing schedule trigger (every 4 hours) as safety net when no pushes to main occur to keep Dependabot PR queue flushed - Update AGENTS comment to document the schedule trigger - Update APP_ID comment to include contents:write permission Closes #132 Co-authored-by: Don Petry <don-petry@users.noreply.github.com>
|
Warning Rate limit exceeded
You’ve run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the 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 configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe Dependabot automation workflow gains time-based and manual triggering capabilities. A scheduled cron trigger (every 4 hours) acts as a safety net when pushes are infrequent, and a manual trigger enables on-demand queue flushing. Documentation comments clarify the trigger roles and GitHub App permission requirements. ChangesDependabot Workflow Triggers and Permissions
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related issues
Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
|
@petry-projects/org-leads — this PR is ready for review and merge. Summary: Syncs Closes #132 |
There was a problem hiding this comment.
Pull request overview
Updates the repository’s dependabot-rebase.yml thin-caller workflow stub to align with the org canonical template by adding a scheduled trigger and refining the embedded guidance comments.
Changes:
- Added a
scheduletrigger (every 4 hours) as a fallback whenpushtomaindoes not occur. - Updated AGENTS guidance to include the new trigger and clarify intent.
- Updated the
APP_IDcomment to note required GitHub App permissions includecontents:write.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| on: | ||
| push: | ||
| branches: | ||
| - main | ||
| schedule: | ||
| - cron: '0 */4 * * *' # every 4 hours — safety net when no pushes to main trigger the chain | ||
| workflow_dispatch: # allow manual trigger to flush Dependabot PR queue |
| # when no PR merges have occurred recently; `workflow_dispatch` allows | ||
| # manual queue flushes). |
|
Auto-rebase blocked — the base branch contains Please rebase this branch manually: |
donpetry-bot
left a comment
There was a problem hiding this comment.
Automated review — APPROVED ✓
Risk: LOW
Reviewed commit: 7a4967a220b6b2abd4800f4e065d28823addbe97
Review mode: triage-approved (single reviewer)
Summary
Single-line change pinning actions/checkout@v5 to its commit SHA (93cb6efe18208431cddfb8368fd83d5badbf9bfd) in .github/workflows/pr-review.yml. This is a standard security best practice that prevents supply chain attacks via tag mutation.
Linked issue analysis
Closes #51 — a compliance finding (action-pinning / error severity) from the weekly audit flagging actions/checkout@v5 as unpinned. The fix directly addresses the finding by pinning to the exact SHA while retaining a # v5 comment for readability.
Findings
- ✅ SHA
93cb6efe18208431cddfb8368fd83d5badbf9bfdverified as the correct commit foractions/checkouttagv5 - ✅ Version comment (
# v5) retained for maintainability - ✅ No functional changes to the workflow logic
- ✅ No security concerns
CI status
| Check | Status |
|---|---|
| Claude Code | ✅ SUCCESS |
| CodeQL (actions) | ✅ SUCCESS |
| CodeQL (javascript-typescript) | ✅ SUCCESS |
| CodeQL (python) | ✅ SUCCESS |
| CodeRabbit | ✅ SUCCESS |
Reviewed automatically by the PR-review agent (single-reviewer). Reply if you need a human review.
donpetry-bot
left a comment
There was a problem hiding this comment.
Automated review — APPROVED ✓
Risk: MEDIUM
Reviewed commit: dd027e55048b2f66c989c4c23bc2f77951173054
Review mode: triage-approved (single reviewer)
Summary
Confirms triage's low-risk classification. This is a security-improving, config-only PR: it adds a SHA-pinned gitleaks secret-scan job, a comprehensive .gitignore secrets baseline (391 lines, verbatim from the petry-projects org standard), and registers gitleaks as a required status check in the code-quality ruleset. The prior audit (cycle 1 on SHA 4ff2f9c) approved the same content set with MEDIUM risk; the new HEAD is a merge commit bringing main forward, with no further substantive changes.
Linked issue analysis
Closes #68 (push-protection compliance). The PR substantively addresses the three code-side compliance gaps (secret_scan_ci_job_present, gitignore_secrets_block, ruleset required-check coverage). The remaining security_and_analysis_unavailable finding is correctly scoped out of code: it requires an admin API call documented in the PR body.
Findings
Info
- [info]
.github/workflows/ci.yml:54—continue-on-error: trueon the gitleaks job is still present from the original bootstrapping window. The CI run on this HEAD shows the gitleaks job passed (Secret scan (gitleaks): SUCCESS), so the GITLEAKS_LICENSE org secret has clearly been provisioned and the flag is now dormant in practice. It should be removed in a follow-up so the required status check actually enforces; this is also the open CodeRabbit comment. Not a blocker — current state introduces no security gap because the underlying check succeeds. - [info] Action SHA pins remain valid (verified in prior audit):
actions/checkout@de0fac2…= v6.0.2,gitleaks/gitleaks-action@ff98106…= v2.3.9. - [info] Workflow permissions are minimal (
contents: read,security-events: write); triggers arepush/pull_request(nopull_request_target); no expression-injection vectors. Org-secret exposure is limited toGITLEAKS_LICENSEon the gitleaks step. - [info]
.gitignorestructure correctly orders negations after their broad patterns; encrypted variants (*.enc.yaml,*.sops.yaml, etc.) are explicitly re-allowed; no over-broad directory ignores.
CI status
All required checks green on HEAD dd027e55: Lint ✓, Format ✓, Secret scan (gitleaks) ✓, SonarCloud ✓ (Quality Gate passed, 0 new issues / 0 security hotspots), AgentShield ✓, CodeQL ✓, dependency-audit ✓, claude-code ✓. mergeStateStatus is BLOCKED only because the org-leads team review is still requested.
Reviewed automatically by the PR-review agent (single-reviewer mode: opus 4.7). Reply if you need a human review.
Superseded by automated re-review at 36d3e57.
Pull request was closed
|
@dev-lead - please fix this PR |
|
|
Auto-rebase failed — merge conflict — this branch has conflicts with dev-lead will attempt to resolve this automatically. If it cannot, a follow-up comment will explain what needs manual attention. To resolve manually instead: |
|
Auto-rebase failed — merge conflict — this branch has conflicts with dev-lead will attempt to resolve this automatically. If it cannot, a follow-up comment will explain what needs manual attention. To resolve manually instead: |
|
Auto-rebase failed — merge conflict — this branch has conflicts with dev-lead will attempt to resolve this automatically. If it cannot, a follow-up comment will explain what needs manual attention. To resolve manually instead: |
|
Auto-rebase failed — merge conflict — this branch has conflicts with dev-lead will attempt to resolve this automatically. If it cannot, a follow-up comment will explain what needs manual attention. To resolve manually instead: |
|
Auto-rebase failed — merge conflict — this branch has conflicts with dev-lead will attempt to resolve this automatically. If it cannot, a follow-up comment will explain what needs manual attention. To resolve manually instead: |
|
Auto-rebase failed — merge conflict — this branch has conflicts with dev-lead will attempt to resolve this automatically. If it cannot, a follow-up comment will explain what needs manual attention. To resolve manually instead: |
|
Auto-rebase failed — merge conflict — this branch has conflicts with dev-lead will attempt to resolve this automatically. If it cannot, a follow-up comment will explain what needs manual attention. To resolve manually instead: |
|
Auto-rebase failed — merge conflict — this branch has conflicts with dev-lead will attempt to resolve this automatically. If it cannot, a follow-up comment will explain what needs manual attention. To resolve manually instead: |
|
Auto-rebase failed — merge conflict — this branch has conflicts with dev-lead will attempt to resolve this automatically. If it cannot, a follow-up comment will explain what needs manual attention. To resolve manually instead: |
|
Auto-rebase failed — merge conflict — this branch has conflicts with dev-lead will attempt to resolve this automatically. If it cannot, a follow-up comment will explain what needs manual attention. To resolve manually instead: |
|
Auto-rebase failed — merge conflict — this branch has conflicts with dev-lead will attempt to resolve this automatically. If it cannot, a follow-up comment will explain what needs manual attention. To resolve manually instead: |
|
Auto-rebase failed — merge conflict — this branch has conflicts with dev-lead will attempt to resolve this automatically. If it cannot, a follow-up comment will explain what needs manual attention. To resolve manually instead: |
|
Auto-rebase failed — merge conflict — this branch has conflicts with dev-lead will attempt to resolve this automatically. If it cannot, a follow-up comment will explain what needs manual attention. To resolve manually instead: |
|
Auto-rebase failed — merge conflict — this branch has conflicts with dev-lead will attempt to resolve this automatically. If it cannot, a follow-up comment will explain what needs manual attention. To resolve manually instead: |
|
Auto-rebase failed — merge conflict — this branch has conflicts with dev-lead will attempt to resolve this automatically. If it cannot, a follow-up comment will explain what needs manual attention. To resolve manually instead: |
|
Auto-rebase failed — merge conflict — this branch has conflicts with dev-lead will attempt to resolve this automatically. If it cannot, a follow-up comment will explain what needs manual attention. To resolve manually instead: |
|
Auto-rebase failed — merge conflict — this branch has conflicts with dev-lead will attempt to resolve this automatically. If it cannot, a follow-up comment will explain what needs manual attention. To resolve manually instead: |
|
Auto-rebase failed — merge conflict — this branch has conflicts with dev-lead will attempt to resolve this automatically. If it cannot, a follow-up comment will explain what needs manual attention. To resolve manually instead: |
|
Closing due to merge conflict that cannot be auto-rebased. Re-implementing from fresh main via dev-lead. |
Pull request was closed



Summary
dependabot-rebase.ymlstub with the canonical version frompetry-projects/.github/standards/workflows/dependabot-rebase.ymlscheduletrigger (cron: '0 */4 * * *') as a safety net when no pushes tomainoccur, keeping the Dependabot PR queue flushedscheduletriggerAPP_IDcomment to includecontents:writepermission noteThe
uses:SHA (b51e2edf830ea085be0277bcf3174c7b3ec8f958 # v1) is unchanged — it already matches the org standard.Closes #132
Generated with Claude Code
Summary by CodeRabbit
Release Notes