Migrates to vite@6 to drop base64 inlined worker source from all bundles

[ruiconti]

Investigation: #1578 (comment)
Motivation: #1568 (comment)

Opening this tentatively to see if there's interest in merging this, so consumers don't have to rely on patches to fix this issue. If there is, happy to comply with all the contributing guidelines, rebases to master, etc.

Resolves #1578

[Juice10]

Thanks for taking your time to upgrade vite Rui! This won't have the intended result I'm afraid since we are explicitly requesting the worker to be inlined (see &inline code below):

rrweb/packages/rrweb/src/record/observers/canvas/canvas-manager.ts

Line 18 in 9cd28b7

import ImageBitmapDataURLWorker from '../../workers/image-bitmap-data-url-worker?worker&inline';

Edit: I take this back, I hadn't read the code changes in vite 6, this will totally fix the issue, and I'm happy to put in the time to get this over the finish line.

[changeset-bot]

🦋 Changeset detected

Latest commit: aacb7c4

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 19 packages

Name	Type
@rrweb/all	Patch
@rrweb/packer	Patch
@rrweb/record	Patch
rrweb-snapshot	Patch
rrweb	Patch
@rrweb/web-extension	Patch
rrdom	Patch
rrdom-nodejs	Patch
rrweb-player	Patch
@rrweb/replay	Patch
@rrweb/types	Patch
@rrweb/utils	Patch
rrvideo	Patch
@rrweb/rrweb-plugin-console-record	Patch
@rrweb/rrweb-plugin-console-replay	Patch
@rrweb/rrweb-plugin-sequential-id-record	Patch
@rrweb/rrweb-plugin-sequential-id-replay	Patch
@rrweb/rrweb-plugin-canvas-webrtc-record	Patch
@rrweb/rrweb-plugin-canvas-webrtc-replay	Patch

Not sure what this means? Click here to learn what changesets are.

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

[Juice10]

@ruiconti could you rebase this PR on the latest master branch? Failing CI is fixed there.

Edit: looks like this PR needs to be merged first before a rebase would fix the issue: #1647

[pauldambra]

👀

[dzcora]

@ruiconti @Juice10 @pauldambra When did this one merge?

[ebloom19]

🙏

[pauldambra]

fly-by that I was testing this and a map file is being output to dist/assets and not being picked up by my front end bundler when I try and use it

yep, if I manually copy that map file out into my final bundled JS files then all is well

[ruiconti]

Apologies for the delay, somehow I missed the notifications from this PR.

@Juice10 Thanks for creating the changeset. I've just rebased from master, and I'll check back until CI is passing.

[pauldambra]

PostHog/posthog-js#1464 (comment)

users are confirming that a vite6 bundled rrweb passes chrome extension validation for them

[colingm]

@Juice10 @ruiconti are there any blockers left on this PR? Anything that can be done to help?