feat: add XML sitemap via @astrojs/sitemap - #393
Open
isvictorious wants to merge 1 commit into
Open
Conversation
The docs site had no sitemap; Starlight doesn't generate one by default, leaving search engines without a canonical page index. Add @astrojs/sitemap, which inherits `site` + `base` so URLs resolve as https://www.tigerdata.com/docs/... - filter() excludes the local-preview REST placeholder - serialize() sets an accurate per-page `lastmod` from each doc's real git history (the field Google actually uses), plus section `priority` (home 1.0, get-started 0.9, rest 0.7) and `changefreq: weekly` Verified locally: 674 URLs, per-page lastmod on all file-backed pages. Part of EDU-622 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
isvictorious
marked this pull request as ready for review
July 14, 2026 19:58
|
Contributor
|
hey @isvictorious, we currently have a sitemap under https://www.tigerdata.com/docs/sitemap-0.xml |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Describe your changes
The docs site had no XML sitemap — Starlight doesn't generate one by default, so search engines had no canonical index of our pages. This adds the
@astrojs/sitemapintegration.Because
site(https://www.tigerdata.com) andbase(/docs) are already set inastro.config.ts, generated URLs correctly resolve ashttps://www.tigerdata.com/docs/.... It regenerates on every build, so new pages are included automatically with no per-page maintenance.What's in the config:
filter()excludes the local-preview REST placeholder so it's never a canonical URL.serialize()sets an accurate per-pagelastmodfrom each doc's real git history (the one field Google actually uses — a blanket build-day date would be misleading), plus sectionpriority(home1.0,get-started0.9, everything else0.7) andchangefreq: weekly. Pages with no source file (the generated REST reference) gracefully get nolastmod.Verified locally (
build:local): 674 URLs, per-pagelastmodon all file-backed pages, placeholder excluded, home at priority1.0.Affected pages
Once you open the PR and the build runs, links to changed pages will appear here automatically. Please review them to make sure everything is OK, including rendered output, links, code blocks, and images.
Related Issues
Issue: EDU-622 (sub-task: EDU-623)
Checklist before requesting a review
🤖 Generated with Claude Code