From ad090304696ba9ff50dd237f39a154c86476878f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Nicol=C3=B2=20Ribaudo?= <nicolo.ribaudo@gmail.com>
Date: Thu, 22 Apr 2021 23:58:52 +0200
Subject: [PATCH] Parse string epxort names by default (`moduleStringNames`)

---
 packages/babel-parser/src/parser/statement.js                | 1 -
 .../module-string-names/default-import/input.js              | 0
 .../module-string-names/default-import/output.json           | 0
 .../duplicate-exported-binding-check/input.js                | 0
 .../duplicate-exported-binding-check/output.json             | 0
 .../module-string-names/export-from/input.js                 | 0
 .../module-string-names/export-from/output.json              | 0
 .../export-name-has-lone-surrogate/input.js                  | 0
 .../export-name-has-lone-surrogate/output.json               | 0
 .../module-string-names/import-local-is-string/input.js      | 0
 .../module-string-names/import-local-is-string/options.json  | 0
 .../module-string-names/mixed/input.js                       | 0
 .../module-string-names/mixed/output.json                    | 0
 .../module-string-names/named-export/input.js                | 0
 .../module-string-names/named-export/output.json             | 0
 .../module-string-names/named-import/input.js                | 0
 .../module-string-names/named-import/output.json             | 0
 .../module-string-names/namespace-export/input.js            | 0
 .../module-string-names/namespace-export/output.json         | 0
 .../test/fixtures/es2022/module-string-names/options.json    | 3 +++
 .../string-exported-binding-without-from/input.js            | 0
 .../string-exported-binding-without-from/output.json         | 0
 .../_no-plugin/module-string-names-export/input.js           | 2 --
 .../_no-plugin/module-string-names-export/options.json       | 5 -----
 .../_no-plugin/module-string-names-import/input.js           | 1 -
 .../_no-plugin/module-string-names-import/options.json       | 5 -----
 .../fixtures/experimental/module-string-names/options.json   | 4 ----
 27 files changed, 3 insertions(+), 18 deletions(-)
 rename packages/babel-parser/test/fixtures/{experimental => es2022}/module-string-names/default-import/input.js (100%)
 rename packages/babel-parser/test/fixtures/{experimental => es2022}/module-string-names/default-import/output.json (100%)
 rename packages/babel-parser/test/fixtures/{experimental => es2022}/module-string-names/duplicate-exported-binding-check/input.js (100%)
 rename packages/babel-parser/test/fixtures/{experimental => es2022}/module-string-names/duplicate-exported-binding-check/output.json (100%)
 rename packages/babel-parser/test/fixtures/{experimental => es2022}/module-string-names/export-from/input.js (100%)
 rename packages/babel-parser/test/fixtures/{experimental => es2022}/module-string-names/export-from/output.json (100%)
 rename packages/babel-parser/test/fixtures/{experimental => es2022}/module-string-names/export-name-has-lone-surrogate/input.js (100%)
 rename packages/babel-parser/test/fixtures/{experimental => es2022}/module-string-names/export-name-has-lone-surrogate/output.json (100%)
 rename packages/babel-parser/test/fixtures/{experimental => es2022}/module-string-names/import-local-is-string/input.js (100%)
 rename packages/babel-parser/test/fixtures/{experimental => es2022}/module-string-names/import-local-is-string/options.json (100%)
 rename packages/babel-parser/test/fixtures/{experimental => es2022}/module-string-names/mixed/input.js (100%)
 rename packages/babel-parser/test/fixtures/{experimental => es2022}/module-string-names/mixed/output.json (100%)
 rename packages/babel-parser/test/fixtures/{experimental => es2022}/module-string-names/named-export/input.js (100%)
 rename packages/babel-parser/test/fixtures/{experimental => es2022}/module-string-names/named-export/output.json (100%)
 rename packages/babel-parser/test/fixtures/{experimental => es2022}/module-string-names/named-import/input.js (100%)
 rename packages/babel-parser/test/fixtures/{experimental => es2022}/module-string-names/named-import/output.json (100%)
 rename packages/babel-parser/test/fixtures/{experimental => es2022}/module-string-names/namespace-export/input.js (100%)
 rename packages/babel-parser/test/fixtures/{experimental => es2022}/module-string-names/namespace-export/output.json (100%)
 create mode 100644 packages/babel-parser/test/fixtures/es2022/module-string-names/options.json
 rename packages/babel-parser/test/fixtures/{experimental => es2022}/module-string-names/string-exported-binding-without-from/input.js (100%)
 rename packages/babel-parser/test/fixtures/{experimental => es2022}/module-string-names/string-exported-binding-without-from/output.json (100%)
 delete mode 100644 packages/babel-parser/test/fixtures/experimental/_no-plugin/module-string-names-export/input.js
 delete mode 100644 packages/babel-parser/test/fixtures/experimental/_no-plugin/module-string-names-export/options.json
 delete mode 100644 packages/babel-parser/test/fixtures/experimental/_no-plugin/module-string-names-import/input.js
 delete mode 100644 packages/babel-parser/test/fixtures/experimental/_no-plugin/module-string-names-import/options.json
 delete mode 100644 packages/babel-parser/test/fixtures/experimental/module-string-names/options.json

diff --git a/packages/babel-parser/src/parser/statement.js b/packages/babel-parser/src/parser/statement.js
index 3daf7a22cf00..21f7ea15e05a 100644
--- a/packages/babel-parser/src/parser/statement.js
+++ b/packages/babel-parser/src/parser/statement.js
@@ -2089,7 +2089,6 @@ export default class StatementParser extends ExpressionParser {
   // https://tc39.es/ecma262/#prod-ModuleExportName
   parseModuleExportName(): N.StringLiteral | N.Identifier {
     if (this.match(tt.string)) {
-      this.expectPlugin("moduleStringNames");
       const result = this.parseLiteral<N.StringLiteral>(
         this.state.value,
         "StringLiteral",
diff --git a/packages/babel-parser/test/fixtures/experimental/module-string-names/default-import/input.js b/packages/babel-parser/test/fixtures/es2022/module-string-names/default-import/input.js
similarity index 100%
rename from packages/babel-parser/test/fixtures/experimental/module-string-names/default-import/input.js
rename to packages/babel-parser/test/fixtures/es2022/module-string-names/default-import/input.js
diff --git a/packages/babel-parser/test/fixtures/experimental/module-string-names/default-import/output.json b/packages/babel-parser/test/fixtures/es2022/module-string-names/default-import/output.json
similarity index 100%
rename from packages/babel-parser/test/fixtures/experimental/module-string-names/default-import/output.json
rename to packages/babel-parser/test/fixtures/es2022/module-string-names/default-import/output.json
diff --git a/packages/babel-parser/test/fixtures/experimental/module-string-names/duplicate-exported-binding-check/input.js b/packages/babel-parser/test/fixtures/es2022/module-string-names/duplicate-exported-binding-check/input.js
similarity index 100%
rename from packages/babel-parser/test/fixtures/experimental/module-string-names/duplicate-exported-binding-check/input.js
rename to packages/babel-parser/test/fixtures/es2022/module-string-names/duplicate-exported-binding-check/input.js
diff --git a/packages/babel-parser/test/fixtures/experimental/module-string-names/duplicate-exported-binding-check/output.json b/packages/babel-parser/test/fixtures/es2022/module-string-names/duplicate-exported-binding-check/output.json
similarity index 100%
rename from packages/babel-parser/test/fixtures/experimental/module-string-names/duplicate-exported-binding-check/output.json
rename to packages/babel-parser/test/fixtures/es2022/module-string-names/duplicate-exported-binding-check/output.json
diff --git a/packages/babel-parser/test/fixtures/experimental/module-string-names/export-from/input.js b/packages/babel-parser/test/fixtures/es2022/module-string-names/export-from/input.js
similarity index 100%
rename from packages/babel-parser/test/fixtures/experimental/module-string-names/export-from/input.js
rename to packages/babel-parser/test/fixtures/es2022/module-string-names/export-from/input.js
diff --git a/packages/babel-parser/test/fixtures/experimental/module-string-names/export-from/output.json b/packages/babel-parser/test/fixtures/es2022/module-string-names/export-from/output.json
similarity index 100%
rename from packages/babel-parser/test/fixtures/experimental/module-string-names/export-from/output.json
rename to packages/babel-parser/test/fixtures/es2022/module-string-names/export-from/output.json
diff --git a/packages/babel-parser/test/fixtures/experimental/module-string-names/export-name-has-lone-surrogate/input.js b/packages/babel-parser/test/fixtures/es2022/module-string-names/export-name-has-lone-surrogate/input.js
similarity index 100%
rename from packages/babel-parser/test/fixtures/experimental/module-string-names/export-name-has-lone-surrogate/input.js
rename to packages/babel-parser/test/fixtures/es2022/module-string-names/export-name-has-lone-surrogate/input.js
diff --git a/packages/babel-parser/test/fixtures/experimental/module-string-names/export-name-has-lone-surrogate/output.json b/packages/babel-parser/test/fixtures/es2022/module-string-names/export-name-has-lone-surrogate/output.json
similarity index 100%
rename from packages/babel-parser/test/fixtures/experimental/module-string-names/export-name-has-lone-surrogate/output.json
rename to packages/babel-parser/test/fixtures/es2022/module-string-names/export-name-has-lone-surrogate/output.json
diff --git a/packages/babel-parser/test/fixtures/experimental/module-string-names/import-local-is-string/input.js b/packages/babel-parser/test/fixtures/es2022/module-string-names/import-local-is-string/input.js
similarity index 100%
rename from packages/babel-parser/test/fixtures/experimental/module-string-names/import-local-is-string/input.js
rename to packages/babel-parser/test/fixtures/es2022/module-string-names/import-local-is-string/input.js
diff --git a/packages/babel-parser/test/fixtures/experimental/module-string-names/import-local-is-string/options.json b/packages/babel-parser/test/fixtures/es2022/module-string-names/import-local-is-string/options.json
similarity index 100%
rename from packages/babel-parser/test/fixtures/experimental/module-string-names/import-local-is-string/options.json
rename to packages/babel-parser/test/fixtures/es2022/module-string-names/import-local-is-string/options.json
diff --git a/packages/babel-parser/test/fixtures/experimental/module-string-names/mixed/input.js b/packages/babel-parser/test/fixtures/es2022/module-string-names/mixed/input.js
similarity index 100%
rename from packages/babel-parser/test/fixtures/experimental/module-string-names/mixed/input.js
rename to packages/babel-parser/test/fixtures/es2022/module-string-names/mixed/input.js
diff --git a/packages/babel-parser/test/fixtures/experimental/module-string-names/mixed/output.json b/packages/babel-parser/test/fixtures/es2022/module-string-names/mixed/output.json
similarity index 100%
rename from packages/babel-parser/test/fixtures/experimental/module-string-names/mixed/output.json
rename to packages/babel-parser/test/fixtures/es2022/module-string-names/mixed/output.json
diff --git a/packages/babel-parser/test/fixtures/experimental/module-string-names/named-export/input.js b/packages/babel-parser/test/fixtures/es2022/module-string-names/named-export/input.js
similarity index 100%
rename from packages/babel-parser/test/fixtures/experimental/module-string-names/named-export/input.js
rename to packages/babel-parser/test/fixtures/es2022/module-string-names/named-export/input.js
diff --git a/packages/babel-parser/test/fixtures/experimental/module-string-names/named-export/output.json b/packages/babel-parser/test/fixtures/es2022/module-string-names/named-export/output.json
similarity index 100%
rename from packages/babel-parser/test/fixtures/experimental/module-string-names/named-export/output.json
rename to packages/babel-parser/test/fixtures/es2022/module-string-names/named-export/output.json
diff --git a/packages/babel-parser/test/fixtures/experimental/module-string-names/named-import/input.js b/packages/babel-parser/test/fixtures/es2022/module-string-names/named-import/input.js
similarity index 100%
rename from packages/babel-parser/test/fixtures/experimental/module-string-names/named-import/input.js
rename to packages/babel-parser/test/fixtures/es2022/module-string-names/named-import/input.js
diff --git a/packages/babel-parser/test/fixtures/experimental/module-string-names/named-import/output.json b/packages/babel-parser/test/fixtures/es2022/module-string-names/named-import/output.json
similarity index 100%
rename from packages/babel-parser/test/fixtures/experimental/module-string-names/named-import/output.json
rename to packages/babel-parser/test/fixtures/es2022/module-string-names/named-import/output.json
diff --git a/packages/babel-parser/test/fixtures/experimental/module-string-names/namespace-export/input.js b/packages/babel-parser/test/fixtures/es2022/module-string-names/namespace-export/input.js
similarity index 100%
rename from packages/babel-parser/test/fixtures/experimental/module-string-names/namespace-export/input.js
rename to packages/babel-parser/test/fixtures/es2022/module-string-names/namespace-export/input.js
diff --git a/packages/babel-parser/test/fixtures/experimental/module-string-names/namespace-export/output.json b/packages/babel-parser/test/fixtures/es2022/module-string-names/namespace-export/output.json
similarity index 100%
rename from packages/babel-parser/test/fixtures/experimental/module-string-names/namespace-export/output.json
rename to packages/babel-parser/test/fixtures/es2022/module-string-names/namespace-export/output.json
diff --git a/packages/babel-parser/test/fixtures/es2022/module-string-names/options.json b/packages/babel-parser/test/fixtures/es2022/module-string-names/options.json
new file mode 100644
index 000000000000..2104ca43283f
--- /dev/null
+++ b/packages/babel-parser/test/fixtures/es2022/module-string-names/options.json
@@ -0,0 +1,3 @@
+{
+  "sourceType": "module"
+}
diff --git a/packages/babel-parser/test/fixtures/experimental/module-string-names/string-exported-binding-without-from/input.js b/packages/babel-parser/test/fixtures/es2022/module-string-names/string-exported-binding-without-from/input.js
similarity index 100%
rename from packages/babel-parser/test/fixtures/experimental/module-string-names/string-exported-binding-without-from/input.js
rename to packages/babel-parser/test/fixtures/es2022/module-string-names/string-exported-binding-without-from/input.js
diff --git a/packages/babel-parser/test/fixtures/experimental/module-string-names/string-exported-binding-without-from/output.json b/packages/babel-parser/test/fixtures/es2022/module-string-names/string-exported-binding-without-from/output.json
similarity index 100%
rename from packages/babel-parser/test/fixtures/experimental/module-string-names/string-exported-binding-without-from/output.json
rename to packages/babel-parser/test/fixtures/es2022/module-string-names/string-exported-binding-without-from/output.json
diff --git a/packages/babel-parser/test/fixtures/experimental/_no-plugin/module-string-names-export/input.js b/packages/babel-parser/test/fixtures/experimental/_no-plugin/module-string-names-export/input.js
deleted file mode 100644
index 0eed97348183..000000000000
--- a/packages/babel-parser/test/fixtures/experimental/_no-plugin/module-string-names-export/input.js
+++ /dev/null
@@ -1,2 +0,0 @@
-export { foo as "some exports" };
-var foo;
diff --git a/packages/babel-parser/test/fixtures/experimental/_no-plugin/module-string-names-export/options.json b/packages/babel-parser/test/fixtures/experimental/_no-plugin/module-string-names-export/options.json
deleted file mode 100644
index 577ec6148c27..000000000000
--- a/packages/babel-parser/test/fixtures/experimental/_no-plugin/module-string-names-export/options.json
+++ /dev/null
@@ -1,5 +0,0 @@
-{
-  "throws": "This experimental syntax requires enabling the parser plugin: 'moduleStringNames' (1:16)",
-  "sourceType": "module",
-  "plugins": []
-}
diff --git a/packages/babel-parser/test/fixtures/experimental/_no-plugin/module-string-names-import/input.js b/packages/babel-parser/test/fixtures/experimental/_no-plugin/module-string-names-import/input.js
deleted file mode 100644
index 79106029227c..000000000000
--- a/packages/babel-parser/test/fixtures/experimental/_no-plugin/module-string-names-import/input.js
+++ /dev/null
@@ -1 +0,0 @@
-import { "foo" as bar, "default" as qux } from "module-a";
diff --git a/packages/babel-parser/test/fixtures/experimental/_no-plugin/module-string-names-import/options.json b/packages/babel-parser/test/fixtures/experimental/_no-plugin/module-string-names-import/options.json
deleted file mode 100644
index e6b9c1ad0636..000000000000
--- a/packages/babel-parser/test/fixtures/experimental/_no-plugin/module-string-names-import/options.json
+++ /dev/null
@@ -1,5 +0,0 @@
-{
-  "throws": "This experimental syntax requires enabling the parser plugin: 'moduleStringNames' (1:9)",
-  "sourceType": "module",
-  "plugins": []
-}
diff --git a/packages/babel-parser/test/fixtures/experimental/module-string-names/options.json b/packages/babel-parser/test/fixtures/experimental/module-string-names/options.json
deleted file mode 100644
index db7db2aa9480..000000000000
--- a/packages/babel-parser/test/fixtures/experimental/module-string-names/options.json
+++ /dev/null
@@ -1,4 +0,0 @@
-{
-  "sourceType": "module",
-  "plugins": ["moduleStringNames"]
-}