os lyrics

This commit is contained in:
Kendall Garner 2024-02-01 23:53:10 -08:00
parent 9e4664a54c
commit 73cd647486
No known key found for this signature in database
GPG key ID: 18D2767419676C87
9 changed files with 182 additions and 46 deletions

View file

@ -50,6 +50,8 @@ import type {
LyricsResponse, LyricsResponse,
ServerInfo, ServerInfo,
ServerInfoArgs, ServerInfoArgs,
StructuredLyricsArgs,
StructuredLyric,
} from '/@/renderer/api/types'; } from '/@/renderer/api/types';
import { ServerType } from '/@/renderer/types'; import { ServerType } from '/@/renderer/types';
import { DeletePlaylistResponse, RandomSongListArgs } from './types'; import { DeletePlaylistResponse, RandomSongListArgs } from './types';
@ -90,6 +92,7 @@ export type ControllerEndpoint = Partial<{
getServerInfo: (args: ServerInfoArgs) => Promise<ServerInfo>; getServerInfo: (args: ServerInfoArgs) => Promise<ServerInfo>;
getSongDetail: (args: SongDetailArgs) => Promise<SongDetailResponse>; getSongDetail: (args: SongDetailArgs) => Promise<SongDetailResponse>;
getSongList: (args: SongListArgs) => Promise<SongListResponse>; getSongList: (args: SongListArgs) => Promise<SongListResponse>;
getStructuredLyrics: (args: StructuredLyricsArgs) => Promise<StructuredLyric[]>;
getTopSongs: (args: TopSongListArgs) => Promise<TopSongListResponse>; getTopSongs: (args: TopSongListArgs) => Promise<TopSongListResponse>;
getUserList: (args: UserListArgs) => Promise<UserListResponse>; getUserList: (args: UserListArgs) => Promise<UserListResponse>;
removeFromPlaylist: (args: RemoveFromPlaylistArgs) => Promise<RemoveFromPlaylistResponse>; removeFromPlaylist: (args: RemoveFromPlaylistArgs) => Promise<RemoveFromPlaylistResponse>;
@ -135,6 +138,7 @@ const endpoints: ApiController = {
getServerInfo: jfController.getServerInfo, getServerInfo: jfController.getServerInfo,
getSongDetail: jfController.getSongDetail, getSongDetail: jfController.getSongDetail,
getSongList: jfController.getSongList, getSongList: jfController.getSongList,
getStructuredLyrics: undefined,
getTopSongs: jfController.getTopSongList, getTopSongs: jfController.getTopSongList,
getUserList: undefined, getUserList: undefined,
removeFromPlaylist: jfController.removeFromPlaylist, removeFromPlaylist: jfController.removeFromPlaylist,
@ -172,6 +176,7 @@ const endpoints: ApiController = {
getServerInfo: ssController.getServerInfo, getServerInfo: ssController.getServerInfo,
getSongDetail: ndController.getSongDetail, getSongDetail: ndController.getSongDetail,
getSongList: ndController.getSongList, getSongList: ndController.getSongList,
getStructuredLyrics: ssController.getStructuredLyrics,
getTopSongs: ssController.getTopSongList, getTopSongs: ssController.getTopSongList,
getUserList: ndController.getUserList, getUserList: ndController.getUserList,
removeFromPlaylist: ndController.removeFromPlaylist, removeFromPlaylist: ndController.removeFromPlaylist,
@ -206,6 +211,7 @@ const endpoints: ApiController = {
getServerInfo: ssController.getServerInfo, getServerInfo: ssController.getServerInfo,
getSongDetail: undefined, getSongDetail: undefined,
getSongList: undefined, getSongList: undefined,
getStructuredLyrics: ssController.getStructuredLyrics,
getTopSongs: ssController.getTopSongList, getTopSongs: ssController.getTopSongList,
getUserList: undefined, getUserList: undefined,
scrobble: ssController.scrobble, scrobble: ssController.scrobble,
@ -496,6 +502,15 @@ const getServerInfo = async (args: ServerInfoArgs) => {
)?.(args); )?.(args);
}; };
const getStructuredLyrics = async (args: StructuredLyricsArgs) => {
return (
apiController(
'getStructuredLyrics',
args.apiClientProps.server?.type,
) as ControllerEndpoint['getStructuredLyrics']
)?.(args);
};
export const controller = { export const controller = {
addToPlaylist, addToPlaylist,
authenticate, authenticate,
@ -518,6 +533,7 @@ export const controller = {
getServerInfo, getServerInfo,
getSongDetail, getSongDetail,
getSongList, getSongList,
getStructuredLyrics,
getTopSongList, getTopSongList,
getUserList, getUserList,
removeFromPlaylist, removeFromPlaylist,

View file

@ -57,6 +57,14 @@ export const contract = c.router({
200: ssType._response.serverInfo, 200: ssType._response.serverInfo,
}, },
}, },
getStructuredLyrics: {
method: 'GET',
path: 'getLyricsBySongId.view',
query: ssType._parameters.structuredLyrics,
responses: {
200: ssType._response.structuredLyrics,
},
},
getTopSongsList: { getTopSongsList: {
method: 'GET', method: 'GET',
path: 'getTopSongs.view', path: 'getTopSongs.view',

View file

@ -23,6 +23,8 @@ import {
RandomSongListArgs, RandomSongListArgs,
ServerInfo, ServerInfo,
ServerInfoArgs, ServerInfoArgs,
StructuredLyricsArgs,
StructuredLyric,
} from '/@/renderer/api/types'; } from '/@/renderer/api/types';
import { randomString } from '/@/renderer/utils'; import { randomString } from '/@/renderer/utils';
@ -397,6 +399,51 @@ const getServerInfo = async (args: ServerInfoArgs): Promise<ServerInfo> => {
return { features, id: apiClientProps.server?.id, version: ping.body.serverVersion }; return { features, id: apiClientProps.server?.id, version: ping.body.serverVersion };
}; };
export const getStructuredLyrics = async (
args: StructuredLyricsArgs,
): Promise<StructuredLyric[]> => {
const { query, apiClientProps } = args;
const res = await ssApiClient(apiClientProps).getStructuredLyrics({
query: {
id: query.songId,
},
});
if (res.status !== 200) {
throw new Error('Failed to get server extensions');
}
const lyrics = res.body.lyricsList?.structuredLyrics;
if (!lyrics) {
return [];
}
return lyrics.map((lyric) => {
const baseLyric = {
artist: lyric.displayArtist || '',
lang: lyric.lang,
name: lyric.displayTitle || '',
remote: false,
source: apiClientProps.server?.name || 'music server',
};
if (lyric.synced) {
return {
...baseLyric,
lyrics: lyric.line.map((line) => [line.start!, line.value]),
synced: true,
};
}
return {
...baseLyric,
lyrics: lyric.line.map((line) => [line.value]).join('\n'),
synced: false,
};
});
};
export const ssController = { export const ssController = {
authenticate, authenticate,
createFavorite, createFavorite,
@ -404,6 +451,7 @@ export const ssController = {
getMusicFolderList, getMusicFolderList,
getRandomSongList, getRandomSongList,
getServerInfo, getServerInfo,
getStructuredLyrics,
getTopSongList, getTopSongList,
removeFavorite, removeFavorite,
scrobble, scrobble,

View file

@ -221,6 +221,32 @@ const serverInfo = z.object({
openSubsonicExtensions: z.array(extension), openSubsonicExtensions: z.array(extension),
}); });
const structuredLyricsParameters = z.object({
id: z.string(),
});
const lyricLine = z.object({
start: z.number().optional(),
value: z.string(),
});
const structuredLyric = z.object({
displayArtist: z.string().optional(),
displayTitle: z.string().optional(),
lang: z.string(),
line: z.array(lyricLine),
offset: z.number().optional(),
synced: z.boolean(),
});
const structuredLyrics = z.object({
lyricsList: z
.object({
structuredLyrics: z.array(structuredLyric).optional(),
})
.optional(),
});
export const ssType = { export const ssType = {
_parameters: { _parameters: {
albumList: albumListParameters, albumList: albumListParameters,
@ -232,6 +258,7 @@ export const ssType = {
scrobble: scrobbleParameters, scrobble: scrobbleParameters,
search3: search3Parameters, search3: search3Parameters,
setRating: setRatingParameters, setRating: setRatingParameters,
structuredLyrics: structuredLyricsParameters,
topSongsList: topSongsListParameters, topSongsList: topSongsListParameters,
}, },
_response: { _response: {
@ -252,6 +279,7 @@ export const ssType = {
serverInfo, serverInfo,
setRating, setRating,
song, song,
structuredLyrics,
topSongsList, topSongsList,
}, },
}; };

View file

@ -1092,17 +1092,11 @@ export type InternetProviderLyricSearchResponse = {
source: LyricSource; source: LyricSource;
}; };
export type SynchronizedLyricMetadata = { export type FullLyricsMetadata = {
lyrics: SynchronizedLyricsArray; lyrics: LyricsResponse;
remote: boolean; remote: boolean;
} & Omit<InternetProviderLyricResponse, 'lyrics'>; source: string;
} & Omit<InternetProviderLyricResponse, 'id' | 'lyrics' | 'source'>;
export type UnsynchronizedLyricMetadata = {
lyrics: string;
remote: boolean;
} & Omit<InternetProviderLyricResponse, 'lyrics'>;
export type FullLyricsMetadata = SynchronizedLyricMetadata | UnsynchronizedLyricMetadata;
export type LyricOverride = Omit<InternetProviderLyricResponse, 'lyrics'>; export type LyricOverride = Omit<InternetProviderLyricResponse, 'lyrics'>;
@ -1153,3 +1147,21 @@ export type ServerInfo = {
id?: string; id?: string;
version: string; version: string;
}; };
export type StructuredLyricsArgs = {
query: LyricsQuery;
} & BaseEndpointArgs;
export type StructuredUnsyncedLyric = {
lyrics: string;
synced: false;
} & Omit<FullLyricsMetadata, 'lyrics'>;
export type StructuredSyncedLyric = {
lyrics: SynchronizedLyricsArray;
synced: true;
} & Omit<FullLyricsMetadata, 'lyrics'>;
export type StructuredLyric = {
lang: string;
} & (StructuredUnsyncedLyric | StructuredSyncedLyric);

View file

@ -1,21 +1,19 @@
import { useCallback, useEffect, useState } from 'react'; import { useCallback, useEffect, useMemo, useState } from 'react';
import { Center, Group } from '@mantine/core'; import { Center, Group } from '@mantine/core';
import { AnimatePresence, motion } from 'framer-motion'; import { AnimatePresence, motion } from 'framer-motion';
import { ErrorBoundary } from 'react-error-boundary'; import { ErrorBoundary } from 'react-error-boundary';
import { RiInformationFill } from 'react-icons/ri'; import { RiInformationFill } from 'react-icons/ri';
import styled from 'styled-components'; import styled from 'styled-components';
import { useSongLyricsByRemoteId, useSongLyricsBySong } from './queries/lyric-query'; import { useSongLyricsByRemoteId, useSongLyricsBySong } from './queries/lyric-query';
import { SynchronizedLyrics } from './synchronized-lyrics'; import { SynchronizedLyrics, SynchronizedLyricsProps } from './synchronized-lyrics';
import { Spinner, TextTitle } from '/@/renderer/components'; import { Select, Spinner, TextTitle } from '/@/renderer/components';
import { ErrorFallback } from '/@/renderer/features/action-required'; import { ErrorFallback } from '/@/renderer/features/action-required';
import { UnsynchronizedLyrics } from '/@/renderer/features/lyrics/unsynchronized-lyrics';
import { useCurrentSong, usePlayerStore } from '/@/renderer/store';
import { import {
FullLyricsMetadata, UnsynchronizedLyrics,
LyricsOverride, UnsynchronizedLyricsProps,
SynchronizedLyricMetadata, } from '/@/renderer/features/lyrics/unsynchronized-lyrics';
UnsynchronizedLyricMetadata, import { useCurrentSong, usePlayerStore } from '/@/renderer/store';
} from '/@/renderer/api/types'; import { FullLyricsMetadata, LyricSource, LyricsOverride } from '/@/renderer/api/types';
import { LyricsActions } from '/@/renderer/features/lyrics/lyrics-actions'; import { LyricsActions } from '/@/renderer/features/lyrics/lyrics-actions';
import { queryKeys } from '/@/renderer/api/query-keys'; import { queryKeys } from '/@/renderer/api/query-keys';
import { queryClient } from '/@/renderer/lib/react-query'; import { queryClient } from '/@/renderer/lib/react-query';
@ -84,17 +82,9 @@ const ScrollContainer = styled(motion.div)`
} }
`; `;
function isSynchronized(
data: Partial<FullLyricsMetadata> | undefined,
): data is SynchronizedLyricMetadata {
// Type magic. The only difference between Synchronized and Unsynchhronized is
// the datatype of lyrics. This makes Typescript happier later...
if (!data) return false;
return Array.isArray(data.lyrics);
}
export const Lyrics = () => { export const Lyrics = () => {
const currentSong = useCurrentSong(); const currentSong = useCurrentSong();
const [index, setIndex] = useState(0);
const { data, isInitialLoading } = useSongLyricsBySong( const { data, isInitialLoading } = useSongLyricsBySong(
{ {
@ -139,7 +129,7 @@ export const Lyrics = () => {
}, },
query: { query: {
remoteSongId: override?.id, remoteSongId: override?.id,
remoteSource: override?.source, remoteSource: override?.source as LyricSource | undefined,
song: currentSong, song: currentSong,
}, },
serverId: currentSong?.serverId, serverId: currentSong?.serverId,
@ -150,6 +140,7 @@ export const Lyrics = () => {
(state) => state.current.song, (state) => state.current.song,
() => { () => {
setOverride(undefined); setOverride(undefined);
setIndex(0);
}, },
{ equalityFn: (a, b) => a?.id === b?.id }, { equalityFn: (a, b) => a?.id === b?.id },
); );
@ -159,16 +150,29 @@ export const Lyrics = () => {
}; };
}, []); }, []);
const [lyrics, synced] = useMemo(() => {
if (Array.isArray(data)) {
if (data.length > 0) {
const selectedLyric = data[Math.min(index, data.length)];
return [selectedLyric, selectedLyric.synced];
}
} else if (data?.lyrics) {
return [data, Array.isArray(data.lyrics)];
}
return [undefined, false];
}, [data, index]);
const languages = useMemo(() => {
if (Array.isArray(data)) {
return data.map((lyric, idx) => ({ label: lyric.lang, value: idx.toString() }));
}
return [];
}, [data]);
const isLoadingLyrics = isInitialLoading || isOverrideLoading; const isLoadingLyrics = isInitialLoading || isOverrideLoading;
const hasNoLyrics = !data?.lyrics; const hasNoLyrics = !lyrics;
const lyricsMetadata:
| Partial<SynchronizedLyricMetadata>
| Partial<UnsynchronizedLyricMetadata>
| undefined = data;
const isSynchronizedLyrics = isSynchronized(lyricsMetadata);
return ( return (
<ErrorBoundary FallbackComponent={ErrorFallback}> <ErrorBoundary FallbackComponent={ErrorFallback}>
@ -198,11 +202,11 @@ export const Lyrics = () => {
initial={{ opacity: 0 }} initial={{ opacity: 0 }}
transition={{ duration: 0.5 }} transition={{ duration: 0.5 }}
> >
{isSynchronizedLyrics ? ( {synced ? (
<SynchronizedLyrics {...lyricsMetadata} /> <SynchronizedLyrics {...(lyrics as SynchronizedLyricsProps)} />
) : ( ) : (
<UnsynchronizedLyrics <UnsynchronizedLyrics
{...(lyricsMetadata as UnsynchronizedLyricMetadata)} {...(lyrics as UnsynchronizedLyricsProps)}
/> />
)} )}
</ScrollContainer> </ScrollContainer>
@ -210,6 +214,16 @@ export const Lyrics = () => {
</AnimatePresence> </AnimatePresence>
)} )}
<ActionsContainer> <ActionsContainer>
{languages.length > 1 && (
<Select
clearable={false}
data={languages}
style={{ bottom: 50, position: 'absolute' }}
value={index.toString()}
onChange={(value) => setIndex(parseInt(value!, 10))}
/>
)}
<LyricsActions <LyricsActions
onRemoveLyric={handleOnRemoveLyric} onRemoveLyric={handleOnRemoveLyric}
onResetLyric={handleOnResetLyric} onResetLyric={handleOnResetLyric}

View file

@ -7,6 +7,7 @@ import {
FullLyricsMetadata, FullLyricsMetadata,
LyricGetQuery, LyricGetQuery,
SubsonicExtensions, SubsonicExtensions,
StructuredLyric,
} from '/@/renderer/api/types'; } from '/@/renderer/api/types';
import { QueryHookArgs } from '/@/renderer/lib/react-query'; import { QueryHookArgs } from '/@/renderer/lib/react-query';
import { getServerById, useLyricsSettings } from '/@/renderer/store'; import { getServerById, useLyricsSettings } from '/@/renderer/store';
@ -81,7 +82,7 @@ export const useServerLyrics = (
export const useSongLyricsBySong = ( export const useSongLyricsBySong = (
args: QueryHookArgs<LyricsQuery>, args: QueryHookArgs<LyricsQuery>,
song: QueueSong | undefined, song: QueueSong | undefined,
): UseQueryResult<FullLyricsMetadata> => { ): UseQueryResult<FullLyricsMetadata | StructuredLyric[]> => {
const { query } = args; const { query } = args;
const { fetch } = useLyricsSettings(); const { fetch } = useLyricsSettings();
const server = getServerById(song?.serverId); const server = getServerById(song?.serverId);
@ -90,7 +91,7 @@ export const useSongLyricsBySong = (
cacheTime: Infinity, cacheTime: Infinity,
enabled: !!song && !!server, enabled: !!song && !!server,
onError: () => {}, onError: () => {},
queryFn: async ({ signal }) => { queryFn: async ({ signal }): Promise<FullLyricsMetadata | StructuredLyric[] | null> => {
if (!server) throw new Error('Server not found'); if (!server) throw new Error('Server not found');
if (!song) return null; if (!song) return null;
@ -112,7 +113,16 @@ export const useSongLyricsBySong = (
}; };
} }
} else if (server.features && SubsonicExtensions.SONG_LYRICS in server.features) { } else if (server.features && SubsonicExtensions.SONG_LYRICS in server.features) {
console.log(1234); const subsonicLyrics = await api.controller
.getStructuredLyrics({
apiClientProps: { server, signal },
query: { songId: song.id },
})
.catch(console.error);
if (subsonicLyrics) {
return subsonicLyrics;
}
} else if (song.lyrics) { } else if (song.lyrics) {
return { return {
artist: song.artists?.[0]?.name, artist: song.artists?.[0]?.name,

View file

@ -46,7 +46,7 @@ const SynchronizedLyricsContainer = styled.div<{ $gap: number }>`
} }
`; `;
interface SynchronizedLyricsProps extends Omit<FullLyricsMetadata, 'lyrics'> { export interface SynchronizedLyricsProps extends Omit<FullLyricsMetadata, 'lyrics'> {
lyrics: SynchronizedLyricsArray; lyrics: SynchronizedLyricsArray;
} }

View file

@ -4,7 +4,7 @@ import { LyricLine } from '/@/renderer/features/lyrics/lyric-line';
import { FullLyricsMetadata } from '/@/renderer/api/types'; import { FullLyricsMetadata } from '/@/renderer/api/types';
import { useLyricsSettings } from '/@/renderer/store'; import { useLyricsSettings } from '/@/renderer/store';
interface UnsynchronizedLyricsProps extends Omit<FullLyricsMetadata, 'lyrics'> { export interface UnsynchronizedLyricsProps extends Omit<FullLyricsMetadata, 'lyrics'> {
lyrics: string; lyrics: string;
} }