Skip to content

Conversation

@azman0101
Copy link
Owner

@azman0101 azman0101 commented Aug 24, 2025

This pull request refactors the YouTube transcript and comments retrieval logic to use the yt-dlp Python module directly, instead of invoking the command-line tool via subprocess. This change simplifies the code, improves error handling, and makes the application easier to test and maintain.

Migration to direct yt-dlp Python API:

  • Replaced all uses of subprocess calls to the yt-dlp command-line tool with direct calls to the yt-dlp Python module (YoutubeDL), removing the run_yt_dlp helper and related subprocess logic. [1] [2] [3]
  • Updated the fetch_comments and fetch_transcript functions to use YoutubeDL.extract_info with appropriate options for extracting comments and subtitles, improving code clarity and error handling. [1] [2]

Documentation and UI updates:

  • Updated documentation and user instructions to clarify that the Python yt-dlp module is used internally, not the standalone command-line tool. [1] [2]

Copilot AI review requested due to automatic review settings August 24, 2025 13:27
@semanticdiff-com
Copy link

semanticdiff-com bot commented Aug 24, 2025

Review changes with  SemanticDiff

Changed Files
File Status
  app.py  26% smaller
  README.md Unsupported file format
  requirements.txt Unsupported file format

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

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.

2 participants