Skip to content

Comments

Fix Node 20 CI install failure by aligning Rollup peer dependency#474

Merged
ThomasK33 merged 3 commits intomasterfrom
copilot/combine-version-bumps
Feb 21, 2026
Merged

Fix Node 20 CI install failure by aligning Rollup peer dependency#474
ThomasK33 merged 3 commits intomasterfrom
copilot/combine-version-bumps

Conversation

Copy link
Contributor

Copilot AI commented Feb 21, 2026

Node 20 CI was failing at npm ci with an ERESOLVE conflict after the version-bump consolidation. The failure came from a peer range mismatch between rollup@4.x and @rollup/plugin-json@6.0.0.

  • Root cause

    • @rollup/plugin-json@6.0.0 declared rollup peer support up to ^3, while the repo now pins rollup@4.52.5.
    • This made dependency resolution fail in CI before build/test steps.
  • Dependency alignment

    • Bumped @rollup/plugin-json from ^6.0.0 to ^6.1.0 in package.json.
    • Updated package-lock.json to reflect the resolved graph.
  • Resulting behavior

    • Keeps the Rollup 4 bump intact.
    • Removes the install-time peer conflict for Node 20 CI.
{
  "devDependencies": {
    "@rollup/plugin-json": "^6.1.0",
    "rollup": "4.52.5"
  }
}

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

@coveralls
Copy link

coveralls commented Feb 21, 2026

Pull Request Test Coverage Report for Build 22259340279

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%

Totals Coverage Status
Change from base Build 16596068830: 0.0%
Covered Lines: 278
Relevant Lines: 278

💛 - Coveralls

Co-authored-by: ThomasK33 <2198487+ThomasK33@users.noreply.github.com>
Copilot AI changed the title [WIP] Combine version bumps into a single pull request Consolidate open version bumps into one PR (deps + workflow action pins) Feb 21, 2026
Copilot AI requested a review from ThomasK33 February 21, 2026 15:22
Co-authored-by: ThomasK33 <2198487+ThomasK33@users.noreply.github.com>
Copilot AI changed the title Consolidate open version bumps into one PR (deps + workflow action pins) Fix Node 20 CI install failure by aligning Rollup peer dependency Feb 21, 2026
Copilot AI requested a review from ThomasK33 February 21, 2026 15:28
@ThomasK33 ThomasK33 marked this pull request as ready for review February 21, 2026 15:30
@ThomasK33 ThomasK33 merged commit 73d5200 into master Feb 21, 2026
10 checks passed
@ThomasK33 ThomasK33 deleted the copilot/combine-version-bumps branch February 21, 2026 15:30
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