fix: avoid duplicate loop and do not skip for decorated class

packages/babel-helper-create-class-features-plugin/src/index.js

@@ -99,19 +99,6 @@ export function createClassFeaturePlugin({
         const computedPaths = [];
         const privateNames = new Set();
         const classElements = path.get("body").get("body");
-        const isPublicClassFieldsTransformRequired = isRequired(
-          "public_class_fields",
-          targets,
-          { compatData },
-        );
-        if (
-          !isPublicClassFieldsTransformRequired &&
-          classElements.every(
-            element => element.isClassMethod() || element.isClassProperty(),
-          )
-        ) {
-          return;
-        }
 
         for (const path of classElements) {
           verifyUsedFeatures(path, this.file);
@@ -182,7 +169,17 @@ export function createClassFeaturePlugin({
           }
         }
 
-        if (!props.length && !isDecorated) return;
+        if (!isDecorated) {
+          if (!props.length) return;
+          const isPublicClassFieldsTransformRequired = isRequired(
+            "public_class_fields",
+            targets,
+            { compatData },
+          );
+          if (!isPublicClassFieldsTransformRequired && !privateNames.size) {
+            return;
+          }
+        }
 
         let ref;
 

packages/babel-plugin-proposal-class-properties/test/fixtures/targets-support-public-fields/decorated/input.js

@@ -0,0 +1,6 @@
+class C {
+  @dec p = 1;
+  static P = 2;
+  m() {}
+  static M() {}
+}

packages/babel-plugin-proposal-class-properties/test/fixtures/targets-support-public-fields/decorated/options.json

@@ -0,0 +1,6 @@
+{
+  "targets": {
+    "node": "12"
+  },
+  "plugins": ["proposal-class-properties", ["proposal-decorators", { "decoratorsBeforeExport": true }], ["external-helpers", { "helperVersion": "7.100.0" }]]
+}

packages/babel-plugin-proposal-class-properties/test/fixtures/targets-support-public-fields/decorated/output.js

@@ -0,0 +1,42 @@
+let C = babelHelpers.decorate(null, function (_initialize) {
+  "use strict";
+
+  class C {
+    constructor() {
+      _initialize(this);
+    }
+
+  }
+
+  return {
+    F: C,
+    d: [{
+      kind: "field",
+      decorators: [dec],
+      key: "p",
+
+      value() {
+        return 1;
+      }
+
+    }, {
+      kind: "field",
+      static: true,
+      key: "P",
+
+      value() {
+        return 2;
+      }
+
+    }, {
+      kind: "method",
+      key: "m",
+      value: function m() {}
+    }, {
+      kind: "method",
+      static: true,
+      key: "M",
+      value: function M() {}
+    }]
+  };
+});