refactor(router): Pass headline directly to details

- Removed HeadlineDetailsBloc
- Passed headline as extra
- Simplified article details pages
- Added error handling for missing data

Author: fulleni

lib/router/router.dart

@@ -19,7 +19,7 @@ import 'package:ht_main/authentication/bloc/authentication_bloc.dart';
 import 'package:ht_main/authentication/view/authentication_page.dart';
 import 'package:ht_main/authentication/view/email_code_verification_page.dart';
 import 'package:ht_main/authentication/view/request_code_page.dart';
-import 'package:ht_main/headline-details/bloc/headline_details_bloc.dart';
+// import 'package:ht_main/headline-details/bloc/headline_details_bloc.dart'; // Removed
 import 'package:ht_main/headline-details/view/headline_details_page.dart';
 import 'package:ht_main/headlines-feed/bloc/categories_filter_bloc.dart'; // Import new BLoC
 // import 'package:ht_main/headlines-feed/bloc/countries_filter_bloc.dart'; // Import new BLoC - REMOVED
@@ -341,15 +341,18 @@ GoRouter createRouter({
                     path: 'article/:id', // Relative path
                     name: Routes.articleDetailsName,
                     builder: (context, state) {
-                      final id = state.pathParameters['id']!;
-                      return BlocProvider(
-                        create:
-                            (context) => HeadlineDetailsBloc(
-                              headlinesRepository:
-                                  context.read<HtDataRepository<Headline>>(),
-                            )..add(HeadlineDetailsRequested(headlineId: id)),
-                        child: HeadlineDetailsPage(headlineId: id),
-                      );
+                      // final id = state.pathParameters['id']!; // No longer needed
+                      final headline = state.extra as Headline?;
+                      if (headline == null) {
+                        // Handle missing headline data, perhaps redirect or show error
+                        // For now, returning a placeholder, but a robust app
+                        // might redirect to an error page or back.
+                        return const Scaffold(
+                          body: Center(child: Text('Error: Headline data missing')),
+                        );
+                      }
+                      // No longer need BlocProvider for HeadlineDetailsBloc
+                      return HeadlineDetailsPage(headline: headline);
                     },
                   ),
                   // Sub-route for notifications (placeholder) - MOVED HERE
@@ -464,15 +467,14 @@ GoRouter createRouter({
                     path: 'article/:id', // Relative path
                     name: Routes.searchArticleDetailsName, // New route name
                     builder: (context, state) {
-                      final id = state.pathParameters['id']!;
-                      return BlocProvider(
-                        create:
-                            (context) => HeadlineDetailsBloc(
-                              headlinesRepository:
-                                  context.read<HtDataRepository<Headline>>(),
-                            )..add(HeadlineDetailsRequested(headlineId: id)),
-                        child: HeadlineDetailsPage(headlineId: id),
-                      );
+                      // final id = state.pathParameters['id']!; // No longer needed
+                      final headline = state.extra as Headline?;
+                      if (headline == null) {
+                        return const Scaffold(
+                          body: Center(child: Text('Error: Headline data missing')),
+                        );
+                      }
+                      return HeadlineDetailsPage(headline: headline);
                     },
                   ),
                 ],
@@ -615,18 +617,14 @@ GoRouter createRouter({
                         path: Routes.accountArticleDetails, // 'article/:id'
                         name: Routes.accountArticleDetailsName,
                         builder: (context, state) {
-                          final id = state.pathParameters['id']!;
-                          return BlocProvider(
-                            create:
-                                (context) => HeadlineDetailsBloc(
-                                  headlinesRepository:
-                                      context
-                                          .read<HtDataRepository<Headline>>(),
-                                )..add(
-                                  HeadlineDetailsRequested(headlineId: id),
-                                ),
-                            child: HeadlineDetailsPage(headlineId: id),
-                          );
+                          // final id = state.pathParameters['id']!; // No longer needed
+                          final headline = state.extra as Headline?;
+                          if (headline == null) {
+                            return const Scaffold(
+                              body: Center(child: Text('Error: Headline data missing')),
+                            );
+                          }
+                          return HeadlineDetailsPage(headline: headline);
                         },
                       ),
                     ],