fix: use stable edge functions bootstrap package for globalThis.Netlify

Author: JakeChampion

packages/edge-bundler/deno/config.ts

@@ -1,14 +1,11 @@
-// this needs to be updated whenever there's a change to globalThis.Netlify in bootstrap
-import { Netlify } from "https://64e8753eae24930008fac6d9--edge.netlify.app/bootstrap/index-combined.ts"
-
-const [functionURL, collectorURL, rawExitCodes] = Deno.args
+const [functionURL, collectorURL, bootstrapURL, rawExitCodes] = Deno.args
 const exitCodes = JSON.parse(rawExitCodes)
 
-globalThis.Netlify = Netlify
-
 let func
 
 try {
+  const { Netlify } = await import(bootstrapURL)
+  globalThis.Netlify = Netlify
   func = await import(functionURL)
 } catch (error) {
   console.error(error)

packages/edge-bundler/node/bundler.ts

@@ -22,6 +22,7 @@ import { writeManifest } from './manifest.js'
 import { vendorNPMSpecifiers } from './npm_dependencies.js'
 import { ensureLatestTypes } from './types.js'
 import { nonNullable } from './utils/non_nullable.js'
+import { getURL as getBootstrapURL } from '@netlify/edge-functions-bootstrap/version'
 
 export interface BundleOptions {
   basePath?: string
@@ -60,6 +61,7 @@ export const bundle = async (
     userLogger,
     systemLogger,
     vendorDirectory,
+    bootstrapURL = getBootstrapURL(),
   }: BundleOptions = {},
 ) => {
   const logger = getLogger(systemLogger, userLogger, debug)
@@ -161,10 +163,10 @@ export const bundle = async (
   // Retrieving a configuration object for each function.
   // Run `getFunctionConfig` in parallel as it is a non-trivial operation and spins up deno
   const internalConfigPromises = internalFunctions.map(
-    async (func) => [func.name, await getFunctionConfig({ func, importMap, deno, log: logger })] as const,
+    async (func) => [func.name, await getFunctionConfig({ func, importMap, deno, log: logger, bootstrapURL })] as const,
   )
   const userConfigPromises = userFunctions.map(
-    async (func) => [func.name, await getFunctionConfig({ func, importMap, deno, log: logger })] as const,
+    async (func) => [func.name, await getFunctionConfig({ func, importMap, deno, log: logger, bootstrapURL })] as const,
   )
 
   // Creating a hash of function names to configuration objects.

packages/edge-bundler/node/config.test.ts

@@ -14,6 +14,8 @@ import { FunctionConfig, getFunctionConfig } from './config.js'
 import type { Declaration } from './declaration.js'
 import { ImportMap } from './import_map.js'
 import { RateLimitAction, RateLimitAggregator } from './rate_limit.js'
+import { getURL as getBootstrapURL } from '@netlify/edge-functions-bootstrap/version'
+const bootstrapURL = getBootstrapURL()
 
 const importMapFile = {
   baseURL: new URL('file:///some/path/import-map.json'),
@@ -202,6 +204,7 @@ describe('`getFunctionConfig` extracts configuration properties from function fi
         importMap: new ImportMap([importMapFile]),
         deno,
         log: logger,
+        bootstrapURL,
       })
 
     if (func.error) {
@@ -393,6 +396,7 @@ test('Passes validation if default export exists and is a function', async () =>
       importMap: new ImportMap([importMapFile]),
       deno,
       log: logger,
+      bootstrapURL,
     }),
   ).resolves.not.toThrow()
 
@@ -429,6 +433,7 @@ test('Fails validation if default export is not function', async () => {
     importMap: new ImportMap([importMapFile]),
     deno,
     log: logger,
+    bootstrapURL,
   })
 
   await expect(config).rejects.toThrowError(invalidDefaultExportErr(path))
@@ -465,6 +470,7 @@ test('Fails validation if default export is not present', async () => {
     importMap: new ImportMap([importMapFile]),
     deno,
     log: logger,
+    bootstrapURL,
   })
 
   await expect(config).rejects.toThrowError(invalidDefaultExportErr(path))

packages/edge-bundler/node/config.ts

@@ -78,11 +78,13 @@ export const getFunctionConfig = async ({
   func,
   importMap,
   deno,
+  bootstrapURL,
   log,
 }: {
   func: EdgeFunction
   importMap: ImportMap
   deno: DenoBridge
+  bootstrapURL: string
   log: Logger
 }) => {
   // The extractor is a Deno script that will import the function and run its
@@ -117,6 +119,7 @@ export const getFunctionConfig = async ({
       extractorPath,
       pathToFileURL(func.path).href,
       pathToFileURL(collector.path).href,
+      bootstrapURL,
       JSON.stringify(ConfigExitCode),
     ].filter(Boolean),
     { rejectOnExitCode: false },

packages/edge-bundler/node/server/server.ts

@@ -148,7 +148,7 @@ const prepareServer = ({
 
     if (options.getFunctionsConfig) {
       functionsConfig = await Promise.all(
-        functions.map((func) => getFunctionConfig({ func, importMap, deno, log: logger })),
+        functions.map((func) => getFunctionConfig({ func, importMap, deno, bootstrapURL, log: logger })),
       )
     }