fix(commonjs): preserve the class body property keys even if they are special keywords (#1688)

Author: TrickyPi

packages/commonjs/src/transform-commonjs.js

@@ -291,7 +291,12 @@ export default async function transformCommonjs(
           return;
         case 'Identifier': {
           const { name } = node;
-          if (!isReference(node, parent) || scope.contains(name)) return;
+          if (
+            !isReference(node, parent) ||
+            scope.contains(name) ||
+            (parent.type === 'PropertyDefinition' && parent.key === node)
+          )
+            return;
           switch (name) {
             case 'require':
               uses.require = true;

packages/commonjs/test/fixtures/function/class-body-property-keys-are-special-keywords/_config.js

@@ -0,0 +1,3 @@
+module.exports = {
+  description: 'preserve the class body property keys even if they are special keywords',
+};

packages/commonjs/test/fixtures/function/class-body-property-keys-are-special-keywords/main.js

@@ -0,0 +1,7 @@
+class Rollup {
+  define;
+  require;
+  global;
+}
+
+exports.Rollup = Rollup;

packages/commonjs/test/snapshots/function.js.md

@@ -241,6 +241,30 @@ Generated by [AVA](https://avajs.dev).
       `,
     }
 
+## class-body-property-keys-are-special-keywords
+
+> Snapshot 1
+
+    {
+      'main.js': `'use strict';␊
+      ␊
+      Object.defineProperty(exports, '__esModule', { value: true });␊
+      ␊
+      var main = {};␊
+      ␊
+      class Rollup {␊
+        define;␊
+        require;␊
+        global;␊
+      }␊
+      ␊
+      var Rollup_1 = main.Rollup = Rollup;␊
+      ␊
+      exports.Rollup = Rollup_1;␊
+      exports.default = main;␊
+      `,
+    }
+
 ## compiled-esm-default-is-module-exports-false
 
 > Snapshot 1