ship cutting a stretch out of a recording, plus cleanup and preset renames - #19
Merged
Conversation
a name that says what it does to the file, not where it is going; 'chat' left the reader to infer 'small' with no clue in the menu itself
closes the gap chat/tiny left: nothing pushed crf the other way, and a reviewer reading a diff or dense UI needs that more than a small file
a plain word beats an abbreviation in a menu with no room for a tooltip
noxend
force-pushed
the
simplify-config
branch
from
August 13, 2026 09:57
b59d914 to
85434ae
Compare
the setting itself was already dropped when output naming switched to the preset name; this example line in the shipped config was missed at the time
ffmpeg's trim= just clamps an out-of-bounds range to the real length, so it was accepted, logged as 'cut applied', and shipped the source untouched; found reviewing PR #19 before merge, since this is exactly the redaction use case the feature exists for
the two mv calls were independent statements, so a locked or permission-denied source left its sidecar archived alone while the video stayed stuck in input/; found reviewing PR #19 before merge
a bare prefix glob also matched an unrelated recording sharing the first few characters of its name, e.g. clip.mov / clip 2.mov, and warned about a sidecar that was never meant for the file being processed; found reviewing PR #19 before merge
0-0:20 already worked (0 is unambiguous), so only the end side needed anything new. A bare trailing dash (2:30-) would have worked too, but a leading one (-0:20) reads exactly like a negative number to anyone who has not read this file's own rules -- end is a real word instead
noxend
force-pushed
the
simplify-config
branch
from
August 15, 2026 12:15
d2b6da0 to
64d1a09
Compare
noxend
added a commit
that referenced
this pull request
Aug 15, 2026
ffmpeg's trim= just clamps an out-of-bounds range to the real length, so it was accepted, logged as 'cut applied', and shipped the source untouched; found reviewing PR #19 before merge, since this is exactly the redaction use case the feature exists for
noxend
added a commit
that referenced
this pull request
Aug 15, 2026
the two mv calls were independent statements, so a locked or permission-denied source left its sidecar archived alone while the video stayed stuck in input/; found reviewing PR #19 before merge
noxend
added a commit
that referenced
this pull request
Aug 15, 2026
a bare prefix glob also matched an unrelated recording sharing the first few characters of its name, e.g. clip.mov / clip 2.mov, and warned about a sidecar that was never meant for the file being processed; found reviewing PR #19 before merge
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.
Closes #1.
Cuts a stretch out of a recording via a
.cutssidecar next to the video, one range per line.mark-cutsQuick Action opens it for editing.0-0:20cuts the first 20s,2:30-endcuts from 2:30 to the end.Found and fixed three bugs going back over it: an out-of-bounds range said "cut applied" but did nothing, a locked source could leave its
.cutssidecar archived without the video, and the near-miss warning could flag an unrelated file with a similar name.Rest is cleanup from the same stretch of work:
output_dirgone,output/is a symlink, filenames come from the preset name now,chatrenamed totiny, newsharppreset for higher quality.