valtech-components 2.0.759 → 2.0.761

package/lib/services/auth/notification-action.service.d.ts

@@ -4,7 +4,6 @@ import { HandoffService } from './handoff.service';
 import { OrgSwitchService } from './org-switch.service';
 import { ValtechAuthConfig } from './types';
 import { NotificationDocument, NotificationsService } from '../firebase/notifications.service';
-import { NotificationClickEvent } from '../firebase/types';
 import * as i0 from "@angular/core";
 /** Resultado descriptivo del open() — útil para tests y telemetría. */
 export type NotificationOpenResult = 'navigated' | 'navigated-after-switch-org' | 'redirected-cross-app' | 'no-action-route' | 'cross-app-unconfigured' | 'handoff-failed';
@@ -24,16 +23,6 @@ export declare class NotificationActionService {
      * técnicos quedan en console.warn.
      */
     open(notif: NotificationDocument): Promise<NotificationOpenResult>;
-    /**
-     * Variante para clicks de FCM push (background SW → postMessage).
-     * Mapea el payload del FCM a la forma de `NotificationDocument` y delega en `open()`.
-     *
-     * Diferencias con clicks de inbox Firestore:
-     *  - No hay `id` Firestore → markAsRead se skipea (no aplica).
-     *  - `appId`/`orgId`/`actionRoute` se extraen de `notification.data` (lo que el
-     *    backend `push.SendPush` puso allí) o de `event.action` (parsed por MessagingService).
-     */
-    openFromFCM(event: NotificationClickEvent): Promise<NotificationOpenResult>;
     private activeOrg;
     /**
      * Construye URL absoluta para handoff cross-app preservando pathname y otros

package/lib/version.d.ts

@@ -2,4 +2,4 @@
  * Current version of valtech-components.
  * This is automatically updated during the publish process.
  */
-export declare const VERSION = "2.0.759";
+export declare const VERSION = "2.0.761";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "valtech-components",
-  "version": "2.0.759",
+  "version": "2.0.761",
   "private": false,
   "bin": {
     "valtech-firebase-config": "./src/lib/services/firebase/scripts/generate-sw-config.js"

package/src/lib/services/firebase/firebase-messaging-sw.js

@@ -96,28 +96,33 @@ if (!self.FIREBASE_CONFIG) {
     event.waitUntil(
       clients
         .matchAll({ type: 'window', includeUncontrolled: true })
-        .then(async (clientList) => {
-          // Estrategia: el SW NO navega. Solo da foco al client y emite postMessage.
-          // La app (MessagingService → NotificationActionService.openFromFCM) hace
-          // routing via Angular Router → SPA navigation + handoff cross-app + switch-org.
-          // `client.navigate()` haría full reload y compite con Router → ambos pierden.
-          //
-          // Solo abrimos nueva ventana (con URL completa) cuando NO hay client abierto;
-          // ese caso es el único donde el SW SÍ define la URL inicial.
-          if (clientList.length > 0) {
-            const client = clientList[0];
-            // postMessage al primer client — el wire de la app (APP_INITIALIZER de
-            // provideValtechAuth) recibe y dispatcha NotificationActionService.
+        .then((clientList) => {
+          // Siempre enviar postMessage para que la app pueda reaccionar
+          for (const client of clientList) {
             client.postMessage(notificationPayload);
-            if ('focus' in client) {
-              await client.focus();
+          }
+
+          // Navegar si hay route o url
+          if (targetUrl !== '/') {
+            for (const client of clientList) {
+              if ('navigate' in client) {
+                return client.navigate(targetUrl).then((c) => c?.focus());
+              }
+            }
+            // Si no hay cliente abierto, abrir nueva ventana
+            if (clients.openWindow) {
+              return clients.openWindow(targetUrl);
             }
-            return;
           }
 
-          // Sin clients abiertos: openWindow con la URL final ya resuelta.
+          // Solo hacer focus si no hay navegación
+          for (const client of clientList) {
+            if ('focus' in client) {
+              return client.focus();
+            }
+          }
           if (clients.openWindow) {
-            return clients.openWindow(targetUrl);
+            return clients.openWindow('/');
           }
         })
     );