Skip to content

fix: handle array subtitle format in video export#437

Merged
Movm merged 4 commits intotest-branchfrom
fix/nginx-dns-caching
Feb 1, 2026
Merged

fix: handle array subtitle format in video export#437
Movm merged 4 commits intotest-branchfrom
fix/nginx-dns-caching

Conversation

@Movm
Copy link
Copy Markdown
Collaborator

@Movm Movm commented Feb 1, 2026

Summary

  • Fix video export failing when subtitles arrive as a pre-parsed array (with start/end keys) instead of the raw string format
  • Normalise both subtitle shapes before passing to ASS generation
  • Fix pre-existing empty catch block lint errors in processingController

Test plan

  • Export a video with subtitles and verify the export completes successfully
  • Test with both manually edited and auto-generated subtitles

Movm added 4 commits February 1, 2026 03:16
- Only set cameraState=ready from jscanify import callback
- Always capture photo, fall back to raw frame without scanner
Drop the 8MB OpenCV CDN dependency and jscanify edge detection.
Camera now captures raw frames directly via react-webcam — simpler,
faster to load, and no WASM initialization issues.
The subtitles payload can arrive as a pre-parsed array with start/end
keys instead of the raw string format. Normalise both shapes before
passing to ASS generation to prevent export failures.
- Fix TS2322/TS2362 in processingController by wrapping unknown types with Number()/String()
- Replace @ts-ignore with @ts-expect-error in FluxImageService
- Add Sentry error tracking integration for api, web, and sites packages
- Update scraper and package dependencies
@Movm Movm merged commit 9545930 into test-branch Feb 1, 2026
10 of 11 checks passed
@Movm Movm deleted the fix/nginx-dns-caching branch March 10, 2026 16:15
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.

1 participant