fix: further limit extraneous events from playback reporting

src/screens/Music/stacks/components/TrackListView.tsx

@@ -6,7 +6,6 @@ import { useNavigation } from '@react-navigation/native';
 import { useAppDispatch, useTypedSelector } from '@/store';
 import TouchableHandler from '@/components/TouchableHandler';
 import useCurrentTrack from '@/utility/useCurrentTrack';
-import TrackPlayer from 'react-native-track-player';
 import Play from '@/assets/icons/play.svg';
 import Shuffle from '@/assets/icons/shuffle.svg';
 import useDefaultStyles from '@/components/Colors';
@@ -150,9 +149,7 @@ const TrackListView: React.FC<TrackListViewProps> = ({
     const playEntity = useCallback(() => { playTracks(trackIds); }, [playTracks, trackIds]);
     const shuffleEntity = useCallback(() => { playTracks(trackIds, { shuffle: true }); }, [playTracks, trackIds]);
     const selectTrack = useCallback(async (index: number) => {
-        await playTracks(trackIds, { play: false });
+        await playTracks(trackIds, { playIndex: index });
-        await TrackPlayer.skip(index);
-        await TrackPlayer.play();
     }, [playTracks, trackIds]);
     const longPressTrack = useCallback((index: number) => {
         navigation.navigate('TrackPopupMenu', { trackId: trackIds[index].toString() });

src/utility/JellyfinApi/playback.ts

@@ -31,6 +31,9 @@ export async function sendPlaybackEvent(
         TrackPlayer.getPlaybackState(),
     ]);
 
+    // GUARD: Ensure that no empty events are sent out
+    if (!activeTrack?.backendId) return;
+
     // Generate a payload from the gathered data
     const payload = {
         VolumeLevel: volume * 100,
@@ -41,8 +44,8 @@ export async function sendPlaybackEvent(
         PositionTicks: Math.round((lastPosition || currentPosition) * 10_000_000),
         PlaybackRate: 1,
         PlayMethod: 'transcode',
-        MediaSourceId: activeTrack?.backendId || null,
+        MediaSourceId: activeTrack.backendId,
-        ItemId: activeTrack?.backendId || null,
+        ItemId: activeTrack.backendId,
         CanSeek: true,
         PlaybackStartTimeTicks: null,
     };

src/utility/PlaybackService.ts

@@ -40,15 +40,16 @@ export default async function() {
     TrackPlayer.addEventListener(Event.PlaybackActiveTrackChanged, async (e) => {
         // Retrieve the current settings from the Redux store
         const settings = store.getState().settings;
+        console.log('TrackChanged', e?.track?.title);
 
         // GUARD: Only report playback when the settings is enabled
         if (settings.enablePlaybackReporting && 'track' in e) {
             // GUARD: End the previous track if it's about to end
             if (e.lastTrack) {
-                await sendPlaybackEvent('/Sessions/Playing/Stopped', e.lastTrack, e.lastPosition);
+                sendPlaybackEvent('/Sessions/Playing/Stopped', e.lastTrack, e.lastPosition);
             }
 
-            await sendPlaybackEvent('/Sessions/Playing', e.track);
+            sendPlaybackEvent('/Sessions/Playing', e.track);
         }
     });
 

src/utility/usePlayTracks.ts

@@ -8,6 +8,14 @@ interface PlayOptions {
     play: boolean;
     shuffle: boolean;
     method: 'add-to-end' | 'add-after-currently-playing' | 'replace';
+    /** 
+     * The index for the track that should start out playing. This ensures that
+     * no intermediate tracks are played (however briefly) while the queue skips
+     * to this index.
+     * 
+     * NOTE: This option is only available with the `replace` method. 
+     */
+    playIndex?: number;
 }
 
 const defaults: PlayOptions = {
@@ -103,11 +111,29 @@ export default function usePlayTracks() {
                 break;
             }
             case 'replace': {
+                // Reset the queue first
                 await TrackPlayer.reset();
-                await TrackPlayer.add(newTracks);
 
-                if (play) {
+                // GUARD: Check if we need to skip to a particular index
-                    await TrackPlayer.play();
+                if (options.playIndex) {
+                    // If so, we'll split the tracks into tracks before the
+                    // index that should be played, and the queue of tracks that
+                    // will start playing
+                    const before = newTracks.slice(0, options.playIndex);
+                    const current = newTracks.slice(options.playIndex);
+
+                    // First, we'll add the current queue and (optionally) force
+                    // it to start playing.
+                    await TrackPlayer.add(current);
+                    if (play) await TrackPlayer.play();
+
+                    // Then, we'll insert the "previous" tracks after the queue
+                    // has started playing. This ensures that these tracks won't
+                    // trigger any events on the track player.
+                    await TrackPlayer.add(before, options.playIndex);
+                } else {
+                    await TrackPlayer.add(newTracks);
+                    if (play) await TrackPlayer.play();
                 }
 
                 break;