Skip to content

feat: Integrate yt-dlp as a new downloader - #30

Merged
Tanq16 merged 1 commit into
refactor-v2from
feat/ytdlp-integration-14014072662368274453
May 17, 2026
Merged

feat: Integrate yt-dlp as a new downloader#30
Tanq16 merged 1 commit into
refactor-v2from
feat/ytdlp-integration-14014072662368274453

Conversation

@Tanq16

@Tanq16 Tanq16 commented May 17, 2026

Copy link
Copy Markdown
Owner

This pull request implements the yt-dlp integration plan.

  • New Job Implementation: Created internal/jobs/ytdlp/job.go implementing highway.Job with methods to parse yt-dlp standard output in JSON format natively tracking progress and merger events. State marshaling functions are also supported for pause/resume semantics.
  • CLI Command: Added cmd/ytdlp.go which registers the ytdlp Cobra command.
  • Registrations: Successfully registered the module with the root Cobra CLI (cmd/root.go) and the highway job unmarshaler (cmd/highway.go).

PR created automatically by Jules for task 14014072662368274453 started by @Tanq16

Implemented a new job package for yt-dlp that executes the binary using
its JSON progress template for seamless highway architecture integration.
Added `ytdlp` as a supported command and registered it in the CLI.

Co-authored-by: Tanq16 <37408906+Tanq16@users.noreply.github.com>
@google-labs-jules

Copy link
Copy Markdown
Contributor

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

New to Jules? Learn more at jules.google/docs.


For security, I will only act on instructions from the user who triggered this task.

@Tanq16
Tanq16 merged commit 81a7b33 into refactor-v2 May 17, 2026
3 checks passed
@Tanq16
Tanq16 deleted the feat/ytdlp-integration-14014072662368274453 branch May 17, 2026 15:40
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