Don't check type annotations when deciding params scope (#11349)

* Don't check type annotations when deciding params scope

* Also type params

packages/babel-plugin-transform-parameters/src/params.js

@@ -36,6 +36,9 @@ const iifeVisitor = {
       path.stop();
     }
   },
+  // type annotations don't use or introduce "real" bindings
+  "TypeAnnotation|TSTypeAnnotation|TypeParameterDeclaration|TSTypeParameterDeclaration": path =>
+    path.skip(),
 };
 
 export default function convertFunctionParams(path, loose) {

packages/babel-plugin-transform-parameters/test/fixtures/regression/11344-flow/input.js

@@ -0,0 +1,7 @@
+function a(b: (c) => void = {}) {
+  let c;
+}
+
+function d(e = <T>() => {}) {
+  let T;
+}

packages/babel-plugin-transform-parameters/test/fixtures/regression/11344-flow/options.json

@@ -0,0 +1,6 @@
+{
+  "plugins": [
+    "transform-parameters",
+    "syntax-flow"
+  ]
+}

packages/babel-plugin-transform-parameters/test/fixtures/regression/11344-flow/output.js

@@ -0,0 +1,9 @@
+function a() {
+  let b: (c) => void = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : {};
+  let c;
+}
+
+function d() {
+  let e = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : <T>() => {};
+  let T;
+}

packages/babel-plugin-transform-parameters/test/fixtures/regression/11344-typescript/input.js

@@ -0,0 +1,7 @@
+function a(b: (c) => void = {}) {
+  let c;
+}
+
+function d(e = <T>() => {}) {
+  let T;
+}

packages/babel-plugin-transform-parameters/test/fixtures/regression/11344-typescript/options.json

@@ -0,0 +1,6 @@
+{
+  "plugins": [
+    "transform-parameters",
+    "syntax-typescript"
+  ]
+}

packages/babel-plugin-transform-parameters/test/fixtures/regression/11344-typescript/output.js

@@ -0,0 +1,9 @@
+function a() {
+  let b: (c) => void = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : {};
+  let c;
+}
+
+function d() {
+  let e = arguments.length > 0 && arguments[0] !== undefined ? arguments[0] : <T>() => {};
+  let T;
+}