Skip to content

chore(ci): enable Dependabot version updates#4000

Open
MRmarioruci wants to merge 3 commits into
mainfrom
chore/ci-enable-dependabot
Open

chore(ci): enable Dependabot version updates#4000
MRmarioruci wants to merge 3 commits into
mainfrom
chore/ci-enable-dependabot

Conversation

@MRmarioruci

Copy link
Copy Markdown
Contributor

Problem

The repo runs Dependabot security updates (reactive, vulnerability-triggered) but has no dependabot.yml, so version updates never fire. Deps drift forward silently and only surface once they become security issues — exactly what update-deps.yml (#3998) does for the toolchain, but for libraries.

Changes

  • New .github/dependabot.yml covering 4 ecosystems (npm, cargo, github-actions, docker).
  • L3 grouping with two carve-outs:
    • npm: bulk PR + agent-js carve-out (@dfinity/* tied to IC release cadence).
    • cargo: bulk PR + pocket-ic carve-out (replica-coupled).
  • Weekly Mon staggered (08/09/10 UTC) so reviews don't pile.
  • Majors ignored across the board — manual upgrade path.
  • CODEOWNERS already auto-requests reviews from @dfinity/identity.
  • Commit prefix chore(deps) / chore(deps-dev) to match repo convention.

No auto-merge — every Dependabot PR gets human eyes on diff and CI before landing.

Tests

  • YAML schema validates locally before commit.
  • First real fire Mon 08:00 UTC post-merge; expected ≤6 PRs/week steady state.

Adds .github/dependabot.yml covering npm, cargo, github-actions, and
docker. Weekly Mon staggered (08/09/10 UTC), aggressive L3 grouping
collapsed to one PR per ecosystem with two carve-outs:

  - npm: bulk + @dfinity/* (IC-coupled release cadence)
  - cargo: bulk + pocket-ic (replica-coupled)

Majors ignored — manual upgrade path. CODEOWNERS auto-requests reviews
from @dfinity/identity. Commit prefix matches repo convention.

Complements the toolchain workflow (#3998); this catches library
drift before it becomes a security alert.
@MRmarioruci MRmarioruci force-pushed the chore/ci-enable-dependabot branch from 40f0c7f to 4d1fa41 Compare June 9, 2026 13:51
@MRmarioruci MRmarioruci marked this pull request as ready for review June 9, 2026 13:51
@MRmarioruci MRmarioruci requested a review from a team as a code owner June 9, 2026 13:51
Copilot AI review requested due to automatic review settings June 9, 2026 13:51
@zeropath-ai

zeropath-ai Bot commented Jun 9, 2026

Copy link
Copy Markdown

No security or compliance issues detected. Reviewed everything up to 2a10282.

Security Overview
Detected Code Changes
Change Type Relevant files
Configuration changes ► .github/dependabot.yml
    Configure Dependabot for cargo, npm, github-actions, and docker
    Set up weekly dependency updates with specific schedules and limits
    Define ignore rules for major version updates
    Create groups for dependency updates (e.g., cargo-all, npm-all, actions-all)

Copilot AI 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.

Pull request overview

This PR introduces a new .github/dependabot.yml so the repository gets regular (non-security) dependency update PRs across key ecosystems, reducing drift and catching issues earlier.

Changes:

  • Adds Dependabot version-update configuration for Cargo, npm, GitHub Actions, and Docker.
  • Sets weekly schedules (Mon) with grouping and carve-outs (pocket-ic, @dfinity/*) and ignores semver-major updates.
  • Applies consistent commit-message prefixes (chore(deps) / chore(deps-dev)).

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

Comment thread .github/dependabot.yml
Comment thread .github/dependabot.yml
Comment thread .github/dependabot.yml
MRmarioruci and others added 2 commits June 9, 2026 16:06
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Comment thread .github/dependabot.yml
timezone: Etc/UTC
open-pull-requests-limit: 5
commit-message:
prefix: "chore(deps)"

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Can we factor out the common stuff like PR prefix?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Not really. The dependabot yaml parser rejects things like anchors/aliases, shared defaults blocks etc.. dependabot/dependabot-core#1582. There are workarounds with scripts etc.. but seems too dirty to me for this.

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.

3 participants