Fix hangs on invalid v-for (#12113)

changelog_unreleased/vue/12113.md

@@ -0,0 +1,21 @@
+#### Fix hangs on invalid `v-for` (#12113 by @fisker)
+
+<!-- prettier-ignore -->
+```jsx
+// Input
+<template>
+  <div>
+    <div v-for=" a in ">aaaaa</div>
+  </div>
+</template>
+
+// Prettier stable
+// Hangs
+
+// Prettier main
+<template>
+  <div>
+    <div v-for=" a in ">aaaaa</div>
+  </div>
+</template>;
+```

src/language-html/syntax-vue.js

@@ -40,8 +40,13 @@ function parseVueFor(value) {
   if (!inMatch) {
     return;
   }
+
   const res = {};
   res.for = inMatch[3].trim();
+  if (!res.for) {
+    return;
+  }
+
   const alias = inMatch[1].trim().replace(stripParensRE, "");
   const iteratorMatch = alias.match(forIteratorRE);
   if (iteratorMatch) {
@@ -54,10 +59,18 @@ function parseVueFor(value) {
     res.alias = alias;
   }
 
+  const left = [res.alias, res.iterator1, res.iterator2];
+  if (
+    left.some(
+      (part, index) =>
+        !part && (index === 0 || left.slice(index + 1).some(Boolean))
+    )
+  ) {
+    return;
+  }
+
   return {
-    left: `${[res.alias, res.iterator1, res.iterator2]
-      .filter(Boolean)
-      .join(",")}`,
+    left: left.filter(Boolean).join(","),
     operator: inMatch[2],
     right: res.for,
   };

tests/format/vue/invalid/__snapshots__/jsfmt.spec.js.snap

@@ -0,0 +1,52 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`v-for.vue format 1`] = `
+====================================options=====================================
+parsers: ["vue"]
+printWidth: 80
+                                                                                | printWidth
+=====================================input======================================
+<template>
+  <div>
+    <div
+      v-for="    _       in          "
+      v-for="            in _        "
+      v-for="            in          "
+      v-for="    _,      in a        "
+      v-for="    ,_      in a        "
+
+      v-for="    a, b,      in a        "
+      v-for="    a,  , c     in a        "
+
+      v-for="     , b, c     in a        "
+      v-for="    a, b,       in a        "
+
+      v-for="     , b, c     in a        "
+      v-for="    a,  , c     in a        "
+      v-for="    (,a,b) of 'abcd'        "
+    ></div>
+  </div>
+</template>
+
+=====================================output=====================================
+<template>
+  <div>
+    <div
+      v-for="    _       in          "
+      v-for="            in _        "
+      v-for="            in          "
+      v-for="_ in a"
+      v-for="    ,_      in a        "
+      v-for="(a, b) in a"
+      v-for="    a,  , c     in a        "
+      v-for="     , b, c     in a        "
+      v-for="(a, b) in a"
+      v-for="     , b, c     in a        "
+      v-for="    a,  , c     in a        "
+      v-for="    (,a,b) of 'abcd'        "
+    ></div>
+  </div>
+</template>
+
+================================================================================
+`;

tests/format/vue/invalid/jsfmt.spec.js

@@ -0,0 +1 @@
+run_spec(__dirname, ["vue"]);

tests/format/vue/invalid/v-for.vue

@@ -0,0 +1,21 @@
+<template>
+  <div>
+    <div
+      v-for="    _       in          "
+      v-for="            in _        "
+      v-for="            in          "
+      v-for="    _,      in a        "
+      v-for="    ,_      in a        "
+
+      v-for="    a, b,      in a        "
+      v-for="    a,  , c     in a        "
+
+      v-for="     , b, c     in a        "
+      v-for="    a, b,       in a        "
+
+      v-for="     , b, c     in a        "
+      v-for="    a,  , c     in a        "
+      v-for="    (,a,b) of 'abcd'        "
+    ></div>
+  </div>
+</template>