fix: fix @netlify/edge-functions exports (#289)

* fix: fix edge functions exports

* fix: re-export `getURL`

* chore: fix test

Author: eduardoboucas

packages/edge-functions/dev/deno/config.mjs

@@ -1,8 +1,8 @@
 // @ts-check
 
 /**
- * @typedef {import('./workers/types.js').ConfigRequestMessage} ConfigRequestMessage
- * @typedef {import('./workers/types.js').Message} Message
+ * @typedef {import('./workers/types.ts').ConfigRequestMessage} ConfigRequestMessage
+ * @typedef {import('./workers/types.ts').Message} Message
  */
 
 /**

packages/edge-functions/dev/shared/types.ts

@@ -1,8 +1,3 @@
-import type { Context } from '@netlify/types'
-import type { Config } from '../../src/lib/config.ts'
-
-export type EdgeFunction = { config?: Config; default: (req: Request, context: Context) => Promise<Response> }
-
 export interface RunOptions {
   bootstrapURL: string
   denoPort: number

packages/edge-functions/src/lib/edge-function.ts

@@ -0,0 +1,6 @@
+import type { Context } from '@netlify/types'
+
+export type EdgeFunction = (
+  request: Request,
+  context: Context,
+) => Response | Promise<Response> | URL | Promise<URL> | undefined | Promise<void>

packages/edge-functions/src/main.ts

@@ -1,3 +1,3 @@
 export type { Context, Cookie } from '@netlify/types'
 export type { Config, IntegrationsConfig, Manifest, ManifestFunction } from './lib/config.js'
-export type { EdgeFunction } from '../dev/shared/types.js'
+export type { EdgeFunction } from './lib/edge-function.js'

packages/edge-functions/src/version.test.ts

@@ -1,13 +1,10 @@
 import { describe, expect, test } from 'vitest'
 
-import packageJSON from '../package.json' with { type: 'json' }
+import * as bootstrap from '@netlify/edge-functions-bootstrap/version'
 import { getURL } from './version.js'
 
 describe('`getURL`', () => {
   test('Returns the URL of the bootstrap entry point', async () => {
-    const { version } = packageJSON
-    const branch = `v${version.split('.').join('-')}`
-
-    expect(await getURL()).toBe(`https://${branch}--edge.netlify.com/bootstrap/index-combined.ts`)
+    expect(await getURL()).toBe(await bootstrap.getURL())
   })
 })

packages/edge-functions/src/version.ts

@@ -1,29 +1 @@
-import { promises as fs } from 'node:fs'
-import path from 'node:path'
-import { fileURLToPath } from 'node:url'
-
-const packageJsonPath = path.resolve(fileURLToPath(import.meta.url), '../../package.json')
-
-/**
- * Returns the URL of the edge functions bootstrap entry file.
- *
- * @returns {Promise<string>}
- */
-export const getURL = async () => {
-  try {
-    const data = await fs.readFile(packageJsonPath, 'utf8')
-    const { version } = JSON.parse(data) as { version: string }
-
-    // Releasing version 1.2.3 of the `@netlify/edge-functions` module will
-    // make release-please create a GitHub release and corresponding tag of
-    // `v1.2.3`. Because the `edge` Netlify site is configured to build all
-    // branches, the tag will generate a `v1-2-3--edge.netlify.com` URL,
-    // which serves the bootstrap for this version. We're computing that
-    // URL here.
-    const branch = `v${version.split('.').join('-')}`
-
-    return `https://${branch}--edge.netlify.com/bootstrap/index-combined.ts`
-  } catch (error) {
-    throw new Error(`Could not find edge bootstrap version: ${error}`)
-  }
-}
+export { getURL } from '@netlify/edge-functions-bootstrap/version'