fixup! path separator

Author: vicb

packages/cloudflare/src/cli/build/patches/plugins/dynamic-requires.ts

@@ -93,7 +93,7 @@ rule:
       field: name
       regex: ^NodeModuleLoader$
 fix: |
-  async load($ID) {  
+  async load($ID) {
     ${getRequires("$ID", files, serverDir)}
   }`;
 }
@@ -119,9 +119,8 @@ function requirePage($PAGE, $DIST_DIR, $IS_APP_PATH) {
     }, // Inline fs access and dynamic require that are not supported by workerd.
     fix: `
 function requirePage($PAGE, $DIST_DIR, $IS_APP_PATH) {
-  const { platform } = require('process');
-  const pagePath = platform === 'win32' ? getPagePath($$$ARGS).replaceAll('\\\\', '/') : getPagePath($$$ARGS);
-  
+  const pagePath = getPagePath($$$ARGS).replaceAll(${JSON.stringify(sep)}, ${JSON.stringify(posix.sep)});
+
   // html
   ${(
     await Promise.all(

packages/cloudflare/src/cli/build/patches/plugins/eval-manifest.ts

@@ -3,7 +3,7 @@
  * that are not supported by workerd.
  */
 
-import { join, relative } from "node:path";
+import { join, posix, relative, sep } from "node:path";
 
 import { type BuildOptions, getPackagePath } from "@opennextjs/aws/build/helper.js";
 import { getCrossPlatformPathRegex } from "@opennextjs/aws/utils/regex.js";
@@ -62,8 +62,7 @@ function evalManifest($PATH, $$$ARGS) {
     },
     fix: `
 function evalManifest($PATH, $$$ARGS) {
-  const { platform } = require('process');
-  $PATH = platform === 'win32' ? $PATH.replaceAll('\\\\', '/') : $PATH;
+  $PATH = $PATH.replaceAll(${JSON.stringify(sep)}, ${JSON.stringify(posix.sep)});
   ${returnManifests}
   throw new Error(\`Unexpected evalManifest(\${$PATH}) call!\`);
 }`,

packages/cloudflare/src/cli/build/patches/plugins/find-dir.ts

@@ -3,7 +3,7 @@
  */
 
 import { existsSync } from "node:fs";
-import { join } from "node:path";
+import { join, posix, sep } from "node:path";
 
 import { type BuildOptions, getPackagePath } from "@opennextjs/aws/build/helper.js";
 import { getCrossPlatformPathRegex } from "@opennextjs/aws/utils/regex.js";
@@ -35,8 +35,7 @@ rule:
   pattern: function findDir($DIR, $NAME) { $$$_ }
 fix: |-
   function findDir($DIR, $NAME) {
-    const { platform } = require('process');
-    $DIR = platform === 'win32' ? $DIR.replaceAll('\\\\', '/') : $DIR;
+    $DIR = $DIR.replaceAll(${JSON.stringify(sep)}, ${JSON.stringify(posix.sep)});
     if ($DIR.endsWith(".next/server")) {
       if ($NAME === "app") {
         return ${appExists};

packages/cloudflare/src/cli/build/patches/plugins/load-manifest.ts

@@ -3,7 +3,7 @@
  */
 
 import { readFile } from "node:fs/promises";
-import { join, relative } from "node:path";
+import { join, posix, relative, sep } from "node:path";
 
 import { type BuildOptions, getPackagePath } from "@opennextjs/aws/build/helper.js";
 import { getCrossPlatformPathRegex } from "@opennextjs/aws/utils/regex.js";
@@ -53,8 +53,7 @@ function loadManifest($PATH, $$$ARGS) {
     },
     fix: `
 function loadManifest($PATH, $$$ARGS) {
-  const { platform } = require('process');
-  $PATH = platform === 'win32' ? $PATH.replaceAll('\\\\', '/') : $PATH;
+  $PATH = $PATH.replaceAll(${JSON.stringify(sep)}, ${JSON.stringify(posix.sep)});
   ${returnManifests}
   throw new Error(\`Unexpected loadManifest(\${$PATH}) call!\`);
 }`,