feat(app): navigate using go_router in AppBloc

- Pass context to AppNavigationIndexChanged
- Navigate using goNamed in AppBloc

Author: fulleni

lib/app/bloc/app_bloc.dart

@@ -1,23 +1,36 @@
 import 'package:bloc/bloc.dart';
 import 'package:equatable/equatable.dart';
 import 'package:flutter/material.dart';
+import 'package:go_router/go_router.dart';
+import 'package:ht_main/router/routes.dart';
 
 part 'app_event.dart';
 part 'app_state.dart';
 
 class AppBloc extends Bloc<AppEvent, AppState> {
   AppBloc() : super(const AppState()) {
-    on<AppNavigationIndexChanged>((event, emit) {
-      emit(state.copyWith(selectedBottomNavigationIndex: event.index));
-    });
-    on<AppThemeChanged>((event, emit) {
-      emit(
-        state.copyWith(
-          themeMode: state.themeMode == ThemeMode.light
-              ? ThemeMode.dark
-              : ThemeMode.light,
-        ),
-      );
-    });
+    on<AppNavigationIndexChanged>(_onAppNavigationIndexChanged);
+    on<AppThemeChanged>(_onAppThemeChanged);
+  }
+
+  void _onAppNavigationIndexChanged(
+    AppNavigationIndexChanged event,
+    Emitter<AppState> emit,
+  ) {
+    emit(state.copyWith(selectedBottomNavigationIndex: event.index));
+    event.context.goNamed(Routes.getRouteNameByIndex(event.index));
+  }
+
+  void _onAppThemeChanged(
+    AppThemeChanged event,
+    Emitter<AppState> emit,
+  ) {
+    emit(
+      state.copyWith(
+        themeMode: state.themeMode == ThemeMode.light
+            ? ThemeMode.dark
+            : ThemeMode.light,
+      ),
+    );
   }
 }

lib/app/bloc/app_event.dart

@@ -4,9 +4,10 @@ part of 'app_bloc.dart';
 sealed class AppEvent {}
 
 final class AppNavigationIndexChanged extends AppEvent {
-  AppNavigationIndexChanged({required this.index});
+  AppNavigationIndexChanged({required this.index, required this.context});
 
   final int index;
+  final BuildContext context;
 }
 
 final class AppThemeChanged extends AppEvent {}

lib/app/view/app_scaffold.dart

@@ -24,9 +24,9 @@ class AppScaffold extends StatelessWidget {
             largeSecondaryBody: AdaptiveScaffold.emptyBuilder,
             selectedIndex: state.selectedBottomNavigationIndex,
             onSelectedIndexChange: (index) {
-              context
-                  .read<AppBloc>()
-                  .add(AppNavigationIndexChanged(index: index));
+              context.read<AppBloc>().add(
+                    AppNavigationIndexChanged(index: index, context: context),
+                  );
             },
             destinations: const [
               NavigationDestination(

lib/router/routes.dart

@@ -8,4 +8,17 @@ abstract final class Routes {
   static const account = '/account';
   static const accountName = 'account';
   static const articleDetailsName = 'articleDetails'; // For the sub-route
+
+  static String getRouteNameByIndex(int index) {
+    switch (index) {
+      case 0:
+        return headlinesFeedName;
+      case 1:
+        return searchName;
+      case 2:
+        return accountName;
+      default:
+        throw ArgumentError('Invalid index: $index');
+    }
+  }
 }

memory-bank/.clinerules

@@ -22,3 +22,4 @@
 - Explicitly document no-op operations with a comment explaining the reason.
 - Document exceptions associated with calling a function in its documentation comments.
 - Define descriptive exceptions by implementing `Exception` with descriptive names, rather than throwing generic `Exception`.
+- When using `go_router` for navigation, `BuildContext` is required for methods like `context.goNamed()`. Since BLoCs don't have inherent access to `BuildContext`, it needs to be passed through the event triggering the navigation.

pubspec.yaml

@@ -1,6 +1,6 @@
 name: ht_main
 description: main headlines toolkit mobile app.
-version: 0.15.1
+version: 0.17.0
 publish_to: none
 repository: https://github.com/headlines-toolkit/ht-main
 environment: