@timtebeek this is to support autoformat for text blocks where idents are not spaces. Looks like the code currently is biased towards spaces based indents. The indent for multiline string is parent AST node indent + 2 indents - this seesm to be the same to JDT and IntelliJ.

@timtebeek i see there is a test failure for aligned text blocks. Trying to figure it out... I don't know why we're making a check !literal.getPrefix().getLastWhitespace().contains("\n") - I'd align the string even if """ on the new line... Something blows up with the cursor is the source position service if I remove this check. If you @Jenson3210 and @timtebeek have ideas/opinions/directions for me, please don't be shy :-)

Okay, I think i came up with a good fix for the alignment things. Should be good to go now

@BoykoAlex, thanks for identifying this gap!

I made some suggestions which I think will make the code behave even more accurate. Let me know if you have thoughts here.

@Jenson3210 thank you for the review :-)

Quick note that we're seeing two failures downstream that I think are related: https://ge.openrewrite.org/s/gtr3ruly7ncng

SourceSpecTextBlockNewLineTest > newlineBetweenMultipleTextBlocks() FAILED
    org.opentest4j.AssertionFailedError: [Unexpected result in "MyRecipeTest.java":
    diff --git a/MyRecipeTest.java b/MyRecipeTest.java
    index 54a59ee..bdfba39 100644
    --- a/MyRecipeTest.java
    +++ b/MyRecipeTest.java
    @@ -15,14 +15,14 @@ 
                  }
                  """,
                   """
    -                class Test {
    -                
    -                    
    -                    void test() {
    -                        System.out.println("Hello, world!");
    -                    }
    -                }
    -                """
    +                      class Test {
    +                      
    +                          
    +                          void test() {
    +                              System.out.println("Hello, world!");
    +                          }
    +                      }
    +                      """
              )
           );
         }
    ] 

Could either of you have a look? I want to prevent us having to field related questions after today's release.