Skip to content

Post-deploy fixes: BreadcrumbList dedupe, relative 301s, sitemap lastmod - #224

Merged
buddies2705 merged 2 commits into
mainfrom
docs/post-deploy-fixes
Jul 24, 2026
Merged

Post-deploy fixes: BreadcrumbList dedupe, relative 301s, sitemap lastmod#224
buddies2705 merged 2 commits into
mainfrom
docs/post-deploy-fixes

Conversation

@buddies2705

@buddies2705buddies2705 commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Three cleanups found by testing the live site after the docs-audit deploy.

1. Duplicate BreadcrumbList JSON-LD. The Docusaurus theme already emits a BreadcrumbList for its visual breadcrumbs, so plugins/tech-article-jsonld.js was adding a second one on every page. Keep only TechArticle in the plugin (removes the dead breadcrumb/humanize helpers). Verified: BreadcrumbList per page 2 → 1, TechArticle unchanged.

2. Internal 301s emitted http:// Location. Behind TLS termination nginx sees plain http, so rewrite … permanent produced http://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>. Set lastmod: "date" so every doc URL carries a <lastmod> from its last git commit (showLastUpdateTime is 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

Gaurav agarwaland others added 2 commits July 24, 2026 21:12
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>
@buddies2705buddies2705 changed the title Post-deploy fixes: dedupe BreadcrumbList JSON-LD + relative 301 schemePost-deploy fixes: BreadcrumbList dedupe, relative 301s, sitemap lastmodJul 24, 2026
@buddies2705
buddies2705 merged commit 6332c29 into mainJul 24, 2026
1 check passed
Kshitij0O7 pushed a commit to Kshitij0O7/streaming-data-platform-docs that referenced this pull request Aug 5, 2026
node:20-alpine ships without git, and Docusaurus derives several things by
shelling out to `git log`:
- sitemap <lastmod> (added in bitquery#224 — currently emits nothing live)
- "Last updated on … by …" (showLastUpdateTime/Author — silently missing
in production on ALL pages, pre-existing)
- TechArticle datePublished/dateModified (tech-article-jsonld plugin)
Verified on the deployed site: 0 <lastmod> in sitemap.xml, no "Last updated"
footer, and no date fields in the TechArticle JSON-LD — while the same commit
built locally (where git exists) produces 532 <lastmod> entries and full dates.
Adding `apk add --no-cache git` to the builder stage fixes all three.
Requires the build context to include .git (there is no .dockerignore, so it
does today) and a non-shallow checkout.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Sign up for freeto 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

@buddies2705