Several bugfixes: (#23)

* Fix player auto-expanding again the first time you try and collapse it
* Fix tapping the notification bringing you to a "not found" screen
* Fix being able to navigate to the same screen over and over, causing a confusing navigation experience

src/app/(tabs)/(library)/_layout.tsx

@@ -1,6 +1,9 @@
 import { useSessionStore } from "@/src/stores/session";
 import { Redirect, Stack } from "expo-router";
 
+const getId = ({ params }: { params?: Record<string, any> | undefined }) =>
+  params?.id;
+
 export default function AppLayout() {
   const session = useSessionStore((state) => state.session);
 
@@ -11,10 +14,10 @@ export default function AppLayout() {
   return (
     <Stack>
       <Stack.Screen name="index" options={{ title: "Library" }} />
-      <Stack.Screen name="media/[id]" options={{ title: "" }} />
-      <Stack.Screen name="person/[id]" options={{ title: "" }} />
-      <Stack.Screen name="series/[id]" options={{ title: "" }} />
-      <Stack.Screen name="book/[id]" options={{ title: "" }} />
+      <Stack.Screen name="media/[id]" getId={getId} />
+      <Stack.Screen name="person/[id]" getId={getId} />
+      <Stack.Screen name="series/[id]" getId={getId} />
+      <Stack.Screen name="book/[id]" getId={getId} />
     </Stack>
   );
 }

src/app/(tabs)/(library)/media/[id].tsx

@@ -777,7 +777,7 @@ function OtherEditions({
     <View className="mt-8">
       <Pressable
         onPress={() => {
-          router.push({
+          router.navigate({
             pathname: "/book/[id]",
             params: { id: media[0].book.id },
           });
@@ -881,7 +881,7 @@ function OtherBooksInSeries({
     <View className="mt-8">
       <Pressable
         onPress={() => {
-          router.push({
+          router.navigate({
             pathname: "/series/[id]",
             params: { id: series.id },
           });
@@ -1038,7 +1038,7 @@ function OtherBooksByAuthor({
     <View className="mt-8">
       <Pressable
         onPress={() => {
-          router.push({
+          router.navigate({
             pathname: "/person/[id]",
             params: { id: author.person.id },
           });
@@ -1212,7 +1212,7 @@ function OtherMediaByNarrator({
     <View className="mt-8">
       <Pressable
         onPress={() => {
-          router.push({
+          router.navigate({
             pathname: "/person/[id]",
             params: { id: narrator.person.id },
           });

src/app/(tabs)/downloads.tsx

@@ -82,7 +82,7 @@ function DownloadRow({ session, download }: DownloadRowProps) {
   const [isModalVisible, setIsModalVisible] = useState(false);
 
   const navigateToBook = () => {
-    router.push({
+    router.navigate({
       pathname: "/media/[id]",
       params: {
         id: download.media.id,

src/app/+native-intent.tsx

@@ -0,0 +1,15 @@
+import { useTrackPlayerStore } from "@/src/stores/trackPlayer";
+
+type PathArgs = {
+  path: string;
+  initial: boolean;
+};
+
+export function redirectSystemPath({ path }: PathArgs) {
+  if (path === "trackplayer://notification.click") {
+    useTrackPlayerStore.getState().requestExpandPlayer();
+    return null;
+  } else {
+    return path;
+  }
+}

src/app/sign-in.tsx

@@ -6,7 +6,7 @@ import { useState } from "react";
 import { Button, Text, TextInput, View } from "react-native";
 import colors from "tailwindcss/colors";
 
-export default function SignIn() {
+export default function SignInScreen() {
   const { session, error, isLoading, signIn, clearError } = useSessionStore(
     (state) => state,
   );

src/components/TabBarWithPlayer.tsx

@@ -62,8 +62,8 @@ export default function TabBarWithPlayer({
   useEffect(() => {
     if (!expanded && lastPlayerExpandRequest) {
       expandLocal();
-      expandPlayerHandled();
     }
+    expandPlayerHandled();
   }, [expandLocal, expanded, lastPlayerExpandRequest, expandPlayerHandled]);
 
   const tabBarHeight = 50 + insets.bottom;
@@ -333,7 +333,7 @@ export default function TabBarWithPlayer({
                     } else {
                       collapseLocal();
                       setTimeout(() => {
-                        router.push({
+                        router.navigate({
                           pathname: "/media/[id]",
                           params: { id: media.id, title: media.book.title },
                         });
@@ -400,7 +400,7 @@ export default function TabBarWithPlayer({
                   onPress={() => {
                     collapseLocal();
                     setTimeout(() => {
-                      router.push({
+                      router.navigate({
                         pathname: "/media/[id]",
                         params: { id: media.id, title: media.book.title },
                       });

src/components/Tiles.tsx

@@ -94,12 +94,12 @@ export function Tile({ book, media, seriesBook, style }: TileProps) {
 
   const navigateToBook = () => {
     if (media.length === 1) {
-      router.push({
+      router.navigate({
         pathname: "/media/[id]",
         params: { id: media[0].id, title: book.title },
       });
     } else {
-      router.push({
+      router.navigate({
         pathname: "/book/[id]",
         params: { id: book.id, title: book.title },
       });
@@ -148,7 +148,7 @@ export function PersonTile(props: PersonTileProps) {
   const router = useRouter();
 
   const navigateToPerson = () => {
-    router.push({
+    router.navigate({
       pathname: "/person/[id]",
       params: { id: personId, title: realName },
     });