Skip to content

ci(gh-actions): group vitest dependabot updates - #2655

Merged
madhavilosetty-intel merged 2 commits into
mainfrom
ci/group-vitest-updates
Sep 8, 2026
Merged

ci(gh-actions): group vitest dependabot updates#2655
madhavilosetty-intel merged 2 commits into
mainfrom
ci/group-vitest-updates

Conversation

@madhavilosetty-intel

@madhavilosetty-intel madhavilosetty-intel commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Adds an npm groups entry so Dependabot raises vitest and @vitest/* as a single PR.

Why

@vitest/coverage-v8 declares an exact peer on the matching vitest version. When Dependabot bumps them separately, the resulting pair cannot install:

npm error code ERESOLVE
npm error While resolving: @vitest/coverage-v8@5.0.0
npm error Found: vitest@4.1.11
npm error   peer vitest@"5.0.0" from @vitest/coverage-v8@5.0.0

That is exactly what happened in #2652 — one PR bumping only the coverage package, failing all four build checks. #2654 fixes the current bump by hand; this change stops it recurring.

The github-actions ecosystem already uses this pattern for codeql-action, for the same lockstep reason.

Note

Grouping only affects future Dependabot PRs; it does not retroactively combine existing ones.

vitest and @vitest/* release in lockstep and declare exact peers on each other,
so Dependabot raising them as separate PRs produces a version pair that cannot
install. #2652 bumped @vitest/coverage-v8 to 5.0.0 on its own and failed every
build job with ERESOLVE, because that release requires vitest@5.0.0 exactly.

Grouping them means one PR that moves both together.

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.

🟢 Approval recommended

The Dependabot grouping is correctly scoped and syntactically valid, and it directly addresses the documented lockstep peer-dependency issue without impacting runtime code.

Pull request overview

Configures Dependabot to group vitest and @vitest/* npm dependency updates into a single PR to avoid peer-dependency resolution failures caused by lockstep versioning (notably @vitest/coverage-v8 requiring an exact matching vitest version).

Changes:

  • Adds a Dependabot groups.vitest rule for the npm ecosystem.
  • Groups vitest and @vitest/* patterns so related updates land together.
File summaries
File Description
.github/dependabot.yml Adds an npm Dependabot group to combine vitest + @vitest/* updates into one PR to prevent split update install failures.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@madhavilosetty-intel
madhavilosetty-intel merged commit 8514eb9 into main Sep 8, 2026
10 checks passed
@madhavilosetty-intel
madhavilosetty-intel deleted the ci/group-vitest-updates branch September 8, 2026 20:22
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