pr changes

mobile/analysis_options.yaml

@@ -94,6 +94,7 @@ custom_lint:
         - lib/repositories/*_api.repository.dart
         - lib/domain/models/sync/sync_event.model.dart
         - lib/{domain,infrastructure}/**/sync_stream.*
+        - lib/{domain,infrastructure}/**/sync_api.*
         - lib/infrastructure/repositories/*_api.repository.dart
         - lib/infrastructure/utils/*.converter.dart
         # acceptable exceptions for the time being

mobile/lib/domain/interfaces/sync_api.interface.dart

@@ -1,9 +1,8 @@
 import 'package:immich_mobile/domain/models/sync/sync_event.model.dart';
+import 'package:openapi/api.dart';
 
 abstract interface class ISyncApiRepository {
   Future<void> ack(List<String> data);
 
-  Stream<List<SyncEvent>> watchUserSyncEvent();
-
-  Stream<List<SyncEvent>> watchPartnerSyncEvent();
+  Stream<List<SyncEvent>> getSyncEvents(List<SyncRequestType> type);
 }

mobile/lib/domain/services/sync_stream.service.dart

@@ -64,7 +64,8 @@ class SyncStreamService {
 
   Future<void> syncUsers() async {
     _logger.info("Syncing User Changes");
-    _userSyncSubscription = _syncApiRepository.watchUserSyncEvent().listen(
+    _userSyncSubscription =
+        _syncApiRepository.getSyncEvents([SyncRequestType.usersV1]).listen(
       _handleSyncEvents,
       onDone: () {
         _userSyncCompleter.complete();
@@ -77,7 +78,7 @@ class SyncStreamService {
   Future<void> syncPartners() async {
     _logger.info("Syncing Partner Changes");
     _partnerSyncSubscription =
-        _syncApiRepository.watchPartnerSyncEvent().listen(
+        _syncApiRepository.getSyncEvents([SyncRequestType.partnersV1]).listen(
       _handleSyncEvents,
       onDone: () {
         _partnerSyncCompleter.complete();

mobile/lib/infrastructure/repositories/sync_api.repository.dart

@@ -14,13 +14,8 @@ class SyncApiRepository implements ISyncApiRepository {
   SyncApiRepository(this._api);
 
   @override
-  Stream<List<SyncEvent>> watchUserSyncEvent() {
-    return _getSyncStream(SyncStreamDto(types: [SyncRequestType.usersV1]));
-  }
-
-  @override
-  Stream<List<SyncEvent>> watchPartnerSyncEvent() {
-    return _getSyncStream(SyncStreamDto(types: [SyncRequestType.partnersV1]));
+  Stream<List<SyncEvent>> getSyncEvents(List<SyncRequestType> type) {
+    return _getSyncStream(SyncStreamDto(types: type));
   }
 
   @override

mobile/lib/utils/background_sync.dart

@@ -7,37 +7,21 @@ import 'package:immich_mobile/providers/infrastructure/sync_stream.provider.dart
 import 'package:immich_mobile/utils/isolate.dart';
 import 'package:logging/logging.dart';
 
-class _SyncStreamDriver {
-  final _userSyncCache = AsyncCache.ephemeral();
-  final _partnerSyncCache = AsyncCache.ephemeral();
-
-  Future<void> syncUsers() => _userSyncCache.fetch(
-        () async => runInIsolate(
-          (ref) => ref.read(syncStreamServiceProvider).syncUsers(),
-        ),
-      );
-
-  Future<void> syncPartners() => _partnerSyncCache.fetch(
-        () async => runInIsolate(
-          (ref) => ref.read(syncStreamServiceProvider).syncPartners(),
-        ),
-      );
-}
-
 class BackgroundSyncManager {
   final Logger _logger = Logger('BackgroundSyncManager');
   Timer? _timer;
   final Duration _duration;
   // This allows us to keep synching in the background while allowing ondemand syncs
-  final _driver = _SyncStreamDriver();
+  final _userSyncCache = AsyncCache.ephemeral();
+  final _partnerSyncCache = AsyncCache.ephemeral();
 
   BackgroundSyncManager({required Duration duration}) : _duration = duration;
 
   Timer _createTimer() {
     return Timer.periodic(_duration, (timer) async {
       _logger.info('Background sync started');
-      await _driver.syncUsers();
-      await _driver.syncPartners();
+      await syncUsers();
+      await syncPartners();
       _logger.info('Background sync completed');
     });
   }
@@ -53,6 +37,14 @@ class BackgroundSyncManager {
     _timer = null;
   }
 
-  Future<void> syncUsers() => _driver.syncUsers();
-  Future<void> syncPartners() => _driver.syncPartners();
+  Future<void> syncUsers() => _userSyncCache.fetch(
+        () async => runInIsolate(
+          (ref) => ref.read(syncStreamServiceProvider).syncUsers(),
+        ),
+      );
+  Future<void> syncPartners() => _partnerSyncCache.fetch(
+        () async => runInIsolate(
+          (ref) => ref.read(syncStreamServiceProvider).syncPartners(),
+        ),
+      );
 }