From eff6051f026c221298c6288213e290d82a0a3817 Mon Sep 17 00:00:00 2001
From: SinhSinh An <sinhsinhan@gmail.com>
Date: Wed, 22 Apr 2026 14:49:12 -0500
Subject: [PATCH] fix(prettier-plugin-liquid): route standalone externals to
 prettier/standalone

The webpack config for the browser `standalone.js` bundle declared
the `prettier` external with `commonjs: 'prettier'` and
`commonjs2: 'prettier'`. The resulting UMD wrapper emitted
`require('prettier')` in both the `module.exports` and `exports[...]`
branches, so downstream bundlers (webpack, rollup, vite) processing
`standalone.js` for a browser target resolved prettier's Node-flavored
main entry. That entry imports `module`, `url`, and `path` and crashes
browser builds with "Module not found" errors.

Route the `commonjs` and `commonjs2` keys to `prettier/standalone`,
matching the `amd` branch, so downstream bundlers land on prettier's
browser-safe entry. Keep `root: 'prettier'` so <script>-tag consumers
still bind against the `window.prettier` global populated by
`prettier/standalone.js` on the page.

Closes #1098
---
 .changeset/fix-standalone-prettier-import.md      |  5 +++++
 packages/prettier-plugin-liquid/webpack.config.js | 13 +++++++++++--
 2 files changed, 16 insertions(+), 2 deletions(-)
 create mode 100644 .changeset/fix-standalone-prettier-import.md

diff --git a/.changeset/fix-standalone-prettier-import.md b/.changeset/fix-standalone-prettier-import.md
new file mode 100644
index 000000000..fbbb1fccd
--- /dev/null
+++ b/.changeset/fix-standalone-prettier-import.md
@@ -0,0 +1,5 @@
+---
+'@shopify/prettier-plugin-liquid': patch
+---
+
+Fix standalone build importing `prettier` instead of `prettier/standalone`, which caused browser bundlers (webpack, rollup, vite) consuming `standalone.js` to try to resolve Node builtins (`module`, `url`, `path`) from prettier's main entry
diff --git a/packages/prettier-plugin-liquid/webpack.config.js b/packages/prettier-plugin-liquid/webpack.config.js
index 5a8d7ba53..3c435e5b7 100644
--- a/packages/prettier-plugin-liquid/webpack.config.js
+++ b/packages/prettier-plugin-liquid/webpack.config.js
@@ -15,9 +15,18 @@ module.exports = {
     },
   },
   externals: {
+    // The standalone bundle is browser-only. Downstream bundlers that consume
+    // standalone.js (webpack, rollup, vite) see a UMD wrapper whose `commonjs`
+    // and `commonjs2` branches `require('prettier')`. That import resolves to
+    // prettier's Node-flavored main entry, which pulls in `module`, `url`, and
+    // `path` and breaks browser builds. Route those branches to
+    // `prettier/standalone` so downstream bundlers land on prettier's
+    // browser-safe entry. `root: 'prettier'` is kept so <script>-tag loading
+    // still works against the global `window.prettier` populated by
+    // `prettier/standalone.js` on the page.
     prettier: {
-      commonjs: 'prettier',
-      commonjs2: 'prettier',
+      commonjs: 'prettier/standalone',
+      commonjs2: 'prettier/standalone',
       amd: 'prettier/standalone',
       root: 'prettier',
     },