Merge pull request #2790 from lpsinger/builtin-fetch

Use builtin Web Fetch; polyfill not needed in Node.js >= 20

remix.config.js

@@ -45,6 +45,15 @@ const esmOnlyModules = [
   /^vfile/,
 ]
 
+// These packages should never be bundled.
+const neverBundledModules = [
+  // Included in AWS Lambda base image
+  /@?aws-sdk(?:\/|$)/,
+  // Used to polyfill Web Fetch; not needed for Node.js >= 20
+  'undici',
+  '@remix-run/web-fetch',
+]
+
 /** @type {import('@remix-run/dev').AppConfig} */
 export default {
   mdx: {
@@ -80,7 +89,15 @@ export default {
   serverModuleFormat: 'cjs',
   serverDependenciesToBundle: [
     ...esmOnlyModules,
-    ...(isProduction ? [/^(?!@?aws-sdk(\/|$))/] : []),
+    ...(isProduction
+      ? [
+          new RegExp(
+            `^(?!${neverBundledModules
+              .map((item) => (item instanceof RegExp ? item.source : item))
+              .join('|')})`
+          ),
+        ]
+      : []),
   ],
   future: { v3_relativeSplatPath: true },
 }

server.ts

@@ -1,11 +1,9 @@
 import { type RequestHandler, createRequestHandler } from '@remix-run/architect'
 import * as build from '@remix-run/dev/server-build'
-import { installGlobals } from '@remix-run/node'
 import { type APIGatewayProxyStructuredResultV2 } from 'aws-lambda'
 import sourceMapSupport from 'source-map-support'
 
 sourceMapSupport.install()
-installGlobals({ nativeFetch: true })
 
 const remixHandler = createRequestHandler({
   build,