mediasnacks 0.14.0 → 0.16.1

package/.github/workflows/test.yml

@@ -0,0 +1,25 @@
+name: Test
+
+on: [ push, pull_request, workflow_dispatch ]
+
+permissions:
+  contents: read
+  pull-requests: write
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v6
+      - uses: docker/setup-buildx-action@v4
+      - uses: docker/build-push-action@v7
+        with:
+          context: .
+          load: true
+          tags: app-test
+          cache-from: type=gha
+          cache-to: type=gha,mode=max
+
+      - name: Tests
+        run: docker run --rm app-test

package/.zsh/completions/_mediasnacks

@@ -2,25 +2,26 @@
 
 _mediasnacks_commands=(
   'avif:Converts images to AVIF'
-  'resize:Resizes videos or images'
   'sqcrop:Square crops images'
-  'moov2front:Rearranges metadata for fast-start streaming'
   'dropdups:Removes duplicate frames in a video'
-  'seqcheck:Finds missing sequence number'
-  'qdir:Sequentially runs all *.sh files in a folder'
-  'hev1tohvc1:Fixes video thumbnails not rendering in macOS Finder'
+  'edgespic:Extracts first and last frames'
   'framediff:ffplay with a filter for diffing adjacent frames'
-  'vdiff:Plays a video with the difference of two videos'
+  'gif:Video to GIF'
+  'hev1tohvc1:Fixes video thumbnails not rendering in macOS Finder'
+  'moov2front:Rearranges metadata for fast-start streaming'
+  'resize:Resizes videos or images'
   'vconcat:Concatenates videos'
+  'vdiff:Plays a video with the difference of two videos'
+  'vsplit:Splits a video into multiple clips from CSV timestamps'
   'vtrim:Trims video from start to end time'
+  'flattendir:Moves unique files to the top dir and deletes empty dirs'
+  'seqcheck:Finds missing sequence number'
+  'qdir:Sequentially runs all *.sh files in a folder'
   'dlaudio: yt-dlp best audio'
   'dlvideo: yt-dlp best video'
   'unemoji:Removes emojis from filenames'
   'rmcover:Removes cover art'
   'curltime:Measures request response timings'
-  'gif:Video to GIF'
-  'vsplit:Split video at multiple time points'
-  'flattendir:Moves unique files to the top dir and deletes empty dirs'
 )
 
 if (( CURRENT == 2 )); then
@@ -30,7 +31,7 @@ fi
 
 local cmd="$words[2]"
 case "$cmd" in
-	avif|resize|sqcrop|moov2front|dropdups|seqcheck|hev1tohvc1|framediff|vdiff|vconcat|vtrim|dlaudio|dlvideo|unemoji|rmcover|curltime|gif|vsplit|flattendir)
+	avif|resize|sqcrop|moov2front|dropdups|edgespic|seqcheck|hev1tohvc1|framediff|vdiff|vconcat|vsplit|vtrim|dlaudio|dlvideo|unemoji|rmcover|curltime|gif|flattendir)
 		_files
 		;;
 	qdir)

package/Dockerfile

@@ -0,0 +1,13 @@
+FROM mwader/static-ffmpeg:8.0.1 AS ffmpeg
+FROM node:24-bookworm-slim
+
+COPY --from=ffmpeg /ffmpeg /usr/local/bin/ffmpeg
+COPY --from=ffmpeg /ffprobe /usr/local/bin/ffprobe
+
+ENV FORCE_COLOR=1
+WORKDIR /workspace
+
+COPY src/ src/
+COPY package.json .
+
+CMD ["node", "--test"]

package/README.md

@@ -3,28 +3,33 @@
 Utilities optimizing and preparing video and images for the web.
 
 
-## Usage Overview
+## Overview
 **FFmpeg and Node.js must be installed.**
 
 ```shell
 npx mediasnacks <command> <args>
 ```
 
-Commands:
+### Commands
 - `avif` Converts images to AVIF
-- `resize` Resizes videos or images
 - `sqcrop` Square crops images
-- `moov2front` Rearranges .mov and .mp4 metadata for fast-start streaming
+ 
 - `dropdups` Removes duplicate frames in a video
-- `seqcheck` Finds missing sequence number
-- `qdir` Sequentially runs all *.sh files in a folder
-- `hev1tohvc1`: Fixes video thumbnails not rendering in macOS Finder
-
+- `edgespic` Extracts first and last frames
 - `framediff`: Plays a video of adjacent frames diff
-- `vdiff`: Plays a video with the difference of two videos
+- `gif`: Video to GIF
+- `hev1tohvc1`: Fixes video thumbnails not rendering in macOS Finder
+- `moov2front` Rearranges .mov and .mp4 metadata for fast-start streaming
+- `resize` Resizes videos or images
 - `vconcat`: Concatenates videos
+- `vdiff`: Plays a video with the difference of two videos
+- `vsplit`: Splits a video into multiple clips from CSV timestamps
 - `vtrim`: Trims video from start to end time
 
+- `flattendir`: Moves unique files to the top dir and deletes empty dirs
+- `qdir` Sequentially runs all *.sh files in a folder
+- `seqcheck` Finds missing sequence number
+
 - `dlaudio`: yt-dlp best audio
 - `dlvideo`: yt-dlp best video
 
@@ -32,32 +37,26 @@ Commands:
 - `rmcover`: Removes cover art
 
 - `curltime`: Measures request response timings
-- `gif`: Video to GIF
-- `vsplit`: Split video at multiple time points
-
-- `flattendir`: Moves unique files to the top dir and deletes empty dirs
-
-### Glob Patterns and Literal Filenames
-
-Most commands accept glob patterns (like `*.png` or `file[234].png`) to match multiple
-files. By default, these patterns are expanded by Node.js to match existing files.
 
-To treat arguments as literal filenames instead of
-glob patterns, use the `--` (double dash) separator:
+### Globs
+Glob patterns are expanded by Node.js to match existing files.
 
 ```shell
-# Expands to: file2.png, file3.png, file4.png
 npx mediasnacks avif file[234].png
+  # Expands to: file2.png, file3.png, file4.png
+```
 
-# Literal filename: "file[234].png"
+```shell
 npx mediasnacks avif -- file[234].png
-
-# Mixed: expand first pattern, treat second as literal
-npx mediasnacks avif file2.png -- file[234].png
+  # Literal filename: "file[234].png"
 ```
 
 <br/>
 
+---
+
+## Commands
+
 ### Converting Images to AVIF
 ```shell
 npx mediasnacks avif [-y | --overwrite] [--output-dir=<dir>] <images> 
@@ -103,5 +102,26 @@ What is Fast Start?
 
 <br/>
 
-### License
-MIT
+---
+
+## Adding a macOS Quick Action
+
+
+![](./docs/macos-quick-action.png)
+
+
+For example, for `dropdups -n2 file.mov`
+
+- Open Automator
+- Select: Quick Action
+- Workflow receives current: `movie files` in `Finder.app`
+- Action: `Run Shell Script`
+```shell
+export PATH="/opt/homebrew/bin"
+for f in "$@"; do
+  $HOME/bin/mediasnacks dropdups -n2 "$f"
+done
+```
+
+It will be saved to `~/Library/Services` 
+

package/generate-fixtures.sh

@@ -0,0 +1,11 @@
+#!/bin/sh
+
+docker run --rm \
+  -v "$(pwd)/src/fixtures:/workspace/src/fixtures" \
+  $(docker build -q .) \
+  /bin/bash -c "
+node --test 
+cp /tmp/avif*/lenna.avif /workspace/src/fixtures/ 
+cp /tmp/edgespic*/edgespic/*.png /workspace/src/fixtures/edgespic/
+"
+

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "mediasnacks",
-  "version": "0.14.0",
-  "description": "Utilities for preparing videos, images, and audio for the web",
+  "version": "0.16.1",
+  "description": "Utilities for optimizing and preparing videos and images for the web",
   "license": "MIT",
   "author": "Eric Fortis",
   "type": "module",
@@ -9,6 +9,6 @@
 		"mediasnacks": "src/cli.js"
 	},
 	"scripts": {
-		"test": "node --test"
+		"test": "docker run --rm $(docker build -q .)"
 	}
 }

package/src/avif.js

@@ -19,8 +19,8 @@ async function main() {
 
 	const { values, files } = await parseOptions({
 		'output-dir': { type: 'string', default: '' },
-		overwrite: { short: 'y', type: 'boolean', default: false },
-		help: { short: 'h', type: 'boolean', default: false },
+		overwrite: { short: 'y', type: 'boolean' },
+		help: { short: 'h', type: 'boolean' },
 	})
 
 	if (values.help) {

package/src/avif.test.js

@@ -0,0 +1,19 @@
+import { join } from 'node:path'
+import { test } from 'node:test'
+import { deepEqual } from 'node:assert/strict'
+
+import { videoAttrs } from './utils/ffmpeg.js'
+import { mkTempDir, cli } from './utils/test-utils.js'
+
+const rel = f => join(import.meta.dirname, f)
+
+test('PNG to AVIF', async () => {
+	const tmp = mkTempDir('avif')
+	cli('avif', '--output-dir', tmp, rel('fixtures/lenna.png'))
+
+	deepEqual(
+		await videoAttrs(join(tmp, 'lenna.avif')),
+		await videoAttrs(rel('fixtures/lenna.avif')))
+	// That's because we use non-deterministic avif.
+	// Claude says: avif is deterministic only when it's single-threaded: '-threads 1'
+})

package/src/cli.js

@@ -7,39 +7,39 @@ import pkgJSON from '../package.json' with { type: 'json' }
 
 const COMMANDS = {
 	avif: ['avif.js', 'Converts images to AVIF'],
-	resize: ['resize.js', 'Resizes videos or images'],
-	sqcrop: ['sqcrop.js', 'Square crops images'],
-	moov2front: ['moov2front.js', 'Rearranges .mov and .mp4 metadata for fast-start streaming'],
+	sqcrop: ['sqcrop.js', 'Square crops images\n'],
 
 	dropdups: ['dropdups.js', 'Removes duplicate frames in a video'],
-	seqcheck: ['seqcheck.js', 'Finds missing sequence number'],
-	qdir: ['qdir.js', 'Sequentially runs all *.sh files in a folder'],
-	hev1tohvc1: ['hev1tohvc1.js', 'Fixes video thumbnails not rendering in macOS Finder '],
-
+	edgespic: ['edgespic.js', 'Extracts first and last frames'],
 	framediff: ['framediff.sh', 'Plays a video of adjacent frames diff'],
+	gif: ['gif.sh', 'Video to GIF'],
+	hev1tohvc1: ['hev1tohvc1.js', 'Fixes video thumbnails not rendering in macOS Finder '],
+	moov2front: ['moov2front.js', 'Rearranges .mov and .mp4 metadata for fast-start streaming'],
+	resize: ['resize.js', 'Resizes videos or images'],
+	vconcat: ['vconcat.sh', 'Concatenates videos'],
 	vdiff: ['vdiff.sh', 'Plays a video with the difference of two videos'],
+	vsplit: ['vsplit.js', 'Splits a video into multiple clips from CSV timestamps'],
+	vtrim: ['vtrim.sh', 'Trims video from start to end time\n'],
 
-	vconcat: ['vconcat.sh', 'Concatenates videos'],
-	vtrim: ['vtrim.sh', 'Trims video from start to end time'],
+	flattendir: ['flattendir.sh', 'Moves all files to top dir and deletes dirs'],
+	qdir: ['qdir.js', 'Sequentially runs all *.sh files in a folder'],
+	seqcheck: ['seqcheck.js', 'Finds missing sequence number\n'],
 
 	dlaudio: ['dlaudio.sh', 'yt-dlp best audio'],
-	dlvideo: ['dlvideo.sh', 'yt-dlp best video'],
+	dlvideo: ['dlvideo.sh', 'yt-dlp best video\n'],
 
 	unemoji: ['unemoji.sh', 'Removes emojis from filenames'],
-	rmcover: ['rmcover.sh', 'Removes cover art'],
-	
+	rmcover: ['rmcover.sh', 'Removes cover art\n'],
+
 	curltime: ['curltime.sh', 'Measures request response timings'],
-	gif: ['gif.sh', 'Video to GIF'],
-	vsplit: ['vsplit.sh', 'Split video at multiple time points'],
-	flattendir: ['flattendir.sh', 'Moves all files to top dir and deletes dirs']
 }
 
 const USAGE = `
-Usage: npx mediasnacks <command> <args>
+Usage: mediasnacks <command> <args>
 
 Commands:
 ${Object.entries(COMMANDS).map(([cmd, [, title]]) =>
-	`    ${cmd}\t${title}`).join('\n')}
+	`   ${cmd.padEnd(12, ' ')}\t${title}`).join('\n')}
 `.trim()
 
 

package/src/dropdups.js

@@ -36,7 +36,7 @@ async function main() {
 
 	const { values, files } = await parseOptions({
 		'bad-frame-number': { short: 'n', type: 'string', default: '' },
-		help: { short: 'h', type: 'boolean', default: false },
+		help: { short: 'h', type: 'boolean' },
 	})
 
 	if (values.help) {

package/src/edgespic.js

@@ -0,0 +1,77 @@
+#!/usr/bin/env node
+
+import { basename, extname, join, parse } from 'node:path'
+
+import { mkDir } from './utils/fs-utils.js'
+import { parseOptions } from './utils/parseOptions.js'
+import { ffmpeg, videoAttrs, assertUserHasFFmpeg } from './utils/ffmpeg.js'
+
+
+const USAGE = `
+Usage: npx mediasnacks edgespic [--width=<num>] <files>
+
+Extracts the first and last frames from each video and saves them to the 'edgepics/' subfolder.
+--width defaults to 640px and The aspect ratio is preserved.
+
+Example:
+    npx mediasnacks edgespic --width 800 *.mov
+    npx mediasnacks edgespic -w 600 'videos/**/*.mp4'
+`.trim()
+
+
+async function main() {
+	await assertUserHasFFmpeg()
+
+	const { values, files } = await parseOptions({
+		'width': { short: 'w', type: 'string', default: '640' },
+		help: { short: 'h', type: 'boolean' },
+	})
+
+	if (values.help) {
+		console.log(USAGE)
+		process.exit(0)
+	}
+
+	const width = Number(values['width'])
+	if (width <= 0 || !Number.isInteger(width))
+		throw new Error('--width must be a positive number')
+
+	if (!files.length)
+		throw new Error('No video files specified')
+
+	const outDir = join(parse(files[0]).dir, 'edgespic')
+	await mkDir(outDir) 
+
+	console.log('Extracting edge frames…')
+	for (const file of files)
+		await extractEdgeFrames(file, width, outDir)
+}
+
+
+async function extractEdgeFrames(video, width, outDir) {
+	const { r_frame_rate } = await videoAttrs(video)
+	const name = basename(video, extname(video))
+
+	await ffmpeg([
+		'-y',
+		'-i', video,
+		'-vf', `scale=${width}:-1`,
+		'-frames:v', '1',
+		join(outDir, `${name}_first.png`)
+	])
+
+	await ffmpeg([
+		'-y',
+		'-sseof', -1 / eval(r_frame_rate),
+		'-i', video,
+		'-vf', `scale=${width}:-1`,
+		'-frames:v', '1',
+		join(outDir, `${name}_last.png`)
+	])
+}
+
+
+main().catch(err => {
+	console.error(err.message)
+	process.exit(1)
+})

package/src/edgespic.test.js

@@ -0,0 +1,32 @@
+import { join } from 'node:path'
+import { ok, equal } from 'node:assert/strict'
+import { describe, test } from 'node:test'
+import { cpSync, readdirSync, } from 'node:fs'
+
+import { cli, mkTempDir, sha1 } from './utils/test-utils.js'
+
+const rel = f => join(import.meta.dirname, f)
+
+describe('edgespic', () => {
+	const tmp = mkTempDir('edgespic')
+	const inputFile = join(tmp, '60fps.mp4')
+	cpSync(rel('fixtures/60fps.mp4'), inputFile)
+	cli('edgespic', inputFile)
+
+	test('creates output directory', () => {
+		const files = readdirSync(join(tmp, 'edgespic'))
+		ok(files.length === 2, `Expected 2 PNG files, got ${files.length}`)
+	})
+
+	test('extracts first frame', () => {
+		const fixture = rel('fixtures/edgespic/60fps_first.png')
+		const generated = join(tmp, 'edgespic', '60fps_first.png')
+		equal(sha1(generated), sha1(fixture))
+	})
+
+	test('extracts last frame', () => {
+		const fixture = rel('fixtures/edgespic/60fps_last.png')
+		const generated = join(tmp, 'edgespic', '60fps_last.png')
+		equal(sha1(generated), sha1(fixture))
+	})
+})

package/src/fixtures/60fps.csv

@@ -0,0 +1,7 @@
+start,end
+0,5
+5,10
+10,15
+15,20
+20,25
+25,30

package/src/gif.sh

@@ -1,6 +1,6 @@
-#!/bin/zsh
+#!/bin/sh
 
-# Convert to GIF
+# Converts to GIF
 
 FPS=10
 WIDTH=600
@@ -10,7 +10,7 @@ usage() {
   exit 1
 }
 
-while [[ $# -gt 0 ]]; do
+while [ $# -gt 0 ]; do
   case "$1" in
     --fps)
       FPS="$2";
@@ -26,10 +26,8 @@ while [[ $# -gt 0 ]]; do
   esac
 done
 
-[[ -z "$file" ]] && usage
-
-outfile="${file:r}.gif"
+[ -z "$file" ] && usage
 
 ffmpeg -v error -i "$file" \
   -vf "fps=${FPS},scale=${WIDTH}:-1" \
-  "$outfile"
+  "${file%.*}.gif"

package/src/qdir.js

@@ -12,17 +12,14 @@ import { isFile } from './utils/fs-utils.js'
 const USAGE = `
 Usage: npx mediasnacks qdir [folder]
 
-Sequentially runs all *.sh files in a folder. 
-It uses the current working directory by default.
-
--h, --help
+Sequentially runs all *.sh files in a folder (cwd by default). 
 `.trim()
 
 
 async function main() {
 	const { values, positionals } = parseArgs({
 		options: {
-			help: { short: 'h', type: 'boolean', default: false },
+			help: { short: 'h', type: 'boolean' },
 		},
 		allowPositionals: true,
 	})
@@ -95,7 +92,7 @@ function sleep(ms) {
 }
 
 
-if (fileURLToPath(import.meta.url) === process.argv[1])
+if (import.meta.main)
 	main().catch(err => {
 		console.error(err.message || err)
 		process.exit(1)

package/src/qdir.test.js

@@ -0,0 +1,24 @@
+import { test } from 'node:test'
+import { join } from 'node:path'
+import { equal, deepEqual } from 'node:assert/strict'
+import { cpSync, readdirSync } from 'node:fs'
+
+import { qdir } from './qdir.js'
+import { mkTempDir } from './utils/test-utils.js'
+
+const rel = f => join(import.meta.dirname, f)
+
+test('qdir-jobs get renamed and failed have their exit status code', async () => {
+	const tmp = mkTempDir('qdir')
+	cpSync(rel('fixtures/qdir-jobs'), tmp, { recursive: true })
+
+	const err = await qdir(tmp, 0.2)
+	equal(err, null)
+
+	deepEqual(readdirSync(tmp).sort(), [
+		'job1_good.sh.done',
+		'job2_bad.sh.failed.1',
+		'job3_good.sh.done',
+		'job4_bad.sh.failed.1'
+	])
+})

package/src/resize.js

@@ -34,8 +34,8 @@ async function main() {
 		width: { type: 'string', default: '-2' },
 		height: { type: 'string', default: '-2' },
 		'output-dir': { type: 'string', default: '' },
-		overwrite: { short: 'y', type: 'boolean', default: false },
-		help: { short: 'h', type: 'boolean', default: false },
+		overwrite: { short: 'y', type: 'boolean' },
+		help: { short: 'h', type: 'boolean' },
 	})
 
 	if (values.help) {

package/src/seqcheck.js

@@ -22,7 +22,7 @@ function main() {
 		options: {
 			'left-delimiter': { type: 'string', default: '_' },
 			'right-delimiter': { type: 'string', default: '.' },
-			help: { short: 'h', type: 'boolean', default: false },
+			help: { short: 'h', type: 'boolean' },
 		},
 		allowPositionals: true,
 	})
@@ -68,5 +68,5 @@ function escapeRegex(str) {
 }
 
 
-if (fileURLToPath(import.meta.url) === process.argv[1])
+if (import.meta.main)
 	main()

package/{tests → src}/seqcheck.test.js

@@ -1,6 +1,6 @@
 import { test } from 'node:test'
 import { deepEqual } from 'node:assert/strict'
-import { extractSeqNums, findMissingNumbers } from '../src/seqcheck.js'
+import { extractSeqNums, findMissingNumbers } from './seqcheck.js'
 
 
 test('extractSeqNums extracts sequence numbers from filenames', () => {

package/src/sqcrop.js

@@ -1,8 +1,8 @@
 #!/usr/bin/env node
 
 import { join } from 'node:path'
-
 import { rename } from 'node:fs/promises'
+
 import { ffmpeg, assertUserHasFFmpeg } from './utils/ffmpeg.js'
 import { lstat, uniqueFilenameFor } from './utils/fs-utils.js'
 import { parseOptions } from './utils/parseOptions.js'
@@ -20,8 +20,8 @@ async function main() {
 
 	const { values, files } = await parseOptions({
 		'output-dir': { type: 'string', default: '' },
-		overwrite: { short: 'y', type: 'boolean', default: false },
-		help: { short: 'h', type: 'boolean', default: false },
+		overwrite: { short: 'y', type: 'boolean' },
+		help: { short: 'h', type: 'boolean' },
 	})
 
 	if (values.help) {

package/src/utils/fs-utils.js

@@ -1,6 +1,6 @@
 import { lstatSync } from 'node:fs'
 import { randomUUID } from 'node:crypto'
-import { unlink, rename } from 'node:fs/promises'
+import { mkdir, unlink, rename } from 'node:fs/promises'
 import { dirname, extname, join } from 'node:path'
 
 
@@ -23,3 +23,13 @@ export async function overwrite(src, target) {
 	await unlink(target)
 	await rename(src, target)
 }
+
+export async function mkDir(path) {
+	try {
+		await mkdir(path, { recursive: true })
+	}
+	catch (err) {
+		if (err.code !== 'EEXIST')
+			throw err
+	}
+}

package/src/utils/parseOptions.test.js

@@ -1,7 +1,7 @@
 import { join } from 'node:path'
 import { tmpdir } from 'node:os'
 import { equal, deepEqual } from 'node:assert/strict'
-import { mkdtemp, writeFile, rmdir } from 'node:fs/promises'
+import { mkdtemp, writeFile, rm } from 'node:fs/promises'
 import { test, describe, before, after } from 'node:test'
 
 import { parseOptions } from './parseOptions.js'
@@ -18,7 +18,7 @@ describe('parseOptions', () => {
 			await writeFile(inTmpDir(file), '')
 	})
 
-	after(() => rmdir(testDir, { recursive: true }))
+	after(() => rm(testDir, { recursive: true }))
 
 	test('parses args and globs files', async () => {
 		const { values, files } = await parseOptions({

package/src/utils/test-utils.js

@@ -0,0 +1,21 @@
+import { join } from 'node:path'
+import { tmpdir } from 'node:os'
+import { spawnSync } from 'node:child_process'
+import { createHash } from 'node:crypto'
+import { mkdtempSync, readFileSync } from 'node:fs'
+
+const rel = f => join(import.meta.dirname, f)
+
+export function mkTempDir(prefix = 'test-') {
+	return mkdtempSync(join(tmpdir(), prefix))
+}
+
+export function cli(...args) {
+	spawnSync(rel('../cli.js'), args)
+}
+
+export function sha1(filePath) {
+	return createHash('sha1')
+		.update(readFileSync(filePath))
+		.digest('base64')
+}

package/src/vconcat.sh

@@ -1,22 +1,24 @@
-#!/bin/zsh
+#!/bin/sh
 
-if (( $# < 2 )); then
+if [ "$#" -lt 2 ]; then
   cat << EOF
 Usage:
-    $(basename $0) vid1.mov vid2.mov [...]
-    $(basename $0) *.mp4
+    $(basename "$0") vid1.mov vid2.mov [...]
+    $(basename "$0") *.mp4
 EOF
   exit 1
 fi
 
 list_file=$(mktemp -p .)
 for file in "$@"; do
-  printf "file %q\n" "$file" >> "$list_file"
+  fname=$(printf '%s' "$file" | sed "s/'/'\\\\''/g") # Escape single quotes
+  printf "file '%s'\n" "$fname" >> "$list_file"
 done
 
 first_video="$1"
-outfile="${first_video:r}.concat.${first_video:e}"
+name="${first_video%.*}"
+ext="${first_video##*.}"
+outfile="${name}.concat.${ext}"
 
 ffmpeg -v error -f concat -safe 0 -i "$list_file" -c copy "$outfile"
-
 rm "$list_file"

package/src/vconcat.test.js

@@ -0,0 +1,23 @@
+import { ok } from 'node:assert/strict'
+import { join } from 'node:path'
+import { test } from 'node:test'
+import { cpSync } from 'node:fs'
+
+import { videoAttrs } from './utils/ffmpeg.js'
+import { mkTempDir, cli } from './utils/test-utils.js'
+
+const rel = f => join(import.meta.dirname, f)
+
+test('vconcat concatenates videos with single quotes in filenames', async () => {
+	const tmp = mkTempDir('vconcat')
+
+	const file1 = join(tmp, `video'1.mp4`)
+	const file2 = join(tmp, `video'2.mp4`)
+	cpSync(rel('fixtures/60fps.mp4'), file1)
+	cpSync(rel('fixtures/60fps.mp4'), file2)
+
+	cli('vconcat', file1, file2)
+
+	const { duration } = await videoAttrs(join(tmp, `video'1.concat.mp4`))
+	ok(parseFloat(duration) === 60, `Duration should be 60s, got ${duration}s`)
+})

package/src/vsplit.js

@@ -0,0 +1,100 @@
+#!/usr/bin/env node
+
+import { readFileSync } from 'node:fs'
+import { resolve, parse, join } from 'node:path'
+
+import { parseOptions } from './utils/parseOptions.js'
+import { assertUserHasFFmpeg, run } from './utils/ffmpeg.js'
+
+
+// TODO looks like it's missing a frame (perhaps becaue of -c copy)
+
+const USAGE = `
+Usage: npx mediasnacks vsplit <csv> <video>
+
+Splits a video into multiple clips from CSV timestamps.
+
+Arguments:
+  <csv>    CSV file with start,end columns (in seconds)
+  <video>  Video file to split
+
+Example:
+  npx mediasnacks vsplit clips.csv video.mov
+
+  Given clips.csv:
+    start,end
+    0,5
+    5,10
+    10,15
+
+  Outputs: video_1.mov, video_2.mov, video_3.mov
+`.trim()
+
+
+async function main() {
+	await assertUserHasFFmpeg()
+
+	const { values, files } = await parseOptions({
+		help: { short: 'h', type: 'boolean' },
+	})
+
+	if (values.help) {
+		console.log(USAGE)
+		process.exit(0)
+	}
+
+	if (files.length !== 2)
+		throw new Error('Expected 2 arguments: CSV file and video file. See npx mediasnacks vsplit --help')
+
+	const [csvPath, videoPath] = files.map(f => resolve(f))
+
+	const clips = parseCSV(csvPath)
+	if (!clips.length)
+		throw new Error('CSV file contains no clips')
+
+	console.log(`Splitting video into ${clips.length} clip${clips.length === 1 ? '' : 's'}…`)
+	await splitVideo(videoPath, clips)
+}
+
+function parseCSV(csvPath) {
+	const content = readFileSync(csvPath, 'utf8')
+	const lines = content.split('\n').filter(line => line.trim())
+
+	if (!lines.length)
+		throw new Error('CSV file is empty')
+
+	const clips = []
+	for (let i = 1; i < lines.length; i++) { // unconditionally skips header 
+		const parts = lines[i].split(',').map(s => s.trim())
+
+		if (parts.length !== 2)
+			throw new Error(`Invalid CSV format at line ${i + 1}: expected 2 columns, got ${parts.length}`)
+
+		clips.push(parts)
+	}
+	return clips
+}
+
+async function splitVideo(videoPath, clips) {
+	const { dir, name, ext } = parse(videoPath)
+	const seqLen = Math.log10(clips.length) + 1 | 0
+
+	for (let i = 0; i < clips.length; i++) {
+		const [start, end] = clips[i]
+		const seq = String(i + 1).padStart(seqLen, '0')
+		await run('ffmpeg', [
+			'-v', 'error',
+			'-ss', start,
+			'-to', end,
+			'-i', videoPath,
+			'-c', 'copy',
+			'-y',
+			join(dir, `${name}_${seq}${ext}`)
+		])
+	}
+}
+
+main().catch(err => {
+	console.error(err.message || err)
+	process.exit(1)
+})

package/src/vsplit.test.js

@@ -0,0 +1,32 @@
+import { ok } from 'node:assert/strict'
+import { join } from 'node:path'
+import { describe, test } from 'node:test'
+import { cpSync, readdirSync } from 'node:fs'
+
+import { videoAttrs } from './utils/ffmpeg.js'
+import { mkTempDir, cli } from './utils/test-utils.js'
+
+
+const rel = f => join(import.meta.dirname, f)
+
+describe('vsplit splits video into multiple clips from CSV', () => {
+	const tmp = mkTempDir('vsplit')
+
+	const csvFile = join(tmp, '60fps.csv')
+	const inputFile = join(tmp, '60fps.mp4')
+
+	cpSync(rel('fixtures/60fps.csv'), csvFile)
+	cpSync(rel('fixtures/60fps.mp4'), inputFile)
+	cli('vsplit', csvFile, inputFile)
+
+	test('all 6 clips were created', () => {
+		const files = readdirSync(tmp).filter(f => f.startsWith('60fps_'))
+		ok(files.length === 6, `Expected 6 clips, got ${files.length}`)
+	})
+
+	test('first clip has correct duration (5 seconds)', async () => {
+		const { duration } = await videoAttrs(join(tmp, '60fps_1.mp4'))
+		const EPSILON = 0.05
+		ok(Math.abs(parseFloat(duration) - 5) < EPSILON, `Duration should be 5s, got ${duration}s`)
+	})
+})

package/src/vtrim.sh

@@ -1,4 +1,4 @@
-#!/bin/zsh
+#!/bin/sh
 
 if [ "$#" -ne 3 ]; then
 	cat << EOF
@@ -27,9 +27,10 @@ DIRNAME=$(dirname "$VIDEO")
 EXT="${BASENAME##*.}"
 NAME="${BASENAME%.*}"
 
-outfile="$DIRNAME/${NAME}.trim.$EXT"
-
-ffmpeg -v error -y -i "$VIDEO" \
-	-ss "$START" \
+# For speed, we copy without re-encoding (with -ss before -i), but 
+# that means that the output isn’t going to be exact
+ffmpeg -v error -y \
+  -ss "$START" \
 	-to "$END" \
-	-c copy "$outfile"
+  -i "$VIDEO" \
+	-c copy "$DIRNAME/${NAME}.trim.$EXT"

package/src/vtrim.test.js

@@ -0,0 +1,22 @@
+import { ok } from 'node:assert/strict'
+import { join } from 'node:path'
+import { test } from 'node:test'
+import { cpSync } from 'node:fs'
+
+import { videoAttrs } from './utils/ffmpeg.js'
+import { mkTempDir, cli } from './utils/test-utils.js'
+
+const rel = f => join(import.meta.dirname, f)
+
+
+test('vtrim trims video from start to end time', async () => {
+	const tmp = mkTempDir('vtrim')
+
+	const inputFile = join(tmp, '60fps.mp4')
+	cpSync(rel('fixtures/60fps.mp4'), inputFile)
+	cli('vtrim', 5, 10, inputFile)
+
+	const { duration } = await videoAttrs(join(tmp, '60fps.trim.mp4'))
+	const EPSILON = 0.05
+	ok(Math.abs(parseFloat(duration) - 5) < EPSILON, `Duration should be 5s, got ${duration}s`)
+})

package/src/vsplit.sh

@@ -1,50 +0,0 @@
-#!/bin/zsh
-
-if [ "$#" -lt 2 ]; then
-	cat << EOF
-Usage:
-    $(basename $0) <split-time-1> [split-time-2 ...] <video-file>
-
-Examples:
-    $(basename $0) 123.45 video.mp4
-    $(basename $0) 60 120 180 video.mov
-    $(basename $0) 00:01:00 00:02:00 00:03:00 video.mkv
-EOF
-	exit 1
-fi
-
-VIDEO="${@[-1]}"
-SPLITS=("${@[1,-2]}")
-
-if [ ! -f "$VIDEO" ]; then
-	echo "Error: file not found: $VIDEO"
-	exit 1
-fi
-
-DIRNAME=$(dirname "$VIDEO")
-BASENAME=$(basename "$VIDEO")
-EXT="${BASENAME##*.}"
-NAME="${BASENAME%.*}"
-
-N_CLIPS=$((${#SPLITS[@]} + 1))
-
-for (( i=1; i<=$N_CLIPS; i++ )); do
-	outfile="$DIRNAME/${NAME}_${i}.$EXT"
-
-	if [ $i -eq 1 ]; then # First clip: [start, first_split]
-		ffmpeg -v error -y -i "$VIDEO" \
-			-t "${SPLITS[1]}" \
-			-c copy "$outfile"
-
-	elif [ $i -eq $N_CLIPS ]; then # Last clip: [last_split, end]
-		ffmpeg -v error -y -i "$VIDEO" \
-			-ss "${SPLITS[-1]}" \
-			-c copy "$outfile"
-
-	else # Middle clip: [split[i-1], split[i]]
-		ffmpeg -v error -y -i "$VIDEO" \
-			-ss "${SPLITS[$((i-1))]}" \
-			-to "${SPLITS[$i]}" \
-			-c copy "$outfile"
-	fi
-done

package/tests/avif.test.js

@@ -1,15 +0,0 @@
-import { join } from 'node:path'
-import { test } from 'node:test'
-import { equal } from 'node:assert/strict'
-import { tmpdir } from 'node:os'
-import { execSync } from 'node:child_process'
-import { mkdtempSync } from 'node:fs'
-
-import { sha1 } from './utils.js'
-
-
-test('PNG to AVIF', () => {
-	const tmp = mkdtempSync(join(tmpdir(), 'avif-test-'))
-	execSync(`src/cli.js avif --output-dir ${tmp} tests/fixtures/lenna.png` )
-	equal(sha1(join(tmp, 'lenna.avif')), sha1('tests/fixtures/lenna.avif'))
-})

package/tests/qdir/qdir.test.js

@@ -1,26 +0,0 @@
-import test from 'node:test'
-import { join } from 'node:path'
-import { tmpdir } from 'node:os'
-import { equal, deepEqual } from 'node:assert/strict'
-import { mkdtempSync, cpSync, readdirSync } from 'node:fs'
-
-import { qdir } from '../../src/qdir.js'
-
-
-test('jobs get renamed and failed have their exit status code', async () => {
-	const fixtures = join(import.meta.dirname, 'jobs')
-	const tmp = join(mkdtempSync(join(tmpdir(), 'qdir-')))
-
-	cpSync(fixtures, tmp, { recursive: true })
-
-	const err = await qdir(tmp, 0.2)
-	equal(err, null)
-
-	const files = readdirSync(tmp).sort()
-	deepEqual(files, [
-		'job1_good.sh.done',
-		'job2_bad.sh.failed.1',
-		'job3_good.sh.done',
-		'job4_bad.sh.failed.1'
-	])
-})

package/tests/utils.js

@@ -1,6 +0,0 @@
-import { createHash } from 'node:crypto'
-import { readFileSync } from 'node:fs'
-
-export function sha1(filePath) {
-	return createHash('sha1').update(readFileSync(filePath)).digest('hex')
-}

package/tests/vsplit.test.js

@@ -1,22 +0,0 @@
-import { join } from 'node:path'
-import { test } from 'node:test'
-import { equal } from 'node:assert/strict'
-import { tmpdir } from 'node:os'
-import { execSync } from 'node:child_process'
-import { mkdtempSync, cpSync } from 'node:fs'
-import { sha1 } from './utils.js'
-
-test('vsplit splits video at multiple time points', () => {
-	const tmp = mkdtempSync(join(tmpdir(), 'vsplit-test-'))
-
-	const inputFile = join(tmp, '60fps.mp4')
-	cpSync('tests/fixtures/60fps.mp4', inputFile)
-
-	execSync(`src/cli.js vsplit 5 10 15 20 25 ${inputFile}`)
-
-	for (let i = 1; i <= 6; i++) {
-		const generatedFile = join(tmp, `60fps_${i}.mp4`)
-		const expectedFile = `tests/fixtures/60fps_${i}.mp4`
-		equal(sha1(generatedFile), sha1(expectedFile), `60fps_${i}.mp4 hash should match expected`)
-	}
-})

package/tests/vtrim.test.js

@@ -1,20 +0,0 @@
-import { join } from 'node:path'
-import { test } from 'node:test'
-import { equal } from 'node:assert/strict'
-import { tmpdir } from 'node:os'
-import { execSync } from 'node:child_process'
-import { mkdtempSync, cpSync } from 'node:fs'
-import { sha1 } from './utils.js'
-
-test('vtrim trims video from start to end time', () => {
-	const tmp = mkdtempSync(join(tmpdir(), 'vtrim-test-'))
-
-	const inputFile = join(tmp, '60fps.mp4')
-	cpSync('tests/fixtures/60fps.mp4', inputFile)
-
-	execSync(`src/cli.js vtrim 5 10 ${inputFile}`)
-
-	const out = join(tmp, '60fps.trim.mp4')
-	const expected = 'tests/fixtures/60fps_2.mp4'
-	equal(sha1(out), sha1(expected), 'Trimmed video (5-10s) should match 60fps_2.mp4')
-})