refactor(router): improve routing logic

- Updated redirect logic
- Improved navigation flow

Author: fulleni

lib/authentication/bloc/authentication_bloc.dart

@@ -20,7 +20,8 @@ class AuthenticationBloc
         super(AuthenticationInitial()) {
     on<AuthenticationUserChanged>(_onAuthenticationUserChanged);
     on<AuthenticationEmailSignInRequested>(
-        _onAuthenticationEmailSignInRequested,);
+      _onAuthenticationEmailSignInRequested,
+    );
     on<AuthenticationGoogleSignInRequested>(
       _onAuthenticationGoogleSignInRequested,
     );

lib/authentication/view/authentication_page.dart

@@ -22,6 +22,8 @@ class _AuthenticationView extends StatelessWidget {
 
   @override
   Widget build(BuildContext context) {
-    return const Placeholder(child: Text('AUTHENTICATION PAGE'),);
+    return const Placeholder(
+      child: Text('AUTHENTICATION PAGE'),
+    );
   }
 }

lib/headlines-feed/view/headlines_feed_page.dart

@@ -88,7 +88,7 @@ class _HeadlinesFeedViewState extends State<_HeadlinesFeedView> {
           IconButton(
             icon: const Icon(Icons.search),
             onPressed: () {
-              context.pushNamed(Routes.searchName);
+              context.goNamed(Routes.searchName);
             },
           ),
           BlocBuilder<HeadlinesFeedBloc, HeadlinesFeedState>(

lib/router/router.dart

@@ -12,11 +12,25 @@ final appRouter = GoRouter(
   initialLocation: Routes.headlinesFeed,
   redirect: (BuildContext context, GoRouterState state) {
     final appStatus = context.read<AppBloc>().state.status;
+    const authenticationPath = Routes.authentication;
+    const headlinesFeedPath = Routes.headlinesFeed;
+
+    // If the user is not authenticated, redirect to the headlines feed
+    // unless they are already on a route within the headlines feed.
     if (appStatus != AppStatus.authenticated) {
-      return Routes.headlinesFeed;
-    } else {
-      return Routes.authentication;
+      if (!state.matchedLocation.startsWith(headlinesFeedPath)) {
+        return headlinesFeedPath;
+      }
+    }
+    // If the user is authenticated, redirect to the authentication page
+    // unless they are already on a route within the authentication section.
+    else {
+      if (!state.matchedLocation.startsWith(authenticationPath)) {
+        return authenticationPath;
+      }
     }
+    // Otherwise, allow the navigation to proceed.
+    return null;
   },
   routes: [
     GoRoute(