eslint-config-airbnb-extended 0.3.1 → 0.5.0

package/README.md

@@ -53,3 +53,13 @@ If you're working in a monorepo setup, it's recommended to run the installation
 ### Why did we switch from `import` to `import-x`?
 
 The switch from the `import` ESLint plugin to `import-x` is due to several improvements. `import-x` provides **better TypeScript support**, ensuring more accurate linting for TypeScript projects. It is **actively maintained**, with regular updates and bug fixes, unlike the original plugin. It also has **fewer issues reported on GitHub**, indicating better stability. Additionally, `import-x` offers a **more performant and lightweight version**, reducing linting overhead and improving build performance. These factors make `import-x` a more reliable and efficient choice.
+
+### Why are `plugins` separated from the `config` in this package?
+
+The main reason for separating `plugins` from the config is to avoid a common issue developers face when extending multiple ESLint configurations. Specifically, many run into the error:
+`Config "package": Key "plugins": Cannot redefine plugin "key".`
+Unfortunately, there's no built-in way to resolve this conflict when plugins are directly included within shared configs.
+
+This package combines multiple ESLint configurations (see [Packages Used](https://github.com/NishargShah/eslint-config-airbnb-extended/tree/master/packages/eslint-config-airbnb-extended#packages-used)), and based on experience, it's better to keep `plugins` separate. If you'd like to use the recommended plugins, you can import them directly from the `plugins` export provided by the package.
+
+By doing this, you can safely use this package alongside official ESLint configs without running into plugin redefinition issues.

package/dist/@types/index.d.ts

@@ -0,0 +1,22 @@
+/**
+ * Direct export not allowed, it will increase the size of d.ts
+ */
+export declare const rules: {
+    base: Record<"imports" | "variables" | "bestPractices" | "errors" | "es6" | "node" | "strict" | "style" | "stylistic" | "importsStrict" | "reactStrict", import("eslint").Linter.Config>;
+    react: Record<"react" | "reactA11y" | "reactHooks", import("eslint").Linter.Config>;
+    next: import("eslint").Linter.Config;
+    typescript: Record<"imports" | "base" | "typescriptEslint", import("eslint").Linter.Config>;
+};
+export declare const configs: {
+    base: Record<"all" | "recommended" | "typescript", import("eslint").Linter.Config[]>;
+    react: Record<"all" | "recommended" | "typescript", import("eslint").Linter.Config[]>;
+    next: Record<"all" | "recommended" | "typescript", import("eslint").Linter.Config[]>;
+    node: Record<"all" | "recommended" | "typescript", import("eslint").Linter.Config[]>;
+};
+export declare const plugins: {
+    next: import("eslint").Linter.Config;
+    react: import("eslint").Linter.Config;
+    reactA11y: import("eslint").Linter.Config;
+    reactHooks: import("eslint").Linter.Config;
+    typescriptEslint: import("eslint").Linter.Config;
+};

package/dist/configs/base/index.js

@@ -0,0 +1,26 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const best_practices_1 = __importDefault(require("../../rules/best-practices"));
+const errors_1 = __importDefault(require("../../rules/errors"));
+const es6_1 = __importDefault(require("../../rules/es6"));
+const imports_1 = __importDefault(require("../../rules/imports"));
+const node_1 = __importDefault(require("../../rules/node"));
+const strict_1 = __importDefault(require("../../rules/strict"));
+const style_1 = __importDefault(require("../../rules/style"));
+const stylistic_1 = __importDefault(require("../../rules/stylistic"));
+const variables_1 = __importDefault(require("../../rules/variables"));
+const baseConfig = {
+    bestPractices: best_practices_1.default,
+    errors: errors_1.default,
+    es6: es6_1.default,
+    imports: imports_1.default,
+    node: node_1.default,
+    strict: strict_1.default,
+    style: style_1.default,
+    stylistic: stylistic_1.default,
+    variables: variables_1.default,
+};
+exports.default = baseConfig;

package/dist/{base → configs/base}/recommended.js

@@ -3,9 +3,9 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-const base_1 = __importDefault(require("../base"));
-exports.default = [
-    ...Object.values(base_1.default),
+const index_1 = __importDefault(require("../../configs/base/index"));
+const baseRecommendedConfig = [
+    ...Object.values(index_1.default),
     {
         name: 'airbnb/config/language-configurations',
         languageOptions: {
@@ -14,6 +14,6 @@ exports.default = [
                 sourceType: 'module',
             },
         },
-        rules: {},
     },
 ];
+exports.default = baseRecommendedConfig;

package/dist/configs/index.js

@@ -0,0 +1,34 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const recommended_1 = __importDefault(require("../configs/base/recommended"));
+const recommended_2 = __importDefault(require("../configs/react/recommended"));
+const recommended_3 = __importDefault(require("../configs/typescript/recommended"));
+const next_1 = __importDefault(require("../rules/next"));
+const base = {
+    recommended: recommended_1.default,
+    typescript: recommended_3.default.base,
+    all: [...recommended_1.default, ...recommended_3.default.base],
+};
+const react = {
+    recommended: recommended_2.default,
+    typescript: recommended_3.default.react,
+    all: [...recommended_2.default, ...recommended_3.default.react],
+};
+const next = {
+    recommended: [...react.recommended, next_1.default],
+    typescript: react.typescript,
+    all: [...react.all, next_1.default],
+};
+/**
+ * as is given due to less size of index.d.ts
+ */
+const configs = {
+    base: base,
+    react: react,
+    next: next,
+    node: base,
+};
+exports.default = configs;

package/dist/{react → configs/react}/index.js

@@ -3,11 +3,12 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-const react_1 = __importDefault(require("../rules/react"));
-const react_a11y_1 = __importDefault(require("../rules/react-a11y"));
-const react_hooks_1 = __importDefault(require("../rules/react-hooks"));
-exports.default = {
+const react_1 = __importDefault(require("../../rules/react"));
+const react_a11y_1 = __importDefault(require("../../rules/react-a11y"));
+const react_hooks_1 = __importDefault(require("../../rules/react-hooks"));
+const reactConfig = {
     react: react_1.default,
     reactA11y: react_a11y_1.default,
     reactHooks: react_hooks_1.default,
 };
+exports.default = reactConfig;

package/dist/{react → configs/react}/recommended.js

@@ -3,12 +3,10 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-const recommended_1 = __importDefault(require("../base/recommended"));
-const react_1 = __importDefault(require("../react"));
-const utils_1 = require("../utils");
-exports.default = [
-    ...recommended_1.default,
-    ...Object.values(react_1.default),
+const index_1 = __importDefault(require("../../configs/react/index"));
+const utils_1 = require("../../utils");
+const reactRecommendedConfig = [
+    ...Object.values(index_1.default),
     {
         name: 'airbnb/config/react-settings',
         settings: {
@@ -20,3 +18,4 @@ exports.default = [
         },
     },
 ];
+exports.default = reactRecommendedConfig;

package/dist/configs/typescript/index.js

@@ -0,0 +1,14 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const typescriptBase_1 = __importDefault(require("../../rules/typescript/typescriptBase"));
+const typescriptEslint_1 = __importDefault(require("../../rules/typescript/typescriptEslint"));
+const typescriptImports_1 = __importDefault(require("../../rules/typescript/typescriptImports"));
+const typescriptConfig = {
+    base: typescriptBase_1.default,
+    typescriptEslint: typescriptEslint_1.default,
+    imports: typescriptImports_1.default,
+};
+exports.default = typescriptConfig;

package/dist/{typescript → configs/typescript}/recommended.js

@@ -3,16 +3,16 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-const typescript_1 = __importDefault(require("../typescript"));
-const utils_1 = require("../utils");
-exports.default = {
-    base: typescript_1.default.typescript,
+const index_1 = __importDefault(require("../../configs/typescript/index"));
+const utils_1 = require("../../utils");
+const typescriptRecommendedConfig = {
+    base: Object.values(index_1.default),
     react: [
-        ...typescript_1.default.typescript,
         {
             name: 'airbnb/config/typescript-react',
             rules: {
-                // Append 'tsx' to Airbnb 'react/jsx-filename-extension' rule
+                // only .jsx and .tsx files may have JSX
+                // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/jsx-filename-extension.md
                 'react/jsx-filename-extension': [
                     'error',
                     {
@@ -33,3 +33,4 @@ exports.default = {
         },
     ],
 };
+exports.default = typescriptRecommendedConfig;

package/dist/helpers/getDevDepsList.js

@@ -1,6 +1,5 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.getDevDepsList = void 0;
 const getDevDepsList = (extensions) => [
     'test/**',
     'tests/**',
@@ -28,4 +27,4 @@ const getDevDepsList = (extensions) => [
     `**/eslint.config.{${extensions}}`,
     `**/prettier.config.{${extensions}}`,
 ];
-exports.getDevDepsList = getDevDepsList;
+exports.default = getDevDepsList;

package/dist/index.js

@@ -1,42 +1,16 @@
 "use strict";
+/* eslint-disable import-x/no-rename-default, unicorn/prefer-export-from */
 var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.configs = exports.rules = void 0;
-const base_1 = __importDefault(require("./base"));
-const recommended_1 = __importDefault(require("./base/recommended"));
-const react_1 = __importDefault(require("./react"));
-const recommended_2 = __importDefault(require("./react/recommended"));
-const typescript_1 = __importDefault(require("./typescript"));
-const recommended_3 = __importDefault(require("./typescript/recommended"));
-const importsStrict_1 = __importDefault(require("./rules/importsStrict"));
-const next_1 = __importDefault(require("./rules/next"));
-exports.rules = {
-    base: Object.assign(Object.assign({}, base_1.default), { importsStrict: importsStrict_1.default }),
-    react: react_1.default,
-    next: next_1.default,
-    typescript: typescript_1.default,
-};
-exports.configs = {
-    base: {
-        recommended: recommended_1.default,
-        typescript: recommended_3.default.base,
-        all: [...recommended_1.default, ...recommended_3.default.base],
-    },
-    react: {
-        recommended: recommended_2.default,
-        typescript: recommended_3.default.react,
-        all: [...recommended_2.default, ...recommended_3.default.react],
-    },
-    next: {
-        recommended: [...recommended_2.default, next_1.default],
-        typescript: recommended_3.default.react,
-        all: [...recommended_2.default, ...recommended_3.default.react, next_1.default],
-    },
-    node: {
-        recommended: recommended_1.default,
-        typescript: recommended_3.default.base,
-        all: [...recommended_1.default, ...recommended_3.default.base],
-    },
-};
+exports.plugins = exports.configs = exports.rules = void 0;
+const configs_1 = __importDefault(require("./configs"));
+const plugins_1 = __importDefault(require("./plugins"));
+const rules_1 = __importDefault(require("./rules"));
+/**
+ * Direct export not allowed, it will increase the size of d.ts
+ */
+exports.rules = rules_1.default;
+exports.configs = configs_1.default;
+exports.plugins = plugins_1.default;

package/dist/plugins/index.js

@@ -0,0 +1,21 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const nextPlugin_1 = __importDefault(require("../plugins/nextPlugin"));
+const reactA11yPlugin_1 = __importDefault(require("../plugins/reactA11yPlugin"));
+const reactHooksPlugin_1 = __importDefault(require("../plugins/reactHooksPlugin"));
+const reactPlugin_1 = __importDefault(require("../plugins/reactPlugin"));
+const typescriptEslintPlugin_1 = __importDefault(require("../plugins/typescriptEslintPlugin"));
+/**
+ * as is given due to less size of index.d.ts
+ */
+const plugins = {
+    next: nextPlugin_1.default,
+    react: reactPlugin_1.default,
+    reactA11y: reactA11yPlugin_1.default,
+    reactHooks: reactHooksPlugin_1.default,
+    typescriptEslint: typescriptEslintPlugin_1.default,
+};
+exports.default = plugins;

package/dist/plugins/nextPlugin.js

@@ -0,0 +1,12 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+// @ts-expect-error eslint-plugin-import not working in import
+// eslint-disable-next-line @typescript-eslint/no-require-imports,unicorn/prefer-module
+const ESLintPluginNext = require('@next/eslint-plugin-next');
+const nextPlugin = {
+    name: 'airbnb/config/plugin/next',
+    plugins: {
+        '@next/next': ESLintPluginNext,
+    },
+};
+exports.default = nextPlugin;

package/dist/plugins/reactA11yPlugin.js

@@ -0,0 +1,12 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+// @ts-expect-error eslint-plugin-import not working in import
+// eslint-disable-next-line @typescript-eslint/no-require-imports,unicorn/prefer-module
+const EsLintPluginJSXA11y = require('eslint-plugin-jsx-a11y');
+const reactA11yPlugin = {
+    name: 'airbnb/config/plugin/react-a11y',
+    plugins: {
+        'jsx-a11y': EsLintPluginJSXA11y,
+    },
+};
+exports.default = reactA11yPlugin;

package/dist/plugins/reactHooksPlugin.js

@@ -0,0 +1,44 @@
+"use strict";
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
+Object.defineProperty(exports, "__esModule", { value: true });
+// eslint-disable-next-line import-x/no-namespace
+const EsLintPluginReactHooks = __importStar(require("eslint-plugin-react-hooks"));
+const reactHooksPlugin = {
+    name: 'airbnb/config/plugin/react-hooks',
+    plugins: {
+        'react-hooks': EsLintPluginReactHooks,
+    },
+};
+exports.default = reactHooksPlugin;

package/dist/plugins/reactPlugin.js

@@ -0,0 +1,13 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const eslint_plugin_react_1 = __importDefault(require("eslint-plugin-react"));
+const reactPlugin = {
+    name: 'airbnb/config/plugin/react',
+    plugins: {
+        react: eslint_plugin_react_1.default,
+    },
+};
+exports.default = reactPlugin;

package/dist/plugins/typescriptEslintPlugin.js

@@ -0,0 +1,13 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const typescript_eslint_1 = require("typescript-eslint");
+const typescriptEslintPlugin = {
+    name: 'airbnb/config/plugin/typescript-eslint',
+    plugins: {
+        '@typescript-eslint': typescript_eslint_1.plugin,
+    },
+    languageOptions: {
+        parser: typescript_eslint_1.parser,
+    },
+};
+exports.default = typescriptEslintPlugin;

package/dist/rules/best-practices.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.default = {
+const bestPracticesRules = {
     name: 'airbnb/config/best-practices',
     rules: {
         // enforces getter/setter pairs in objects
@@ -8,13 +8,15 @@ exports.default = {
         'accessor-pairs': 'off',
         // enforces return statements in callbacks of array's methods
         // https://eslint.org/docs/rules/array-callback-return
-        'array-callback-return': ['error', { allowImplicit: true }],
+        'array-callback-return': [
+            'error',
+            {
+                allowImplicit: true,
+            },
+        ],
         // treat var statements as if they were block scoped
         // https://eslint.org/docs/rules/block-scoped-var
         'block-scoped-var': 'error',
-        // specify the maximum cyclomatic complexity allowed in a program
-        // https://eslint.org/docs/rules/complexity
-        complexity: ['off', 20],
         // enforce that class methods use "this"
         // https://eslint.org/docs/rules/class-methods-use-this
         'class-methods-use-this': [
@@ -23,15 +25,23 @@ exports.default = {
                 exceptMethods: [],
             },
         ],
+        // specify the maximum cyclomatic complexity allowed in a program
+        // https://eslint.org/docs/rules/complexity
+        complexity: 'off',
         // require return statements to either always or never specify values
         // https://eslint.org/docs/rules/consistent-return
         'consistent-return': 'error',
         // specify curly brace conventions for all control statements
         // https://eslint.org/docs/rules/curly
-        curly: ['error', 'multi-line'], // multiline
+        curly: ['error', 'multi-line'],
         // require default case in switch statements
         // https://eslint.org/docs/rules/default-case
-        'default-case': ['error', { commentPattern: '^no default$' }],
+        'default-case': [
+            'error',
+            {
+                commentPattern: '^no default$',
+            },
+        ],
         // Enforce default clauses in switch statements to be last
         // https://eslint.org/docs/rules/default-case-last
         'default-case-last': 'error',
@@ -39,13 +49,21 @@ exports.default = {
         'default-param-last': 'error',
         // encourages use of dot notation whenever possible
         // https://eslint.org/docs/rules/dot-notation
-        'dot-notation': ['error', { allowKeywords: true }],
-        // enforces consistent newlines before or after dots
-        // https://eslint.org/docs/rules/dot-location
-        'dot-location': ['error', 'property'],
+        'dot-notation': [
+            'error',
+            {
+                allowKeywords: true,
+            },
+        ],
         // require the use of === and !==
         // https://eslint.org/docs/rules/eqeqeq
-        eqeqeq: ['error', 'always', { null: 'ignore' }],
+        eqeqeq: [
+            'error',
+            'always',
+            {
+                null: 'ignore',
+            },
+        ],
         // Require grouped accessor pairs in object literals and classes
         // https://eslint.org/docs/rules/grouped-accessor-pairs
         'grouped-accessor-pairs': 'error',
@@ -57,7 +75,6 @@ exports.default = {
         'max-classes-per-file': ['error', 1],
         // disallow the use of alert, confirm, and prompt
         // https://eslint.org/docs/rules/no-alert
-        // TODO: enable, semver-major
         'no-alert': 'warn',
         // disallow use of arguments.caller or arguments.callee
         // https://eslint.org/docs/rules/no-caller
@@ -73,7 +90,12 @@ exports.default = {
         'no-div-regex': 'off',
         // disallow else after a return in an if
         // https://eslint.org/docs/rules/no-else-return
-        'no-else-return': ['error', { allowElseIf: false }],
+        'no-else-return': [
+            'error',
+            {
+                allowElseIf: false,
+            },
+        ],
         // disallow empty functions, except for standalone funcs/arrows
         // https://eslint.org/docs/rules/no-empty-function
         'no-empty-function': [
@@ -87,7 +109,6 @@ exports.default = {
         'no-empty-pattern': 'error',
         // Disallow empty static blocks
         // https://eslint.org/docs/latest/rules/no-empty-static-block
-        // TODO: semver-major, enable
         'no-empty-static-block': 'off',
         // disallow comparisons to null without a type-checking operator
         // https://eslint.org/docs/rules/no-eq-null
@@ -107,15 +128,14 @@ exports.default = {
         // disallow fallthrough of case statements
         // https://eslint.org/docs/rules/no-fallthrough
         'no-fallthrough': 'error',
-        // disallow the use of leading or trailing decimal points in numeric literals
-        // https://eslint.org/docs/rules/no-floating-decimal
-        'no-floating-decimal': 'error',
         // disallow reassignments of native objects or read-only globals
         // https://eslint.org/docs/rules/no-global-assign
-        'no-global-assign': ['error', { exceptions: [] }],
-        // deprecated in favor of no-global-assign
-        // https://eslint.org/docs/rules/no-native-reassign
-        'no-native-reassign': 'off',
+        'no-global-assign': [
+            'error',
+            {
+                exceptions: [],
+            },
+        ],
         // disallow implicit type conversions
         // https://eslint.org/docs/rules/no-implicit-coercion
         'no-implicit-coercion': [
@@ -141,7 +161,13 @@ exports.default = {
         'no-iterator': 'error',
         // disallow use of labels for anything other than loops and switches
         // https://eslint.org/docs/rules/no-labels
-        'no-labels': ['error', { allowLoop: false, allowSwitch: false }],
+        'no-labels': [
+            'error',
+            {
+                allowLoop: false,
+                allowSwitch: false,
+            },
+        ],
         // disallow unnecessary nested blocks
         // https://eslint.org/docs/rules/no-lone-blocks
         'no-lone-blocks': 'error',
@@ -159,14 +185,6 @@ exports.default = {
                 detectObjects: false,
             },
         ],
-        // disallow use of multiple spaces
-        // https://eslint.org/docs/rules/no-multi-spaces
-        'no-multi-spaces': [
-            'error',
-            {
-                ignoreEOLComments: false,
-            },
-        ],
         // disallow use of multiline strings
         // https://eslint.org/docs/rules/no-multi-str
         'no-multi-str': 'error',
@@ -277,9 +295,6 @@ exports.default = {
         // disallow use of assignment in return statement
         // https://eslint.org/docs/rules/no-return-assign
         'no-return-assign': ['error', 'always'],
-        // disallow redundant `return await`
-        // https://eslint.org/docs/rules/no-return-await
-        'no-return-await': 'error',
         // disallow use of `javascript:` urls.
         // https://eslint.org/docs/rules/no-script-url
         'no-script-url': 'error',
@@ -334,22 +349,32 @@ exports.default = {
         // disallow use of void operator
         // https://eslint.org/docs/rules/no-void
         'no-void': 'error',
-        // disallow usage of configurable warning terms in comments: e.g. todo
+        // disallow usage of configurable warning terms in comments
         // https://eslint.org/docs/rules/no-warning-comments
-        'no-warning-comments': ['off', { terms: ['todo', 'fixme', 'xxx'], location: 'start' }],
+        'no-warning-comments': [
+            'off',
+            {
+                terms: ['todo', 'fixme', 'remember'],
+                location: 'start',
+            },
+        ],
         // disallow use of the with statement
         // https://eslint.org/docs/rules/no-with
         'no-with': 'error',
         // require using Error objects as Promise rejection reasons
         // https://eslint.org/docs/rules/prefer-promise-reject-errors
-        'prefer-promise-reject-errors': ['error', { allowEmptyReject: true }],
+        'prefer-promise-reject-errors': [
+            'error',
+            {
+                allowEmptyReject: true,
+            },
+        ],
         // Suggest using named capture group in regular expression
         // https://eslint.org/docs/rules/prefer-named-capture-group
         'prefer-named-capture-group': 'off',
         // Prefer Object.hasOwn() over Object.prototype.hasOwnProperty.call()
         // https://eslint.org/docs/rules/prefer-object-has-own
-        // TODO: semver-major: enable thus rule, once eslint v8.5.0 is required
-        'prefer-object-has-own': 'off',
+        'prefer-object-has-own': 'error',
         // https://eslint.org/docs/rules/prefer-regex-literals
         'prefer-regex-literals': [
             'error',
@@ -369,11 +394,9 @@ exports.default = {
         // requires to declare all vars on top of their containing scope
         // https://eslint.org/docs/rules/vars-on-top
         'vars-on-top': 'error',
-        // require immediate function invocation to be wrapped in parentheses
-        // https://eslint.org/docs/rules/wrap-iife.html
-        'wrap-iife': ['error', 'outside', { functionPrototypeMethods: false }],
         // require or disallow Yoda conditions
         // https://eslint.org/docs/rules/yoda
         yoda: 'error',
     },
 };
+exports.default = bestPracticesRules;