From 4301e50ab36f6c62e43a9b9f44167ef74c5d24f8 Mon Sep 17 00:00:00 2001
From: HangerThem <f.borisjuk@hangerthem.com>
Date: Mon, 19 Oct 2026 14:15:57 +0200
Subject: [PATCH] fix: tag/category routes, recently updated list, feed and
 no-js flash

- Slugify tag/category static params so pre-rendered pages match links;
  disable dynamic params and return a real 404 via notFound()
- Drop bogus generateStaticParams from the tags/categories index pages
- Add a styled not-found page
- Sort 'Recently updated' by last modification before slicing
- Atom feed: correct rel=self URL, use updated dates, valid content type
- Swap no-js -> js class in an inline head script before first paint
- Markdown links: only route internal paths through next/link and
  tolerate missing hrefs
---
 .../categories/[categorySlug]/page.tsx        | 16 ++++++------
 app/(withRecent)/categories/page.tsx          |  7 ------
 app/(withRecent)/tags/[tagSlug]/page.tsx      | 16 ++++++------
 app/(withRecent)/tags/page.tsx                |  7 ------
 app/feed.xml/route.ts                         | 14 ++++++++---
 app/layout.tsx                                | 20 +++++++++------
 app/not-found.tsx                             | 16 ++++++++++++
 components/post/RecentlyUpdatedPosts.tsx      | 25 +++++++++----------
 lib/posts.tsx                                 | 16 +++++++-----
 9 files changed, 75 insertions(+), 62 deletions(-)
 create mode 100644 app/not-found.tsx

diff --git a/app/(withRecent)/categories/[categorySlug]/page.tsx b/app/(withRecent)/categories/[categorySlug]/page.tsx
index 3bbaa97..7eec203 100644
--- a/app/(withRecent)/categories/[categorySlug]/page.tsx
+++ b/app/(withRecent)/categories/[categorySlug]/page.tsx
@@ -3,6 +3,8 @@ import { getAllCategories, getCategoryBySlug, getPostsByCategory } from '@/lib/c
 import { dateFormatter } from '@/utils/time'
 import type { Metadata } from 'next'
 import Link from 'next/link'
+import { notFound } from 'next/navigation'
+import { slugify } from '@/utils/slug'
 
 type CategoryPageProps = {
   params: Promise<{
@@ -13,13 +15,16 @@ type CategoryPageProps = {
 export async function generateStaticParams() {
   const categories = getAllCategories()
   return Object.keys(categories).map((category) => ({
-    categorySlug: category,
+    categorySlug: slugify(category),
   }))
 }
 
+export const dynamicParams = false
+
 export async function generateMetadata({ params }: CategoryPageProps): Promise<Metadata> {
   const { categorySlug } = await params
   const category = getCategoryBySlug(categorySlug)
+  if (!category) notFound()
 
   return {
     title: `${category} | ${metadata.title!.toString()}`,
@@ -39,14 +44,7 @@ export default async function CategoryPage({ params }: CategoryPageProps) {
   const posts = getPostsByCategory(categorySlug)
   const category = getCategoryBySlug(categorySlug)
 
-  if (!category) {
-    return (
-      <main className="max-w-3xl 2xl:max-w-4xl mx-auto px-4 py-8 lg:py-16 w-full">
-        <h1>Category not found</h1>
-        <Link href="/categories">Back to categories</Link>
-      </main>
-    )
-  }
+  if (!category) notFound()
 
   return (
     <main className="max-w-3xl 2xl:max-w-4xl mx-auto px-4 py-8 mb-8 lg:py-16 w-full">
diff --git a/app/(withRecent)/categories/page.tsx b/app/(withRecent)/categories/page.tsx
index cd9d8ba..45bc592 100644
--- a/app/(withRecent)/categories/page.tsx
+++ b/app/(withRecent)/categories/page.tsx
@@ -6,13 +6,6 @@ import Link from 'next/link'
 import { metadata } from '@/app/metadata'
 import { dateFormatter } from '@/utils/time'
 
-export async function generateStaticParams() {
-  const categories = getAllCategories()
-  return Object.keys(categories).map((category) => ({
-    categorySlug: category,
-  }))
-}
-
 export async function generateMetadata(): Promise<Metadata> {
   return {
     title: `Categories | ${metadata.title!.toString()}`,
diff --git a/app/(withRecent)/tags/[tagSlug]/page.tsx b/app/(withRecent)/tags/[tagSlug]/page.tsx
index aacdd63..83588d2 100644
--- a/app/(withRecent)/tags/[tagSlug]/page.tsx
+++ b/app/(withRecent)/tags/[tagSlug]/page.tsx
@@ -3,6 +3,8 @@ import { getAllTags, getPostsByTag, getTagBySlug } from '@/lib/tags'
 import { dateFormatter } from '@/utils/time'
 import type { Metadata } from 'next'
 import Link from 'next/link'
+import { notFound } from 'next/navigation'
+import { slugify } from '@/utils/slug'
 
 type TagPageProps = {
   params: Promise<{
@@ -13,13 +15,16 @@ type TagPageProps = {
 export async function generateStaticParams() {
   const tags = getAllTags()
   return Object.keys(tags).map((tag) => ({
-    tagSlug: tag,
+    tagSlug: slugify(tag),
   }))
 }
 
+export const dynamicParams = false
+
 export async function generateMetadata({ params }: TagPageProps): Promise<Metadata> {
   const { tagSlug } = await params
   const tag = getTagBySlug(tagSlug)
+  if (!tag) notFound()
 
   return {
     title: `${tag} | ${metadata.title!.toString()}`,
@@ -39,14 +44,7 @@ export default async function TagPage({ params }: TagPageProps) {
   const posts = getPostsByTag(tagSlug)
   const tag = getTagBySlug(tagSlug)
 
-  if (!tag) {
-    return (
-      <main className="max-w-3xl 2xl:max-w-4xl mx-auto px-4 py-8 lg:py-16 w-full">
-        <h1>Tag not found</h1>
-        <Link href="/tags">Back to tags</Link>
-      </main>
-    )
-  }
+  if (!tag) notFound()
 
   return (
     <main className="max-w-3xl 2xl:max-w-4xl mx-auto px-4 py-8 mb-8 lg:py-16 w-full">
diff --git a/app/(withRecent)/tags/page.tsx b/app/(withRecent)/tags/page.tsx
index 207f829..b023a7e 100644
--- a/app/(withRecent)/tags/page.tsx
+++ b/app/(withRecent)/tags/page.tsx
@@ -6,13 +6,6 @@ import Link from 'next/link'
 import { metadata } from '@/app/metadata'
 import { dateFormatter } from '@/utils/time'
 
-export async function generateStaticParams() {
-  const tags = getAllTags()
-  return Object.keys(tags).map((tag) => ({
-    tagSlug: tag,
-  }))
-}
-
 export async function generateMetadata(): Promise<Metadata> {
   return {
     title: `Tags | ${metadata.title!.toString()}`,
diff --git a/app/feed.xml/route.ts b/app/feed.xml/route.ts
index e797839..1c027c1 100644
--- a/app/feed.xml/route.ts
+++ b/app/feed.xml/route.ts
@@ -1,10 +1,13 @@
 import { metadata } from '@/app/metadata'
 import { getAllPostsMeta } from '@/lib/posts'
+import type { PostMeta } from '@/types/Post.type'
 
 export const dynamic = 'force-static'
 
 const SITE_URL = process.env.SITE_URL
 
+const lastModified = (p: PostMeta) => new Date(p.updated ?? p.date)
+
 function escapeXml(str: string) {
   return str
     .replace(/&/g, '&amp;')
@@ -18,20 +21,23 @@ export async function GET(req: Request) {
   const posts = getAllPostsMeta()
 
   const updated =
-    posts.length > 0 ? new Date(posts[0].date).toISOString() : new Date().toISOString()
+    posts.length > 0
+      ? new Date(Math.max(...posts.map((p) => lastModified(p).getTime()))).toISOString()
+      : new Date().toISOString()
 
   const entries = posts
     .map((p) => {
       const url = `${SITE_URL}/posts/${p.slug}`
       const date = new Date(p.date).toISOString()
+      const modified = lastModified(p).toISOString()
       return `
   <entry>
     <title>${escapeXml(p.title)}</title>
     <link href="${url}" rel="alternate" type="text/html" />
     <id>${url}</id>
     <published>${date}</published>
-    <updated>${date}</updated>
-    <content type="html" src="${url}" />
+    <updated>${modified}</updated>
+    <content type="text/html" src="${url}" />
     <summary>${escapeXml(p.description)}</summary>
   </entry>`
     })
@@ -45,7 +51,7 @@ export async function GET(req: Request) {
   <author>
     <name>${escapeXml(metadata.creator!.toString())}</name>
   </author>
-  <link href="${SITE_URL}/feed" rel="self" type="application/atom+xml" />
+  <link href="${SITE_URL}/feed.xml" rel="self" type="application/atom+xml" />
   <link href="${SITE_URL}" rel="alternate" type="text/html" />
   <rights> © ${new Date().getFullYear()} Frank Borisjuk </rights>
   ${entries}
diff --git a/app/layout.tsx b/app/layout.tsx
index e05d642..32fc18a 100644
--- a/app/layout.tsx
+++ b/app/layout.tsx
@@ -1,5 +1,4 @@
 import { Space_Grotesk } from 'next/font/google'
-import Script from 'next/script'
 import { Sidebar } from '@/components/layout/Sidebar'
 import { MobileNavbar } from '@/components/layout/MobileNavbar'
 import { metadata } from '@/app/metadata'
@@ -18,12 +17,19 @@ export default function RootLayout({
   children: React.ReactNode
 }>) {
   return (
-    <html lang="en" className={`${spaceGrotesk.variable} h-full antialiased no-js`}>
-      <Script
-        dangerouslySetInnerHTML={{
-          __html: `document.documentElement.classList.replace('no-js','js')`,
-        }}
-      />
+    <html
+      lang="en"
+      className={`${spaceGrotesk.variable} h-full antialiased no-js`}
+      suppressHydrationWarning
+    >
+      <head>
+        {/* Runs before first paint so `js-only` elements don't flash in after hydration */}
+        <script
+          dangerouslySetInnerHTML={{
+            __html: `document.documentElement.classList.replace('no-js','js')`,
+          }}
+        />
+      </head>
       <body className="min-h-full flex">
         <Sidebar />
         <MobileNavbar />
diff --git a/app/not-found.tsx b/app/not-found.tsx
new file mode 100644
index 0000000..36d976b
--- /dev/null
+++ b/app/not-found.tsx
@@ -0,0 +1,16 @@
+import Link from 'next/link'
+
+export default function NotFound() {
+  return (
+    <main className="max-w-3xl 2xl:max-w-4xl mx-auto px-4 py-8 mb-8 lg:py-16 w-full">
+      <div className="font-mono text-sm text-fg-dim mb-2">~/404</div>
+      <h1 className="text-4xl font-bold mb-4">Page not found</h1>
+      <p className="text-fg-dim mb-6">
+        Whatever you were looking for isn&apos;t here. It may have moved, or never existed.
+      </p>
+      <Link href="/" className="text-accent hover:underline">
+        Back to the homepage
+      </Link>
+    </main>
+  )
+}
diff --git a/components/post/RecentlyUpdatedPosts.tsx b/components/post/RecentlyUpdatedPosts.tsx
index 74af931..e6c8154 100644
--- a/components/post/RecentlyUpdatedPosts.tsx
+++ b/components/post/RecentlyUpdatedPosts.tsx
@@ -2,24 +2,23 @@ import { getAllPostsMeta } from '@/lib/posts'
 import Link from 'next/link'
 
 export function RecentlyUpdatedPosts() {
-  const posts = getAllPostsMeta().slice(0, 5)
+  const posts = getAllPostsMeta()
+    .toSorted(
+      (a, b) => new Date(b.updated ?? b.date).getTime() - new Date(a.updated ?? a.date).getTime(),
+    )
+    .slice(0, 5)
   return (
     <aside className="w-80 2xl:w-100 hidden lg:block">
       <div className="sticky top-12 border-l border-border h-fit pl-4 py-4">
         <h2 className="text-xl font-bold mb-4">Recently updated</h2>
         <ul className="flex flex-col gap-2 ml-1">
-          {posts
-            .toSorted(
-              (a, b) =>
-                new Date(b.updated ?? b.date).getTime() - new Date(a.updated ?? a.date).getTime(),
-            )
-            .map((post) => (
-              <li key={post.slug} className="text-sm text-fg-dim">
-                <Link href={`/posts/${post.slug}`} className="hover:underline">
-                  {post.title}
-                </Link>
-              </li>
-            ))}
+          {posts.map((post) => (
+            <li key={post.slug} className="text-sm text-fg-dim">
+              <Link href={`/posts/${post.slug}`} className="hover:underline">
+                {post.title}
+              </Link>
+            </li>
+          ))}
         </ul>
       </div>
     </aside>
diff --git a/lib/posts.tsx b/lib/posts.tsx
index 04746cb..37079d0 100644
--- a/lib/posts.tsx
+++ b/lib/posts.tsx
@@ -97,12 +97,16 @@ export async function getPostBySlug(slug: string, postDirectory: string = postsD
         return <CodeWrapper language={language}>{children}</CodeWrapper>
       },
       img: ({ src, alt }: { src: string; alt: string }) => <ImageWrapper src={src} alt={alt} />,
-      a: ({ href, children }: { href: string; children: ReactNode }) =>
-        href?.startsWith('http') ? (
-          <ExternalLink href={href}>{children}</ExternalLink>
-        ) : (
-          <Link href={href}>{children}</Link>
-        ),
+      a: ({ href, children, ...props }: { href?: string; children: ReactNode }) => {
+        if (href?.startsWith('http')) return <ExternalLink href={href}>{children}</ExternalLink>
+        // Only internal routes go through next/link; anchors, mailto:, footnotes etc. stay plain
+        if (href?.startsWith('/')) return <Link href={href}>{children}</Link>
+        return (
+          <a href={href} {...props}>
+            {children}
+          </a>
+        )
+      },
       blockquote: ({ children }: { children: ReactNode }) => {
         const textContent = getTextContent(children).trimStart()
         const type = textContent.match(/^\[!(\w+)\]/)?.[1]?.toLowerCase() as NoteType | undefined