[PR #1722] [MERGED] Rename media plugin to ytdlp #4483

New issue

Closed

opened 2026-03-15 01:47:09 +03:00 by kerem · 0 comments

kerem commented

2026-03-15 01:47:09 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/ArchiveBox/ArchiveBox/pull/1722
Author: @pirate
Created: 12/29/2025
Status: ✅ Merged
Merged: 12/29/2025
Merged by: @pirate

Base: dev ← Head: claude/rename-media-to-ytdlp-TpcSW

📝 Commits (2)

a5654e8 rename media plugin to ytdlp with backwards-compatible aliases
ac64c77 move default yt-dlp args to config.json YTDLP_ARGS for user override

📊 Changes

20 files changed (+206 additions, -173 deletions)

View changed files

📝 README.md (+2 -2)
📝 TODO_hook_concurrency.md (+3 -3)
📝 archivebox/cli/tests_piping.py (+1 -1)
📝 archivebox/plugins/gallerydl/on_Snapshot__64_gallerydl.bg.py (+1 -17)
➖ archivebox/plugins/media/config.json (+0 -58)
📝 archivebox/plugins/ytdlp/binaries.jsonl (+0 -0)
➕ archivebox/plugins/ytdlp/config.json (+71 -0)
📝 archivebox/plugins/ytdlp/on_Snapshot__63_ytdlp.bg.py (+63 -50)
📝 archivebox/plugins/ytdlp/templates/embed.html (+2 -2)
📝 archivebox/plugins/ytdlp/templates/fullscreen.html (+2 -2)
📝 archivebox/plugins/ytdlp/templates/icon.html (+0 -0)
📝 archivebox/plugins/ytdlp/templates/thumbnail.html (+3 -3)
📝 archivebox/plugins/ytdlp/tests/test_ytdlp.py (+47 -24)
📝 archivebox/tests/test_migrations_08_to_09.py (+1 -1)
📝 archivebox/tests/test_migrations_helpers.py (+1 -1)
📝 docker-compose.yml (+1 -1)
📝 etc/ArchiveBox.conf.default (+2 -2)
📝 tests/fixtures.py (+1 -1)
📝 tests/test_cli_config.py (+3 -3)
📝 tests/test_recursive_crawl.py (+2 -2)

📄 Description

Rename archivebox/plugins/media/ → archivebox/plugins/ytdlp/
Rename hook script on_Snapshot__63_media.bg.py → on_Snapshot__63_ytdlp.bg.py
Update config.json: YTDLP_* as primary keys, MEDIA_* as x-aliases
Update templates CSS classes: media-* → ytdlp-*
Fix gallerydl bug: remove incorrect dependency on media plugin output
Update all codebase references to use YTDLP_* and SAVE_YTDLP
Add backwards compatibility test for MEDIA_ENABLED alias

Summary

Changes these areas

Bugfixes
Feature behavior
Command line interface
Configuration options
Internal architecture
Snapshot data layout on disk

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/ArchiveBox/ArchiveBox/pull/1722 **Author:** [@pirate](https://github.com/pirate) **Created:** 12/29/2025 **Status:** ✅ Merged **Merged:** 12/29/2025 **Merged by:** [@pirate](https://github.com/pirate) **Base:** `dev` ← **Head:** `claude/rename-media-to-ytdlp-TpcSW` --- ### 📝 Commits (2) - [`a5654e8`](https://github.com/ArchiveBox/ArchiveBox/commit/a5654e877f64e7c97d7d334ecec8b4a05f2e66cc) rename media plugin to ytdlp with backwards-compatible aliases - [`ac64c77`](https://github.com/ArchiveBox/ArchiveBox/commit/ac64c77341dd130ec4bcdefd9ae4b5f2dcd32199) move default yt-dlp args to config.json YTDLP_ARGS for user override ### 📊 Changes **20 files changed** (+206 additions, -173 deletions) <details> <summary>View changed files</summary> 📝 `README.md` (+2 -2) 📝 `TODO_hook_concurrency.md` (+3 -3) 📝 `archivebox/cli/tests_piping.py` (+1 -1) 📝 `archivebox/plugins/gallerydl/on_Snapshot__64_gallerydl.bg.py` (+1 -17) ➖ `archivebox/plugins/media/config.json` (+0 -58) 📝 `archivebox/plugins/ytdlp/binaries.jsonl` (+0 -0) ➕ `archivebox/plugins/ytdlp/config.json` (+71 -0) 📝 `archivebox/plugins/ytdlp/on_Snapshot__63_ytdlp.bg.py` (+63 -50) 📝 `archivebox/plugins/ytdlp/templates/embed.html` (+2 -2) 📝 `archivebox/plugins/ytdlp/templates/fullscreen.html` (+2 -2) 📝 `archivebox/plugins/ytdlp/templates/icon.html` (+0 -0) 📝 `archivebox/plugins/ytdlp/templates/thumbnail.html` (+3 -3) 📝 `archivebox/plugins/ytdlp/tests/test_ytdlp.py` (+47 -24) 📝 `archivebox/tests/test_migrations_08_to_09.py` (+1 -1) 📝 `archivebox/tests/test_migrations_helpers.py` (+1 -1) 📝 `docker-compose.yml` (+1 -1) 📝 `etc/ArchiveBox.conf.default` (+2 -2) 📝 `tests/fixtures.py` (+1 -1) 📝 `tests/test_cli_config.py` (+3 -3) 📝 `tests/test_recursive_crawl.py` (+2 -2) </details> ### 📄 Description - Rename archivebox/plugins/media/ → archivebox/plugins/ytdlp/ - Rename hook script on_Snapshot__63_media.bg.py → on_Snapshot__63_ytdlp.bg.py - Update config.json: YTDLP_* as primary keys, MEDIA_* as x-aliases - Update templates CSS classes: media-* → ytdlp-* - Fix gallerydl bug: remove incorrect dependency on media plugin output - Update all codebase references to use YTDLP_* and SAVE_YTDLP - Add backwards compatibility test for MEDIA_ENABLED alias  # Summary  # Related issues  # Changes these areas - [ ] Bugfixes - [ ] Feature behavior - [ ] Command line interface - [ ] Configuration options - [ ] Internal architecture - [ ] Snapshot data layout on disk --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

kerem

2026-03-15 01:47:09 +03:00

closed this issue
added the
pull-request
label

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

starred/ArchiveBox#4483

No description provided.

Rows
Columns