Skip to content

fix: preserve blog media paths under developers mount - #828

Merged
up-the-tempo merged 1 commit into
mainfrom
centaur/fix-blog-media-paths-1787158124
Aug 19, 2026
Merged

fix: preserve blog media paths under developers mount#828
up-the-tempo merged 1 commit into
mainfrom
centaur/fix-blog-media-paths-1787158124

Conversation

@decofe

@decofe decofe commented Aug 19, 2026

Copy link
Copy Markdown
Member

Summary

  • Rewrite rendered /blog/... media URLs as mount-relative URLs.
  • Preserve /blog paths in previews and /developers/blog paths on tempo.xyz.
  • Add a regression test covering image, video source, and fallback link URLs.

Production evidence

  • Images currently request /blog/... and return 404 text/html.
  • Videos currently receive the same HTML 404 page as 206 text/html.
  • The corresponding /developers/blog/... assets return 200 image/png and 200 video/mp4.

Testing

  • pnpm exec biome check src/marketing/blogPlugin.ts src/marketing/blogPlugin.test.ts
  • pnpm check:types
  • Generated virtual blog data contains mount-safe paths for every Voight-Kampff image and video
  • Vitest startup is locally blocked by the existing external OpenAPI fetch timeout; CI will run the committed regression test

Prompted by: tom

@vercel

vercel Bot commented Aug 19, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
tempo-docs Ready Ready Preview Aug 19, 2026 4:53pm

Request Review

@up-the-tempo
up-the-tempo merged commit 27ab041 into main Aug 19, 2026
11 checks passed
@up-the-tempo
up-the-tempo deleted the centaur/fix-blog-media-paths-1787158124 branch August 19, 2026 16:56
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.

2 participants