@meteorlxy/eslint-config 3.1.1 → 3.3.0

package/README.md

@@ -30,6 +30,7 @@ Configure `.vscode/settings.json`:
   "eslint.validate": [
     "javascript",
     "javascriptreact",
+    "markdown",
     "typescript",
     "typescriptreact",
     "vue",

package/dist/index.cjs

@@ -288,18 +288,6 @@ const builtinRules = {
   "no-extra-bind": "error",
   "no-extra-boolean-cast": "error",
   "no-extra-label": "error",
-  "no-extra-parens": [
-    "off",
-    "all",
-    {
-      conditionalAssign: true,
-      nestedBinaryExpressions: false,
-      returnAssign: false,
-      ignoreJSX: "all",
-      enforceForArrowConditionals: false
-    }
-  ],
-  "no-extra-semi": "error",
   "no-fallthrough": "error",
   "no-floating-decimal": "error",
   "no-func-assign": "error",
@@ -595,6 +583,7 @@ const builtinRules = {
   "one-var": ["error", "never"],
   "operator-assignment": ["error", "always"],
   "padding-line-between-statements": "off",
+  // deprecated
   "prefer-arrow-callback": [
     "error",
     {
@@ -1133,6 +1122,42 @@ const reactRules = {
 };
 
 const typescriptShimRules = {
+  /**
+   * Rules that are checked by typescript or recommended to disable in typescript.
+   */
+  "constructor-super": "off",
+  "getter-return": "off",
+  "no-const-assign": "off",
+  "no-dupe-args": "off",
+  "no-dupe-class-members": "off",
+  "no-dupe-keys": "off",
+  "no-func-assign": "off",
+  "no-invalid-this": "off",
+  "no-import-assign": "off",
+  "no-new-symbol": "off",
+  "no-obj-calls": "off",
+  "no-redeclare": "off",
+  "no-setter-return": "off",
+  "no-this-before-super": "off",
+  "no-undef": "off",
+  "no-unreachable": "off",
+  "no-unsafe-negation": "off",
+  "valid-typeof": "off",
+  /**
+   * Rules from eslint-plugin-import that should be disabled
+   *
+   * @see https://typescript-eslint.io/troubleshooting/performance-troubleshooting#eslint-plugin-import
+   */
+  "import/default": "off",
+  "import/named": "off",
+  "import/namespace": "off",
+  "import/no-named-as-default-member": "off",
+  "import/no-unresolved": "off",
+  // might run them in CI
+  "import/no-cycle": "off",
+  "import/no-deprecated": "off",
+  "import/no-named-as-default": "off",
+  "import/no-unused-modules": "off",
   /**
    * Rules that have equivalents in typescript-eslint plugin.
    */
@@ -1179,44 +1204,7 @@ const typescriptShimRules = {
   "semi": "off",
   "space-before-blocks": "off",
   "space-before-function-paren": "off",
-  "space-infix-ops": "off",
-  /**
-   * Rules that are checked by typescript or recommended to disable in typescript.
-   */
-  // builtin rules
-  "constructor-super": "off",
-  "getter-return": "off",
-  "no-const-assign": "off",
-  "no-dupe-args": "off",
-  "no-dupe-class-members": "off",
-  "no-dupe-keys": "off",
-  "no-func-assign": "off",
-  "no-invalid-this": "off",
-  "no-import-assign": "off",
-  "no-new-symbol": "off",
-  "no-obj-calls": "off",
-  "no-redeclare": "off",
-  "no-setter-return": "off",
-  "no-this-before-super": "off",
-  "no-undef": "off",
-  "no-unreachable": "off",
-  "no-unsafe-negation": "off",
-  "valid-typeof": "off",
-  /**
-   * Rules from eslint-plugin-import that should be disabled
-   *
-   * @see https://typescript-eslint.io/troubleshooting/performance-troubleshooting#eslint-plugin-import
-   */
-  "import/default": "off",
-  "import/named": "off",
-  "import/namespace": "off",
-  "import/no-named-as-default-member": "off",
-  "import/no-unresolved": "off",
-  // might run them in CI
-  "import/no-cycle": "off",
-  "import/no-deprecated": "off",
-  "import/no-named-as-default": "off",
-  "import/no-unused-modules": "off"
+  "space-infix-ops": "off"
 };
 
 const typescriptRules = {
@@ -1447,12 +1435,22 @@ const typescriptRules = {
 };
 
 const vueRules = {
+  "vue/block-lang": [
+    "off",
+    {
+      script: { lang: "ts" },
+      style: { allowNoLang: true }
+    }
+  ],
   "vue/block-order": [
     "error",
     {
       order: ["script", "template", "style"]
     }
   ],
+  "vue/camelcase": "off",
+  // builtin extension - checked by typescript
+  "vue/component-api-style": ["error", ["script-setup", "composition"]],
   "vue/component-name-in-template-casing": [
     "error",
     "PascalCase",
@@ -1464,12 +1462,43 @@ const vueRules = {
   "vue/component-options-name-casing": ["error", "PascalCase"],
   "vue/component-tags-order": "off",
   // deprecated
+  "vue/custom-event-name-casing": [
+    "error",
+    "camelCase",
+    {
+      ignores: []
+    }
+  ],
+  "vue/define-emits-declaration": ["error", "type-literal"],
   "vue/define-macros-order": [
     "error",
     {
       order: ["defineOptions", "defineProps", "defineEmits", "defineSlots"]
     }
   ],
+  "vue/define-props-declaration": ["error", "type-based"],
+  "vue/dot-notation": "off",
+  // builtin extension - checked by typescript?
+  "vue/enforce-style-attribute": [
+    "off",
+    { allow: ["scoped", "module", "plain"] }
+  ],
+  "vue/eqeqeq": [
+    "error",
+    "always",
+    {
+      null: "ignore"
+    }
+  ],
+  // builtin extension
+  "vue/html-button-has-type": [
+    "error",
+    {
+      button: true,
+      submit: true,
+      reset: true
+    }
+  ],
   "vue/match-component-file-name": [
     "error",
     {
@@ -1477,15 +1506,177 @@ const vueRules = {
       shouldMatchCase: false
     }
   ],
+  "vue/match-component-import-name": "error",
+  "vue/max-lines-per-block": [
+    "off",
+    {
+      script: 0,
+      template: 0,
+      style: 0,
+      skipBlankLines: true
+    }
+  ],
+  "vue/next-tick-style": ["error", "promise"],
+  "vue/no-bare-strings-in-template": ["off"],
+  "vue/no-boolean-default": ["off", "no-default"],
+  "vue/no-console": "error",
+  // builtin extension
+  "vue/no-constant-condition": "warn",
+  // builtin extension
+  "vue/no-custom-modifiers-on-v-model": "off",
+  // vue2?
+  "vue/no-deprecated-model-definition": "error",
+  "vue/no-duplicate-attr-inheritance": "error",
+  "vue/no-empty-component-block": "warn",
+  "vue/no-empty-pattern": "error",
+  // builtin extension
+  "vue/no-invalid-model-keys": "off",
+  // deprecated
+  "vue/no-irregular-whitespace": "error",
+  // builtin extension
+  "vue/no-loss-of-precision": "off",
+  // builtin extension - checked by typescript?
+  "vue/no-multiple-objects-in-class": "error",
+  "vue/no-multiple-template-root": "off",
+  // vue2 - better to have in vue3, too?
+  "vue/no-potential-component-option-typo": "off",
+  // options api
+  "vue/no-ref-object-destructure": "off",
+  // deprecated
+  "vue/no-ref-object-reactivity-loss": "warn",
+  "vue/no-required-prop-with-default": ["warn", { autofix: false }],
+  "vue/no-restricted-block": ["off"],
+  "vue/no-restricted-call-after-await": ["off"],
+  "vue/no-restricted-class": ["off"],
+  "vue/no-restricted-component-names": ["off"],
+  "vue/no-restricted-component-options": ["off"],
+  "vue/no-restricted-custom-event": ["off"],
+  "vue/no-restricted-html-elements": ["off"],
+  "vue/no-restricted-props": ["off"],
+  "vue/no-restricted-static-attribute": ["off"],
   "vue/no-restricted-syntax": [
     "error",
     "DebuggerStatement",
     "LabeledStatement",
     "WithStatement"
   ],
+  // builtin extension
+  "vue/no-restricted-v-bind": ["off"],
+  "vue/no-restricted-v-on": ["off"],
+  "vue/no-root-v-if": "error",
+  "vue/no-setup-props-destructure": "off",
+  // deprecated
+  "vue/no-setup-props-reactivity-loss": "error",
   "vue/no-sparse-arrays": "error",
+  // builtin extension
+  "vue/no-static-inline-styles": [
+    "error",
+    {
+      allowBinding: false
+    }
+  ],
+  "vue/no-template-target-blank": [
+    "error",
+    {
+      allowReferrer: false,
+      enforceDynamicLinks: "always"
+    }
+  ],
+  "vue/no-this-in-before-route-enter": "off",
+  // options api & vue-router
+  "vue/no-undef-components": "off",
+  "vue/no-undef-properties": "error",
+  "vue/no-unsupported-features": "off",
+  "vue/no-unused-emit-declarations": "error",
+  "vue/no-unused-properties": [
+    "error",
+    {
+      groups: ["props"],
+      deepData: false,
+      ignorePublicMembers: false,
+      unreferencedOptions: []
+    }
+  ],
   "vue/no-unused-refs": "error",
-  "vue/no-useless-v-bind": "error"
+  "vue/no-use-v-else-with-v-for": "error",
+  "vue/no-use-v-if-with-v-for": "error",
+  "vue/no-useless-concat": "error",
+  // builtin extension
+  "vue/no-useless-mustaches": [
+    "off",
+    {
+      ignoreIncludesComment: false,
+      ignoreStringEscape: true
+    }
+  ],
+  "vue/no-useless-v-bind": "error",
+  "vue/no-v-for-template-key": "off",
+  // vue2
+  "vue/no-v-for-template-key-on-child": "error",
+  "vue/no-v-model-argument": "off",
+  // vue2
+  "vue/no-v-text": "off",
+  "vue/object-shorthand": [
+    "error",
+    "always",
+    {
+      ignoreConstructors: false,
+      avoidQuotes: true
+    }
+  ],
+  // builtin extension
+  "vue/prefer-define-options": "error",
+  "vue/prefer-prop-type-boolean-first": "error",
+  "vue/prefer-separate-static-class": "error",
+  "vue/prefer-template": "error",
+  // builtin extension
+  "vue/prefer-true-attribute-shorthand": "off",
+  "vue/require-direct-export": [
+    "error",
+    {
+      disallowFunctionalComponentFunction: false
+    }
+  ],
+  "vue/require-explicit-slots": "error",
+  "vue/require-expose": "error",
+  "vue/require-emit-validator": "off",
+  // enable?
+  "vue/require-macro-variable-name": [
+    "error",
+    {
+      defineProps: "props",
+      defineEmits: "emits",
+      defineSlots: "slots",
+      useSlots: "slots",
+      useAttrs: "attrs"
+    }
+  ],
+  "vue/require-name-property": "error",
+  "vue/require-prop-comment": [
+    "warn",
+    {
+      type: "JSDoc"
+    }
+  ],
+  "vue/require-typed-object-prop": "error",
+  "vue/require-typed-ref": "error",
+  "vue/sort-keys": "off",
+  // builtin extension
+  "vue/script-setup-uses-vars": "off",
+  // deprecated
+  "vue/static-class-names-order": "error",
+  "vue/v-if-else-key": "off",
+  // vue2
+  "vue/v-on-handler-style": "off",
+  "vue/v-on-function-call": "off",
+  // deprecated
+  "vue/valid-define-emits": "error",
+  "vue/valid-define-options": "error",
+  "vue/valid-define-props": "error",
+  "vue/valid-model-definition": "off",
+  // vue2
+  "vue/valid-v-bind-sync": "off"
+  // vue2
 };
 
 const index$1 = {
@@ -1755,13 +1946,13 @@ const env = {
 };
 
 const meteorlxy = async (options, ...customConfigs) => [
-  // .eslintignore is not supported by flat config
+  // global ignores
   ...ignores(options.ignores),
   // javascript core rules
   ...javascript(options.javascript),
-  // react rules
+  // react rules - should be placed before typescript rules
   ...options.react ? await react(options.react === true ? {} : options.react) : [],
-  // typescript rules - should be placed after react rules
+  // typescript rules
   ...typescript(options.typescript),
   // vue rules - should be placed after typescript rules
   ...options.vue ? await vue(options.vue === true ? {} : options.vue) : [],