wip

Author: crogoz

palantir-java-format/src/main/java/com/palantir/javaformat/java/JavaInputAstVisitor.java

@@ -2655,6 +2655,7 @@ void visitDot(ExpressionTree node0) {
         List<ExpressionTree> items = new ArrayList<>(stack);
 
         boolean needDot = false;
+        boolean isTextBlock = false;
 
         // The dot chain started with a primary expression: output it normally, and indent
         // the rest of the chain +4.
@@ -2666,17 +2667,26 @@ void visitDot(ExpressionTree node0) {
                 scan(getArrayBase(node), null);
                 token(".");
             } else {
+                // Special case for text blocks: if the node is a string literal that ends with """,
+                // don't add a break after it
+                if (node instanceof LiteralTree && node.getKind() == Tree.Kind.STRING_LITERAL) {
+                    String sourceForNode = getSourceForNode(node, getCurrentPath());
+                    isTextBlock = sourceForNode.trim().endsWith("\"\"\"");
+                }
+
                 builder.open(OpenOp.builder()
                         .debugName("visitDot")
                         .plusIndent(plusFour)
                         .breakBehaviour(BreakBehaviours.preferBreakingLastInnerLevel(true))
                         .breakabilityIfLastLevel(
                                 LastLevelBreakability.ACCEPT_INLINE_CHAIN_IF_SIMPLE_OTHERWISE_CHECK_INNER)
-                        .columnLimitBeforeLastBreak(METHOD_CHAIN_COLUMN_LIMIT)
+                        .columnLimitBeforeLastBreak(isTextBlock ? Integer.MAX_VALUE : METHOD_CHAIN_COLUMN_LIMIT)
                         .isSimple(false)
                         .build());
                 scan(getArrayBase(node), null);
-                builder.breakOp();
+                if (!isTextBlock) {
+                    builder.breakOp();
+                }
                 needDot = true;
             }
             formatArrayIndices(getArrayIndices(node));

palantir-java-format/src/test/java/com/palantir/javaformat/java/FormatterIntegrationTest.java

@@ -33,7 +33,7 @@
 @Execution(ExecutionMode.CONCURRENT)
 public class FormatterIntegrationTest {
 
-    private static FileBasedTests tests = new FileBasedTests(FormatterIntegrationTest.class, "testdata");
+    private static FileBasedTests tests = new FileBasedTests(FormatterIntegrationTest.class, "testdata2");
 
     @ParameterizedClass.Parameters(name = "{0}")
     public static List<Object[]> data() throws IOException {

palantir-java-format/src/test/resources/com/palantir/javaformat/java/testdata2/crogoz.input

@@ -0,0 +1,20 @@
+class RSLs {
+    {
+        """
+        No tools or answer found in the message. Please try again, following the instructions:\s
+
+        %s
+        """
+            .someOtherValue(e -> e.getValue().print()).myValue(System.err::println).formatted(toolFormatter.usage()).myOtherValue().someOtherValue().somethingElse();
+
+        myValue.someOtherValue(e -> e.getValue().print()).myValue(System.err::println).formatted(toolFormatter.usage()).myOtherValue().someOtherValue().somethingElse();
+
+        """
+        No tools or answer found in the message. Please try again, following the i
+        """.codePoints();
+        """
+        No tools or answer found in the message. Please try again, following the instructions:
+        """
+            .codePoints().forEach(System.err::println);
+    }
+}

palantir-java-format/src/test/resources/com/palantir/javaformat/java/testdata2/crogoz.output

@@ -0,0 +1,4 @@
+ class RSLs {
+     {
+     }
+}