eslint-config-airbnb-extended 0.4.0 → 0.5.0

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

@@ -2,7 +2,7 @@
  * 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" | "importsStrict", import("eslint").Linter.Config>;
+    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>;

package/dist/configs/base/index.js

@@ -10,6 +10,7 @@ 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,
@@ -19,6 +20,7 @@ const baseConfig = {
     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/configs/base/recommended.js

@@ -14,7 +14,6 @@ const baseRecommendedConfig = [
                 sourceType: 'module',
             },
         },
-        rules: {},
     },
 ];
 exports.default = baseRecommendedConfig;

package/dist/configs/index.js

@@ -15,7 +15,7 @@ const base = {
 const react = {
     recommended: recommended_2.default,
     typescript: recommended_3.default.react,
-    all: [...recommended_1.default, ...recommended_2.default, ...recommended_3.default.react],
+    all: [...recommended_2.default, ...recommended_3.default.react],
 };
 const next = {
     recommended: [...react.recommended, next_1.default],

package/dist/configs/typescript/recommended.js

@@ -8,11 +8,11 @@ const utils_1 = require("../../utils");
 const typescriptRecommendedConfig = {
     base: Object.values(index_1.default),
     react: [
-        ...Object.values(index_1.default),
         {
             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',
                     {

package/dist/plugins/reactHooksPlugin.js

@@ -40,12 +40,5 @@ const reactHooksPlugin = {
     plugins: {
         'react-hooks': EsLintPluginReactHooks,
     },
-    languageOptions: {
-        parserOptions: {
-            ecmaFeatures: {
-                jsx: true,
-            },
-        },
-    },
 };
 exports.default = reactHooksPlugin;

package/dist/plugins/typescriptEslintPlugin.js

@@ -8,9 +8,6 @@ const typescriptEslintPlugin = {
     },
     languageOptions: {
         parser: typescript_eslint_1.parser,
-        parserOptions: {
-            projectService: true,
-        },
     },
 };
 exports.default = typescriptEslintPlugin;

package/dist/rules/best-practices.js

@@ -8,13 +8,15 @@ const bestPracticesRules = {
         '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 @@ const bestPracticesRules = {
                 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 @@ const bestPracticesRules = {
         '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 @@ const bestPracticesRules = {
         '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 @@ const bestPracticesRules = {
         '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 @@ const bestPracticesRules = {
         '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 @@ const bestPracticesRules = {
         // 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 @@ const bestPracticesRules = {
         '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 @@ const bestPracticesRules = {
                 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 @@ const bestPracticesRules = {
         // 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 @@ const bestPracticesRules = {
         // 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,9 +394,6 @@ const bestPracticesRules = {
         // 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',

package/dist/rules/errors.js

@@ -8,7 +8,12 @@ const errorsRules = {
         'for-direction': 'error',
         // Enforces that a return statement is present in property getters
         // https://eslint.org/docs/rules/getter-return
-        'getter-return': ['error', { allowImplicit: true }],
+        'getter-return': [
+            'error',
+            {
+                allowImplicit: true,
+            },
+        ],
         // disallow using an async function as a Promise executor
         // https://eslint.org/docs/rules/no-async-promise-executor
         'no-async-promise-executor': 'error',
@@ -24,7 +29,6 @@ const errorsRules = {
         'no-console': 'warn',
         // Disallows expressions where the operation doesn't affect the value
         // https://eslint.org/docs/rules/no-constant-binary-expression
-        // TODO: semver-major, enable
         'no-constant-binary-expression': 'off',
         // disallow use of constant expressions in conditions
         'no-constant-condition': 'warn',
@@ -50,21 +54,6 @@ const errorsRules = {
         // disallow double-negation boolean casts in a boolean context
         // https://eslint.org/docs/rules/no-extra-boolean-cast
         'no-extra-boolean-cast': 'error',
-        // disallow unnecessary parentheses
-        // https://eslint.org/docs/rules/no-extra-parens
-        'no-extra-parens': [
-            'off',
-            'all',
-            {
-                conditionalAssign: true,
-                nestedBinaryExpressions: false,
-                returnAssign: false,
-                ignoreJSX: 'all', // delegate to eslint-plugin-react
-                enforceForArrowConditionals: false,
-            },
-        ],
-        // disallow unnecessary semicolons
-        'no-extra-semi': 'error',
         // disallow overwriting functions written as function declarations
         'no-func-assign': 'error',
         // https://eslint.org/docs/rules/no-import-assign
@@ -85,14 +74,13 @@ const errorsRules = {
         'no-obj-calls': 'error',
         // Disallow new operators with global non-constructor functions
         // https://eslint.org/docs/latest/rules/no-new-native-nonconstructor
-        // TODO: semver-major, enable
-        'no-new-native-nonconstructor': 'off',
-        // Disallow returning values from Promise executor functions
-        // https://eslint.org/docs/rules/no-promise-executor-return
-        'no-promise-executor-return': 'error',
+        'no-new-native-nonconstructor': 'error',
         // disallow use of Object.prototypes builtins directly
         // https://eslint.org/docs/rules/no-prototype-builtins
         'no-prototype-builtins': 'error',
+        // Disallow returning values from Promise executor functions
+        // https://eslint.org/docs/rules/no-promise-executor-return
+        'no-promise-executor-return': 'error',
         // disallow multiple spaces in a regular expression literal
         'no-regex-spaces': 'error',
         // Disallow returning values from setters
@@ -124,29 +112,32 @@ const errorsRules = {
         'no-unsafe-negation': 'error',
         // disallow use of optional chaining in contexts where the undefined value is not allowed
         // https://eslint.org/docs/rules/no-unsafe-optional-chaining
-        'no-unsafe-optional-chaining': ['error', { disallowArithmeticOperators: true }],
+        'no-unsafe-optional-chaining': [
+            'error',
+            {
+                disallowArithmeticOperators: true,
+            },
+        ],
         // Disallow Unused Private Class Members
         // https://eslint.org/docs/rules/no-unused-private-class-members
-        // TODO: enable once eslint 7 is dropped (which is semver-major)
         'no-unused-private-class-members': 'off',
         // Disallow useless backreferences in regular expressions
         // https://eslint.org/docs/rules/no-useless-backreference
         'no-useless-backreference': 'error',
-        // disallow negation of the left operand of an in expression
-        // deprecated in favor of no-unsafe-negation
-        'no-negated-in-lhs': 'off',
         // Disallow assignments that can lead to race conditions due to usage of await or yield
         // https://eslint.org/docs/rules/require-atomic-updates
         // note: not enabled because it is very buggy
         'require-atomic-updates': 'off',
         // disallow comparisons with the value NaN
         'use-isnan': 'error',
-        // ensure JSDoc comments are valid
-        // https://eslint.org/docs/rules/valid-jsdoc
-        'valid-jsdoc': 'off',
         // ensure that the results of typeof are compared against a valid string
         // https://eslint.org/docs/rules/valid-typeof
-        'valid-typeof': ['error', { requireStringLiterals: true }],
+        'valid-typeof': [
+            'error',
+            {
+                requireStringLiterals: true,
+            },
+        ],
     },
 };
 exports.default = errorsRules;

package/dist/rules/es6.js

@@ -28,28 +28,11 @@ const es6Rules = {
                 requireReturnForObjectLiteral: false,
             },
         ],
-        // require parens in arrow function arguments
-        // https://eslint.org/docs/rules/arrow-parens
-        'arrow-parens': ['error', 'always'],
-        // require space before/after arrow function's arrow
-        // https://eslint.org/docs/rules/arrow-spacing
-        'arrow-spacing': ['error', { before: true, after: true }],
         // verify super() callings in constructors
         'constructor-super': 'error',
-        // enforce the spacing around the * in generator functions
-        // https://eslint.org/docs/rules/generator-star-spacing
-        'generator-star-spacing': ['error', { before: false, after: true }],
         // disallow modifying variables of class declarations
         // https://eslint.org/docs/rules/no-class-assign
         'no-class-assign': 'error',
-        // disallow arrow functions where they could be confused with comparisons
-        // https://eslint.org/docs/rules/no-confusing-arrow
-        'no-confusing-arrow': [
-            'error',
-            {
-                allowParens: true,
-            },
-        ],
         // disallow modifying variables that are declared using const
         'no-const-assign': 'error',
         // disallow duplicate class members
@@ -59,9 +42,6 @@ const es6Rules = {
         // https://eslint.org/docs/rules/no-duplicate-imports
         // replaced by https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/no-duplicates.md
         'no-duplicate-imports': 'off',
-        // disallow symbol constructor
-        // https://eslint.org/docs/rules/no-new-symbol
-        'no-new-symbol': 'error',
         // Disallow specified names in exports
         // https://eslint.org/docs/rules/no-restricted-exports
         'no-restricted-exports': [
@@ -150,9 +130,6 @@ const es6Rules = {
         // disallow parseInt() in favor of binary, octal, and hexadecimal literals
         // https://eslint.org/docs/rules/prefer-numeric-literals
         'prefer-numeric-literals': 'error',
-        // suggest using Reflect methods where applicable
-        // https://eslint.org/docs/rules/prefer-reflect
-        'prefer-reflect': 'off',
         // use rest parameters instead of arguments
         // https://eslint.org/docs/rules/prefer-rest-params
         'prefer-rest-params': 'error',
@@ -165,9 +142,6 @@ const es6Rules = {
         // disallow generator functions that do not have yield
         // https://eslint.org/docs/rules/require-yield
         'require-yield': 'error',
-        // enforce spacing between object rest-spread
-        // https://eslint.org/docs/rules/rest-spread-spacing
-        'rest-spread-spacing': ['error', 'never'],
         // import sorting
         // https://eslint.org/docs/rules/sort-imports
         'sort-imports': [
@@ -182,12 +156,6 @@ const es6Rules = {
         // require a Symbol description
         // https://eslint.org/docs/rules/symbol-description
         'symbol-description': 'error',
-        // enforce usage of spacing in template strings
-        // https://eslint.org/docs/rules/template-curly-spacing
-        'template-curly-spacing': 'error',
-        // enforce spacing around the * in yield* expressions
-        // https://eslint.org/docs/rules/yield-star-spacing
-        'yield-star-spacing': ['error', 'after'],
     },
 };
 exports.default = es6Rules;

package/dist/rules/imports.js

@@ -52,7 +52,7 @@ const importsRules = Object.assign(Object.assign({}, eslint_plugin_import_x_1.fl
         // disallow non-import statements appearing before import statements
         // https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/imports-first.md
         // @deprecated: use `import-x/first`
-        'import-x/imports-first': 'off',
+        // 'import-x/imports-first': 'off',
         // Forbid modules to have too many dependencies
         // https://github.com/un-ts/eslint-plugin-import-x/blob/master/docs/rules/max-dependencies.md
         'import-x/max-dependencies': 'off',

package/dist/rules/index.js

@@ -8,7 +8,8 @@ const react_1 = __importDefault(require("../configs/react"));
 const typescript_1 = __importDefault(require("../configs/typescript"));
 const importsStrict_1 = __importDefault(require("../rules/importsStrict"));
 const next_1 = __importDefault(require("../rules/next"));
-const baseRules = Object.assign(Object.assign({}, base_1.default), { importsStrict: importsStrict_1.default });
+const reactStrict_1 = __importDefault(require("../rules/reactStrict"));
+const baseRules = Object.assign(Object.assign({}, base_1.default), { importsStrict: importsStrict_1.default, reactStrict: reactStrict_1.default });
 /**
  * as is given due to less size of index.d.ts
  */