From 6ac44748e1efbcf75a456560ab34e9a808004f82 Mon Sep 17 00:00:00 2001
From: Ralph Torres <mail@ralphptorr.es>
Date: Sat, 11 Apr 2026 13:37:14 +0000
Subject: [PATCH] prioritize synced lyrics in fetching

get returns only synced lyrics, falls back to search if none found.
search iterates to return first result with synced lyrics.
plain lyrics only returned as last resort

closes #76
---
 services/lrclib/lrclib.go | 18 ++++++++++++++++--
 1 file changed, 16 insertions(+), 2 deletions(-)

diff --git a/services/lrclib/lrclib.go b/services/lrclib/lrclib.go
index 6a159ef..e6bebe7 100644
--- a/services/lrclib/lrclib.go
+++ b/services/lrclib/lrclib.go
@@ -24,7 +24,10 @@ type Client struct {
 // Client implements lyrics.Provider
 func (c *Client) Lyrics(artist, track string) ([]lyrics.Line, error) {
 	if artist != "" && track != "" {
-		return c.get(artist, track)
+		lines, err := c.get(artist, track)
+		if lines != nil || err != nil {
+			return lines, err
+		}
 	}
 
 	return c.search(artist + track)
@@ -60,7 +63,11 @@ func (c *Client) get(artist, track string) ([]lyrics.Line, error) {
 		return nil, err
 	}
 
-	return parseTrack(response), nil
+	if response.SyncedLyrics != "" {
+		return parseSynced(response), nil
+	}
+
+	return nil, nil
 }
 
 func (c *Client) search(query string) ([]lyrics.Line, error) {
@@ -91,6 +98,13 @@ func (c *Client) search(query string) ([]lyrics.Line, error) {
 	if len(response) == 0 {
 		return nil, nil
 	}
+
+	for _, track := range response {
+		if track.SyncedLyrics != "" {
+			return parseSynced(track), nil
+		}
+	}
+
 	return parseTrack(response[0]), nil
 }