fix text block

Author: crogoz

debugger/package-lock.json

@@ -3,16 +3,16 @@
   "version": "0.1.0",
   "private": true,
   "dependencies": {
+    "@blueprintjs/core": "^3.20.0",
     "@types/jest": "24.0.23",
     "@types/node": "12.12.7",
     "@types/react": "16.9.11",
     "@types/react-dom": "16.9.4",
     "react": "^16.11.0",
     "react-dom": "^16.11.0",
-    "react-scripts": "3.2.0",
-    "typescript": "3.7.2",
-    "@blueprintjs/core": "^3.20.0",
-    "react-treebeard": "^3.2.4"
+    "react-scripts": "^3.2.0",
+    "react-treebeard": "^3.2.4",
+    "typescript": "3.7.2"
   },
   "scripts": {
     "start": "react-scripts start",

debugger/package.json

@@ -104,8 +104,14 @@ public void add(DocBuilder builder) {
         builder.add(this);
     }
 
+    // TODO(crogoz): this is where the token gets computed. I need to differentiate between multiLine Strings and single
+    // Strings
     @Override
     protected float computeWidth() {
+        if (token.getTok().getOriginalText().startsWith("\"\"\"")) {
+            // hack hack hack to not consider the size of the textBlock such that we won't break
+            return 0;
+        }
         return token.getTok().length();
     }
 
@@ -123,6 +129,10 @@ protected Range<Integer> computeRange() {
     public State computeBreaks(
             CommentsHelper commentsHelper, int maxWidth, State state, Obs.ExplorationNode observationNode) {
         String text = token.getTok().getOriginalText();
+        if (token.getTok().getOriginalText().startsWith("\"\"\"")) {
+            // hack hack hack to not consider the size of the textBlock such that we won't break
+            return state.withColumn(state.column() + 3);
+        }
         return state.withColumn(state.column() + text.length());
     }
 

debugger/yarn.lock

@@ -2677,10 +2677,15 @@ void visitDot(ExpressionTree node0) {
                 builder.open(OpenOp.builder()
                         .debugName("visitDot")
                         .plusIndent(plusFour)
-                        .breakBehaviour(BreakBehaviours.preferBreakingLastInnerLevel(true))
+                        .breakBehaviour(
+                                isTextBlock
+                                        ? BreakBehaviours.breakOnlyIfInnerLevelsThenFitOnOneLine(false)
+                                        : BreakBehaviours.preferBreakingLastInnerLevel(true))
                         .breakabilityIfLastLevel(
-                                LastLevelBreakability.ACCEPT_INLINE_CHAIN_IF_SIMPLE_OTHERWISE_CHECK_INNER)
-                        .columnLimitBeforeLastBreak(isTextBlock ? Integer.MAX_VALUE : METHOD_CHAIN_COLUMN_LIMIT)
+                                isTextBlock
+                                        ? LastLevelBreakability.ACCEPT_INLINE_CHAIN
+                                        : LastLevelBreakability.ACCEPT_INLINE_CHAIN_IF_SIMPLE_OTHERWISE_CHECK_INNER)
+                        .columnLimitBeforeLastBreak(METHOD_CHAIN_COLUMN_LIMIT)
                         .isSimple(false)
                         .build());
                 scan(getArrayBase(node), null);
@@ -2793,6 +2798,7 @@ private void visitRegularDot(List<ExpressionTree> items, boolean needDot) {
         int length = needDot0 ? minLength : 0;
         for (ExpressionTree e : items) {
             if (needDot) {
+                // Don't add a break after a text block
                 if (length > minLength) {
                     builder.breakOp(Break.builder()
                             .fillMode(FillMode.UNIFIED)

palantir-java-format/src/main/java/com/palantir/javaformat/doc/Token.java

@@ -33,7 +33,7 @@
 @Execution(ExecutionMode.CONCURRENT)
 public class FormatterIntegrationTest {
 
-    private static FileBasedTests tests = new FileBasedTests(FormatterIntegrationTest.class, "testdata2");
+    private static FileBasedTests tests = new FileBasedTests(FormatterIntegrationTest.class, "testdata");
 
     @ParameterizedClass.Parameters(name = "{0}")
     public static List<Object[]> data() throws IOException {
@@ -73,7 +73,7 @@ public void format() {
         assumeJavaVersionForTest(name);
         try {
             Formatter formatter = createFormatter();
-            String output = createFormatter().formatSource(input);
+            String output = formatter.formatSource(input);
             output = StringWrapper.wrap(Style.PALANTIR.maxLineLength(), output, formatter);
             if (isRecreate()) {
                 tests.writeFormatterOutput(name, output);

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

@@ -52,7 +52,9 @@ class RSLs {
     """, 42);
 
     """
-    hello %s
+    hello %s dsadasdsadad  addkadkadakdakda hdsafdaf
+    daskdabs
+    adasd
     """
         .formatted("world");
             f(
@@ -63,7 +65,7 @@ class RSLs {
         bar
         """);
     """
-    hello
+    hello my dasdasdagdafkasfas fsdgaidahdahdafhalsdflas hasdsasadasdsadasdasdasds
     """.codePoints().forEach(System.err::println);
     String s =
         """
@@ -104,4 +106,12 @@ class RSLs {
           this very long string that does something using arguments %s %s %s
           """,
             "@", "@", "something");
+
+    {
+        """
+        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();
+    }
 }