fix(seo): remove deprecated docs sitemap hints - #10
Open
hasanalituran wants to merge 1 commit into
Open
hasanalituran wants to merge 1 commit into
hasanalituran wants to merge 1 commit into
Conversation
This branch has not been deployed
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
1 participant
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.
Summary\n\n- Remove deprecated changefreq and priority sitemap hints from the generated Docusaurus sitemap.\n- Preserve source-derived lastmod values and the existing URL filtering.\n\n## Evidence and affected URLs\n\n- Current live sitemap: https://docs.testream.app/sitemap.xml\n- The live sitemap currently returns 200, contains 36 canonical docs URLs, and emits changefreq and priority for all 36 entries.\n- The generated sitemap from this branch keeps the same 36-URL set while removing those deprecated tags.\n- Representative affected URLs: https://docs.testream.app/, https://docs.testream.app/getting-started/quick-start, https://docs.testream.app/reporters/vitest, and https://docs.testream.app/reporters/jasmine.\n\n## Validation performed\n\n- npm run typecheck passed.\n- npm run build passed.\n- npm run postbuild passed; the existing 404 noindex behavior was preserved.\n- npm run test:indexnow passed: 3 tests.\n- npm run indexnow:dry-run -- --sitemap build/sitemap.xml passed and prepared 36 URLs.\n- Generated sitemap assertions passed: 36 unique URLs, zero changefreq or priority tags, no search or plan URLs, and one 404 noindex marker.\n- Generated and live sitemap URL sets matched.\n- git diff origin/main...HEAD --check passed.\n\n## Risks and data gaps\n\n- This is metadata cleanup only; it does not change page copy, canonicals, index controls, or the URL set.\n- The live sitemap will remain unchanged until the docs branch is merged and deployed.\n- No provider UI action or recrawl request was performed.\n- Existing docs PR #6 contains the older version of this same fix and is currently marked dirty; this branch is based on the current docs origin/main so it can be reviewed cleanly.\n\n## Manual tasks for Hasan\n\n- [ ] Review and approve this focused sitemap configuration change; no copy approval is required.\n- [ ] If this PR is accepted, close or supersede the older dirty PR #6.\n- [ ] Merge and deploy through the normal GitHub Pages workflow.\n- [ ] After deployment, verify https://docs.testream.app/sitemap.xml returns 200, still lists 36 canonical URLs, and no longer contains changefreq or priority.\n- [ ] Allow or request the normal Google/Bing recrawl after deployment.\n- [ ] Perform manual QA on the docs homepage and one reporter page after deployment.\n- [ ] No credential or admin action is required.