Skip to content

Migrate from Dependabot to Renovate - #975

Open
ludeeus wants to merge 5 commits into
mainfrom
claude/gracious-brahmagupta-u8eais
Open

ludeeus wants to merge 5 commits into
mainfrom
claude/gracious-brahmagupta-u8eais

Conversation

@ludeeus

@ludeeus ludeeus commented Oct 18, 2026

Copy link
Copy Markdown
Member

Summary

  • Replace .github/dependabot.yml with .github/renovate.json based on config:best-practices.
  • Weekly schedule (before 6am on Monday, Europe/Amsterdam), 7-day minimum release age, PR limit of 10.
  • Exact version pins for npm dependencies; GitHub Actions and the devcontainer image are pinned to digests.
  • OSV vulnerability alerts and a dependency dashboard summary; security updates skip the schedule, wait 3 days and get a security label.
  • Weekly lock file maintenance.
  • Sentry packages stay grouped through the group:monorepos preset.
  • wrangler.yml: read the wrangler version with ltrimstr("^") so the deploy keeps working with exact versions.

Follow-up

  • Enable the Renovate GitHub app for this repository.
  • Turn off Dependabot security updates in repository settings (keep Dependabot alerts on).

Generated by Claude Code

Replace the Dependabot config with a Renovate config based on
config:best-practices: weekly Monday schedule in Europe/Amsterdam,
7-day minimum release age, OSV vulnerability alerts, exact version
pins and weekly lock file maintenance. Security updates bypass the
schedule, wait 3 days and get a security label.

The Sentry packages stay grouped through config:recommended's
group:monorepos preset (sentry-javascript).

Read the wrangler version with ltrimstr("^") instead of cut -c2- so
the deploy step keeps working once package.json holds exact versions.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PvRm4BkK7RdvnMAS87zfDp
Copilot AI balanced review requested due to automatic review settings October 18, 2026 13:52

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

🟡 Changes recommended

An inherited npm rule overrides the intended seven-day release delay with three days.

1 open issue
What changed in this PR

Migrates dependency update automation from Dependabot to Renovate.

Changes:

  • Adds weekly Renovate updates, version pinning, security alerts, and lockfile maintenance.
  • Removes Dependabot configuration.
  • Adjusts Wrangler version extraction to support exact versions.
File Description
.github/​workflows/​wrangler.yml Preserves exact Wrangler versions during extraction.
.github/​renovate.json Configures Renovate update and security policies.
.github/​dependabot.yml Removes the previous update configuration.

Review effort: Balanced

Comment thread .github/renovate.json Resolved
config:best-practices extends security:minimumReleaseAgeNpm, whose npm
package rule sets a 3-day minimum release age and takes precedence over
the root-level value. Add a repository rule for npm major, minor and
patch updates so they wait 7 days, keeping the preset's exceptions for
pin, replacement and lock file maintenance updates.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PvRm4BkK7RdvnMAS87zfDp

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

🔵 Needs a closer look

Devcontainer digest pinning needs an explicit override, and missing MCR release timestamps block image updates.

No open issues 🎉

2 previously missed issues

In code that hasn't changed since last review

Medium severity Allow MCR images without release timestamps

.github/​renovate.json:23

The global seven-day release-age requirement also applies to .devcontainer.json:3, which uses an image from mcr.microsoft.com. Renovate's Docker datasource provides release timestamps only for Docker Hub, so the default timestamp-required behavior leaves this image's updates pending indefinitely. Add a timestamp-optional exception for MCR images while retaining the age requirement wherever timestamps are available. This is separate from enabling devcontainer digest pinning.

Medium severity Enable digest pinning for devcontainer images

.github/​renovate.json:28

The inherited docker:pinDigests preset explicitly sets pinDigests: false for the devcontainer manager. As a result, .devcontainer.json's image will remain unpinned despite the stated goal. Add a package rule overriding that exception for devcontainers; the release-timestamp restriction also needs to be addressed separately.

1 resolved since last review

Review effort: Balanced

docker:pinDigests disables digest pinning for the devcontainer manager,
so the devcontainer image stayed unpinned. Turn it back on.

Renovate's Docker datasource only gets release timestamps from Docker
Hub. With the default timestamp-required behaviour, images from other
registries, such as the mcr.microsoft.com devcontainer image, never
pass minimumReleaseAge and never get updates. Use timestamp-optional
for Docker images; images with a timestamp still wait 7 days.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PvRm4BkK7RdvnMAS87zfDp
Comment thread .github/renovate.json Resolved
Comment thread .github/renovate.json Resolved

This branch has not been deployed

No deployments
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