linter-bundle 2.1.0 → 2.3.0

package/CHANGELOG.md

@@ -6,7 +6,48 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 
 ## [Unreleased]
 
-[Show all code changes](https://github.com/jens-duttke/linter-bundle/compare/v2.1.0...HEAD)
+[Show all code changes](https://github.com/jens-duttke/linter-bundle/compare/v2.2.0...HEAD)
+
+## [2.2.0] - 2022.01.27
+
+### Changed
+
+- [eslint] In the `.vscode/settings.json` the "configFile" option in "eslint.options" has been renamed to "overrideConfigFile"
+- [eslint] Re-added [`eslint-plugin-jsx-a11y`](https://www.npmjs.com/package/eslint-plugin-jsx-a11y) v6.5.1
+- [eslint] Re-added [`eslint-plugin-promise`](https://www.npmjs.com/package/eslint-plugin-promise) v6.0.0
+- [eslint] Re-added [`eslint-plugin-react`](https://www.npmjs.com/package/eslint-plugin-react) v7.28.0
+- [stylelint] Re-added [`stylelint-declaration-block-no-ignored-properties`](https://www.npmjs.com/package/stylelint-declaration-block-no-ignored-properties) v2.5.0
+- [eslint] Updated `eslint-plugin-react-hooks` from v4.2.1-alpha-13455d26d-20211104 to v4.3.0
+- [eslint] Re-added [`react/no-invalid-html-attribute`](https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/no-invalid-html-attribute.md) rule
+- [eslint] Re-added [`react/no-arrow-function-lifecycle`](https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/no-arrow-function-lifecycle.md) rule
+- [eslint] Re-added [`react/no-unused-class-component-methods`](https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/no-unused-class-component-methods.md) rule
+- [eslint] Disabled [`unicorn/prefer-object-has-own`](https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-object-has-own.md) for now, because of limited engine support
+- [eslint] Disabled `allowThrowingAny` and `allowThrowingUnknown` of the [`@typescript-eslint/no-throw-literal`](https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/no-throw-literal.md) rule
+- [eslint] Reactivated [`import/no-import-module-exports`](https://github.com/import-js/eslint-plugin-import/blob/master/docs/rules/no-import-module-exports.md) rule
+- [eslint] Added new [`import/no-unused-modules`](https://github.com/import-js/eslint-plugin-import/blob/master/docs/rules/no-unused-modules.md) rule
+- [eslint] Added new [`jest/prefer-comparison-matcher`](https://github.com/jest-community/eslint-plugin-jest/blob/main/docs/rules/prefer-comparison-matcher.md) rule
+- [eslint] Added new [`jest/prefer-equality-matcher`](https://github.com/jest-community/eslint-plugin-jest/blob/main/docs/rules/prefer-equality-matcher.md) rule
+- [eslint] Added new [`unicorn/no-await-expression-member`](https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/no-await-expression-member.md) rule
+- [eslint] Added new [`unicorn/prefer-code-point`](https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-code-point.md) rule
+- [eslint] Added new [`unicorn/no-thenable`](https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/no-thenable.md) rule
+- [eslint] Added new [`unicorn/no-useless-promise-resolve-reject`](https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/no-useless-promise-resolve-reject.md) rule
+- [eslint] Added new [`unicorn/relative-url-style`](https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/relative-url-style.md) rule
+- [eslint] Added new [`unicorn/prefer-json-parse-buffer`](https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-json-parse-buffer.md) rule
+- [eslint] Updated `@typescript-eslint` from v5.3.0 to v5.10.1
+- [eslint] Updated `eslint` from v8.2.0 to v8.7.0
+- [eslint] Updated `eslint-plugin-functional` from v4.0.2 to v4.1.1
+- [eslint] Updated `eslint-plugin-import` from v2.25.2 to v2.25.4
+- [eslint] Updated `eslint-plugin-jest` from v25.2.3 to v26.0.0
+- [eslint] Updated `eslint-plugin-jsdoc` from v37.0.3 to v37.7.0
+- [eslint] Updated `eslint-plugin-unicode` from v38.0.0 to v40.1.0
+- [markdownlint] Updated `markdownlint-cli` from v0.29.0 to v0.30.0
+- [stylelint] Updated `stylelint` from v14.0.1 to v14.3.0
+- [stylelint] Updated `stylelint-scss` from v4.0.0 to v4.1.0
+- [stylelint] Added new [`scss/dollar-variable-no-namespaced-assignment`](https://github.com/stylelint-scss/stylelint-scss/tree/master/src/rules/dollar-variable-no-namespaced-assignment) rule
+- [stylelint] Added new [`scss/at-use-no-unnamespaced`](https://github.com/stylelint-scss/stylelint-scss/tree/master/src/rules/at-use-no-unnamespaced) rule
+- [stylelint] Added but disabled [`rule-selector-property-disallowed-list`](https://github.com/stylelint/stylelint/tree/main/lib/rules/rule-selector-property-disallowed-list) rule
+
+[Show all code changes](https://github.com/jens-duttke/linter-bundle/compare/v2.1.0...v2.2.0)
 
 ## [2.1.0] - 2021.11.07
 
@@ -23,7 +64,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 
 - [eslint] [`eslint-plugin-jsx-a11y`](https://www.npmjs.com/package/eslint-plugin-jsx-a11y) has been removed, because it seems not to be regularly maintained anymore, which blocks us from updating to ESLint v8
 - [eslint] [`eslint-plugin-promise`](https://www.npmjs.com/package/eslint-plugin-promise) has been removed, because it seems not to be regularly maintained anymore, which blocks us from updating to ESLint v8
-- [eslint] [`eslint-plugin-react`](https://www.npmjs.com/package/eslint-plugin-jsx-a11y) has been removed, because it's relying on unmaintained dependencies, which blocks us from updating to ESLint v8
+- [eslint] [`eslint-plugin-react`](https://www.npmjs.com/package/eslint-plugin-react) has been removed, because it's relying on unmaintained dependencies, which blocks us from updating to ESLint v8
 - [stylelint] [`stylelint-declaration-block-no-ignored-properties`](https://www.npmjs.com/package/stylelint-declaration-block-no-ignored-properties) has been removed, because it seems not to be regularly maintained anymore, which blocks us from updating to Stylelint v14
 - [stylelint] [`stylelint-use-nesting`](https://www.npmjs.com/package/stylelint-use-nesting) has been removed, because it seems not to be regularly maintained anymore, which blocks us from updating to Stylelint v14
 

package/README.md

@@ -28,8 +28,11 @@ This setup is using the following additional plugins:
 - [eslint-plugin-import](https://www.npmjs.com/package/eslint-plugin-import)
 - [eslint-plugin-jest](https://www.npmjs.com/package/eslint-plugin-jest)
 - [eslint-plugin-jsdoc](https://www.npmjs.com/package/eslint-plugin-jsdoc)
+- [eslint-plugin-jsx-a11y](https://www.npmjs.com/package/eslint-plugin-jsx-a11y)
 - [eslint-plugin-node](https://www.npmjs.com/package/eslint-plugin-node)
+- [eslint-plugin-promise](https://www.npmjs.com/package/eslint-plugin-promise)
 - [eslint-plugin-react-hooks](https://www.npmjs.com/package/eslint-plugin-react-hooks)
+- [eslint-plugin-react](https://www.npmjs.com/package/eslint-plugin-react)
 - [eslint-plugin-unicorn](https://www.npmjs.com/package/eslint-plugin-unicorn)
 
 Beside that, the following additional rules are part of this bundle:
@@ -38,22 +41,19 @@ Beside that, the following additional rules are part of this bundle:
 
 ### stylelint
 
+- [stylelint-declaration-block-no-ignored-properties](https://www.npmjs.com/package/stylelint-declaration-block-no-ignored-properties)
 - [stylelint-high-performance-animation](https://www.npmjs.com/package/stylelint-high-performance-animation) (Forked version)
 - [stylelint-order](https://www.npmjs.com/package/stylelint-order)
 - [stylelint-scss](https://www.npmjs.com/package/stylelint-scss)
 - [stylelint-selector-no-empty](https://www.npmjs.com/package/stylelint-selector-no-empty)
-- [stylelint-use-logical-spec](https://www.npmjs.com/package/stylelint-use-logical-spec)
 - [stylelint-selector-tag-no-without-class](https://www.npmjs.com/package/stylelint-selector-tag-no-without-class) (Forked version)
+- [stylelint-use-logical-spec](https://www.npmjs.com/package/stylelint-use-logical-spec)
 
 ### Previously used, but now unmaintained plugins
 
 Unfortunately a couple of previously used plugins are not regularly maintained or depend on unmaintained third-party code which blocks them from updating, so they don't provide updates for the major releases of the linters (ESLint and Stylelint).
 For that reason the following plugins are not used anymore:
 
-- [eslint-plugin-jsx-a11y](https://www.npmjs.com/package/eslint-plugin-jsx-a11y) (removed on 2021-11-05)
-- [eslint-plugin-promise](https://www.npmjs.com/package/eslint-plugin-promise) (removed on 2021-11-05)
-- [eslint-plugin-react](https://www.npmjs.com/package/eslint-plugin-react) (removed on 2021-11-05)
-- [stylelint-declaration-block-no-ignored-properties](https://www.npmjs.com/package/stylelint-declaration-block-no-ignored-properties) (removed on 2021-11-05)
 - [stylelint-use-nesting](https://www.npmjs.com/package/stylelint-use-nesting) (removed on 2021-11-05)
 
 If these plugins are maintained again, the plugins will also be used again.
@@ -318,7 +318,7 @@ This can be done by adding these options to your `.vscode/settings.json`:
 {
   "eslint.nodePath": "./node_modules/linter-bundle/node_modules/eslint",
   "eslint.options": {
-    "configFile": "./.eslintrc.js",
+    "overrideConfigFile": "./.eslintrc.js",
     "resolvePluginsRelativeTo": "./node_modules/linter-bundle",
     "rulePaths": ["./node_modules/linter-bundle/eslint/rules"],
     "reportUnusedDisableDirectives": "error",

package/eslint/index.js

@@ -22,7 +22,10 @@ module.exports = {
 		'eslint-comments',
 		'functional',
 		'import',
+		'jsx-a11y',
+		'promise',
 		'react-hooks',
+		'react',
 		'unicorn'
 	],
 	extends: [
@@ -639,7 +642,10 @@ module.exports = {
 		'@typescript-eslint/no-require-imports': 'error',
 		'@typescript-eslint/no-shadow': 'error',
 		'@typescript-eslint/no-this-alias': 'error',
-		'@typescript-eslint/no-throw-literal': 'error',
+		'@typescript-eslint/no-throw-literal': ['error', {
+			allowThrowingAny: false,
+			allowThrowingUnknown: false
+		}],
 		'@typescript-eslint/no-type-alias': ['off', { // @todo There should be an option like 'sub-in-unions-and-intersections', which allows `type A = (string | number)[];`
 			allowAliases: 'always',
 			allowCallbacks: 'always',
@@ -764,7 +770,7 @@ module.exports = {
 				'workbox-window/utils/WorkboxEvent'
 			]
 		}],
-		'import/no-import-module-exports': 'off', // @todo The rule reports files, which don't use `exports`. This must be a bug, report it!
+		'import/no-import-module-exports': 'error',
 		'import/no-mutable-exports': 'error',
 		'import/no-named-as-default-member': 'error',
 		'import/no-named-as-default': 'error',
@@ -809,6 +815,43 @@ module.exports = {
 		'import/prefer-default-export': 'off',
 		'import/unambiguous': 'off',
 
+		/**
+		 * eslint-plugin-jsx-a11y
+		 *
+		 * @see https://github.com/jsx-eslint/eslint-plugin-jsx-a11y
+		 */
+		'jsx-a11y/alt-text': 'error',
+		'jsx-a11y/anchor-has-content': 'error',
+		'jsx-a11y/anchor-is-valid': 'error',
+		'jsx-a11y/aria-activedescendant-has-tabindex': 'error',
+		'jsx-a11y/aria-props': 'error',
+		'jsx-a11y/aria-proptypes': 'error',
+		'jsx-a11y/aria-role': 'error',
+		'jsx-a11y/aria-unsupported-elements': 'error',
+		'jsx-a11y/autocomplete-valid': 'error',
+		'jsx-a11y/click-events-have-key-events': 'error',
+		'jsx-a11y/heading-has-content': 'error',
+		'jsx-a11y/html-has-lang': 'off', // <html> elements set by react-helmet automatically have a lang attribute set
+		'jsx-a11y/iframe-has-title': 'error',
+		'jsx-a11y/img-redundant-alt': 'error',
+		'jsx-a11y/interactive-supports-focus': 'error',
+		'jsx-a11y/label-has-associated-control': 'error',
+		'jsx-a11y/media-has-caption': 'error',
+		'jsx-a11y/mouse-events-have-key-events': 'error',
+		'jsx-a11y/no-access-key': 'off', // If you explicitly specify an access key, you usually have a reason for this, so it should not be prevented by a rule
+		'jsx-a11y/no-autofocus': ['error', { ignoreNonDOM: true }],
+		'jsx-a11y/no-distracting-elements': 'error',
+		'jsx-a11y/no-interactive-element-to-noninteractive-role': 'error',
+		'jsx-a11y/no-noninteractive-element-interactions': 'error',
+		'jsx-a11y/no-noninteractive-element-to-interactive-role': 'error',
+		'jsx-a11y/no-noninteractive-tabindex': 'error',
+		'jsx-a11y/no-redundant-roles': 'error',
+		'jsx-a11y/no-static-element-interactions': 'error',
+		'jsx-a11y/role-has-required-aria-props': 'error',
+		'jsx-a11y/role-supports-aria-props': 'error',
+		'jsx-a11y/scope': 'error',
+		'jsx-a11y/tabindex-no-positive': 'error',
+
 		/**
 		 * eslint-plugin-eslint-comments
 		 *
@@ -824,6 +867,26 @@ module.exports = {
 		'eslint-comments/no-use': 'off',
 		'eslint-comments/require-description': ['error', { ignore: ['eslint-enable'] }],
 
+		/**
+		 * eslint-plugin-promise
+		 *
+		 * @see https://github.com/xjamundx/eslint-plugin-promise
+		 */
+		'promise/always-return': 'off', // If the result of an `.then()` is not used, there is no need to return something.
+		'promise/avoid-new': 'off',
+		'promise/catch-or-return': 'error',
+		'promise/no-callback-in-promise': 'off',
+		'promise/no-native': 'off',
+		'promise/no-nesting': 'off',
+		'promise/no-new-statics': 'error',
+		'promise/no-promise-in-callback': 'off',
+		'promise/no-return-in-finally': 'error',
+		'promise/no-return-wrap': 'error',
+		'promise/param-names': 'off', // @todo Disabled until this issue is fixed: https://github.com/xjamundx/eslint-plugin-promise/issues/206
+		'promise/prefer-await-to-callbacks': 'off', // It's not always possible to use avoid callbacks.
+		'promise/prefer-await-to-then': 'off', // Depending on the use-case `.then()`/`.catch()` might be easier to understand
+		'promise/valid-params': 'off', // TypeScript ensures that
+
 		/**
 		 * eslint-plugin-unicorn
 		 *
@@ -852,11 +915,12 @@ module.exports = {
 		'unicorn/import-style': 'error',
 		'unicorn/new-for-builtins': 'error',
 		'unicorn/no-abusive-eslint-disable': 'error',
-		'unicorn/no-array-callback-reference': 'off', // If I use functions, they are specially developed for this use-case
+		'unicorn/no-array-callback-reference': 'off', // If I use functions, they are the best option for this use-case
 		'unicorn/no-array-for-each': 'error',
 		'unicorn/no-array-method-this-argument': 'error',
 		'unicorn/no-array-push-push': 'error',
 		'unicorn/no-array-reduce': ['error', { allowSimpleOperations: true }],
+		'unicorn/no-await-expression-member': 'error',
 		'unicorn/no-console-spaces': 'error',
 		'unicorn/no-document-cookie': 'error',
 		'unicorn/no-empty-file': 'error',
@@ -872,11 +936,13 @@ module.exports = {
 		'unicorn/no-object-as-default-parameter': 'error',
 		'unicorn/no-process-exit': 'error',
 		'unicorn/no-static-only-class': 'error',
+		'unicorn/no-thenable': 'error',
 		'unicorn/no-this-assignment': 'error',
 		'unicorn/no-unreadable-array-destructuring': 'error',
 		'unicorn/no-unsafe-regex': 'off',
 		'unicorn/no-unused-properties': 'error',
 		'unicorn/no-useless-fallback-in-spread': 'error',
+		'unicorn/no-useless-promise-resolve-reject': 'error',
 		'unicorn/no-invalid-remove-event-listener': 'error',
 		'unicorn/no-useless-length-check': 'error',
 		'unicorn/no-useless-spread': 'error',
@@ -891,6 +957,7 @@ module.exports = {
 		'unicorn/prefer-array-index-of': 'error',
 		'unicorn/prefer-array-some': 'error',
 		'unicorn/prefer-at': 'off', // @todo Disabled for now, since `at` is not supported by TypeScript type definitions yet.
+		'unicorn/prefer-code-point': 'error',
 		'unicorn/prefer-date-now': 'error',
 		'unicorn/prefer-default-parameters': 'error',
 		'unicorn/prefer-dom-node-append': 'error',
@@ -899,13 +966,14 @@ module.exports = {
 		'unicorn/prefer-dom-node-text-content': 'error',
 		'unicorn/prefer-export-from': 'error',
 		'unicorn/prefer-includes': 'error',
+		'unicorn/prefer-json-parse-buffer': 'error',
 		'unicorn/prefer-keyboard-event-key': 'error',
 		'unicorn/prefer-math-trunc': 'error',
 		'unicorn/prefer-modern-dom-apis': 'error',
 		'unicorn/prefer-module': 'off',
 		'unicorn/prefer-negative-index': 'error',
 		'unicorn/prefer-number-properties': 'error',
-		'unicorn/prefer-object-has-own': 'error',
+		'unicorn/prefer-object-has-own': 'off', // Not widely supported yet. Can be activated in 2024
 		'unicorn/prefer-object-from-entries': 'error',
 		'unicorn/prefer-optional-catch-binding': 'error',
 		'unicorn/prefer-prototype-methods': 'error',
@@ -923,6 +991,7 @@ module.exports = {
 		'unicorn/prefer-regexp-test': 'error',
 		'unicorn/prefer-type-error': 'error',
 		'unicorn/prevent-abbreviations': ['error', { ignore: ['args', 'i', 'j', 'i18n', 'ref', 'Ref', 'params', 'props', 'Props'] }],
+		'unicorn/relative-url-style': 'error',
 		'unicorn/require-array-join-separator': 'error',
 		'unicorn/require-number-to-fixed-digits-argument': 'error',
 		'unicorn/require-post-message-target-origin': 'off', // False-positive with Workers which don't support a `targetOrigin`

package/eslint/overrides-jest.js

@@ -88,6 +88,8 @@ module.exports = {
 				'jest/prefer-expect-resolves': 'off', // We prefer `expect(await promise)` enforced by 'jest/no-restricted-matchers'
 				'jest/prefer-hooks-on-top': 'error',
 				'jest/require-hook': 'error',
+				'jest/prefer-comparison-matcher': 'error',
+				'jest/prefer-equality-matcher': 'error',
 				'jest/prefer-spy-on': 'error',
 				'jest/prefer-strict-equal': 'error',
 				'jest/prefer-to-be': 'error',

package/eslint/overrides-jsdoc.js

@@ -24,8 +24,8 @@ module.exports = {
 				 */
 				'jsdoc/check-access': 'error',
 				'jsdoc/check-alignment': 'error',
-				'jsdoc/check-examples': 'off', // @todo need to be configured to allow text-based examples
-				'jsdoc/check-indentation': ['off', { excludeTags: ['typedef'] }],
+				'jsdoc/check-examples': 'off', // @todo Need to be configured to allow text-based examples
+				'jsdoc/check-indentation': ['off', { excludeTags: ['typedef'] }], // @todo Why is this disabled?
 				'jsdoc/check-param-names': 'error',
 				'jsdoc/check-syntax': 'error',
 				'jsdoc/check-line-alignment': 'error',

package/eslint/overrides-react.js

@@ -2,6 +2,8 @@
  * @file Settings for React code in TypeScript (TSX) files.
  */
 
+const ensureType = require('../helper/ensure-type');
+
 module.exports = {
 	overrides: [
 		{
@@ -21,7 +23,143 @@ module.exports = {
 				 * @see https://reactjs.org/docs/hooks-rules.html
 				 */
 				'react-hooks/rules-of-hooks': 'error',
-				'react-hooks/exhaustive-deps': 'error'
+				'react-hooks/exhaustive-deps': 'error',
+
+				/**
+				 * eslint-plugin-react
+				 *
+				 * @see https://github.com/yannickcr/eslint-plugin-react
+				 */
+				'react/boolean-prop-naming': 'error',
+				'react/button-has-type': 'error',
+				'react/default-props-match-prop-types': 'error',
+				'react/destructuring-assignment': 'off', // Sometimes destructuring makes sense, sometimes it doesn't - that depends on the context.
+				'react/display-name': 'error',
+				'react/forbid-component-props': ['error', { forbid: [
+					{
+						propName: 'className',
+						allowedFor: ensureType.array(global.linterBundleSettings?.overrides?.react?.['react/forbid-component-props']?.allowClassNameFor)
+					},
+					{
+						propName: 'style',
+						allowedFor: ensureType.array(global.linterBundleSettings?.overrides?.react?.['react/forbid-component-props']?.allowStyleFor)
+					}
+				] }],
+				'react/forbid-dom-props': 'error',
+				'react/forbid-elements': 'error',
+				'react/forbid-prop-types': 'error',
+				'react/forbid-foreign-prop-types': 'error',
+				'react/function-component-definition': ['off', { // @todo Doesn't work with Component-Factories, which shall return named components `return function MyComponent () {}`, while in modules, `const MyComponent = () => {}` shall be used
+					namedComponents: 'arrow-function',
+					unnamedComponents: 'arrow-function'
+				}],
+				'react/no-access-state-in-setstate': 'error',
+				'react/no-array-index-key': 'error',
+				'react/no-arrow-function-lifecycle': 'error',
+				'react/no-children-prop': 'error',
+				'react/no-danger': 'off',
+				'react/no-danger-with-children': 'error',
+				'react/no-deprecated': 'error',
+				'react/no-did-mount-set-state': 'error',
+				'react/no-did-update-set-state': 'error',
+				'react/no-direct-mutation-state': 'error',
+				'react/no-find-dom-node': 'error',
+				'react/no-invalid-html-attribute': 'error',
+				'react/no-is-mounted': 'error',
+				'react/no-multi-comp': 'error',
+				'react/no-namespace': 'error',
+				'react/no-redundant-should-component-update': 'error',
+				'react/no-render-return-value': 'error',
+				'react/no-set-state': 'off',
+				'react/no-typos': 'error',
+				'react/no-string-refs': 'error',
+				'react/no-this-in-sfc': 'error',
+				'react/no-unescaped-entities': 'error',
+				'react/no-unknown-property': 'error',
+				'react/no-unsafe': 'error',
+				'react/no-unstable-nested-components': 'error',
+				'react/no-unused-class-component-methods': 'error',
+				'react/no-unused-prop-types': 'error',
+				'react/no-unused-state': 'error',
+				'react/no-will-update-set-state': 'error',
+				'react/prefer-es6-class': 'error',
+				'react/prefer-exact-props': 'off', // With TypeScript interfaces, this rule is not required
+				'react/prefer-read-only-props': 'error',
+				'react/prefer-stateless-function': 'error',
+				'react/prop-types': 'off',
+				'react/react-in-jsx-scope': 'error',
+				'react/require-default-props': ['error', { forbidDefaultForRequired: true, ignoreFunctionalComponents: true }], // @see https://medium.com/@matanbobi/react-defaultprops-is-dying-whos-the-contender-443c19d9e7f1
+				'react/require-optimization': 'error',
+				'react/require-render-return': 'error',
+				'react/self-closing-comp': 'error',
+				'react/sort-comp': ['error', {
+					order: [
+						'type-annotations',
+						'static-variables',
+						'instance-variables',
+						'getters',
+						'setters',
+						'lifecycle',
+						'render',
+						'/^render.+$/',
+						'/^handle.+$/',
+						'everything-else',
+						'instance-methods',
+						'static-methods'
+					]
+				}],
+				'react/sort-prop-types': 'error',
+				'react/state-in-constructor': 'error',
+				'react/static-property-placement': 'error',
+				'react/style-prop-object': 'error',
+				'react/void-dom-elements-no-children': 'error',
+
+				// JSX-specific rules
+				'react/jsx-boolean-value': ['error', 'always'],
+				'react/jsx-child-element-spacing': 'off', // @todo Why is this disabled? Could it be, that the faulty behaviour is fixed in the meantime?
+				'react/jsx-closing-bracket-location': 'error',
+				'react/jsx-closing-tag-location': 'error',
+				'react/jsx-curly-newline': 'off',
+				'react/jsx-curly-spacing': 'error',
+				'react/jsx-equals-spacing': 'error',
+				'react/jsx-filename-extension': ['error', { extensions: ['.tsx'] }],
+				'react/jsx-first-prop-new-line': 'error',
+				'react/jsx-handler-names': 'off', // @todo There should be an option which checks if the function is used multiple times in a class (like this.closeTooltip()) - in that case, the 'handle' prefix should not be mandatory
+				'react/jsx-indent': ['error', 'tab', { checkAttributes: true, indentLogicalExpressions: true }],
+				'react/jsx-indent-props': ['error', 'tab'],
+				'react/jsx-key': ['error', { checkKeyMustBeforeSpread: true }],
+				'react/jsx-max-depth': ['error', { max: 8 }],
+				'react/jsx-max-props-per-line': ['error', { maximum: { single: 5, multi: 1 } }],
+				'react/no-adjacent-inline-elements': 'off', // @todo There is an issue if inline and block elements are mixed. Simple example: `<span>Text</span><br />` here, the space between the SPAN and BR should not be forced, because a space at the end of a line does not make sense.
+				'react/jsx-newline': 'off',
+				'react/jsx-no-bind': ['error', { ignoreDOMComponents: true }],
+				'react/jsx-no-comment-textnodes': 'error',
+				'react/jsx-no-constructed-context-values': 'error',
+				'react/jsx-no-duplicate-props': 'error',
+				'react/jsx-no-literals': 'off',
+				'react/jsx-no-script-url': 'error',
+				'react/jsx-no-target-blank': ['error', { allowReferrer: true, forms: true }],
+				'react/jsx-no-undef': 'error',
+				'react/jsx-no-useless-fragment': 'error',
+				'react/jsx-one-expression-per-line': ['off', { allow: 'single-child' }], // @todo Doesn't work with something like "Text <a href="...">Link</a> More Text", which should be valid
+				'react/jsx-curly-brace-presence': 'error',
+				'react/jsx-fragments': ['error', 'element'],
+				'react/jsx-pascal-case': ['error', { allowAllCaps: true }],
+				'react/jsx-props-no-multi-spaces': 'error',
+				'react/jsx-props-no-spreading': 'off',
+				'react/jsx-sort-default-props': 'error',
+				'react/jsx-sort-props': ['error', {
+					ignoreCase: true,
+					callbacksLast: true,
+					shorthandFirst: true,
+					shorthandLast: false,
+					noSortAlphabetically: true,
+					reservedFirst: true
+				}],
+				'react/jsx-tag-spacing': 'error',
+				'react/jsx-uses-react': 'error',
+				'react/jsx-uses-vars': 'error',
+				'react/jsx-wrap-multilines': 'error'
 			}
 		}
 	]

package/lint.js

@@ -121,6 +121,7 @@ const jobs = getTasksToRun(process.argv.splice(2)).map(({ taskName, config }) =>
 	throw new Error(`"${taskName}" is not a valid task.`);
 });
 
+// eslint-disable-next-line unicorn/prefer-top-level-await -- The minimum required Node.js version is v14.8, but we are also support down till v14.0
 void (async () => {
 	for (const { jobTitle, taskSetup, job } of jobs) {
 		process.stdout.write(jobTitle);

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "linter-bundle",
-	"version": "2.1.0",
+	"version": "2.3.0",
 	"description": "Ready-to use bundle of linting tools, containing configurations for ESLint, stylelint and markdownlint.",
 	"keywords": [
 		"eslint",
@@ -39,24 +39,28 @@
 		"check-outdated": "npx --yes -- check-outdated --ignore-pre-releases"
 	},
 	"dependencies": {
-		"@typescript-eslint/eslint-plugin": "5.3.0",
-		"@typescript-eslint/parser": "5.3.0",
-		"eslint": "8.2.0",
+		"@typescript-eslint/eslint-plugin": "5.10.1",
+		"@typescript-eslint/parser": "5.10.1",
+		"eslint": "8.7.0",
 		"eslint-import-resolver-typescript": "2.5.0",
 		"eslint-import-resolver-webpack": "0.13.2",
 		"eslint-plugin-eslint-comments": "3.2.0",
-		"eslint-plugin-functional": "4.0.2",
-		"eslint-plugin-import": "2.25.2",
-		"eslint-plugin-jest": "25.2.3",
-		"eslint-plugin-jsdoc": "37.0.3",
+		"eslint-plugin-functional": "4.1.1",
+		"eslint-plugin-import": "2.25.4",
+		"eslint-plugin-jest": "26.0.0",
+		"eslint-plugin-jsdoc": "37.7.0",
+		"eslint-plugin-jsx-a11y": "6.5.1",
 		"eslint-plugin-node": "11.1.0",
-		"eslint-plugin-react-hooks": "4.2.1-alpha-13455d26d-20211104",
-		"eslint-plugin-unicorn": "38.0.0",
-		"markdownlint-cli": "0.29.0",
-		"postcss-scss": "4.0.2",
-		"stylelint": "14.0.1",
+		"eslint-plugin-promise": "6.0.0",
+		"eslint-plugin-react": "7.28.0",
+		"eslint-plugin-react-hooks": "4.3.0",
+		"eslint-plugin-unicorn": "40.1.0",
+		"markdownlint-cli": "0.30.0",
+		"postcss-scss": "4.0.3",
+		"stylelint": "14.3.0",
+		"stylelint-declaration-block-no-ignored-properties": "2.5.0",
 		"stylelint-order": "5.0.0",
-		"stylelint-scss": "4.0.0",
+		"stylelint-scss": "4.1.0",
 		"stylelint-selector-no-empty": "1.0.8",
 		"stylelint-use-logical-spec": "3.2.2"
 	},
@@ -65,9 +69,9 @@
 		"postcss-value-parser": "*"
 	},
 	"devDependencies": {
-		"@types/eslint": "7.28.2",
-		"@types/node": "16.11.6",
+		"@types/eslint": "8.4.1",
+		"@types/node": "17.0.12",
 		"stylelint-find-new-rules": "^3.0.4",
-		"typescript": "4.4.4"
+		"typescript": "4.5.5"
 	}
 }

package/stylelint/index.js

@@ -12,6 +12,7 @@ module.exports = {
 	reportDescriptionlessDisables: true,
 	customSyntax: 'postcss-scss',
 	plugins: [
+		'stylelint-declaration-block-no-ignored-properties',
 		'stylelint-order',
 		'stylelint-scss',
 		'stylelint-selector-no-empty',
@@ -242,6 +243,7 @@ module.exports = {
 		],
 		'property-no-unknown': true,
 		'property-no-vendor-prefix': true,
+		'rule-selector-property-disallowed-list': null,
 		'rule-empty-line-before': [
 			'always-multi-line',
 			{
@@ -315,6 +317,13 @@ module.exports = {
 		'value-list-max-empty-lines': 0,
 		'value-no-vendor-prefix': true,
 
+		/**
+		 * stylelint-declaration-block-no-ignored-properties
+		 *
+		 * @see https://www.npmjs.com/package/stylelint-declaration-block-no-ignored-properties
+		 */
+		'plugin/declaration-block-no-ignored-properties': true,
+
 		/**
 		 * stylelint-high-performance-animation
 		 *
@@ -839,6 +848,7 @@ module.exports = {
 		'scss/at-mixin-pattern': '^[a-z]+(-[a-z]+)*$',
 		'scss/at-rule-conditional-no-parentheses': true,
 		'scss/at-rule-no-unknown': true,
+		'scss/at-use-no-unnamespaced': true,
 		'scss/comment-no-empty': true,
 		'scss/comment-no-loud': true,
 		'scss/declaration-nested-properties-no-divided-groups': true,
@@ -862,6 +872,7 @@ module.exports = {
 		],
 		'scss/dollar-variable-first-in-block': [true, { ignore: ['comments', 'imports'] }],
 		'scss/dollar-variable-no-missing-interpolation': true,
+		'scss/dollar-variable-no-namespaced-assignment': true,
 		'scss/dollar-variable-pattern': '^[a-z]+(-[a-z]+)*$',
 		'scss/double-slash-comment-empty-line-before': [
 			'always',