Repository navigation
3 second hook cut; background work as one inline line beside Videos - #90
Merged
Merged
Conversation
… strip above the player
…o of moments with a single hook
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
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.
None of the 6 clips on Circulate (2026-11-10) got a hook. The hook was rounded out to whole transcript lines and dropped past 6 s. The transcript's lines are 4.2 s median, so a hook line that ran from one transcript line into the next became about 8 s and was thrown away.
The hook is now what an editor means by it: a 3 second cut at the start of the timeline, taken from where the clip's strongest line starts, followed by the moment.
hook_startonly. It's snapped back to the line's own start when it is within 0.6 s, so the cut never starts mid-word.The clips proposal also says what it does: it reads the video and makes new videos in this project, and this cut stays as it is. It used to say 'watch a clip … applies as one step you can undo'.
Tests: 307 unit tests (new: a hook line across two transcript lines, the Circulate case) and all e2e pass.
The job strip becomes one inline line
Feedback from Circulate: the 'Made 6 clips' strip above the player took space it didn't need. Inside Clawnify, background work now shows as one short line beside the Videos menu, and only while there is something to say:
Hidden on phones, where the AI button's spinner already shows a job running.
Separate videos, or one video of the moments
'5/6 cut downs, with a strong hook and interesting first 3 seconds' can mean separate videos, each with its own hook, or one video of the moments behind a single hook.
make_clips { together: true }makes ONE new video in the project: a 3 second hook from the strongest moment (its hook line, or its own opening when it has none, skipped if the video already starts there), then the moments in order, framed, with captions.Tests: 310 unit tests (3 new for one-video mode) and all e2e pass.