fix: more commonjs cases

Author: marvinhagemeister

packages/plugin-vite/src/plugins/commonjs.ts

@@ -134,6 +134,70 @@ export function cjsPlugin(
               }
             }
           }
+        } else if (expr.isCallExpression()) {
+          if (
+            t.isMemberExpression(expr.node.callee) &&
+            t.isIdentifier(expr.node.callee.object) &&
+            expr.node.callee.object.name === "Object" &&
+            t.isIdentifier(expr.node.callee.property) &&
+            expr.node.callee.property.name === "defineProperty" &&
+            expr.node.arguments.length === 3 &&
+            t.isIdentifier(expr.node.arguments[0]) &&
+            expr.node.arguments[0].name === "exports" &&
+            t.isStringLiteral(expr.node.arguments[1]) &&
+            expr.node.arguments[1].value !== "__esModule" &&
+            t.isObjectExpression(expr.node.arguments[2])
+          ) {
+            const named = expr.node.arguments[1].value;
+            const obj = expr.node.arguments[2];
+
+            let right: types.Expression = t.nullLiteral();
+
+            for (let i = 0; i < obj.properties.length; i++) {
+              const prop = obj.properties[i];
+
+              if (t.isObjectProperty(prop)) {
+                if (t.isIdentifier(prop.key)) {
+                  if (prop.key.name === "get") {
+                    if (
+                      t.isFunctionExpression(prop.value) ||
+                      t.isArrowFunctionExpression(prop.value)
+                    ) {
+                      right = t.callExpression(
+                        t.parenthesizedExpression(
+                          t.cloneNode(prop.value, true),
+                        ),
+                        [],
+                      );
+                    }
+                  }
+                }
+              } else if (t.isObjectMethod(prop)) {
+                if (t.isIdentifier(prop.key)) {
+                  if (prop.key.name === "get") {
+                    right = t.callExpression(
+                      t.parenthesizedExpression(
+                        t.functionExpression(
+                          null,
+                          [],
+                          t.cloneNode(prop.body, true),
+                        ),
+                      ),
+                      [],
+                    );
+                  }
+                }
+              }
+            }
+
+            path.replaceWith(
+              t.exportNamedDeclaration(
+                t.variableDeclaration("let", [
+                  t.variableDeclarator(t.identifier(named), right),
+                ]),
+              ),
+            );
+          }
         }
       },
     },

packages/plugin-vite/src/plugins/commonjs_test.ts

@@ -174,6 +174,49 @@ exports.trace = 'foo'`,
   });
 });
 
+Deno.test("commonjs - cleared exports", () => {
+  runTest({
+    input: `Object.defineProperty(exports, "__esModule", { value: true });
+exports.foo = exports.bar = void 0;
+exports.foo = 'foo'`,
+    expected: `export let foo = 'foo';`,
+  });
+});
+
+Deno.test("commonjs - define exports", () => {
+  runTest({
+    input: `var utils_1 = require("./bar");
+Object.defineProperty(exports, "foo", { enumerable: true, get: function () { return utils_1.foo; } });`,
+    expected: `import * as _mod from "./bar";
+var utils_1 = _mod.default ?? _mod;
+export let foo = (function () {
+  return utils_1.foo;
+})();`,
+  });
+});
+
+Deno.test("commonjs - define exports #2", () => {
+  runTest({
+    input: `var utils_1 = require("./bar");
+Object.defineProperty(exports, "foo", { enumerable: true, get() { return utils_1.foo; } });`,
+    expected: `import * as _mod from "./bar";
+var utils_1 = _mod.default ?? _mod;
+export let foo = (function () {
+  return utils_1.foo;
+})();`,
+  });
+});
+
+Deno.test("commonjs - define exports #3", () => {
+  runTest({
+    input: `Object.defineProperty(exports, "__esModule", { value: true });
+exports._globalThis = void 0;
+exports._globalThis = typeof globalThis === 'object' ? globalThis : global;`,
+    expected:
+      `export let _globalThis = typeof globalThis === 'object' ? globalThis : global;`,
+  });
+});
+
 // I've never seen this, seems rare. Skipping for now.
 Deno.test.ignore("commonjs - require", () => {
   runTest({