Limit playlist to 100 items for performance

tsubery · Oct 7, 2024 · 0a55c6a · 0a55c6a
1 parent 1e750ea
commit 0a55c6a
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/app/services/youtube/cli.rb b/app/services/youtube/cli.rb
@@ -2,6 +2,8 @@ module Youtube
   class CLI
     class << self
       DL_BINARY = "./bin/yt-dlp".freeze
+      PLAYLIST_LENGTH=100
+
       def sanitize_id!(id)
         unless id =~ /\A[a-z0-9\-_]+\z/i
           raise(ArgumentError, "The id #{id.inspect} can be dangerous!")
@@ -34,9 +36,9 @@ def get_video_information(url)
         end
       end
 
-      def get_playlist_information(id)
+      def get_playlist_information(id, length: PLAYLIST_LENGTH)
         sanitize_id!(id)
-        stdout, stderr, status = cmd("-j --flat-playlist \"#{id}\"")
+        stdout, stderr, status = cmd("-j -I1:#{length} --flat-playlist \"#{id}\"")
         if status.exitstatus.zero?
           stdout
         else

diff --git a/test/fixtures/youtube_cli/get_playlist_information-291387ba69c6f12c2d43aed1aa63d5d6.json b/test/fixtures/youtube_cli/get_playlist_information-291387ba69c6f12c2d43aed1aa63d5d6.json