fix(enrich): не помечать done при сбое запроса iTunes (промах не застывает)
Отличаем сбой запроса (сеть/HTTP-ошибка → ретраить, оставляем pending) от чистого 'не найдено' (done). Раньше транзиентный сбой iTunes под нагрузкой навсегда лишал трек обложки. fetchItunes теперь бросает при !res.ok. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This commit is contained in:
@@ -117,7 +117,14 @@ export class EnrichmentService {
|
||||
|
||||
// iTunes: обложка (покрытие почти как у Record) + альбом/год/жанр как
|
||||
// фолбэк к Discogs. Гибрид: стили и лейбл — только Discogs.
|
||||
const itunes = await this.fetchItunes(track.artist, track.song);
|
||||
// Отличаем сбой запроса (ретраить) от чистого «не найдено» (done).
|
||||
let itunes: Awaited<ReturnType<typeof this.fetchItunes>> = null;
|
||||
let itunesFailed = false;
|
||||
try {
|
||||
itunes = await this.fetchItunes(track.artist, track.song);
|
||||
} catch {
|
||||
itunesFailed = true;
|
||||
}
|
||||
|
||||
// Обложка → WebP к себе (если ещё не наша)
|
||||
let coverUrl = track.coverUrl;
|
||||
@@ -136,9 +143,10 @@ export class EnrichmentService {
|
||||
itunes?.releaseDate ??
|
||||
(data?.year ? new Date(Date.UTC(data.year, 0, 1)) : null);
|
||||
|
||||
// Без токена Discogs стили/лейбл не получим — оставляем pending, чтобы
|
||||
// добрать позже (но обложку/жанр-iTunes уже сохранили).
|
||||
const enriched = this.discogs.enabled;
|
||||
// Помечаем done, если обогатились. НЕ помечаем (оставляем pending для
|
||||
// ретрая), если: нет токена Discogs, ИЛИ запрос к iTunes упал И обложку
|
||||
// так и не получили (транзиентный сбой — промах не должен застывать).
|
||||
const enriched = this.discogs.enabled && !(itunesFailed && !coverUrl);
|
||||
|
||||
await this.prisma.track.update({
|
||||
where: { id: trackId },
|
||||
@@ -179,7 +187,6 @@ export class EnrichmentService {
|
||||
releaseDate: Date | null;
|
||||
genre: string | null;
|
||||
} | null> {
|
||||
try {
|
||||
// Пунктуация в названии («St.Thomas», «feat.») ломает поиск iTunes —
|
||||
// заменяем все не-буквенно-цифровые символы на пробел и схлопываем.
|
||||
const clean = `${artist} ${song}`
|
||||
@@ -188,10 +195,11 @@ export class EnrichmentService {
|
||||
.trim();
|
||||
const term = encodeURIComponent(clean);
|
||||
const url = `https://itunes.apple.com/search?term=${term}&entity=song&limit=1`;
|
||||
// Бросаем при сетевой/HTTP-ошибке — это сбой, а не «не найдено».
|
||||
const res = await fetch(url, {
|
||||
headers: { 'User-Agent': 'radiOLA/1.0 +https://radiola.app' },
|
||||
});
|
||||
if (!res.ok) return null;
|
||||
if (!res.ok) throw new Error(`iTunes ${res.status}`);
|
||||
const data = (await res.json()) as {
|
||||
results?: Array<{
|
||||
artworkUrl100?: string;
|
||||
@@ -201,7 +209,7 @@ export class EnrichmentService {
|
||||
}>;
|
||||
};
|
||||
const r = data.results?.[0];
|
||||
if (!r) return null;
|
||||
if (!r) return null; // чистое «не найдено»
|
||||
|
||||
const cover = r.artworkUrl100
|
||||
? r.artworkUrl100.replace(/\/\d+x\d+bb\./, '/600x600bb.')
|
||||
@@ -216,9 +224,6 @@ export class EnrichmentService {
|
||||
releaseDate: validDate,
|
||||
genre: r.primaryGenreName ?? null,
|
||||
};
|
||||
} catch {
|
||||
return null;
|
||||
}
|
||||
}
|
||||
|
||||
private isSelfHosted(url: string): boolean {
|
||||
|
||||
Reference in New Issue
Block a user