Formatting is not preserved in imports renamed with =>

If I have the following imports (formatted with spaces.inImportCurlyBraces = true in scalafmt, which adds spaces within the curly braces), and if they are already correctly organized:

import bar.{ Bar => Baz }
import foo.{ Foo1, Foo2 }

then organize-imports preserves the formatting of the second line, but not the first line (the spaces within the curly braces are removed):

import bar.{Bar => Baz}
import foo.{ Foo1, Foo2 }

I have only noticed this in lines where an import is renamed using the => syntax.

This is not a problem in my opinion for formatting code (we just run organize-imports first, and then scalafmt, which makes sense anyway) ; but it’s an issue when checking already formatted code with the -check option, since it detects a difference.

I tested with the following configuration:

OrganizeImports {
  coalesceToWildcardImportThreshold = 50
  expandRelative = false
  groupExplicitlyImportedImplicitsSeparately = false
  groupedImports = Merge
  groups = ["re:javax?\\.", "scala.", "*", "my.app.package"]
  importSelectorsOrder = SymbolsFirst
  importsOrder = SymbolsFirst
  removeUnused = true
}

[guilgaly]

I can confirm that it also works on my real project. 👍

BTW, I had already tested this tool with a colleague, over the past few weeks, on another project (with different formatting rules, so it didn’t show the same issue) and we’ve been super happy with it. 🙂

[liancheng]

Thanks for the feedback, I’m closing this one and will prepare for the 0.4.4 release to include this fix.