Skip to content

Commit 7ca30c3

Browse files
JessyTsuiclaude
andauthored
Fix unified pipeline: use YtDlpVideoDownloader for YouTube videos (#97)
Unified pipeline was using HttpVideoDownloader which cannot download YouTube videos (requires yt-dlp with cookies/auth). Switch to YtDlpVideoDownloader which handles cookies, proxy, and signatures. Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent e70a83b commit 7ca30c3

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

workers/unified/pipeline.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,8 @@
2626
HeuristicFrameAnalyzer,
2727
HeuristicSceneDetector,
2828
HttpVideoDownloader,
29+
KnowledgeVideoDownloader,
30+
YtDlpVideoDownloader,
2931
extract_keywords,
3032
summarize_text,
3133
)
@@ -74,7 +76,7 @@ def __init__(
7476
pixabay_client: PixabayClient | None = None,
7577
frame_analyzer: HeuristicFrameAnalyzer | None = None,
7678
scene_detector: HeuristicSceneDetector | None = None,
77-
video_downloader: HttpVideoDownloader | None = None,
79+
video_downloader: KnowledgeVideoDownloader | None = None,
7880
summary_generator: GeminiFlashSummaryGenerator | None = None,
7981
frame_uploader: R2FrameUploader | None = None,
8082
temp_dir_root: str | None = None,
@@ -89,7 +91,7 @@ def __init__(
8991
self._pixabay_client = pixabay_client or PixabayClient()
9092
self._frame_analyzer = frame_analyzer or HeuristicFrameAnalyzer()
9193
self._scene_detector = scene_detector or HeuristicSceneDetector()
92-
self._video_downloader = video_downloader or HttpVideoDownloader()
94+
self._video_downloader = video_downloader or YtDlpVideoDownloader()
9395
self._summary_generator = summary_generator or GeminiFlashSummaryGenerator()
9496
self._frame_uploader = frame_uploader or R2FrameUploader()
9597
self._temp_dir_root = temp_dir_root

0 commit comments

Comments
 (0)