Parse string export names by default (moduleStringNames) (#13195)

* Parse string epxort names by default (`moduleStringNames`) * Remove remaining references
babel · Apr 23, 2021 · a9f5e07 · a9f5e07
1 parent bf053df
commit a9f5e07
Show file tree

Hide file tree

Showing 30 changed files with 6 additions and 28 deletions.
diff --git 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/...dule-string-names/default-import/input.js → ...dule-string-names/default-import/input.js b/...dule-string-names/default-import/input.js → ...dule-string-names/default-import/input.js
diff --git a/...e-string-names/default-import/output.json → ...e-string-names/default-import/output.json b/...e-string-names/default-import/output.json → ...e-string-names/default-import/output.json
diff --git a/...duplicate-exported-binding-check/input.js → ...duplicate-exported-binding-check/input.js b/...duplicate-exported-binding-check/input.js → ...duplicate-exported-binding-check/input.js
diff --git a/...licate-exported-binding-check/output.json → ...licate-exported-binding-check/output.json b/...licate-exported-binding-check/output.json → ...licate-exported-binding-check/output.json
diff --git a/.../module-string-names/export-from/input.js → .../module-string-names/export-from/input.js b/.../module-string-names/export-from/input.js → .../module-string-names/export-from/input.js
diff --git a/...dule-string-names/export-from/output.json → ...dule-string-names/export-from/output.json b/...dule-string-names/export-from/output.json → ...dule-string-names/export-from/output.json
diff --git a/...s/export-name-has-lone-surrogate/input.js → ...s/export-name-has-lone-surrogate/input.js b/...s/export-name-has-lone-surrogate/input.js → ...s/export-name-has-lone-surrogate/input.js
diff --git a/...xport-name-has-lone-surrogate/output.json → ...xport-name-has-lone-surrogate/output.json b/...xport-name-has-lone-surrogate/output.json → ...xport-name-has-lone-surrogate/output.json
diff --git a/...ing-names/import-local-is-string/input.js → ...ing-names/import-local-is-string/input.js b/...ing-names/import-local-is-string/input.js → ...ing-names/import-local-is-string/input.js
diff --git a/...names/import-local-is-string/options.json → ...names/import-local-is-string/options.json b/...names/import-local-is-string/options.json → ...names/import-local-is-string/options.json
@@ -1,7 +1,4 @@
 {
   "sourceType": "module",
-  "plugins": [
-    "moduleStringNames"
-  ],
   "throws": "A string literal cannot be used as an imported binding.\n- Did you mean `import { \"foo\" as foo }`? (1:9)"
-}
+}
diff --git a/...mental/module-string-names/mixed/input.js → ...es2022/module-string-names/mixed/input.js b/...mental/module-string-names/mixed/input.js → ...es2022/module-string-names/mixed/input.js
diff --git a/...tal/module-string-names/mixed/output.json → ...022/module-string-names/mixed/output.json b/...tal/module-string-names/mixed/output.json → ...022/module-string-names/mixed/output.json
diff --git a/...module-string-names/named-export/input.js → ...module-string-names/named-export/input.js b/...module-string-names/named-export/input.js → ...module-string-names/named-export/input.js
diff --git a/...ule-string-names/named-export/output.json → ...ule-string-names/named-export/output.json b/...ule-string-names/named-export/output.json → ...ule-string-names/named-export/output.json
diff --git a/...module-string-names/named-import/input.js → ...module-string-names/named-import/input.js b/...module-string-names/named-import/input.js → ...module-string-names/named-import/input.js
diff --git a/...ule-string-names/named-import/output.json → ...ule-string-names/named-import/output.json b/...ule-string-names/named-import/output.json → ...ule-string-names/named-import/output.json
diff --git a/...le-string-names/namespace-export/input.js → ...le-string-names/namespace-export/input.js b/...le-string-names/namespace-export/input.js → ...le-string-names/namespace-export/input.js
diff --git a/...string-names/namespace-export/output.json → ...string-names/namespace-export/output.json b/...string-names/namespace-export/output.json → ...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
@@ -0,0 +1,3 @@
+{
+  "sourceType": "module"
+}
diff --git a/...ng-exported-binding-without-from/input.js → ...ng-exported-binding-without-from/input.js b/...ng-exported-binding-without-from/input.js → ...ng-exported-binding-without-from/input.js
diff --git a/...exported-binding-without-from/output.json → ...exported-binding-without-from/output.json b/...exported-binding-without-from/output.json → ...exported-binding-without-from/output.json
diff --git a/...es/babel-parser/test/fixtures/experimental/_no-plugin/module-string-names-export/input.js b/...es/babel-parser/test/fixtures/experimental/_no-plugin/module-string-names-export/input.js
diff --git a/...abel-parser/test/fixtures/experimental/_no-plugin/module-string-names-export/options.json b/...abel-parser/test/fixtures/experimental/_no-plugin/module-string-names-export/options.json
diff --git a/...es/babel-parser/test/fixtures/experimental/_no-plugin/module-string-names-import/input.js b/...es/babel-parser/test/fixtures/experimental/_no-plugin/module-string-names-import/input.js
diff --git a/...abel-parser/test/fixtures/experimental/_no-plugin/module-string-names-import/options.json b/...abel-parser/test/fixtures/experimental/_no-plugin/module-string-names-import/options.json
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
diff --git a/...s/babel-parser/test/fixtures/flow/module-string-names/import-local-is-string/options.json b/...s/babel-parser/test/fixtures/flow/module-string-names/import-local-is-string/options.json
@@ -1,8 +1,5 @@
 {
   "sourceType": "module",
-  "plugins": [
-    "moduleStringNames",
-    "flow"
-  ],
+  "plugins": ["flow"],
   "throws": "A string literal cannot be used as an imported binding.\n- Did you mean `import { \"foo\" as foo }`? (1:9)"
 }
diff --git a/packages/babel-parser/test/fixtures/flow/module-string-names/options.json b/packages/babel-parser/test/fixtures/flow/module-string-names/options.json
@@ -1,4 +1,4 @@
 {
   "sourceType": "module",
-  "plugins": ["flow", "moduleStringNames"]
+  "plugins": ["flow"]
 }
diff --git a/packages/babel-traverse/test/introspection.js b/packages/babel-traverse/test/introspection.js
@@ -176,7 +176,6 @@ describe("path/introspection", function () {
     it('rejects the "export called *" trick', function () {
       const program = getPath(`import * as ns from "source"; ns["*"].nested;`, {
         sourceType: "module",
-        plugins: ["moduleStringNames"],
       });
       const reference = program.get("body.1.expression");
       expect(reference.referencesImport("source", "nested")).toBe(false);