Spotify: correctly propagate TOTALTRACKS and TOTALDISCS metadata #15

Merged

maleszka merged 7 commits from fix/totaltracks into main

2026-07-06 15:48:32 +02:00

maleszka commented

2026-07-06 15:48:23 +02:00

Owner

closes #2

maleszka added 7 commits

2026-07-06 15:48:23 +02:00

[spec] design logic how TOTALTRACKS metadata should be populated 948c487def

[provider] thread a resolved Album into map_track for total_tracks/total_discs 5ee5a2f00b

track.album is only the per-track metadata stub and never populates discs,
so map_track now takes the caller's resolved Album instead of reading the
stub. Call sites still pass the stub for now; each will be fixed to pass a
real Album in the following commits.

[provider] reuse the fetched Album for total_tracks/total_discs on album downloads e0e5a09152

resolve_track_list's album branch already fetches the full Album to build
track_uris and previously dropped it; now it's kept and passed into
map_track for every track in the album, fixing TOTALTRACKS/TOTALDISCS for
the common full-album download case with no extra network round-trip.

[provider] resolve per-track Album for single-track, playlist, and search paths 6c06ba0ad9

resolve_track, resolve_track_list's playlist branch, and search all lacked a
pre-fetched Album to reuse (playlists and search results can span different
albums), so each still passed the always-empty stub. Add a per-batch
Album cache (resolve_album_cached/AlbumCache) shared by the playlist branch
and search to avoid an Album::get round-trip per track when consecutive
results share an album, and fetch the track's own Album directly for the
single-track path. Also adds an ignored real-session integration test that
downloads a known multi-disc track and checks the written OGG's
TOTALDISCS/TOTALTRACKS.

[README] add asciinema screencast 8038a7d977

[main] log errors only without --debug flag 095ee3cc61

[spec] propagate changes to core architecture 41ceac938a