From b6fb314571d01dfcc78c0b0f5733b3ffd01cafad Mon Sep 17 00:00:00 2001 From: DARKZOUL5 Date: Mon, 19 Oct 2026 22:17:55 +0300 Subject: [PATCH] fix: resolve core utility lint errors --- src/app/core/utils/deps.py | 4 ++-- src/app/core/utils/version.py | 6 ++++-- src/app/core/utils/yt.py | 2 +- 3 files changed, 7 insertions(+), 5 deletions(-) diff --git a/src/app/core/utils/deps.py b/src/app/core/utils/deps.py index 8b1c091..8cb4fa8 100644 --- a/src/app/core/utils/deps.py +++ b/src/app/core/utils/deps.py @@ -28,7 +28,7 @@ def get_yt_dlp_version() -> str | None: # yt-dlp exposes version in different places depending on install return getattr(yt_dlp, "__version__", None) or getattr(getattr(yt_dlp, "version", None), "__version__", None) - except Exception: + except (ImportError, AttributeError, TypeError): return None @@ -78,7 +78,7 @@ def ensure_ffmpeg_available(ffmpeg_hint: str | None) -> tuple[str, str]: """ ffmpeg_exe = "ffmpeg.exe" if sys.platform.startswith("win") else "ffmpeg" - ffmpeg_path, ffmpeg_dir = _resolve_tool_paths(ffmpeg_hint, ffmpeg_exe) + ffmpeg_path, _ = _resolve_tool_paths(ffmpeg_hint, ffmpeg_exe) if not ffmpeg_path: raise DependencyError("ffmpeg not found. Install ffmpeg or set 'ffmpeg_path' in config.") diff --git a/src/app/core/utils/version.py b/src/app/core/utils/version.py index c9f76bf..f1eb47c 100644 --- a/src/app/core/utils/version.py +++ b/src/app/core/utils/version.py @@ -17,9 +17,11 @@ def _read_text(path: Path) -> str | None: if path.exists(): text = path.read_text(encoding="utf-8").strip() return text or None - except Exception: - pass + except (OSError, UnicodeError): + return None return None + + def get_app_version() -> str: """ Returns the packaged app version. diff --git a/src/app/core/utils/yt.py b/src/app/core/utils/yt.py index 1dc9053..6cf1cc1 100644 --- a/src/app/core/utils/yt.py +++ b/src/app/core/utils/yt.py @@ -10,5 +10,5 @@ def extract_playlist_id(url: str) -> str | None: if "list" in qs and qs.get("list"): return qs.get("list", [None])[0] return None - except Exception: + except (TypeError, ValueError): return None