From 41f4eeb91efe5f27d4075f4f68583efe64a78771 Mon Sep 17 00:00:00 2001 From: BoulderBadgeDad Date: Mon, 8 Jun 2026 08:52:49 -0700 Subject: [PATCH] Watchlist: log when the release-type filter skips an album/EP/single MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Sokhi reported "14 singles, 0 albums" from a scan and setting lookback to "All" didn't help — because it was never lookback. The cause is the per-artist release-type filter: with "Albums" toggled off, _should_include_release drops every 7+-track release (the full discography is still fetched, then filtered). That skip was completely silent, so there was no way to see it in the log. Now logs each skipped release with its type + the artist's albums/eps/singles toggles, so "why didn't my albums get added" is answerable from app.log. No behaviour change — diagnostics only. --- core/watchlist_scanner.py | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/core/watchlist_scanner.py b/core/watchlist_scanner.py index 0ce55e89..e2a345aa 100644 --- a/core/watchlist_scanner.py +++ b/core/watchlist_scanner.py @@ -1313,6 +1313,19 @@ class WatchlistScanner: logger.info("Skipping album with placeholder tracks: %s", album_name) continue if not self._should_include_release(len(tracks), artist): + # Make the type-filter skip visible — otherwise a user + # with "Albums" toggled off just sees missing tracks + # with no explanation (Sokhi #815-adjacent: 14 singles, + # 0 albums because include_albums was off). + _n = len(tracks) + _kind = 'album' if _n >= 7 else ('EP' if _n >= 4 else 'single') + logger.info( + "Skipping %s '%s' (%d tracks) — release type filter " + "(albums=%s, eps=%s, singles=%s) excludes it", + _kind, album_name, _n, + getattr(artist, 'include_albums', True), + getattr(artist, 'include_eps', True), + getattr(artist, 'include_singles', True)) continue album_image_url = ''