conrad/media
Watch
1
0
Fork
You've already forked media
 0
Code Pull requests Releases Packages Wiki Activity Actions

Complete cached media using Safari-compatible OPFS rename #12

Merged
conrad merged 1 commit from codex/safari-cache-rename into main 2026-09-06 04:37:08 -04:00
Conversation 0 Commits 1 Files changed 2 +8 -7
conrad commented 2026-09-06 04:36:41 -04:00
Owner
Copy link

Safari exposes FileSystemFileHandle.move but requires both the destination directory and the new name. The one-argument call failed after downloading all bytes, leaving media partly cached at 100% and preventing books from opening. Pass both arguments and retain the existing copy fallback for browsers without move.

Validation: 55 blob cache tests, TypeScript, and the production build pass. A real Safari dedicated worker reproduced the old TypeError and verified that the corrected OpfsStore rename preserves file bytes and removes the old entry. Temporary verification fixtures were removed before building.

Safari exposes FileSystemFileHandle.move but requires both the destination directory and the new name. The one-argument call failed after downloading all bytes, leaving media partly cached at 100% and preventing books from opening. Pass both arguments and retain the existing copy fallback for browsers without move. Validation: 55 blob cache tests, TypeScript, and the production build pass. A real Safari dedicated worker reproduced the old TypeError and verified that the corrected OpfsStore rename preserves file bytes and removes the old entry. Temporary verification fixtures were removed before building.
Complete cached media with Safari-compatible OPFS rename
All checks were successful
deploy-fly / validate (push) Successful in 9m15s
Details
deploy-fly / deploy (push) Successful in 6m27s
Details
2704f570fe
conrad merged commit 2704f570fe into main 2026-09-06 04:37:08 -04:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
conrad/media!12
No description provided.