eslint-plugin-crisp 1.0.14 → 1.0.16

package/index.js

@@ -28,6 +28,7 @@ module.exports = {
     "two-lines-between-class-members": require("./rules/two-lines-between-class-members"),
     "variable-names": require("./rules/variable-names"),
     "align-consecutive-class-assignements": require("./rules/align-consecutive-class-assignements"),
+    "vue-html-quotes": require("./rules/vue-html-quotes"),
     "vue-props-declaration-order": require("./rules/vue-props-declaration-order")
   }
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-crisp",
-  "version": "1.0.14",
+  "version": "1.0.16",
   "description": "Custom EsLint Rules for Crisp",
   "main": "index.js",
   "author": "Crisp IM SAS",
@@ -11,7 +11,8 @@
   },
   "peerDependencies": {
     "eslint": "8.45.0",
-    "eslint-plugin-vue": "8.5.0",
-    "eslint-plugin-vue-pug": "0.6.0"
+    "eslint-plugin-vue": "8.7.1",
+    "eslint-plugin-vue-pug": "0.6.0",
+    "vue-eslint-parser": "9.3.1"
   }
 }

package/recommended-vue.js

@@ -11,7 +11,8 @@ module.exports = {
   extends: [
     "eslint:recommended",
     "plugin:jsdoc/recommended",
-    "plugin:vue/vue3-recommended"
+    "plugin:vue/vue3-recommended",
+    "plugin:vue-pug/vue3-recommended"
   ],
 
   rules: {
@@ -105,29 +106,43 @@ module.exports = {
     "crisp/jsdocs-align-params": "error",
 
     // General Vue rules
-    "vue/multi-word-component-names": "error",
     "vue/no-v-html": "off",
+    "vue/html-quotes": "off",
+    "vue/component-tags-order": [
+      "error",
+
+      {
+        "order": [ "template", "script", "style" ]
+      }
+    ],
     "vue/attributes-order": [
       "error",
+
       {
         order: [
-          "OTHER_DIRECTIVES",
+          "RENDER_MODIFIERS",
           "CONDITIONALS",
+          "OTHER_DIRECTIVES",
           "LIST_RENDERING",
-          "RENDER_MODIFIERS",
           "SLOT",
           "TWO_WAY_BINDING",
           "CONTENT",
           "EVENTS",
-          "GLOBAL",
           "DEFINITION",
-          "UNIQUE",
-          "OTHER_ATTR"
-          ["ATTR_DYNAMIC", "ATTR_STATIC", "ATTR_SHORTHAND_BOOL"]
+          ["GLOBAL", "UNIQUE", "OTHER_ATTR"]
         ],
         "alphabetical": false
       }
     ],
+    "vue/v-slot-style": [
+      "error",
+
+      {
+        "atComponent": "v-slot",
+        "default": "v-slot",
+        "named": "longform"
+      }
+    ],
 
     // Crisp JS rules
     "crisp/header-check": "error",
@@ -149,6 +164,7 @@ module.exports = {
     ],
 
     // Crisp Vue rules
+    "crisp/vue-html-quotes": "error",
     "crisp/vue-props-declaration-order": "error"
   },
 

package/rules/vue-html-quotes.js

@@ -0,0 +1,76 @@
+module.exports = {
+  meta: {
+    type: "layout",
+    docs: {
+      description: "enforce quotes style of HTML attributes"
+    },
+
+    schema: [
+      { enum: ["double", "single"] },
+      {
+        type: "object",
+        properties: {
+          avoidEscape: {
+            type: "boolean"
+          }
+        },
+        additionalProperties: false
+      }
+    ],
+
+    messages: {
+      expected: "Expected to be enclosed by {{kind}}."
+    }
+  },
+
+  create(context) {
+    const sourceCode = context.getSourceCode()
+    const double = context.options[0] !== "single"
+    const avoidEscape =
+      context.options[1] && context.options[1].avoidEscape === true
+    const quoteChar = double ? '"' : "'"
+    const quoteName = double ? "double quotes" : "single quotes"
+
+    return context.parserServices.defineTemplateBodyVisitor(
+      {
+        "VAttribute[value!=null]"(node) {
+          const text = sourceCode.getText(node.value)
+          const firstChar = text[0]
+
+          if (firstChar !== quoteChar) {
+            const contentText = text.slice(1, -1);
+
+            const quoted = firstChar === "'" || firstChar === '"';
+
+            if (avoidEscape && quoted) {
+              if (contentText.includes(quoteChar)) {
+
+                return
+              }
+            }
+
+            // Attribute value is an object or an array
+            if (firstChar === "[" || firstChar === "{") {
+              // Matching quote in the object
+              if (contentText.includes(quoteChar)) {
+                return
+              }
+
+              // No quotes in the object or array
+              if (!contentText.includes("'") && !contentText.includes('"')) {
+                return
+              }
+            }
+
+            context.report({
+              node: node.value,
+              loc: node.value.loc,
+              messageId: "expected",
+              data: { kind: quoteName }
+            })
+          }
+        }
+      }
+    )
+  }
+}