chore: remove rollup web worker processing (#60)

since we're using vite's built in

Author: pauldambra

CHANGELOG.md

@@ -1,7 +1,5 @@
 ## 0.0.12 - 2025-03-17
 
-
-
 ## 0.0.11 - 2025-03-14
 
 ## 0.0.10 - 2025-02-28

packages/rrweb-snapshot/test/integration.test.ts

@@ -112,8 +112,8 @@ describe('integration tests', function (this: ISuite) {
   });
 
   afterAll(async () => {
-    await browser.close();
-    await server.close();
+    await browser?.close();
+    await server?.close();
   });
 
   for (const html of htmls) {

packages/rrweb/package.json

@@ -44,12 +44,10 @@
             }
         },
         "./dist/rrweb.css": "./dist/rrweb.css",
-        "./dist/style.css": "./dist/style.css",
-        "./dist/assets/*": "./dist/assets/*"
+        "./dist/style.css": "./dist/style.css"
     },
     "files": [
         "dist",
-        "dist/assets",
         "package.json"
     ],
     "sideEffects": false,
@@ -89,4 +87,4 @@
         "@posthog/rrdom": "*",
         "@posthog/rrweb-snapshot": "*"
     }
-}
+}

packages/rrweb/rollup.config.js

@@ -3,7 +3,6 @@ import esbuild from 'rollup-plugin-esbuild';
 import resolve from '@rollup/plugin-node-resolve';
 import postcss from 'rollup-plugin-postcss';
 import renameNodeModules from 'rollup-plugin-rename-node-modules';
-import webWorkerLoader from 'rollup-plugin-web-worker-loader';
 import pkg from './package.json';
 
 function toRecordPath(path) {
@@ -122,12 +121,6 @@ function getPlugins(options = {}) {
   const { minify = false, sourceMap = false } = options;
   return [
     resolve({ browser: true }),
-    webWorkerLoader({
-      targetPlatform: 'browser',
-      inline: true,
-      preserveSource: true,
-      sourceMap,
-    }),
     esbuild({
       minify,
     }),
@@ -144,13 +137,6 @@ for (const c of baseConfigs) {
   const basePlugins = [
     resolve({ browser: true }),
 
-    // supports bundling `web-worker:..filename`
-    webWorkerLoader({
-      targetPlatform: 'browser',
-      inline: true,
-      preserveSource: true,
-    }),
-
     typescript(),
   ];
   const plugins = basePlugins.concat(

packages/rrweb/test/integration.test.ts

@@ -55,8 +55,8 @@ describe('record integration tests', function (this: ISuite) {
   });
 
   afterAll(async () => {
-    await browser.close();
-    server.close();
+    await browser?.close();
+    server?.close();
   });
 
   it('can record clicks', async () => {

packages/rrweb/vite.config.js

@@ -1,4 +1,28 @@
 import config from '../../vite.config.default';
+import { defineConfig } from 'vite';
+import { mergeConfig } from 'vite';
 
 // export default config('src/index.ts', 'rrweb', { outputDir: 'dist/main' });
-export default config('src/index.ts', 'rrweb');
+const baseConfig = config('src/index.ts', 'rrweb');
+
+export default defineConfig((configEnv) =>
+    mergeConfig(
+        baseConfig(configEnv),
+        {
+            plugins: [{
+                name: 'move-worker-sourcemap',
+                generateBundle(options, bundle) {
+                    Object.entries(bundle).forEach(([fileName, output]) => {
+                        if (fileName.includes('worker') && fileName.endsWith('.map')) {
+                            console.log('Moving worker sourcemap:', fileName);
+                            const newFileName = fileName.replace('assets/', '');
+                            bundle[newFileName] = output;
+                            output.fileName = newFileName;  // Update the fileName property
+                            delete bundle[fileName];
+                        }
+                    });
+                }
+            }]
+        }
+    )
+);

vite.config.default.ts

@@ -10,11 +10,6 @@ import { visualizer } from 'rollup-plugin-visualizer';
 
 // don't empty out dir if --watch flag is passed
 const emptyOutDir = !process.argv.includes('--watch');
-/**
- * Chrome web store does not allow base64 inline workers.
- * For chrome extension, we need to disable worker inlining to pass the review.
- */
-const disableWorkerInlining = process.env.DISABLE_WORKER_INLINING === 'true';
 
 function minifyAndUMDPlugin({
   name,
@@ -162,19 +157,6 @@ export default function (
         filename: resolve(__dirname, name + '-bundle-analysis.html'), // Path for the HTML report
         open: false, // don't Automatically open the report in the browser
       }),
-      {
-        name: 'remove-worker-inline',
-        enforce: 'pre',
-        transform(code, id) {
-          if (!disableWorkerInlining) return;
-          if (/\.(js|ts|jsx|tsx)$/.test(id)) {
-            return {
-              code: code.replace(/\?worker&inline/g, '?worker'),
-              map: null,
-            };
-          }
-        },
-      },
       ...plugins,
     ],
   }));