Skip to content

fix: report FFmpeg input worker startup failure#43

Merged
abrichr merged 1 commit into
mainfrom
agent/fix-ffmpeg-worker-start
Jul 24, 2026
Merged

fix: report FFmpeg input worker startup failure#43
abrichr merged 1 commit into
mainfrom
agent/fix-ffmpeg-worker-start

Conversation

@abrichr

@abrichr abrichr commented Jul 24, 2026

Copy link
Copy Markdown
Member

What changed

  • translate input-worker thread startup failures into the public FFmpegEncodingError contract
  • kill and reap the already-started encoder before returning the failure
  • add a regression test for process/thread cleanup

Why

This closes the last untested startup lifecycle edge in the new direct FFmpeg stream and provides the conventional patch signal needed to publish the already-merged encoder as v1.0.4.

Validation

  • focused video suite: 29 passed
  • Ruff and diff checks pass

@abrichr
abrichr marked this pull request as ready for review July 24, 2026 01:14
@abrichr
abrichr merged commit c11969b into main Jul 24, 2026
12 checks passed
@abrichr
abrichr deleted the agent/fix-ffmpeg-worker-start branch July 24, 2026 01:14
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