refactor Genre domain type to include serverType and serverId

src/renderer/api/navidrome/navidrome-controller.ts

@@ -370,7 +370,7 @@ export const NavidromeController: InternalControllerEndpoint = {
         }
 
         return {
-            items: res.body.data.map((genre) => ndNormalize.genre(genre)),
+            items: res.body.data.map((genre) => ndNormalize.genre(genre, apiClientProps.server)),
             startIndex: query.startIndex || 0,
             totalRecordCount: Number(res.body.headers.get('x-total-count') || 0),
         };

src/renderer/api/subsonic/subsonic-controller.ts

@@ -647,7 +647,7 @@ export const SubsonicController: InternalControllerEndpoint = {
                 break;
         }
 
-        const genres = results.map(ssNormalize.genre);
+        const genres = results.map((genre) => ssNormalize.genre(genre, apiClientProps.server));
 
         return {
             items: genres,

src/shared/api/jellyfin/jellyfin-normalize.ts

@@ -504,6 +504,8 @@ const normalizeGenre = (
     server: null | ServerListItem,
 ): Genre => {
     return {
+        _serverId: server?.id || '',
+        _serverType: ServerType.JELLYFIN,
         albumCount: undefined,
         id: item.Id,
         imageUrl: getGenreCoverArtUrl({ baseUrl: server?.url || '', item, size: 200 }),

src/shared/api/navidrome/navidrome-normalize.ts

@@ -430,8 +430,13 @@ const normalizePlaylist = (
     };
 };
 
-const normalizeGenre = (item: z.infer<typeof ndType._response.genre>): Genre => {
+const normalizeGenre = (
+    item: z.infer<typeof ndType._response.genre>,
+    server: null | ServerListItem,
+): Genre => {
     return {
+        _serverId: server?.id || 'unknown',
+        _serverType: ServerType.NAVIDROME,
         albumCount: undefined,
         id: item.id,
         imageUrl: null,

src/shared/api/subsonic/subsonic-normalize.ts

@@ -94,9 +94,12 @@ const getGenres = (
         | z.infer<typeof ssType._response.album>
         | z.infer<typeof ssType._response.albumListEntry>
         | z.infer<typeof ssType._response.song>,
+    server?: null | ServerListItemWithCredential,
 ): Genre[] => {
     return item.genres
         ? item.genres.map((genre) => ({
+              _serverId: server?.id || 'unknown',
+              _serverType: ServerType.SUBSONIC,
               id: genre.name,
               imageUrl: null,
               itemType: LibraryItem.GENRE,
@@ -105,6 +108,8 @@ const getGenres = (
         : item.genre
           ? [
                 {
+                    _serverId: server?.id || 'unknown',
+                    _serverType: ServerType.SUBSONIC,
                     id: item.genre,
                     imageUrl: null,
                     itemType: LibraryItem.GENRE,
@@ -161,7 +166,7 @@ const normalizeSong = (
                       track: item.replayGain.trackGain,
                   }
                 : null,
-        genres: getGenres(item),
+        genres: getGenres(item, server),
         id: item.id.toString(),
         imagePlaceholderUrl: null,
         imageUrl,
@@ -260,7 +265,7 @@ const normalizeAlbum = (
                 : item.explicitStatus === 'clean'
                   ? ExplicitStatus.CLEAN
                   : null,
-        genres: getGenres(item),
+        genres: getGenres(item, server),
         id: item.id.toString(),
         imagePlaceholderUrl: null,
         imageUrl,
@@ -320,8 +325,13 @@ const normalizePlaylist = (
     };
 };
 
-const normalizeGenre = (item: z.infer<typeof ssType._response.genre>): Genre => {
+const normalizeGenre = (
+    item: z.infer<typeof ssType._response.genre>,
+    server: null | ServerListItemWithCredential,
+): Genre => {
     return {
+        _serverId: server?.id || 'unknown',
+        _serverType: ServerType.SUBSONIC,
         albumCount: item.albumCount,
         id: item.value,
         imageUrl: null,

src/shared/types/domain-types.ts

@@ -262,6 +262,8 @@ export type GainInfo = {
 };
 
 export type Genre = {
+    _serverId: string;
+    _serverType: ServerType;
     albumCount?: number;
     id: string;
     imageUrl: null | string;