fix(seo): keep versioned landings renderable

Author: KevinVandy

src/routes/-library-landing-route.tsx

@@ -31,13 +31,6 @@ export function beforeLoadLibraryLanding(
     throw redirect({ href: `/${libraryId}/latest`, statusCode: 308 })
   })
 
-  if (version === library.latestVersion) {
-    throw redirect({
-      href: href.replace(`/${libraryId}/${version}`, `/${libraryId}/latest`),
-      statusCode: 308,
-    })
-  }
-
   library.handleRedirects?.(href)
 }
 

src/routes/_library/$libraryId/index.tsx

@@ -1,13 +1,10 @@
 import { redirect, createFileRoute } from '@tanstack/react-router'
 
 export const Route = createFileRoute('/_library/$libraryId/')({
-  beforeLoad: ({ params, location }) => {
+  beforeLoad: ({ params }) => {
     throw redirect({
-      href: location.href.replace(
-        `/${params.libraryId}`,
-        `/${params.libraryId}/latest`,
-      ),
-      statusCode: 308,
+      to: '/$libraryId/$version',
+      params: { libraryId: params.libraryId, version: 'latest' } as never,
     })
   },
 })

src/utils/library-sitemap.ts

@@ -5,7 +5,6 @@ import { getDocsManifest } from '~/utils/docs'
 import { getPartnerSitemapEntries } from '~/utils/partner-pages'
 import { SITE_URL } from '~/utils/site'
 import { getChartsCatalogSitemapEntries } from './charts-catalog'
-import { getLibraryLandingSitemapEntries } from './library-sitemap'
 
 export type SitemapEntry = {
   path: string
@@ -48,6 +47,20 @@ function asLastModified(value: string) {
   return new Date(`${value}T12:00:00.000Z`).toISOString()
 }
 
+function getLibraryEntries(): Array<SitemapEntry> {
+  return libraries.flatMap((library) => {
+    if (
+      library.visible === false ||
+      !library.latestVersion ||
+      library.sitemap?.includeLandingPage !== true
+    ) {
+      return []
+    }
+    const basePath = `/${library.id}/latest`
+    return [{ path: basePath }]
+  })
+}
+
 function isHighValueDocsSlug(slug: string) {
   const segments = slug.split('/').filter(Boolean)
 
@@ -104,7 +117,7 @@ export async function getSitemapEntries(): Promise<Array<SitemapEntry>> {
 
   const entries = [
     ...HIGH_VALUE_NON_DOC_PAGES.map((path) => ({ path })),
-    ...getLibraryLandingSitemapEntries(),
+    ...getLibraryEntries(),
     ...docsEntries.flat(),
     ...getBlogEntries(),
     ...getPartnerSitemapEntries(),