vndangkhoa/kv-music
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Revert "Fix v2 artists API: correct data access paths"

Browse source
This commit is contained in:
Samidy 2026-04-19 01:49:00 +03:00 committed by GitHub
parent cfe1239c79
commit b528720e8b
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 89 additions and 210 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

292
js/HiFi.ts
View file

@ -100,48 +100,26 @@ export interface TidalArtistProfile {
id: number; id: number;
/** Artist display name. */ /** Artist display name. */
name: string; name: string;
/** /** Roles this artist holds on TIDAL, e.g. `["ARTIST", "CONTRIBUTOR"]`. */
* Roles this artist holds on TIDAL, e.g. `["ARTIST", "CONTRIBUTOR"]`. artistTypes: string[];
* Present in v1 responses; absent from v2.
*/
artistTypes?: string[];
/** Canonical TIDAL artist URL. */ /** Canonical TIDAL artist URL. */
url: string; url: string;
/** Picture UUID, or `null` if no image is available. */ /** Picture UUID, or `null` if no image is available. */
picture: string | null; picture: string | null;
/** /** Fallback album cover UUID used when no artist picture exists, or `null`. */
* Fallback album cover UUID used when no artist picture exists, or `null`. selectedAlbumCoverFallback: string | null;
* Present in v1 responses; absent from v2. /** Popularity score (0-100). */
*/
selectedAlbumCoverFallback?: string | null;
/** Popularity score (0-100; the raw v2 float (0-1) is multiplied by 100 and rounded). */
popularity: number; popularity: number;
/** /** List of credited roles for this artist. */
* List of credited roles for this artist. artistRoles: TidalArtistRole[];
* Present in v1 responses; absent from v2. /** Map of mix type → mix ID, e.g. `{ "ARTIST_MIX": "000ff..." }`. */
*/ mixes: Record<string, string>;
artistRoles?: TidalArtistRole[]; /** TIDAL handle, or `null` if not set. */
/** handle: string | null;
* Map of mix type → mix ID, e.g. `{ "ARTIST_MIX": "000ff..." }`. /** Associated TIDAL user ID, or `null`. */
* Present in v1 responses; absent from v2. userId: number | null;
*/
mixes?: Record<string, string>;
/**
* TIDAL handle, or `null` if not set.
* Present in v1 responses; absent from v2.
*/
handle?: string | null;
/**
* Associated TIDAL user ID, or `null`.
* Present in v1 responses; absent from v2.
*/
userId?: number | null;
/** Whether the artist is currently spotlighted. */ /** Whether the artist is currently spotlighted. */
spotlighted?: boolean; spotlighted: boolean;
/** Whether artist contributions are enabled (v2 only). */
contributionsEnabled?: boolean;
/** Owner type, e.g. `"LABEL"` (v2 only). */
ownerType?: string;
} }
/** /**
@ -598,35 +576,13 @@ export interface ArtistCover {
/** /**
* Response returned by the `/artist` route when an `id` query parameter is supplied. * Response returned by the `/artist` route when an `id` query parameter is supplied.
* Contains the artist's full profile, optional cover image URL, and (when using the * Contains the artist's full profile and optional cover image URL.
* v2 OpenAPI endpoint) the artist's albums and tracks inline.
*/ */
export interface ArtistByIdResponse extends VersionedResponse { export interface ArtistByIdResponse extends VersionedResponse {
/** Full TIDAL artist profile data. */ /** Full TIDAL artist profile data. */
artist: TidalArtistProfile; artist: TidalArtistProfile;
/** Cover image URL at 750 px, or `null` if no picture is available. */ /** Cover image URL at 750 px, or `null` if no picture is available. */
cover: ArtistCover | null; cover: ArtistCover | null;
/**
* Albums associated with the artist (populated by the v2 endpoint).
* Items are partial because the v2 artist endpoint does not return every
* field present in a v1 full-album response.
* Absent in v1 responses.
*/
albums?: { items: Partial<TidalAlbum>[] };
/**
* Top tracks for the artist (populated by the v2 endpoint).
* Items are partial because the v2 artist endpoint does not return every
* field present in a v1 full-track response.
* Absent in v1 responses.
*/
tracks?: Partial<TidalTrack>[];
/**
* Inline biography extracted from the v2 `included` array.
* Only `text` and `source` are available from this endpoint;
* for the full biography use the `/artist/bio` route.
* Absent in v1 responses and when no biography is available.
*/
biography?: { text: string; source?: string } | null;
} }
/** /**
@ -1054,14 +1010,12 @@ interface JsonApiIncludeAttributes {
externalLinks?: Array<{ href: string; meta: { type: string } }>; externalLinks?: Array<{ href: string; meta: { type: string } }>;
spotlighted?: boolean; spotlighted?: boolean;
contributionsEnabled?: boolean; contributionsEnabled?: boolean;
ownerType?: string;
selectedAlbumCoverFallback?: string | null; selectedAlbumCoverFallback?: string | null;
files?: Array<{ href: string }>; files?: Array<{ href: string }>;
title?: string; title?: string;
barcodeId?: string; barcodeId?: string;
numberOfVolumes?: number; numberOfVolumes?: number;
numberOfItems?: number; numberOfItems?: number;
/** ISO 8601 duration string, e.g. `"PT3M45S"`. */
duration?: string; duration?: string;
explicit?: boolean; explicit?: boolean;
releaseDate?: string; releaseDate?: string;
@ -1072,8 +1026,6 @@ interface JsonApiIncludeAttributes {
albumType?: string; albumType?: string;
createdAt?: string; createdAt?: string;
type?: string; type?: string;
text?: string;
source?: string;
} }
/** An included resource node from a TIDAL OpenAPI JSON:API response. */ /** An included resource node from a TIDAL OpenAPI JSON:API response. */
@ -1081,26 +1033,12 @@ interface JsonApiInclude {
id: string; id: string;
type: string; type: string;
attributes: JsonApiIncludeAttributes; attributes: JsonApiIncludeAttributes;
/** relationships?: Record<string, { data?: JsonApiRef[] }>;
* Relationships map. `data` may be a single ref (e.g. `biography`) or an array
* (e.g. `profileArt`, `coverArt`, `artists`).
*/
relationships?: Record<string, { data?: JsonApiRef | JsonApiRef[]; links?: Record<string, unknown> }>;
} }
/** A TIDAL OpenAPI JSON:API list response for relationship endpoints (e.g. similar-artists/albums). */ /** A TIDAL OpenAPI JSON:API list response (similar-artists/albums). */
interface JsonApiListResponse { interface JsonApiListResponse {
/** Top-level data array returned by v2 list/relationship endpoints. */
data?: JsonApiRef[]; data?: JsonApiRef[];
/** Top-level included resources returned alongside the data array. */
included?: JsonApiInclude[];
}
/** A TIDAL OpenAPI JSON:API artist detail response (v2 /artists/{id} endpoint). */
interface JsonApiArtistResponse {
/** Single artist resource object. */
data?: JsonApiInclude;
/** Included side-loaded resources (albums, tracks, artworks, biographies, etc.). */
included?: JsonApiInclude[]; included?: JsonApiInclude[];
} }
@ -1147,7 +1085,7 @@ export enum HiFiClientEvents {
} }
class HiFiClient { class HiFiClient {
static readonly API_VERSION = '2.9'; static readonly API_VERSION = '2.7';
static readonly BROWSER_CLIENT_ID = 'txNoH4kkV41MfH25'; static readonly BROWSER_CLIENT_ID = 'txNoH4kkV41MfH25';
static readonly BROWSER_CLIENT_SECRET = 'dQjy0MinCEvxi1O4UmxvxWnDjt4cgHBPw8ll6nYBk98='; static readonly BROWSER_CLIENT_SECRET = 'dQjy0MinCEvxi1O4UmxvxWnDjt4cgHBPw8ll6nYBk98=';
@ -1494,17 +1432,6 @@ class HiFiClient {
return parts.length >= 9 ? parts.slice(4, 9).join('-') : null; return parts.length >= 9 ? parts.slice(4, 9).join('-') : null;
} }
/**
* Parses an ISO 8601 duration string (e.g. `"PT3M45S"`) into whole seconds.
* Returns `0` for missing or unparseable values.
*/
static #parseDuration(iso?: string | null): number {
if (!iso) return 0;
const m = iso.match(/^PT(?:(\d+)H)?(?:(\d+)M)?(?:(\d+(?:\.\d+)?)S)?$/);
if (!m) return 0;
return Number(m[1] ?? 0) * 3600 + Number(m[2] ?? 0) * 60 + Math.round(Number(m[3] ?? 0));
}
async #withAlbumTrackSlot<T>(fn: () => Promise<T>) { async #withAlbumTrackSlot<T>(fn: () => Promise<T>) {
if (this.#albumTracksActive >= this.#albumTracksMax) { if (this.#albumTracksActive >= this.#albumTracksMax) {
await new Promise<void>((res) => this.#albumTracksQueue.push(res)); await new Promise<void>((res) => this.#albumTracksQueue.push(res));
@ -1673,9 +1600,10 @@ class HiFiClient {
} }
return { return {
...attr,
id: Number(aid), id: Number(aid),
name: attr.name ?? '', name: attr.name ?? '',
picture: pic_id ?? null, picture: pic_id ?? attr.selectedAlbumCoverFallback ?? null,
url: `http://www.tidal.com/artist/${aid}`, url: `http://www.tidal.com/artist/${aid}`,
relationType: 'SIMILAR_ARTIST', relationType: 'SIMILAR_ARTIST',
popularity: attr.popularity ?? 0, popularity: attr.popularity ?? 0,
@ -1752,27 +1680,13 @@ class HiFiClient {
} }
return { return {
...attr,
id: Number(aid), id: Number(aid),
title: attr.title ?? '', title: attr.title ?? '',
barcodeId: attr.barcodeId ?? '',
numberOfVolumes: attr.numberOfVolumes ?? 1,
numberOfItems: attr.numberOfItems ?? 0,
duration: attr.duration ?? '',
explicit: attr.explicit ?? false,
releaseDate: attr.releaseDate ?? '',
copyright: attr.copyright ?? { text: '' },
popularity: attr.popularity ?? 0,
accessType: attr.accessType ?? '',
availability: attr.availability ?? [],
mediaTags: attr.mediaTags ?? [],
externalLinks: attr.externalLinks ?? [],
type: attr.type ?? '',
albumType: attr.albumType ?? '',
createdAt: attr.createdAt,
cover: cover_id ?? '', cover: cover_id ?? '',
artists: artist_list, artists: artist_list,
url: `http://www.tidal.com/album/${aid}`, url: `http://www.tidal.com/album/${aid}`,
}; } as TidalSimilarAlbum;
}; };
return HiFiClient.#jsonResponse({ return HiFiClient.#jsonResponse({
@ -1807,63 +1721,53 @@ class HiFiClient {
if (!id && !f) throw new ResponseError(400, 'Provide id or f query param'); if (!id && !f) throw new ResponseError(400, 'Provide id or f query param');
if (id) { if (id) {
// Fetch the full v1 artist profile (has artistTypes, artistRoles, mixes, integer const artist_url = `https://openapi.tidal.com/v2/artists/${id}`;
// popularity, handle, userId, selectedAlbumCoverFallback, etc.) and the v2 artist const payload = await this.#fetchJson<any>(
// resource (for albums, tracks, biography, and profile/cover art) in parallel. artist_url,
const [v1Artist, v2Payload] = await Promise.all([ {
this.#fetchJson<TidalArtistProfile>( countryCode: this.#countryCode,
`https://api.tidal.com/v1/artists/${id}`, include: 'albums,albums.coverArt,tracks,tracks.albums,biography,profileArt',
{ countryCode: this.#countryCode }, collapseBy: 'FINGERPRINT',
signal },
), signal
this.#fetchJson<JsonApiArtistResponse>( );
`https://openapi.tidal.com/v2/artists/${id}`,
{
countryCode: this.#countryCode,
include: 'albums,albums.coverArt,tracks,tracks.albums,biography,profileArt',
collapseBy: 'FINGERPRINT',
},
signal
).catch((): JsonApiArtistResponse | null => null),
]);
const includedMap = new Map<string, JsonApiInclude>(); const includedMap = new Map<string, any>();
if (Array.isArray(v2Payload?.included)) { if (Array.isArray(payload?.included)) {
for (const item of v2Payload.included) { for (const item of payload.included) {
includedMap.set(`${item.type}:${item.id}`, item); includedMap.set(`${item.type}:${item.id}`, item);
} }
} }
const getPic = (item: JsonApiInclude | undefined, relName: string): string | null => { const getPic = (item: any, relName: string) => {
const relData = item?.relationships?.[relName]?.data; if (item?.relationships?.[relName]?.data?.[0]) {
const picRef = Array.isArray(relData) ? relData[0] : undefined; const picRef = item.relationships[relName].data[0];
if (!picRef) return null; const pic = includedMap.get(`artworks:${picRef.id}`);
const pic = includedMap.get(`artworks:${picRef.id}`); return pic?.attributes?.files?.[0]?.href
const href = pic?.attributes?.files?.[0]?.href; ? HiFiClient.#extractUuidFromTidalUrl(pic.attributes.files[0].href)
return href ? HiFiClient.#extractUuidFromTidalUrl(href) : null; : null;
}
return null;
}; };
const v2Data = v2Payload?.data; const data = payload?.data;
let biography: any = null;
// Biography: v2 returns a single-ref relationship, not an array if (data?.relationships?.biography?.data) {
const bioRelData = v2Data?.relationships?.biography?.data; const bioRef = data.relationships.biography.data;
const bioRef = Array.isArray(bioRelData) ? bioRelData[0] : bioRelData; const bioItem =
const bioItem = bioRef includedMap.get(`biographies:${bioRef.id}`) || includedMap.get(`biography:${bioRef.id}`);
? (includedMap.get(`${bioRef.type}:${bioRef.id}`) ?? if (bioItem) {
includedMap.get(`biographies:${bioRef.id}`) ?? biography = { text: bioItem.attributes?.text, source: bioItem.attributes?.source };
includedMap.get(`biography:${bioRef.id}`)) }
: undefined;
// Use the full v1 artist profile as-is. It already carries all fields the UI
// needs: name, picture UUID, popularity (0-100 integer), artistTypes, artistRoles,
// mixes, handle, userId, selectedAlbumCoverFallback, url, spotlighted, etc.
const artist_data: TidalArtistProfile = v1Artist;
// Fall back to the v2 profileArt UUID when v1 has no picture.
if (!artist_data.picture) {
artist_data.picture = getPic(v2Data, 'profileArt');
} }
const artist_data: any = {
id: Number(data?.id || id),
name: data?.attributes?.name || '',
picture: getPic(data, 'profileArt') || data?.attributes?.selectedAlbumCoverFallback || null,
biography: biography,
};
const picture = artist_data.picture; const picture = artist_data.picture;
let cover: ArtistCover | null = null; let cover: ArtistCover | null = null;
if (picture) { if (picture) {
@ -1875,80 +1779,60 @@ class HiFiClient {
}; };
} }
const albums: Partial<TidalAlbum>[] = []; const albums: any[] = [];
const tracks: Partial<TidalTrack>[] = []; const tracks: any[] = [];
const artistRef: TidalArtistRef = { if (data?.relationships?.albums?.data) {
id: artist_data.id, for (const ref of data.relationships.albums.data) {
name: artist_data.name,
type: 'MAIN',
picture: artist_data.picture,
handle: artist_data.handle ?? null,
};
const albumsRelData = v2Data?.relationships?.albums?.data;
if (Array.isArray(albumsRelData)) {
for (const ref of albumsRelData) {
const al = includedMap.get(`albums:${ref.id}`); const al = includedMap.get(`albums:${ref.id}`);
if (al) { if (al) {
albums.push({ albums.push({
id: Number(al.id), id: Number(al.id),
title: al.attributes?.title ?? '', title: al.attributes?.title,
duration: HiFiClient.#parseDuration(al.attributes?.duration), duration: al.attributes?.duration ? 100 : undefined,
numberOfTracks: al.attributes?.numberOfItems ?? 0, numberOfTracks: al.attributes?.numberOfItems,
releaseDate: al.attributes?.releaseDate ?? '', releaseDate: al.attributes?.releaseDate,
type: al.attributes?.albumType ?? '', type: al.attributes?.albumType,
cover: getPic(al, 'coverArt') ?? '', cover: getPic(al, 'coverArt'),
artist: artistRef, artist: { id: artist_data.id, name: artist_data.name },
artists: [artistRef],
}); });
} }
} }
} }
const tracksRelData = v2Data?.relationships?.tracks?.data; if (data?.relationships?.tracks?.data) {
if (Array.isArray(tracksRelData)) { for (const ref of data.relationships.tracks.data) {
for (const ref of tracksRelData) {
const tr = includedMap.get(`tracks:${ref.id}`); const tr = includedMap.get(`tracks:${ref.id}`);
if (tr) { if (tr) {
const albumRelData = tr.relationships?.albums?.data; let albumInfo = undefined;
const albumRef = Array.isArray(albumRelData) ? albumRelData[0] : undefined; if (tr.relationships?.albums?.data?.[0]) {
const aItem = albumRef ? includedMap.get(`albums:${albumRef.id}`) : undefined; const aRef = tr.relationships.albums.data[0];
const albumInfo: TidalTrackAlbumRef | undefined = aItem const aItem = includedMap.get(`albums:${aRef.id}`);
? { if (aItem) {
id: Number(aItem.id), albumInfo = {
title: aItem.attributes?.title ?? '', id: Number(aItem.id),
cover: getPic(aItem, 'coverArt') ?? '', title: aItem.attributes?.title,
vibrantColor: '', cover: getPic(aItem, 'coverArt'),
videoCover: null, };
} }
: undefined; }
tracks.push({ tracks.push({
id: Number(tr.id), id: Number(tr.id),
title: tr.attributes?.title ?? '', title: tr.attributes?.title,
duration: HiFiClient.#parseDuration(tr.attributes?.duration), duration: tr.attributes?.duration ? 100 : undefined,
// v2 popularity is a 0-1 float; normalise to 0-100 so the consumer
// can sort tracks the same way it sorts v1 tracks.
popularity: Math.round((tr.attributes?.popularity ?? 0) * 100),
album: albumInfo, album: albumInfo,
artist: artistRef, artist: { id: artist_data.id, name: artist_data.name },
artists: [artistRef],
}); });
} }
} }
} }
const biography = bioItem
? { text: bioItem.attributes?.text ?? '', source: bioItem.attributes?.source }
: null;
return HiFiClient.#jsonResponse({ return HiFiClient.#jsonResponse({
version: HiFiClient.API_VERSION, version: HiFiClient.API_VERSION,
artist: artist_data, artist: artist_data,
cover, cover,
albums: { items: albums }, albums: { items: albums },
tracks, tracks,
biography,
}); });
} }

7
js/api.js
View file

@ -1167,12 +1167,7 @@ export class LosslessAPI {
// Enrich tracks with album release dates // Enrich tracks with album release dates
const tracks = options.lightweight ? topTracks : await this.enrichTracksWithAlbumDates(topTracks); const tracks = options.lightweight ? topTracks : await this.enrichTracksWithAlbumDates(topTracks);
// Biography is included inline when the id-path response carries it (from v2 included const result = { ...artist, albums, eps, tracks, videos };
// resources). When absent (v1-only profile with no v2 call, or no biography available),
// getArtistBiography is called lazily when the page renders and artist.biography is falsy.
const biography = primaryData.biography || null;
const result = { ...artist, albums, eps, tracks, videos, biography };
if (!(primaryResponse instanceof TidalResponse)) { if (!(primaryResponse instanceof TidalResponse)) {
await this.cache.set('artist', cacheKey, result); await this.cache.set('artist', cacheKey, result);