Captions opt-in + YouTube downloads go anonymous-first - #84
Closed
SecretivePie495 wants to merge 5 commits into
Closed
SecretivePie495 wants to merge 5 commits into
SecretivePie495 wants to merge 5 commits into
Conversation
Users can now drag the hook overlay to any point on the frame instead of only top/center/bottom. Threaded x_pct/y_pct through the ffmpeg overlay path (hooks.py, /api/hook) and both Remotion HookOverlay copies (client preview + render-service) so drag position, live preview, and the final burned video all agree. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Fits the source's native 16:9 frame into a 1080x1920 canvas with black bars top/bottom instead of cropping, for creators who want Shorts/Reels/TikTok delivery without losing any of the shot. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
…choice deb.debian.org drops the connection mid-batch on the deploy VM, failing image builds; retry once with --fix-missing. Also remember the last picked output format (vertical/square/horizontal/Boxed) in localStorage like the other upload preferences. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Captions move from always-on to opt-in on every surface: the dashboard sends an explicit captions flag (checkbox in advanced options, remembered in localStorage), AUTO_CAPTIONS defaults off in main.py, and the MCP process_video tool defaults captions=false. Bare rerender/reframe calls no longer re-burn a layer the caller never asked for; add_subtitles stays the way to caption a finished clip.
Every download and probe attempt now leaves without the account cookiejar: with YOUTUBE_COOKIES attached YouTube answers UNPLAYABLE for every client on a share of videos (prod, 9-sep: cookies -> unavailable, anonymous on the same static -> 1080p), and the PO-token provider serves the HD path without them. A single authed attempt rides at the end of the chain, on a free route, for age-gated/private content and the static-IP bot-check bursts the cookies actually fix. DIRECT_FIRST no longer requires a cookie jar. Dockerfile: NodeSource Node 22 — bgutil's script-node PO provider needs >=22, Debian's apt node is 20.
Author
|
Closing: landed via my fork's own flow instead (no write access to upstream). |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
feat(captions): ship clips without burned subtitles unless asked
fix(youtube): go out anonymously first, cookies only as the last try