jsx-no-leaked-render causes code to be incorrect by removing parentheses

[tcl333]

// maybeObject: object | undefined
// isFoo: boolean
{maybeObject && (isFoo ? <Aaa /> : <Bbb />)}

// autofixes to:
{!!maybeObject && isFoo ? <Aaa /> : <Bbb />}

This has an entirely different meaning. My original code was hiding everything if maybeObject was undefined. The new code will display <Bbb /> instead.

To show how absurd this is, in other spot in my codebase, the else condition of the ternary was also maybeObject. This meant that my new code had a runtime error.

[ljharb]

I agree; this is changing the semantics, which is unacceptable. The parens should be retained.

[akulsr0]

@ljharb I have made a PR for it's fix, can you pls check once?