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

package/rules/plugin-import-rules.js

@@ -12,17 +12,23 @@ export default {
     plugins: { import: importPlugin },
 
     rules: {
+        /**
+         * Enforce or ban the use of inline type-only markers for named imports.
+         * https://github.com/un-ts/eslint-plugin-import-x/blob/v4.5.0/docs/rules/consistent-type-specifier-style.md
+         */
+        "import/consistent-type-specifier-style": ["error", "prefer-inline"],
+
         /**
          * Ensure default import coupled with default export
          * https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/default.md.
          */
-        "import/default": "off",
+        // "import/default": "off",
 
         /**
          * Dynamic imports require a leading comment with a webpackChunkName
          * https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/dynamic-import-chunkname.md.
          */
-        "import/dynamic-import-chunkname": ["off", { importFunctions: [], webpackChunknameFormat: "[0-9a-zA-Z-_/.]+" }],
+        // "import/dynamic-import-chunkname": ["off", { importFunctions: [], webpackChunknameFormat: "[0-9a-zA-Z-_/.]+" }],
 
         /**
          * Disallow invalid exports, e.g. Multiple defaults
@@ -34,7 +40,7 @@ export default {
          * This rule enforces that all exports are declared at the bottom of the file.
          * Https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/exports-last.md.
          */
-        "import/exports-last": "off",
+        // "import/exports-last": "off",
 
         /**
          * Ensure consistent use of file extension within the import path
@@ -42,7 +48,7 @@ export default {
          *
          * Only warn, do not error on this, certain bundlers might accept it.
          */
-        "import/extensions": ["off", "always", { ignorePackages: true }],
+        // "import/extensions": ["off", "always", { ignorePackages: true }],
 
         /**
          * Disallow non-import statements appearing before import statements
@@ -56,13 +62,13 @@ export default {
          * in a single file.
          * Https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/group-exports.md.
          */
-        "import/group-exports": "off",
+        // "import/group-exports": "off",
 
         /**
          * Forbid modules to have too many dependencies
          * https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/group-exports.md.
          */
-        "import/max-dependencies": ["off", { max: 10 }],
+        // "import/max-dependencies": ["off", { max: 10 }],
 
         /**
          * Ensure named imports coupled with named exports
@@ -74,7 +80,7 @@ export default {
          * Enforces names exist at the time they are dereferenced, when imported as a full namespace
          * https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/namespace.md.
          */
-        "import/namespace": "off",
+        // "import/namespace": "off",
 
         /**
          * Require a newline after the last import/require in a group
@@ -98,23 +104,23 @@ export default {
          * Reports if a module"s default export is unnamed
          * https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/no-anonymous-default-export.md.
          */
-        "import/no-anonymous-default-export": [
-            "off",
-            {
-                allowArray: false,
-                allowArrowFunction: false,
-                allowAnonymousClass: false,
-                allowAnonymousFunction: false,
-                allowLiteral: false,
-                allowObject: false,
-            },
-        ],
+        // "import/no-anonymous-default-export": [
+        //     "off",
+        //     {
+        //         allowArray: false,
+        //         allowArrowFunction: false,
+        //         allowAnonymousClass: false,
+        //         allowAnonymousFunction: false,
+        //         allowLiteral: false,
+        //         allowObject: false,
+        //     },
+        // ],
 
         /**
          * Disallow require()
          * https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/no-commonjs.md.
          */
-        "import/no-commonjs": "off",
+        // "import/no-commonjs": "off",
 
         /**
          * Forbid cyclical dependencies between modules
@@ -126,13 +132,13 @@ export default {
          * Forbid default exports. This is a terrible rule, do not use it.
          * Https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/no-default-export.md.
          */
-        "import/no-default-export": "off",
+        // "import/no-default-export": "off",
 
         /**
          * Disallow use of jsdoc-marked-deprecated imports
          * https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/no-deprecated.md.
          */
-        "import/no-deprecated": "off",
+        // "import/no-deprecated": "off",
 
         /**
          * Disallow duplicate imports
@@ -151,36 +157,36 @@ export default {
          * paths are treated both as absolute paths, and relative to process.cwd()
          * https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/no-extraneous-dependencies.md.
          */
-        "import/no-extraneous-dependencies": [
-            "off",
-            {
-                devDependencies: [
-                    "test/**", // tape, common npm pattern
-                    "tests/**", // also common npm pattern
-                    "spec/**", // mocha, rspec-like pattern
-                    "**/__tests__/**", // jest pattern
-                    "**/__mocks__/**", // jest pattern
-                    "test.{js,jsx}", // repos with a single test file
-                    "test-*.{js,jsx}", // repos with multiple top-level test files
-                    "**/*{.,_}{test,spec}.{js,jsx}", // tests where the extension or filename suffix denotes that it is a test
-                    "**/jest.config.js", // jest config
-                    "**/jest.setup.js", // jest setup
-                    "**/vue.config.js", // vue-cli config
-                    "**/webpack.config.js", // webpack config
-                    "**/webpack.config.*.js", // webpack config
-                    "**/rollup.config.js", // rollup config
-                    "**/rollup.config.*.js", // rollup config
-                    "**/gulpfile.js", // gulp config
-                    "**/gulpfile.*.js", // gulp config
-                    "**/Gruntfile{,.js}", // grunt config
-                    "**/protractor.conf.js", // protractor config
-                    "**/protractor.conf.*.js", // protractor config
-                    "**/karma.conf.js", // karma config
-                    "**/.eslintrc.js", // eslint config
-                ],
-                optionalDependencies: false,
-            },
-        ],
+        // "import/no-extraneous-dependencies": [
+        //     "off",
+        //     {
+        //         devDependencies: [
+        //             "test/**", // tape, common npm pattern
+        //             "tests/**", // also common npm pattern
+        //             "spec/**", // mocha, rspec-like pattern
+        //             "**/__tests__/**", // jest pattern
+        //             "**/__mocks__/**", // jest pattern
+        //             "test.{js,jsx}", // repos with a single test file
+        //             "test-*.{js,jsx}", // repos with multiple top-level test files
+        //             "**/*{.,_}{test,spec}.{js,jsx}", // tests where the extension or filename suffix denotes that it is a test
+        //             "**/jest.config.js", // jest config
+        //             "**/jest.setup.js", // jest setup
+        //             "**/vue.config.js", // vue-cli config
+        //             "**/webpack.config.js", // webpack config
+        //             "**/webpack.config.*.js", // webpack config
+        //             "**/rollup.config.js", // rollup config
+        //             "**/rollup.config.*.js", // rollup config
+        //             "**/gulpfile.js", // gulp config
+        //             "**/gulpfile.*.js", // gulp config
+        //             "**/Gruntfile{,.js}", // grunt config
+        //             "**/protractor.conf.js", // protractor config
+        //             "**/protractor.conf.*.js", // protractor config
+        //             "**/karma.conf.js", // karma config
+        //             "**/.eslintrc.js", // eslint config
+        //         ],
+        //         optionalDependencies: false,
+        //     },
+        // ],
 
         /**
          * Reports the use of import declarations with CommonJS exports in any module except for the main module.
@@ -192,7 +198,7 @@ export default {
          * Prevent importing the submodules of other modules
          * https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/no-internal-modules.md.
          */
-        "import/no-internal-modules": ["off", { allow: [] }],
+        // "import/no-internal-modules": ["off", { allow: [] }],
 
         /**
          * Forbid mutable exports
@@ -222,19 +228,19 @@ export default {
          * Prohibit named exports. This is a terrible rule, do not use it.
          * Https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/no-named-export.md.
          */
-        "import/no-named-export": "off",
+        // "import/no-named-export": "off",
 
         /**
          * Disallow namespace imports
          * https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/no-namespace.md.
          */
-        "import/no-namespace": "off",
+        // "import/no-namespace": "off",
 
         /**
          * No Node.js builtin modules
          * https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/no-nodejs-modules.md.
          */
-        "import/no-nodejs-modules": "off",
+        // "import/no-nodejs-modules": "off",
 
         /**
          * Use this rule to prevent importing packages through relative paths.
@@ -246,13 +252,13 @@ export default {
          * Use this rule to prevent imports to folders in relative parent paths.
          * Https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/no-relative-parent-imports.md.
          */
-        "import/no-relative-parent-imports": "off",
+        // "import/no-relative-parent-imports": "off",
 
         /**
          * Restrict which files can be imported in a given folder
          * https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/no-restricted-paths.md.
          */
-        "import/no-restricted-paths": "off",
+        // "import/no-restricted-paths": "off",
 
         /**
          * Forbid a module from importing itself
@@ -265,7 +271,7 @@ export default {
          * importing for side effects is perfectly acceptable, if you need side effects.
          * Https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/no-unassigned-import.md.
          */
-        "import/no-unassigned-import": "off",
+        // "import/no-unassigned-import": "off",
 
         /**
          * Ensure imports point to files/modules that can be resolved
@@ -278,14 +284,14 @@ export default {
          * https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/no-unused-modules.md
          * TODO: enable once it supports CJS.
          */
-        "import/no-unused-modules": [
-            "off",
-            {
-                ignoreExports: [],
-                missingExports: true,
-                unusedExports: true,
-            },
-        ],
+        // "import/no-unused-modules": [
+        //     "off",
+        //     {
+        //         ignoreExports: [],
+        //         missingExports: true,
+        //         unusedExports: true,
+        //     },
+        // ],
 
         /**
          * Ensures that there are no useless path segments
@@ -322,7 +328,7 @@ export default {
          * Require modules with a single export to use a default export
          * https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/prefer-default-export.md.
          */
-        "import/prefer-default-export": "off",
+        // "import/prefer-default-export": "off",
 
         /**
          * Warn if a module could be mistakenly parsed as a script by a consumer
@@ -331,6 +337,6 @@ export default {
          * this should not be enabled until this proposal has at least been *presented* to TC39.
          * At the moment, it"s not a thing.
          */
-        "import/unambiguous": "off",
+        // "import/unambiguous": "off",
     },
 };

package/rules/plugin-jest-rules.js

@@ -18,7 +18,7 @@ export default {
          * Disallow confusing usages of jest.setTimeout
          * https://github.com/jest-community/eslint-plugin-jest/blob/main/docs/rules/no-confusing-set-timeout.md
          */
-        "jest/no-confusing-set-timeout": "off",
+        // "jest/no-confusing-set-timeout": "off",
 
         /**
          * Disallow use of deprecated functions ✅ 🔧
@@ -48,13 +48,13 @@ export default {
          * Disallow specific jest. methods
          * https://github.com/jest-community/eslint-plugin-jest/blob/main/docs/rules/no-restricted-jest-methods.md
          */
-        "jest/no-restricted-jest-methods": "off",
+        // "jest/no-restricted-jest-methods": "off",
 
         /**
          * Disallow using jest.mock() factories without an explicit type parameter
          * https://github.com/jest-community/eslint-plugin-jest/blob/main/docs/rules/no-untyped-mock-factory.md
          */
-        "jest/no-untyped-mock-factory": "off",
+        // "jest/no-untyped-mock-factory": "off",
 
         /**
          * Require promises that have expectations in their chain to be valid ✅

package/rules/plugin-jsdoc-rules.js

@@ -28,19 +28,19 @@ export default {
          * Ensures that (JavaScript) examples within JSDoc adhere to ESLint rules
          * https://github.com/gajus/eslint-plugin-jsdoc#eslint-plugin-jsdoc-rules-check-examples.
          */
-        "jsdoc/check-examples": "off",
+        // "jsdoc/check-examples": "off",
 
         /**
          * Reports invalid padding inside JSDoc blocks
          * https://github.com/gajus/eslint-plugin-jsdoc#eslint-plugin-jsdoc-rules-check-indentation.
          */
-        "jsdoc/check-indentation": "off",
+        // "jsdoc/check-indentation": "off",
 
         /**
          * Reports invalid alignment of JSDoc block lines
          * https://github.com/gajus/eslint-plugin-jsdoc#eslint-plugin-jsdoc-rules-check-line-alignment.
          */
-        "jsdoc/check-line-alignment": "off",
+        // "jsdoc/check-line-alignment": "off",
 
         /**
          * Ensures that parameter names in JSDoc match those in the function declaration ✅
@@ -146,7 +146,7 @@ export default {
          * Reports when certain comment structures are always expected
          * https://github.com/gajus/eslint-plugin-jsdoc#eslint-plugin-jsdoc-rules-no-missing-syntax.
          */
-        "jsdoc/no-missing-syntax": "off",
+        // "jsdoc/no-missing-syntax": "off",
 
         /**
          * ✅
@@ -158,7 +158,7 @@ export default {
          * Reports when certain comment structures are present
          * https://github.com/gajus/eslint-plugin-jsdoc#eslint-plugin-jsdoc-rules-no-restricted-syntax.
          */
-        "jsdoc/no-restricted-syntax": "off",
+        // "jsdoc/no-restricted-syntax": "off",
 
         /**
          * This rule reports types being used on `@param` or `@returns`
@@ -204,25 +204,25 @@ export default {
          *
          * If you cannot do that, PR reviewers should call you out on it.
          */
-        "jsdoc/require-description-complete-sentence": "off",
+        // "jsdoc/require-description-complete-sentence": "off",
 
         /**
          * Requires that all functions have examples
          * https://github.com/gajus/eslint-plugin-jsdoc#eslint-plugin-jsdoc-rules-require-example.
          */
-        "jsdoc/require-example": "off",
+        // "jsdoc/require-example": "off",
 
         /**
          * Checks that all files have one `@file`, `@fileoverview`, or `@overview` tag at the beginning of the file
          * https://github.com/gajus/eslint-plugin-jsdoc#eslint-plugin-jsdoc-rules-require-file-overview.
          */
-        "jsdoc/require-file-overview": "off",
+        // "jsdoc/require-file-overview": "off",
 
         /**
          * Requires a hyphen before the `@param` description
          * https://github.com/gajus/eslint-plugin-jsdoc#eslint-plugin-jsdoc-rules-require-hyphen-before-param-description.
          */
-        "jsdoc/require-hyphen-before-param-description": "off",
+        // "jsdoc/require-hyphen-before-param-description": "off",
 
         /**
          * Require JSDoc comments ✅
@@ -277,14 +277,14 @@ export default {
          * Requires that each `@param` tag has a `type` value ✅
          * https://github.com/gajus/eslint-plugin-jsdoc#eslint-plugin-jsdoc-rules-require-param-type.
          */
-        "jsdoc/require-param-type": "off",
+        // "jsdoc/require-param-type": "off",
 
         /**
          * Requires that all `@typedef` and `@namespace` tags have `@property` when their type ✅
          * is a plain `object`, `Object`, or `PlainObject
          * https://github.com/gajus/eslint-plugin-jsdoc#eslint-plugin-jsdoc-rules-require-property.
          */
-        "jsdoc/require-property": "off",
+        // "jsdoc/require-property": "off",
 
         /**
          * Requires that each `@property` tag has a `description` value ✅
@@ -326,7 +326,7 @@ export default {
          * Requires that `@returns` tag has `type` value ✅
          * https://github.com/gajus/eslint-plugin-jsdoc#eslint-plugin-jsdoc-rules-require-returns-type.
          */
-        "jsdoc/require-returns-type": "off",
+        // "jsdoc/require-returns-type": "off",
 
         /**
          * Requires that throw statements are documented

package/rules/plugin-jsx-a11y-rules.js

@@ -39,7 +39,7 @@ export default {
          * Enforce <a> text to not exactly match "click here", "here", "link", or "a link".
          * Https://github.com/jsx-eslint/eslint-plugin-jsx-a11y/blob/master/docs/rules/anchor-ambiguous-text.md.
          */
-        "jsx-a11y/anchor-ambiguous-text": "off",
+        // "jsx-a11y/anchor-ambiguous-text": "off",
 
         /**
          * Enforce that anchors have content.  ✅ 🔒
@@ -95,7 +95,7 @@ export default {
          * Ensure the autocomplete attribute is correct and suitable for the form field it is used with.  ✅ 🔒
          * https://github.com/jsx-eslint/eslint-plugin-jsx-a11y/blob/29c68596b15c4ff0a40daae6d4a2670e36e37d35/docs/rules/autocomplete-valid.md.
          */
-        "jsx-a11y/autocomplete-valid": ["off", { inputComponents: [] }],
+        "jsx-a11y/autocomplete-valid": ["error", { inputComponents: ["Input", "FormField"] }],
 
         /**
          * Require onClick be accompanied by onKeyUp/onKeyDown/onKeyPress. ✅ 🔒

package/rules/plugin-node-rules.js

@@ -25,13 +25,13 @@ export default {
          * Enforce return after a callback.
          * https://github.com/eslint-community/eslint-plugin-n/blob/master/docs/rules/callback-return.md.
          */
-        "node/callback-return": "off",
+        // "node/callback-return": "off",
 
         /**
          * Enforce either module.exports or exports.
          * https://github.com/eslint-community/eslint-plugin-n/blob/master/docs/rules/exports-style.md.
          */
-        "node/exports-style": "off",
+        // "node/exports-style": "off",
 
         /**
          * Enforce file extensions in import declarations (ignores packages by default).
@@ -49,7 +49,7 @@ export default {
          * Enforces error handling in callbacks (node environment)
          * https://github.com/eslint-community/eslint-plugin-n/blob/master/docs/rules/handle-callback-err.md.
          */
-        "node/handle-callback-err": "off",
+        // "node/handle-callback-err": "off",
 
         /**
          * Require correct usage of hashbang.
@@ -61,7 +61,7 @@ export default {
          * Disallow mixing regular variable and require declarations
          * https://github.com/eslint-community/eslint-plugin-n/blob/master/docs/rules/no-mixed-requires.md.
          */
-        "node/no-mixed-requires": ["off", false],
+        // "node/no-mixed-requires": ["off", false],
 
         /**
          * Disallow use of new operator with the require function
@@ -79,13 +79,13 @@ export default {
          * Disallow use of process.env
          * https://github.com/eslint-community/eslint-plugin-n/blob/master/docs/rules/no-process-env.md.
          */
-        "/node/no-process-env": "off",
+        // "/node/no-process-env": "off",
 
         /**
          * Disallow process.exit()
          * https://github.com/eslint-community/eslint-plugin-n/blob/master/docs/rules/no-process-exit.md.
          */
-        "node/no-process-exit": "off",
+        // "node/no-process-exit": "off",
 
         /**
          * Disallow use of the Buffer() constructor

package/rules/plugin-prefer-arrow-rules.js

@@ -1,16 +1,20 @@
-import preferArrowPlugin from "eslint-plugin-prefer-arrow";
+import preferArrowPlugin from "eslint-plugin-prefer-arrow-functions";
 
 export default {
-    plugins: { "prefer-arrow": preferArrowPlugin },
+    plugins: { "prefer-arrow-functions": preferArrowPlugin },
 
+    /**
+     * See https://www.npmjs.com/package/eslint-plugin-prefer-arrow-functions
+     */
     rules: {
-        "prefer-arrow/prefer-arrow-functions": [
+        "prefer-arrow-functions/prefer-arrow-functions": [
             "warn",
             {
-                disallowPrototype: true,
-                singleReturnOnly: false,
+                allowNamedFunctions: true,
                 classPropertiesAllowed: false,
-                allowStandaloneDeclarations: true,
+                disallowPrototype: false,
+                returnStyle: "unchanged",
+                singleReturnOnly: false,
             },
         ],
     },