fix(player): cdn breaking due to node modules mangling

mihar-22 · mihar-22 · commit 7dc7588c4863 · 2024-04-09T14:39:27.000+10:00
diff --git a/packages/vidstack/rollup.config.js b/packages/vidstack/rollup.config.js
@@ -5,7 +5,7 @@ import { fileURLToPath } from 'node:url';
 import { nodeResolve } from '@rollup/plugin-node-resolve';
 import chokidar from 'chokidar';
 import * as eslexer from 'es-module-lexer';
-import { build } from 'esbuild';
+import { build, transform as esbuildTransform } from 'esbuild';
 import fsExtra from 'fs-extra';
 import { globbySync } from 'globby';
 import { defineConfig } from 'rollup';
@@ -231,16 +231,28 @@ function defineNPMBundle({ target, type, minify }) {
         platform: isServer ? 'node' : 'browser',
         minify: minify,
         legalComments: 'none',
-        mangleProps: shouldMangle ? /^_/ : undefined,
-        mangleCache: shouldMangle ? MANGLE_CACHE : undefined,
-        reserveProps: shouldMangle ? /^__/ : undefined,
         define: {
           __DEV__: !isProd && !isServer ? 'true' : 'false',
           __SERVER__: isServer ? 'true' : 'false',
           __CDN__: MODE_CDN ? 'true' : 'false',
           __TEST__: 'false',
         },
       }),
+      shouldMangle && {
+        name: 'mangle',
+        async transform(code, id) {
+          if (id.includes('node_modules')) return null;
+          return (
+            await esbuildTransform(code, {
+              target: 'esnext',
+              platform: 'neutral',
+              mangleProps: /^_/,
+              mangleCache: MANGLE_CACHE,
+              reserveProps: /^__/,
+            })
+          ).code;
+        },
+      },
     ],
   };
 }
