Remove `multilineGroup` (#450)

commands.md

@@ -24,11 +24,12 @@ line, but if it doesn't fit it will break the outermost group first
 and try again. It will continue breaking groups until everything fits
 (or there are no more groups to break).
 
-### multilineGroup
-
-This is the same as `group`, but with an additional behavior: if this
-group spans any other groups that have hard breaks (see below) this
-group *always* breaks. Otherwise it acts the same as `group`.
+A document can force parent groups to break by including `breakParent`
+(see below). A hard and literal line automatically include this so
+they always break parent groups. Breaks are propagated to all parent
+groups, so if a deeply nested expression has a hard break, everything
+with break. This only matters for "hard" breaks, i.e. newlines that
+are printed no matter what and can be statically analyzed.
 
 For example, an array will try to fit on one line:
 
@@ -53,6 +54,20 @@ Functions always break after the opening curly brace no matter what,
 so the array breaks as well for consistent formatting. See the
 implementation of `ArrayExpression` for an example.
 
+### breakParent
+
+Include this anywhere to force all parent groups to break. See `group`
+for more info. Example:
+
+```js
+group(concat([
+  " ",
+  expr,
+  " ",
+  breakParent
+]))
+```
+
 ### join
 
 Join an array of items with a separator.

src/doc-builders.js

@@ -33,13 +33,6 @@ function group(contents, opts) {
   };
 }
 
-function multilineGroup(contents, opts) {
-  return group(
-    contents,
-    Object.assign(opts || {}, { shouldBreak: willBreak(contents) })
-  );
-}
-
 function conditionalGroup(states, opts) {
   return group(
     states[0],
@@ -62,7 +55,7 @@ const breakParent =  { type: "break-parent" };
 const line = { type: "line" };
 const softline = { type: "line", soft: true };
 const hardline = concat([{ type: "line", hard: true }, breakParent ]);
-const literalline = { type: "line", hard: true, literal: true };
+const literalline = concat([{ type: "line", hard: true, literal: true }, breakParent ]);
 
 function join(sep, arr) {
   var res = [];
@@ -86,7 +79,6 @@ module.exports = {
   hardline,
   literalline,
   group,
-  multilineGroup,
   conditionalGroup,
   breakParent,
   ifBreak,

src/doc-debug.js

@@ -19,8 +19,8 @@ function flattenDoc(doc) {
 
   } else if (doc.type === "if-break") {
     return Object.assign({}, doc, {
-      breakContents: doc.breakContents ? flattenDoc(doc.breakContents) : null,
-      flatContents: doc.flatContents ? flattenDoc(doc.flatContents) : null
+      breakContents: doc.breakContents != null ? flattenDoc(doc.breakContents) : null,
+      flatContents: doc.flatContents != null ? flattenDoc(doc.flatContents) : null
     });
 
   } else if (doc.type === "group") {

src/printer.js

@@ -13,7 +13,6 @@ var hardline = docBuilders.hardline;
 var softline = docBuilders.softline;
 var literalline = docBuilders.literalline;
 var group = docBuilders.group;
-var multilineGroup = docBuilders.multilineGroup;
 var indent = docBuilders.indent;
 var conditionalGroup = docBuilders.conditionalGroup;
 var ifBreak = docBuilders.ifBreak;
@@ -270,6 +269,8 @@ function genericPrintNoParens(path, options, print) {
       const body = path.call(print, "body");
       const collapsed = concat([ concat(parts), " ", body ]);
 
+      // We want to always keep these types of nodes on the same line
+      // as the arrow.
       if (
         n.body.type === "ArrayExpression" ||
           n.body.type === "ObjectExpression" ||
@@ -278,13 +279,21 @@ function genericPrintNoParens(path, options, print) {
         return group(collapsed);
       }
 
-      return conditionalGroup([
-        collapsed,
-        concat([
-          concat(parts),
-          indent(options.tabWidth, concat([ line, body ]))
-        ])
-      ]);
+      // These nested groups are a little wonky, but because
+      // `conditionalGroup` suppresses break propagation, we want to
+      // re-propagate it. We still want to allow the printer to choose
+      // the more collapsed version, but still break parents if there
+      // are any hard breaks in the content.
+      return group(
+        conditionalGroup([
+          collapsed,
+          concat([
+            concat(parts),
+            indent(options.tabWidth, concat([ line, body ]))
+          ])
+        ]),
+        { shouldBreak: willBreak(body) }
+      );
     case "MethodDefinition":
       if (n.static) {
         parts.push("static ");
@@ -425,7 +434,7 @@ function genericPrintNoParens(path, options, print) {
 
         if (grouped.length > 0) {
           parts.push(
-            multilineGroup(
+            group(
               concat([
                 "{",
                 indent(
@@ -560,7 +569,7 @@ function genericPrintNoParens(path, options, print) {
       if (props.length === 0) {
         return "{}";
       } else {
-        return multilineGroup(
+        return group(
           concat([
             leftBrace,
             indent(
@@ -641,7 +650,7 @@ function genericPrintNoParens(path, options, print) {
           lastElem === null;
 
         parts.push(
-          multilineGroup(
+          group(
             concat([
               "[",
               indent(
@@ -781,7 +790,7 @@ function genericPrintNoParens(path, options, print) {
         parts.push(";");
       }
 
-      return multilineGroup(concat(parts));
+      return group(concat(parts));
     case "VariableDeclarator":
       return n.init
         ? concat([ path.call(print, "id"), " = ", path.call(print, "init") ])
@@ -1086,17 +1095,15 @@ function genericPrintNoParens(path, options, print) {
         concat([
           "<",
           path.call(print, "name"),
-          multilineGroup(
-            concat([
-              indent(
-                options.tabWidth,
-                concat(
-                  path.map(attr => concat([ line, print(attr) ]), "attributes")
-                )
-              ),
-              n.selfClosing ? line : softline
-            ])
-          ),
+          concat([
+            indent(
+              options.tabWidth,
+              concat(
+                path.map(attr => concat([ line, print(attr) ]), "attributes")
+              )
+            ),
+            n.selfClosing ? line : softline
+          ]),
           n.selfClosing ? "/>" : ">"
         ])
       );
@@ -1310,7 +1317,7 @@ function genericPrintNoParens(path, options, print) {
 
       parts.push(path.call(print, "typeParameters"));
 
-      parts.push(multilineGroup(printFunctionParams(path, print, options)));
+      parts.push(group(printFunctionParams(path, print, options)));
 
       // The returnType is not wrapped in a TypeAnnotation, so the colon
       // needs to be added separately.
@@ -1655,7 +1662,7 @@ function printMethod(path, options, print) {
   parts.push(
     key,
     path.call(print, "value", "typeParameters"),
-    multilineGroup(
+    group(
       concat([
         path.call(
           function(valuePath) {
@@ -1726,7 +1733,7 @@ function printArgumentsList(path, options, print) {
     );
   }
 
-  return multilineGroup(
+  return group(
     concat([
       "(",
       indent(
@@ -1811,7 +1818,7 @@ function printObjectMethod(path, options, print) {
   }
 
   parts.push(
-    multilineGroup(
+    group(
       concat([
         printFunctionParams(path, print, options),
         printReturnType(path, print)
@@ -1879,7 +1886,7 @@ function printExportDeclaration(path, options, print) {
       } else {
         parts.push(
           decl.exportKind === "type" ? "type " : "",
-          multilineGroup(
+          group(
             concat([
               "{",
               indent(
@@ -2276,7 +2283,7 @@ function maybeWrapJSXElementInParens(path, elem, options) {
     return elem;
   }
 
-  return multilineGroup(
+  return group(
     concat([
       ifBreak("("),
       indent(options.tabWidth, concat([ softline, elem ])),

tests/comments/__snapshots__/jsfmt.spec.js.snap

@@ -179,7 +179,8 @@ Observable
 
 /* Comments in JSX tag are placed in a non optimal way*/
 <div
-/* comment*/ />;
+/* comment*/
+/>;
 
 // Comments disappear in empty blocks
 if (1) {
@@ -379,7 +380,8 @@ Observable
 
 // Comments in JSX tag are placed in a non optimal way
 <// comment
-div />;
+div
+/>;
 
 // Comments disappear in empty blocks
 if (1) {

tests/trailing_whitespace/__snapshots__/jsfmt.spec.js.snap

@@ -18,7 +18,8 @@ export type Result<T, V> =
 \`
    
    
-\` + \`
+\` +
+  \`
     
     
 \`;