fix: rewriteBindingInitVisitor should skip on scopable node (#10764)

babel · Nov 26, 2019 · 4e774b7 · 4e774b7
1 parent 7f732ad
commit 4e774b7
Show file tree

Hide file tree

Showing 4 changed files with 24 additions and 4 deletions.
diff --git a/packages/babel-helper-module-transforms/src/rewrite-live-references.js b/packages/babel-helper-module-transforms/src/rewrite-live-references.js
@@ -80,10 +80,7 @@ export default function rewriteLiveReferences(
  * A visitor to inject export update statements during binding initialization.
  */
 const rewriteBindingInitVisitor = {
-  ClassProperty(path) {
-    path.skip();
-  },
-  Function(path) {
+  Scope(path) {
     path.skip();
   },
   ClassDeclaration(path) {

diff --git a/packages/babel-plugin-transform-modules-amd/test/fixtures/regression/9346/input.mjs b/packages/babel-plugin-transform-modules-amd/test/fixtures/regression/9346/input.mjs
@@ -0,0 +1,4 @@
+export function bug() {}
+{
+  let bug = 2;
+}
diff --git a/packages/babel-plugin-transform-modules-amd/test/fixtures/regression/9346/options.json b/packages/babel-plugin-transform-modules-amd/test/fixtures/regression/9346/options.json
@@ -0,0 +1,5 @@
+{
+  "plugins": [
+    "transform-modules-amd"
+  ]
+}
diff --git a/packages/babel-plugin-transform-modules-amd/test/fixtures/regression/9346/output.js b/packages/babel-plugin-transform-modules-amd/test/fixtures/regression/9346/output.js
@@ -0,0 +1,14 @@
+define(["exports"], function (_exports) {
+  "use strict";
+
+  Object.defineProperty(_exports, "__esModule", {
+    value: true
+  });
+  _exports.bug = bug;
+
+  function bug() {}
+
+  {
+    let bug = 2;
+  }
+});