Remove unused shouldBreak check

leaysgur · leaysgur · commit b6ac6360ad18 · 2025-11-28T15:56:57.000+09:00
diff --git a/src/language-js/print/ternary-old.js b/src/language-js/print/ternary-old.js
@@ -1,23 +1,18 @@
 import {
   align,
-  breakParent,
   dedent,
   group,
   ifBreak,
   indent,
   line,
   softline,
 } from "../../document/index.js";
-import hasNewlineInRange from "../../utils/has-newline-in-range.js";
-import { locEnd, locStart } from "../loc.js";
 import {
-  hasComment,
   isBinaryCastExpression,
   isCallExpression,
   isJsxElement,
   isMemberExpression,
 } from "../utils/index.js";
-import isBlockComment from "../utils/is-block-comment.js";
 
 /**
  * @import {Doc} from "../../document/index.js"
@@ -323,31 +318,8 @@ function printTernaryOld(path, options, print) {
     );
   }
 
-  // We want a whole chain of ConditionalExpressions to all
-  // break if any of them break. That means we should only group around the
-  // outer-most ConditionalExpression.
-  const shouldBreak = [
-    consequentNodePropertyName,
-    alternateNodePropertyName,
-    ...testNodePropertyNames,
-  ].some((property) =>
-    hasComment(
-      node[property],
-      (comment) =>
-        isBlockComment(comment) &&
-        hasNewlineInRange(
-          options.originalText,
-          locStart(comment),
-          locEnd(comment),
-        ),
-    ),
-  );
   const maybeGroup = (doc) =>
-    parent === firstNonConditionalParent
-      ? group(doc, { shouldBreak })
-      : shouldBreak
-        ? [doc, breakParent]
-        : doc;
+    parent === firstNonConditionalParent ? group(doc) : doc;
 
   // Break the closing paren to keep the chain right after it:
   // (a
