From 3c51f27e975563ea946e809815766b192d1eed22 Mon Sep 17 00:00:00 2001
From: Broque Thomas <26755000+Nezreka@users.noreply.github.com>
Date: Fri, 20 Mar 2026 12:45:36 -0700
Subject: [PATCH] Fix Spotify enrichment worker rejecting every track via
 fallback

Worker checked self.client.sp (non-None even without Spotify auth due
to fallback) instead of is_spotify_authenticated(). Searched via
iTunes/Deezer fallback, got numeric IDs, rejected them all with
warnings. Now sleeps when Spotify isn't authenticated instead of
making pointless fallback searches.
---
 core/spotify_worker.py | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/core/spotify_worker.py b/core/spotify_worker.py
index 739154b1..254f43da 100644
--- a/core/spotify_worker.py
+++ b/core/spotify_worker.py
@@ -152,10 +152,10 @@ class SpotifyWorker:
                 # We intentionally avoid calling is_spotify_authenticated() here
                 # because it makes an API probe that can re-trigger rate limits
                 # and lock users in an infinite rate-limit loop.
-                if not self.client.sp:
+                if not self.client.is_spotify_authenticated():
                     self.client.reload_config()
-                    if not self.client.sp:
-                        logger.debug("Spotify not configured, sleeping 30s...")
+                    if not self.client.is_spotify_authenticated():
+                        logger.debug("Spotify not authenticated, sleeping 30s...")
                         time.sleep(30)
                         continue