refactor: Introduce session scope for DI and remove SessionScope widget

This commit introduces a dedicated session scope for dependency injection using `injectable`. This replaces the previous `SessionScope` InheritedWidget.

Key changes:
- **`SessionModule`:** A new module is added to provide `StreamFeedsClient` within the session scope.
- **`HomeScreen`:** Now manages the session scope lifecycle. It initializes the scope in `initState` and disposes of it in `dispose`.
- **Removed `SessionScope` widget:** The `SessionScope` InheritedWidget and its associated extension have been removed.
- **Updated `UserFeedScreen`:** Now retrieves `StreamFeedsClient` and the current user directly from the DI container within the session scope.
- **`AuthGuard`:** Simplified logging.

This change centralizes session-specific dependency management and aligns with standard DI practices.

Author: xsahil03x

sample_app/lib/core/di/auth_module.dart

@@ -0,0 +1,12 @@
+import 'package:injectable/injectable.dart';
+import 'package:stream_feeds/stream_feeds.dart';
+
+import '../../app/content/auth_controller.dart';
+
+@module
+abstract class SessionModule {
+  @Singleton(scope: 'session')
+  StreamFeedsClient authenticatedFeeds(AuthController auth) {
+    return (auth.value as Authenticated).client;
+  }
+}

sample_app/lib/core/di/di_initializer.config.dart

@@ -44,3 +44,4 @@ class AppRouter extends RootStackRouter {
     ];
   }
 }
+

sample_app/lib/navigation/app_router.dart

@@ -14,12 +14,8 @@ class AuthGuard extends AutoRouteGuard {
   @override
   void onNavigation(NavigationResolver resolver, StackRouter router) {
     final isAuthenticated = _authController.value is Authenticated;
-    debugPrint('AuthGuard: isAuthenticated = $isAuthenticated');
-
     // If the user is authenticated, allow navigation to the requested route.
     if (isAuthenticated) return resolver.next();
-
-    print('AuthGuard: User is not authenticated, redirecting to login.');
     // Otherwise, redirect to the Choose user page.
     resolver.redirectUntil(const ChooseUserRoute(), replace: true);
   }

sample_app/lib/navigation/app_router.gr.dart

@@ -1,27 +1,30 @@
 import 'package:auto_route/auto_route.dart';
 import 'package:flutter/material.dart';
 
-import '../../app/content/auth_controller.dart';
+import '../../core/di/di_initializer.config.dart';
 import '../../core/di/di_initializer.dart';
-import 'session_scope.dart';
 
 @RoutePage()
-class HomeScreen extends StatelessWidget {
+class HomeScreen extends StatefulWidget {
   const HomeScreen({super.key});
 
   @override
-  Widget build(BuildContext context) {
-    final authController = locator<AuthController>();
-    final state = authController.value;
-    final (user, client) = switch (state) {
-      Authenticated(:final user, :final client) => (user, client),
-      _ => throw Exception('User not authenticated'),
-    };
+  State<HomeScreen> createState() => _HomeScreenState();
+}
 
-    return SessionScope(
-      user: user,
-      client: client,
-      child: const AutoRouter(),
-    );
+class _HomeScreenState extends State<HomeScreen> {
+  @override
+  void initState() {
+    super.initState();
+    locator.initSessionScope();
   }
+
+  @override
+  void dispose() {
+    locator.popScope();
+    super.dispose();
+  }
+
+  @override
+  Widget build(BuildContext context) => const AutoRouter();
 }

sample_app/lib/navigation/guards/auth_guard.dart

@@ -7,7 +7,6 @@ import '../../../theme/extensions/theme_extensions.dart';
 import '../../../widgets/user_avatar.dart';
 import '../../app/content/auth_controller.dart';
 import '../../core/di/di_initializer.dart';
-import '../home/session_scope.dart';
 import 'widgets/activity_comments_view.dart';
 import 'widgets/activity_content.dart';
 import 'widgets/create_activity_bottom_sheet.dart';
@@ -23,19 +22,21 @@ class UserFeedScreen extends StatefulWidget {
 }
 
 class _UserFeedScreenState extends State<UserFeedScreen> {
-  late final feed = context.client.feedFromQuery(
+  StreamFeedsClient get client => locator<StreamFeedsClient>();
+
+  late final feed = client.feedFromQuery(
     FeedQuery(
-      fid: FeedId(group: 'user', id: context.currentUser.id),
+      fid: FeedId(group: 'user', id: client.user.id),
       data: FeedInputData(
         visibility: FeedVisibility.public,
-        members: [FeedMemberRequestData(userId: context.currentUser.id)],
+        members: [FeedMemberRequestData(userId: client.user.id)],
       ),
     ),
   );
 
   @override
-  void didChangeDependencies() {
-    super.didChangeDependencies();
+  void initState() {
+    super.initState();
     feed.getOrCreate();
   }
 
@@ -52,6 +53,8 @@ class _UserFeedScreenState extends State<UserFeedScreen> {
 
   @override
   Widget build(BuildContext context) {
+    final currentUser = client.user;
+
     final wideScreen = MediaQuery.sizeOf(context).width > 600;
 
     return StateNotifierBuilder(
@@ -100,7 +103,7 @@ class _UserFeedScreenState extends State<UserFeedScreen> {
                             text: baseActivity.text ?? '',
                             attachments: baseActivity.attachments,
                             data: activity,
-                            currentUserId: context.currentUser.id,
+                            currentUserId: currentUser.id,
                             onCommentClick: () =>
                                 _onCommentClick(context, activity),
                             onHeartClick: (isAdding) =>
@@ -123,7 +126,7 @@ class _UserFeedScreenState extends State<UserFeedScreen> {
             feedWidget,
             onLogout: _onLogout,
             onProfileTap: () {
-              _showProfileBottomSheet(context, context.client, feed);
+              _showProfileBottomSheet(context, client, feed);
             },
           );
         }
@@ -134,7 +137,7 @@ class _UserFeedScreenState extends State<UserFeedScreen> {
             children: [
               SizedBox(
                 width: 250,
-                child: UserProfileView(feedsClient: context.client, feed: feed),
+                child: UserProfileView(feedsClient: client, feed: feed),
               ),
               const SizedBox(width: 16),
               Expanded(child: feedWidget),
@@ -157,7 +160,7 @@ class _UserFeedScreenState extends State<UserFeedScreen> {
         leading: GestureDetector(
           onTap: onProfileTap,
           child: Center(
-            child: UserAvatar.appBar(user: context.currentUser),
+            child: UserAvatar.appBar(user: client.user),
           ),
         ),
         title: Text(
@@ -202,7 +205,7 @@ class _UserFeedScreenState extends State<UserFeedScreen> {
       builder: (context) => ActivityCommentsView(
         activityId: activity.id,
         feed: feed,
-        client: context.client,
+        client: client,
       ),
     );
   }
@@ -227,7 +230,7 @@ class _UserFeedScreenState extends State<UserFeedScreen> {
       isScrollControlled: true,
       backgroundColor: Colors.transparent,
       builder: (context) => CreateActivityBottomSheet(
-        currentUser: context.currentUser,
+        currentUser: client.user,
         feedId: feed.query.fid,
       ),
     );