@prowise/eslint-config-linting-rules 2.0.3 → 2.0.5

package/README.md

@@ -42,3 +42,7 @@ It is best NOT to use paths redirects.
 #### Installing a new version
 When trying to update the installed lining version by `npm i -D "@prowise/eslint-config-linting-rules@latest` you might get an error on a peer dependency.
 In this case you can uninstall the linter by `npm uninstall "@prowise/eslint-config-linting-rules` and reinstall it again with `npm i -D "@prowise/eslint-config-linting-rules@latest`
+
+## Version 2
+This version contains several breaking changes. This version is only tested on a backend project.
+For the moment: DO NOT USE this version on a frontend project! You best use the latest version 1.2.13

package/front-react.js

@@ -1,9 +1,11 @@
 import globals from "globals";
+import base from "./base.js";
 import react from "./rules/plugin-react-rules.js";
+import typescript from "./rules/plugin-typescript-rules.js";
 import vitest from "./rules/plugin-vitest-rules.js";
-import typescript from "./typescript.js";
 
 export default [
+    base,
     typescript,
     react,
     vitest,

package/front-solid.js

@@ -1,9 +1,11 @@
 import globals from "globals";
+import base from "./base.js";
 import solid from "./rules/plugin-solid-rules.js";
+import typescript from "./rules/plugin-typescript-rules.js";
 import vitest from "./rules/plugin-vitest-rules.js";
-import typescript from "./typescript.js";
 
 export default [
+    base,
     typescript,
     solid,
     vitest,

package/package.json

@@ -1 +1 @@
-{"name":"@prowise/eslint-config-linting-rules","version":"2.0.3","description":"Default Prowise linting rules for npm packages.","main":"index.js","type":"module","exports":{"./typescript":"./typescript.js","./backend":"./backend.js","./front-react":"./front-react.js","./front-solid":"./front-solid.js"},"scripts":{"lint":"eslint","inspect":"eslint --inspect-config"},"repository":{"type":"git","url":"git+ssh://git@bitbucket.org/prowise/npm-linting-rules.git"},"author":"Prowise B.V.","license":"UNLICENSED","homepage":"https://bitbucket.org/prowise/npm-linting-rules#readme","dependencies":{"@stylistic/eslint-plugin":"^2.8.0","confusing-browser-globals":"^1.0.11","eslint":"^9.11.1","eslint-import-resolver-typescript":"^3.6.3","eslint-plugin-import-x":"^4.3.0","eslint-plugin-jest":"^28.8.3","eslint-plugin-jsdoc":"^50.2.4","eslint-plugin-jsx-a11y":"^6.10.0","eslint-plugin-n":"^17.10.3","eslint-plugin-prefer-arrow":"^1.2.3","eslint-plugin-react":"^7.36.1","eslint-plugin-solid":"^0.14.3","eslint-plugin-vitest":"^0.5.4","globals":"^15.9.0","typescript-eslint":"^8.7.0"},"devDependencies":{"@eslint/config-inspector":"^0.5.4"}}
+{"name":"@prowise/eslint-config-linting-rules","version":"2.0.5","description":"Default Prowise linting rules for npm packages.","main":"index.js","type":"module","exports":{"./typescript":"./typescript.js","./backend":"./backend.js","./front-react":"./front-react.js","./front-solid":"./front-solid.js"},"scripts":{"lint":"eslint","inspect":"eslint --inspect-config"},"repository":{"type":"git","url":"git+ssh://git@bitbucket.org/prowise/npm-linting-rules.git"},"author":"Prowise B.V.","license":"UNLICENSED","homepage":"https://bitbucket.org/prowise/npm-linting-rules#readme","dependencies":{"@stylistic/eslint-plugin":"^2.11.0","confusing-browser-globals":"^1.0.11","eslint":"^9.16.0","eslint-import-resolver-typescript":"^3.7.0","eslint-plugin-import-x":"^4.5.0","eslint-plugin-jest":"^28.9.0","eslint-plugin-jsdoc":"^50.6.0","eslint-plugin-jsx-a11y":"^6.10.2","eslint-plugin-n":"^17.14.0","eslint-plugin-prefer-arrow-functions":"^3.4.1","eslint-plugin-react":"^7.37.2","eslint-plugin-solid":"^0.14.4","eslint-plugin-vitest":"^0.5.4","globals":"^15.13.0","typescript-eslint":"^8.17.0"},"devDependencies":{"@eslint/config-inspector":"^0.5.6"}}

package/rules/baseSettings.js

@@ -46,7 +46,7 @@ export default {
          * Disallow await inside of loops
          * https://eslint.org/docs/latest/rules/no-await-in-loop.
          */
-        "no-await-in-loop": "off",
+        // "no-await-in-loop": "off",
 
         /**
          * Disallow modifying variables of class declarations ✅
@@ -154,7 +154,8 @@ export default {
          * Disallow empty destructuring patterns ✅
          * https://eslint.org/docs/latest/rules/no-empty-pattern.
          */
-        "no-empty-static-block": "off",
+        // "no-empty-static-block": "off",
+
         /**
          * Disallow assigning to the exception in a catch block ✅
          * https://eslint.org/docs/latest/rules/no-ex-assign.
@@ -165,7 +166,7 @@ export default {
          * Disallow fallthrough of case statements ✅
          * https://eslint.org/docs/latest/rules/no-fallthrough.
          */
-        "no-fallthrough": "off",
+        // "no-fallthrough": "off",
 
         /**
          * Disallow overwriting functions written as function declarations ✅
@@ -285,7 +286,7 @@ export default {
          * Disallow unmodified conditions of loops
          * https://eslint.org/docs/latest/rules/no-unmodified-loop-condition.
          */
-        "no-unmodified-loop-condition": "off",
+        // "no-unmodified-loop-condition": "off",
 
         /**
          * Disallow unreachable statements after a return, throw, continue, or break statement ✅
@@ -346,7 +347,7 @@ export default {
          * note: not enabled because it is very buggy
          * https://eslint.org/docs/latest/rules/require-atomic-updates.
          */
-        "require-atomic-updates": "off",
+        // "require-atomic-updates": "off",
 
         /**
          * Disallow comparisons with the value NaN ✅
@@ -366,7 +367,7 @@ export default {
          * Enforces getter/setter pairs in objects
          * https://eslint.org/docs/latest/rules/accessor-pairs.
          */
-        "accessor-pairs": "off",
+        // "accessor-pairs": "off",
 
         /**
          * Enforces no braces where they can be omitted 🔧
@@ -390,56 +391,56 @@ export default {
          * Enforce or disallow capitalization of the first letter of a comment 🔧
          * https://eslint.org/docs/latest/rules/capitalized-comments.
          */
-        "capitalized-comments": [
-            "off",
-            "never",
-            {
-                line: {
-                    ignorePattern: ".*",
-                    ignoreInlineComments: true,
-                    ignoreConsecutiveComments: true,
-                },
-                block: {
-                    ignorePattern: ".*",
-                    ignoreInlineComments: true,
-                    ignoreConsecutiveComments: true,
-                },
-            },
-        ],
+        // "capitalized-comments": [
+        //     "off",
+        //     "never",
+        //     {
+        //         line: {
+        //             ignorePattern: ".*",
+        //             ignoreInlineComments: true,
+        //             ignoreConsecutiveComments: true,
+        //         },
+        //         block: {
+        //             ignorePattern: ".*",
+        //             ignoreInlineComments: true,
+        //             ignoreConsecutiveComments: true,
+        //         },
+        //     },
+        // ],
 
         /**
          * Enforce that class methods use "this"
          * https://eslint.org/docs/latest/rules/class-methods-use-this.
          */
-        "class-methods-use-this": [
-            "off",
-            {
-                exceptMethods: [
-                    "render",
-                    "getInitialState",
-                    "getDefaultProps",
-                    "getChildContext",
-                    "componentWillMount",
-                    "UNSAFE_componentWillMount",
-                    "componentDidMount",
-                    "componentWillReceiveProps",
-                    "UNSAFE_componentWillReceiveProps",
-                    "shouldComponentUpdate",
-                    "componentWillUpdate",
-                    "UNSAFE_componentWillUpdate",
-                    "componentDidUpdate",
-                    "componentWillUnmount",
-                    "componentDidCatch",
-                    "getSnapshotBeforeUpdate",
-                ],
-            },
-        ],
+        // "class-methods-use-this": [
+        //     "off",
+        //     {
+        //         exceptMethods: [
+        //             "render",
+        //             "getInitialState",
+        //             "getDefaultProps",
+        //             "getChildContext",
+        //             "componentWillMount",
+        //             "UNSAFE_componentWillMount",
+        //             "componentDidMount",
+        //             "componentWillReceiveProps",
+        //             "UNSAFE_componentWillReceiveProps",
+        //             "shouldComponentUpdate",
+        //             "componentWillUpdate",
+        //             "UNSAFE_componentWillUpdate",
+        //             "componentDidUpdate",
+        //             "componentWillUnmount",
+        //             "componentDidCatch",
+        //             "getSnapshotBeforeUpdate",
+        //         ],
+        //     },
+        // ],
 
         /**
          * Specify the maximum cyclomatic complexity allowed in a program
          * https://eslint.org/docs/latest/rules/complexity.
          */
-        "complexity": ["off", 20],
+        // "complexity": ["off", 20],
 
         /**
          * Require return statements to either always or never specify values
@@ -451,7 +452,7 @@ export default {
          * Enforces consistent naming when capturing the current execution context
          * https://eslint.org/docs/latest/rules/consistent-this.
          */
-        "consistent-this": "off",
+        // "consistent-this": "off",
 
         /**
          * Specify curly brace conventions for all control statements 🔧
@@ -493,17 +494,17 @@ export default {
          * Requires function names to match the name of the variable or property to which they are assigned
          * https://eslint.org/docs/latest/rules/func-name-matching.
          */
-        "func-name-matching": [
-            "off",
-            "always",
-            { includeCommonJSModuleExports: false, considerPropertyDescriptor: true },
-        ],
+        // "func-name-matching": [
+        //     "off",
+        //     "always",
+        //     { includeCommonJSModuleExports: false, considerPropertyDescriptor: true },
+        // ],
 
         /**
          * Require function expressions to have a name
          * https://eslint.org/docs/latest/rules/func-names.
          */
-        "func-names": "off",
+        // "func-names": "off",
 
         /**
          * Enforces use of function declarations
@@ -522,7 +523,7 @@ export default {
          * Off: not really a thing anymore
          * https://eslint.org/docs/latest/rules/guard-for-in.
          */
-        "guard-for-in": "off",
+        // "guard-for-in": "off",
 
         /**
          * Disallow specified identifiers
@@ -536,7 +537,7 @@ export default {
          * (variable names, property names etc.)
          * https://eslint.org/docs/latest/rules/id-length.
          */
-        "id-length": "off",
+        // "id-length": "off",
 
         /**
          * Require identifiers to match the provided regular expression
@@ -548,57 +549,57 @@ export default {
          * Enforce or disallow variable initializations at definition
          * https://eslint.org/docs/latest/rules/init-declarations.
          */
-        "init-declarations": "off",
+        // "init-declarations": "off",
 
         /**
          * Enforce a maximum number of classes per file
          * https://eslint.org/docs/latest/rules/max-classes-per-file.
          */
-        "max-classes-per-file": "off",
+        // "max-classes-per-file": "off",
 
         /**
          * Specify the maximum depth that blocks can be nested
          * https://eslint.org/docs/latest/rules/max-depth.
          */
-        "max-depth": ["off", 4],
+        // "max-depth": ["off", 4],
 
         /**
          * Specify the max number of lines in a file
          * https://eslint.org/docs/latest/rules/max-lines.
          */
-        "max-lines": ["off", { max: 300, skipBlankLines: true, skipComments: true }],
+        // "max-lines": ["off", { max: 300, skipBlankLines: true, skipComments: true }],
 
         /**
          * Enforce a maximum function length
          * https://eslint.org/docs/latest/rules/max-lines-per-function.
          */
-        "max-lines-per-function": [
-            "off",
-            {
-                max: 50,
-                skipBlankLines: true,
-                skipComments: true,
-                IIFEs: true,
-            },
-        ],
+        // "max-lines-per-function": [
+        //     "off",
+        //     {
+        //         max: 50,
+        //         skipBlankLines: true,
+        //         skipComments: true,
+        //         IIFEs: true,
+        //     },
+        // ],
 
         /**
          * Specify the maximum depth callbacks can be nested
          * https://eslint.org/docs/latest/rules/max-nested-callbacks.
          */
-        "max-nested-callbacks": "off",
+        // "max-nested-callbacks": "off",
 
         /**
          * Limits the number of parameters that can be used in the function declaration.
          * Https://eslint.org/docs/latest/rules/max-params.
          */
-        "max-params": ["off", 3],
+        // "max-params": ["off", 3],
 
         /**
          * Specify the maximum number of statement allowed in a function
          * https://eslint.org/docs/latest/rules/max-statements.
          */
-        "max-statements": ["off", 10],
+        // "max-statements": ["off", 10],
 
         /**
          * Require a capital letter for constructors
@@ -648,13 +649,13 @@ export default {
          * Disallow use of console
          * https://eslint.org/docs/latest/rules/no-console.
          */
-        "no-console": "off",
+        // "no-console": "off",
 
         /**
          * Disallow use of the continue statement
          * https://eslint.org/docs/latest/rules/no-continue.
          */
-        "no-continue": "off",
+        // "no-continue": "off",
 
         /**
          * Disallow deletion of variables ✅
@@ -666,7 +667,7 @@ export default {
          * Disallow division operators explicitly at beginning of regular expression 🔧
          * https://eslint.org/docs/latest/rules/no-div-regex.
          */
-        "no-div-regex": "off",
+        // "no-div-regex": "off",
 
         /**
          * Disallow else after a return in an if 🔧
@@ -695,7 +696,7 @@ export default {
          * Disallow comparisons to null without a type-checking operator
          * https://eslint.org/docs/latest/rules/no-eq-null.
          */
-        "no-eq-null": "off",
+        // "no-eq-null": "off",
 
         /**
          * Disallow use of eval()
@@ -737,21 +738,21 @@ export default {
          * Disallow implicit type conversions 🔧
          * https://eslint.org/docs/latest/rules/no-implicit-coercion.
          */
-        "no-implicit-coercion": [
-            "off",
-            {
-                boolean: false,
-                number: true,
-                string: true,
-                allow: [],
-            },
-        ],
+        // "no-implicit-coercion": [
+        //     "off",
+        //     {
+        //         boolean: false,
+        //         number: true,
+        //         string: true,
+        //         allow: [],
+        //     },
+        // ],
 
         /**
          * Disallow var and named functions in global scope
          * https://eslint.org/docs/latest/rules/no-implicit-globals.
          */
-        "no-implicit-globals": "off",
+        // "no-implicit-globals": "off",
 
         /**
          * Disallow use of eval()-like methods
@@ -763,13 +764,13 @@ export default {
          * Disallow comments inline after code
          * https://eslint.org/docs/latest/rules/no-inline-comments.
          */
-        "no-inline-comments": "off",
+        // "no-inline-comments": "off",
 
         /**
          * Disallow this keywords outside of classes or class-like objects
          * https://eslint.org/docs/latest/rules/no-invalid-this.
          */
-        "no-invalid-this": "off",
+        // "no-invalid-this": "off",
 
         /**
          * Disallow usage of __iterator__ property
@@ -811,15 +812,15 @@ export default {
          * Disallow magic numbers
          * https://eslint.org/docs/latest/rules/no-magic-numbers.
          */
-        "no-magic-numbers": [
-            "off",
-            {
-                ignore: [],
-                ignoreArrayIndexes: true,
-                enforceConst: true,
-                detectObjects: false,
-            },
-        ],
+        // "no-magic-numbers": [
+        //     "off",
+        //     {
+        //         ignore: [],
+        //         ignoreArrayIndexes: true,
+        //         enforceConst: true,
+        //         detectObjects: false,
+        //     },
+        // ],
 
         /**
          * Disallow use of chained assignment expressions
@@ -837,13 +838,13 @@ export default {
          * Disallow negated conditions
          * https://eslint.org/docs/latest/rules/no-negated-condition.
          */
-        "no-negated-condition": "off",
+        // "no-negated-condition": "off",
 
         /**
          * Disallow nested ternary expressions
          * https://eslint.org/docs/latest/rules/no-nested-ternary.
          */
-        "no-nested-ternary": "off",
+        // "no-nested-ternary": "off",
 
         /**
          * Disallow use of new operator when not part of the assignment or comparison
@@ -897,7 +898,7 @@ export default {
          * Disallow use of unary operators, ++ and --
          * https://eslint.org/docs/latest/rules/no-plusplus.
          */
-        "no-plusplus": "off",
+        // "no-plusplus": "off",
 
         /**
          * Disallow usage of __proto__ property
@@ -951,7 +952,7 @@ export default {
          * Disallow specific imports
          * https://eslint.org/docs/latest/rules/no-restricted-imports.
          */
-        "no-restricted-imports": ["off", { paths: [], patterns: [] }],
+        // "no-restricted-imports": ["off", { paths: [], patterns: [] }],
 
         /**
          * Disallow certain object properties
@@ -1031,7 +1032,7 @@ export default {
          * Disallow use of assignment in return statement
          * https://eslint.org/docs/latest/rules/no-return-assign.
          */
-        "no-return-assign": "off",
+        // "no-return-assign": "off",
 
         /**
          * Disallow use of `javascript:` urls.
@@ -1061,7 +1062,7 @@ export default {
          * Disallow the use of ternary operators
          * https://eslint.org/docs/latest/rules/no-ternary.
          */
-        "no-ternary": "off",
+        // "no-ternary": "off",
 
         /**
          * Restrict what can be thrown as an exception
@@ -1079,22 +1080,22 @@ export default {
          * Disallow use of undefined variable
          * https://eslint.org/docs/latest/rules/no-undefined.
          */
-        "no-undefined": "off",
+        // "no-undefined": "off",
 
         /**
          * Disallow dangling underscores in identifiers
          * https://eslint.org/docs/latest/rules/no-underscore-dangle.
          */
-        "no-underscore-dangle": [
-            "off",
-            {
-                allow: [],
-                allowAfterThis: true,
-                allowFunctionParams: true,
-                allowAfterSuper: false,
-                enforceInMethodNames: true,
-            },
-        ],
+        // "no-underscore-dangle": [
+        //     "off",
+        //     {
+        //         allow: [],
+        //         allowAfterThis: true,
+        //         allowFunctionParams: true,
+        //         allowAfterSuper: false,
+        //         enforceInMethodNames: true,
+        //     },
+        // ],
 
         /**
          * Disallow the use of Boolean literals in conditional expressions 🔧
@@ -1187,13 +1188,13 @@ export default {
          * Disallow use of void operator
          * https://eslint.org/docs/latest/rules/no-void.
          */
-        "no-void": "off",
+        // "no-void": "off",
 
         /**
          * Disallow usage of configurable warning terms in comments: e.g. Todo
          * https://eslint.org/docs/latest/rules/no-warning-comments.
          */
-        "no-warning-comments": ["off", { terms: ["todo", "fixme", "xxx"], location: "start" }],
+        // "no-warning-comments": ["off", { terms: ["todo", "fixme", "xxx"], location: "start" }],
 
         /**
          * Disallow use of the with statement ✅
@@ -1256,7 +1257,7 @@ export default {
          * Suggest using named capture group in regular expression
          * https://eslint.org/docs/latest/rules/prefer-named-capture-group.
          */
-        "prefer-named-capture-group": "off",
+        // "prefer-named-capture-group": "off",
 
         /**
          * Disallow parseInt() in favor of binary, octal, and hexadecimal literals 🔧
@@ -1269,7 +1270,7 @@ export default {
          * https://eslint.org/docs/latest/rules/prefer-object-has-own.
          * Turned off: only supported with ES2022 and node 18: can be turned on when all projects support node18/ES2022.
          */
-        "prefer-object-has-own": "off",
+        // "prefer-object-has-own": "off",
 
         /**
          * Prefer use of an object spread over Object.assign 🔧
@@ -1317,13 +1318,13 @@ export default {
          * Require `await` in `async function` (note: this is a horrible rule that should never be used)
          * https://eslint.org/docs/latest/rules/require-await.
          */
-        "require-await": "off",
+        // "require-await": "off",
 
         /**
          * Enforce the use of u flag on RegExp
          * https://eslint.org/docs/latest/rules/require-unicode-regexp.
          */
-        "require-unicode-regexp": "off",
+        // "require-unicode-regexp": "off",
 
         /**
          * Disallow generator functions that do not have yield ✅
@@ -1335,27 +1336,27 @@ export default {
          * Import sorting 🔧
          * https://eslint.org/docs/latest/rules/sort-imports.
          */
-        "sort-imports": [
-            "off",
-            {
-                ignoreCase: false,
-                ignoreDeclarationSort: false,
-                ignoreMemberSort: false,
-                memberSyntaxSortOrder: ["none", "all", "multiple", "single"],
-            },
-        ],
+        // "sort-imports": [
+        //     "off",
+        //     {
+        //         ignoreCase: false,
+        //         ignoreDeclarationSort: false,
+        //         ignoreMemberSort: false,
+        //         memberSyntaxSortOrder: ["none", "all", "multiple", "single"],
+        //     },
+        // ],
 
         /**
          * Requires object keys to be sorted
          * https://eslint.org/docs/latest/rules/sort-keys.
          */
-        "sort-keys": ["off", "asc", { caseSensitive: false, natural: true }],
+        // "sort-keys": ["off", "asc", { caseSensitive: false, natural: true }],
 
         /**
          * Sort variables within the same declaration block 🔧
          * https://eslint.org/docs/latest/rules/sort-vars.
          */
-        "sort-vars": "off",
+        // "sort-vars": "off",
 
         /**
          * Require or disallow strict mode directives 🔧