Skip to content

♻️ Drop @rollup/plugin-replace for rolldown builtin#6675

Merged
dubzzz merged 4 commits into
mainfrom
copilot/drop-rollup-plugin-replace
Mar 6, 2026
Merged

♻️ Drop @rollup/plugin-replace for rolldown builtin#6675
dubzzz merged 4 commits into
mainfrom
copilot/drop-rollup-plugin-replace

Conversation

Copilot AI commented Mar 2, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Replace @rollup/plugin-replace with Rolldown's built-in replacePlugin from rolldown/plugins.

  • rolldown.common.config.js: Switch to replacePlugin(values, { preventAssignment: true }) — the builtin separates values from options
  • packages/fast-check/rolldown.config.js: Return only replacement values from callback; preventAssignment moves to common config
  • package.json: Remove @rollup/plugin-replace devDependency

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • rolldown.rs
    • Triggering command: /home/REDACTED/work/_temp/ghcca-node/node/bin/node /home/REDACTED/work/_temp/ghcca-node/node/bin/node --enable-source-maps /home/REDACTED/work/_temp/copilot-developer-action-main/dist/index.js (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

drop rollup plugin replace in favor of https://rolldown.rs/builtin-plugins/replace

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

@changeset-bot

changeset-bot Bot commented Mar 2, 2026 •
edited
Loading

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 621c34b

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copilot AI changed the title [WIP] Remove rollup-plugin-replace in favor of Rolldown replace ♻️ Drop @rollup/plugin-replace for rolldown builtin Mar 2, 2026
@dubzzz
dubzzz marked this pull request as ready for review March 2, 2026 20:06
@dubzzz
dubzzz enabled auto-merge (squash) March 2, 2026 20:06
@pkg-pr-new

pkg-pr-new Bot commented Mar 2, 2026 •
edited
Loading

Copy link
Copy Markdown
@fast-check/ava

npm i https://pkg.pr.new/@fast-check/ava@6675

fast-check

npm i https://pkg.pr.new/fast-check@6675

@fast-check/jest

npm i https://pkg.pr.new/@fast-check/jest@6675

@fast-check/packaged

npm i https://pkg.pr.new/@fast-check/packaged@6675

@fast-check/poisoning

npm i https://pkg.pr.new/@fast-check/poisoning@6675

@fast-check/vitest

npm i https://pkg.pr.new/@fast-check/vitest@6675

@fast-check/worker

npm i https://pkg.pr.new/@fast-check/worker@6675

commit: 1c65fc7

@codecov

codecov Bot commented Mar 2, 2026 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.72%. Comparing base (8b5c54f) to head (621c34b).
⚠️ Report is 8 commits behind head on main.

Additional details and impacted files 
@@           Coverage Diff           @@
##             main    #6675   +/-   ##
=======================================
  Coverage   94.72%   94.72%           
=======================================
  Files         209      209           
  Lines        5666     5666           
  Branches     1501     1500    -1     
=======================================
  Hits         5367     5367           
  Misses        286      286           
  Partials       13       13
Flag Coverage Δ
tests 94.72% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions
💄 Format code with prettier
3be3348 
  Applied by GitHub Action workflow: PR Format
  Run ID: 22598367723
  Target PR: 6675
@dubzzz
dubzzz merged commit 8e47b45 into main Mar 6, 2026
47 checks passed
@dubzzz
dubzzz deleted the copilot/drop-rollup-plugin-replace branch March 6, 2026 21:15
@ghost-of-high-heart ghost-of-high-heart Bot mentioned this pull request Apr 3, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer
@dubzzz dubzzz dubzzz approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

@dubzzz dubzzz

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@dubzzz