[pigment-css] Fix evaluation of undefined variables (#41569)

mui · Mar 20, 2024 · 4ca7231 · 4ca7231
1 parent f6a89c1
commit 4ca7231
Show file tree

Hide file tree

Showing 5 changed files with 15 additions and 0 deletions.
diff --git a/packages/pigment-css-unplugin/package.json b/packages/pigment-css-unplugin/package.json
@@ -32,6 +32,7 @@
     "@pigment-css/react": "workspace:^",
     "@wyw-in-js/shared": "^0.5.0",
     "@wyw-in-js/transform": "^0.5.0",
+    "babel-plugin-define-var": "^0.1.0",
     "babel-plugin-transform-react-remove-prop-types": "^0.4.24",
     "unplugin": "^1.7.1"
   },

diff --git a/packages/pigment-css-unplugin/src/index.ts b/packages/pigment-css-unplugin/src/index.ts
@@ -222,6 +222,7 @@ export const plugin = createUnplugin<PigmentOptions, true>((options) => {
               ...rest.babelOptions,
               plugins: [
                 ['babel-plugin-transform-react-remove-prop-types', { mode: 'remove' }],
+                'babel-plugin-define-var', // A fix for undefined variables in the eval phase of wyw-in-js, more details on https://github.com/siriwatknp/babel-plugin-define-var?tab=readme-ov-file#problem
                 ...(rest.babelOptions?.plugins ?? []),
               ],
             },

diff --git a/packages/pigment-css-vite-plugin/package.json b/packages/pigment-css-vite-plugin/package.json
@@ -33,6 +33,7 @@
     "@pigment-css/react": "workspace:^",
     "@wyw-in-js/shared": "^0.5.0",
     "@wyw-in-js/transform": "^0.5.0",
+    "babel-plugin-define-var": "^0.1.0",
     "babel-plugin-transform-react-remove-prop-types": "^0.4.24"
   },
   "devDependencies": {

diff --git a/packages/pigment-css-vite-plugin/src/vite-plugin.ts b/packages/pigment-css-vite-plugin/src/vite-plugin.ts
@@ -193,6 +193,7 @@ export default function wywVitePlugin({
                   ...rest.babelOptions,
                   plugins: [
                     ['babel-plugin-transform-react-remove-prop-types', { mode: 'remove' }],
+                    'babel-plugin-define-var', // A fix for undefined variables in the eval phase of wyw-in-js, more details on https://github.com/siriwatknp/babel-plugin-define-var?tab=readme-ov-file#problem
                     ...(rest.babelOptions?.plugins ?? []),
                   ],
                   presets: Array.from(presets),

diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml