fix: correctly pass locals from Netlify edge middleware (#488)

* fix: correctly pass locals from Netlify edge middleware

* Format

* Remove console

.changeset/violet-laws-wonder.md

@@ -0,0 +1,5 @@
+---
+'@astrojs/netlify': patch
+---
+
+Correctly pass Netlify context in edge middleware

.gitignore

@@ -12,6 +12,7 @@ package-lock.json
 .pnpm-store
 .idea/
 **/fixtures/**/.astro
+**/hosted/**/.astro
 
 # ignore top-level vscode settings
 /.vscode/settings.json

packages/netlify/src/index.ts

@@ -289,9 +289,9 @@ export default function netlifyIntegration(
 			export default async (request, context) => {
 				const ctx = createContext({
 					request,
-					params: {}
+					params: {},
+					locals: { netlify: { context } }
 				});
-				ctx.locals.netlify = { context } 
 				// https://docs.netlify.com/edge-functions/api/#return-a-rewrite
 				ctx.rewrite = (target) => {
 					if(target instanceof Request) {

packages/netlify/test/hosted/hosted-astro-project/astro.config.mjs

@@ -4,7 +4,9 @@ import { defineConfig } from 'astro/config';
 // https://astro.build/config
 export default defineConfig({
 	output: 'server',
-	adapter: netlify(),
+	adapter: netlify({
+		edgeMiddleware: true,
+	}),
 	image: {
 		remotePatterns: [
 			{

packages/netlify/test/hosted/hosted-astro-project/package.json

@@ -7,6 +7,6 @@
   },
   "dependencies": {
     "@astrojs/netlify": "workspace:*",
-    "astro": "^5.0.0"
+    "astro": "^5.0.5"
   }
 }

packages/netlify/test/hosted/hosted-astro-project/src/middleware.ts

@@ -0,0 +1,11 @@
+import https from 'node:https';
+
+export const onRequest = (context, next) => {
+	console.log(context.netlify);
+	context.locals.middleware = context?.locals?.netlify?.context?.geo?.country?.code ?? null;
+	context.locals.runtime = 'Deno' in globalThis ? 'Deno' : 'Node';
+	context.locals.title = 'Middleware';
+	context.locals.nodePrefixedImportExists = !!https;
+
+	return next();
+};

packages/netlify/test/hosted/hosted-astro-project/src/pages/country.astro

@@ -0,0 +1,7 @@
+---
+const country = Astro.locals.middleware;
+---
+
+<h1>{country}</h1>
+<h3>{country ? 'has context' : 'no context'}</h3>
+<h2>{Astro.locals.runtime}</h2>

packages/netlify/test/hosted/hosted.test.js

@@ -12,10 +12,17 @@ describe('Hosted Netlify Tests', () => {
 		assert.equal(image.status, 200);
 	});
 
+	it('passes context from edge middleware', async () => {
+		const response = await fetch(`${NETLIFY_TEST_URL}/country`);
+		const body = await response.text();
+		assert.match(body, /has context/);
+		assert.match(body, /Deno/);
+	});
+
 	it('Server returns fresh content', async () => {
-		const responseOne = await fetch(`${NETLIFY_TEST_URL}/time`);
+		const responseOne = await fetch(`${NETLIFY_TEST_URL}/time`).then((res) => res.text());
 
-		const responseTwo = await fetch(`${NETLIFY_TEST_URL}/time`);
+		const responseTwo = await fetch(`${NETLIFY_TEST_URL}/time`).then((res) => res.text());
 
 		assert.notEqual(responseOne.body, responseTwo.body);
 	});