Ditch lazyloading polyfill in shimmie utiity, causes issues with 3rd …

…party ESM deps. Fixes #96 v5.0.0 failing to import Prettier plugins (`ERR_MODULE_NOT_FOUND`)
hyperfocalHQ · Jul 16, 2024 · f7440e5 · f7440e5
1 parent efd2a8e
commit f7440e5
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 27 deletions.
diff --git a/rollup.config.js b/rollup.config.js
@@ -4,21 +4,6 @@ import typescript from "rollup-plugin-ts";
 import { preserveShebangs } from "rollup-plugin-preserve-shebangs";
 import { terser } from "rollup-plugin-terser";
 
-function retainImports() {
-  return {
-    name: "retain-import-expression",
-    resolveDynamicImport() {
-      return false;
-    },
-    renderDynamicImport() {
-      return {
-        left: "import(",
-        right: ")",
-      };
-    },
-  };
-}
-
 export default [
   {
     input: "src/utils/index.ts",
@@ -42,7 +27,6 @@ export default [
       resolve(),
       typescript({ outDir: "." }),
       commonjs(),
-      retainImports(),
       preserveShebangs(),
     ],
   },

diff --git a/src/utils/shimmie.ts b/src/utils/shimmie.ts
@@ -1,4 +1,6 @@
-import type { CSSVarsPonyfillOptions } from "css-vars-ponyfill";
+import cssVarsPonyfill, {
+  type CSSVarsPonyfillOptions,
+} from "css-vars-ponyfill";
 /**
  * Conditionally load and apply a shim for CSS variables in IE
  */
@@ -11,16 +13,14 @@ export function shimmie({
 
   if (!test) {
     document.body.style.visibility = "hidden";
-    import("css-vars-ponyfill").then(({ default: cssVarsPonyfill }) => {
-      cssVarsPonyfill({
-        watch: true,
-        updateURLs: false,
-        onComplete() {
-          onComplete?.();
-          document.body.style.visibility = "visible";
-        },
-        ...config,
-      });
+    cssVarsPonyfill({
+      watch: true,
+      updateURLs: false,
+      onComplete() {
+        onComplete?.();
+        document.body.style.visibility = "visible";
+      },
+      ...config,
     });
   }
 }