refactor: Migrate manage snaps providers

packages/app_center/lib/manage/local_snap_providers.dart

@@ -1,22 +1,49 @@
 import 'package:app_center/manage/manage_model.dart';
 import 'package:app_center/snapd/snapd.dart';
 import 'package:flutter_riverpod/flutter_riverpod.dart';
+import 'package:riverpod_annotation/riverpod_annotation.dart';
+import 'package:snapd/snapd.dart';
+
+part 'local_snap_providers.g.dart';
 
 final localSnapFilterProvider = StateProvider.autoDispose<String>((_) => '');
 final showLocalSystemAppsProvider = StateProvider<bool>((_) => false);
 final localSnapSortOrderProvider =
     StateProvider<SnapSortOrder>((_) => SnapSortOrder.alphabeticalAsc);
-final localSnapsProvider = Provider.autoDispose(
-  (ref) => ref
-      .watch(manageModelProvider.select((m) => m.nonRefreshableSnaps))
-      .where(
-        (snap) => snap.titleOrName
-            .toLowerCase()
-            .contains(ref.watch(localSnapFilterProvider).toLowerCase()),
-      )
-      .where(
-        (snap) =>
-            ref.watch(showLocalSystemAppsProvider) || snap.apps.isNotEmpty,
-      )
-      .sortedSnaps(ref.watch(localSnapSortOrderProvider)),
-);
+
+@riverpod
+class FilteredLocalSnaps extends _$FilteredLocalSnaps {
+  @override
+  Future<Iterable<Snap>> build() async {
+    final nonRefreshableSnaps = await ref.watch(
+      manageModelProvider.future.select(
+        (m) => m.then((value) => value.nonRefreshableSnaps),
+      ),
+    );
+    void refreshFunction(_, __) => _refreshWithFilters(nonRefreshableSnaps);
+    ref.listen(localSnapFilterProvider, refreshFunction);
+    ref.listen(showLocalSystemAppsProvider, refreshFunction);
+    ref.listen(localSnapSortOrderProvider, refreshFunction);
+    return _refreshWithFilters(nonRefreshableSnaps, updateState: false);
+  }
+
+  Iterable<Snap> _refreshWithFilters(
+    Iterable<Snap> nonRefreshableSnaps, {
+    bool updateState = true,
+  }) {
+    final filter = ref.read(localSnapFilterProvider).toLowerCase();
+    final showSystemApps = ref.read(showLocalSystemAppsProvider);
+    final sortOrder = ref.read(localSnapSortOrderProvider);
+    final filteredSnaps = nonRefreshableSnaps
+        .where(
+          (snap) =>
+              snap.titleOrName.toLowerCase().contains(filter) &&
+              (showSystemApps || snap.apps.isNotEmpty),
+        )
+        .sortedSnaps(sortOrder);
+    if (updateState) {
+      state = AsyncData(filteredSnaps);
+    }
+    return filteredSnaps;
+  }
+}

packages/app_center/lib/manage/manage_model.dart

@@ -1,63 +1,23 @@
+import 'package:app_center/manage/manage_snaps_data.dart';
 import 'package:app_center/snapd/snapd.dart';
-import 'package:collection/collection.dart';
-import 'package:flutter/foundation.dart';
-import 'package:flutter_riverpod/flutter_riverpod.dart';
-import 'package:snapd/snapd.dart';
+import 'package:riverpod_annotation/riverpod_annotation.dart';
 import 'package:ubuntu_service/ubuntu_service.dart';
 
-final manageModelProvider = ChangeNotifierProvider.autoDispose(
-  (ref) => ManageModel(
-    snapd: getService<SnapdService>(),
-    updatesModel: ref.read(updatesModelProvider),
-  )..init(),
-);
+part 'manage_model.g.dart';
 
-class ManageModel extends ChangeNotifier {
-  ManageModel({
-    required this.snapd,
-    required this.updatesModel,
-  }) : _state = const AsyncValue.loading();
-  final SnapdService snapd;
-  final UpdatesModel updatesModel;
-
-  AsyncValue<void> get state => _state;
-  AsyncValue<void> _state;
-
-  List<Snap>? _installedSnaps;
-  List<String>? _refreshableSnapNames;
-
-  bool _isRefreshable(Snap snap) => updatesModel.hasUpdate(snap.name);
-  Iterable<Snap> get refreshableSnaps =>
-      _installedSnaps?.where(_isRefreshable) ?? const Iterable.empty();
-  Iterable<Snap> get nonRefreshableSnaps =>
-      _installedSnaps?.whereNot(_isRefreshable) ?? const Iterable.empty();
-
-  void _getRefreshableSnapNames() {
-    final refreshableSnapNames = updatesModel.refreshableSnapNames.toList();
-    if (!listEquals(refreshableSnapNames, _refreshableSnapNames)) {
-      _refreshableSnapNames = refreshableSnapNames;
-      notifyListeners();
-    }
-  }
-
-  // TODO: cache local snaps
-  Future<void> init() async {
-    updatesModel.addListener(_getRefreshableSnapNames);
-    _state = await AsyncValue.guard(() async {
-      await _getInstalledSnaps();
-      notifyListeners();
-    });
-  }
+@Riverpod(keepAlive: true)
+class ManageModel extends _$ManageModel {
+  late final _snapd = getService<SnapdService>();
 
   @override
-  void dispose() {
-    updatesModel.removeListener(_getRefreshableSnapNames);
-    super.dispose();
-  }
-
-  Future<void> _getInstalledSnaps() async {
-    _installedSnaps = await snapd.getSnaps().then(
-          (snaps) => snaps.sortedBy((snap) => snap.titleOrName.toLowerCase()),
-        );
+  Future<ManageSnapsData> build() async {
+    final installedSnaps = await _snapd.getSnaps();
+    final refreshableSnapNames =
+        (await ref.watch(updatesModelProvider.future).onError((_, __) => []))
+            .snapNames;
+    return ManageSnapsData(
+      installedSnaps: installedSnaps,
+      refreshableSnapNames: refreshableSnapNames,
+    );
   }
 }