feat: get changelog from app's specific github path (ugly)

lib/services/github_api.dart

@@ -6,6 +6,16 @@ import 'package:timeago/timeago.dart';
 class GithubAPI {
   final GitHub _github = GitHub();
 
+  final Map<String, String> repoAppPath = {
+    'com.google.android.youtube': 'youtube',
+    'com.google.android.apps.youtube.music': 'music',
+    'com.twitter.android': 'twitter',
+    'com.reddit.frontpage': 'reddit',
+    'com.zhiliaoapp.musically': 'tiktok',
+    'de.dwd.warnapp': 'warnwetter',
+    'com.garzotto.pflotsh.ecmwf_a': 'ecmwf',
+  };
+
   Future<String?> latestReleaseVersion(String org, String repoName) async {
     try {
       var latestRelease = await _github.repositories.getLatestRelease(
@@ -61,9 +71,19 @@ class GithubAPI {
     )).toList();
   }
 
-  Future<List<RepositoryCommit>> getCommits(String org, String repoName) async {
+  Future<List<RepositoryCommit>> getCommits(
-    return await (_github.repositories.listCommits(
+    String packageName,
-      RepositorySlug(org, repoName),
+    String org,
+    String repoName,
+  ) async {
+    String path =
+        'src/main/kotlin/app/revanced/patches/${repoAppPath[packageName]}';
+    return await (PaginationHelper(_github)
+        .objects<Map<String, dynamic>, RepositoryCommit>(
+      'GET',
+      '/repos/$org/$repoName/commits',
+      (i) => RepositoryCommit.fromJson(i),
+      params: <String, dynamic>{'path': path},
     )).toList();
   }
 }

lib/services/manager_api.dart

@@ -15,11 +15,9 @@ class ManagerAPI {
   final GithubAPI _githubAPI = GithubAPI();
   final RootAPI _rootAPI = RootAPI();
   late SharedPreferences _prefs;
-  late List<RepositoryCommit> _commits = [];
 
   Future<void> initialize() async {
     _prefs = await SharedPreferences.getInstance();
-    _commits = (await _githubAPI.getCommits(ghOrg, patchesRepo)).toList();
   }
 
   Future<File?> downloadPatches(String extension) async {
@@ -62,16 +60,16 @@ class ManagerAPI {
         .toList();
   }
 
-  void setPatchedApps(List<PatchedApplication> patchedApps) {
+  Future<void> setPatchedApps(List<PatchedApplication> patchedApps) async {
-    _prefs.setStringList('patchedApps',
+    await _prefs.setStringList('patchedApps',
         patchedApps.map((a) => json.encode(a.toJson())).toList());
   }
 
-  void savePatchedApp(PatchedApplication app) {
+  Future<void> savePatchedApp(PatchedApplication app) async {
     List<PatchedApplication> patchedApps = getPatchedApps();
     patchedApps.removeWhere((a) => a.packageName == app.packageName);
     patchedApps.add(app);
-    setPatchedApps(patchedApps);
+    await setPatchedApps(patchedApps);
   }
 
   Future<void> reAssessSavedApps() async {
@@ -83,20 +81,12 @@ class ManagerAPI {
       if (isRemove) {
         toRemove.add(app);
       } else {
-        List<String> newChangelog = getAppChangelog(
+        app.hasUpdates = await hasAppUpdates(app.packageName, app.patchDate);
-          app.packageName,
+        app.changelog = await getAppChangelog(app.packageName, app.patchDate);
-          app.patchDate,
-        );
-        if (newChangelog.isNotEmpty) {
-          app.changelog = newChangelog;
-          app.hasUpdates = true;
-        } else {
-          app.hasUpdates = false;
-        }
       }
     }
     patchedApps.removeWhere((a) => toRemove.contains(a));
-    setPatchedApps(patchedApps);
+    await setPatchedApps(patchedApps);
   }
 
   Future<bool> isAppUninstalled(PatchedApplication app, bool isRoot) async {
@@ -108,24 +98,38 @@ class ManagerAPI {
     return !existsRoot && !existsNonRoot;
   }
 
-  List<String> getAppChangelog(String packageName, DateTime patchedDate) {
+  Future<bool> hasAppUpdates(String packageName, DateTime patchDate) async {
-    List<String> newCommits = _commits
+    List<RepositoryCommit> commits =
+        await _githubAPI.getCommits(packageName, ghOrg, patchesRepo);
+    return commits.any((c) =>
+        c.commit != null &&
+        c.commit!.author != null &&
+        c.commit!.author!.date != null &&
+        c.commit!.author!.date!.isAfter(patchDate));
+  }
+
+  Future<List<String>> getAppChangelog(
+    String packageName,
+    DateTime patchDate,
+  ) async {
+    List<RepositoryCommit> commits =
+        await _githubAPI.getCommits(packageName, ghOrg, patchesRepo);
+    List<String> newCommits = commits
         .where((c) =>
             c.commit != null &&
-            c.commit!.message != null &&
             c.commit!.author != null &&
             c.commit!.author!.date != null &&
-            c.commit!.author!.date!.isAfter(patchedDate))
+            c.commit!.author!.date!.isAfter(patchDate) &&
+            c.commit!.message != null)
         .map((c) => c.commit!.message!)
         .toList();
-    if (newCommits.isNotEmpty) {
+    if (newCommits.isEmpty) {
-      int firstChore = newCommits.indexWhere((c) => c.startsWith('chore'));
+      newCommits = commits
-      int secondChore =
+          .where((c) => c.commit != null && c.commit!.message != null)
-          newCommits.indexWhere((c) => c.startsWith('chore'), firstChore + 1);
+          .take(3)
-      if (firstChore >= 0 && secondChore > firstChore) {
+          .map((c) => c.commit!.message!)
-        return newCommits.sublist(firstChore + 1, secondChore);
+          .toList();
-      }
     }
-    return List.empty();
+    return newCommits;
   }
 }

lib/ui/views/home/home_view.dart

@@ -17,7 +17,6 @@ class HomeView extends StatelessWidget {
   Widget build(BuildContext context) {
     return ViewModelBuilder<HomeViewModel>.reactive(
       disposeViewModel: false,
-      fireOnModelReadyOnce: true,
       onModelReady: (model) => model.initialize(),
       viewModelBuilder: () => locator<HomeViewModel>(),
       builder: (context, model, child) => Scaffold(
@@ -111,8 +110,8 @@ class HomeView extends StatelessWidget {
                     ),
                     const SizedBox(height: 14),
                     model.showUpdatableApps
-                        ? const AvailableUpdatesCard()
+                        ? AvailableUpdatesCard()
-                        : const InstalledAppsCard()
+                        : InstalledAppsCard(),
                   ],
                 ),
               ),

lib/ui/views/home/home_viewmodel.dart

@@ -26,14 +26,14 @@ class HomeViewModel extends BaseViewModel {
   List<PatchedApplication> patchedUpdatableApps = [];
 
   Future<void> initialize() async {
-    await _getPatchedApps();
     await flutterLocalNotificationsPlugin.initialize(
       const InitializationSettings(
         android: AndroidInitializationSettings('ic_notification'),
       ),
       onSelectNotification: (p) => DeviceApps.openApp('app.revanced.manager'),
     );
-    _managerAPI.reAssessSavedApps().then((_) => notifyListeners());
+    _getPatchedApps();
+    _managerAPI.reAssessSavedApps().then((_) => _getPatchedApps());
   }
 
   void toggleUpdatableApps(bool value) {
@@ -49,11 +49,8 @@ class HomeViewModel extends BaseViewModel {
     locator<MainViewModel>().setIndex(1);
   }
 
-  Future<void> _getPatchedApps() async {
+  void _getPatchedApps() {
-    patchedInstalledApps = _managerAPI
+    patchedInstalledApps = _managerAPI.getPatchedApps().toList();
-        .getPatchedApps()
-        .where((app) => app.hasUpdates == false)
-        .toList();
     patchedUpdatableApps = _managerAPI
         .getPatchedApps()
         .where((app) => app.hasUpdates == true)

lib/ui/views/installer/installer_viewmodel.dart

@@ -136,7 +136,7 @@ class InstallerViewModel extends BaseViewModel {
         update(1.0, 'Installed!', 'Installed!');
         _app!.patchDate = DateTime.now();
         _app!.appliedPatches = _patches.map((p) => p.name).toList();
-        _managerAPI.savePatchedApp(_app!);
+        await _managerAPI.savePatchedApp(_app!);
       } else {
         update(1.0, 'Aborting...', 'An error occurred! Aborting');
       }

lib/ui/views/root_checker/root_checker_viewmodel.dart

@@ -20,7 +20,7 @@ class RootCheckerViewModel extends BaseViewModel {
 
   Future<void> navigateToHome() async {
     final prefs = await SharedPreferences.getInstance();
-    prefs.setBool('isRooted', isRooted);
+    await prefs.setBool('isRooted', isRooted);
     _navigationService.navigateTo(Routes.navigation);
     notifyListeners();
   }

lib/ui/widgets/homeView/available_updates_card.dart

@@ -1,21 +1,24 @@
 import 'package:flutter/material.dart';
 import 'package:revanced_manager/app/app.locator.dart';
+import 'package:revanced_manager/models/patched_application.dart';
 import 'package:revanced_manager/ui/views/home/home_viewmodel.dart';
 import 'package:revanced_manager/ui/widgets/shared/application_item.dart';
 
 class AvailableUpdatesCard extends StatelessWidget {
-  const AvailableUpdatesCard({
+  AvailableUpdatesCard({
     Key? key,
   }) : super(key: key);
 
+  final List<PatchedApplication> apps =
+      locator<HomeViewModel>().patchedUpdatableApps;
+
   @override
   Widget build(BuildContext context) {
     return ListView(
       shrinkWrap: true,
       padding: EdgeInsets.zero,
       physics: const NeverScrollableScrollPhysics(),
-      children: locator<HomeViewModel>()
+      children: apps
-          .patchedUpdatableApps
           .map((app) => ApplicationItem(
                 icon: app.icon,
                 name: app.name,

lib/ui/widgets/homeView/installed_apps_card.dart

@@ -1,22 +1,25 @@
 import 'package:device_apps/device_apps.dart';
 import 'package:flutter/material.dart';
 import 'package:revanced_manager/app/app.locator.dart';
+import 'package:revanced_manager/models/patched_application.dart';
 import 'package:revanced_manager/ui/views/home/home_viewmodel.dart';
 import 'package:revanced_manager/ui/widgets/shared/application_item.dart';
 
 class InstalledAppsCard extends StatelessWidget {
-  const InstalledAppsCard({
+  InstalledAppsCard({
     Key? key,
   }) : super(key: key);
 
+  final List<PatchedApplication> apps =
+      locator<HomeViewModel>().patchedInstalledApps;
+
   @override
   Widget build(BuildContext context) {
     return ListView(
       shrinkWrap: true,
       padding: EdgeInsets.zero,
       physics: const NeverScrollableScrollPhysics(),
-      children: locator<HomeViewModel>()
+      children: apps
-          .patchedInstalledApps
           .map((app) => ApplicationItem(
                 icon: app.icon,
                 name: app.name,

lib/ui/widgets/shared/application_item.dart

@@ -105,7 +105,7 @@ class ApplicationItem extends StatelessWidget {
             ),
             const SizedBox(height: 4),
             Text(
-              '    - ${changelog.join('\n- ')}',
+              '\u2022 ${changelog.join('\n\u2022 ')}',
               style: kRobotoTextStyle,
             ),
           ],