Post-deploy fixes: BreadcrumbList dedupe, relative 301s, sitemap lastmod#224
Merged
Conversation
Two cleanups found while testing the deployed site: - tech-article-jsonld.js: stop injecting BreadcrumbList — the Docusaurus theme already emits a BreadcrumbList for its visual breadcrumbs, so every page had a duplicate. Keep TechArticle only; remove the now-dead breadcrumb/humanize helpers. - nginx: add `absolute_redirect off` — behind TLS termination nginx emitted http:// Location headers on internal 301s (old PumpFun casing, legacy MCP path), forcing an extra http->https upgrade hop. Relative Location keeps https. Verified live pre-fix: real 404s, gzip, all 301s, new pages, robots/llms/sitemap, favicon, GA4 dedupe all working — these were the only two issues. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Set sitemap `lastmod: "date"` so every doc URL carries a <lastmod> derived from its last git commit (showLastUpdateTime is already enabled). 532/573 URLs now have lastmod — the remainder are auto-generated category routes with no source file. lastmod is the one sitemap field Google actually uses for recrawl. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
buddies2705
changed the title
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.
Three cleanups found by testing the live site after the docs-audit deploy.
1. Duplicate
BreadcrumbListJSON-LD. The Docusaurus theme already emits a BreadcrumbList for its visual breadcrumbs, soplugins/tech-article-jsonld.jswas adding a second one on every page. Keep only TechArticle in the plugin (removes the deadbreadcrumb/humanizehelpers). Verified: BreadcrumbList per page 2 → 1, TechArticle unchanged.2. Internal 301s emitted
http://Location. Behind TLS termination nginx sees plain http, sorewrite … permanentproducedhttp://docs.bitquery.io/…Location headers (old PumpFun casing, legacy/docs/usecases/MCP), causing an extra http→https upgrade hop.absolute_redirect off;makes Location relative so the client keeps https.3. Sitemap
<lastmod>. Setlastmod: "date"so every doc URL carries a<lastmod>from its last git commit (showLastUpdateTimeis already on). 532/573 URLs now have lastmod (the rest are auto-generated category routes). lastmod is the one sitemap field Google actually uses for recrawl.Everything else from the audit deploy tested clean (real 404s, gzip, redirects resolve to 200, new pages, robots/llms/sitemap, favicon, GA4 dedupe).
🤖 Generated with Claude Code