[parser] Make decoratorsBeforeExport default to false (#14695)

packages/babel-parser/src/plugin-utils.ts

@@ -87,14 +87,10 @@ export function validatePlugins(plugins: PluginList) {
       "decorators",
       "decoratorsBeforeExport",
     );
-    if (decoratorsBeforeExport == null) {
-      throw new Error(
-        "The 'decorators' plugin requires a 'decoratorsBeforeExport' option," +
-          " whose value must be a boolean. If you are migrating from" +
-          " Babylon/Babel 6 or want to use the old decorators proposal, you" +
-          " should use the 'decorators-legacy' plugin instead of 'decorators'.",
-      );
-    } else if (typeof decoratorsBeforeExport !== "boolean") {
+    if (
+      decoratorsBeforeExport != null &&
+      typeof decoratorsBeforeExport !== "boolean"
+    ) {
       throw new Error("'decoratorsBeforeExport' must be a boolean.");
     }
   }

packages/babel-parser/test/fixtures/es2022/class-properties/arguments-in-nested-class-decorator-call-expression/options.json

@@ -1,3 +1,3 @@
 {
-  "plugins": [["decorators", { "decoratorsBeforeExport": false }]]
+  "plugins": ["decorators"]
 }

packages/babel-parser/test/fixtures/experimental/decorators/class-property/options.json

@@ -1,3 +1,3 @@
 {
-  "plugins": [["decorators", { "decoratorsBeforeExport": false }]]
+  "plugins": ["decorators"]
 }

packages/babel-parser/test/fixtures/experimental/decorators/compued-property/options.json

@@ -1,3 +1,3 @@
 {
-  "plugins": [["decorators", { "decoratorsBeforeExport": false }]]
+  "plugins": ["decorators"]
 }

packages/babel-parser/test/fixtures/experimental/decorators/no-object-methods/options.json

@@ -1,10 +1,3 @@
 {
-  "plugins": [
-    [
-      "decorators",
-      {
-        "decoratorsBeforeExport": false
-      }
-    ]
-  ]
+  "plugins": ["decorators"]
 }

packages/babel-parser/test/fixtures/experimental/decorators/no-semi/options.json

@@ -1,11 +1,4 @@
 {
-  "plugins": [
-    [
-      "decorators",
-      {
-        "decoratorsBeforeExport": false
-      }
-    ]
-  ],
+  "plugins": ["decorators"],
   "throws": "Decorators must not be followed by a semicolon. (2:5)"
-}
+}

packages/babel-parser/test/fixtures/experimental/decorators/options.json

@@ -1,3 +1,3 @@
 {
-  "plugins": [["decorators", { "decoratorsBeforeExport": false }]]
+  "plugins": ["decorators"]
 }

packages/babel-parser/test/fixtures/experimental/decorators/parenthesized-createParenthesizedExpressions/options.json

@@ -1,4 +1,4 @@
 {
-  "plugins": [["decorators", { "decoratorsBeforeExport": false }]],
+  "plugins": ["decorators"],
   "createParenthesizedExpressions": true
 }

packages/babel-parser/test/fixtures/experimental/decorators/private-property/options.json

@@ -1,3 +1,3 @@
 {
-  "plugins": [["decorators", { "decoratorsBeforeExport": false }]]
+  "plugins": ["decorators"]
 }

packages/babel-parser/test/fixtures/experimental/decorators/restricted-1/options.json

@@ -1,11 +1,4 @@
 {
-  "plugins": [
-    [
-      "decorators",
-      {
-        "decoratorsBeforeExport": false
-      }
-    ]
-  ],
+  "plugins": ["decorators"],
   "throws": "Leading decorators must be attached to a class declaration. (1:6)"
-}
+}

packages/babel-parser/test/fixtures/experimental/decorators/restricted-2/options.json

@@ -1,11 +1,4 @@
 {
-  "plugins": [
-    [
-      "decorators",
-      {
-        "decoratorsBeforeExport": false
-      }
-    ]
-  ],
+  "plugins": ["decorators"],
   "throws": "Leading decorators must be attached to a class declaration. (1:6)"
-}
+}

packages/babel-parser/test/fixtures/experimental/pipeline-operator/hack-double-at-proposal-class-decorator-with-bare-identifier/options.json

@@ -1,6 +1,6 @@
 {
   "plugins": [
     ["pipelineOperator", { "proposal": "hack", "topicToken": "@@" }],
-    ["decorators", { "decoratorsBeforeExport": false }]
+    "decorators"
   ]
 }

packages/babel-parser/test/fixtures/experimental/pipeline-operator/hack-double-at-proposal-class-decorator-with-parenthesized-identifier/options.json

@@ -1,6 +1,6 @@
 {
   "plugins": [
     ["pipelineOperator", { "proposal": "hack", "topicToken": "@@" }],
-    ["decorators", { "decoratorsBeforeExport": false }]
+    "decorators"
   ]
 }

packages/babel-parser/test/fixtures/experimental/pipeline-operator/hack-double-at-proposal-class-decorator-with-parenthesized-topic/options.json

@@ -1,6 +1,6 @@
 {
   "plugins": [
     ["pipelineOperator", { "proposal": "hack", "topicToken": "@@" }],
-    ["decorators", { "decoratorsBeforeExport": false }]
+    "decorators"
   ]
 }

packages/babel-parser/test/fixtures/experimental/pipeline-operator/hack-double-caret-proposal-class-decorator-with-bare-identifier/options.json

@@ -1,6 +1,6 @@
 {
   "plugins": [
     ["pipelineOperator", { "proposal": "hack", "topicToken": "^^" }],
-    ["decorators", { "decoratorsBeforeExport": false }]
+    "decorators"
   ]
 }

packages/babel-parser/test/fixtures/experimental/pipeline-operator/hack-double-caret-proposal-class-decorator-with-parenthesized-identifier/options.json

@@ -1,6 +1,6 @@
 {
   "plugins": [
     ["pipelineOperator", { "proposal": "hack", "topicToken": "^^" }],
-    ["decorators", { "decoratorsBeforeExport": false }]
+    "decorators"
   ]
 }

packages/babel-parser/test/fixtures/experimental/pipeline-operator/hack-double-caret-proposal-class-decorator-with-parenthesized-topic/options.json

@@ -1,6 +1,6 @@
 {
   "plugins": [
     ["pipelineOperator", { "proposal": "hack", "topicToken": "^^" }],
-    ["decorators", { "decoratorsBeforeExport": false }]
+    "decorators"
   ]
 }

packages/babel-parser/test/fixtures/flow/class-properties/declare-after-decorators/options.json

@@ -1,4 +1,4 @@
 {
   "sourceType": "module",
-  "plugins": ["flow", ["decorators", { "decoratorsBeforeExport": false }]]
+  "plugins": ["flow", "decorators"]
 }

packages/babel-parser/test/fixtures/flow/class-properties/declare-before-decorators/options.json

@@ -1,5 +1,5 @@
 {
   "sourceType": "module",
-  "plugins": ["flow", ["decorators", { "decoratorsBeforeExport": false }]],
+  "plugins": ["flow", "decorators"],
   "throws": "Unexpected token (2:10)"
 }