Skip to content

fix(docs): cache public videos - #901

Merged
zahlekhan merged 3 commits into
thesysdev:mainfrom
zahlekhan:codex/reduce-vercel-fast-data-transfer
Jul 30, 2026
Merged

fix(docs): cache public videos#901
zahlekhan merged 3 commits into
thesysdev:mainfrom
zahlekhan:codex/reduce-vercel-fast-data-transfer

Conversation

@zahlekhan

@zahlekhan zahlekhan commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Summary

  • change the homepage videos from preload="metadata" to preload="none"
  • cache every public .mp4 response in the browser for 24 hours
  • preserve the original video assets, autoplay, looping, sources, and accordion auto-advance behavior

Root cause

The public MP4 assets were served with immediate revalidation, so repeat visits could transfer the same video data again. Metadata preloading could also initiate requests independently of playback.

Impact

Repeat visits within the cache window can reuse local video responses, and the preload hint no longer independently requests metadata. Existing playback and accordion behavior remains unchanged.

Validation

  • git diff --check
  • pnpm exec eslint --rule 'prettier/prettier: off' on the changed source/config files
  • pnpm --filter @openuidev/docs types:check
  • pnpm --filter @openuidev/docs build
  • verified runtime response includes Cache-Control: public, max-age=86400, immutable

@zahlekhan zahlekhan changed the title fix(docs): reduce homepage video transfer fix(docs): cache homepage videos Jul 29, 2026
@zahlekhan zahlekhan changed the title fix(docs): cache homepage videos fix(docs): cache public videos Jul 29, 2026
@zahlekhan
zahlekhan marked this pull request as ready for review July 29, 2026 17:25
@zahlekhan
zahlekhan enabled auto-merge (squash) July 29, 2026 17:25
@zahlekhan
zahlekhan merged commit 21a3be7 into thesysdev:main Jul 30, 2026
2 checks passed
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