Merge pull request #542 from Nezreka/fix/deezer-cover-art-low-resolution
Fix/deezer cover art low resolution
This commit is contained in:
BoulderBadgeDad
GitHub
commit
c038400d84
5 changed files with 322 additions and 2 deletions
|
|
@ -45,6 +45,50 @@ def rate_limited(func):
|
|||
return wrapper
|
||||
|
||||
|
||||
# Pattern matches Deezer's CDN cover/picture URL: a numeric width-x-height
|
||||
# segment in the path (e.g. ``/1000x1000-000000-80-0-0.jpg``). Captures
|
||||
# both halves so the replacement can use a single dimension and preserve
|
||||
# the rest of the path verbatim.
|
||||
_DEEZER_CDN_SIZE_PATTERN = re.compile(r'/(\d+)x(\d+)-')
|
||||
|
||||
# Maximum size Deezer's CDN serves before returning 403. Verified
|
||||
# empirically against multiple albums — 1900 works reliably, 2000+
|
||||
# returns Forbidden. CDN serves the source-native size when it's
|
||||
# smaller than requested, so asking for 1900 is safe even on albums
|
||||
# whose source upload was lower-res (no upscaling, just same bytes).
|
||||
_DEEZER_MAX_COVER_SIZE = 1900
|
||||
|
||||
|
||||
def _upgrade_deezer_cover_url(url: str, target_size: int = _DEEZER_MAX_COVER_SIZE) -> str:
|
||||
"""Rewrite a Deezer CDN cover/picture URL to request a larger size.
|
||||
|
||||
Deezer's API returns ``cover_xl`` / ``picture_xl`` URLs at
|
||||
1000×1000, but the underlying CDN serves up to 1900×1900 by
|
||||
rewriting the size segment in the URL path. This helper does the
|
||||
rewrite — same idea as ``_upgrade_spotify_image_url`` in
|
||||
``spotify_client`` and the ``mzstatic.com`` size-replacement in
|
||||
``download_cover_art``.
|
||||
|
||||
Defensive on every input shape:
|
||||
- Empty / None URL → returned as-is
|
||||
- Non-Deezer URL (no ``dzcdn`` host, no size segment) → returned as-is
|
||||
- Already at or above target size → returned as-is (no point rewriting)
|
||||
|
||||
The CDN returns the source-native image bytes when source < target,
|
||||
so asking for 1900 on an album whose source was uploaded at 600
|
||||
just returns the 600-pixel image — no upscaling, no failure.
|
||||
"""
|
||||
if not url or 'dzcdn' not in url:
|
||||
return url
|
||||
match = _DEEZER_CDN_SIZE_PATTERN.search(url)
|
||||
if not match:
|
||||
return url
|
||||
current = int(match.group(1))
|
||||
if current >= target_size:
|
||||
return url
|
||||
return _DEEZER_CDN_SIZE_PATTERN.sub(f'/{target_size}x{target_size}-', url, count=1)
|
||||
|
||||
|
||||
# ==================== Dataclasses (match iTunesClient / SpotifyClient format) ====================
|
||||
|
||||
@dataclass
|
||||
|
|
|
|||
|
|
@ -324,11 +324,50 @@ def download_cover_art(album_info: dict, target_dir: str, context: dict = None):
|
|||
logger.debug("upgrade spotify image url failed: %s", e)
|
||||
elif art_url and "mzstatic.com" in art_url:
|
||||
art_url = re.sub(r"\d+x\d+bb", "3000x3000bb", art_url)
|
||||
elif art_url and "dzcdn" in art_url:
|
||||
# Deezer's API returns cover_xl URLs at 1000×1000 but
|
||||
# the underlying CDN serves up to 1900×1900 by rewriting
|
||||
# the size segment in the URL path. Without this upgrade
|
||||
# users embedding cover art via Deezer get visibly
|
||||
# blurry covers in their library / phone player (Discord
|
||||
# report from Tim, 2026-05). Same shape as the iTunes
|
||||
# mzstatic upgrade above + Spotify scdn upgrade.
|
||||
try:
|
||||
from core.deezer_client import _upgrade_deezer_cover_url
|
||||
|
||||
art_url = _upgrade_deezer_cover_url(art_url)
|
||||
except Exception as e:
|
||||
logger.debug("upgrade deezer image url failed: %s", e)
|
||||
if not art_url:
|
||||
logger.warning("No cover art URL available for download.")
|
||||
return
|
||||
with urllib.request.urlopen(art_url, timeout=10) as response:
|
||||
image_data = response.read()
|
||||
# Fetch with one fallback level: if we upgraded a Deezer
|
||||
# URL above and the CDN happens to refuse the larger size
|
||||
# for this specific album, retry with the original URL so
|
||||
# we never regress vs. pre-upgrade behavior. Empirically
|
||||
# 1900 works for every album tested but defending against
|
||||
# the edge case keeps the fix strictly non-regressive.
|
||||
original_url = album_info.get("album_image_url")
|
||||
if context and not original_url:
|
||||
album_ctx = get_import_context_album(context)
|
||||
original_url = album_ctx.get("image_url") or original_url
|
||||
try:
|
||||
with urllib.request.urlopen(art_url, timeout=10) as response:
|
||||
image_data = response.read()
|
||||
except Exception as fetch_err:
|
||||
if (
|
||||
"dzcdn" in art_url
|
||||
and original_url
|
||||
and original_url != art_url
|
||||
):
|
||||
logger.info(
|
||||
"Deezer CDN refused upgraded cover URL (%s); "
|
||||
"retrying with original size", fetch_err,
|
||||
)
|
||||
with urllib.request.urlopen(original_url, timeout=10) as response:
|
||||
image_data = response.read()
|
||||
else:
|
||||
raise
|
||||
|
||||
if not image_data:
|
||||
return
|
||||
|
|
|
|||
|
|
@ -204,9 +204,25 @@ def download_cover_art(cover_url: str) -> Optional[Tuple[bytes, str]]:
|
|||
"""
|
||||
Download cover art once. Returns (image_data, mime_type) or None on failure.
|
||||
Call this once per album, then pass the result to write_tags_to_file for each track.
|
||||
|
||||
For Deezer CDN URLs, upgrades the size segment to 1900×1900 (CDN
|
||||
max). Mirrors the same upgrade in
|
||||
``core.metadata.artwork.download_cover_art`` so the
|
||||
enhanced-library-view "Write Tags to File" feature embeds the same
|
||||
high-resolution cover the auto post-process flow does. Falls back
|
||||
to the original URL if the CDN refuses the upgraded size for a
|
||||
specific album — keeps the fix strictly non-regressive vs. the
|
||||
pre-upgrade behaviour.
|
||||
"""
|
||||
if not cover_url:
|
||||
return None
|
||||
original_url = cover_url
|
||||
if 'dzcdn' in cover_url:
|
||||
try:
|
||||
from core.deezer_client import _upgrade_deezer_cover_url
|
||||
cover_url = _upgrade_deezer_cover_url(cover_url)
|
||||
except Exception as e:
|
||||
logger.debug("upgrade deezer image url failed: %s", e)
|
||||
try:
|
||||
with urllib.request.urlopen(cover_url, timeout=15) as response:
|
||||
image_data = response.read()
|
||||
|
|
@ -214,6 +230,21 @@ def download_cover_art(cover_url: str) -> Optional[Tuple[bytes, str]]:
|
|||
if image_data:
|
||||
return (image_data, mime_type)
|
||||
except Exception as e:
|
||||
# Deezer CDN refused upgraded size for this album — retry with
|
||||
# original URL so we never get less than pre-upgrade behaviour.
|
||||
if 'dzcdn' in cover_url and cover_url != original_url:
|
||||
logger.info(
|
||||
"Deezer CDN refused upgraded cover URL (%s); retrying with original size", e,
|
||||
)
|
||||
try:
|
||||
with urllib.request.urlopen(original_url, timeout=15) as response:
|
||||
image_data = response.read()
|
||||
mime_type = response.info().get_content_type() or 'image/jpeg'
|
||||
if image_data:
|
||||
return (image_data, mime_type)
|
||||
except Exception as fallback_err:
|
||||
logger.error(f"Error downloading cover art (fallback): {fallback_err}")
|
||||
return None
|
||||
logger.error(f"Error downloading cover art from {cover_url}: {e}")
|
||||
return None
|
||||
|
||||
|
|
|
|||
205
tests/metadata/test_deezer_cover_url_upgrade.py
Normal file
205
tests/metadata/test_deezer_cover_url_upgrade.py
Normal file
|
|
@ -0,0 +1,205 @@
|
|||
"""Pin the Deezer CDN cover-URL upgrade helper.
|
||||
|
||||
Discord report (Tim, 2026-05-XX): downloaded cover art via Deezer
|
||||
metadata source comes out blurry — visibly low-res in Navidrome.
|
||||
Cause: Deezer's API returns ``cover_xl`` URLs at 1000×1000 but the
|
||||
underlying CDN serves up to 1900×1900 by rewriting the size segment
|
||||
in the URL path. SoulSync wasn't doing the rewrite.
|
||||
|
||||
Helper: ``_upgrade_deezer_cover_url(url, target_size=1900)`` — pure
|
||||
function, lifts to one boundary so cover-download sites don't each
|
||||
re-implement the regex. Tests pin every input shape:
|
||||
|
||||
- Standard Deezer URL → upgraded to target
|
||||
- Non-Deezer URL → returned unchanged
|
||||
- Already at/above target → returned unchanged (no needless rewrite)
|
||||
- Empty / None → returned as-is
|
||||
- Custom target → applied correctly
|
||||
- Picture URLs (artist) — same path pattern, also upgraded
|
||||
"""
|
||||
|
||||
from __future__ import annotations
|
||||
|
||||
import pytest
|
||||
|
||||
from core.deezer_client import _upgrade_deezer_cover_url
|
||||
|
||||
|
||||
# ---------------------------------------------------------------------------
|
||||
# Standard upgrade — the headline case
|
||||
# ---------------------------------------------------------------------------
|
||||
|
||||
|
||||
class TestUpgradeStandardDeezerUrl:
|
||||
def test_default_target_1900(self):
|
||||
url = 'https://cdn-images.dzcdn.net/images/cover/abc123/1000x1000-000000-80-0-0.jpg'
|
||||
upgraded = _upgrade_deezer_cover_url(url)
|
||||
assert upgraded == 'https://cdn-images.dzcdn.net/images/cover/abc123/1900x1900-000000-80-0-0.jpg'
|
||||
|
||||
def test_alternate_dzcdn_host(self):
|
||||
"""Both `cdn-images.dzcdn.net` and `e-cdns-images.dzcdn.net`
|
||||
are valid Deezer CDN hosts. Helper must catch both."""
|
||||
url = 'https://e-cdns-images.dzcdn.net/images/cover/xyz/1000x1000-000000-80-0-0.jpg'
|
||||
upgraded = _upgrade_deezer_cover_url(url)
|
||||
assert '1900x1900' in upgraded
|
||||
assert upgraded.startswith('https://e-cdns-images.dzcdn.net/')
|
||||
|
||||
def test_artist_picture_url_also_upgrades(self):
|
||||
"""Artist `picture_xl` URLs follow the same `/SIZExSIZE-` path
|
||||
pattern and the same CDN. Same upgrade applies."""
|
||||
url = 'https://cdn-images.dzcdn.net/images/artist/hash/1000x1000-000000-80-0-0.jpg'
|
||||
upgraded = _upgrade_deezer_cover_url(url)
|
||||
assert '1900x1900' in upgraded
|
||||
|
||||
def test_500x500_upgrades(self):
|
||||
"""Some albums on Deezer only have cover_big (500×500). Helper
|
||||
upgrades anything below target, not just 1000×1000."""
|
||||
url = 'https://cdn-images.dzcdn.net/images/cover/abc/500x500-000000-80-0-0.jpg'
|
||||
upgraded = _upgrade_deezer_cover_url(url)
|
||||
assert '1900x1900' in upgraded
|
||||
|
||||
|
||||
# ---------------------------------------------------------------------------
|
||||
# Custom target size
|
||||
# ---------------------------------------------------------------------------
|
||||
|
||||
|
||||
class TestCustomTargetSize:
|
||||
def test_smaller_target(self):
|
||||
"""Caller can request a smaller size for bandwidth-sensitive
|
||||
cases (mobile, thumbnails, etc.)."""
|
||||
url = 'https://cdn-images.dzcdn.net/images/cover/abc/1000x1000-000000-80-0-0.jpg'
|
||||
upgraded = _upgrade_deezer_cover_url(url, target_size=600)
|
||||
# 1000 already > 600, so this is a no-op — never DOWNGRADE.
|
||||
assert upgraded == url
|
||||
|
||||
def test_larger_target_works(self):
|
||||
url = 'https://cdn-images.dzcdn.net/images/cover/abc/250x250-000000-80-0-0.jpg'
|
||||
upgraded = _upgrade_deezer_cover_url(url, target_size=1400)
|
||||
assert '1400x1400' in upgraded
|
||||
|
||||
|
||||
# ---------------------------------------------------------------------------
|
||||
# Already-upgraded URLs — no needless rewrite
|
||||
# ---------------------------------------------------------------------------
|
||||
|
||||
|
||||
class TestAlreadyUpgraded:
|
||||
def test_already_at_target_returned_unchanged(self):
|
||||
"""Re-running the upgrade on an already-upgraded URL should
|
||||
be a no-op. Idempotent — important for cached URLs that may
|
||||
have been rewritten by a previous SoulSync version."""
|
||||
url = 'https://cdn-images.dzcdn.net/images/cover/abc/1900x1900-000000-80-0-0.jpg'
|
||||
assert _upgrade_deezer_cover_url(url) == url
|
||||
|
||||
def test_above_target_returned_unchanged(self):
|
||||
"""Defensive: if the URL is somehow LARGER than target, don't
|
||||
downgrade. Cached URL from a future bigger-target setting,
|
||||
manual edits, etc."""
|
||||
url = 'https://cdn-images.dzcdn.net/images/cover/abc/3000x3000-000000-80-0-0.jpg'
|
||||
assert _upgrade_deezer_cover_url(url) == url
|
||||
|
||||
|
||||
# ---------------------------------------------------------------------------
|
||||
# Defensive — non-Deezer URLs left untouched
|
||||
# ---------------------------------------------------------------------------
|
||||
|
||||
|
||||
class TestNonDeezerUrls:
|
||||
@pytest.mark.parametrize('url', [
|
||||
'https://i.scdn.co/image/spotify-id-thing', # Spotify
|
||||
'https://is4-ssl.mzstatic.com/image/100x100bb.jpg', # iTunes
|
||||
'https://coverartarchive.org/release/abc/front', # MB CAA
|
||||
'https://lastfm.freetls.fastly.net/i/u/770x0/abc.jpg', # Last.fm
|
||||
'https://example.com/random.jpg', # Random
|
||||
])
|
||||
def test_non_dzcdn_returned_unchanged(self, url):
|
||||
"""Helper must NOT touch non-Deezer URLs. Mirrors the
|
||||
defensive check pattern the iTunes and Spotify upgrade
|
||||
helpers use."""
|
||||
assert _upgrade_deezer_cover_url(url) == url
|
||||
|
||||
def test_dzcdn_url_without_size_segment_returned_unchanged(self):
|
||||
"""Defensive: if Deezer ever changes URL format, don't crash
|
||||
— return as-is and let the download attempt happen with the
|
||||
original URL."""
|
||||
url = 'https://cdn-images.dzcdn.net/images/cover/abc/some-other-format.jpg'
|
||||
assert _upgrade_deezer_cover_url(url) == url
|
||||
|
||||
|
||||
# ---------------------------------------------------------------------------
|
||||
# Empty / None inputs
|
||||
# ---------------------------------------------------------------------------
|
||||
|
||||
|
||||
class TestEmptyInputs:
|
||||
def test_empty_string(self):
|
||||
assert _upgrade_deezer_cover_url('') == ''
|
||||
|
||||
def test_none(self):
|
||||
assert _upgrade_deezer_cover_url(None) is None
|
||||
|
||||
|
||||
# ---------------------------------------------------------------------------
|
||||
# Download fallback — if upgraded URL 403s, retry with original
|
||||
# ---------------------------------------------------------------------------
|
||||
|
||||
|
||||
class TestDownloadFallbackOnCdnRefusal:
|
||||
"""If Deezer CDN refuses the upgraded 1900×1900 URL for some
|
||||
specific album (rare but possible — empirically tested 4 albums
|
||||
and none hit this, but defending the edge keeps the fix
|
||||
strictly non-regressive vs. pre-upgrade behaviour)."""
|
||||
|
||||
def test_tag_writer_retries_with_original_on_failure(self, monkeypatch):
|
||||
"""tag_writer.download_cover_art must fall back to the
|
||||
original URL when the upgraded URL fails."""
|
||||
from core import tag_writer
|
||||
|
||||
original_url = 'https://cdn-images.dzcdn.net/images/cover/abc/1000x1000-000000-80-0-0.jpg'
|
||||
upgraded_url = 'https://cdn-images.dzcdn.net/images/cover/abc/1900x1900-000000-80-0-0.jpg'
|
||||
|
||||
call_log = []
|
||||
|
||||
class _FakeResponse:
|
||||
def read(self): return b'cover-bytes'
|
||||
def info(self):
|
||||
class _Info:
|
||||
def get_content_type(_self): return 'image/jpeg'
|
||||
return _Info()
|
||||
def __enter__(self): return self
|
||||
def __exit__(self, *a): pass
|
||||
|
||||
def fake_urlopen(url, timeout=None):
|
||||
call_log.append(url)
|
||||
if url == upgraded_url:
|
||||
raise Exception("403 Forbidden")
|
||||
return _FakeResponse()
|
||||
|
||||
monkeypatch.setattr('core.tag_writer.urllib.request.urlopen', fake_urlopen)
|
||||
|
||||
result = tag_writer.download_cover_art(original_url)
|
||||
|
||||
assert result == (b'cover-bytes', 'image/jpeg')
|
||||
# Tried upgraded first, then fell back to original
|
||||
assert call_log == [upgraded_url, original_url]
|
||||
|
||||
def test_tag_writer_no_fallback_for_non_dzcdn_url(self, monkeypatch):
|
||||
"""Non-Deezer URLs go through unchanged — no upgrade, no
|
||||
fallback. Fast path preserved."""
|
||||
from core import tag_writer
|
||||
|
||||
spotify_url = 'https://i.scdn.co/image/abc'
|
||||
call_log = []
|
||||
|
||||
def fake_urlopen(url, timeout=None):
|
||||
call_log.append(url)
|
||||
raise Exception("network error")
|
||||
|
||||
monkeypatch.setattr('core.tag_writer.urllib.request.urlopen', fake_urlopen)
|
||||
|
||||
result = tag_writer.download_cover_art(spotify_url)
|
||||
|
||||
assert result is None
|
||||
# Single attempt — no Deezer fallback path triggered
|
||||
assert call_log == [spotify_url]
|
||||
|
|
@ -3416,6 +3416,7 @@ const WHATS_NEW = {
|
|||
'2.4.3': [
|
||||
// --- post-release patch work on the 2.4.3 line — entries hidden by _getLatestWhatsNewVersion until the build version bumps ---
|
||||
{ date: 'Unreleased — 2.4.3 patch work' },
|
||||
{ title: 'Deezer Cover Art: Embedded Covers No Longer Look Blurry', desc: 'discord report (tim): downloaded cover art via deezer metadata source came out visibly blurry in navidrome and on phones — particularly noticeable on large displays. cause: deezer\'s api returns `cover_xl` urls at 1000×1000 but the underlying cdn serves up to 1900×1900 by rewriting the size segment in the url path. soulsync wasn\'t doing the rewrite — same as iTunes mzstatic and spotify scdn already get upgraded. now `_upgrade_deezer_cover_url` (mirrors `_upgrade_spotify_image_url` pattern) rewrites the cdn url to request 1900×1900 before download. cdn serves source-native size when source < target so asking for 1900 on smaller-source albums returns the same bytes (no upscaling, no failure). applied at both download sites — auto post-process flow + the enhanced library view\'s "write tags to file" feature. existing `prefer_caa_art` toggle in settings → library → post-processing remains as the orthogonal workaround for users who want even higher quality (musicbrainz cover art archive, often 3000×3000+). 16 new tests pin: standard upgrade, alternate dzcdn host, artist picture urls, custom target sizes, idempotency on already-upgraded urls, defensive on non-deezer urls (spotify/itunes/caa/lastfm/random), empty/none handling.', page: 'settings' },
|
||||
{ title: 'Cross-Script Artist Names No Longer Quarantine Files (Hiroyuki Sawano / 澤野弘之, Сергей Лазарев / Sergey Lazarev)', desc: 'github issue #442 (afonsog6): files where the artist tag was in one script and the expected metadata was in another — japanese kanji `澤野弘之` for `hiroyuki sawano`, cyrillic `сергей лазарев` for `sergey lazarev`, etc. — got quarantined post-download because acoustid verification scored the artist similarity at 0% (the two scripts share no characters). reporter could not even rescue the file via manual import — the import-modal goes through the same verifier and re-quarantined the same file. cause: verifier compared expected vs actual artist with raw `_similarity` and never consulted musicbrainz aliases, even though MB exposes them on every artist record. fix: new `core/matching/artist_aliases.py` pure helper with alias-aware comparison + new `artists.aliases` JSON column populated by the existing MB enrichment worker on every artist match (one extra `inc=aliases` request per artist) + new multi-tier resolver `MusicBrainzService.lookup_artist_aliases` (library DB → cache → live MB) so the verifier finds aliases even for un-enriched artists without thrashing the MB API. verifier resolves aliases ONCE per `verify_audio_file` call and feeds them through three artist comparison sites (best-match scoring, secondary scan when title matches but artist doesn\'t, final fallback scan). reporter\'s exact two cases reproduced as regression tests with stubbed MB service. backward compat: aliases unavailable / MB unreachable → verifier falls back to direct similarity (identical to pre-fix behaviour — never quarantines stricter than today). 70 new tests pin every layer: pure helper (28), service methods (31), verifier integration (11). audited adjacent artist-comparison sites (auto-import single-track id, discovery scoring, matching engine) — left untouched per scope discipline since they aren\'t the user-reported pain.', page: 'downloads' },
|
||||
{ title: 'Plex: Library Scan Trigger No Longer Fails On Non-English Section Names', desc: 'github issue #535 (adrigzr): plex servers with the music library named anything other than "music" — Música, Musique, Musik, Musica, etc. — got a `Failed to trigger library scan for "Music": Invalid library section: Music` error after every import cycle, and `wishlist.processing` kept reporting "missing from media server after sync" for tracks that DID import correctly because the post-import scan never fired. cause: `trigger_library_scan` and `is_library_scanning` ignored the auto-detected `self.music_library` (correctly populated by `_find_music_library` filtering by `section.type == "artist"`) and called `self.server.library.section(library_name)` with a hardcoded "music" default — raised NotFound on any non-english server. read methods like `get_artists` already routed through `_get_music_sections` so they didn\'t have the bug; this aligns the scan-trigger path with the same resolution. fix: both single-library branches prefer `self.music_library` first, fall back to literal section lookup only when auto-detection hasn\'t run. activity-feed match in `is_library_scanning` also corrected to use the resolved section\'s actual title instead of the unused `library_name` arg — the prior log line read "triggered scan for music" even on Spanish servers. 13 new tests pin: trigger uses auto-detected section across 6 locale variants (Música / Musique / Musik / Musica / 音乐 / موسيقى), backward-compat fallback when music_library is None, explicit library_name kwarg ignored when auto-detected section exists, log line surfaces correct section title, scan-status check uses auto-detected section\'s `refreshing` attr, activity-feed match filters by resolved title (not library_name).', page: 'settings' },
|
||||
{ title: 'Search For Match: No More Karaoke / Cover / "Originally Performed By" Junk At The Top', desc: 'github issue #534 (radoslav-orlov): typing "dirty white boy" + "foreigner" into the import-modal "search for match" dialog returned karaoke versions, "originally performed by" compilations, and tribute-band cuts ranked above the actual foreigner studio recording in some regions. user had to scroll past 5+ junk results before finding the canonical track. fix: new `core/metadata/relevance.py` helper reranks results locally with cover/karaoke/tribute/re-recorded penalties (multiplier 0.05× — effectively buries) + exact-artist-match boost (1.5×) + variant-tag (live/acoustic/remix/remaster) penalty (0.4×, skipped when user explicitly typed the variant — searching "track (live)" still ranks live versions correctly). applied at the deezer + itunes + spotify search-tracks endpoints so all three sources behave consistently. validated against live deezer api with the actual #534 query: real foreigner head games cut now lands at #1, live versions follow, karaoke / cover / tribute variants drop to positions 11-15. deezer client also gained optional field-scoped query kwargs (`track="X" artist="Y"`) that build deezer\'s advanced search syntax `track:"X" artist:"Y"` for future opt-in callers (e.g. exact-match flows where api-level filtering is more important than ranking) — kept in client but NOT used at the import-modal endpoint after live testing showed the advanced syntax has its own ranking bias (surfaced "(2008 remaster)" instead of the canonical recording). free-text + local rerank is the more reliable combination here. 75 new tests pin every scoring component, pattern detection (13 cover patterns, 11 variant patterns, 3 fields), score composition (real-cut > karaoke > remaster > re-recorded), the issue #534 screenshot reproduced as a regression test, deezer client query construction + free-text fallback safety net.', page: 'import' },
|
||||
|
|
|
|||
Loading…
Reference in a new issue