Skip to content

Fix image cache for website#9591

Merged
michaelstaib merged 4 commits intomainfrom
mst/fix-images-website
Apr 23, 2026
Merged

Fix image cache for website#9591
michaelstaib merged 4 commits intomainfrom
mst/fix-images-website

Conversation

@michaelstaib
Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings April 23, 2026 10:18
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the website’s image optimization caching behavior to avoid producing pages that reference optimized image variants that are not present on disk (especially in CI deploys), and includes a small UI styling adjustment to the docs article version navigation.

Changes:

  • Add a disk-presence verification step for cached optimized image variants before reusing the cache manifest.
  • Adjust docs article version dialog sizing/nowrap to better fit version labels.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
website/src/components/articles/doc-article-navigation.tsx Tweaks version switcher dialog/link styling to prevent wrapping and allow content-sized width.
website/lib/image-optimization.ts Validates cached optimized image variants exist in public/optimized before trusting the persisted manifest cache.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread website/lib/image-optimization.ts
{latestBlogPost.featuredImage && (
<TeaserImage>
<img
src={latestBlogPost.featuredImage}
@michaelstaib michaelstaib merged commit 871816d into main Apr 23, 2026
134 of 136 checks passed
@michaelstaib michaelstaib deleted the mst/fix-images-website branch April 23, 2026 10:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants