@griffel/transform 2.0.1 → 2.0.2

package/CHANGELOG.md

@@ -1,16 +1,19 @@
 # Change Log - @griffel/transform
 
-This log was last generated on Thu, 12 Mar 2026 17:19:24 GMT and should not be manually modified.
+This log was last generated on Fri, 13 Mar 2026 16:59:28 GMT and should not be manually modified.
 
 <!-- Start content -->
 
-## 2.0.1
+## 2.0.2
 
-Thu, 12 Mar 2026 17:19:24 GMT
+Fri, 13 Mar 2026 16:59:28 GMT
 
 ### Patches
 
-- Bump @griffel/transform-shaker to v1.0.1
+- fix: wrap VM errors as host Error with filename context (olfedias@microsoft.com)
+- perf: skip eval cache for __mkPreval entry-point evaluations (olfedias@microsoft.com)
+- fix: wrap VM errors as host Error with filename context, defer CJS export assignments for IIFE patterns (olfedias@microsoft.com)
+- Bump @griffel/transform-shaker to v1.0.2
 
 ## 2.0.0
 

package/evaluation/module.d.mts

@@ -25,7 +25,7 @@ export declare class Module {
         cache: Record<string, Module>;
         resolve: (id: string) => string;
     };
-    evaluate(text: string, only?: string[] | null): void;
+    evaluate(text: string, only?: string[] | null, useEvalCache?: boolean): void;
     static invalidate: () => void;
     static invalidateEvalCache: () => void;
     static _nodeModulePaths: (filename: string) => string[];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@griffel/transform",
-  "version": "2.0.1",
+  "version": "2.0.2",
   "description": "A package that performs build time transforms for Griffel",
   "license": "MIT",
   "repository": {
@@ -17,7 +17,7 @@
   },
   "dependencies": {
     "@griffel/core": "^1.20.1",
-    "@griffel/transform-shaker": "^1.0.1",
+    "@griffel/transform-shaker": "^1.0.2",
     "debug": "^4.3.0",
     "magic-string": "^0.30.19",
     "oxc-parser": "^0.116.0",

package/transform.js

@@ -51,6 +51,7 @@ function convertESMtoCJS(code, filename) {
     return code;
   }
   const ms = new MagicString(code);
+  const deferredExports = [];
   for (const node of program.body) {
     switch (node.type) {
       case "ImportDeclaration": {
@@ -105,14 +106,11 @@ function convertESMtoCJS(code, filename) {
             const names = prop(decl, "declarations").flatMap(
               (d) => extractDeclaredNames(prop(d, "id"))
             );
-            const exportsCode = names.map((name) => `exports.${name} = ${name};`).join(" ");
-            ms.appendLeft(node.end, "\n" + exportsCode);
+            deferredExports.push(...names);
           } else if (decl.type === "FunctionDeclaration" || decl.type === "ClassDeclaration") {
             const id = prop(decl, "id");
             if (id) {
-              const name = prop(id, "name");
-              ms.appendLeft(node.end, `
-exports.${name} = ${name};`);
+              deferredExports.push(prop(id, "name"));
             }
           }
         } else if (prop(node, "source")) {
@@ -173,6 +171,9 @@ exports.default = ${prop(id, "name")};`);
       }
     }
   }
+  if (deferredExports.length > 0) {
+    ms.append("\n" + deferredExports.map((name) => `exports.${name} = ${name};`).join("\n"));
+  }
   ms.prepend('Object.defineProperty(exports, "__esModule", { value: true });\n');
   return ms.toString();
 }
@@ -293,6 +294,9 @@ const debug = createDebug("griffel:module");
 let cache = {};
 const NOOP = () => {
 };
+function isError$1(e) {
+  return e != null && typeof e === "object" && "message" in e && "stack" in e;
+}
 const createCustomDebug = (depth) => (namespaces, arg1, ...args) => {
   const modulePrefix = depth === 0 ? "module" : `sub-module-${depth}`;
   debug(`${modulePrefix}:${namespaces}`, arg1, ...args);
@@ -383,7 +387,7 @@ class Module {
       resolve: (id) => this.resolveFilename(id, this).path
     }
   );
-  evaluate(text, only = null) {
+  evaluate(text, only = null, useEvalCache = true) {
     const { filename } = this;
     let action = "ignore";
     for (let i = this.rules.length - 1; i >= 0; i--) {
@@ -394,7 +398,7 @@ class Module {
       }
     }
     const cacheKey = [this.filename, ...only ?? []];
-    if (has(cacheKey, text)) {
+    if (useEvalCache && has(cacheKey, text)) {
       this.exports = get(cacheKey, text);
       return;
     }
@@ -417,25 +421,36 @@ ${code}`);
 })(exports);`, {
       filename: this.filename
     });
-    script.runInContext(
-      vm.createContext({
-        clearImmediate: NOOP,
-        clearInterval: NOOP,
-        clearTimeout: NOOP,
-        setImmediate: NOOP,
-        setInterval: NOOP,
-        setTimeout: NOOP,
-        fetch: NOOP,
-        global,
-        process: mockProcess,
-        module: this,
-        exports: this.exports,
-        require: this.require,
-        __filename: this.filename,
-        __dirname: path.dirname(this.filename)
-      })
-    );
-    set(cacheKey, text, this.exports);
+    try {
+      script.runInContext(
+        vm.createContext({
+          clearImmediate: NOOP,
+          clearInterval: NOOP,
+          clearTimeout: NOOP,
+          setImmediate: NOOP,
+          setInterval: NOOP,
+          setTimeout: NOOP,
+          fetch: NOOP,
+          global,
+          process: mockProcess,
+          module: this,
+          exports: this.exports,
+          require: this.require,
+          __filename: this.filename,
+          __dirname: path.dirname(this.filename)
+        })
+      );
+    } catch (vmError) {
+      const message = isError$1(vmError) ? vmError.message : String(vmError);
+      const hostError = new Error(message);
+      if (isError$1(vmError)) {
+        hostError.stack = vmError.stack;
+      }
+      throw hostError;
+    }
+    if (useEvalCache) {
+      set(cacheKey, text, this.exports);
+    }
   }
   static invalidate = () => {
     cache = {};
@@ -529,7 +544,12 @@ export const __mkPreval = (() => {
 `;
   try {
     const mod = new Module(filename, evaluationRules, resolveFilename);
-    mod.evaluate(codeForEvaluation, ["__mkPreval"]);
+    mod.evaluate(
+      codeForEvaluation,
+      ["__mkPreval"],
+      /* useEvalCache */
+      false
+    );
     const result = mod.exports.__mkPreval;
     if (isError(result)) {
       return { confident: false, error: result };
@@ -765,7 +785,9 @@ function transformSync(sourceCode, options) {
         }
         const functionKind = importedName;
         if (node.arguments.length !== 1) {
-          throw new Error(`${functionKind}() function accepts only a single param`);
+          throw new Error(
+            `${functionKind}() function accepts only a single param, got ${node.arguments.length} in ${filename}`
+          );
         }
         matchedSpecifiers.set(declaration.node.start, {
           start: declaration.node.start,