danvics/soulsync
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

Remove redundant repetition of lookback period change during watchlist scan

Browse source 
Unnecessary noise on the logs
This commit is contained in:
Antti Kettunen 2026-04-11 14:01:49 +03:00
parent 1dcdccb282
commit 4946ff0d03
1 changed files with 10 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
core/watchlist_scanner.py
View file

@ -353,6 +353,7 @@ class WatchlistScanner:
self._database = None self._database = None
self._wishlist_service = None self._wishlist_service = None
self._matching_engine = None self._matching_engine = None
self._rescan_cutoff_log_marker = None
if metadata_service: if metadata_service:
self._metadata_service = metadata_service self._metadata_service = metadata_service
@ -932,7 +933,9 @@ class WatchlistScanner:
# Check if a lookback period change requires a one-time wider window # Check if a lookback period change requires a one-time wider window
rescan_cutoff = self._get_rescan_cutoff() rescan_cutoff = self._get_rescan_cutoff()
if rescan_cutoff == 'all': if rescan_cutoff == 'all':
logger.info(f"Lookback period changed to 'all' — returning full discography") if self._rescan_cutoff_log_marker != 'all':
logger.info(f"Lookback period changed to 'all' — returning full discography")
self._rescan_cutoff_log_marker = 'all'
cutoff_timestamp = None cutoff_timestamp = None
needs_full_discog = True needs_full_discog = True
elif rescan_cutoff is not None: elif rescan_cutoff is not None:
@ -942,7 +945,10 @@ class WatchlistScanner:
if rescan_cutoff.tzinfo is None: if rescan_cutoff.tzinfo is None:
rescan_cutoff = rescan_cutoff.replace(tzinfo=timezone.utc) rescan_cutoff = rescan_cutoff.replace(tzinfo=timezone.utc)
if rescan_cutoff < scan_ts: if rescan_cutoff < scan_ts:
logger.info(f"Lookback period change detected — expanding cutoff from {cutoff_timestamp} to {rescan_cutoff}") marker = rescan_cutoff.isoformat()
if self._rescan_cutoff_log_marker != marker:
logger.info(f"Lookback period change detected — expanding cutoff from {cutoff_timestamp} to {rescan_cutoff}")
self._rescan_cutoff_log_marker = marker
cutoff_timestamp = rescan_cutoff cutoff_timestamp = rescan_cutoff
else: else:
# No scan timestamp — first scan, use lookback period # No scan timestamp — first scan, use lookback period
@ -1248,6 +1254,7 @@ class WatchlistScanner:
cursor.execute("DELETE FROM metadata WHERE key = 'watchlist_rescan_cutoff'") cursor.execute("DELETE FROM metadata WHERE key = 'watchlist_rescan_cutoff'")
conn.commit() conn.commit()
logger.info("Cleared watchlist rescan cutoff flag") logger.info("Cleared watchlist rescan cutoff flag")
self._rescan_cutoff_log_marker = None
except Exception as e: except Exception as e:
logger.debug(f"Error clearing rescan cutoff: {e}") logger.debug(f"Error clearing rescan cutoff: {e}")
@ -3266,4 +3273,4 @@ def get_watchlist_scanner(spotify_client: SpotifyClient) -> WatchlistScanner:
global _watchlist_scanner_instance global _watchlist_scanner_instance
if _watchlist_scanner_instance is None: if _watchlist_scanner_instance is None:
_watchlist_scanner_instance = WatchlistScanner(spotify_client) _watchlist_scanner_instance = WatchlistScanner(spotify_client)
return _watchlist_scanner_instance return _watchlist_scanner_instance