fix: should not remove let binding even it is wrapped in closure

Fixes #10339

packages/babel-plugin-transform-block-scoping/src/index.js

@@ -453,9 +453,7 @@ class BlockScoping {
       if (binding.kind === "let" || binding.kind === "const") {
         binding.kind = "var";
 
-        if (wrappedInClosure) {
-          scope.removeBinding(ref.name);
-        } else {
+        if (!wrappedInClosure) {
           scope.moveBindingTo(ref.name, parentScope);
         }
       }

packages/babel-plugin-transform-block-scoping/test/fixtures/general/issue-10339/input.js

@@ -0,0 +1,4 @@
+for (const {foo, ...bar} of {}) {
+  () => foo;
+  bar;
+}

packages/babel-plugin-transform-block-scoping/test/fixtures/general/issue-10339/options.json

@@ -0,0 +1,3 @@
+{
+  "plugins": ["transform-block-scoping", ["proposal-object-rest-spread", { "loose": true }]]
+}

packages/babel-plugin-transform-block-scoping/test/fixtures/general/issue-10339/output.js

@@ -0,0 +1,16 @@
+function _objectWithoutPropertiesLoose(source, excluded) { if (source == null) return {}; var target = {}; var sourceKeys = Object.keys(source); var key, i; for (i = 0; i < sourceKeys.length; i++) { key = sourceKeys[i]; if (excluded.indexOf(key) >= 0) continue; target[key] = source[key]; } return target; }
+
+var _loop = function (foo, bar) {
+  () => foo;
+
+  bar;
+};
+
+for (var _ref of {}) {
+  var {
+    foo
+  } = _ref,
+      bar = _objectWithoutPropertiesLoose(_ref, ["foo"]);
+
+  _loop(foo, bar);
+}