From 67330479424c80c0768a07f70df0ae8a1fa5e1e0 Mon Sep 17 00:00:00 2001
From: Kendall Garner <17521368+kgarner7@users.noreply.github.com>
Date: Fri, 10 Oct 2025 19:32:11 -0700
Subject: [PATCH] improve jellyfin participants

---
 .../features/item-details/components/item-details-modal.tsx | 2 +-
 src/shared/api/jellyfin/jellyfin-normalize.ts               | 6 ++++++
 2 files changed, 7 insertions(+), 1 deletion(-)

diff --git a/src/renderer/features/item-details/components/item-details-modal.tsx b/src/renderer/features/item-details/components/item-details-modal.tsx
index e6bba6be..aa96d959 100644
--- a/src/renderer/features/item-details/components/item-details-modal.tsx
+++ b/src/renderer/features/item-details/components/item-details-modal.tsx
@@ -81,7 +81,7 @@ const formatArtists = (artists: null | RelatedArtist[] | undefined) =>
                     {artist.name || '—'}
                 </Text>
             ) : (
-                <Text overflow="visible" size="md">
+                <Text component="span" overflow="visible" size="md">
                     {artist.name || '-'}
                 </Text>
             )}
diff --git a/src/shared/api/jellyfin/jellyfin-normalize.ts b/src/shared/api/jellyfin/jellyfin-normalize.ts
index 5a87d981..7e692ba7 100644
--- a/src/shared/api/jellyfin/jellyfin-normalize.ts
+++ b/src/shared/api/jellyfin/jellyfin-normalize.ts
@@ -127,12 +127,18 @@ const getPlaylistCoverArtUrl = (args: { baseUrl: string; item: JFPlaylist; size:
 
 type AlbumOrSong = z.infer<typeof jfType._response.album> | z.infer<typeof jfType._response.song>;
 
+const KEYS_TO_OMIT = new Set(['AlbumArtist', 'Artist']);
+
 const getPeople = (item: AlbumOrSong): null | Record<string, RelatedArtist[]> => {
     if (item.People) {
         const participants: Record<string, RelatedArtist[]> = {};
 
         for (const person of item.People) {
             const key = person.Type || '';
+            if (KEYS_TO_OMIT.has(key)) {
+                continue;
+            }
+
             const item: RelatedArtist = {
                 // for other roles, we just want to display this and not filter.
                 // filtering (and links) would require a separate field, PersonIds