-rewrite -no-indent breaks code with Optional Braces for Method Arguments

[scalway]

Compiler version

3.3.0-RC5

Minimized code

//  BadRewrite.scala

//> using scala "3.3.0-RC5"
//> using options "-rewrite", "-no-indent"

def test[T](body:T):T = body

object Test:
  test:
    println("test 1")

Output

when we call scala-cli compile BadRewrite.scala our file is rewriten to:

//Result clearly incorrect
//> using scala "3.3.0-RC5"
//> using options "-rewrite", "-no-indent"

def test[T](name:String)(body:T):T = body

object Test {
  test {
    println("test 1")
}

and it clearly misses single bracket. It happens only when you use "optional braces for method arguments" as last thing in block.

If we add anything after call on the same level (here object Test) it will produce proper result:

object Test:
  test:
    println("test 1")
  val dummy = 2 //even comment is enough!!

translates to

//Result as expected 
object Test {
  test {
    println("test 1")
  }
  val dummy = 2 //even comment is enough!!
}

Weird Behavior of End Marker

object Test:
  test:
    println("test 1")
end Test

translates to:

object Test {
  test {
    println("test 1")
} // end Test

but with extra \n:

object Test:
  test:
    println("test 1")

end Test

translates to:

object Test {
  test {
    println("test 1")
}

end Test

[som-snytt]

Presumptive duplicate of #17187 which was just fixed.

[som-snytt]

Confirmed. Before I updated my repo just now:

➜  snips diff bad-rewr*a
4,5c4,5
< object Test {
<   test {
---
> object Test:
>   test:
7d6
< }

After:

➜  snips diff bad-rewr*a
4,5c4,5
< object Test {
<   test {
---
> object Test:
>   test:
7,8d6
<   }
< }

Someday, I'll use scala-cli to test nightlies, instead of downgrading from JDK 20 in order to build dotty...