evo-cli 0.20.0__tar.gz → 0.20.2__tar.gz

{evo_cli-0.20.0 → evo_cli-0.20.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: evo_cli
-Version: 0.20.0
+Version: 0.20.2
 Summary: Evolution CLI - a developer toolbox for setting up dev machines
 Author: maycuatroi
 Project-URL: Homepage, https://github.com/maycuatroi/evo-cli

evo_cli-0.20.2/evo_cli/VERSION

@@ -0,0 +1 @@
+0.20.2

{evo_cli-0.20.0 → evo_cli-0.20.2}/evo_cli/commands/download.py

@@ -28,6 +28,12 @@ CONTAINERS = ["mp4", "mkv", "webm", "auto"]
 BROWSERS = ["chrome", "edge", "firefox", "brave", "chromium", "opera", "vivaldi", "safari"]
 JS_RUNTIMES = ["deno", "bun", "node"]
 YTDLP_STALE_DAYS = 60
+# Facebook reels and Instagram posts hand back the whole caption as the title,
+# which walks straight past the 255-byte limit filesystems put on a single path
+# component. `.<N>B` truncates the field to N bytes (before yt-dlp swaps `:` and
+# friends for their fullwidth twins, so leave room for that) and the suffixes
+# yt-dlp tacks on while downloading - `.f<format_id>`, `.mp4`, `.part`.
+TITLE_BYTE_LIMIT = 120
 
 DIRECT_EXTS = {
     ".7z",
@@ -256,12 +262,13 @@ def format_selector(quality, container, has_ffmpeg):
 
 
 def output_template(outdir, playlist, name):
+    title = f"%(title).{TITLE_BYTE_LIMIT}B"
     if name:
         template = name
     elif playlist:
-        template = "%(playlist_title)s/%(playlist_index)03d - %(title)s.%(ext)s"
+        template = f"%(playlist_title).{TITLE_BYTE_LIMIT}B/%(playlist_index)03d - {title}.%(ext)s"
     else:
-        template = "%(title)s.%(ext)s"
+        template = f"{title}.%(ext)s"
     return str(Path(outdir) / template)
 
 

{evo_cli-0.20.0 → evo_cli-0.20.2}/evo_cli.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: evo_cli
-Version: 0.20.0
+Version: 0.20.2
 Summary: Evolution CLI - a developer toolbox for setting up dev machines
 Author: maycuatroi
 Project-URL: Homepage, https://github.com/maycuatroi/evo-cli

{evo_cli-0.20.0 → evo_cli-0.20.2}/tests/test_download.py

@@ -92,6 +92,26 @@ def test_output_template_respects_name(tmp_path):
     assert dl.output_template(tmp_path, False, "%(id)s.%(ext)s").endswith("%(id)s.%(ext)s")
 
 
+@pytest.mark.parametrize("playlist", [False, True])
+def test_output_template_caps_the_title(tmp_path, playlist):
+    # A Facebook reel titles itself with its whole caption; uncapped, the file
+    # name overruns the 255-byte limit and yt-dlp dies with "File name too long".
+    template = dl.output_template(tmp_path, playlist, name=None)
+    assert f"%(title).{dl.TITLE_BYTE_LIMIT}B" in template
+    assert "%(title)s" not in template
+
+
+def test_output_template_caps_the_playlist_folder(tmp_path):
+    template = dl.output_template(tmp_path, playlist=True, name=None)
+    assert f"%(playlist_title).{dl.TITLE_BYTE_LIMIT}B" in template
+
+
+def test_title_cap_leaves_room_for_the_suffixes_ytdlp_appends():
+    # yt-dlp downloads to `<name>.f<format_id>.<ext>.part` and the sanitizer
+    # swaps `:` `|` `?` for 3-byte fullwidth twins before any of that.
+    assert dl.TITLE_BYTE_LIMIT + len(".f1049104804294604v.mp4.part") < 255
+
+
 def test_build_args_default(opts):
     args = dl.build_ytdlp_args(opts, has_ffmpeg=True)
     assert "--no-playlist" in args

evo_cli-0.20.0/evo_cli/VERSION

@@ -1 +0,0 @@
-0.20.0