refactor: pass environment to AccountBloc

- Pass environment to AccountBloc
- Add delay in demo mode

Author: fulleni

lib/account/bloc/account_bloc.dart

@@ -4,18 +4,22 @@ import 'package:bloc/bloc.dart';
 import 'package:equatable/equatable.dart';
 import 'package:ht_auth_repository/ht_auth_repository.dart';
 import 'package:ht_data_repository/ht_data_repository.dart';
+import 'package:ht_main/app/config/config.dart' as local_config;
 import 'package:ht_shared/ht_shared.dart';
 
 part 'account_event.dart';
 part 'account_state.dart';
 
+
 class AccountBloc extends Bloc<AccountEvent, AccountState> {
   AccountBloc({
     required HtAuthRepository authenticationRepository,
     required HtDataRepository<UserContentPreferences>
     userContentPreferencesRepository,
+    required local_config.AppEnvironment environment, // Added environment
   }) : _authenticationRepository = authenticationRepository,
        _userContentPreferencesRepository = userContentPreferencesRepository,
+       _environment = environment, // Initialize environment
        super(const AccountState()) {
     // Listen to user changes from HtAuthRepository
     _userSubscription = _authenticationRepository.authStateChanges.listen((
@@ -37,6 +41,7 @@ class AccountBloc extends Bloc<AccountEvent, AccountState> {
   final HtAuthRepository _authenticationRepository;
   final HtDataRepository<UserContentPreferences>
   _userContentPreferencesRepository;
+  final local_config.AppEnvironment _environment; // New field for environment
   late StreamSubscription<User?> _userSubscription;
 
   Future<void> _onAccountUserChanged(
@@ -72,6 +77,17 @@ class AccountBloc extends Bloc<AccountEvent, AccountState> {
         ),
       );
     } on NotFoundException {
+      // In demo mode, a short delay is introduced here to mitigate a race
+      // condition during anonymous to authenticated data migration.
+      // This ensures that the DemoDataMigrationService has a chance to
+      // complete its migration of UserContentPreferences before AccountBloc
+      // attempts to create a new default preference for the authenticated user.
+      // This is a temporary stub for the demo environment only and is not
+      // needed in production/development where backend handles migration.
+      if (_environment == local_config.AppEnvironment.demo) {
+        // ignore: inference_failure_on_instance_creation
+        await Future.delayed(const Duration(seconds: 1));
+      }
       // If preferences not found, create a default one for the user
       final defaultPreferences = UserContentPreferences(id: event.userId);
       try {

lib/app/view/app.dart

@@ -101,13 +101,14 @@ class App extends StatelessWidget {
           htUserContentPreferencesRepository:
               _htUserContentPreferencesRepository,
           htAppConfigRepository: _htAppConfigRepository,
-          environment: _environment,
+          environment: _environment, // Pass environment
         ),
       ),
     );
   }
 }
 
+
 class _AppView extends StatefulWidget {
   const _AppView({
     required this.htAuthenticationRepository,
@@ -159,6 +160,7 @@ class _AppViewState extends State<_AppView> {
       htUserContentPreferencesRepository:
           widget.htUserContentPreferencesRepository,
       htAppConfigRepository: widget.htAppConfigRepository,
+      environment: widget.environment, // Pass environment
     );
 
     // Removed Dynamic Link Initialization

lib/router/router.dart

@@ -14,6 +14,7 @@ import 'package:ht_main/account/view/manage_followed_items/sources/add_source_to
 import 'package:ht_main/account/view/manage_followed_items/sources/followed_sources_list_page.dart'; // New
 import 'package:ht_main/account/view/saved_headlines_page.dart'; // Import SavedHeadlinesPage
 import 'package:ht_main/app/bloc/app_bloc.dart';
+import 'package:ht_main/app/config/config.dart' as local_config;
 import 'package:ht_main/app/view/app_shell.dart';
 import 'package:ht_main/authentication/bloc/authentication_bloc.dart';
 import 'package:ht_main/authentication/view/authentication_page.dart';
@@ -62,11 +63,13 @@ GoRouter createRouter({
   required HtDataRepository<UserContentPreferences>
   htUserContentPreferencesRepository,
   required HtDataRepository<AppConfig> htAppConfigRepository,
+  required local_config.AppEnvironment environment, // Added environment
 }) {
   // Instantiate AccountBloc once to be shared
   final accountBloc = AccountBloc(
     authenticationRepository: htAuthenticationRepository,
     userContentPreferencesRepository: htUserContentPreferencesRepository,
+    environment: environment, // Pass environment to AccountBloc
   );
 
   return GoRouter(