Skip to content

feat(site): add canonical sitemap generation#1485

Merged
chaliy merged 1 commit intomainfrom
feat/site-sitemap
Apr 29, 2026
Merged

feat(site): add canonical sitemap generation#1485
chaliy merged 1 commit intomainfrom
feat/site-sitemap

Conversation

@chaliy
Copy link
Copy Markdown
Contributor

@chaliy chaliy commented Apr 29, 2026

What

Add canonical sitemap generation for bashkit.sh. The site now builds /sitemap.xml, adds <lastmod> entries, removes Astro's intermediate sitemap files, and points robots.txt at the canonical sitemap.

Why

The site advertised a sitemap URL but did not generate one. Crawlers should have a single canonical /sitemap.xml target.

How

  • Add @astrojs/sitemap to the Astro site
  • Add a post-build sitemap enhancer modeled after the Everruns docs app
  • Add a post-build verifier that checks all generated HTML routes and rejects sitemap-index.xml
  • Run deploy through npm run build so verification executes before Wrangler deploys

Risk

  • Low
  • Site build could fail if Astro changes sitemap output naming; verifier catches that before deploy.

Checklist

  • Tests added or updated
  • Backward compatibility considered

Verified:

  • npm run build
  • npm run check
  • just pre-pr

@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
bashkit b73ad48 Commit Preview URL

Branch Preview URL
Apr 29 2026, 10:23 PM

@chaliy chaliy merged commit 05ff1b2 into main Apr 29, 2026
16 checks passed
@chaliy chaliy deleted the feat/site-sitemap branch April 29, 2026 22:32
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 29, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@chaliy chaliy mentioned this pull request Apr 30, 2026
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant