chore: fix some review comments

Author: nsbarsukov

libs/notification/src/services/notification.service.ts

@@ -22,6 +22,8 @@ const NOT_SUPPORTED_ERROR$ = throwError(
     () => new Error('Notification API is not supported in your browser'),
 );
 
+const mapToVoid = map(() => undefined);
+
 @Injectable({
     providedIn: 'root',
 })
@@ -86,17 +88,15 @@ export class NotificationService {
         targetNotification: Notification,
         eventName: E,
     ): Observable<{action: string} | void> {
-        const mapToVoid = map(() => undefined);
+        const isTargetNotification = ({timestamp}: {timestamp?: number}) =>
+            timestamp === targetNotification.timestamp;
 
         return this.swRegistration$.pipe(
             switchMap(swRegistration => {
                 if (!swRegistration) {
                     return fromEvent(targetNotification, eventName).pipe(mapToVoid);
                 }
 
-                const isTargetNotification = (notification: {timestamp?: number}) =>
-                    notification.timestamp === targetNotification.timestamp;
-
                 switch (eventName) {
                     case 'click':
                         return this.notificationSwClicks$.pipe(

libs/notification/src/tokens/notification-clicks.ts

@@ -8,7 +8,7 @@ export const NOTIFICATION_SW_CLICKS = new InjectionToken(
         factory: () => {
             const swPush = inject(SwPush, InjectFlags.Optional);
 
-            return swPush && swPush.isEnabled ? swPush.notificationClicks : NEVER;
+            return swPush?.isEnabled ? swPush.notificationClicks : NEVER;
         },
     },
 );

libs/notification/src/tokens/notification-closes.ts

@@ -23,8 +23,8 @@ export const NOTIFICATION_SW_CLOSES = new InjectionToken(
          * return swPush && swPush.isEnabled ? swPush.notificationCloses : NEVER;
          * ```
          */
-        factory: (): Observable<Notification> => {
-            return combineLatest([
+        factory: (): Observable<Notification> =>
+            combineLatest([
                 from(inject(SERVICE_WORKER).getRegistration()),
                 inject(ANIMATION_FRAME),
             ]).pipe(
@@ -37,7 +37,6 @@ export const NOTIFICATION_SW_CLOSES = new InjectionToken(
                 ),
                 zoneOptimized(inject(NgZone)),
                 share(),
-            );
-        },
+            ),
     },
 );