allowlist replace_in_metadata in yt_dlp_opts#20
Conversation
|
@bradleesand can you change this to merge into arr_integration branch? I'm about to release v1.0.0 from that branch and I do think this PR can go there cleanly! thanks |
replace_in_metadata is a yt-dlp post-processor option that rewrites metadata fields using regex substitution. It's useful for stripping noise from titles (e.g. remaster suffixes, channel name suffixes) without affecting the download itself, making it safe to allowlist.
The music-search-benchmark workflow requires CHANGELOG.md to be updated for any change that touches engine/job_queue.py.
783c17e to
64ca860
Compare
|
Sorry I missed the v1.0.0 release, the past two weeks have been extra busy for me. Congrats on shipping 1.0.0! Rebased onto current |
|
thank you! |
What
Adds
replace_in_metadatato_YTDLP_DOWNLOAD_ALLOWLISTso it can be passed viayt_dlp_optsin config.Why
replace_in_metadatais a yt-dlp post-processor option that rewrites metadata fields using regex substitution (e.g. stripping remaster suffixes or channel name suffixes from titles). It doesn't affect the download itself — it only transforms metadata — so it's safe to allowlist alongside options likeratelimitandretries.Without this, users who set
replace_in_metadatainyt_dlp_optshave it silently dropped.Test
Added a test to
tests/test_ytdlp_download_opts.pyverifying the option passes throughbuild_ytdlp_optsfor download operations.