Changed not to autofix, if it becomes an invalid attribute

Author: ota-meshi

lib/rules/syntaxes/slot-scope-attribute.js

@@ -8,6 +8,21 @@ module.exports = {
   supported: '2.5.0',
   createTemplateBodyVisitor (context, { fixToUpgrade } = {}) {
     const sourceCode = context.getSourceCode()
+
+    /**
+     * Checks whether the given node can convert to the `v-slot`.
+     * @param {VAttribute | null} slotAttr node of `slot`
+     * @returns {boolean} `true` if the given node can convert to the `v-slot`
+     */
+    function canConvertToVSlot (slotAttr) {
+      if (!slotAttr || !slotAttr.value) {
+        return true
+      }
+      const slotName = slotAttr.value.value
+      // If non-Latin characters are included it can not be converted.
+      return !/[^a-z]/i.test(slotName)
+    }
+
     /**
      * Convert to `v-slot`.
      * @param {object} fixer fixer
@@ -16,7 +31,7 @@ module.exports = {
      * @returns {*} fix data
      */
     function fixSlotToVSlot (fixer, slotAttr, scopeAttr) {
-      const nameArgument = slotAttr && slotAttr.value && slotAttr && slotAttr.value.value
+      const nameArgument = slotAttr && slotAttr.value && slotAttr.value.value
         ? `:${slotAttr.value.value}`
         : ''
       const scopeValue = scopeAttr && scopeAttr.value
@@ -47,6 +62,9 @@ module.exports = {
             const element = scopeAttr.parent
             const slotAttr = element.attributes
               .find(attr => attr.directive === false && attr.key.name === 'slot')
+            if (!canConvertToVSlot(slotAttr)) {
+              return null
+            }
             return fixSlotToVSlot(fixer, slotAttr, scopeAttr)
           }
           : null

tests/lib/rules/no-deprecated-slot-scope-attribute.js

@@ -123,6 +123,22 @@ tester.run('no-deprecated-slot-scope-attribute', rule, {
           line: 4
         }
       ]
+    },
+    // cannot fix
+    {
+      code: `
+      <template>
+        <LinkList>
+          <a slot-scope="{a}" slot="f o o"/>
+        </LinkList>
+      </template>`,
+      output: null,
+      errors: [
+        {
+          message: '`slot-scope` are deprecated.',
+          line: 4
+        }
+      ]
     }
   ]
 })