Files
jellytau/src/lib/components/library/ArtistDetailView.svelte
T
dtourolle d54d8cc7c4 refactor(logging): route frontend console calls through the logger
TRACES: | DR-204

484 ungated `console.*` calls across 63 non-test frontend files shipped to
end users with no way to turn them off. Mechanical substitution, no control
flow, error handling or message semantics changed:

  console.log / console.debug -> log.debug
  console.info                -> log.info
  console.warn                -> log.warn
  console.error               -> log.error

Hand-written `"[Scope] …"` prefixes are dropped where the logger's scope
now carries them; scope names that already existed are preserved verbatim
(`[Auth]`, `[VideoPlayer]`, `[PiP]`, …) and inferred from the filename
where a file had none. `src/routes/player/[id]/+page.svelte` keeps its
`NextEpisode` and `AutoPlay` sub-scopes as separate loggers rather than
flattening them into the page scope.

`grep -rn 'console\.' src/` now matches nothing outside the tests and the
facade itself.
2026-08-20 19:29:59 +02:00

255 lines
8.1 KiB
Svelte

<script lang="ts">
import { onMount } from "svelte";
import { truncateMiddle } from "$lib/utils/truncateMiddle";
import { auth } from "$lib/stores/auth";
import type { MediaItem } from "$lib/api/types";
import LibraryGrid from "./LibraryGrid.svelte";
import TrackList from "./TrackList.svelte";
import CachedImage from "$lib/components/common/CachedImage.svelte";
import FavoriteButton from "$lib/components/FavoriteButton.svelte";
import { favoriteOverrides, resolveIsFavorite } from "$lib/stores/favorites";
import { createLogger } from "$lib/utils/logger";
const log = createLogger("ArtistDetailView");
interface Props {
artist: MediaItem;
}
let { artist }: Props = $props();
let albums = $state<MediaItem[]>([]);
let singles = $state<MediaItem[]>([]);
let topTracks = $state<MediaItem[]>([]);
let relatedArtists = $state<MediaItem[]>([]);
let albumsLoading = $state(true);
let singlesLoading = $state(true);
let tracksLoading = $state(true);
let artistsLoading = $state(true);
let showSingles = $state(false);
let showAppears = $state(false);
onMount(async () => {
await loadArtistContent();
});
async function loadArtistContent() {
try {
const repo = auth.getRepository();
if (!repo) return;
// Load albums
try {
const albumsResult = await repo.getItems(artist.id, {
includeItemTypes: ["MusicAlbum"],
limit: 50,
sortBy: "DateCreated",
sortOrder: "Descending"
});
albums = albumsResult.items.filter(item => item.kind === "album");
} catch (e) {
log.warn("Failed to load albums:", e);
} finally {
albumsLoading = false;
}
// Load top tracks
try {
const tracksResult = await repo.getItems(artist.id, {
includeItemTypes: ["Audio"],
limit: 10,
sortBy: "CommunityRating",
sortOrder: "Descending"
});
topTracks = tracksResult.items.filter(item => item.kind === "track");
} catch (e) {
log.warn("Failed to load tracks:", e);
} finally {
tracksLoading = false;
}
// Load related artists (by genre)
try {
if (artist.genres && artist.genres.length > 0) {
const relatedResult = await repo.getItems(artist.id, {
includeItemTypes: ["MusicArtist"],
genres: artist.genres.slice(0, 2),
limit: 12,
sortBy: "CommunityRating",
sortOrder: "Descending"
});
relatedArtists = relatedResult.items
.filter(item => item.id !== artist.id && item.kind === "artist")
.slice(0, 6);
}
} catch (e) {
log.warn("Failed to load related artists:", e);
} finally {
artistsLoading = false;
}
singlesLoading = false;
} catch (e) {
log.error("Error loading artist content:", e);
}
}
</script>
<div class="space-y-8">
<!-- Hero Section -->
<div class="relative">
<!-- Backdrop -->
{#if artist.backdropImageTags?.[0]}
<div class="absolute inset-0 -z-10 h-96 overflow-hidden rounded-lg">
<CachedImage
itemId={artist.id}
imageType="Backdrop"
tag={artist.backdropImageTags[0]}
maxWidth={1920}
class="w-full h-full object-cover opacity-40"
/>
<div class="absolute inset-0 bg-gradient-to-b from-transparent to-[var(--color-background)]"></div>
</div>
{/if}
<!-- Artist Info -->
<div class="flex flex-col items-center text-center py-12">
<!-- Artist Image -->
{#if artist.imageId}
<div class="mb-6 rounded-full overflow-hidden w-40 h-40 shadow-lg">
<CachedImage
itemId={artist.id}
imageType="Primary"
tag={artist.imageId}
maxWidth={400}
alt={artist.name}
class="w-full h-full object-cover"
/>
</div>
{/if}
<!-- Artist Name -->
<div class="flex items-center gap-2 mb-4">
<h1 class="text-4xl font-bold text-white">{artist.name}</h1>
<!-- TRACES: UR-068 | DR-119 -->
<FavoriteButton
itemId={artist.id}
isFavorite={resolveIsFavorite(artist, $favoriteOverrides)}
size="lg"
/>
</div>
<!-- Bio -->
{#if artist.overview}
<p class="text-gray-300 leading-relaxed max-w-3xl">
{artist.overview}
</p>
{/if}
</div>
</div>
<!-- Albums Section -->
<div class="space-y-4">
<h2 class="text-2xl font-semibold text-white">Albums</h2>
{#if albumsLoading}
<div class="grid grid-cols-2 md:grid-cols-4 lg:grid-cols-6 gap-4">
{#each Array(6) as _}
<div class="animate-pulse">
<div class="aspect-square bg-[var(--color-surface)] rounded-lg mb-2"></div>
<div class="h-4 bg-[var(--color-surface)] rounded w-3/4 mb-1"></div>
<div class="h-3 bg-[var(--color-surface)] rounded w-1/2"></div>
</div>
{/each}
</div>
{:else if albums.length === 0}
<p class="text-gray-400">No albums found</p>
{:else}
<div class="grid grid-cols-2 md:grid-cols-4 lg:grid-cols-6 gap-4">
{#each albums as album (album.id)}
<a
href="/library/{album.id}"
class="group cursor-pointer"
>
<div class="aspect-square bg-[var(--color-surface)] rounded-lg overflow-hidden mb-2 group-hover:opacity-80 transition-opacity">
{#if album.imageId}
<CachedImage
itemId={album.id}
imageType="Primary"
tag={album.imageId}
maxWidth={200}
alt={album.name}
class="w-full h-full object-cover"
/>
{/if}
</div>
<p class="text-sm font-medium text-white truncate group-hover:text-[var(--color-jellyfin)] transition-colors">
{truncateMiddle(album.name, 40)}
</p>
{#if album.productionYear}
<p class="text-xs text-gray-400">{album.productionYear}</p>
{/if}
</a>
{/each}
</div>
{/if}
</div>
<!-- Top Tracks Section -->
{#if topTracks.length > 0}
<div class="space-y-4">
<h2 class="text-2xl font-semibold text-white">Top Tracks</h2>
<TrackList
tracks={topTracks}
loading={tracksLoading}
showAlbum={true}
showArtist={false}
showDownload={false}
/>
</div>
{/if}
<!-- Related Artists Section -->
{#if relatedArtists.length > 0}
<div class="space-y-4">
<h2 class="text-2xl font-semibold text-white">Similar Artists</h2>
{#if artistsLoading}
<div class="grid grid-cols-2 md:grid-cols-4 lg:grid-cols-6 gap-4">
{#each Array(6) as _}
<div class="animate-pulse text-center">
<div class="w-32 h-32 bg-[var(--color-surface)] rounded-full mb-2 mx-auto"></div>
<div class="h-4 bg-[var(--color-surface)] rounded w-3/4 mx-auto"></div>
</div>
{/each}
</div>
{:else}
<div class="grid grid-cols-2 md:grid-cols-4 lg:grid-cols-6 gap-4">
{#each relatedArtists as relatedArtist (relatedArtist.id)}
<a
href="/library/{relatedArtist.id}"
class="group text-center"
>
<div class="w-32 h-32 bg-[var(--color-surface)] rounded-full overflow-hidden mb-2 mx-auto group-hover:opacity-80 transition-opacity">
{#if relatedArtist.imageId}
<CachedImage
itemId={relatedArtist.id}
imageType="Primary"
tag={relatedArtist.imageId}
maxWidth={200}
alt={relatedArtist.name}
class="w-full h-full object-cover"
/>
{/if}
</div>
<p class="text-sm font-medium text-white truncate group-hover:text-[var(--color-jellyfin)] transition-colors">
{relatedArtist.name}
</p>
</a>
{/each}
</div>
{/if}
</div>
{/if}
</div>