eslint-config-airbnb-extended 0.3.1 → 0.5.0

package/dist/rules/reactStrict.js

@@ -0,0 +1,86 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const reactStrictRules = {
+    name: 'airbnb/config/react/strict',
+    rules: {
+        // Enforces consistent naming for boolean props
+        // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/boolean-prop-naming.md
+        'react/boolean-prop-naming': [
+            'error',
+            {
+                validateNested: true,
+            },
+        ],
+        // This rule enforces onChange or readonly attribute for checked property of input elements.
+        // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/checked-requires-onchange-or-readonly.md
+        'react/checked-requires-onchange-or-readonly': 'error',
+        // Enforce a specific function type for function components
+        // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/function-component-definition.md
+        'react/function-component-definition': [
+            'error',
+            {
+                namedComponents: 'arrow-function',
+                unnamedComponents: 'arrow-function',
+            },
+        ],
+        // Ensure destructuring and symmetric naming of useState hook value and setter variables
+        // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/hook-use-state.md
+        'react/hook-use-state': [
+            'error',
+            {
+                allowDestructuredState: true,
+            },
+        ],
+        // Ensures inline tags are not rendered without spaces between them
+        // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/jsx-child-element-spacing.md
+        'react/jsx-child-element-spacing': 'warn',
+        // Enforce shorthand or standard form for React fragments
+        // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/jsx-fragments.md
+        'react/jsx-fragments': ['error', 'element'],
+        // Validate JSX has key prop when in array or iterator
+        // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/jsx-key.md
+        // Turned off because it has too many false positives
+        'react/jsx-key': [
+            'error',
+            {
+                checkFragmentShorthand: true,
+                checkKeyMustBeforeSpread: false,
+                warnOnDuplicates: true,
+            },
+        ],
+        // Prevent problematic leaked values from being rendered
+        // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/jsx-no-leaked-render.md
+        'react/jsx-no-leaked-render': 'error',
+        // Enforce propTypes declarations alphabetical sorting
+        // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/sort-prop-types.md
+        'react/jsx-sort-props': [
+            'error',
+            {
+                callbacksLast: false,
+                shorthandFirst: true,
+                shorthandLast: false,
+                multiline: 'last',
+                ignoreCase: true,
+                noSortAlphabetically: false,
+                reservedFirst: ['key', 'ref'],
+                locale: 'auto',
+            },
+        ],
+        // Prevent usage of UNSAFE_ methods
+        // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/no-unsafe.md
+        'react/no-unsafe': 'error',
+        // Require stateless functions when not using lifecycle methods, setState or ref
+        // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/prefer-stateless-function.md
+        'react/prefer-stateless-function': [
+            'error',
+            {
+                ignorePureComponents: false,
+            },
+        ],
+        // Prevent missing props validation in a React component definition
+        // https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/prop-types.md
+        // Disabling it because prop-types are deprecated
+        'react/prop-types': 'off',
+    },
+};
+exports.default = reactStrictRules;

package/dist/rules/strict.js

@@ -1,9 +1,10 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.default = {
+const strictRules = {
     name: 'airbnb/config/strict',
     rules: {
         // babel inserts `'use strict';` for us
         strict: ['error', 'never'],
     },
 };
+exports.default = strictRules;

package/dist/rules/style.js

@@ -1,27 +1,18 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.default = {
+const styleRules = {
     name: 'airbnb/config/style',
     rules: {
-        // enforce line breaks after opening and before closing array brackets
-        // https://eslint.org/docs/rules/array-bracket-newline
-        // TODO: enable? semver-major
-        'array-bracket-newline': ['off', 'consistent'], // object option alternative: { multiline: true, minItems: 3 }
-        // enforce line breaks between array elements
-        // https://eslint.org/docs/rules/array-element-newline
-        // TODO: enable? semver-major
-        'array-element-newline': ['off', { multiline: true, minItems: 3 }],
-        // enforce spacing inside array brackets
-        'array-bracket-spacing': ['error', 'never'],
-        // enforce spacing inside single-line blocks
-        // https://eslint.org/docs/rules/block-spacing
-        'block-spacing': ['error', 'always'],
-        // enforce one true brace style
-        'brace-style': ['error', '1tbs', { allowSingleLine: true }],
         // require camel case names
-        camelcase: ['error', { properties: 'never', ignoreDestructuring: false }],
+        camelcase: [
+            'error',
+            {
+                properties: 'never',
+                ignoreDestructuring: false,
+            },
+        ],
         // enforce or disallow capitalization of the first letter of a comment
-        // https://eslint.org/docs/rules/capitalized-comments
+        // https://eslint.org/docs/latest/rules/capitalized-comments
         'capitalized-comments': [
             'off',
             'never',
@@ -38,53 +29,11 @@ exports.default = {
                 },
             },
         ],
-        // require trailing commas in multiline object literals
-        'comma-dangle': [
-            'error',
-            {
-                arrays: 'always-multiline',
-                objects: 'always-multiline',
-                imports: 'always-multiline',
-                exports: 'always-multiline',
-                functions: 'always-multiline',
-            },
-        ],
-        // enforce spacing before and after comma
-        'comma-spacing': ['error', { before: false, after: true }],
-        // enforce one true comma style
-        'comma-style': [
-            'error',
-            'last',
-            {
-                exceptions: {
-                    ArrayExpression: false,
-                    ArrayPattern: false,
-                    ArrowFunctionExpression: false,
-                    CallExpression: false,
-                    FunctionDeclaration: false,
-                    FunctionExpression: false,
-                    ImportDeclaration: false,
-                    ObjectExpression: false,
-                    ObjectPattern: false,
-                    VariableDeclaration: false,
-                    NewExpression: false,
-                },
-            },
-        ],
-        // disallow padding inside computed properties
-        'computed-property-spacing': ['error', 'never'],
         // enforces consistent naming when capturing the current execution context
         'consistent-this': 'off',
-        // enforce newline at the end of file, with no multiple empty lines
-        'eol-last': ['error', 'always'],
-        // https://eslint.org/docs/rules/function-call-argument-newline
-        'function-call-argument-newline': ['error', 'consistent'],
-        // enforce spacing between functions and their invocations
-        // https://eslint.org/docs/rules/func-call-spacing
-        'func-call-spacing': ['error', 'never'],
         // requires function names to match the name of the variable or property to which they are
         // assigned
-        // https://eslint.org/docs/rules/func-name-matching
+        // https://eslint.org/docs/latest/rules/func-name-matching
         'func-name-matching': [
             'off',
             'always',
@@ -94,119 +43,20 @@ exports.default = {
             },
         ],
         // require function expressions to have a name
-        // https://eslint.org/docs/rules/func-names
+        // https://eslint.org/docs/latest/rules/func-names
         'func-names': 'warn',
         // enforces use of function declarations or expressions
-        // https://eslint.org/docs/rules/func-style
+        // https://eslint.org/docs/latest/rules/func-style
         // TODO: enable
         'func-style': ['off', 'expression'],
-        // require line breaks inside function parentheses if there are line breaks between parameters
-        // https://eslint.org/docs/rules/function-paren-newline
-        'function-paren-newline': ['error', 'multiline-arguments'],
         // disallow specified identifiers
-        // https://eslint.org/docs/rules/id-denylist
+        // https://eslint.org/docs/latest/rules/id-denylist
         'id-denylist': 'off',
         // this option enforces minimum and maximum identifier lengths
         // (variable names, property names etc.)
         'id-length': 'off',
         // require identifiers to match the provided regular expression
         'id-match': 'off',
-        // Enforce the location of arrow function bodies with implicit returns
-        // https://eslint.org/docs/rules/implicit-arrow-linebreak
-        'implicit-arrow-linebreak': ['error', 'beside'],
-        // this option sets a specific tab width for your code
-        // https://eslint.org/docs/rules/indent
-        indent: [
-            'error',
-            2,
-            {
-                SwitchCase: 1,
-                VariableDeclarator: 1,
-                outerIIFEBody: 1,
-                // MemberExpression: null,
-                FunctionDeclaration: {
-                    parameters: 1,
-                    body: 1,
-                },
-                FunctionExpression: {
-                    parameters: 1,
-                    body: 1,
-                },
-                CallExpression: {
-                    arguments: 1,
-                },
-                ArrayExpression: 1,
-                ObjectExpression: 1,
-                ImportDeclaration: 1,
-                flatTernaryExpressions: false,
-                // list derived from https://github.com/benjamn/ast-types/blob/HEAD/def/jsx.js
-                ignoredNodes: [
-                    'JSXElement',
-                    'JSXElement > *',
-                    'JSXAttribute',
-                    'JSXIdentifier',
-                    'JSXNamespacedName',
-                    'JSXMemberExpression',
-                    'JSXSpreadAttribute',
-                    'JSXExpressionContainer',
-                    'JSXOpeningElement',
-                    'JSXClosingElement',
-                    'JSXFragment',
-                    'JSXOpeningFragment',
-                    'JSXClosingFragment',
-                    'JSXText',
-                    'JSXEmptyExpression',
-                    'JSXSpreadChild',
-                ],
-                ignoreComments: false,
-            },
-        ],
-        // specify whether double or single quotes should be used in JSX attributes
-        // https://eslint.org/docs/rules/jsx-quotes
-        'jsx-quotes': ['off', 'prefer-double'],
-        // enforces spacing between keys and values in object literal properties
-        'key-spacing': ['error', { beforeColon: false, afterColon: true }],
-        // require a space before & after certain keywords
-        'keyword-spacing': [
-            'error',
-            {
-                before: true,
-                after: true,
-                overrides: {
-                    return: { after: true },
-                    throw: { after: true },
-                    case: { after: true },
-                },
-            },
-        ],
-        // enforce position of line comments
-        // https://eslint.org/docs/rules/line-comment-position
-        // TODO: enable?
-        'line-comment-position': [
-            'off',
-            {
-                position: 'above',
-                ignorePattern: '',
-                applyDefaultPatterns: true,
-            },
-        ],
-        // disallow mixed 'LF' and 'CRLF' as linebreaks
-        // https://eslint.org/docs/rules/linebreak-style
-        'linebreak-style': ['error', 'unix'],
-        // require or disallow an empty line between class members
-        // https://eslint.org/docs/rules/lines-between-class-members
-        'lines-between-class-members': ['error', 'always', { exceptAfterSingleLine: false }],
-        // enforces empty lines around comments
-        'lines-around-comment': 'off',
-        // require or disallow newlines around directives
-        // https://eslint.org/docs/rules/lines-around-directive
-        'lines-around-directive': [
-            'error',
-            {
-                before: 'always',
-                after: 'always',
-            },
-        ],
         // Require or disallow logical assignment logical operator shorthand
         // https://eslint.org/docs/latest/rules/logical-assignment-operators
         // TODO, semver-major: enable
@@ -219,22 +69,8 @@ exports.default = {
         ],
         // specify the maximum depth that blocks can be nested
         'max-depth': ['off', 4],
-        // specify the maximum length of a line in your program
-        // https://eslint.org/docs/rules/max-len
-        'max-len': [
-            'error',
-            100,
-            2,
-            {
-                ignoreUrls: true,
-                ignoreComments: false,
-                ignoreRegExpLiterals: true,
-                ignoreStrings: true,
-                ignoreTemplateLiterals: true,
-            },
-        ],
         // specify the max number of lines in a file
-        // https://eslint.org/docs/rules/max-lines
+        // https://eslint.org/docs/latest/rules/max-lines
         'max-lines': [
             'off',
             {
@@ -244,7 +80,7 @@ exports.default = {
             },
         ],
         // enforce a maximum function length
-        // https://eslint.org/docs/rules/max-lines-per-function
+        // https://eslint.org/docs/latest/rules/max-lines-per-function
         'max-lines-per-function': [
             'off',
             {
@@ -260,16 +96,6 @@ exports.default = {
         'max-params': ['off', 3],
         // specify the maximum number of statement allowed in a function
         'max-statements': ['off', 10],
-        // restrict the number of statements per line
-        // https://eslint.org/docs/rules/max-statements-per-line
-        'max-statements-per-line': ['off', { max: 1 }],
-        // enforce a particular style for multiline comments
-        // https://eslint.org/docs/rules/multiline-comment-style
-        'multiline-comment-style': ['off', 'starred-block'],
-        // require multiline ternary
-        // https://eslint.org/docs/rules/multiline-ternary
-        // TODO: enable?
-        'multiline-ternary': ['off', 'never'],
         // require a capital letter for constructors
         'new-cap': [
             'error',
@@ -280,71 +106,32 @@ exports.default = {
                 capIsNewExceptions: ['Immutable.Map', 'Immutable.Set', 'Immutable.List'],
             },
         ],
-        // disallow the omission of parentheses when invoking a constructor with no arguments
-        // https://eslint.org/docs/rules/new-parens
-        'new-parens': 'error',
-        // allow/disallow an empty newline after var statement
-        'newline-after-var': 'off',
-        // https://eslint.org/docs/rules/newline-before-return
-        'newline-before-return': 'off',
-        // enforces new line after each method call in the chain to make it
-        // more readable and easy to maintain
-        // https://eslint.org/docs/rules/newline-per-chained-call
-        'newline-per-chained-call': ['error', { ignoreChainWithDepth: 4 }],
         // disallow use of the Array constructor
         'no-array-constructor': 'error',
         // disallow use of bitwise operators
-        // https://eslint.org/docs/rules/no-bitwise
+        // https://eslint.org/docs/latest/rules/no-bitwise
         'no-bitwise': 'error',
         // disallow use of the continue statement
-        // https://eslint.org/docs/rules/no-continue
+        // https://eslint.org/docs/latest/rules/no-continue
         'no-continue': 'error',
         // disallow comments inline after code
         'no-inline-comments': 'off',
         // disallow if as the only statement in an else block
-        // https://eslint.org/docs/rules/no-lonely-if
+        // https://eslint.org/docs/latest/rules/no-lonely-if
         'no-lonely-if': 'error',
-        // disallow un-paren'd mixes of different operators
-        // https://eslint.org/docs/rules/no-mixed-operators
-        'no-mixed-operators': [
-            'error',
-            {
-                // the list of arithmetic groups disallows mixing `%` and `**`
-                // with other arithmetic operators.
-                groups: [
-                    ['%', '**'],
-                    ['%', '+'],
-                    ['%', '-'],
-                    ['%', '*'],
-                    ['%', '/'],
-                    ['/', '*'],
-                    ['&', '|', '<<', '>>', '>>>'],
-                    ['==', '!=', '===', '!=='],
-                    ['&&', '||'],
-                ],
-                allowSamePrecedence: false,
-            },
-        ],
-        // disallow mixed spaces and tabs for indentation
-        'no-mixed-spaces-and-tabs': 'error',
         // disallow use of chained assignment expressions
-        // https://eslint.org/docs/rules/no-multi-assign
+        // https://eslint.org/docs/latest/rules/no-multi-assign
         'no-multi-assign': ['error'],
-        // disallow multiple empty lines, only one newline at the end, and no new lines at the beginning
-        // https://eslint.org/docs/rules/no-multiple-empty-lines
-        'no-multiple-empty-lines': ['error', { max: 1, maxBOF: 0, maxEOF: 0 }],
         // disallow negated conditions
-        // https://eslint.org/docs/rules/no-negated-condition
+        // https://eslint.org/docs/latest/rules/no-negated-condition
         'no-negated-condition': 'off',
         // disallow nested ternary expressions
         'no-nested-ternary': 'error',
-        // disallow use of the Object constructor
-        'no-new-object': 'error',
         // disallow use of unary operators, ++ and --
-        // https://eslint.org/docs/rules/no-plusplus
+        // https://eslint.org/docs/latest/rules/no-plusplus
         'no-plusplus': 'error',
         // disallow certain syntax forms
-        // https://eslint.org/docs/rules/no-restricted-syntax
+        // https://eslint.org/docs/latest/rules/no-restricted-syntax
         'no-restricted-syntax': [
             'error',
             {
@@ -364,23 +151,10 @@ exports.default = {
                 message: '`with` is disallowed in strict mode because it makes code impossible to predict and optimize.',
             },
         ],
-        // disallow space between function identifier and application
-        // deprecated in favor of func-call-spacing
-        'no-spaced-func': 'off',
-        // disallow tab characters entirely
-        'no-tabs': 'error',
         // disallow the use of ternary operators
         'no-ternary': 'off',
-        // disallow trailing whitespace at the end of lines
-        'no-trailing-spaces': [
-            'error',
-            {
-                skipBlankLines: false,
-                ignoreComments: false,
-            },
-        ],
         // disallow dangling underscores in identifiers
-        // https://eslint.org/docs/rules/no-underscore-dangle
+        // https://eslint.org/docs/latest/rules/no-underscore-dangle
         'no-underscore-dangle': [
             'error',
             {
@@ -392,139 +166,38 @@ exports.default = {
         ],
         // disallow the use of Boolean literals in conditional expressions
         // also, prefer `a || b` over `a ? a : b`
-        // https://eslint.org/docs/rules/no-unneeded-ternary
-        'no-unneeded-ternary': ['error', { defaultAssignment: false }],
-        // disallow whitespace before properties
-        // https://eslint.org/docs/rules/no-whitespace-before-property
-        'no-whitespace-before-property': 'error',
-        // enforce the location of single-line statements
-        // https://eslint.org/docs/rules/nonblock-statement-body-position
-        'nonblock-statement-body-position': ['error', 'beside', { overrides: {} }],
-        // require padding inside curly braces
-        'object-curly-spacing': ['error', 'always'],
-        // enforce line breaks between braces
-        // https://eslint.org/docs/rules/object-curly-newline
-        'object-curly-newline': [
+        // https://eslint.org/docs/latest/rules/no-unneeded-ternary
+        'no-unneeded-ternary': [
             'error',
             {
-                ObjectExpression: { minProperties: 4, multiline: true, consistent: true },
-                ObjectPattern: { minProperties: 4, multiline: true, consistent: true },
-                ImportDeclaration: { minProperties: 4, multiline: true, consistent: true },
-                ExportDeclaration: { minProperties: 4, multiline: true, consistent: true },
-            },
-        ],
-        // enforce "same line" or "multiple line" on object properties.
-        // https://eslint.org/docs/rules/object-property-newline
-        'object-property-newline': [
-            'error',
-            {
-                allowAllPropertiesOnSameLine: true,
+                defaultAssignment: false,
             },
         ],
         // allow just one var statement per function
         'one-var': ['error', 'never'],
-        // require a newline around variable declaration
-        // https://eslint.org/docs/rules/one-var-declaration-per-line
-        'one-var-declaration-per-line': ['error', 'always'],
         // require assignment operator shorthand where possible or prohibit it entirely
-        // https://eslint.org/docs/rules/operator-assignment
+        // https://eslint.org/docs/latest/rules/operator-assignment
         'operator-assignment': ['error', 'always'],
-        // Requires operator at the beginning of the line in multiline statements
-        // https://eslint.org/docs/rules/operator-linebreak
-        'operator-linebreak': ['error', 'before', { overrides: { '=': 'none' } }],
-        // disallow padding within blocks
-        'padded-blocks': [
-            'error',
-            {
-                blocks: 'never',
-                classes: 'never',
-                switches: 'never',
-            },
-            {
-                allowSingleLineBlocks: true,
-            },
-        ],
-        // Require or disallow padding lines between statements
-        // https://eslint.org/docs/rules/padding-line-between-statements
-        'padding-line-between-statements': 'off',
         // Disallow the use of Math.pow in favor of the ** operator
-        // https://eslint.org/docs/rules/prefer-exponentiation-operator
+        // https://eslint.org/docs/latest/rules/prefer-exponentiation-operator
         'prefer-exponentiation-operator': 'error',
         // Prefer use of an object spread over Object.assign
-        // https://eslint.org/docs/rules/prefer-object-spread
+        // https://eslint.org/docs/latest/rules/prefer-object-spread
         'prefer-object-spread': 'error',
-        // require quotes around object literal property names
-        // https://eslint.org/docs/rules/quote-props.html
-        'quote-props': ['error', 'as-needed', { keywords: false, unnecessary: true, numbers: false }],
-        // specify whether double or single quotes should be used
-        quotes: ['error', 'single', { avoidEscape: true }],
-        // do not require jsdoc
-        // https://eslint.org/docs/rules/require-jsdoc
-        'require-jsdoc': 'off',
-        // require or disallow use of semicolons instead of ASI
-        semi: ['error', 'always'],
-        // enforce spacing before and after semicolons
-        'semi-spacing': ['error', { before: false, after: true }],
-        // Enforce location of semicolons
-        // https://eslint.org/docs/rules/semi-style
-        'semi-style': ['error', 'last'],
         // requires object keys to be sorted
-        'sort-keys': ['off', 'asc', { caseSensitive: false, natural: true }],
-        // sort variables within the same declaration block
-        'sort-vars': 'off',
-        // require or disallow space before blocks
-        'space-before-blocks': 'error',
-        // require or disallow space before function opening parenthesis
-        // https://eslint.org/docs/rules/space-before-function-paren
-        'space-before-function-paren': [
-            'error',
-            {
-                anonymous: 'always',
-                named: 'never',
-                asyncArrow: 'always',
-            },
-        ],
-        // require or disallow spaces inside parentheses
-        'space-in-parens': ['error', 'never'],
-        // require spaces around operators
-        'space-infix-ops': 'error',
-        // Require or disallow spaces before/after unary operators
-        // https://eslint.org/docs/rules/space-unary-ops
-        'space-unary-ops': [
-            'error',
-            {
-                words: true,
-                nonwords: false,
-                overrides: {},
-            },
-        ],
-        // require or disallow a space immediately following the // or /* in a comment
-        // https://eslint.org/docs/rules/spaced-comment
-        'spaced-comment': [
-            'error',
-            'always',
+        'sort-keys': [
+            'off',
+            'asc',
             {
-                line: {
-                    exceptions: ['-', '+'],
-                    markers: ['=', '!', '/'], // space here to support sprockets directives, slash for TS /// comments
-                },
-                block: {
-                    exceptions: ['-', '+'],
-                    markers: ['=', '!', ':', '::'], // space here to support sprockets directives and flow comment types
-                    balanced: true,
-                },
+                caseSensitive: false,
+                natural: true,
             },
         ],
-        // Enforce spacing around colons of switch statements
-        // https://eslint.org/docs/rules/switch-colon-spacing
-        'switch-colon-spacing': ['error', { after: true, before: false }],
-        // Require or disallow spacing between template tags and their literals
-        // https://eslint.org/docs/rules/template-tag-spacing
-        'template-tag-spacing': ['error', 'never'],
+        // sort variables within the same declaration block
+        'sort-vars': 'off',
         // require or disallow the Unicode Byte Order Mark
-        // https://eslint.org/docs/rules/unicode-bom
+        // https://eslint.org/docs/latest/rules/unicode-bom
         'unicode-bom': ['error', 'never'],
-        // require regex literals to be wrapped in parentheses
-        'wrap-regex': 'off',
     },
 };
+exports.default = styleRules;