refactor(database): update migration class and log message

- Update RefactorAdConfigToRoleBased migration class constructor parameters
- Improve log message in up() method to include PR ID and date

Author: fulleni

lib/src/database/migrations/20250924084800__refactor_ad_config_to_role_based.dart

@@ -20,34 +20,37 @@ import 'package:mongo_dart/mongo_dart.dart';
 class RefactorAdConfigToRoleBased extends Migration {
   /// {@macro refactor_ad_config_to_role_based}
   RefactorAdConfigToRoleBased()
-      : super(
-          version: '20250924084800',
-          description: 'Refactor adConfig to use role-based visibleTo maps',
-          gitHubPullRequest:
-              'https://github.com/flutter-news-app-full-source-code/core/pull/50',
-        );
+    : super(
+        prDate: '20250924084800',
+        prSummary: 'Refactor adConfig to use role-based visibleTo maps',
+        prId: '50',
+      );
 
   @override
   Future<void> up(Db db, Logger log) async {
-    log.info('Applying migration: Refactor adConfig to role-based visibleTo maps.');
+    log.info(
+      'Applying migration PR#$prId (Date: $prDate): $prSummary.',
+    );
 
     final remoteConfigCollection = db.collection('remote_configs');
 
     // Define default FeedAdFrequencyConfig for roles
-    const defaultGuestFeedAdFrequency =
-        FeedAdFrequencyConfig(adFrequency: 5, adPlacementInterval: 3);
-    const defaultStandardUserFeedAdFrequency =
-        FeedAdFrequencyConfig(adFrequency: 10, adPlacementInterval: 5);
-    const defaultPremiumUserFeedAdFrequency =
-        FeedAdFrequencyConfig(adFrequency: 0, adPlacementInterval: 0);
-
+    const defaultGuestFeedAdFrequency = FeedAdFrequencyConfig(
+      adFrequency: 5,
+      adPlacementInterval: 3,
+    );
+    const defaultStandardUserFeedAdFrequency = FeedAdFrequencyConfig(
+      adFrequency: 10,
+      adPlacementInterval: 5,
+    );
     // Define default InterstitialAdFrequencyConfig for roles
-    const defaultGuestInterstitialAdFrequency =
-        InterstitialAdFrequencyConfig(transitionsBeforeShowingInterstitialAds: 5);
+    const defaultGuestInterstitialAdFrequency = InterstitialAdFrequencyConfig(
+      transitionsBeforeShowingInterstitialAds: 5,
+    );
     const defaultStandardUserInterstitialAdFrequency =
-        InterstitialAdFrequencyConfig(transitionsBeforeShowingInterstitialAds: 10);
-    const defaultPremiumUserInterstitialAdFrequency =
-        InterstitialAdFrequencyConfig(transitionsBeforeShowingInterstitialAds: 50000);
+        InterstitialAdFrequencyConfig(
+          transitionsBeforeShowingInterstitialAds: 10,
+        );
 
     // Define default ArticleAdSlot visibility for roles
     final defaultArticleAdSlots = {
@@ -57,25 +60,35 @@ class RefactorAdConfigToRoleBased extends Migration {
 
     final result = await remoteConfigCollection.updateMany(
       // Find documents that still have the old structure (e.g., old frequency fields)
-      where.exists('adConfig.feedAdConfiguration.frequencyConfig.guestAdFrequency'),
+      where.exists(
+        'adConfig.feedAdConfiguration.frequencyConfig.guestAdFrequency',
+      ),
       ModifierBuilder()
         // --- FeedAdConfiguration Transformation ---
         // Remove old frequencyConfig fields
         ..unset('adConfig.feedAdConfiguration.frequencyConfig.guestAdFrequency')
-        ..unset('adConfig.feedAdConfiguration.frequencyConfig.guestAdPlacementInterval')
-        ..unset('adConfig.feedAdConfiguration.frequencyConfig.authenticatedAdFrequency')
-        ..unset('adConfig.feedAdConfiguration.frequencyConfig.authenticatedAdPlacementInterval')
-        ..unset('adConfig.feedAdConfiguration.frequencyConfig.premiumAdFrequency')
-        ..unset('adConfig.feedAdConfiguration.frequencyConfig.premiumAdPlacementInterval')
+        ..unset(
+          'adConfig.feedAdConfiguration.frequencyConfig.guestAdPlacementInterval',
+        )
+        ..unset(
+          'adConfig.feedAdConfiguration.frequencyConfig.authenticatedAdFrequency',
+        )
+        ..unset(
+          'adConfig.feedAdConfiguration.frequencyConfig.authenticatedAdPlacementInterval',
+        )
+        ..unset(
+          'adConfig.feedAdConfiguration.frequencyConfig.premiumAdFrequency',
+        )
+        ..unset(
+          'adConfig.feedAdConfiguration.frequencyConfig.premiumAdPlacementInterval',
+        )
         // Set the new visibleTo map for FeedAdConfiguration
         ..set(
           'adConfig.feedAdConfiguration.visibleTo',
           {
             AppUserRole.guestUser.name: defaultGuestFeedAdFrequency.toJson(),
-            AppUserRole.standardUser.name:
-                defaultStandardUserFeedAdFrequency.toJson(),
-            AppUserRole.premiumUser.name:
-                defaultPremiumUserFeedAdFrequency.toJson(),
+            AppUserRole.standardUser.name: defaultStandardUserFeedAdFrequency
+                .toJson(),
           },
         )
         // --- ArticleAdConfiguration Transformation ---
@@ -87,24 +100,27 @@ class RefactorAdConfigToRoleBased extends Migration {
           {
             AppUserRole.guestUser.name: defaultArticleAdSlots,
             AppUserRole.standardUser.name: defaultArticleAdSlots,
-            AppUserRole.premiumUser.name: defaultArticleAdSlots,
           },
         )
         // --- InterstitialAdConfiguration Transformation ---
         // Remove old feedInterstitialAdFrequencyConfig fields
-        ..unset('adConfig.interstitialAdConfiguration.feedInterstitialAdFrequencyConfig.guestTransitionsBeforeShowingInterstitialAds')
-        ..unset('adConfig.interstitialAdConfiguration.feedInterstitialAdFrequencyConfig.standardUserTransitionsBeforeShowingInterstitialAds')
-        ..unset('adConfig.interstitialAdConfiguration.feedInterstitialAdFrequencyConfig.premiumUserTransitionsBeforeShowingInterstitialAds')
+        ..unset(
+          'adConfig.interstitialAdConfiguration.feedInterstitialAdFrequencyConfig.guestTransitionsBeforeShowingInterstitialAds',
+        )
+        ..unset(
+          'adConfig.interstitialAdConfiguration.feedInterstitialAdFrequencyConfig.standardUserTransitionsBeforeShowingInterstitialAds',
+        )
+        ..unset(
+          'adConfig.interstitialAdConfiguration.feedInterstitialAdFrequencyConfig.premiumUserTransitionsBeforeShowingInterstitialAds',
+        )
         // Set the new visibleTo map for InterstitialAdConfiguration
         ..set(
           'adConfig.interstitialAdConfiguration.visibleTo',
           {
-            AppUserRole.guestUser.name:
-                defaultGuestInterstitialAdFrequency.toJson(),
+            AppUserRole.guestUser.name: defaultGuestInterstitialAdFrequency
+                .toJson(),
             AppUserRole.standardUser.name:
                 defaultStandardUserInterstitialAdFrequency.toJson(),
-            AppUserRole.premiumUser.name:
-                defaultPremiumUserInterstitialAdFrequency.toJson(),
           },
         ),
     );
@@ -125,7 +141,9 @@ class RefactorAdConfigToRoleBased extends Migration {
     // Reverting to the old structure would require re-introducing the old fields
     // and potentially losing data if the new structure was used.
     // For simplicity in this example, we'll just unset the new fields.
-    final result = await db.collection('remote_configs').updateMany(
+    final result = await db
+        .collection('remote_configs')
+        .updateMany(
           where.exists('adConfig.feedAdConfiguration.visibleTo'),
           ModifierBuilder()
             ..unset('adConfig.feedAdConfiguration.visibleTo')