add esbuild fn support

Author: dcousens

.changeset/add-custom-esbuild.md

@@ -2,4 +2,4 @@
 "@keystone-6/core": minor
 ---
 
-Add support for developers to add `esbuild.keystone.js` to the working directory to replace the default esbuild configuration
+Add support for developers to add `esbuild.keystone.js` to the working directory to mutate the default esbuild configuration

packages/core/src/lib/esbuild.ts

@@ -1,13 +1,16 @@
 // WARNING: be careful not to import `esbuild` within next
 import { type BuildOptions } from 'esbuild'
 
+function identity (x: BuildOptions) { return x }
+
 export function getEsbuildConfig (cwd: string): BuildOptions {
+  let esbuildFn: typeof identity | undefined
   try {
-    return require(require.resolve(`${cwd}/esbuild.keystone.js`))
+    esbuildFn = require(require.resolve(`${cwd}/esbuild.keystone.js`))
   } catch (e) {}
+  esbuildFn ??= identity
 
-  // default fallback
-  return {
+  return esbuildFn({
     entryPoints: ['./keystone'],
     absWorkingDir: cwd,
     bundle: true,
@@ -20,18 +23,15 @@ export function getEsbuildConfig (cwd: string): BuildOptions {
       {
         name: 'external-node_modules',
         setup (build) {
-          build.onResolve(
-            {
-              // don't bundle anything that is NOT a relative import
-              //   WARNING: we can't use a negative lookahead/lookbehind because esbuild uses Go
-              filter: /(?:^[^.])|(?:^\.[^/.])|(?:^\.\.[^/])/,
-            },
-            ({ path }) => {
-              return { external: true, path }
-            }
-          )
+          build.onResolve({
+            // don't bundle anything that is NOT a relative import
+            //   WARNING: we can't use a negative lookahead/lookbehind because esbuild uses Go
+            filter: /(?:^[^.])|(?:^\.[^/.])|(?:^\.\.[^/])/,
+          }, ({ path }) => {
+            return { external: true, path }
+          })
         },
       },
     ],
-  }
+  } satisfies BuildOptions)
 }