From ad1829f0b64e812f007baa9bdcf77b35288e815e Mon Sep 17 00:00:00 2001
From: Sarath Francis <sarathfrancis90@gmail.com>
Date: Fri, 12 Jun 2026 08:52:56 -0400
Subject: [PATCH] Stabilize XML declaration formatting on reformat

The XMLDeclOpen token ("<?xml") carries the single whitespace character
that follows it, since its lexer pattern is /<\?xml[ \t\r\n]/. Printing that
character verbatim is not stable: when the declaration wraps onto multiple
lines, the captured character is a newline, and re-formatting then renders an
extra blank line right after "<?xml". As a result prettier's own output of a
wrapped declaration is not idempotent and fails prettier --check.

Print a normalized "<?xml" and let the surrounding layout provide the
spacing, so a wrapped declaration round-trips unchanged.
---
 src/printer.js      | 10 ++++++++--
 test/format.test.js | 10 ++++++++++
 2 files changed, 18 insertions(+), 2 deletions(-)

diff --git a/src/printer.js b/src/printer.js
index 753ba610..898cb9ac 100644
--- a/src/printer.js
+++ b/src/printer.js
@@ -542,10 +542,16 @@ function printMisc(path, opts, print) {
 
 function printProlog(path, opts, print) {
   const { XMLDeclOpen, attribute, SPECIAL_CLOSE } = path.node;
-  const parts = [XMLDeclOpen];
+
+  // The XMLDeclOpen token includes the single whitespace character that follows
+  // "<?xml" (its pattern is /<\?xml[ \t\r\n]/). Printing that trailing
+  // whitespace verbatim is not stable: when the declaration breaks onto
+  // multiple lines a captured newline turns into an extra blank line on the
+  // next pass. Drop it and let the layout below supply the spacing.
+  const parts = ["<?xml"];
 
   if (attribute) {
-    parts.push(indent([softline, join(line, path.map(print, "attribute"))]));
+    parts.push(indent([line, join(line, path.map(print, "attribute"))]));
   }
 
   return group([
diff --git a/test/format.test.js b/test/format.test.js
index 5448dc26..c16bb371 100644
--- a/test/format.test.js
+++ b/test/format.test.js
@@ -104,3 +104,13 @@ test("xmlQuoteAttributes => double", async () => {
 
   expect(formatted).toMatchSnapshot();
 });
+
+test("wrapped XML declaration is stable on reformat", async () => {
+  const input = '<?xml version="1.0" encoding="UTF-8"?>\n<root />\n';
+  const opts = { printWidth: 20 };
+
+  const once = await format(input, opts);
+  const twice = await format(once, opts);
+
+  expect(twice).toEqual(once);
+});