@tb-dev/eslint-config 4.4.1 → 5.0.0

package/LICENSE

@@ -1,21 +1,21 @@
-MIT License
-
-Copyright (c) 2023 Andrew Ferreira
-
-Permission is hereby granted, free of charge, to any person obtaining a copy
-of this software and associated documentation files (the "Software"), to deal
-in the Software without restriction, including without limitation the rights
-to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
-copies of the Software, and to permit persons to whom the Software is
-furnished to do so, subject to the following conditions:
-
-The above copyright notice and this permission notice shall be included in all
-copies or substantial portions of the Software.
-
-THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
-IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
-FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
-AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
-LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
-OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
-SOFTWARE.
+MIT License
+
+Copyright (c) 2023 Andrew Ferreira
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.

package/dist/index.cjs

@@ -10,7 +10,6 @@ var Glob = /* @__PURE__ */ ((Glob2) => {
   Glob2["Javascript"] = "**/*.?([cm])js?(x)";
   Glob2["Json"] = "**/*.json?([c5])";
   Glob2["Typescript"] = "**/*.?([cm])ts?(x)";
-  Glob2["Vitest"] = "**/*.{test,spec}.[jt]s";
   Glob2["Vue"] = "**/*.vue";
   return Glob2;
 })(Glob || {});
@@ -482,49 +481,6 @@ async function jsonc(options) {
   ];
 }
 
-async function vitest(options) {
-  if (!options.features?.vitest) return {};
-  const plugin = await interopDefault(import('eslint-plugin-vitest'));
-  return {
-    plugins: { vitest: plugin },
-    files: [Glob.Vitest],
-    rules: {
-      "vitest/consistent-test-it": ["error", { fn: "it", withinDescribe: "it" }],
-      "vitest/expect-expect": ["error", { assertFunctionNames: ["expect"] }],
-      "vitest/max-expects": ["error", { max: 5 }],
-      "vitest/max-nested-describe": ["error", { max: 1 }],
-      "vitest/no-alias-methods": "error",
-      "vitest/no-commented-out-tests": "error",
-      "vitest/no-conditional-expect": "error",
-      "vitest/no-conditional-in-test": "error",
-      "vitest/no-conditional-tests": "error",
-      "vitest/no-done-callback": "error",
-      "vitest/no-duplicate-hooks": "error",
-      "vitest/no-identical-title": "error",
-      "vitest/no-import-node-test": "error",
-      "vitest/no-standalone-expect": "error",
-      "vitest/no-test-return-statement": "error",
-      "vitest/prefer-comparison-matcher": "error",
-      "vitest/prefer-each": "error",
-      "vitest/prefer-equality-matcher": "error",
-      "vitest/prefer-expect-resolves": "error",
-      "vitest/prefer-hooks-in-order": "error",
-      "vitest/prefer-hooks-on-top": "error",
-      "vitest/prefer-lowercase-title": "error",
-      "vitest/prefer-mock-promise-shorthand": "error",
-      "vitest/prefer-spy-on": "error",
-      "vitest/prefer-to-be-object": "error",
-      "vitest/prefer-to-contain": "error",
-      "vitest/prefer-to-have-length": "error",
-      "vitest/prefer-todo": "error",
-      "vitest/require-top-level-describe": ["error", { maxNumberOfTopLevelDescribes: 10 }],
-      "vitest/valid-describe-callback": "error",
-      "vitest/valid-expect": "error",
-      ...options.overrides?.vitest
-    }
-  };
-}
-
 async function unicorn(options) {
   const { unicorn: enabled = true } = options.features ?? {};
   if (!enabled) return {};
@@ -544,6 +500,7 @@ async function unicorn(options) {
       "unicorn/no-await-in-promise-methods": "error",
       "unicorn/no-invalid-fetch-options": "error",
       "unicorn/no-invalid-remove-event-listener": "error",
+      "unicorn/no-length-as-slice-end": "error",
       "unicorn/no-magic-array-flat-depth": "error",
       "unicorn/no-negation-in-equality-check": "error",
       "unicorn/no-single-promise-in-promise-methods": "error",
@@ -579,10 +536,11 @@ async function unicorn(options) {
 async function tailwind(options) {
   if (!options.features?.tailwind) return {};
   const plugin = await interopDefault(import('eslint-plugin-tailwindcss'));
+  const callees = ["classnames", "clsx", "cn", "ctl", "cva", "tv"];
   const rules = {
-    "tailwindcss/classnames-order": "error",
-    "tailwindcss/enforces-shorthand": "error",
-    "tailwindcss/no-contradicting-classname": "error",
+    "tailwindcss/classnames-order": ["error", { callees }],
+    "tailwindcss/enforces-shorthand": ["error", { callees }],
+    "tailwindcss/no-contradicting-classname": ["error", { callees }],
     ...options.overrides?.tailwind
   };
   return {
@@ -662,6 +620,7 @@ function javascript(options) {
       "no-labels": ["error", { allowLoop: false, allowSwitch: false }],
       "no-lone-blocks": "error",
       "no-lonely-if": "error",
+      "no-loss-of-precision": "error",
       "no-misleading-character-class": "error",
       "no-multi-assign": "error",
       "no-multi-str": "error",
@@ -813,7 +772,6 @@ async function typescript(options) {
     "@typescript-eslint/no-dynamic-delete": "error",
     "no-empty-function": "off",
     "@typescript-eslint/no-empty-function": "error",
-    "@typescript-eslint/no-empty-interface": "error",
     "@typescript-eslint/no-empty-object-type": ["error", {
       allowInterfaces: "never",
       allowObjectTypes: "never"
@@ -822,9 +780,10 @@ async function typescript(options) {
     "@typescript-eslint/no-extra-non-null-assertion": "error",
     "@typescript-eslint/no-extraneous-class": "error",
     "@typescript-eslint/no-floating-promises": ["error", {
+      allowForKnownSafePromises: options.knownSafePromises ?? [],
+      checkThenables: false,
       ignoreIIFE: true,
-      ignoreVoid: true,
-      allowForKnownSafePromises: options.knownSafePromises ?? []
+      ignoreVoid: true
     }],
     "@typescript-eslint/no-for-in-array": "error",
     "no-implied-eval": "off",
@@ -837,8 +796,6 @@ async function typescript(options) {
     }],
     "no-loop-func": "off",
     "@typescript-eslint/no-loop-func": "error",
-    "no-loss-of-precision": "off",
-    "@typescript-eslint/no-loss-of-precision": "error",
     "@typescript-eslint/no-meaningless-void-operator": "error",
     "@typescript-eslint/no-misused-new": "error",
     "@typescript-eslint/no-misused-promises": ["error", {
@@ -893,7 +850,6 @@ async function typescript(options) {
     "no-useless-constructor": "off",
     "@typescript-eslint/no-useless-constructor": "error",
     "@typescript-eslint/no-useless-empty-export": "error",
-    "@typescript-eslint/no-useless-template-literals": "error",
     "@typescript-eslint/no-wrapper-object-types": "error",
     "@typescript-eslint/non-nullable-type-assertion-style": "error",
     "no-throw-literal": "off",
@@ -1039,7 +995,6 @@ async function defineConfig(options) {
     stylistic(options),
     perfectionist(options),
     unicorn(options),
-    vitest(options),
     tailwind(options),
     ignores
   ]);

package/dist/index.d.ts

@@ -27,8 +27,6 @@ declare interface ConfigOptions {
         /** @default true */
         unicorn?: boolean;
         /** @default false */
-        vitest?: boolean;
-        /** @default false */
         vue?: boolean;
     };
     /** `tsconfig.json` files for TypeScript. */
@@ -42,7 +40,6 @@ declare interface ConfigOptions {
         tailwind?: Rules;
         typescript?: Rules;
         unicorn?: Rules;
-        vitest?: Rules;
         vue?: Rules;
     };
     /** @see https://typescript-eslint.io/rules/no-floating-promises#allowforknownsafepromises */

package/dist/index.js

@@ -6,7 +6,6 @@ var Glob = /* @__PURE__ */ ((Glob2) => {
   Glob2["Javascript"] = "**/*.?([cm])js?(x)";
   Glob2["Json"] = "**/*.json?([c5])";
   Glob2["Typescript"] = "**/*.?([cm])ts?(x)";
-  Glob2["Vitest"] = "**/*.{test,spec}.[jt]s";
   Glob2["Vue"] = "**/*.vue";
   return Glob2;
 })(Glob || {});
@@ -478,49 +477,6 @@ async function jsonc(options) {
   ];
 }
 
-async function vitest(options) {
-  if (!options.features?.vitest) return {};
-  const plugin = await interopDefault(import('eslint-plugin-vitest'));
-  return {
-    plugins: { vitest: plugin },
-    files: [Glob.Vitest],
-    rules: {
-      "vitest/consistent-test-it": ["error", { fn: "it", withinDescribe: "it" }],
-      "vitest/expect-expect": ["error", { assertFunctionNames: ["expect"] }],
-      "vitest/max-expects": ["error", { max: 5 }],
-      "vitest/max-nested-describe": ["error", { max: 1 }],
-      "vitest/no-alias-methods": "error",
-      "vitest/no-commented-out-tests": "error",
-      "vitest/no-conditional-expect": "error",
-      "vitest/no-conditional-in-test": "error",
-      "vitest/no-conditional-tests": "error",
-      "vitest/no-done-callback": "error",
-      "vitest/no-duplicate-hooks": "error",
-      "vitest/no-identical-title": "error",
-      "vitest/no-import-node-test": "error",
-      "vitest/no-standalone-expect": "error",
-      "vitest/no-test-return-statement": "error",
-      "vitest/prefer-comparison-matcher": "error",
-      "vitest/prefer-each": "error",
-      "vitest/prefer-equality-matcher": "error",
-      "vitest/prefer-expect-resolves": "error",
-      "vitest/prefer-hooks-in-order": "error",
-      "vitest/prefer-hooks-on-top": "error",
-      "vitest/prefer-lowercase-title": "error",
-      "vitest/prefer-mock-promise-shorthand": "error",
-      "vitest/prefer-spy-on": "error",
-      "vitest/prefer-to-be-object": "error",
-      "vitest/prefer-to-contain": "error",
-      "vitest/prefer-to-have-length": "error",
-      "vitest/prefer-todo": "error",
-      "vitest/require-top-level-describe": ["error", { maxNumberOfTopLevelDescribes: 10 }],
-      "vitest/valid-describe-callback": "error",
-      "vitest/valid-expect": "error",
-      ...options.overrides?.vitest
-    }
-  };
-}
-
 async function unicorn(options) {
   const { unicorn: enabled = true } = options.features ?? {};
   if (!enabled) return {};
@@ -540,6 +496,7 @@ async function unicorn(options) {
       "unicorn/no-await-in-promise-methods": "error",
       "unicorn/no-invalid-fetch-options": "error",
       "unicorn/no-invalid-remove-event-listener": "error",
+      "unicorn/no-length-as-slice-end": "error",
       "unicorn/no-magic-array-flat-depth": "error",
       "unicorn/no-negation-in-equality-check": "error",
       "unicorn/no-single-promise-in-promise-methods": "error",
@@ -575,10 +532,11 @@ async function unicorn(options) {
 async function tailwind(options) {
   if (!options.features?.tailwind) return {};
   const plugin = await interopDefault(import('eslint-plugin-tailwindcss'));
+  const callees = ["classnames", "clsx", "cn", "ctl", "cva", "tv"];
   const rules = {
-    "tailwindcss/classnames-order": "error",
-    "tailwindcss/enforces-shorthand": "error",
-    "tailwindcss/no-contradicting-classname": "error",
+    "tailwindcss/classnames-order": ["error", { callees }],
+    "tailwindcss/enforces-shorthand": ["error", { callees }],
+    "tailwindcss/no-contradicting-classname": ["error", { callees }],
     ...options.overrides?.tailwind
   };
   return {
@@ -658,6 +616,7 @@ function javascript(options) {
       "no-labels": ["error", { allowLoop: false, allowSwitch: false }],
       "no-lone-blocks": "error",
       "no-lonely-if": "error",
+      "no-loss-of-precision": "error",
       "no-misleading-character-class": "error",
       "no-multi-assign": "error",
       "no-multi-str": "error",
@@ -809,7 +768,6 @@ async function typescript(options) {
     "@typescript-eslint/no-dynamic-delete": "error",
     "no-empty-function": "off",
     "@typescript-eslint/no-empty-function": "error",
-    "@typescript-eslint/no-empty-interface": "error",
     "@typescript-eslint/no-empty-object-type": ["error", {
       allowInterfaces: "never",
       allowObjectTypes: "never"
@@ -818,9 +776,10 @@ async function typescript(options) {
     "@typescript-eslint/no-extra-non-null-assertion": "error",
     "@typescript-eslint/no-extraneous-class": "error",
     "@typescript-eslint/no-floating-promises": ["error", {
+      allowForKnownSafePromises: options.knownSafePromises ?? [],
+      checkThenables: false,
       ignoreIIFE: true,
-      ignoreVoid: true,
-      allowForKnownSafePromises: options.knownSafePromises ?? []
+      ignoreVoid: true
     }],
     "@typescript-eslint/no-for-in-array": "error",
     "no-implied-eval": "off",
@@ -833,8 +792,6 @@ async function typescript(options) {
     }],
     "no-loop-func": "off",
     "@typescript-eslint/no-loop-func": "error",
-    "no-loss-of-precision": "off",
-    "@typescript-eslint/no-loss-of-precision": "error",
     "@typescript-eslint/no-meaningless-void-operator": "error",
     "@typescript-eslint/no-misused-new": "error",
     "@typescript-eslint/no-misused-promises": ["error", {
@@ -889,7 +846,6 @@ async function typescript(options) {
     "no-useless-constructor": "off",
     "@typescript-eslint/no-useless-constructor": "error",
     "@typescript-eslint/no-useless-empty-export": "error",
-    "@typescript-eslint/no-useless-template-literals": "error",
     "@typescript-eslint/no-wrapper-object-types": "error",
     "@typescript-eslint/non-nullable-type-assertion-style": "error",
     "no-throw-literal": "off",
@@ -1035,7 +991,6 @@ async function defineConfig(options) {
     stylistic(options),
     perfectionist(options),
     unicorn(options),
-    vitest(options),
     tailwind(options),
     ignores
   ]);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tb-dev/eslint-config",
-  "version": "4.4.1",
+  "version": "5.0.0",
   "description": "ESLint config",
   "license": "MIT",
   "type": "module",
@@ -24,32 +24,31 @@
     "*.{?(c|m)@(j|t)s,css,vue,md,json}": "eslint --config eslint.config.js --fix"
   },
   "dependencies": {
-    "@stylistic/eslint-plugin": "^2.3.0",
-    "@typescript-eslint/eslint-plugin": "^7.17.0",
-    "@typescript-eslint/parser": "^7.17.0",
+    "@stylistic/eslint-plugin": "^2.6.1",
+    "@typescript-eslint/eslint-plugin": "^8.0.1",
+    "@typescript-eslint/parser": "^8.0.1",
     "eslint-plugin-jsonc": "^2.16.0",
-    "eslint-plugin-perfectionist": "^3.0.0",
+    "eslint-plugin-perfectionist": "^3.1.2",
     "eslint-plugin-tailwindcss": "^3.17.4",
-    "eslint-plugin-unicorn": "^54.0.0",
-    "eslint-plugin-vitest": "^0.5.4",
+    "eslint-plugin-unicorn": "^55.0.0",
     "eslint-plugin-vue": "^9.27.0",
-    "globals": "^15.8.0",
+    "globals": "^15.9.0",
     "vue-eslint-parser": "^9.4.3"
   },
   "devDependencies": {
-    "@types/node": "^20.14.11",
-    "eslint": "^8.57.0",
-    "husky": "^9.1.1",
-    "lint-staged": "^15.2.7",
+    "@types/node": "^22.1.0",
+    "eslint": "^9.8.0",
+    "husky": "^9.1.4",
+    "lint-staged": "^15.2.8",
     "prettier": "^3.3.3",
     "tslib": "^2.6.3",
     "typescript": "^5.5.4",
-    "vite": "^5.3.4",
-    "vite-plugin-dts": "^3.9.1"
+    "vite": "^5.3.5",
+    "vite-plugin-dts": "3.9.1"
   },
   "peerDependencies": {
-    "eslint": "^8.57.0",
-    "typescript": "^5.4.0"
+    "eslint": "^9.8.0",
+    "typescript": "^5.5.0"
   },
   "engines": {
     "node": ">=22"