fix: hoist variable declaration within do block

packages/babel-plugin-proposal-do-expressions/test/fixtures/do-expressions/variable-declaration-start.js

@@ -1,4 +1,7 @@
-expect(do {
-  var bar = "foo";
-  bar;
-}).toBe("foo");
+expect(
+  do {
+    var bar = "foo";
+    bar;
+  }
+).toBe("foo");
+expect(bar).toBe("foo");

packages/babel-plugin-proposal-do-expressions/test/fixtures/do-expressions/variable-declaration-within-function.js

@@ -0,0 +1,8 @@
+expect(
+  () => do {
+    () => {
+      var bar = "foo";
+    };
+    bar;
+  }
+).toThrow(ReferenceError);

packages/babel-traverse/package.json

@@ -19,6 +19,7 @@
     "@babel/code-frame": "workspace:^7.12.13",
     "@babel/generator": "workspace:^7.13.9",
     "@babel/helper-function-name": "workspace:^7.12.13",
+    "@babel/helper-hoist-variables": "workspace:^7.13.0",
     "@babel/helper-split-export-declaration": "workspace:^7.12.13",
     "@babel/parser": "workspace:^7.13.13",
     "@babel/types": "workspace:^7.13.13",

packages/babel-traverse/src/path/replacement.ts

@@ -6,35 +6,7 @@ import NodePath from "./index";
 import { path as pathCache } from "../cache";
 import { parse } from "@babel/parser";
 import * as t from "@babel/types";
-
-const hoistVariablesVisitor = {
-  Function(path) {
-    path.skip();
-  },
-
-  VariableDeclaration(path) {
-    if (path.node.kind !== "var") return;
-
-    const bindings = path.getBindingIdentifiers();
-    for (const key of Object.keys(bindings)) {
-      path.scope.push({ id: bindings[key] });
-    }
-
-    const exprs = [];
-
-    for (const declar of path.node.declarations as Array<any>) {
-      if (declar.init) {
-        exprs.push(
-          t.expressionStatement(
-            t.assignmentExpression("=", declar.id, declar.init),
-          ),
-        );
-      }
-    }
-
-    path.replaceWithMultiple(exprs);
-  },
-};
+import hoistVariables from "@babel/helper-hoist-variables";
 
 /**
  * Replace a node with an array of multiple. This method performs the following steps:
@@ -238,7 +210,16 @@ export function replaceExpressionWithStatements(
     t.CallExpression & { callee: t.ArrowFunctionExpression }
   >;
 
-  this.traverse(hoistVariablesVisitor);
+  // hoist variable declaration in do block
+  // `(do { var x = 1; x;})` -> `var x; (() => { x = 1; return x; })()`
+  const callee = (this as ThisType).get("callee");
+  hoistVariables(
+    callee.get("body"),
+    id => {
+      this.scope.push({ id });
+    },
+    "var",
+  );
 
   // add implicit returns to all ending expression statements
   const completionRecords: Array<NodePath> = (this as ThisType)

yarn.lock

@@ -3492,6 +3492,7 @@ __metadata:
     "@babel/code-frame": "workspace:^7.12.13"
     "@babel/generator": "workspace:^7.13.9"
     "@babel/helper-function-name": "workspace:^7.12.13"
+    "@babel/helper-hoist-variables": "workspace:^7.13.0"
     "@babel/helper-plugin-test-runner": "workspace:*"
     "@babel/helper-split-export-declaration": "workspace:^7.12.13"
     "@babel/parser": "workspace:^7.13.13"