[windows] Add Windows compatibility for the replacement ESBuild plu…

…gin (#465) * [windows] Add Windows compatibility for the `replacement` ESBuild plugin Previously, the separator for the filter regex was hardcoded to / (Unix separator). Now, we're also taking account for the \ (Windows separator) in the filter regex. Signed-off-by: Costin-Robert Sin <[email protected]> * Create shy-cups-notice.md --------- Signed-off-by: Costin-Robert Sin <[email protected]> Co-authored-by: conico974 <[email protected]>
opennextjs · Jul 8, 2024 · 7beaf82 · 7beaf82
1 parent ab0f8b2
commit 7beaf82
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 3 deletions.
diff --git a/.changeset/shy-cups-notice.md b/.changeset/shy-cups-notice.md
@@ -0,0 +1,5 @@
+---
+"open-next": patch
+---
+
+[windows] Add Windows compatibility for the `replacement` ESBuild plugin
diff --git a/packages/open-next/src/build.ts b/packages/open-next/src/build.ts
@@ -335,7 +335,8 @@ async function createImageOptimizationBundle(config: OpenNextConfig) {
     plugins.push(
       openNextReplacementPlugin({
         name: "opennext-14.1.1-image-optimization",
-        target: /plugins\/image-optimization\/image-optimization\.js/g,
+        target:
+          /plugins(\/|\\)image-optimization(\/|\\)image-optimization\.js/g,
         replacements: [
           require.resolve(
             "./adapters/plugins/image-optimization/image-optimization.replacement.js",

diff --git a/packages/open-next/src/build/createServerBundle.ts b/packages/open-next/src/build/createServerBundle.ts
@@ -207,7 +207,7 @@ async function generateBundle(
   const plugins = [
     openNextReplacementPlugin({
       name: `requestHandlerOverride ${name}`,
-      target: /core\/requestHandler.js/g,
+      target: /core(\/|\\)requestHandler\.js/g,
       deletes: disableNextPrebundledReact ? ["applyNextjsPrebundledReact"] : [],
       replacements: disableRouting
         ? [
@@ -219,7 +219,7 @@ async function generateBundle(
     }),
     openNextReplacementPlugin({
       name: `utilOverride ${name}`,
-      target: /core\/util.js/g,
+      target: /core(\/|\\)util\.js/g,
       deletes: [
         ...(disableNextPrebundledReact ? ["requireHooks"] : []),
         ...(disableRouting ? ["trustHostHeader"] : []),