Remove extra group around object properties (#19310)

changelog_unreleased/javascript/19310.md

@@ -0,0 +1,20 @@
+#### Improve formatting of comments in objects (#19310 by @fisker)
+
+<!-- prettier-ignore -->
+```jsx
+// Input
+const P = {/** X */
+  Y: "z",
+}
+
+// Prettier stable
+const P = {
+  /** X */ Y: "z",
+};
+
+// Prettier main
+const P = {
+  /** X */
+  Y: "z",
+};
+```

src/language-js/print/object.js

@@ -105,7 +105,7 @@ function printObject(path, options, print) {
   /** @type {Doc[]} */
   let separatorParts = [];
   const parts = path.map(({ node }) => {
-    const result = [...separatorParts, group(print())];
+    const result = [...separatorParts, print()];
     separatorParts = [",", line];
     if (isNextLineEmpty(node, options)) {
       separatorParts.push(hardline);

tests/format/js/comments/__snapshots__/format.test.js.snap

@@ -6598,7 +6598,8 @@ var a = { /* comment */
 b };
 =====================================output=====================================
 var a = {
-  /* comment */ b,
+  /* comment */
+  b,
 }
 
 ================================================================================
@@ -6613,7 +6614,8 @@ var a = { /* comment */
 b };
 =====================================output=====================================
 var a = {
-  /* comment */ b,
+  /* comment */
+  b,
 };
 
 ================================================================================
@@ -6629,7 +6631,8 @@ var a = { /* comment */
 b };
 =====================================output=====================================
 var a = {
-  /* comment */ b,
+  /* comment */
+  b,
 }
 
 ================================================================================
@@ -6644,7 +6647,8 @@ var a = { /* comment */
 b };
 =====================================output=====================================
 var a = {
-  /* comment */ b,
+  /* comment */
+  b,
 };
 
 ================================================================================

tests/format/typescript/object-type/comments/11308.ts

@@ -0,0 +1,35 @@
+o = {/** X */
+  Y: "z",
+}
+o = {// X
+  Y: "z",
+}
+o = {
+/** X */
+  Y: "z",
+}
+o = {
+/** X */ Y: "z",
+}
+o = {/** X */
+  Y: "z",
+  /** X */ Z: 1,
+}
+
+type T1 = {/** X */
+  Y: "z",
+}
+type T2 = {// X
+  Y: "z",
+}
+type T3 = {
+/** X */
+  Y: "z",
+}
+type T4 = {
+/** X */ Y: "z",
+}
+type T5 = {/** X */
+  Y: "z",
+  /** X */ Z: 1,
+}

tests/format/typescript/object-type/comments/__snapshots__/format.test.js.snap

@@ -50,3 +50,90 @@ o = /** K */ {
 
 ================================================================================
 `;
+
+exports[`11308.ts format 1`] = `
+====================================options=====================================
+parsers: ["typescript"]
+                                                      printWidth: 80 (default) |
+=====================================input======================================
+o = {/** X */
+  Y: "z",
+}
+o = {// X
+  Y: "z",
+}
+o = {
+/** X */
+  Y: "z",
+}
+o = {
+/** X */ Y: "z",
+}
+o = {/** X */
+  Y: "z",
+  /** X */ Z: 1,
+}
+
+type T1 = {/** X */
+  Y: "z",
+}
+type T2 = {// X
+  Y: "z",
+}
+type T3 = {
+/** X */
+  Y: "z",
+}
+type T4 = {
+/** X */ Y: "z",
+}
+type T5 = {/** X */
+  Y: "z",
+  /** X */ Z: 1,
+}
+
+=====================================output=====================================
+o = {
+  /** X */
+  Y: "z",
+};
+o = {
+  // X
+  Y: "z",
+};
+o = {
+  /** X */
+  Y: "z",
+};
+o = {
+  /** X */ Y: "z",
+};
+o = {
+  /** X */
+  Y: "z",
+  /** X */ Z: 1,
+};
+
+type T1 = {
+  /** X */
+  Y: "z";
+};
+type T2 = {
+  // X
+  Y: "z";
+};
+type T3 = {
+  /** X */
+  Y: "z";
+};
+type T4 = {
+  /** X */ Y: "z";
+};
+type T5 = {
+  /** X */
+  Y: "z";
+  /** X */ Z: 1;
+};
+
+================================================================================
+`;