Skip to content
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 10 additions & 1 deletion src/lib/sitemap.test.ts
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import { describe, expect, it } from 'vitest'
import { finalizeSitemap } from './sitemap'
import { finalizeSitemap, shouldIncludeInSitemap } from './sitemap'

const sitemap = `<?xml version="1.0" encoding="UTF-8"?>
<urlset xmlns="http://www.sitemaps.org/schemas/sitemap/0.9">
Expand Down Expand Up @@ -72,3 +72,12 @@ describe('finalizeSitemap', () => {
).toHaveLength(1)
})
})

describe('shouldIncludeInSitemap', () => {
it('keeps concrete routes and excludes dynamic route templates', () => {
expect(shouldIncludeInSitemap('/blog/t7-network-upgrade')).toBe(true)
expect(shouldIncludeInSitemap('/docs/guide/payments')).toBe(true)
expect(shouldIncludeInSitemap('/blog/[slug]')).toBe(false)
expect(shouldIncludeInSitemap('/examples/[id]/details')).toBe(false)
})
})
4 changes: 4 additions & 0 deletions src/lib/sitemap.ts
Original file line number Diff line number Diff line change
@@ -1,6 +1,10 @@
const TEMPLATE_URL_PATTERN = /<url>\s*<loc>([^<]*\/\[[^\]]+\][^<]*)<\/loc>[\s\S]*?<\/url>\s*/g
const LOCATION_PATTERN = /<loc>([^<]+)<\/loc>/g

export function shouldIncludeInSitemap(path: string): boolean {
return !/\/\[[^\]]+\](?:\/|$)/.test(path)
}

export function finalizeSitemap(
sitemap: string,
blogPostSlugs: readonly string[],
Expand Down
2 changes: 2 additions & 0 deletions vocs.config.ts
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ import { resolveBaseUrl } from './src/lib/base-url'
import { docsRouteDestination, proxiedLegacyDocsRoutes } from './src/lib/docs-routing'
import { createFeedbackAdapter } from './src/lib/feedback-adapter'
import { plainMarkdownComponents } from './src/lib/markdown-output'
import { shouldIncludeInSitemap } from './src/lib/sitemap'

// Only set baseUrl in production — Vocs injects a <base> tag from this value,
// which causes all links to resolve to the absolute URL on preview deployments.
Expand Down Expand Up @@ -125,6 +126,7 @@ export default defineConfig({
},
},
sitemap: {
include: shouldIncludeInSitemap,
lastmod: (path, { lastmod }) => {
const pagePath = typeof path === 'string' ? path : '/'
if (pagePath === '/docs' || pagePath.startsWith('/docs/')) return false
Expand Down
Loading