Apply review suggestion

fisker · fisker · commit c5e3bdd0e1f4 · 2020-09-19T00:13:23.000+08:00
diff --git a/src/language-css/parser-postcss.js b/src/language-css/parser-postcss.js
@@ -303,26 +303,46 @@ function parseNestedCSS(node, options) {
     // `postcss@8` parse custom properties as `css-decl`
     if (options.parser === "css" && node.type === "css-decl") {
       const { prop, value } = node;
-      if (
-        prop.startsWith("--") &&
-        value.startsWith("{") &&
-        value.endsWith("}")
-      ) {
-        const textBefore = options.originalText.slice(
-          0,
-          node.source.start.offset
-        );
-        const nodeText =
-          "a".repeat(prop.length) +
-          options.originalText.slice(
-            node.source.start.offset + prop.length,
-            node.source.end.offset + 1
+      if (prop.startsWith("--") && value.startsWith("{")) {
+        let rules;
+        if (value.endsWith("}")) {
+          const textBefore = options.originalText.slice(
+            0,
+            node.source.start.offset
           );
-        const fakeContent = textBefore.replace(/[^\n]/g, " ") + nodeText;
-        node.value = {
-          type: "css-rule",
-          nodes: parseCss(fakeContent, [], { ...options }).nodes[0].nodes,
-        };
+          const nodeText =
+            "a".repeat(prop.length) +
+            options.originalText.slice(
+              node.source.start.offset + prop.length,
+              node.source.end.offset + 1
+            );
+          const fakeContent = textBefore.replace(/[^\n]/g, " ") + nodeText;
+          let ast;
+          try {
+            ast = parseCss(fakeContent, [], { ...options });
+          } catch (_) {
+            // noop
+          }
+          if (
+            ast &&
+            ast.nodes &&
+            ast.nodes.length === 1 &&
+            ast.nodes[0].type === "css-rule"
+          ) {
+            rules = ast.nodes[0].nodes;
+          }
+        }
+        if (rules) {
+          node.value = {
+            type: "css-rule",
+            nodes: rules,
+          };
+        } else {
+          node.value = {
+            type: "value-unknown",
+            value: node.raws.value.raw
+          }
+        }
         return node;
       }
     }
diff --git a/tests/css/variables/__snapshots__/jsfmt.spec.js.snap b/tests/css/variables/__snapshots__/jsfmt.spec.js.snap
@@ -23,6 +23,20 @@ printWidth: 80
   --without-semi: {color:red;}
 }
 
+:root {
+  --like-a-apply-rule: {
+  color:red;} /* no semi here*/
+  --another-prop: blue;
+}
+
+:root {
+  --like-a-apply-rule: {
+  color:red;} /* no semi here*/
+  --another-one-like-a-apply-rule: {
+    color:red;
+  };
+}
+
 =====================================output=====================================
 /* http://tabatkins.github.io/specs/css-apply-rule/#defining */
 
@@ -43,6 +57,20 @@ printWidth: 80
   };
 }
 
+:root {
+  --like-a-apply-rule: {
+  color:red;} /* no semi here*/
+  --another-prop: blue;
+}
+
+:root {
+  --like-a-apply-rule: {
+  color:red;} /* no semi here*/
+  --another-one-like-a-apply-rule: {
+    color:red;
+  };
+}
+
 ================================================================================
 `;
 
diff --git a/tests/css/variables/apply-rule.css b/tests/css/variables/apply-rule.css
@@ -14,3 +14,17 @@
 :root {
   --without-semi: {color:red;}
 }
+
+:root {
+  --like-a-apply-rule: {
+  color:red;} /* no semi here*/
+  --another-prop: blue;
+}
+
+:root {
+  --like-a-apply-rule: {
+  color:red;} /* no semi here*/
+  --another-one-like-a-apply-rule: {
+    color:red;
+  };
+}
