Bug fix: formatting loop when using bracketSameLine: true and htmlWhitespaceSensitivity: ignore (#447)

* Add failing test

* Fix

* Run pnpm format

* Add changeset

Author: nylexar-sudo

.changeset/brown-oranges-tan.md

@@ -0,0 +1,5 @@
+---
+"prettier-plugin-astro": patch
+---
+
+Fixes a bug with formatting empty elements with bracketSameLine: true and htmlWhitespaceSensitivity: ignore

src/printer/embed.ts

@@ -155,7 +155,12 @@ export const embed = ((path: AstPath, options: Options) => {
 
 			// print
 			const attributes = path.map(print, 'attributes');
-			const openingTag = group(['<script', indent(group(attributes)), softline, '>']);
+			const openingTag = group([
+				'<script',
+				indent(group(attributes)),
+				options.bracketSameLine ? '' : softline,
+				'>',
+			]);
 			return [
 				openingTag,
 				indent([isEmpty ? '' : hardline, formattedScript]),

src/printer/index.ts

@@ -177,7 +177,7 @@ export function print(path: AstPath, opts: ParserOptions, print: printFn): Doc {
 						isTextNodeStartingWithWhitespace(node.children[0]) &&
 						!isPreTagContent(path)
 							? () => line
-							: () => softline;
+							: () => (node.children.length > 0 ? softline : '');
 				} else if (isPreTagContent(path)) {
 					body = () => printRaw(node);
 				} else if (isInlineElement(path, opts, node) && !isPreTagContent(path)) {

test/fixtures/options/option-bracket-same-line-html-true-whitespace-sensitivity-ignore/input.astro

@@ -0,0 +1,5 @@
+<script
+  is:inline
+  src="https://fonts.googleapis.com/css2?family=Roboto&display=swap"
+  crossorigin="anonymous">
+</script>

test/fixtures/options/option-bracket-same-line-html-true-whitespace-sensitivity-ignore/options.json

@@ -0,0 +1,4 @@
+{
+  "bracketSameLine": true,
+  "htmlWhitespaceSensitivity": "ignore"
+}

test/fixtures/options/option-bracket-same-line-html-true-whitespace-sensitivity-ignore/output.astro

@@ -0,0 +1,4 @@
+<script
+  is:inline
+  src="https://fonts.googleapis.com/css2?family=Roboto&display=swap"
+  crossorigin="anonymous"></script>

test/tests/options.test.ts

@@ -205,6 +205,14 @@ test(
 	'options/single-attribute-per-line-false',
 );
 
+// https://prettier.io/docs/options.html#bracket-line
+// https://prettier.io/docs/en/options.html#html-whitespace-sensitivity
+test(
+	'Can format an Astro file with prettier "bracketSameLine: true, htmlWhitespaceSensitivity: ignore" options',
+	files,
+	'options/option-bracket-same-line-html-true-whitespace-sensitivity-ignore',
+);
+
 // // astro option: astroSortOrder
 // test('Can format an Astro file with prettier "astroSortOrder: markup | styles" option',  'option-astro-sort-order-markup-styles');