Scala CLI shebang is not handled

[tgodzik]

Just opening it up here so that I don't forget, but it seems when #!/usr/bin/env -S scala-cli shebang is added scalafix doesn't sort imports.

I will take a look at it probably in April. We usually handle it by just commenting it out for our purposes. Semanticdb is produced correctly there.

Related to scalameta/metals#5942

[bjaglin]

[info]   scala.meta.parsers.ParseException: scala/test/organizeImports/Shebang.sc:1: error: illegal start of definition `identifier`
[info] #!/usr/bin/env -S scala-cli shebang
[info] ^
[info]   at scala.meta.internal.parsers.Reporter.syntaxError(Reporter.scala:16)
[info]   at scala.meta.internal.parsers.Reporter.syntaxError$(Reporter.scala:16)
[info]   at scala.meta.internal.parsers.Reporter$$anon$1.syntaxError(Reporter.scala:22)
[info]   at scala.meta.internal.parsers.Reporter.syntaxError(Reporter.scala:17)
[info]   at scala.meta.internal.parsers.Reporter.syntaxError$(Reporter.scala:17)
[info]   at scala.meta.internal.parsers.Reporter$$anon$1.syntaxError(Reporter.scala:22)
[info]   at scala.meta.internal.parsers.ScalametaParser.statSeqBuf(ScalametaParser.scala:4088)
[info]   at scala.meta.internal.parsers.ScalametaParser.$anonfun$batchSource$13(ScalametaParser.scala:4296)
[info]   at scala.Option.getOrElse(Option.scala:201)
[info]   at scala.meta.internal.parsers.ScalametaParser.$anonfun$batchSource$1(ScalametaParser.scala:4296)
[info]   ...

We could indeed strip the shebang comment in scalafix-core, but shouldn't this be done upstream in ScalametaParser?

[tgodzik]

We probably could do that, I think it was just more complex to do on the scalameta side than on the Metals one previously.

[bjaglin]

fixed by scalameta/scalameta#3812, pending bump

[bjaglin]

fixed in #2047, tested via b22f829