Skip to content

Conversation

@birkskyum
Copy link
Member

@birkskyum birkskyum commented May 14, 2025

POC of what migration to rolldown bundling will look like - so we have that option for later when it's stable and more widely adopted.

It seems like it's already compatible at this time, and able to pass the test suite. In this PR i've focused on keeping the git diff minimal (thus not renamed rollup.config -> rolldown.config etc.)

Rollup plugins for json and commonjs aren't needed with rolldown AFAICT.

Launch Checklist

  • Confirm your changes do not include backports from Mapbox projects (unless with compliant license) - if you are not sure about this, please ask!
  • Briefly describe the changes in this PR.
  • Link to related issues.
  • Include before/after visuals or gifs if this PR includes visual changes.
  • Write tests for all new functionality.
  • Document any changes to public APIs.
  • Post benchmark scores.
  • Add an entry to CHANGELOG.md under the ## main section.

@birkskyum birkskyum changed the title replace rollup with rolldown POC - replace rollup with rolldown May 14, 2025
@codecov-commenter
Copy link

codecov-commenter commented May 14, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.18%. Comparing base (31ae5f8) to head (1a8d7d5).
⚠️ Report is 21 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1132      +/-   ##
==========================================
- Coverage   95.18%   95.18%   -0.01%     
==========================================
  Files         114      114              
  Lines        7359     7358       -1     
  Branches     2318     2318              
==========================================
- Hits         7005     7004       -1     
  Misses        354      354              

☔ 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.

@HarelM
Copy link
Collaborator

HarelM commented Jun 10, 2025

@birkskyum where are we with this PR? Should it be closed or pushed forward?

@birkskyum birkskyum force-pushed the rolldown-bundling branch from e4ff030 to fd84f37 Compare June 16, 2025 19:49
@birkskyum
Copy link
Member Author

birkskyum commented Jun 16, 2025

It's still relevant, and apart from a version bump i don't expect other changes.

Rolldown is moving towards stabilization very fast, and adoption is growing quickly too, even on the beta versions before it'll be auto-updated by many times more:

Screenshot 2025-06-16 at 21 50 24

I think already now, for such a simple repo as we have here, it's probably fine, but it'll be stable in 1-2 months, so we might as well just wait for that.

@birkskyum birkskyum changed the title POC - replace rollup with rolldown Replace rollup with rolldown Jun 16, 2025
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