🐛 Formatter detaches // rome-ignore comment from associated logic expression

[lgarron]

Environment information

Rome CLI version 10.0.1

What happened?

Consider the following code (from @github/webauthn-json):

export function supported(): boolean {
  return !!(
    // rome-ignore lint(style/useOptionalChain): Optional chaining creates more complicated ES2019 code
    navigator.credentials &&
    navigator.credentials.create &&
    navigator.credentials.get &&
    window.PublicKeyCredential
  );
}

Playground link

The // rome-ignore comment works as intended.

However, as the playground shows, the formatter introduces parentheses around the && logic:

export function supported(): boolean {
  return !!(
    // rome-ignore lint(style/useOptionalChain): Optional chaining creates more complicated ES2019 code
    (
      navigator.credentials &&
      navigator.credentials.create &&
      navigator.credentials.get &&
      window.PublicKeyCredential
    )
  );
}

This disconnects the // rome-ignore comment, which in turn causes the linter to error again.

Expected result

The formatter doesn't break linter compatibility of the code.

Code of Conduct

• I agree to follow Rome's Code of Conduct