Prerequisites checklist

• I have read the contributing guidelines.

What is the purpose of this pull request? (put an "X" next to an item)

[ ] Documentation update
[x] Bug fix (template)
[ ] New rule (template)
[ ] Changes an existing rule (template)
[ ] Add autofixing to a rule
[ ] Add a CLI option
[ ] Add something to the core
[ ] Other, please explain:

Tell us about your environment

• ESLint Version: v7.18.0
• Node Version: v12.18.4
• npm Version: v6.14.6

What parser (default, @babel/eslint-parser, @typescript-eslint/parser, etc.) are you using?

Please show your full configuration:

module.exports = {
  parserOptions: {
    ecmaVersion: 2015
  }
};

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

Online Demo

/* eslint no-useless-rename: error */

({ a: (a) } = foo );

({ a: (a) = 1 } = foo );

What did you expect to happen?

2 errors, but not an invalid autofix.

What actually happened? Please include the actual, raw output from ESLint.

2 errors and autofix to:

/* eslint no-useless-rename: error */

({ a) } = foo );

({ (a) = 1 } = foo );

  3:5  error  Parsing error: Unexpected token )

{ a) } is obviously a syntax error.

{ (a) = 1 } is also a syntax error, those parentheses are not allowed in shorthand properties.

What changes did you make? (Give an overview)

Changed the no-useless-rename rule to correctly fix code such as { a: (a) } and avoid autofixing code such as { a: (a) = 1 }.

Is there anything you'd like reviewers to focus on?

This change should be also helpful for future syntax tc39/ecma262#2154, like export { "a b" as "a b" } from "foo";