eslint 6.5.1 → 6.6.0

package/lib/rules/no-confusing-arrow.js

@@ -14,7 +14,7 @@ const astUtils = require("./utils/ast-utils.js");
 
 /**
  * Checks whether or not a node is a conditional expression.
- * @param {ASTNode} node - node to test
+ * @param {ASTNode} node node to test
  * @returns {boolean} `true` if the node is a conditional expression.
  */
 function isConditional(node) {
@@ -59,7 +59,7 @@ module.exports = {
 
         /**
          * Reports if an arrow function contains an ambiguous conditional.
-         * @param {ASTNode} node - A node to check and report.
+         * @param {ASTNode} node A node to check and report.
          * @returns {void}
          */
         function checkArrowFunc(node) {

package/lib/rules/no-console.js

@@ -54,8 +54,7 @@ module.exports = {
 
         /**
          * Checks whether the given reference is 'console' or not.
-         *
-         * @param {eslint-scope.Reference} reference - The reference to check.
+         * @param {eslint-scope.Reference} reference The reference to check.
          * @returns {boolean} `true` if the reference is 'console'.
          */
         function isConsole(reference) {
@@ -67,8 +66,7 @@ module.exports = {
         /**
          * Checks whether the property name of the given MemberExpression node
          * is allowed by options or not.
-         *
-         * @param {ASTNode} node - The MemberExpression node to check.
+         * @param {ASTNode} node The MemberExpression node to check.
          * @returns {boolean} `true` if the property name of the node is allowed.
          */
         function isAllowed(node) {
@@ -80,8 +78,7 @@ module.exports = {
         /**
          * Checks whether the given reference is a member access which is not
          * allowed by options or not.
-         *
-         * @param {eslint-scope.Reference} reference - The reference to check.
+         * @param {eslint-scope.Reference} reference The reference to check.
          * @returns {boolean} `true` if the reference is a member access which
          *      is not allowed by options.
          */
@@ -98,8 +95,7 @@ module.exports = {
 
         /**
          * Reports the given reference as a violation.
-         *
-         * @param {eslint-scope.Reference} reference - The reference to report.
+         * @param {eslint-scope.Reference} reference The reference to report.
          * @returns {void}
          */
         function report(reference) {

package/lib/rules/no-const-assign.js

@@ -33,7 +33,7 @@ module.exports = {
 
         /**
          * Finds and reports references that are non initializer and writable.
-         * @param {Variable} variable - A variable to check.
+         * @param {Variable} variable A variable to check.
          * @returns {void}
          */
         function checkVariable(variable) {

package/lib/rules/no-dupe-args.js

@@ -35,7 +35,7 @@ module.exports = {
 
         /**
          * Checks whether or not a given definition is a parameter's.
-         * @param {eslint-scope.DefEntry} def - A definition to check.
+         * @param {eslint-scope.DefEntry} def A definition to check.
          * @returns {boolean} `true` if the definition is a parameter's.
          */
         function isParameter(def) {

package/lib/rules/no-dupe-class-members.js

@@ -32,8 +32,8 @@ module.exports = {
 
         /**
          * Gets state of a given member name.
-         * @param {string} name - A name of a member.
-         * @param {boolean} isStatic - A flag which specifies that is a static member.
+         * @param {string} name A name of a member.
+         * @param {boolean} isStatic A flag which specifies that is a static member.
          * @returns {Object} A state of a given member name.
          *   - retv.init {boolean} A flag which shows the name is declared as normal member.
          *   - retv.get {boolean} A flag which shows the name is declared as getter.
@@ -55,8 +55,7 @@ module.exports = {
 
         /**
          * Gets the name text of a given node.
-         *
-         * @param {ASTNode} node - A node to get the name.
+         * @param {ASTNode} node A node to get the name.
          * @returns {string} The name text of the node.
          */
         function getName(node) {

package/lib/rules/no-dupe-keys.js

@@ -23,9 +23,10 @@ const SET_KIND = /^(?:init|set)$/u;
  */
 class ObjectInfo {
 
+    // eslint-disable-next-line jsdoc/require-description
     /**
-     * @param {ObjectInfo|null} upper - The information of the outer object.
-     * @param {ASTNode} node - The ObjectExpression node of this information.
+     * @param {ObjectInfo|null} upper The information of the outer object.
+     * @param {ASTNode} node The ObjectExpression node of this information.
      */
     constructor(upper, node) {
         this.upper = upper;
@@ -35,7 +36,7 @@ class ObjectInfo {
 
     /**
      * Gets the information of the given Property node.
-     * @param {ASTNode} node - The Property node to get.
+     * @param {ASTNode} node The Property node to get.
      * @returns {{get: boolean, set: boolean}} The information of the property.
      */
     getPropertyInfo(node) {
@@ -49,7 +50,7 @@ class ObjectInfo {
 
     /**
      * Checks whether the given property has been defined already or not.
-     * @param {ASTNode} node - The Property node to check.
+     * @param {ASTNode} node The Property node to check.
      * @returns {boolean} `true` if the property has been defined.
      */
     isPropertyDefined(node) {
@@ -63,7 +64,7 @@ class ObjectInfo {
 
     /**
      * Defines the given property.
-     * @param {ASTNode} node - The Property node to define.
+     * @param {ASTNode} node The Property node to define.
      * @returns {void}
      */
     defineProperty(node) {

package/lib/rules/no-duplicate-imports.js

@@ -10,8 +10,7 @@
 
 /**
  * Returns the name of the module imported or re-exported.
- *
- * @param {ASTNode} node - A node to get.
+ * @param {ASTNode} node A node to get.
  * @returns {string} the name of the module, or empty string if no name.
  */
 function getValue(node) {
@@ -24,12 +23,11 @@ function getValue(node) {
 
 /**
  * Checks if the name of the import or export exists in the given array, and reports if so.
- *
- * @param {RuleContext} context - The ESLint rule context object.
- * @param {ASTNode} node - A node to get.
- * @param {string} value - The name of the imported or exported module.
- * @param {string[]} array - The array containing other imports or exports in the file.
- * @param {string} messageId - A messageId to be reported after the name of the module
+ * @param {RuleContext} context The ESLint rule context object.
+ * @param {ASTNode} node A node to get.
+ * @param {string} value The name of the imported or exported module.
+ * @param {string[]} array The array containing other imports or exports in the file.
+ * @param {string} messageId A messageId to be reported after the name of the module
  *
  * @returns {void} No return value
  */
@@ -47,16 +45,15 @@ function checkAndReport(context, node, value, array, messageId) {
 
 /**
  * @callback nodeCallback
- * @param {ASTNode} node - A node to handle.
+ * @param {ASTNode} node A node to handle.
  */
 
 /**
  * Returns a function handling the imports of a given file
- *
- * @param {RuleContext} context - The ESLint rule context object.
- * @param {boolean} includeExports - Whether or not to check for exports in addition to imports.
- * @param {string[]} importsInFile - The array containing other imports in the file.
- * @param {string[]} exportsInFile - The array containing other exports in the file.
+ * @param {RuleContext} context The ESLint rule context object.
+ * @param {boolean} includeExports Whether or not to check for exports in addition to imports.
+ * @param {string[]} importsInFile The array containing other imports in the file.
+ * @param {string[]} exportsInFile The array containing other exports in the file.
  *
  * @returns {nodeCallback} A function passed to ESLint to handle the statement.
  */
@@ -78,10 +75,9 @@ function handleImports(context, includeExports, importsInFile, exportsInFile) {
 
 /**
  * Returns a function handling the exports of a given file
- *
- * @param {RuleContext} context - The ESLint rule context object.
- * @param {string[]} importsInFile - The array containing other imports in the file.
- * @param {string[]} exportsInFile - The array containing other exports in the file.
+ * @param {RuleContext} context The ESLint rule context object.
+ * @param {string[]} importsInFile The array containing other imports in the file.
+ * @param {string[]} exportsInFile The array containing other exports in the file.
  *
  * @returns {nodeCallback} A function passed to ESLint to handle the statement.
  */

package/lib/rules/no-else-return.js

@@ -58,7 +58,6 @@ module.exports = {
          *
          * This is not a generic function. In particular, it is assumed that the scope is a function scope or
          * a function's inner scope, and that the names can be valid identifiers in the given scope.
-         *
          * @param {string[]} names Array of variable names.
          * @param {eslint-scope.Scope} scope Function scope or a function's inner scope.
          * @returns {boolean} True if all names can be safely declared, false otherwise.
@@ -134,7 +133,6 @@ module.exports = {
 
         /**
          * Checks whether the removal of `else` and its braces is safe from variable name collisions.
-         *
          * @param {Node} node The 'else' node.
          * @param {eslint-scope.Scope} scope The scope in which the node and the whole 'if' statement is.
          * @returns {boolean} True if it is safe, false otherwise.
@@ -171,7 +169,6 @@ module.exports = {
 
         /**
          * Display the context report if rule is violated
-         *
          * @param {Node} node The 'else' node
          * @returns {void}
          */
@@ -255,7 +252,6 @@ module.exports = {
 
         /**
          * Check to see if the node is a ReturnStatement
-         *
          * @param {Node} node The node being evaluated
          * @returns {boolean} True if node is a return
          */
@@ -267,7 +263,6 @@ module.exports = {
          * Naive return checking, does not iterate through the whole
          * BlockStatement because we make the assumption that the ReturnStatement
          * will be the last node in the body of the BlockStatement.
-         *
          * @param {Node} node The consequent/alternate node
          * @returns {boolean} True if it has a return
          */
@@ -284,7 +279,6 @@ module.exports = {
         /**
          * Check to see if the node is valid for evaluation,
          * meaning it has an else.
-         *
          * @param {Node} node The node being evaluated
          * @returns {boolean} True if the node is valid
          */
@@ -296,7 +290,6 @@ module.exports = {
          * If the consequent is an IfStatement, check to see if it has an else
          * and both its consequent and alternate path return, meaning this is
          * a nested case of rule violation.  If-Else not considered currently.
-         *
          * @param {Node} node The consequent node
          * @returns {boolean} True if this is a nested rule violation
          */
@@ -309,7 +302,6 @@ module.exports = {
          * Check the consequent/body node to make sure it is not
          * a ReturnStatement or an IfStatement that returns on both
          * code paths.
-         *
          * @param {Node} node The consequent or body node
          * @returns {boolean} `true` if it is a Return/If node that always returns.
          */

package/lib/rules/no-empty-function.js

@@ -28,8 +28,7 @@ const ALLOW_OPTIONS = Object.freeze([
 
 /**
  * Gets the kind of a given function node.
- *
- * @param {ASTNode} node - A function node to get. This is one of
+ * @param {ASTNode} node A function node to get. This is one of
  *      an ArrowFunctionExpression, a FunctionDeclaration, or a
  *      FunctionExpression.
  * @returns {string} The kind of the function. This is one of "functions",
@@ -130,8 +129,7 @@ module.exports = {
          * - Not allowed by options.
          * - The body is empty.
          * - The body doesn't have any comments.
-         *
-         * @param {ASTNode} node - A function node to report. This is one of
+         * @param {ASTNode} node A function node to report. This is one of
          *      an ArrowFunctionExpression, a FunctionDeclaration, or a
          *      FunctionExpression.
          * @returns {void}

package/lib/rules/no-eval.js

@@ -22,9 +22,8 @@ const candidatesOfGlobalObject = Object.freeze([
 
 /**
  * Checks a given node is a Identifier node of the specified name.
- *
- * @param {ASTNode} node - A node to check.
- * @param {string} name - A name to check.
+ * @param {ASTNode} node A node to check.
+ * @param {string} name A name to check.
  * @returns {boolean} `true` if the node is a Identifier node of the name.
  */
 function isIdentifier(node, name) {
@@ -33,9 +32,8 @@ function isIdentifier(node, name) {
 
 /**
  * Checks a given node is a Literal node of the specified string value.
- *
- * @param {ASTNode} node - A node to check.
- * @param {string} name - A name to check.
+ * @param {ASTNode} node A node to check.
+ * @param {string} name A name to check.
  * @returns {boolean} `true` if the node is a Literal node of the name.
  */
 function isConstant(node, name) {
@@ -57,9 +55,8 @@ function isConstant(node, name) {
 /**
  * Checks a given node is a MemberExpression node which has the specified name's
  * property.
- *
- * @param {ASTNode} node - A node to check.
- * @param {string} name - A name to check.
+ * @param {ASTNode} node A node to check.
+ * @param {string} name A name to check.
  * @returns {boolean} `true` if the node is a MemberExpression node which has
  *      the specified name's property
  */
@@ -113,8 +110,7 @@ module.exports = {
          *
          * This is used in order to check whether or not `this` binding is a
          * reference to the global object.
-         *
-         * @param {ASTNode} node - A node of the scope. This is one of Program,
+         * @param {ASTNode} node A node of the scope. This is one of Program,
          *      FunctionDeclaration, FunctionExpression, and ArrowFunctionExpression.
          * @returns {void}
          */
@@ -132,7 +128,6 @@ module.exports = {
 
         /**
          * Pops a variable scope from the stack.
-         *
          * @returns {void}
          */
         function exitVarScope() {
@@ -148,8 +143,7 @@ module.exports = {
          * The location of the report is always `eval` `Identifier` (or possibly
          * `Literal`). The type of the report is `CallExpression` if the parent is
          * `CallExpression`. Otherwise, it's the given node type.
-         *
-         * @param {ASTNode} node - A node to report.
+         * @param {ASTNode} node A node to report.
          * @returns {void}
          */
         function report(node) {
@@ -171,8 +165,7 @@ module.exports = {
 
         /**
          * Reports accesses of `eval` via the global object.
-         *
-         * @param {eslint-scope.Scope} globalScope - The global scope.
+         * @param {eslint-scope.Scope} globalScope The global scope.
          * @returns {void}
          */
         function reportAccessingEvalViaGlobalObject(globalScope) {
@@ -205,8 +198,7 @@ module.exports = {
 
         /**
          * Reports all accesses of `eval` (excludes direct calls to eval).
-         *
-         * @param {eslint-scope.Scope} globalScope - The global scope.
+         * @param {eslint-scope.Scope} globalScope The global scope.
          * @returns {void}
          */
         function reportAccessingEval(globalScope) {

package/lib/rules/no-ex-assign.js

@@ -33,7 +33,7 @@ module.exports = {
 
         /**
          * Finds and reports references that are non initializer and writable.
-         * @param {Variable} variable - A variable to check.
+         * @param {Variable} variable A variable to check.
          * @returns {void}
          */
         function checkVariable(variable) {

package/lib/rules/no-extra-bind.js

@@ -47,7 +47,6 @@ module.exports = {
          * Checks if a node is free of side effects.
          *
          * This check is stricter than it needs to be, in order to keep the implementation simple.
-         *
          * @param {ASTNode} node A node to check.
          * @returns {boolean} True if the node is known to be side-effect free, false otherwise.
          */
@@ -57,8 +56,7 @@ module.exports = {
 
         /**
          * Reports a given function node.
-         *
-         * @param {ASTNode} node - A node to report. This is a FunctionExpression or
+         * @param {ASTNode} node A node to report. This is a FunctionExpression or
          *      an ArrowFunctionExpression.
          * @returns {void}
          */
@@ -90,8 +88,7 @@ module.exports = {
          * method.
          *
          * e.g. `(function() {}.bind(foo))`
-         *
-         * @param {ASTNode} node - A node to report. This is a FunctionExpression or
+         * @param {ASTNode} node A node to report. This is a FunctionExpression or
          *      an ArrowFunctionExpression.
          * @returns {boolean} `true` if the node is the callee of `.bind()` method.
          */
@@ -113,8 +110,7 @@ module.exports = {
 
         /**
          * Adds a scope information object to the stack.
-         *
-         * @param {ASTNode} node - A node to add. This node is a FunctionExpression
+         * @param {ASTNode} node A node to add. This node is a FunctionExpression
          *      or a FunctionDeclaration node.
          * @returns {void}
          */
@@ -130,8 +126,7 @@ module.exports = {
          * Removes the scope information object from the top of the stack.
          * At the same time, this reports the function node if the function has
          * `.bind()` and the `this` keywords found.
-         *
-         * @param {ASTNode} node - A node to remove. This node is a
+         * @param {ASTNode} node A node to remove. This node is a
          *      FunctionExpression or a FunctionDeclaration node.
          * @returns {void}
          */
@@ -146,8 +141,7 @@ module.exports = {
         /**
          * Reports a given arrow function if the function is callee of `.bind()`
          * method.
-         *
-         * @param {ASTNode} node - A node to report. This node is an
+         * @param {ASTNode} node A node to report. This node is an
          *      ArrowFunctionExpression.
          * @returns {void}
          */
@@ -159,7 +153,6 @@ module.exports = {
 
         /**
          * Set the mark as the `this` keyword was found in this scope.
-         *
          * @returns {void}
          */
         function markAsThisFound() {

package/lib/rules/no-extra-boolean-cast.js

@@ -49,7 +49,6 @@ module.exports = {
 
         /**
          * Check if a node is in a context where its value would be coerced to a boolean at runtime.
-         *
          * @param {ASTNode} node The node
          * @param {ASTNode} parent Its parent
          * @returns {boolean} If it is in a boolean context
@@ -67,7 +66,6 @@ module.exports = {
 
         /**
          * Check if a node has comments inside.
-         *
          * @param {ASTNode} node The node to check.
          * @returns {boolean} `true` if it has comments inside.
          */

package/lib/rules/no-extra-label.js

@@ -40,8 +40,7 @@ module.exports = {
 
         /**
          * Creates a new scope with a breakable statement.
-         *
-         * @param {ASTNode} node - A node to create. This is a BreakableStatement.
+         * @param {ASTNode} node A node to create. This is a BreakableStatement.
          * @returns {void}
          */
         function enterBreakableStatement(node) {
@@ -54,7 +53,6 @@ module.exports = {
 
         /**
          * Removes the top scope of the stack.
-         *
          * @returns {void}
          */
         function exitBreakableStatement() {
@@ -66,8 +64,7 @@ module.exports = {
          *
          * This ignores it if the body is a breakable statement.
          * In this case it's handled in the `enterBreakableStatement` function.
-         *
-         * @param {ASTNode} node - A node to create. This is a LabeledStatement.
+         * @param {ASTNode} node A node to create. This is a LabeledStatement.
          * @returns {void}
          */
         function enterLabeledStatement(node) {
@@ -85,8 +82,7 @@ module.exports = {
          *
          * This ignores it if the body is a breakable statement.
          * In this case it's handled in the `exitBreakableStatement` function.
-         *
-         * @param {ASTNode} node - A node. This is a LabeledStatement.
+         * @param {ASTNode} node A node. This is a LabeledStatement.
          * @returns {void}
          */
         function exitLabeledStatement(node) {
@@ -97,8 +93,7 @@ module.exports = {
 
         /**
          * Reports a given control node if it's unnecessary.
-         *
-         * @param {ASTNode} node - A node. This is a BreakStatement or a
+         * @param {ASTNode} node A node. This is a BreakStatement or a
          *      ContinueStatement.
          * @returns {void}
          */

package/lib/rules/no-extra-parens.js

@@ -50,7 +50,8 @@ module.exports = {
                                 returnAssign: { type: "boolean" },
                                 ignoreJSX: { enum: ["none", "all", "single-line", "multi-line"] },
                                 enforceForArrowConditionals: { type: "boolean" },
-                                enforceForSequenceExpressions: { type: "boolean" }
+                                enforceForSequenceExpressions: { type: "boolean" },
+                                enforceForNewInMemberExpressions: { type: "boolean" }
                             },
                             additionalProperties: false
                         }
@@ -80,6 +81,8 @@ module.exports = {
             context.options[1].enforceForArrowConditionals === false;
         const IGNORE_SEQUENCE_EXPRESSIONS = ALL_NODES && context.options[1] &&
             context.options[1].enforceForSequenceExpressions === false;
+        const IGNORE_NEW_IN_MEMBER_EXPR = ALL_NODES && context.options[1] &&
+            context.options[1].enforceForNewInMemberExpressions === false;
 
         const PRECEDENCE_OF_ASSIGNMENT_EXPR = precedence({ type: "AssignmentExpression" });
         const PRECEDENCE_OF_UPDATE_EXPR = precedence({ type: "UpdateExpression" });
@@ -88,7 +91,7 @@ module.exports = {
 
         /**
          * Determines if this rule should be enforced for a node given the current configuration.
-         * @param {ASTNode} node - The node to be checked.
+         * @param {ASTNode} node The node to be checked.
          * @returns {boolean} True if the rule should be enforced for this node.
          * @private
          */
@@ -127,7 +130,7 @@ module.exports = {
 
         /**
          * Determines if a node is surrounded by parentheses.
-         * @param {ASTNode} node - The node to be checked.
+         * @param {ASTNode} node The node to be checked.
          * @returns {boolean} True if the node is parenthesised.
          * @private
          */
@@ -137,7 +140,7 @@ module.exports = {
 
         /**
          * Determines if a node is surrounded by parentheses twice.
-         * @param {ASTNode} node - The node to be checked.
+         * @param {ASTNode} node The node to be checked.
          * @returns {boolean} True if the node is doubly parenthesised.
          * @private
          */
@@ -147,7 +150,7 @@ module.exports = {
 
         /**
          * Determines if a node is surrounded by (potentially) invalid parentheses.
-         * @param {ASTNode} node - The node to be checked.
+         * @param {ASTNode} node The node to be checked.
          * @returns {boolean} True if the node is incorrectly parenthesised.
          * @private
          */
@@ -158,7 +161,7 @@ module.exports = {
         /**
          * Determines if a node that is expected to be parenthesised is surrounded by
          * (potentially) invalid extra parentheses.
-         * @param {ASTNode} node - The node to be checked.
+         * @param {ASTNode} node The node to be checked.
          * @returns {boolean} True if the node is has an unexpected extra pair of parentheses.
          * @private
          */
@@ -168,7 +171,7 @@ module.exports = {
 
         /**
          * Determines if a node test expression is allowed to have a parenthesised assignment
-         * @param {ASTNode} node - The node to be checked.
+         * @param {ASTNode} node The node to be checked.
          * @returns {boolean} True if the assignment can be parenthesised.
          * @private
          */
@@ -178,7 +181,7 @@ module.exports = {
 
         /**
          * Determines if a node is in a return statement
-         * @param {ASTNode} node - The node to be checked.
+         * @param {ASTNode} node The node to be checked.
          * @returns {boolean} True if the node is in a return statement.
          * @private
          */
@@ -197,7 +200,7 @@ module.exports = {
 
         /**
          * Determines if a constructor function is newed-up with parens
-         * @param {ASTNode} newExpression - The NewExpression node to be checked.
+         * @param {ASTNode} newExpression The NewExpression node to be checked.
          * @returns {boolean} True if the constructor is called with parens.
          * @private
          */
@@ -217,7 +220,7 @@ module.exports = {
 
         /**
          * Determines if a node is or contains an assignment expression
-         * @param {ASTNode} node - The node to be checked.
+         * @param {ASTNode} node The node to be checked.
          * @returns {boolean} True if the node is or contains an assignment expression.
          * @private
          */
@@ -239,7 +242,7 @@ module.exports = {
 
         /**
          * Determines if a node is contained by or is itself a return statement and is allowed to have a parenthesised assignment
-         * @param {ASTNode} node - The node to be checked.
+         * @param {ASTNode} node The node to be checked.
          * @returns {boolean} True if the assignment can be parenthesised.
          * @private
          */
@@ -261,8 +264,8 @@ module.exports = {
         /**
          * Determines if a node following a [no LineTerminator here] restriction is
          * surrounded by (potentially) invalid extra parentheses.
-         * @param {Token} token - The token preceding the [no LineTerminator here] restriction.
-         * @param {ASTNode} node - The node to be checked.
+         * @param {Token} token The token preceding the [no LineTerminator here] restriction.
+         * @param {ASTNode} node The node to be checked.
          * @returns {boolean} True if the node is incorrectly parenthesised.
          * @private
          */
@@ -567,7 +570,6 @@ module.exports = {
         /**
          * Checks whether the syntax of the given ancestor of an 'in' expression inside a for-loop initializer
          * is preventing the 'in' keyword from being interpreted as a part of an ill-formed for-in loop.
-         *
          * @param {ASTNode} node Ancestor of an 'in' expression.
          * @param {ASTNode} child Child of the node, ancestor of the same 'in' expression or the 'in' expression itself.
          * @returns {boolean} True if the keyword 'in' would be interpreted as the 'in' operator, without any parenthesis.
@@ -599,7 +601,6 @@ module.exports = {
         /**
          * Starts a new reports buffering. Warnings will be stored in a buffer instead of being reported immediately.
          * An additional logic that requires multiple nodes (e.g. a whole subtree) may dismiss some of the stored warnings.
-         *
          * @returns {void}
          */
         function startNewReportsBuffering() {
@@ -895,6 +896,7 @@ module.exports = {
                 }
 
                 if (nodeObjHasExcessParens &&
+                  !IGNORE_NEW_IN_MEMBER_EXPR &&
                   node.object.type === "NewExpression" &&
                   isNewExpressionWithParens(node.object)) {
                     report(node.object);