refactor(demo): remove unused data repositories and fixtures

- Remove DataRepository<Engagement>, DataRepository<Report>, and DataRepository<AppReview> from DemoDataInitializerService and DemoDataMigrationService
- Delete related fixture data lists and initialization/migration methods
- Update dependencies in bootstrap.dart
- Update pubspec.yaml and pubspec.lock with new core ref

Author: fulleni

lib/app/services/demo_data_initializer_service.dart

@@ -21,32 +21,20 @@ class DemoDataInitializerService {
   DemoDataInitializerService({
     required DataRepository<AppSettings> appSettingsRepository,
     required DataRepository<UserContentPreferences>
-    userContentPreferencesRepository,
-    required DataRepository<Engagement> engagementRepository,
-    required DataRepository<Report> reportRepository,
-    required DataRepository<AppReview> appReviewRepository,
+        userContentPreferencesRepository,
     required DataRepository<InAppNotification> inAppNotificationRepository,
     required this.appSettingsFixturesData,
     required this.userContentPreferencesFixturesData,
     required this.inAppNotificationsFixturesData,
-    required this.engagementFixturesData,
-    required this.reportFixturesData,
-    required this.appReviewFixturesData,
   }) : _appSettingsRepository = appSettingsRepository,
        _userContentPreferencesRepository = userContentPreferencesRepository,
-       _engagementRepository = engagementRepository,
-       _reportRepository = reportRepository,
-       _appReviewRepository = appReviewRepository,
        _inAppNotificationRepository = inAppNotificationRepository,
        _logger = Logger('DemoDataInitializerService');
 
   final DataRepository<AppSettings> _appSettingsRepository;
   final DataRepository<UserContentPreferences>
-  _userContentPreferencesRepository;
+      _userContentPreferencesRepository;
   final DataRepository<InAppNotification> _inAppNotificationRepository;
-  final DataRepository<Engagement> _engagementRepository;
-  final DataRepository<Report> _reportRepository;
-  final DataRepository<AppReview> _appReviewRepository;
   final Logger _logger;
 
   /// A list of [AppSettings] fixture data to be used as a template.
@@ -64,15 +52,6 @@ class DemoDataInitializerService {
   /// All items in this list will be cloned for new users.
   final List<InAppNotification> inAppNotificationsFixturesData;
 
-  /// A list of [Engagement] fixture data to be used as a template.
-  final List<Engagement> engagementFixturesData;
-
-  /// A list of [Report] fixture data to be used as a template.
-  final List<Report> reportFixturesData;
-
-  /// A list of [AppReview] fixture data to be used as a template.
-  final List<AppReview> appReviewFixturesData;
-
   /// Initializes essential user-specific data in the in-memory clients
   /// for the given [user].
   ///
@@ -85,14 +64,10 @@ class DemoDataInitializerService {
   /// user in the demo environment.
   Future<void> initializeUserSpecificData(User user) async { 
     _logger.info('Initializing user-specific data for user ID: ${user.id}');
-
     await Future.wait([
       _ensureAppSettingsExist(user.id),
       _ensureUserContentPreferencesExist(user.id),
       _ensureInAppNotificationsExist(user.id),
-      _ensureEngagementsExist(user.id),
-      _ensureReportsExist(user.id),
-      _ensureAppReviewsExist(user.id),
     ]);
 
     _logger.info(
@@ -235,126 +210,4 @@ class DemoDataInitializerService {
       // is not a critical failure for the app's startup.
     }
   }
-
-  /// Ensures that [Engagement]s exist for the given [userId].
-  Future<void> _ensureEngagementsExist(String userId) async {
-    try {
-      final existing = await _engagementRepository.readAll(userId: userId);
-      if (existing.items.isNotEmpty) {
-        _logger.finer('Engagements already exist for user ID: $userId.');
-        return;
-      }
-
-      _logger.fine(
-        'No Engagements found for user ID: $userId. Creating from fixture.',
-      );
-
-      if (engagementFixturesData.isEmpty) {
-        _logger.warning('engagementFixturesData is empty. No items to create.');
-        return;
-      }
-
-      // Filter engagements to only those not belonging to the fixture user.
-      final engagementsToCreate = engagementFixturesData
-          .where((e) => e.userId != 'fixture_user_id')
-          .toList();
-
-      final userItems = engagementsToCreate.map(
-        (i) => i.copyWith(userId: userId),
-      );
-
-      await Future.wait(
-        userItems.map(
-          (item) => _engagementRepository.create(item: item, userId: userId),
-        ),
-      );
-      _logger.fine(
-        '${userItems.length} Engagements from fixture created for user ID: $userId.',
-      );
-    } catch (e, s) {
-      _logger.severe(
-        'Error ensuring Engagements exist for user ID: $userId: $e',
-        e,
-        s,
-      );
-    }
-  }
-
-  /// Ensures that [Report]s exist for the given [userId].
-  Future<void> _ensureReportsExist(String userId) async {
-    try {
-      final existing = await _reportRepository.readAll(userId: userId);
-      if (existing.items.isNotEmpty) {
-        _logger.finer('Reports already exist for user ID: $userId.');
-        return;
-      }
-
-      _logger.fine(
-        'No Reports found for user ID: $userId. Creating from fixture.',
-      );
-
-      if (reportFixturesData.isEmpty) {
-        _logger.warning('reportFixturesData is empty. No items to create.');
-        return;
-      }
-
-      final userItems = reportFixturesData.map(
-        (i) => i.copyWith(reporterUserId: userId),
-      );
-
-      await Future.wait(
-        userItems.map(
-          (item) => _reportRepository.create(item: item, userId: userId),
-        ),
-      );
-      _logger.fine(
-        '${userItems.length} Reports from fixture created for user ID: $userId.',
-      );
-    } catch (e, s) {
-      _logger.severe(
-        'Error ensuring Reports exist for user ID: $userId: $e',
-        e,
-        s,
-      );
-    }
-  }
-
-  /// Ensures that [AppReview]s exist for the given [userId].
-  Future<void> _ensureAppReviewsExist(String userId) async {
-    try {
-      final existing = await _appReviewRepository.readAll(userId: userId);
-      if (existing.items.isNotEmpty) {
-        _logger.finer('AppReviews already exist for user ID: $userId.');
-        return;
-      }
-
-      _logger.fine(
-        'No AppReviews found for user ID: $userId. Creating from fixture.',
-      );
-
-      if (appReviewFixturesData.isEmpty) {
-        _logger.warning('appReviewFixturesData is empty. No items to create.');
-        return;
-      }
-
-      final userItems = appReviewFixturesData.map(
-        (i) => i.copyWith(userId: userId),
-      );
-
-      await Future.wait(
-        userItems.map(
-          (item) => _appReviewRepository.create(item: item, userId: userId),
-        ),
-      );
-      _logger.fine(
-        '${userItems.length} AppReviews from fixture created for user ID: $userId.',
-      );
-    } catch (e, s) {
-      _logger.severe(
-        'Error ensuring AppReviews exist for user ID: $userId: $e',
-        e,
-        s,
-      );
-    }
-  }
 }

lib/app/services/demo_data_migration_service.dart

@@ -17,21 +17,15 @@ class DemoDataMigrationService {
     required DataRepository<UserContentPreferences>
     userContentPreferencesRepository,
     required DataRepository<Engagement> engagementRepository,
-    required DataRepository<Report> reportRepository,
-    required DataRepository<AppReview> appReviewRepository,
   }) : _appSettingsRepository = appSettingsRepository,
        _userContentPreferencesRepository = userContentPreferencesRepository,
        _engagementRepository = engagementRepository,
-       _reportRepository = reportRepository,
-       _appReviewRepository = appReviewRepository,
        _logger = Logger('DemoDataMigrationService');
 
   final DataRepository<AppSettings> _appSettingsRepository;
   final DataRepository<UserContentPreferences>
   _userContentPreferencesRepository;
   final DataRepository<Engagement> _engagementRepository;
-  final DataRepository<Report> _reportRepository;
-  final DataRepository<AppReview> _appReviewRepository;
   final Logger _logger;
 
   /// Migrates user settings and content preferences from an old anonymous
@@ -190,54 +184,5 @@ class DemoDataMigrationService {
       );
     }
 
-    // Migrate Reports
-    try {
-      final oldReports = await _reportRepository.readAll(userId: oldUserId);
-      for (final oldReport in oldReports.items) {
-        final newReport = oldReport.copyWith(reporterUserId: newUserId);
-        await _reportRepository.create(item: newReport, userId: newUserId);
-        await _reportRepository.delete(id: oldReport.id, userId: oldUserId);
-      }
-      _logger.fine(
-        '[DemoDataMigrationService] ${oldReports.items.length} '
-        'reports migrated successfully from $oldUserId to $newUserId.',
-      );
-    } catch (e, s) {
-      _logger.severe(
-        '[DemoDataMigrationService] Error migrating reports from '
-        '$oldUserId to $newUserId: $e',
-        e,
-        s,
-      );
-    }
-
-    // Migrate AppReviews
-    try {
-      final oldAppReviews = await _appReviewRepository.readAll(
-        userId: oldUserId,
-      );
-      for (final oldAppReview in oldAppReviews.items) {
-        final newAppReview = oldAppReview.copyWith(userId: newUserId);
-        await _appReviewRepository.create(
-          item: newAppReview,
-          userId: newUserId,
-        );
-        await _appReviewRepository.delete(
-          id: oldAppReview.id,
-          userId: oldUserId,
-        );
-      }
-      _logger.fine(
-        '[DemoDataMigrationService] ${oldAppReviews.items.length} '
-        'app reviews migrated successfully from $oldUserId to $newUserId.',
-      );
-    } catch (e, s) {
-      _logger.severe(
-        '[DemoDataMigrationService] Error migrating app reviews from '
-        '$oldUserId to $newUserId: $e',
-        e,
-        s,
-      );
-    }
   }
 }

lib/bootstrap.dart

@@ -508,9 +508,9 @@ Future<Widget> bootstrap(
     cacheDuration: const Duration(minutes: 5),
     logger: logger,
   );
-  logger.fine('ContentLimitationService initialized.');
-
-  logger.fine('AppReviewService initialized.');
+  logger
+    ..fine('ContentLimitationService initialized.')
+    ..fine('AppReviewService initialized.');
 
   // Conditionally instantiate DemoDataMigrationService
   final demoDataMigrationService =
@@ -519,8 +519,6 @@ Future<Widget> bootstrap(
           appSettingsRepository: appSettingsRepository,
           userContentPreferencesRepository: userContentPreferencesRepository,
           engagementRepository: engagementRepository,
-          reportRepository: reportRepository,
-          appReviewRepository: appReviewRepository,
         )
       : null;
   logger.fine(
@@ -538,16 +536,10 @@ Future<Widget> bootstrap(
           appSettingsRepository: appSettingsRepository,
           userContentPreferencesRepository: userContentPreferencesRepository,
           inAppNotificationRepository: inAppNotificationRepository,
-          engagementRepository: engagementRepository,
-          reportRepository: reportRepository,
-          appReviewRepository: appReviewRepository,
           appSettingsFixturesData: appSettingsFixturesData,
           userContentPreferencesFixturesData:
               getUserContentPreferencesFixturesData(),
           inAppNotificationsFixturesData: inAppNotificationsFixturesData,
-          engagementFixturesData: getEngagementsFixturesData(),
-          reportFixturesData: getReportsFixturesData(),
-          appReviewFixturesData: getAppReviewsFixturesData(),
         )
       : null;
   logger

pubspec.lock

@@ -185,8 +185,8 @@ packages:
     dependency: "direct main"
     description:
       path: "."
-      ref: b1110a45182e5b4439b24e1a6a18bdeb5e048fb7
-      resolved-ref: b1110a45182e5b4439b24e1a6a18bdeb5e048fb7
+      ref: "8e02a696ee8921c68160cd15bd1c55effb5212a7"
+      resolved-ref: "8e02a696ee8921c68160cd15bd1c55effb5212a7"
       url: "https://github.com/flutter-news-app-full-source-code/core.git"
     source: git
     version: "1.3.1"

pubspec.yaml

@@ -147,7 +147,7 @@ dependency_overrides:
   core:
     git:
       url: https://github.com/flutter-news-app-full-source-code/core.git
-      ref: b1110a45182e5b4439b24e1a6a18bdeb5e048fb7
+      ref: 8e02a696ee8921c68160cd15bd1c55effb5212a7
   http_client:
     git:
       url: https://github.com/flutter-news-app-full-source-code/http-client.git