fix(notifications): sync background push toggle when OS permission denied

src/app/features/settings/notifications/NativePushNotifications.ts

@@ -69,7 +69,7 @@ function getPushGatewayUrl(clientConfig: ClientConfig): string {
   return pushGatewayUrl;
 }
 
-export async function isNativePushPermissionGranted(): Promise<boolean> {
+export async function isNativePushPermissionGranted(): Promise<boolean | null> {
   const api = await getNativePushNotificationsApi();
   return api.isPermissionGranted();
 }

src/app/features/settings/notifications/NativePushNotificationsApiClient.ts

@@ -7,7 +7,7 @@ export type NativePushRegistration = {
 };
 
 export type NativePushNotificationsApi = {
-  isPermissionGranted: () => Promise<boolean>;
+  isPermissionGranted: () => Promise<boolean | null>;
   requestPermission: () => Promise<NotificationPermission>;
   registerForPushNotifications: (vapid?: string) => Promise<NativePushRegistration>;
   unregisterForPushNotifications: () => Promise<void>;

src/app/features/settings/notifications/NotificationTransportRuntimeFeature.tsx

@@ -7,19 +7,29 @@ import { useClientConfig } from '$hooks/useClientConfig';
 import { useSetting } from '$state/hooks/settings';
 import { settingsAtom } from '$state/settings';
 import { enableUnifiedPush } from './UnifiedPushNotifications';
-import { enableNativePush } from './NativePushNotifications';
+import { enableNativePush, isNativePushPermissionGranted } from './NativePushNotifications';
+import { isUnifiedPushPermissionGranted } from './UnifiedPushTransport';
 import {
   NotificationTransportRuntime,
   type NotificationTransportRuntimeContext,
 } from './NotificationTransportRuntime';
 import {
   type NotificationTransportPlatform,
+  type NotificationTransportProvider,
   normalizeNotificationTransportMode,
   resolvePreferredNotificationTransportProvider,
 } from './NotificationTransport';
 
 const log = createLogger('NotificationTransportRuntimeFeature');
 
+async function checkPushPermission(
+  provider: NotificationTransportProvider | null
+): Promise<boolean | null> {
+  if (provider === 'unifiedpush') return isUnifiedPushPermissionGranted();
+  if (provider === 'native') return isNativePushPermissionGranted();
+  return null;
+}
+
 function currentPlatform(): NotificationTransportPlatform {
   if (!isTauri()) return 'web';
   const platform = osType();
@@ -31,8 +41,14 @@ function currentPlatform(): NotificationTransportPlatform {
 export function NotificationTransportRuntimeFeature() {
   const mx = useMatrixClient();
   const clientConfig = useClientConfig();
-  const [backgroundPushEnabled] = useSetting(settingsAtom, 'backgroundPushEnabled');
-  const [backgroundPushProvider] = useSetting(settingsAtom, 'backgroundPushProvider');
+  const [backgroundPushEnabled, setBackgroundPushEnabled] = useSetting(
+    settingsAtom,
+    'backgroundPushEnabled'
+  );
+  const [backgroundPushProvider, setBackgroundPushProvider] = useSetting(
+    settingsAtom,
+    'backgroundPushProvider'
+  );
   const [pushTransportMode] = useSetting(settingsAtom, 'pushTransportMode');
   const [pushTransportOverride] = useSetting(settingsAtom, 'pushTransportOverride');
   const [useInAppNotifications] = useSetting(settingsAtom, 'useInAppNotifications');
@@ -107,26 +123,33 @@ export function NotificationTransportRuntimeFeature() {
   useEffect(() => {
     if (!mx) return undefined;
 
-    if (provider === 'unifiedpush') {
-      enableUnifiedPush(mx, upConfigRef.current)
-        .then((result) => {
+    void (async () => {
+      const granted = await checkPushPermission(provider);
+      if (granted === false) {
+        setBackgroundPushEnabled(false);
+        setBackgroundPushProvider(null);
+        return;
+      }
+
+      try {
+        if (provider === 'unifiedpush') {
+          const result = await enableUnifiedPush(mx, upConfigRef.current);
           lastEndpointRef.current = result.endpoint;
-        })
-        .catch((error) => {
-          log.warn('UnifiedPush pusher registration failed at startup', error);
-        });
-    } else if (provider === 'native') {
-      enableNativePush(mx, clientConfigRef.current)
-        .then((token) => {
+        } else if (provider === 'native') {
+          const token = await enableNativePush(mx, clientConfigRef.current);
           lastEndpointRef.current = token;
-        })
-        .catch((error) => {
-          log.warn('Native push pusher registration failed at startup', error);
-        });
-    }
+        }
+      } catch (error) {
+        log.warn('Pusher registration failed at startup', error);
+        if ((await checkPushPermission(provider)) === false) {
+          setBackgroundPushEnabled(false);
+          setBackgroundPushProvider(null);
+        }
+      }
+    })();
 
     return () => {};
-  }, [provider, mx]);
+  }, [provider, mx, setBackgroundPushEnabled, setBackgroundPushProvider]);
 
   useEffect(
     () => () => {

src/app/features/settings/notifications/UnifiedPushTransport.ts

@@ -110,6 +110,11 @@ async function getUnifiedPushPermissionState(): Promise<UnifiedPushPermissionSta
   return permission === 'granted' ? 'granted' : permission;
 }
 
+export async function isUnifiedPushPermissionGranted(): Promise<boolean | null> {
+  const api = await getUnifiedPushTransportApi();
+  return api.isPermissionGranted();
+}
+
 export async function getUnifiedPushDistributors(): Promise<string[]> {
   const api = await getUnifiedPushTransportApi();
   return api.listDistributors();

src/app/features/settings/notifications/UnifiedPushTransportApiClient.ts

@@ -7,7 +7,7 @@ export type UnifiedPushRegistration = {
 };
 
 export type UnifiedPushTransportApi = {
-  isPermissionGranted: () => Promise<boolean>;
+  isPermissionGranted: () => Promise<boolean | null>;
   requestPermission: () => Promise<NotificationPermission>;
   registerForPushNotifications: (vapid?: string) => Promise<UnifiedPushRegistration>;
   unregisterForPushNotifications: () => Promise<void>;

src/types/tauri-plugin-notifications-api.d.ts

@@ -62,7 +62,7 @@ declare module '@choochmeque/tauri-plugin-notifications-api' {
     unregister: () => Promise<void> | void;
   };
 
-  export function isPermissionGranted(): Promise<boolean>;
+  export function isPermissionGranted(): Promise<boolean | null>;
   export function requestPermission(): Promise<NotificationPermission>;
   export function sendNotification(options: NotificationOptions | string): Promise<void>;
   export function registerForPushNotifications(): Promise<string>;