Felix Förtsch
d12dd80209
All checks were successful
Build and Push Docker Image / build (push) Successful in 1m12s
Bug: every approve path (buildCommand used by review approve/approve-all/ series approve-all/season approve-all/retry/detail preview) was building an ffmpeg command that -map'd only the 'keep' streams and dropped all subtitles. For a file like Wuthering Heights with 37 embedded subs, the run would delete every sub into the void — user expected extraction to sidecar files per the pipeline contract. buildPipelineCommand already did the right thing (extract every subtitle with -map 0:s:N -c:s copy 'basename.lang.srt', then remux kept streams) but it was only reached by tests. buildCommand now delegates to it — one call site, subtitle extraction always runs, predictExtractedFiles records the sidecar paths after job success (same logic, same basePath). Added a regression test: buildCommand on a 2-subtitle file contains -map 0:s:0, -map 0:s:1 and the expected 'basename.en.srt'/'.de.srt' paths.