Optimize caching and deployment workflow

Use five-minute revalidation for published content, posts, and
sitemaps, while pre-rendering known article routes. Make Netlify
deployments manual and cancel superseded runs.

Author: Llewellyn500

.github/workflows/deploy-to-netlify.yml

@@ -1,8 +1,11 @@
 name: Deploy to Netlify
 
 on:
-  push:
-    branches: [main]
+  workflow_dispatch:
+
+concurrency:
+  group: netlify-production-${{ github.repository }}
+  cancel-in-progress: true
 
 permissions:
   contents: read
@@ -41,9 +44,6 @@ jobs:
           restore-keys: |
             ${{ runner.os }}-nextjs-
 
-      - name: Build project
-        run: npm run build
-
       - name: Install Netlify CLI
         run: npm install -g netlify-cli
 

app/[...not_found]/page.tsx

@@ -490,7 +490,7 @@ export default function AccountPage() {
       try {
         await uploadBytes(avatarRef, sanitized.blob, {
           contentType: sanitized.contentType,
-          cacheControl: "public,max-age=3600",
+          cacheControl: "public,max-age=300",
         });
       } finally {
         URL.revokeObjectURL(sanitized.previewUrl);

app/account/page.tsx

@@ -24,9 +24,16 @@ type RouteParams = {
   title: string;
 };
 
-export const dynamic = "force-dynamic";
+export const revalidate = 300;
 export const dynamicParams = true;
 
+export async function generateStaticParams() {
+  const articles = await getPublishedArticles();
+  return articles.map((article) => ({
+    title: article.slug,
+  }));
+}
+
 function buildKeywords(
   articleTitle: string,
   articleDescription: string,

app/posts/[title]/opengraph-image/route.ts

@@ -2,7 +2,7 @@ import { MetadataRoute } from "next";
 import { buildTopicSummaries, getAllAuthors, getPublishedArticles } from "@/lib/content";
 import { absoluteUrl } from "@/lib/seo";
 
-export const dynamic = "force-dynamic";
+export const revalidate = 300;
 
 export default async function sitemap(): Promise<MetadataRoute.Sitemap> {
   // Static routes

app/posts/[title]/page.tsx

@@ -365,9 +365,7 @@ function normalizeRouteSlug(slug: string) {
   }
 }
 
-async function getPublishedArticleDocuments(
-  revalidateSeconds: number | false = 300,
-): Promise<ContentDocument[]> {
+async function getPublishedArticleDocuments(): Promise<ContentDocument[]> {
   const { apiKey, projectId } = getRequiredFirebaseConfig();
   const params = new URLSearchParams({ key: apiKey });
   const response = await fetch(
@@ -389,9 +387,7 @@ async function getPublishedArticleDocuments(
           },
         },
       }),
-      ...(revalidateSeconds === false
-        ? { cache: "no-store" as const }
-        : { next: { revalidate: revalidateSeconds } }),
+      next: { revalidate: 300 },
     },
   );
 
@@ -548,12 +544,9 @@ export async function getPublishedArticleBySlug(slug: string) {
   const authors = await getAllAuthors();
   const authorLookup = createAuthorLookup(authors);
   const documents = await getPublishedArticleDocuments();
-  const cached = documents.find((entry) => entry.data.slug === normalizedSlug);
-  const document =
-    cached ||
-    (await getPublishedArticleDocuments(false)).find(
-      (entry) => entry.data.slug === normalizedSlug,
-    );
+  const document = documents.find(
+    (entry) => entry.data.slug === normalizedSlug,
+  );
 
   if (!document) return null;