no-extra-parens incorrectly reports returned IIFE

[smbape]

Tell us about your environment

• ESLint Version: 4.5.0
• Node Version: 8.1.3
• npm Version: 5.3.0

What parser (default, Babel-ESLint, etc.) are you using?
default

Please show your full configuration:

Configuration

node node_modules/eslint/bin/eslint.js --no-eslintrc --rule '{"wrap-iife": [2, "any"], "no-extra-parens": 2, "no-trailing-spaces": 2, "quotes":[2, "double"]}' --fix test/compiled.js

What did you do? Please include the actual source code causing the issue.

(function() {
    // trailing spaces    
    console.log('quotes');

    // extra parens
    return (function() {
        // trailing spaces    
        console.log('quotes');
    }());
})();

What did you expect to happen?
Fix option to be executed on nested iife

What actually happened? Please include the actual, raw output from ESLint.
When applying the command line above on the code above, quotes fix is applied on the top iife but not on the neted iife.

Without the wrap-iife everything is fine.

I haven't tested if the fix is not applied only for those specified rules.

On v4.5.0 here is the output

  6:12  error  Gratuitous parentheses around expression  no-extra-parens
  8:21  error  Strings must use doublequote              quotes

On v3.19.0 here is the output

  6:12  error  Gratuitous parentheses around expression  no-extra-parens
  7:30  error  Trailing spaces not allowed               no-trailing-spaces
  8:21  error  Strings must use doublequote              quotes

[not-an-aardvark]

Thanks for the report. This is an issue with the no-extra-parens rule. It's designed to avoid reporting extra parens around IIFEs to avoid conflicting with the wrap-iife rule, but it seems to be incorrectly reporting this case:

/* eslint no-extra-parens: error */
function x() {
  return (function() {}())
}

[smbape]

Thanks for the comment.
I will remove no-extra-parens rule.