Prepare for `rolldown`/`rolldown-vite` usage

[TheAlexLichter]

With Rolldown in beta and rolldown-vite in the technical preview, a few things could be improved regarding the Sentry JS Bundler plugins:

Preparation for Rolldown and rolldown-vite

Vite Plugin

Right now it seems that using @sentry/vite-plugin causes perf slowdown when used with rolldown-vite (example from Twitter). Using hook filters (and generally migrating when rolldown-vite is detected) could help with that.

Rollup plugin

Same improvements regarding plugin hooks should work for Rollup given that hooks are available in Rolldown, Vite and Rollup.

Bundling

Right now, rollup is also used to bundle all the plugins. Like Vite itself, one could either try using Rolldown directly or a lib bundler like tsdown here

---

Related: e18e/ecosystem-issues#171

[Lms24]

Hey @TheAlexLichter thanks for writing in and sorry for the late reply!

The preparation for Rolldown and rolldown-vite sounds all reasonable to me and like something we should do!

Using hook filters (and generally migrating when rolldown-vite is detected) could help with that

I guess my main question here is, is there a way we can use these hooks filters while being backwards compatible with Vite/Rollup versions that don't support them yet? Given this feature was only added recently, I don't think we can drop support for any prior versions anytime soon (admittedly, I might miss something very obvious here, please forgive my ignorance)

Right now, rollup is also used to bundle all the plugins. Like vitejs/vite#20065, one could either try using Rolldown directly or a lib bundler like tsdown here

Possibly yes, though I'm afraid we won't get to this soon! PRs are absolutely welcome though :) Ideally, we can do this without dropping support for Node 14 (which is the current minimum Node version our plugins support).

[TheAlexLichter]

Using hook filters (and generally migrating when rolldown-vite is detected) could help with that

I guess my main question here is, is there a way we can use these hooks filters while being backwards compatible with Vite/Rollup versions that don't support them yet? Given this feature was only added recently, I don't think we can drop support for any prior versions anytime soon (admittedly, I might miss something very obvious here, please forgive my ignorance)

Yes, you can. Just use them but do not remove the checks in the hooks (see e.g. https://github.com/vitejs/vite-plugin-vue/pull/581/files#r2063412633 as example). That way, older Vite (and I think also Rollup versions) will simply call all existing hooks while newer versions will benefit from the hooks.

Possibly yes, though I'm afraid we won't get to this soon! PRs are absolutely welcome though :) Ideally, we can do this without dropping support for Node 14 (which is the current minimum Node version our plugins support).

That should generally be possible 👍

[Lms24]

Thanks for answering! I'll backlog this with high prio for now but agree that we should get to it soon. (full transparency: we're a bit short-staffed at the moment and the plugins are more or less in maintenance only mode). I'll see if we can somehow get this slotted in.