Skip to content

chore(dependabot): security-first posture β€” ignore majors, keep grouped patch/minor - #299

Merged
KrasimirKralev merged 2 commits into
mainfrom
chore/dependabot-security-only
Aug 8, 2026
Merged

chore(dependabot): security-first posture β€” ignore majors, keep grouped patch/minor#299
KrasimirKralev merged 2 commits into
mainfrom
chore/dependabot-security-only

Conversation

@KrasimirKralev

@KrasimirKralev KrasimirKralev commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Why

Dependabot is currently emitting risky major version bumps as individual PRs β€” TypeScript 5β†’7, @types/node 20β†’26 (against a Node 22 runtime), jsdom 28β†’30, jest-dom 6β†’7 β€” while its actual value, security-update PRs, is dormant because repo Dependabot alerts are disabled.

For an edge-deployed, network-exposed device OS, stability beats currency for routine deps. This shifts Dependabot to a security-first, low-churn posture.

What changed

  • Add ignore rules for version-update:semver-major on the bun and github-actions ecosystems.
  • Routine version updates are now grouped patch+minor only; framework majors become an opt-in, by-hand decision.
  • Security-update PRs are unaffected β€” they bypass ignore/group rules and still land when alerts are enabled. Majors that fix a real CVE still come through as security updates.
  • The npm open-pull-requests-limit: 0 (security-only) entry is unchanged.

Companion action (admin, not in this PR)

Enable Dependabot alerts in Settings β†’ Code security β€” that turns on the security-update PRs this config is built around. Without it, the security half stays dormant.

Queue cleanup

The open major-bump PRs (#291, #293, #290, #292, #286, #287, #288) are being closed as superseded by this policy. The grouped patch/minor PR (#289) is kept for a device-build check before merge.

Config file only β€” no runtime code touched.

Summary by CodeRabbit

  • Chores
    • Updated automated dependency update settings to distinguish security updates from routine version updates.
    • Excluded major-version updates for Bun and GitHub Actions.
    • Configured weekly GitHub Actions update checks on the beta branch.
    • Limited open GitHub Actions update proposals to three at a time and grouped minor and patch updates.

…ed patch/minor

Dependabot was emitting risky major bumps (TypeScript 5->7, @types/node 20->26
against a Node 22 runtime, jsdom 28->30, jest-dom 6->7) as ungrouped PRs, while
its actual value β€” security-update PRs β€” was dormant because repo alerts are off.

Add ignore rules for version-update:semver-major on the bun and github-actions
ecosystems so routine churn is grouped patch/minor only. Security-update PRs are
unaffected (they bypass ignore/group rules) and still land when alerts are on;
majors become an opt-in, by-hand decision.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@KrasimirKralev
KrasimirKralev requested a review from a team as a code owner August 6, 2026 18:53
@coderabbitai

coderabbitai Bot commented Aug 6, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. πŸŽ‰

ℹ️ Recent review info
βš™οΈ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: fd5fe83a-c3cc-4a81-950c-346aa2ff3763

πŸ“₯ Commits

Reviewing files that changed from the base of the PR and between 474b694 and f35ee17.

πŸ“’ Files selected for processing (1)
  • .github/dependabot.yml

πŸ“ Walkthrough

Walkthrough

Dependabot documentation now defines security and routine update behavior. Bun major updates are ignored. GitHub Actions updates run weekly against beta, allow three open pull requests, and ignore major versions.

Changes

Dependabot policy

Layer / File(s) Summary
Update rules and documentation
.github/dependabot.yml
The configuration documents security-first updates, groups routine Bun minor and patch updates, ignores Bun major updates, and adds weekly GitHub Actions updates targeting beta with a three-PR limit and major-version exclusions.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Suggested reviewers: georgik77, yalexx

πŸš₯ Pre-merge checks | βœ… 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description clearly explains the configuration change and rationale, but it omits the required change type, testing, checklist, and screenshot/log sections. Add the template sections, select the change type, record testing results, complete the checklist, and state whether screenshots or logs are applicable.
βœ… Passed checks (4 passed)
Check name Status Explanation
Title check βœ… Passed The title clearly and concisely describes the Dependabot policy change to prioritize security updates and ignore routine major updates.
Docstring Coverage βœ… Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check βœ… Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check βœ… Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
πŸ§ͺ Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/dependabot-security-only

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.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

πŸ€– Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In @.github/dependabot.yml:
- Around line 4-6: Update the explanatory comment near the β€œDependabot alerts”
note to require enabling both Dependabot alerts and Dependabot security updates
in repository settings, while preserving the existing description of
security-update PR behavior.
πŸͺ„ Autofix

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
βš™οΈ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 41dada55-08e6-4a2e-9452-43a5f4eb3ac7

πŸ“₯ Commits

Reviewing files that changed from the base of the PR and between 8cb6a43 and 474b694.

πŸ“’ Files selected for processing (1)
  • .github/dependabot.yml

Comment thread .github/dependabot.yml Outdated
…quired

CodeRabbit: security-update PRs need 'Dependabot security updates' enabled in
addition to 'Dependabot alerts' β€” alerts alone surface the CVE but don't open
a PR. Clarify the comment so the requirement is explicit.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Aug 8, 2026

Copy link
Copy Markdown

πŸ¦€ ClawReview

Claws waving β€” here's what this change is about.

This PR tightens ClawBox's Dependabot posture to security-first by adding ignore rules for major-version bumps on the bun and github-actions ecosystems, so routine update PRs only surface minor and patch changes. Security-update PRs are explicitly unaffected and still bypass the ignore/group rules whenever Dependabot alerts are enabled. Only .github/dependabot.yml is modified β€” no runtime code touched.

At a glance

  • 🧹 Chore Β· touches CI / Dependabot config
  • Base branch: main Β· +22 source / +0 tests across 1 file
  • βœ… base main matches the beta-first convention (docs/meta-only change)
  • βœ… conventional PR title

Good to know

β€” ClawReview πŸ¦€, scuttling off. General info only β€” see CodeRabbit for the detailed review. Conventions: docs.

@github-actions

github-actions Bot commented Aug 8, 2026

Copy link
Copy Markdown

CI Summary

βœ… Tests

  • Result: passed
  • View run
  • Coverage: statements 70.1%, branches 60.1%, functions 65.41%, lines 72.07%

βœ… E2E

βœ… E2E Install

@KrasimirKralev
KrasimirKralev merged commit 95db306 into main Aug 8, 2026
9 checks passed
@KrasimirKralev
KrasimirKralev deleted the chore/dependabot-security-only branch August 8, 2026 15:23
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