refactor: rewrite feed generation using jpmonette/feed library

Signed-off-by: Sefa Eyeoglu <[email protected]>

Author: Scrumplex

astro.config.mjs

@@ -6,7 +6,7 @@ import sitemap from "@astrojs/sitemap";
 import mdx from "@astrojs/mdx";
 
 export default defineConfig({
-	site: "https://prismlauncher.org",
+	site: process.env.DEPLOY_URL || "https://prismlauncher.org",
 
 	vite: {
 		plugins: [tailwindcss()],

bun.lock

@@ -10,6 +10,7 @@
         "@tailwindcss/vite": "4.1.11",
         "astro": "5.12.3",
         "astro-icon": "1.1.5",
+        "feed": "5.1.0",
         "rehype-raw": "7.0.0",
         "rehype-stringify": "10.0.1",
         "remark": "15.0.1",
@@ -505,6 +506,8 @@
 
     "fdir": ["[email protected]", "", { "peerDependencies": { "picomatch": "^3 || ^4" }, "optionalPeers": ["picomatch"] }, "sha512-hiFoqpyZcfNm1yc4u8oWCf9A2c4D3QjCrks3zmoVKVxpQRzmPNar1hUJcBG2RQHvEVGDN+Jm81ZheVLAQMK6+w=="],
 
+    "feed": ["[email protected]", "", { "dependencies": { "xml-js": "^1.6.11" } }, "sha512-qGNhgYygnefSkAHHrNHqC7p3R8J0/xQDS/cYUud8er/qD9EFGWyCdUDfULHTJQN1d3H3WprzVwMc9MfB4J50Wg=="],
+
     "flattie": ["[email protected]", "", {}, "sha512-9UbaD6XdAL97+k/n+N7JwX46K/M6Zc6KcFYskrYL8wbBV/Uyk0CTAMY0VT+qiK5PM7AIc9aTWYtq65U7T+aCNQ=="],
 
     "follow-redirects": ["[email protected]", "", {}, "sha512-gew4GsXizNgdoRyqmyfMHyAmXsZDk6mHkSxZFCzW9gwlbtOW44CDtYavM+y+72qD/Vq2l550kMF52DT8fOLJqQ=="],
@@ -1047,6 +1050,8 @@
 
     "wrappy": ["[email protected]", "", {}, "sha512-l4Sp/DRseor9wL6EvV2+TuQn63dMkPjZ/sp9XkghTEbV9KlPS1xUsZ3u7/IQO4wxtcFB4bgpQPRcR3QCvezPcQ=="],
 
+    "xml-js": ["[email protected]", "", { "dependencies": { "sax": "^1.2.4" }, "bin": { "xml-js": "./bin/cli.js" } }, "sha512-7rVi2KMfwfWFl+GpPg6m80IVMWXLRjO+PxTq7V2CDhoGak0wzYzFgUY2m4XJ47OGdXd8eLE8EmwfAmdjw7lC1g=="],
+
     "xxhash-wasm": ["[email protected]", "", {}, "sha512-147y/6YNh+tlp6nd/2pWq38i9h6mz/EuQ6njIrmW8D1BS5nCqs0P6DG+m6zTGnNz5I+uhZ0SHxBs9BsPrwcKDA=="],
 
     "yallist": ["[email protected]", "", {}, "sha512-YgvUTfwqyc7UXVMrB+SImsVYSmTS8X/tSrtdNZMImM+n7+QTriRXyXim0mBrTXNeqzVF0KWGgHPeiyViFFrNDw=="],

package.json

@@ -16,6 +16,7 @@
 		"@tailwindcss/vite": "4.1.11",
 		"astro": "5.12.3",
 		"astro-icon": "1.1.5",
+		"feed": "5.1.0",
 		"rehype-raw": "7.0.0",
 		"rehype-stringify": "10.0.1",
 		"remark": "15.0.1",

src/pages/feed/[feedName].xml.ts

@@ -3,141 +3,74 @@ import { remark } from "remark";
 import remarkRehype from "remark-rehype";
 import rehypeRaw from "rehype-raw";
 import rehypeStringify from "rehype-stringify";
-import { getCollection, type CollectionEntry } from "astro:content";
+import { getCollection } from "astro:content";
+import { Feed } from "feed";
 
-interface Post {
-	title: string;
-	url: string;
-	date: string;
-	description: string;
-	content: string;
-}
-
-const FEED_CONFIG = {
-	title: "Prism Launcher",
-	subtitle:
-		"An Open Source Minecraft launcher with the ability to manage multiple instances, accounts and mods. Focused on user freedom and free redistributability.",
-	email: "[email protected]",
-} as const;
+const DEFAULT_URL = new URL("https://prismlauncher.org");
 
 const processor = remark()
 	.use(remarkRehype, { allowDangerousHtml: true })
 	.use(rehypeRaw)
 	.use(rehypeStringify);
 
-const escapeXml = (text: string): string =>
-	text
-		.replace(/&/g, "&")
-		.replace(/</g, "&lt;")
-		.replace(/>/g, "&gt;")
-		.replace(/"/g, "&quot;");
+export const GET: APIRoute = async ({
+	site = DEFAULT_URL,
+	url,
+	params: { feedName },
+}) => {
+	if (feedName !== "feed" && feedName !== "short") {
+		return new Response(null, {
+			status: 404,
+		});
+	}
+
+	const feed = new Feed({
+		title: "Prism Launcher",
+		description:
+			"An Open Source Minecraft launcher with the ability to manage multiple instances, accounts and mods. Focused on user freedom and free redistributability.",
+		id: site.toString(),
+		feed: url.toString(),
+		copyright: "AGPL-3.0",
+		language: "en",
+		image: `${site.toString()}/img/favicon.png`,
+	});
 
-async function processPost(
-	post: CollectionEntry<"news">,
-	siteUrl: string,
-): Promise<Post> {
-	const slug = post.data.slug || post.slug;
+	const posts = await getCollection("news", ({ data }) => !data.draft).then(
+		(posts) =>
+			posts.sort((a, b) => b.data.date.getTime() - a.data.date.getTime()),
+	);
 
-	try {
+	for (const post of posts) {
+		const slug = post.data.slug || post.slug;
+		const link = new URL(`/news/${slug}`, site).toString();
+
+		// TODO: use Astro's .render() in the future
 		const content = String(await processor.process(post.body))
-			.replace(/href="\/([^"]*)"/g, `href="${siteUrl}$1"`)
-			.replace(/src="\/([^"]*)"/g, `src="${siteUrl}$1"`)
-			.replace(/href="#([^"]+)"/g, `href="${siteUrl}news/${slug}/#$1"`);
+			.replace(/href="\/([^"]*)"/g, `href="${site}$1"`)
+			.replace(/src="\/([^"]*)"/g, `src="${site}$1"`)
+			.replace(/href="#([^"]+)"/g, `href="${link}/#$1"`);
 
-		return {
+		feed.addItem({
 			title: post.data.title,
-			url: `${siteUrl}news/${slug}/`,
-			date: post.data.date.toISOString(),
+			id: link,
+			link,
+			date: post.data.date,
 			description: post.data.description,
-			content,
-		};
-	} catch (error) {
-		console.warn(`Failed to process post "${post.data.title}":`, error);
-		return {
-			title: post.data.title,
-			url: `${siteUrl}news/${slug}/`,
-			date: post.data.date.toISOString(),
-			description: post.data.description || "No description available",
-			content: `<p>${escapeXml(post.data.description || "No description available")}</p>`,
-		};
-	}
-}
-
-function generateFeed(
-	entries: Post[],
-	siteUrl: string,
-	updated: string,
-	short: boolean,
-): string {
-	return `<?xml version="1.0" encoding="utf-8"?>
-<feed xmlns="http://www.w3.org/2005/Atom">
-	<title>${FEED_CONFIG.title}</title>
-	<subtitle>${FEED_CONFIG.subtitle}</subtitle>
-	<link href="${siteUrl}feed/feed.xml" rel="self"/>
-	<link href="${siteUrl}"/>
-	<updated>${updated}</updated>
-	<id>${siteUrl}</id>
-	<author>
-		<name>${FEED_CONFIG.title}</name>
-		<email>${FEED_CONFIG.email}</email>
-	</author>
-${entries
-	.map((entry) => {
-		let content = short
-			? `<content type="string">${escapeXml(entry.description)}</content>`
-			: `<content type="html">${escapeXml(entry.content)}</content>`;
-		return `	<entry>
-		<title>${escapeXml(entry.title)}</title>
-		<link href="${entry.url}"/>
-		<updated>${entry.date}</updated>
-		<id>${entry.url}</id>
-		${content}
-	</entry>`;
-	})
-	.join("\n")}
-</feed>`;
-}
-
-export const GET: APIRoute = async ({ site, params: { feedName } }) => {
-	if (feedName !== "feed" && feedName !== "short") {
-		return new Response(null, {
-			status: 404,
+			content: feedName === "short" ? undefined : content,
+			author: [
+				{
+					name: "Prism Launcher Team",
+				},
+			],
 		});
 	}
-	try {
-		const posts = (await getCollection("news", ({ data }) => !data.draft)).sort(
-			(a, b) => b.data.date.getTime() - a.data.date.getTime(),
-		);
-		const siteUrl =
-			(site?.toString() || "https://prismlauncher.org/").replace(/\/$/, "") +
-			"/";
 
-		return new Response(
-			generateFeed(
-				await Promise.all(posts.map((post) => processPost(post, siteUrl))),
-				siteUrl,
-				(posts[0]?.data.date || new Date()).toISOString(),
-				feedName === "short",
-			),
-			{
-				headers: {
-					"Content-Type": "application/atom+xml; charset=utf-8",
-					"Cache-Control": "public, max-age=3600",
-				},
-			},
-		);
-	} catch (error) {
-		console.error("Feed generation failed:", error);
-		return new Response(
-			import.meta.env.DEV
-				? `Feed error: ${error instanceof Error ? error.message : "Unknown error"}`
-				: "Internal Server Error",
-			{
-				status: 500,
-				headers: { "Content-Type": "text/plain" },
-			},
-		);
-	}
+	return new Response(feed.atom1(), {
+		headers: {
+			"Content-Type": "application/atom+xml; charset=utf-8",
+			"Cache-Control": "public, max-age=3600",
+		},
+	});
 };
 
 export const getStaticPaths = () => {