Worksheet mode adds spaces into multiline strings

[Sporarum]

Describe the bug

The worksheet seems to tabulate the inside of multi-line strings, resulting in unexpected spaces

1. Metals: Create Worksheet
2. Write

val s = """
""".length

4. Worksheet adds comment: // : Int = 5

Expected behavior

Should not add extra characters inside multiline strings

The example should add the comment //: Int = 1

Operating system

Windows

Editor/Extension

VS Code

Version of Metals

0.11.9 ?

Extra context or search terms

No response

[Sporarum]

I wasn't sure how to find the version of metals, but VSCode seems to think it is up to date
I put 0.11.9 as it was the default and what is present at https://scalameta.org/metals/docs/

But the changelog tab of the Extension: Scala (Metals) page in VSCode shows v1.20.0 as the latest

[tgodzik]

Thanks for reporting! I am not sure if we can fix it properly. The way it's currently done is that we wrap the code in additional objects and in order to make sure that Scala 3 works properly we add indentation. Maybe we could do no indentation for the wrapper and then would the indentation work? Something like the below should probably work:

class wrapped{
object A:
   def foo
A.foo
}

The fixed needs to be done in mdoc

[Sporarum]

In the meantime, we should at least emit a warning saying something like "Please use the pattern with | and .stripMargin"
Because this is very surprising behavior, especially given Worksheets might be the first experience people have of Scala

[Sporarum]

Oh and thank you for your reply ^^"

[kubukoz]

just saw it, had the most fun trying to recognize whether I broke my keyboard, filesystem, or switched to some strange newline separator in VSCode 😅

[kubukoz]

BTW. @Sporarum you meant it should print 1, right? That's what I get in the REPL. The newline counts.

[Sporarum]

Oh yes of course, I'll edit the description

Was: "The example should add the comment //: Int = 0"