feat: fix list contributors and add patched apps changelog (wip)

lib/services/github_api.dart

@@ -56,13 +56,14 @@ class GithubAPI {
   }
 
   Future<List<Contributor>> getContributors(String org, repoName) async {
-    try {
+    return await (_github.repositories.listContributors(
-      var contributors = _github.repositories.listContributors(
       RepositorySlug(org, repoName),
-      );
+    )).toList();
-      return contributors.toList();
-    } on Exception {
-      return List.empty();
   }
+
+  Future<List<RepositoryCommit>> getCommits(String org, repoName) async {
+    return await (_github.repositories.listCommits(
+      RepositorySlug(org, repoName),
+    )).toList();
   }
 }

lib/services/manager_api.dart

@@ -94,6 +94,7 @@ class ManagerAPI {
   Future<void> reAssessSavedApps() async {
     List<PatchedApplication> patchedApps = getPatchedApps();
     bool isRoot = isRooted() ?? false;
+    List<PatchedApplication> toRemove = [];
     for (PatchedApplication app in patchedApps) {
       bool existsRoot = false;
       if (isRoot) {
@@ -101,23 +102,23 @@ class ManagerAPI {
       }
       bool existsNonRoot = await DeviceApps.isAppInstalled(app.packageName);
       if (!existsRoot && !existsNonRoot) {
-        patchedApps.remove(app);
+        toRemove.add(app);
       } else if (existsNonRoot) {
         ApplicationWithIcon? application =
             await DeviceApps.getApp(app.packageName, true)
                 as ApplicationWithIcon?;
         if (application != null) {
           int savedVersionInt =
-              int.parse(app.version.replaceFirst('v', '').replaceAll('.', ''));
+              int.parse(app.version.replaceAll(RegExp('[^0-9]'), ''));
-          int currentVersionInt = int.parse(application.versionName!
+          int currentVersionInt = int.parse(
-              .replaceFirst('v', '')
+              application.versionName!.replaceAll(RegExp('[^0-9]'), ''));
-              .replaceAll('.', ''));
           if (savedVersionInt < currentVersionInt) {
-            patchedApps.remove(app);
+            toRemove.add(app);
           }
         }
       }
     }
+    patchedApps.removeWhere((a) => toRemove.contains(a));
     setPatchedApps(patchedApps);
     List<String> apps = await _rootAPI.getInstalledApps();
     for (String packageName in apps) {
@@ -136,8 +137,19 @@ class ManagerAPI {
     return false;
   }
 
-  Future<String> getAppChangelog(String packageName) async {
+  Future<List<String>> getAppChangelog(
-    // TODO: get changelog based on last commits on the folder of this app?
+    String packageName,
-    return 'To be implemented';
+    DateTime lastUpdated,
+  ) async {
+    return (await _githubAPI.getCommits(ghOrg, patchesRepo))
+        .where((c) =>
+            c.commit != null &&
+            c.commit!.message != null &&
+            !c.commit!.message!.startsWith('chore') &&
+            c.commit!.author != null &&
+            c.commit!.author!.date != null)
+        .map((c) => '  - ${c.commit!.message!}')
+        .toList()
+        .sublist(0, 3);
   }
 }

lib/services/root_api.dart

@@ -27,12 +27,15 @@ class RootAPI {
       );
       if (res != null) {
         List<String> apps = res.split('\n');
+        List<String> toRemove = [];
         for (String packageName in apps) {
           bool isInstalled = await isAppInstalled(packageName);
           if (!isInstalled) {
-            apps.remove(packageName);
+            toRemove.add(packageName);
           }
         }
+        apps.removeWhere((a) => toRemove.contains(a));
+        return apps;
       }
     } on Exception {
       return List.empty();

lib/ui/views/contributors/contributors_viewmodel.dart

@@ -32,5 +32,6 @@ class ContributorsViewModel extends BaseViewModel {
       ghOrg,
       managerRepo,
     );
+    notifyListeners();
   }
 }

lib/ui/views/home/home_viewmodel.dart

@@ -65,9 +65,9 @@ class HomeViewModel extends BaseViewModel {
     if (latestVersion != null) {
       try {
         int latestVersionInt =
-            int.parse(latestVersion.replaceFirst('v', '').replaceAll('.', ''));
+            int.parse(latestVersion.replaceAll(RegExp('[^0-9]'), ''));
         int currentVersionInt =
-            int.parse(currentVersion.replaceFirst('v', '').replaceAll('.', ''));
+            int.parse(currentVersion.replaceAll(RegExp('[^0-9]'), ''));
         return latestVersionInt > currentVersionInt;
       } on Exception {
         return false;

lib/ui/widgets/homeView/available_updates_card.dart

@@ -20,23 +20,24 @@ class AvailableUpdatesCard extends StatelessWidget {
       children: [
         FutureBuilder<List<PatchedApplication>>(
           future: locator<HomeViewModel>().getPatchedApps(true),
-          builder: (context, snapshot) =>
+          builder: (context, snapshot) => snapshot.hasData &&
-              snapshot.hasData && snapshot.data!.isNotEmpty
+                  snapshot.data!.isNotEmpty
               ? ListView.builder(
                   shrinkWrap: true,
                   physics: const NeverScrollableScrollPhysics(),
                   padding: EdgeInsets.zero,
                   itemCount: snapshot.data!.length,
-                      itemBuilder: (context, index) => FutureBuilder<String>(
+                  itemBuilder: (context, index) => FutureBuilder<List<String>>(
                     future: _managerAPI.getAppChangelog(
                       snapshot.data![index].packageName,
+                      snapshot.data![index].patchDate,
                     ),
-                        initialData: '',
+                    initialData: List.empty(),
                     builder: (context, snapshot2) => ApplicationItem(
                       icon: snapshot.data![index].icon,
                       name: snapshot.data![index].name,
                       patchDate: snapshot.data![index].patchDate,
-                          changelog: snapshot2.data!,
+                      changelog: '${snapshot2.data!.join('\n')}\n...',
                       isUpdatableApp: true,
                       onPressed: () =>
                           locator<HomeViewModel>().navigateToPatcher(

lib/ui/widgets/homeView/installed_apps_card.dart

@@ -21,23 +21,24 @@ class InstalledAppsCard extends StatelessWidget {
       children: [
         FutureBuilder<List<PatchedApplication>>(
           future: locator<HomeViewModel>().getPatchedApps(false),
-          builder: (context, snapshot) =>
+          builder: (context, snapshot) => snapshot.hasData &&
-              snapshot.hasData && snapshot.data!.isNotEmpty
+                  snapshot.data!.isNotEmpty
               ? ListView.builder(
                   shrinkWrap: true,
                   physics: const NeverScrollableScrollPhysics(),
                   padding: EdgeInsets.zero,
                   itemCount: snapshot.data!.length,
-                      itemBuilder: (context, index) => FutureBuilder<String>(
+                  itemBuilder: (context, index) => FutureBuilder<List<String>>(
                     future: _managerAPI.getAppChangelog(
                       snapshot.data![index].packageName,
+                      snapshot.data![index].patchDate,
                     ),
-                        initialData: '',
+                    initialData: const ['Loading'],
                     builder: (context, snapshot2) => ApplicationItem(
                       icon: snapshot.data![index].icon,
                       name: snapshot.data![index].name,
                       patchDate: snapshot.data![index].patchDate,
-                          changelog: snapshot2.data!,
+                      changelog: '${snapshot2.data!.join('\n')}\n(...)',
                       isUpdatableApp: false,
                       onPressed: () => DeviceApps.openApp(
                         snapshot.data![index].packageName,