patch eval("require") calls to avoid runtime warnings (#187)

dario-piotrowicz · web-flow · commit fad0badea39d · 2024-12-18T12:17:07.000Z
diff --git a/packages/cloudflare/src/cli/build/bundle-server.ts b/packages/cloudflare/src/cli/build/bundle-server.ts
@@ -167,6 +167,7 @@ async function updateWorkerBundledCode(
           // TODO: implement for cf (possibly in @opennextjs/aws)
           .replace("patchAsyncStorage();", "//patchAsyncStorage();"),
     ],
+    ['`eval("require")` calls', (code) => code.replaceAll('eval("require")', "require")],
     [
       "`require.resolve` call",
       // workers do not support dynamic require nor require.resolve

Original file line number	Diff line number	Diff line change
`@@ -167,6 +167,7 @@ async function updateWorkerBundledCode(`
`167`	`167`	`// TODO: implement for cf (possibly in @opennextjs/aws)`
`168`	`168`	`.replace("patchAsyncStorage();", "//patchAsyncStorage();"),`
`169`	`169`	`],`
	`170`	+ ['`eval("require")` calls', (code) => code.replaceAll('eval("require")', "require")],
`170`	`171`	`[`
`171`	`172`	"`require.resolve` call",
`172`	`173`	`// workers do not support dynamic require nor require.resolve`