@alextheman/eslint-plugin 1.13.4 → 1.13.6

package/dist/index.cjs

@@ -3706,7 +3706,7 @@ module.exports = __toCommonJS(index_exports);
 
 // package.json
 var name = "@alextheman/eslint-plugin";
-var version = "1.13.4";
+var version = "1.13.6";
 
 // src/configs/alexPluginBase.ts
 function createAlexPluginBaseConfig(plugin) {
@@ -3724,9 +3724,9 @@ function createAlexPluginBaseConfig(plugin) {
     {
       files: ["**/*.test.ts"],
       rules: {
+        "@alextheman/consistent-test-function": ["error", { preference: "test" }],
         "@alextheman/no-isolated-tests": "error",
-        "@alextheman/no-skipped-tests": "warn",
-        "@alextheman/consistent-test-function": ["error", { preference: "test" }]
+        "@alextheman/no-skipped-tests": "warn"
       }
     }
   ];
@@ -3756,106 +3756,132 @@ var typeScriptBase = [
   import_eslint_config_prettier.default,
   import_eslint_plugin_package_json.default.configs.recommended,
   {
-    name: "@alextheman/eslint-config-typescript-base",
     files: ["**/*.ts", "**/*.tsx"],
+    ignores: ["dist"],
     languageOptions: {
+      globals: __spreadValues(__spreadValues({}, import_globals.default.node), import_globals.default.browser),
       parser: import_parser.default,
       parserOptions: {
         ecmaVersion: "latest",
         sourceType: "module"
-      },
-      globals: __spreadValues(__spreadValues({}, import_globals.default.node), import_globals.default.browser)
-    },
-    settings: {
-      "import/resolver": {
-        typescript: true,
-        node: true
       }
     },
-    ignores: ["dist"],
+    name: "@alextheman/eslint-config-typescript-base",
     plugins: {
       "@typescript-eslint": import_eslint_plugin.default,
       import: import_eslint_plugin_import.default,
-      prettier: import_eslint_plugin_prettier.default,
-      perfectionist: import_eslint_plugin_perfectionist.default
+      perfectionist: import_eslint_plugin_perfectionist.default,
+      prettier: import_eslint_plugin_prettier.default
     },
     rules: {
-      "import/no-unresolved": "error",
+      "@typescript-eslint/consistent-type-imports": "error",
+      "@typescript-eslint/no-unused-vars": [
+        "error",
+        {
+          argsIgnorePattern: "^_",
+          caughtErrorsIgnorePattern: "^_",
+          varsIgnorePattern: "^_"
+        }
+      ],
+      "arrow-body-style": ["error", "always"],
+      curly: ["error", "all"],
+      "dot-notation": "error",
       eqeqeq: "error",
+      "func-style": ["error", "declaration", { allowArrowFunctions: false }],
+      "import/no-unresolved": "error",
+      "no-cond-assign": "error",
       "no-console": ["error", { allow: ["warn", "error"] }],
-      "perfectionist/sort-imports": [
+      "no-else-return": "error",
+      "no-eval": "error",
+      "no-implicit-coercion": ["error", { allow: ["!!"] }],
+      "no-lonely-if": "error",
+      "no-new-wrappers": "error",
+      "no-param-reassign": "error",
+      "no-restricted-imports": [
         "error",
         {
-          type: "alphabetical",
-          order: "asc",
-          ignoreCase: true,
-          partitionByComment: false,
-          partitionByNewLine: false,
-          specialCharacters: "keep",
-          groups: ["type", "builtin", "external", "internal", "object"],
-          newlinesBetween: 1,
-          internalPattern: ["^src/.*"]
+          patterns: [
+            {
+              group: ["node_modules"],
+              message: "What on Earth are you doing? Leave poor node_modules alone!"
+            }
+          ]
         }
       ],
+      "no-undef": "error",
+      // Disable regular no-unused-vars rule since that will flag interface declarations. Only use the TypeScript specific rule for this.
+      "no-unused-vars": "off",
+      "no-useless-rename": "error",
+      "no-useless-return": "error",
+      "operator-assignment": ["error", "always"],
       "perfectionist/sort-exports": [
         "error",
         {
-          type: "alphabetical",
-          order: "asc",
+          customGroups: [],
           fallbackSort: { type: "natural" },
+          groups: ["value-export", "type-export"],
           ignoreCase: true,
-          specialCharacters: "keep",
+          newlinesBetween: 1,
+          order: "asc",
           partitionByComment: false,
           partitionByNewLine: false,
-          newlinesBetween: 1,
-          groups: ["value-export", "type-export"],
-          customGroups: []
+          specialCharacters: "keep",
+          type: "alphabetical"
         }
       ],
-      "@typescript-eslint/no-unused-vars": [
+      "perfectionist/sort-imports": [
         "error",
         {
-          argsIgnorePattern: "^_",
-          varsIgnorePattern: "^_",
-          caughtErrorsIgnorePattern: "^_"
+          groups: ["type", "builtin", "external", "internal", "object"],
+          ignoreCase: true,
+          internalPattern: ["^src/.*"],
+          newlinesBetween: 1,
+          order: "asc",
+          partitionByComment: false,
+          partitionByNewLine: false,
+          specialCharacters: "keep",
+          type: "alphabetical"
         }
       ],
-      // Disable regular no-unused-vars rule since that will flag interface declarations. Only use the TypeScript specific rule for this.
-      "no-unused-vars": "off",
-      "func-style": ["error", "declaration", { allowArrowFunctions: false }],
       "prefer-arrow-callback": ["error", { allowNamedFunctions: false }],
-      "arrow-body-style": ["error", "always"],
-      curly: ["error", "all"],
-      "no-param-reassign": "error",
-      "no-useless-rename": "error",
-      "sort-vars": "error",
-      "no-cond-assign": "error",
-      "no-undef": "error",
-      "@typescript-eslint/consistent-type-imports": "error",
+      "prefer-const": "error",
+      "prefer-destructuring": "error",
+      "prefer-template": "error",
       "prettier/prettier": ["warn", prettierRules_default],
-      "no-restricted-imports": [
-        "error",
-        {
-          patterns: [
-            {
-              group: ["node_modules"],
-              message: "What on Earth are you doing? Leave poor node_modules alone!"
-            }
-          ]
-        }
-      ]
+      "sort-vars": "error"
+    },
+    settings: {
+      "import/resolver": {
+        node: true,
+        typescript: true
+      }
     }
   },
   {
     files: ["**/*.test.ts"],
     rules: {
+      "no-restricted-globals": [
+        "error",
+        {
+          message: "Do not use global describe function. Import test functions from vitest instead.",
+          name: "describe"
+        },
+        {
+          message: "Do not use global test function. Import test functions from vitest instead.",
+          name: "test"
+        },
+        {
+          message: "Do not use global expect function. Import test functions from vitest instead.",
+          name: "expect"
+        }
+      ],
       "no-restricted-imports": [
         "error",
         {
           paths: [
             {
-              name: "node:test",
-              message: "Use test functions from vitest instead."
+              message: "Use test functions from vitest instead.",
+              name: "node:test"
             }
           ]
         }
@@ -3879,30 +3905,30 @@ var import_globals2 = __toESM(require_globals2(), 1);
 var typeScriptReactBase = [
   ...typeScriptBase_default,
   {
-    name: "@alextheman/eslint-config-typescript-react-base",
     languageOptions: {
       ecmaVersion: 2020,
       globals: import_globals2.default.browser
     },
+    name: "@alextheman/eslint-config-typescript-react-base",
     plugins: {
+      react: import_eslint_plugin_react.default,
       "react-hooks": import_eslint_plugin_react_hooks.default,
-      "react-refresh": import_eslint_plugin_react_refresh.default,
-      react: import_eslint_plugin_react.default
+      "react-refresh": import_eslint_plugin_react_refresh.default
     },
     rules: __spreadProps(__spreadValues({}, import_eslint_plugin_react_hooks.default.configs.recommended.rules), {
-      "react-refresh/only-export-components": "off",
-      "react-hooks/exhaustive-deps": "off",
       "no-restricted-imports": [
         "error",
         {
           paths: [
             {
-              regex: "^@mui/[^/]+$",
-              message: 'Please use `import Component from "@mui/[package]/Component"` instead. See https://mui.com/material-ui/guides/minimizing-bundle-size/ for more information.'
+              message: 'Please use `import Component from "@mui/[package]/Component"` instead. See https://mui.com/material-ui/guides/minimizing-bundle-size/ for more information.',
+              regex: "^@mui/[^/]+$"
             }
           ]
         }
-      ]
+      ],
+      "react-hooks/exhaustive-deps": "off",
+      "react-refresh/only-export-components": "off"
     })
   }
 ];
@@ -4187,7 +4213,6 @@ var noPluginConfigAccessFromSrcConfigs = createRule_default({
                 source: node.object
               }
             });
-            return;
           }
         }
       }

package/dist/index.d.cts

@@ -3,7 +3,7 @@ import z from 'zod';
 import { Config } from 'prettier';
 
 var name = "@alextheman/eslint-plugin";
-var version = "1.13.4";
+var version = "1.13.6";
 
 interface AlexPlugin {
     meta: {

package/dist/index.d.ts

@@ -3,7 +3,7 @@ import z from 'zod';
 import { Config } from 'prettier';
 
 var name = "@alextheman/eslint-plugin";
-var version = "1.13.4";
+var version = "1.13.6";
 
 interface AlexPlugin {
     meta: {

package/dist/index.js

@@ -3692,7 +3692,7 @@ var require_globals2 = __commonJS({
 
 // package.json
 var name = "@alextheman/eslint-plugin";
-var version = "1.13.4";
+var version = "1.13.6";
 
 // src/configs/alexPluginBase.ts
 function createAlexPluginBaseConfig(plugin) {
@@ -3710,9 +3710,9 @@ function createAlexPluginBaseConfig(plugin) {
     {
       files: ["**/*.test.ts"],
       rules: {
+        "@alextheman/consistent-test-function": ["error", { preference: "test" }],
         "@alextheman/no-isolated-tests": "error",
-        "@alextheman/no-skipped-tests": "warn",
-        "@alextheman/consistent-test-function": ["error", { preference: "test" }]
+        "@alextheman/no-skipped-tests": "warn"
       }
     }
   ];
@@ -3742,106 +3742,132 @@ var typeScriptBase = [
   prettierConfig,
   packageJson.configs.recommended,
   {
-    name: "@alextheman/eslint-config-typescript-base",
     files: ["**/*.ts", "**/*.tsx"],
+    ignores: ["dist"],
     languageOptions: {
+      globals: __spreadValues(__spreadValues({}, import_globals.default.node), import_globals.default.browser),
       parser: tsparser,
       parserOptions: {
         ecmaVersion: "latest",
         sourceType: "module"
-      },
-      globals: __spreadValues(__spreadValues({}, import_globals.default.node), import_globals.default.browser)
-    },
-    settings: {
-      "import/resolver": {
-        typescript: true,
-        node: true
       }
     },
-    ignores: ["dist"],
+    name: "@alextheman/eslint-config-typescript-base",
     plugins: {
       "@typescript-eslint": eslintPlugin,
       import: importPlugin,
-      prettier: prettierPlugin,
-      perfectionist
+      perfectionist,
+      prettier: prettierPlugin
     },
     rules: {
-      "import/no-unresolved": "error",
+      "@typescript-eslint/consistent-type-imports": "error",
+      "@typescript-eslint/no-unused-vars": [
+        "error",
+        {
+          argsIgnorePattern: "^_",
+          caughtErrorsIgnorePattern: "^_",
+          varsIgnorePattern: "^_"
+        }
+      ],
+      "arrow-body-style": ["error", "always"],
+      curly: ["error", "all"],
+      "dot-notation": "error",
       eqeqeq: "error",
+      "func-style": ["error", "declaration", { allowArrowFunctions: false }],
+      "import/no-unresolved": "error",
+      "no-cond-assign": "error",
       "no-console": ["error", { allow: ["warn", "error"] }],
-      "perfectionist/sort-imports": [
+      "no-else-return": "error",
+      "no-eval": "error",
+      "no-implicit-coercion": ["error", { allow: ["!!"] }],
+      "no-lonely-if": "error",
+      "no-new-wrappers": "error",
+      "no-param-reassign": "error",
+      "no-restricted-imports": [
         "error",
         {
-          type: "alphabetical",
-          order: "asc",
-          ignoreCase: true,
-          partitionByComment: false,
-          partitionByNewLine: false,
-          specialCharacters: "keep",
-          groups: ["type", "builtin", "external", "internal", "object"],
-          newlinesBetween: 1,
-          internalPattern: ["^src/.*"]
+          patterns: [
+            {
+              group: ["node_modules"],
+              message: "What on Earth are you doing? Leave poor node_modules alone!"
+            }
+          ]
         }
       ],
+      "no-undef": "error",
+      // Disable regular no-unused-vars rule since that will flag interface declarations. Only use the TypeScript specific rule for this.
+      "no-unused-vars": "off",
+      "no-useless-rename": "error",
+      "no-useless-return": "error",
+      "operator-assignment": ["error", "always"],
       "perfectionist/sort-exports": [
         "error",
         {
-          type: "alphabetical",
-          order: "asc",
+          customGroups: [],
           fallbackSort: { type: "natural" },
+          groups: ["value-export", "type-export"],
           ignoreCase: true,
-          specialCharacters: "keep",
+          newlinesBetween: 1,
+          order: "asc",
           partitionByComment: false,
           partitionByNewLine: false,
-          newlinesBetween: 1,
-          groups: ["value-export", "type-export"],
-          customGroups: []
+          specialCharacters: "keep",
+          type: "alphabetical"
         }
       ],
-      "@typescript-eslint/no-unused-vars": [
+      "perfectionist/sort-imports": [
         "error",
         {
-          argsIgnorePattern: "^_",
-          varsIgnorePattern: "^_",
-          caughtErrorsIgnorePattern: "^_"
+          groups: ["type", "builtin", "external", "internal", "object"],
+          ignoreCase: true,
+          internalPattern: ["^src/.*"],
+          newlinesBetween: 1,
+          order: "asc",
+          partitionByComment: false,
+          partitionByNewLine: false,
+          specialCharacters: "keep",
+          type: "alphabetical"
         }
       ],
-      // Disable regular no-unused-vars rule since that will flag interface declarations. Only use the TypeScript specific rule for this.
-      "no-unused-vars": "off",
-      "func-style": ["error", "declaration", { allowArrowFunctions: false }],
       "prefer-arrow-callback": ["error", { allowNamedFunctions: false }],
-      "arrow-body-style": ["error", "always"],
-      curly: ["error", "all"],
-      "no-param-reassign": "error",
-      "no-useless-rename": "error",
-      "sort-vars": "error",
-      "no-cond-assign": "error",
-      "no-undef": "error",
-      "@typescript-eslint/consistent-type-imports": "error",
+      "prefer-const": "error",
+      "prefer-destructuring": "error",
+      "prefer-template": "error",
       "prettier/prettier": ["warn", prettierRules_default],
-      "no-restricted-imports": [
-        "error",
-        {
-          patterns: [
-            {
-              group: ["node_modules"],
-              message: "What on Earth are you doing? Leave poor node_modules alone!"
-            }
-          ]
-        }
-      ]
+      "sort-vars": "error"
+    },
+    settings: {
+      "import/resolver": {
+        node: true,
+        typescript: true
+      }
     }
   },
   {
     files: ["**/*.test.ts"],
     rules: {
+      "no-restricted-globals": [
+        "error",
+        {
+          message: "Do not use global describe function. Import test functions from vitest instead.",
+          name: "describe"
+        },
+        {
+          message: "Do not use global test function. Import test functions from vitest instead.",
+          name: "test"
+        },
+        {
+          message: "Do not use global expect function. Import test functions from vitest instead.",
+          name: "expect"
+        }
+      ],
       "no-restricted-imports": [
         "error",
         {
           paths: [
             {
-              name: "node:test",
-              message: "Use test functions from vitest instead."
+              message: "Use test functions from vitest instead.",
+              name: "node:test"
             }
           ]
         }
@@ -3865,30 +3891,30 @@ import reactRefresh from "eslint-plugin-react-refresh";
 var typeScriptReactBase = [
   ...typeScriptBase_default,
   {
-    name: "@alextheman/eslint-config-typescript-react-base",
     languageOptions: {
       ecmaVersion: 2020,
       globals: import_globals2.default.browser
     },
+    name: "@alextheman/eslint-config-typescript-react-base",
     plugins: {
+      react: reactPlugin,
       "react-hooks": reactHooks,
-      "react-refresh": reactRefresh,
-      react: reactPlugin
+      "react-refresh": reactRefresh
     },
     rules: __spreadProps(__spreadValues({}, reactHooks.configs.recommended.rules), {
-      "react-refresh/only-export-components": "off",
-      "react-hooks/exhaustive-deps": "off",
       "no-restricted-imports": [
         "error",
         {
           paths: [
             {
-              regex: "^@mui/[^/]+$",
-              message: 'Please use `import Component from "@mui/[package]/Component"` instead. See https://mui.com/material-ui/guides/minimizing-bundle-size/ for more information.'
+              message: 'Please use `import Component from "@mui/[package]/Component"` instead. See https://mui.com/material-ui/guides/minimizing-bundle-size/ for more information.',
+              regex: "^@mui/[^/]+$"
             }
           ]
         }
-      ]
+      ],
+      "react-hooks/exhaustive-deps": "off",
+      "react-refresh/only-export-components": "off"
     })
   }
 ];
@@ -4173,7 +4199,6 @@ var noPluginConfigAccessFromSrcConfigs = createRule_default({
                 source: node.object
               }
             });
-            return;
           }
         }
       }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@alextheman/eslint-plugin",
-  "version": "1.13.4",
+  "version": "1.13.6",
   "description": "A package to provide custom ESLint rules and configs",
   "license": "ISC",
   "author": "alextheman",