fix(seo): use canonical Charts catalog URLs in the sitemap (#1469)

* fix(seo): emit canonical Charts catalog sitemap URLs

* fix(seo): keep the catalog fallback URL canonical

Author: tannerlinsley

src/utils/charts-catalog.ts

@@ -60,12 +60,12 @@ export function getChartsCatalogSitemapEntries(index: ChartsCatalogIndex) {
   )
 
   return [
-    { path: chartsCatalogBasePath },
+    { path: chartsCatalogBasePath.replace(/\/$/, '') },
     ...collections.map((collection) => ({
-      path: `${chartsCatalogBasePath}collections/${collection.id}/`,
+      path: `${chartsCatalogBasePath}collections/${collection.id}`,
     })),
     ...index.cases.map((catalogCase) => ({
-      path: `${chartsCatalogBasePath}charts/${catalogCase.id}/`,
+      path: `${chartsCatalogBasePath}charts/${catalogCase.id}`,
     })),
   ]
 }

src/utils/sitemap.ts

@@ -145,7 +145,7 @@ async function getPublishedChartsCatalogEntries(): Promise<
     return getChartsCatalogSitemapEntries(publication.index)
   } catch (error) {
     console.error('[sitemap] Charts catalog unavailable', error)
-    return [{ path: '/charts/catalog/' }]
+    return [{ path: '/charts/catalog' }]
   }
 }
 

tests/charts-catalog-site-contracts.test.ts

@@ -12,7 +12,7 @@ import {
   type DocsWebhookSource,
 } from '../src/utils/docs-webhook-sources'
 
-test('sitemap exposes catalog pages but not runtime resources', () => {
+test('sitemap emits canonical catalog URLs without trailing slashes or runtime resources', () => {
   const index = parseChartsCatalogIndex({
     schemaVersion: 2,
     source: {
@@ -46,9 +46,9 @@ test('sitemap exposes catalog pages but not runtime resources', () => {
   })
 
   assert.deepEqual(getChartsCatalogSitemapEntries(index), [
-    { path: '/charts/catalog/' },
-    { path: '/charts/catalog/collections/shadcn/' },
-    { path: '/charts/catalog/charts/01-line/' },
+    { path: '/charts/catalog' },
+    { path: '/charts/catalog/collections/shadcn' },
+    { path: '/charts/catalog/charts/01-line' },
   ])
 })