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

#863: add diagnostic logging to YouTube artist-recovery so we can see per-track what it returns

Browse source
This commit is contained in:
BoulderBadgeDad 2026-06-13 00:03:35 -07:00
parent c72e83bc2f
commit 0577dc92e5
1 changed files with 17 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

27
core/discovery/youtube.py
View file

@ -104,16 +104,23 @@ def run_youtube_discovery_worker(url_hash, deps: YoutubeDiscoveryDeps):
# block for minutes on a big playlist). Per-track cost is hidden
# behind the discovery progress bar; the recovered artist makes the
# match below actually find the song.
if (cleaned_artist == 'Unknown Artist' and deps.recover_youtube_artist
and track.get('id')):
try:
_rec = deps.recover_youtube_artist(track['id'])
except Exception as _rec_err:
logger.debug(f"Artist recovery failed for {track.get('id')}: {_rec_err}")
_rec = ''
if _rec and _rec != 'Unknown Artist':
cleaned_artist = _rec
track['artists'] = [_rec] # persist so retries/UI see it
if cleaned_artist == 'Unknown Artist' and track.get('id'):
if not deps.recover_youtube_artist:
logger.warning("[YT Discovery] artist recovery unavailable (dep not wired) "
"— '%s' stays Unknown", cleaned_title)
else:
try:
_rec = deps.recover_youtube_artist(track['id'])
except Exception as _rec_err:
logger.warning(f"[YT Discovery] artist recovery raised for {track.get('id')}: {_rec_err}")
_rec = ''
if _rec and _rec != 'Unknown Artist':
logger.info(f"[YT Discovery] recovered artist '{_rec}' for '{cleaned_title}' ({track['id']})")
cleaned_artist = _rec
track['artists'] = [_rec] # persist so retries/UI see it
else:
logger.info(f"[YT Discovery] artist recovery returned nothing for "
f"'{cleaned_title}' ({track['id']}) — leaving Unknown")
logger.info(f"Searching {discovery_source} for: '{cleaned_artist}' - '{cleaned_title}'")