@prowise/eslint-config-linting-rules 2.3.0 → 2.3.2

package/package.json

@@ -1 +1 @@
-{"name":"@prowise/eslint-config-linting-rules","version":"2.3.0","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","./next":"./next.js"},"scripts":{"lint":"eslint","test":"node tests/jsx-pascal-case-rule.test.js","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":{"@next/eslint-plugin-next":"^15.3.4","@stylistic/eslint-plugin":"^5.0.0","@vitest/eslint-plugin":"^1.3.3","confusing-browser-globals":"^1.0.11","eslint":"^9.29.0","eslint-import-resolver-typescript":"^4.4.4","eslint-plugin-file-progress":"^3.0.2","eslint-plugin-import-x":"^4.16.1","eslint-plugin-jsdoc":"^51.2.3","eslint-plugin-jsx-a11y":"^6.10.2","eslint-plugin-n":"^17.20.0","eslint-plugin-prefer-arrow-functions":"^3.6.2","eslint-plugin-react":"^7.37.5","eslint-plugin-react-hooks":"^5.2.0","eslint-plugin-solid":"^0.14.5","globals":"^16.2.0","typescript-eslint":"^8.35.0"},"devDependencies":{"@eslint/config-inspector":"^1.1.0"}}
+{"name":"@prowise/eslint-config-linting-rules","version":"2.3.2","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","./next":"./next.js"},"scripts":{"lint":"eslint","test":"node tests/jsx-pascal-case-rule.test.js","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":{"@next/eslint-plugin-next":"^15.3.5","@stylistic/eslint-plugin":"^5.1.0","@vitest/eslint-plugin":"^1.3.4","confusing-browser-globals":"^1.0.11","eslint":"^9.30.1","eslint-import-resolver-typescript":"^4.4.4","eslint-plugin-file-progress":"^3.0.2","eslint-plugin-import-x":"^4.16.1","eslint-plugin-jsdoc":"^51.3.4","eslint-plugin-jsx-a11y":"^6.10.2","eslint-plugin-n":"^17.21.0","eslint-plugin-prefer-arrow-functions":"^3.6.2","eslint-plugin-react":"^7.37.5","eslint-plugin-react-hooks":"^5.2.0","eslint-plugin-solid":"^0.14.5","globals":"^16.3.0","typescript-eslint":"^8.36.0"},"devDependencies":{"@eslint/config-inspector":"^1.1.0"}}

package/rules/plugin-import-rules.js

@@ -126,7 +126,7 @@ export default {
          * Forbid cyclical dependencies between modules
          * https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/no-cycle.md.
          */
-        "import/no-cycle": ["error", { maxDepth: "∞" }],
+        "import/no-cycle": ["error", { ignoreExternal: true }],
 
         /**
          * Forbid default exports. This is a terrible rule, do not use it.

package/rules/plugin-stylistic-rules.js

@@ -393,7 +393,7 @@ export default {
          * Requires operator at the beginning of the line in multiline statements 🔧
          * https://eslint.style/rules/operator-linebreak.
          */
-        "style/operator-linebreak": ["error", "after", { overrides: { "?": "before", ":": "before" } }],
+        "style/operator-linebreak": ["error", "after", { overrides: { "?": "before", ":": "before", "|": "ignore" } }],
 
         /**
          * Disallow padding within blocks 🔧

package/rules/plugin-typescript-rules.js

@@ -17,7 +17,10 @@ export default {
     languageOptions: {
         parser: typescriptPlugin.parser,
         parserOptions: {
-            project: true,
+            ecmaVersion: "latest",
+            sourceType: "module",
+            projectService: true,
+            tsconfigRootDir: import.meta.dirname,
         },
     },
 
@@ -530,7 +533,7 @@ export default {
          * Enforce non-null assertions over explicit type casts 🔒 🔧 💭
          * https://typescript-eslint.io/rules/non-nullable-type-assertion-style.
          */
-        "@typescript-eslint/non-nullable-type-assertion-style": "warn",
+        // "@typescript-eslint/non-nullable-type-assertion-style": "off",
 
         /**
          * Require or disallow parameter properties in class constructors
@@ -705,12 +708,6 @@ export default {
             },
         ],
 
-        /**
-         * Require type annotations in certain places
-         * https://typescript-eslint.io/rules/typedef.
-         */
-        "@typescript-eslint/typedef": "warn",
-
         /**
          * Enforce unbound methods are called with their expected scope ✅ 💭
          * While this rule should be enabled, it gives to many problems with external libraries
@@ -743,20 +740,5 @@ export default {
                 tsx: "never",
             },
         ],
-
-        // "import/no-extraneous-dependencies": [
-        //     importRules.rules["import/no-extraneous-dependencies"][0],
-        //     {
-        //         ...importRules.rules["import/no-extraneous-dependencies"][1],
-        //         devDependencies: importRules.rules["import/no-extraneous-dependencies"][1].devDependencies.reduce(
-        //             (result, devDep) => {
-        //                 const toAppend = [devDep];
-        //                 const devDepWithTs = devDep.replace(/\bjs(x?)\b/g, "ts$1");
-        //                 if (devDepWithTs !== devDep) toAppend.push(devDepWithTs);
-        //                 return [...result, ...toAppend];
-        //             },
-        //         ),
-        //     },
-        // ],
     },
 };