Docker/NovaSonic
0
0 Fork 0
Code Issues 15 Pull Requests Actions Packages Projects 1 Releases 1 Wiki Activity

fix(backend): keep source artist name as-is, no iTunes override

Browse Source 
Artist name from Spotify/source is stored directly in folder, filename,
and tags without being replaced by the iTunes artistName. iTunes is used
only for supplemental metadata (genre, year, cover, duration, trackNum,
albumArtist/collectionArtistName). Sort tags still receive romaji via
romajiForStorage() in ytdlp.js.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
HoneyandClaude Sonnet 4.6 committed 2026-06-21 16:43:15 +02:00
1 parent ff79428683
commit 1616d98fa6
1 file changed
+7 -8
Download Patch File Download Diff File Expand all files Collapse all files
backend/novasonic-server.js
+7 -8
View File
@@ -600,8 +600,8 @@ app.post('/api/download', async (req, res) => {
if (!meta.trackNum && genreMeta.trackNum) { meta.trackNum = genreMeta.trackNum; } if (!meta.trackNum && genreMeta.trackNum) { meta.trackNum = genreMeta.trackNum; }
} }
} }
const finalArtist = meta.itunesArtistName || artist; // Keep source (Spotify) artist name as-is — no iTunes override
const result = await downloadTrack({ jobId, artist: finalArtist, rawArtist: artist, title, ...meta, sourceUrl: sourceUrl || sp.sourceUrl || undefined }); const result = await downloadTrack({ jobId, artist, rawArtist: artist, title, ...meta, sourceUrl: sourceUrl || sp.sourceUrl || undefined });
if (result?.skipped) { if (result?.skipped) {
updateJob(jobId, { status: 'done', progress: 100, message: 'Bereits vorhanden – übersprungen', skipped: true }); updateJob(jobId, { status: 'done', progress: 100, message: 'Bereits vorhanden – übersprungen', skipped: true });
} else if (result?.notFound) { } else if (result?.notFound) {
@@ -688,14 +688,13 @@ app.post('/api/download', async (req, res) => {
log(jobId, `📋 iTunes: "${trackArtist} – ${trackTitle}"`); log(jobId, `📋 iTunes: "${trackArtist} – ${trackTitle}"`);
itunes = await lookupItunesTrackMeta(searchArtist, itunesTitle); itunes = await lookupItunesTrackMeta(searchArtist, itunesTitle);
} }
// Prefer iTunes-resolved name (from server lookup or frontend enrichment) for sort tags / folders. // Keep source (Spotify) artist name as-is — no iTunes override.
// trackArtist stays as rawArtist for main tags and YTM search. // iTunes collectionArtistName is used as albumArtist for consistent album grouping.
const resolvedArtist = itunes.itunesArtistName || track.itunesArtistName || trackArtist;
return downloadTrack({ return downloadTrack({
jobId, jobId,
artist: resolvedArtist, // iTunes English name → used for sort tags + folder names artist: trackArtist,
rawArtist: trackArtist, // original Spotify/Japanese name → used for main tag + YTM search rawArtist: trackArtist,
albumArtist: resolvedArtist, albumArtist: itunes.albumArtist || trackArtist,
title: trackTitle, title: trackTitle,
album: itunes.album || track.collectionName || track.album || '', album: itunes.album || track.collectionName || track.album || '',
year: itunes.year || (track.releaseDate ? new Date(track.releaseDate).getFullYear() : undefined) || track.year, year: itunes.year || (track.releaseDate ? new Date(track.releaseDate).getFullYear() : undefined) || track.year,