no-lonely-if: --fix deletes intervening comments

Using eslint --fix with the unicorn/no-lonely-if rule can result in comments being unexpectedly deleted.

if (
  foo
  // comment one
) {
  // comment two
  if (
    // comment three
    bar
    // comment four
  ) {
    baz();
  }
}

is fixed to

if (
  foo && bar
  // comment one
) {
    baz();
  }

I would expect something more like

if (
  foo &&
  // comment one
  // comment two
  // comment three
  bar
  // comment four
) {
  baz();
}

Configuration:

{
  "plugins": ["unicorn"],
  "rules": {
    "unicorn/no-lonely-if": "error"
  }
}

Versions: eslint@7.16.0, eslint-plugin-unicorn@24.0.0

[fisker]

This is by design, it’s tested https://github.com/sindresorhus/eslint-plugin-unicorn/blob/582ca342b4f2587bb83eddf686a959d1ae940994/test/no-lonely-if.js#L93-L113, but you can improve it if you like.

[fisker]

I’m not against to make the fix more safe, but if we are doing this, we need apply the strategy to all rules.