fix(netlify): externalise sharp when bundling edge middleware (#316)

* fix(netlify): externalise sharp when bundling edge middleware * Deps
withastro · Jul 1, 2024 · d81806a · d81806a
1 parent 1f781f2
commit d81806a
Show file tree

Hide file tree

Showing 4 changed files with 203 additions and 19 deletions.
diff --git a/.changeset/shy-ties-float.md b/.changeset/shy-ties-float.md
@@ -0,0 +1,5 @@
+---
+'@astrojs/netlify': patch
+---
+
+Fixes a regression where edge middleware tried to bundle node builtins
diff --git a/packages/netlify/package.json b/packages/netlify/package.json
@@ -45,7 +45,7 @@
     "@netlify/edge-functions": "^2.0.0",
     "@netlify/edge-handler-types": "^0.34.1",
     "@types/node": "^18.17.8",
-    "astro": "^4.10.1",
+    "astro": "^4.11.3",
     "astro-scripts": "workspace:*",
     "cheerio": "1.0.0-rc.12",
     "execa": "^8.0.1",

diff --git a/packages/netlify/src/index.ts b/packages/netlify/src/index.ts
@@ -309,6 +309,7 @@ export default function netlifyIntegration(
 			format: 'esm',
 			bundle: true,
 			minify: false,
+			external: ['sharp'],
 			banner: {
 				// Import Deno polyfill for `process.env` at the top of the file
 				js: 'import process from "node:process";',