eslint 5.6.0 → 5.9.0

package/lib/rules/id-match.js

@@ -11,6 +11,8 @@
 
 module.exports = {
     meta: {
+        type: "suggestion",
+
         docs: {
             description: "require identifiers to match a specified regular expression",
             category: "Stylistic Issues",
@@ -27,6 +29,12 @@ module.exports = {
                 properties: {
                     properties: {
                         type: "boolean"
+                    },
+                    onlyDeclarations: {
+                        type: "boolean"
+                    },
+                    ignoreDestructuring: {
+                        type: "boolean"
                     }
                 }
             }
@@ -36,15 +44,25 @@ module.exports = {
     create(context) {
 
         //--------------------------------------------------------------------------
-        // Helpers
+        // Options
         //--------------------------------------------------------------------------
-
         const pattern = context.options[0] || "^.+$",
             regexp = new RegExp(pattern);
 
         const options = context.options[1] || {},
             properties = !!options.properties,
-            onlyDeclarations = !!options.onlyDeclarations;
+            onlyDeclarations = !!options.onlyDeclarations,
+            ignoreDestructuring = !!options.ignoreDestructuring;
+
+        //--------------------------------------------------------------------------
+        // Helpers
+        //--------------------------------------------------------------------------
+
+        // contains reported nodes to avoid reporting twice on destructuring with shorthand notation
+        const reported = new Map();
+        const ALLOWED_PARENT_TYPES = new Set(["CallExpression", "NewExpression"]);
+        const DECLARATION_TYPES = new Set(["FunctionDeclaration", "VariableDeclarator"]);
+        const IMPORT_TYPES = new Set(["ImportSpecifier", "ImportNamespaceSpecifier", "ImportDefaultSpecifier"]);
 
         /**
          * Checks if a string matches the provided pattern
@@ -56,6 +74,26 @@ module.exports = {
             return !regexp.test(name);
         }
 
+        /**
+         * Checks if a parent of a node is an ObjectPattern.
+         * @param {ASTNode} node The node to check.
+         * @returns {boolean} if the node is inside an ObjectPattern
+         * @private
+         */
+        function isInsideObjectPattern(node) {
+            let { parent } = node;
+
+            while (parent) {
+                if (parent.type === "ObjectPattern") {
+                    return true;
+                }
+
+                parent = parent.parent;
+            }
+
+            return false;
+        }
+
         /**
          * Verifies if we should report an error or not based on the effective
          * parent node and the identifier name.
@@ -64,9 +102,8 @@ module.exports = {
          * @returns {boolean} whether an error should be reported or not
          */
         function shouldReport(effectiveParent, name) {
-            return effectiveParent.type !== "CallExpression" &&
-                effectiveParent.type !== "NewExpression" &&
-                isInvalid(name);
+            return (!onlyDeclarations || DECLARATION_TYPES.has(effectiveParent.type)) &&
+                !ALLOWED_PARENT_TYPES.has(effectiveParent.type) && isInvalid(name);
         }
 
         /**
@@ -76,14 +113,17 @@ module.exports = {
          * @private
          */
         function report(node) {
-            context.report({
-                node,
-                message: "Identifier '{{name}}' does not match the pattern '{{pattern}}'.",
-                data: {
-                    name: node.name,
-                    pattern
-                }
-            });
+            if (!reported.has(node)) {
+                context.report({
+                    node,
+                    message: "Identifier '{{name}}' does not match the pattern '{{pattern}}'.",
+                    data: {
+                        name: node.name,
+                        pattern
+                    }
+                });
+                reported.set(node, true);
+            }
         }
 
         return {
@@ -106,36 +146,70 @@ module.exports = {
                             report(node);
                         }
 
-                    // Report AssignmentExpressions only if they are the left side of the assignment
+                    // Report AssignmentExpressions left side's assigned variable id
                     } else if (effectiveParent.type === "AssignmentExpression" &&
-                        (effectiveParent.right.type !== "MemberExpression" ||
                         effectiveParent.left.type === "MemberExpression" &&
-                        effectiveParent.left.property.name === name)) {
+                        effectiveParent.left.property.name === node.name) {
+                        if (isInvalid(name)) {
+                            report(node);
+                        }
+
+                    // Report AssignmentExpressions only if they are the left side of the assignment
+                    } else if (effectiveParent.type === "AssignmentExpression" && effectiveParent.right.type !== "MemberExpression") {
                         if (isInvalid(name)) {
                             report(node);
                         }
                     }
 
-                } else if (parent.type === "Property") {
+                /*
+                 * Properties have their own rules, and
+                 * AssignmentPattern nodes can be treated like Properties:
+                 * e.g.: const { no_camelcased = false } = bar;
+                 */
+                } else if (parent.type === "Property" || parent.type === "AssignmentPattern") {
+
+                    if (parent.parent && parent.parent.type === "ObjectPattern") {
+                        if (parent.shorthand && parent.value.left && isInvalid(name)) {
+
+                            report(node);
+                        }
+
+                        const assignmentKeyEqualsValue = parent.key.name === parent.value.name;
 
-                    if (!properties || parent.key.name !== name) {
+                        // prevent checking righthand side of destructured object
+                        if (!assignmentKeyEqualsValue && parent.key === node) {
+                            return;
+                        }
+
+                        const valueIsInvalid = parent.value.name && isInvalid(name);
+
+                        // ignore destructuring if the option is set, unless a new identifier is created
+                        if (valueIsInvalid && !(assignmentKeyEqualsValue && ignoreDestructuring)) {
+                            report(node);
+                        }
+                    }
+
+                    // never check properties or always ignore destructuring
+                    if (!properties || (ignoreDestructuring && isInsideObjectPattern(node))) {
                         return;
                     }
 
-                    if (shouldReport(effectiveParent, name)) {
+                    // don't check right hand side of AssignmentExpression to prevent duplicate warnings
+                    if (parent.right !== node && shouldReport(effectiveParent, name)) {
                         report(node);
                     }
 
-                } else {
-                    const isDeclaration = effectiveParent.type === "FunctionDeclaration" || effectiveParent.type === "VariableDeclarator";
+                // Check if it's an import specifier
+                } else if (IMPORT_TYPES.has(parent.type)) {
 
-                    if (onlyDeclarations && !isDeclaration) {
-                        return;
-                    }
-
-                    if (shouldReport(effectiveParent, name)) {
+                    // Report only if the local imported identifier is invalid
+                    if (parent.local && parent.local.name === node.name && isInvalid(name)) {
                         report(node);
                     }
+
+                // Report anything that is invalid that isn't a CallExpression
+                } else if (shouldReport(effectiveParent, name)) {
+                    report(node);
                 }
             }
 

package/lib/rules/implicit-arrow-linebreak.js

@@ -9,13 +9,17 @@
 //------------------------------------------------------------------------------
 module.exports = {
     meta: {
+        type: "layout",
+
         docs: {
             description: "enforce the location of arrow function bodies",
             category: "Stylistic Issues",
             recommended: false,
             url: "https://eslint.org/docs/rules/implicit-arrow-linebreak"
         },
+
         fixable: "whitespace",
+
         schema: [
             {
                 enum: ["beside", "below"]

package/lib/rules/indent-legacy.js

@@ -21,16 +21,19 @@ const astUtils = require("../util/ast-utils");
 /* istanbul ignore next: this rule has known coverage issues, but it's deprecated and shouldn't be updated in the future anyway. */
 module.exports = {
     meta: {
+        type: "layout",
+
         docs: {
             description: "enforce consistent indentation",
             category: "Stylistic Issues",
             recommended: false,
-            replacedBy: ["indent"],
             url: "https://eslint.org/docs/rules/indent-legacy"
         },
 
         deprecated: true,
 
+        replacedBy: ["indent"],
+
         fixable: "whitespace",
 
         schema: [

package/lib/rules/indent.js

@@ -490,6 +490,8 @@ const ELEMENT_LIST_SCHEMA = {
 
 module.exports = {
     meta: {
+        type: "layout",
+
         docs: {
             description: "enforce consistent indentation",
             category: "Stylistic Issues",

package/lib/rules/init-declarations.js

@@ -44,6 +44,8 @@ function isInitialized(node) {
 
 module.exports = {
     meta: {
+        type: "suggestion",
+
         docs: {
             description: "require or disallow initialization in variable declarations",
             category: "Variables",

package/lib/rules/jsx-quotes.js

@@ -38,6 +38,8 @@ const QUOTE_SETTINGS = {
 
 module.exports = {
     meta: {
+        type: "layout",
+
         docs: {
             description: "enforce the consistent use of either double or single quotes in JSX attributes",
             category: "Stylistic Issues",

package/lib/rules/key-spacing.js

@@ -128,6 +128,8 @@ const messages = {
 
 module.exports = {
     meta: {
+        type: "layout",
+
         docs: {
             description: "enforce consistent spacing between keys and values in object literal properties",
             category: "Stylistic Issues",
@@ -360,10 +362,9 @@ module.exports = {
          */
         function isKeyValueProperty(property) {
             return !(
-                property.method ||
+                (property.method ||
                 property.shorthand ||
-                property.kind !== "init" ||
-                property.type !== "Property" // Could be "ExperimentalSpreadProperty" or "SpreadElement"
+                property.kind !== "init" || property.type !== "Property") // Could be "ExperimentalSpreadProperty" or "SpreadElement"
             );
         }
 

package/lib/rules/keyword-spacing.js

@@ -65,6 +65,8 @@ function isCloseParenOfTemplate(token) {
 
 module.exports = {
     meta: {
+        type: "layout",
+
         docs: {
             description: "enforce consistent spacing before and after keywords",
             category: "Stylistic Issues",

package/lib/rules/line-comment-position.js

@@ -12,6 +12,8 @@ const astUtils = require("../util/ast-utils");
 
 module.exports = {
     meta: {
+        type: "layout",
+
         docs: {
             description: "enforce position of line comments",
             category: "Stylistic Issues",

package/lib/rules/linebreak-style.js

@@ -17,6 +17,8 @@ const astUtils = require("../util/ast-utils");
 
 module.exports = {
     meta: {
+        type: "layout",
+
         docs: {
             description: "enforce consistent linebreak style",
             category: "Stylistic Issues",

package/lib/rules/lines-around-comment.js

@@ -52,6 +52,8 @@ function getCommentLineNums(comments) {
 
 module.exports = {
     meta: {
+        type: "layout",
+
         docs: {
             description: "require empty lines around comments",
             category: "Stylistic Issues",

package/lib/rules/lines-around-directive.js

@@ -14,13 +14,15 @@ const astUtils = require("../util/ast-utils");
 
 module.exports = {
     meta: {
+        type: "layout",
+
         docs: {
             description: "require or disallow newlines around directives",
             category: "Stylistic Issues",
             recommended: false,
-            replacedBy: ["padding-line-between-statements"],
             url: "https://eslint.org/docs/rules/lines-around-directive"
         },
+
         schema: [{
             oneOf: [
                 {
@@ -41,8 +43,10 @@ module.exports = {
                 }
             ]
         }],
+
         fixable: "whitespace",
-        deprecated: true
+        deprecated: true,
+        replacedBy: ["padding-line-between-statements"]
     },
 
     create(context) {

package/lib/rules/lines-between-class-members.js

@@ -12,6 +12,8 @@ const astUtils = require("../util/ast-utils");
 
 module.exports = {
     meta: {
+        type: "layout",
+
         docs: {
             description: "require or disallow an empty line between class members",
             category: "Stylistic Issues",

package/lib/rules/max-classes-per-file.js

@@ -15,18 +15,22 @@
 
 module.exports = {
     meta: {
+        type: "suggestion",
+
         docs: {
             description: "enforce a maximum number of classes per file",
             category: "Best Practices",
             recommended: false,
             url: "https://eslint.org/docs/rules/max-classes-per-file"
         },
+
         schema: [
             {
                 type: "integer",
                 minimum: 1
             }
         ],
+
         messages: {
             maximumExceeded: "Number of classes per file must not exceed {{ max }}"
         }

package/lib/rules/max-depth.js

@@ -11,6 +11,8 @@
 
 module.exports = {
     meta: {
+        type: "suggestion",
+
         docs: {
             description: "enforce a maximum depth that blocks can be nested",
             category: "Stylistic Issues",

package/lib/rules/max-len.js

@@ -65,6 +65,8 @@ const OPTIONS_OR_INTEGER_SCHEMA = {
 
 module.exports = {
     meta: {
+        type: "layout",
+
         docs: {
             description: "enforce a maximum line length",
             category: "Stylistic Issues",

package/lib/rules/max-lines-per-function.js

@@ -69,6 +69,8 @@ function getCommentLineNumbers(comments) {
 
 module.exports = {
     meta: {
+        type: "suggestion",
+
         docs: {
             description: "enforce a maximum number of line of code in a function",
             category: "Stylistic Issues",

package/lib/rules/max-lines.js

@@ -17,6 +17,8 @@ const astUtils = require("../util/ast-utils");
 
 module.exports = {
     meta: {
+        type: "suggestion",
+
         docs: {
             description: "enforce a maximum number of lines per file",
             category: "Stylistic Issues",

package/lib/rules/max-nested-callbacks.js

@@ -11,6 +11,8 @@
 
 module.exports = {
     meta: {
+        type: "suggestion",
+
         docs: {
             description: "enforce a maximum depth that callbacks can be nested",
             category: "Stylistic Issues",

package/lib/rules/max-params.js

@@ -19,6 +19,8 @@ const astUtils = require("../util/ast-utils");
 
 module.exports = {
     meta: {
+        type: "suggestion",
+
         docs: {
             description: "enforce a maximum number of parameters in function definitions",
             category: "Stylistic Issues",

package/lib/rules/max-statements-per-line.js

@@ -16,6 +16,8 @@ const astUtils = require("../util/ast-utils");
 
 module.exports = {
     meta: {
+        type: "layout",
+
         docs: {
             description: "enforce a maximum number of statements allowed per line",
             category: "Stylistic Issues",

package/lib/rules/max-statements.js

@@ -19,6 +19,8 @@ const astUtils = require("../util/ast-utils");
 
 module.exports = {
     meta: {
+        type: "suggestion",
+
         docs: {
             description: "enforce a maximum number of statements allowed in function blocks",
             category: "Stylistic Issues",

package/lib/rules/multiline-comment-style.js

@@ -12,12 +12,15 @@ const astUtils = require("../util/ast-utils");
 
 module.exports = {
     meta: {
+        type: "suggestion",
+
         docs: {
             description: "enforce a particular style for multiline comments",
             category: "Stylistic Issues",
             recommended: false,
             url: "https://eslint.org/docs/rules/multiline-comment-style"
         },
+
         fixable: "whitespace",
         schema: [{ enum: ["starred-block", "separate-lines", "bare-block"] }]
     },

package/lib/rules/multiline-ternary.js

@@ -13,12 +13,15 @@ const astUtils = require("../util/ast-utils");
 
 module.exports = {
     meta: {
+        type: "layout",
+
         docs: {
             description: "enforce newlines between operands of ternary expressions",
             category: "Stylistic Issues",
             recommended: false,
             url: "https://eslint.org/docs/rules/multiline-ternary"
         },
+
         schema: [
             {
                 enum: ["always", "always-multiline", "never"]

package/lib/rules/new-cap.js

@@ -74,6 +74,8 @@ function calculateCapIsNewExceptions(config) {
 
 module.exports = {
     meta: {
+        type: "suggestion",
+
         docs: {
             description: "require constructor names to begin with a capital letter",
             category: "Stylistic Issues",

package/lib/rules/new-parens.js

@@ -21,6 +21,8 @@ const astUtils = require("../util/ast-utils");
 
 module.exports = {
     meta: {
+        type: "layout",
+
         docs: {
             description: "require parentheses when invoking a constructor with no arguments",
             category: "Stylistic Issues",
@@ -29,7 +31,6 @@ module.exports = {
         },
 
         schema: [],
-
         fixable: "code"
     },
 

package/lib/rules/newline-after-var.js

@@ -18,11 +18,12 @@ const astUtils = require("../util/ast-utils");
 
 module.exports = {
     meta: {
+        type: "layout",
+
         docs: {
             description: "require or disallow an empty line after variable declarations",
             category: "Stylistic Issues",
             recommended: false,
-            replacedBy: ["padding-line-between-statements"],
             url: "https://eslint.org/docs/rules/newline-after-var"
         },
 
@@ -34,7 +35,9 @@ module.exports = {
 
         fixable: "whitespace",
 
-        deprecated: true
+        deprecated: true,
+
+        replacedBy: ["padding-line-between-statements"]
     },
 
     create(context) {

package/lib/rules/newline-before-return.js

@@ -11,16 +11,19 @@
 
 module.exports = {
     meta: {
+        type: "layout",
+
         docs: {
             description: "require an empty line before `return` statements",
             category: "Stylistic Issues",
             recommended: false,
-            replacedBy: ["padding-line-between-statements"],
             url: "https://eslint.org/docs/rules/newline-before-return"
         },
+
         fixable: "whitespace",
         schema: [],
-        deprecated: true
+        deprecated: true,
+        replacedBy: ["padding-line-between-statements"]
     },
 
     create(context) {

package/lib/rules/newline-per-chained-call.js

@@ -14,13 +14,17 @@ const astUtils = require("../util/ast-utils");
 
 module.exports = {
     meta: {
+        type: "layout",
+
         docs: {
             description: "require a newline after each call in a method chain",
             category: "Stylistic Issues",
             recommended: false,
             url: "https://eslint.org/docs/rules/newline-per-chained-call"
         },
+
         fixable: "whitespace",
+
         schema: [{
             type: "object",
             properties: {

package/lib/rules/no-alert.js

@@ -74,6 +74,8 @@ function isGlobalThisReferenceOrGlobalWindow(scope, node) {
 
 module.exports = {
     meta: {
+        type: "suggestion",
+
         docs: {
             description: "disallow the use of `alert`, `confirm`, and `prompt`",
             category: "Best Practices",

package/lib/rules/no-array-constructor.js

@@ -11,6 +11,8 @@
 
 module.exports = {
     meta: {
+        type: "suggestion",
+
         docs: {
             description: "disallow `Array` constructors",
             category: "Stylistic Issues",

package/lib/rules/no-async-promise-executor.js

@@ -10,12 +10,15 @@
 
 module.exports = {
     meta: {
+        type: "problem",
+
         docs: {
             description: "disallow using an async function as a Promise executor",
             category: "Possible Errors",
             recommended: false,
             url: "https://eslint.org/docs/rules/no-async-promise-executor"
         },
+
         fixable: null,
         schema: []
     },

package/lib/rules/no-await-in-loop.js

@@ -55,13 +55,17 @@ function isLooped(node, parent) {
 
 module.exports = {
     meta: {
+        type: "problem",
+
         docs: {
             description: "disallow `await` inside of loops",
             category: "Possible Errors",
             recommended: false,
             url: "https://eslint.org/docs/rules/no-await-in-loop"
         },
+
         schema: [],
+
         messages: {
             unexpectedAwait: "Unexpected `await` inside a loop."
         }