fix(neon_framework): Correctly handle disappearing apps and the notif…

…ications app

Signed-off-by: jld3103 <[email protected]>

packages/neon_framework/lib/src/blocs/apps.dart

@@ -61,27 +61,10 @@ class AppsBloc extends InteractiveBloc implements AppsBlocEvents, AppsBlocStates
     _apps.listen((final result) {
       appImplementations
           .add(result.transform((final data) => _filteredAppImplementations(data.map((final a) => a.id))));
-    });
-
-    appImplementations.listen((final result) async {
-      if (!result.hasData) {
-        return;
-      }
 
-      // dispose unsupported apps
-      for (final app in _allAppImplementations) {
-        if (result.requireData.tryFind(app.id) == null) {
-          app.blocsCache.remove(_account);
-        }
+      if (result.hasData) {
+        unawaited(_updateApps());
       }
-
-      final options = _accountsBloc.getOptionsFor(_account);
-      final initialApp = options.initialApp.value ?? _getInitialAppFallback();
-      if (initialApp != null) {
-        await setActiveApp(initialApp, skipAlreadySet: true);
-      }
-
-      unawaited(_checkCompatibility());
     });
 
     _capabilitiesBloc.capabilities.listen((final result) {
@@ -93,12 +76,41 @@ class AppsBloc extends InteractiveBloc implements AppsBlocEvents, AppsBlocStates
         ),
       );
 
-      unawaited(_checkCompatibility());
+      if (result.hasData) {
+        unawaited(_updateApps());
+      }
     });
 
     unawaited(refresh());
   }
 
+  Future<void> _updateApps() async {
+    // dispose unsupported apps
+    for (final app in _allAppImplementations) {
+      if (app.id == AppIDs.notifications) {
+        if (notificationsAppImplementation.hasValue &&
+            !notificationsAppImplementation.value.isCached &&
+            notificationsAppImplementation.value.data == null) {
+          app.blocsCache.remove(_account);
+        }
+        continue;
+      }
+
+      if (appImplementations.hasValue &&
+          !appImplementations.value.isCached &&
+          appImplementations.value.data?.tryFind(app.id) == null) {
+        app.blocsCache.remove(_account);
+      }
+    }
+
+    final initialApp = _getInitialAppFallback();
+    if (initialApp != null) {
+      await setActiveApp(initialApp);
+    }
+
+    await _checkCompatibility();
+  }
+
   /// Determines the appid of initial app.
   ///
   /// It requires [appImplementations] to have both a value and data.
@@ -108,9 +120,13 @@ class AppsBloc extends InteractiveBloc implements AppsBlocEvents, AppsBlocStates
   ///
   /// Returns null when no app is supported by the server.
   String? _getInitialAppFallback() {
-    final supportedApps = appImplementations.value.requireData;
+    final supportedApps = appImplementations.valueOrNull?.data;
+    if (supportedApps == null) {
+      return null;
+    }
 
-    for (final fallback in {AppIDs.dashboard, AppIDs.files}) {
+    final options = _accountsBloc.getOptionsFor(_account);
+    for (final fallback in {options.initialApp.value, AppIDs.dashboard, AppIDs.files}) {
       if (supportedApps.tryFind(fallback) != null) {
         return fallback;
       }