actions: check result.data is not undefined instead of truthy (#11559)

* actions: check result.data is not undefined instead of truthy

* add changeset

* Update .changeset/tasty-rockets-jog.md

---------

Co-authored-by: Bjorn Lu <[email protected]>

.changeset/tasty-rockets-jog.md

@@ -0,0 +1,5 @@
+---
+'astro': patch
+---
+
+Allows actions to return falsy values without an error

packages/astro/src/actions/runtime/route.ts

@@ -40,7 +40,7 @@ export const POST: APIRoute = async (context) => {
 		);
 	}
 	return new Response(JSON.stringify(result.data), {
-		status: result.data ? 200 : 204,
+		status: result.data !== undefined ? 200 : 204,
 		headers: {
 			'Content-Type': 'application/json',
 		},

packages/astro/test/actions.test.js

@@ -228,5 +228,33 @@ describe('Astro Actions', () => {
 			assert.equal($('[data-url]').text(), '/subscribe');
 			assert.equal($('[data-channel]').text(), 'bholmesdev');
 		});
+
+		it('Returns content when the value is 0', async () => {
+			const req = new Request('http://example.com/_actions/zero', {
+				method: 'POST',
+				headers: {
+					'Content-Type': 'application/json',
+					'Content-Length': '0',
+				},
+			});
+			const res = await app.render(req);
+			assert.equal(res.status, 200);
+			const value = await res.json();
+			assert.equal(value, 0);
+		});
+
+		it('Returns content when the value is false', async () => {
+			const req = new Request('http://example.com/_actions/false', {
+				method: 'POST',
+				headers: {
+					'Content-Type': 'application/json',
+					'Content-Length': '0',
+				},
+			});
+			const res = await app.render(req);
+			assert.equal(res.status, 200);
+			const value = await res.json();
+			assert.equal(value, false);
+		});
 	});
 });

packages/astro/test/fixtures/actions/src/actions/index.ts

@@ -64,4 +64,14 @@ export const server = {
 			return;
 		}
 	}),
+	zero: defineAction({
+		handler: async () => {
+			return 0;
+		}
+	}),
+	false: defineAction({
+		handler: async () => {
+			return false;
+		}
+	})
 };