Fix error in empty markdown files with content (#6572)

* test: add test fixture

* test: add test case

* 🤏 fix

* chore: changeset

---------

Co-authored-by: Nate Moore <natemoo-re@users.noreply.github.com>

.changeset/tidy-crabs-warn.md

@@ -0,0 +1,5 @@
+---
+'astro': patch
+---
+
+Properly handle empty markdown files in content collections

packages/astro/src/content/utils.ts

@@ -32,7 +32,7 @@ export const contentConfigParser = z.object({
 export type CollectionConfig = z.infer<typeof collectionConfigParser>;
 export type ContentConfig = z.infer<typeof contentConfigParser>;
 
-type EntryInternal = { rawData: string; filePath: string };
+type EntryInternal = { rawData: string | undefined; filePath: string };
 
 export type EntryInfo = {
 	id: string;
@@ -222,7 +222,8 @@ function hasUnderscoreBelowContentDirectoryPath(
 	return false;
 }
 
-function getFrontmatterErrorLine(rawFrontmatter: string, frontmatterKey: string) {
+function getFrontmatterErrorLine(rawFrontmatter: string | undefined, frontmatterKey: string) {
+	if (!rawFrontmatter) return 0;
 	const indexOfFrontmatterKey = rawFrontmatter.indexOf(`\n${frontmatterKey}`);
 	if (indexOfFrontmatterKey === -1) return 0;
 

packages/astro/test/content-collections.test.js

@@ -215,6 +215,21 @@ describe('Content Collections', () => {
 		});
 	});
 
+	describe('With empty markdown file', () => {
+		it('Throws the right error', async () => {
+			const fixture = await loadFixture({
+				root: './fixtures/content-collections-empty-md-file/',
+			});
+			let error;
+			try {
+				await fixture.build();
+			} catch (e) {
+				error = e.message;
+			}
+			expect(error).to.include('**title**: Required');
+		});
+	});
+
 	describe('SSR integration', () => {
 		let app;
 

packages/astro/test/fixtures/content-collections-empty-md-file/package.json

@@ -0,0 +1,9 @@
+{
+  "name": "@test/content-collections-empty-md-file",
+  "version": "0.0.0",
+  "private": true,
+  "type": "module",
+  "dependencies": {
+    "astro": "workspace:*"
+  }
+}

packages/astro/test/fixtures/content-collections-empty-md-file/src/content/config.ts

@@ -0,0 +1,11 @@
+import { z, defineCollection } from 'astro:content';
+
+const blog = defineCollection({
+	schema: z.object({
+		title: z.string(),
+	}),
+});
+
+export const collections = {
+	blog,
+};

packages/astro/test/fixtures/content-collections-empty-md-file/src/pages/index.astro

@@ -0,0 +1,6 @@
+---
+import { getEntryBySlug } from 'astro:content';
+const blogEntry = await getEntryBySlug('blog', 'empty');
+---
+
+{blogEntry.data.title}