fix(build): Do not minify OpenNext bundles (#144)

This is required to apply string replacement patches.
Fixes #143

Author: vicb

packages/cloudflare/env.d.ts

@@ -9,10 +9,6 @@ declare global {
       OPEN_NEXT_ORIGIN: string;
     }
   }
-
-  interface Window {
-    [key: string]: string | Fetcher;
-  }
 }
 
 export {};

packages/cloudflare/package.json

@@ -59,7 +59,7 @@
     "vitest": "catalog:"
   },
   "dependencies": {
-    "@opennextjs/aws": "https://pkg.pr.new/@opennextjs/aws@2202f36",
+    "@opennextjs/aws": "https://pkg.pr.new/@opennextjs/aws@d2ead58",
     "ts-morph": "catalog:"
   },
   "peerDependencies": {

packages/cloudflare/src/cli/build/bundle-server.ts

@@ -91,7 +91,6 @@ export async function bundleServer(config: Config, openNextOptions: BuildOptions
 // __dirname is used by unbundled js files (which don't inherit the __dirname present in the define field)
 // so we also need to set it on the global scope
 // Note: this was hit in the next/dist/compiled/@opentelemetry/api module
-
 globalThis.__dirname ??= "";
 
 // Do not crash on cache not supported
@@ -168,6 +167,7 @@ async function updateWorkerBundledCode(
 
   patchedCode = patchedCode
     // workers do not support dynamic require nor require.resolve
+    // TODO: implement for cf (possibly in @opennextjs/aws)
     .replace("patchAsyncStorage();", "//patchAsyncStorage();")
     .replace('require.resolve("./cache.cjs")', '"unused"');
 

packages/cloudflare/src/cli/build/index.ts

@@ -41,6 +41,10 @@ export async function build(projectOpts: ProjectOptions): Promise<void> {
   const options = buildHelper.normalizeOptions(config, openNextDistDir, buildDir);
   logger.setLevel(options.debug ? "debug" : "info");
 
+  // Do not minify the code so that we can apply string replacement patch.
+  // Note that wrangler will still minify the bundle.
+  options.minify = false;
+
   // Pre-build validation
   buildHelper.checkRunningInsideNextjsApp(options);
   logger.info(`App directory: ${options.appPath}`);

packages/cloudflare/src/cli/build/open-next/createServerBundle.ts

@@ -125,7 +125,7 @@ async function generateBundle(
   const isBundled = fnOptions.experimentalBundledNextServer ?? false;
   if (isBundled) {
     await bundleNextServer(path.join(outputPath, packagePath), appPath, {
-      debug: options.debug,
+      minify: options.minify,
     });
   }
 

pnpm-lock.yaml

@@ -26,4 +26,4 @@ catalog:
   "tsx": ^4.19.2
   "typescript-eslint": ^8.7.0
   "vitest": ^2.1.1
-  "wrangler": ^3.87.0
+  "wrangler": ^3.91.0