@elliemae/ssf-host 2.22.0 → 2.23.0

package/dist/cjs/guest.js

@@ -142,9 +142,22 @@ class Guest {
   handShake = () => new Promise((resolve) => {
     let handShakeCount = 0;
     const handShakeRetries = 5;
-    const handShakeHandle = setInterval(() => {
+    let handShakeHandle;
+    const onAck = () => {
+      clearInterval(handShakeHandle);
+      this.#remoting.unlisten({
+        messageType: import_microfe_common.MessageType.HandShakeAck,
+        callback: onAck
+      });
+      resolve(true);
+    };
+    handShakeHandle = setInterval(() => {
       if (handShakeCount >= handShakeRetries) {
         clearInterval(handShakeHandle);
+        this.#remoting.unlisten({
+          messageType: import_microfe_common.MessageType.HandShakeAck,
+          callback: onAck
+        });
         resolve(false);
       } else {
         this.#remoting.send({
@@ -158,10 +171,7 @@ class Guest {
     }, 1e3);
     this.#remoting.listen({
       messageType: import_microfe_common.MessageType.HandShakeAck,
-      callback: () => {
-        clearInterval(handShakeHandle);
-        resolve(true);
-      }
+      callback: onAck
     });
   });
   /**

package/dist/cjs/host.js

@@ -61,6 +61,14 @@ class SSFHost {
    * list of guests
    */
   #guests = /* @__PURE__ */ new Map();
+  /**
+   * reverse lookup: window -> guest for O(1) message routing
+   */
+  #guestsByWindow = /* @__PURE__ */ new Map();
+  /**
+   * reverse lookup: url -> guest for O(1) popup dedup
+   */
+  #guestsByUrl = /* @__PURE__ */ new Map();
   /**
    * list of callbacks for guest close.
    */
@@ -107,50 +115,46 @@ class SSFHost {
     this.#remoting.initialize(window);
     this.#connect();
     window.addEventListener("beforeunload", this.#closeAllPopupGuests);
-    this.#monitorPopupGuests();
     this.#logger.debug(
       `host is initialized. hostId: ${this.hostId}, correlationId: ${this.#correlationId}`
     );
   }
   #sendBAEvent = (event, data) => {
     const baEvent = { event, ...data };
-    this.#analyticsObj.sendBAEvent(baEvent).catch(() => {
+    this.#analyticsObj.sendBAEvent(baEvent).catch((e) => {
+      this.#logger.debug(
+        `Analytics sendBAEvent failed: ${e.message}`
+      );
     });
   };
   #startTiming = (name, options) => {
-    this.#analyticsObj.startTiming(name, options).catch(() => {
+    this.#analyticsObj.startTiming(name, options).catch((e) => {
+      this.#logger.debug(
+        `Analytics startTiming failed: ${e.message}`
+      );
     });
   };
   #endTiming = (start, options) => {
-    this.#analyticsObj.endTiming(start, options).catch(() => {
+    this.#analyticsObj.endTiming(start, options).catch((e) => {
+      this.#logger.debug(`Analytics endTiming failed: ${e.message}`);
     });
   };
   #closeAllPopupGuests = () => {
-    for (const guest of this.#guests.values()) {
-      if (guest.openMode === import_types.OpenMode.Popup) {
-        this.unloadGuest(guest.id);
-      }
-    }
+    const popupIds = Array.from(this.#guests.values()).filter((guest) => guest.openMode === import_types.OpenMode.Popup).map((guest) => guest.id);
+    popupIds.forEach((id) => this.unloadGuest(id));
   };
   /**
    * get the reference to the guest application by its window
    * @param guestWindow reference to the guest window
    * @returns reference to the guest
    */
-  #getGuestForWindow = (guestWindow) => Array.from(this.#guests.values()).find(
-    (guest) => guest.window === guestWindow
-  );
+  #getGuestForWindow = (guestWindow) => this.#guestsByWindow.get(guestWindow);
   /**
    * get the reference to the guest application by its window
    * @param url url of the guest application
    * @returns reference to the guest
    */
-  #getGuestForUrl = (url) => {
-    for (const guest of this.#guests.values()) {
-      if (guest.url === url) return guest;
-    }
-    return null;
-  };
+  #getGuestForUrl = (url) => this.#guestsByUrl.get(url) ?? null;
   /**
    * check if a object is a scripting object
    * @param value javascript object
@@ -422,6 +426,9 @@ class SSFHost {
           token
         });
       } catch (error) {
+        this.#logger.warn(
+          `Error in onGuestEventSubscribe callback for event ${eventId}: ${error.message}`
+        );
       }
     }, 0);
   };
@@ -465,6 +472,9 @@ class SSFHost {
           token
         });
       } catch (error) {
+        this.#logger.warn(
+          `Error in onGuestEventUnsubscribe callback for event ${eventId}: ${error.message}`
+        );
       }
     }, 0);
   };
@@ -639,6 +649,8 @@ class SSFHost {
     });
     guest.init();
     this.#guests.set(param.guestId, guest);
+    this.#guestsByWindow.set(guest.window, guest);
+    this.#guestsByUrl.set(guest.url, guest);
     return guest;
   };
   /**
@@ -647,15 +659,17 @@ class SSFHost {
    * @returns guest object
    */
   #findGuest = (guestIdOrWindowOrEle) => {
-    let guest = typeof guestIdOrWindowOrEle === "string" ? this.#guests.get(guestIdOrWindowOrEle) : null;
-    if (!guest) {
-      guest = Array.from(this.#guests.values()).find(
-        (value) => value.window === guestIdOrWindowOrEle || value.domElement === guestIdOrWindowOrEle
-      );
+    if (typeof guestIdOrWindowOrEle === "string") {
+      return this.#guests.get(guestIdOrWindowOrEle);
     }
-    return guest;
+    const byWindow = this.#guestsByWindow.get(guestIdOrWindowOrEle);
+    if (byWindow) return byWindow;
+    return Array.from(this.#guests.values()).find(
+      (value) => value.domElement === guestIdOrWindowOrEle
+    );
   };
   #monitorPopupGuests = () => {
+    if (this.#popupGuestMonitor) return;
     this.#popupGuestMonitor = setInterval(() => {
       const guestsToRemove = [];
       this.#guests.forEach((guest) => {
@@ -668,12 +682,22 @@ class SSFHost {
         this.unloadGuest(id);
         const callbacks = this.#guestCloseCallbackList.get(id);
         callbacks?.forEach((callback) => {
-          callback({ id }).catch(() => {
+          Promise.resolve(callback({ id })).catch(() => {
           });
         });
       });
     }, 1e3);
   };
+  #stopMonitoringPopupGuests = () => {
+    if (!this.#popupGuestMonitor) return;
+    const hasPopups = Array.from(this.#guests.values()).some(
+      (guest) => guest.openMode === import_types.OpenMode.Popup
+    );
+    if (!hasPopups) {
+      clearInterval(this.#popupGuestMonitor);
+      this.#popupGuestMonitor = null;
+    }
+  };
   #openPopupGuest = (param) => {
     const {
       url,
@@ -699,37 +723,29 @@ class SSFHost {
         });
       }
     } else {
-      const windowFeatures = [
-        { key: "width", value: width },
-        { key: "height", value: height },
-        { key: "top", value: top },
-        { key: "left", value: left }
-      ].reduce((acc, cur, curIndex) => {
-        if (curIndex > 0 && cur.value) acc += ",";
-        return cur.value ? `${acc}${cur.key}=${cur.value}` : acc;
-      }, "");
+      const windowFeatures = Object.entries({ width, height, top, left }).filter(([, v]) => v).map(([k, v]) => `${k}=${v}`).join(",");
       const guestWindow = window.open(url, title, `popup, ${windowFeatures}`);
       if (!guestWindow) {
-        try {
-          setTimeout(() => {
+        setTimeout(() => {
+          try {
             onError?.(guestId);
-          }, 0);
-        } catch (error) {
-          this.#logger.debug(
-            `Error occurred in onError for guest with id '${guestId}': ${error.message}`
-          );
-        }
+          } catch (error) {
+            this.#logger.debug(
+              `Error occurred in onError for guest with id '${guestId}': ${error.message}`
+            );
+          }
+        }, 0);
         throw new Error("Failed to open guest application in popup window");
       } else {
-        try {
-          setTimeout(() => {
+        setTimeout(() => {
+          try {
             onLoad?.(guestId);
-          }, 0);
-        } catch (error) {
-          this.#logger.debug(
-            `Error occurred in onLoad for guest with id '${guestId}': ${error.message}`
-          );
-        }
+          } catch (error) {
+            this.#logger.debug(
+              `Error occurred in onLoad for guest with id '${guestId}': ${error.message}`
+            );
+          }
+        }, 0);
       }
       guestWindow.opener = null;
       guest = this.#attachGuest({
@@ -740,6 +756,7 @@ class SSFHost {
         searchParams,
         openMode: import_types.OpenMode.Popup
       });
+      this.#monitorPopupGuests();
     }
     return guest;
   };
@@ -926,7 +943,10 @@ class SSFHost {
    * dispose the resources used by the host application
    */
   close = () => {
-    clearInterval(this.#popupGuestMonitor);
+    if (this.#popupGuestMonitor) {
+      clearInterval(this.#popupGuestMonitor);
+      this.#popupGuestMonitor = null;
+    }
     this.#closeAllPopupGuests();
     this.#remoting.close();
     window.removeEventListener("beforeunload", this.#closeAllPopupGuests);
@@ -979,39 +999,42 @@ class SSFHost {
     }
     const guestPromises = [];
     let timingMetricStarted = false;
-    this.#guests.forEach((guest) => {
+    const dispatchToGuest = (guest) => {
       const guestInfo = guest.getInfo();
-      if (!targetWindow || targetWindow === guest.window) {
-        if (timeout && guest?.capabilities?.eventFeedback) {
-          guestPromises.push(guest.dispatchEvent(eventObj, timeout));
-          if (!timingMetricStarted) {
-            this.#startTiming(
-              `ScriptingObject.Event.${scriptingObject.id}.${name}`,
-              {
-                appId: this.hostId,
-                appUrl: window.location.href
-              }
-            );
-            timingMetricStarted = true;
-          }
-          this.#logger.audit({
-            message: "Event dispatched and awaiting feedback",
-            scriptingEventId: id,
-            ...guestInfo
-          });
-        } else {
-          guest.send({
-            messageType: import_microfe_common.MessageType.ObjectEvent,
-            messageBody: eventObj
-          });
-          this.#logger.audit({
-            message: "Event dispatched",
-            scriptingEventId: id,
-            ...guestInfo
-          });
+      if (timeout && guest?.capabilities?.eventFeedback) {
+        guestPromises.push(guest.dispatchEvent(eventObj, timeout));
+        if (!timingMetricStarted) {
+          this.#startTiming(
+            `ScriptingObject.Event.${scriptingObject.id}.${name}`,
+            {
+              appId: this.hostId,
+              appUrl: window.location.href
+            }
+          );
+          timingMetricStarted = true;
         }
+        this.#logger.audit({
+          message: "Event dispatched and awaiting feedback",
+          scriptingEventId: id,
+          ...guestInfo
+        });
+      } else {
+        guest.send({
+          messageType: import_microfe_common.MessageType.ObjectEvent,
+          messageBody: eventObj
+        });
+        this.#logger.audit({
+          message: "Event dispatched",
+          scriptingEventId: id,
+          ...guestInfo
+        });
       }
-    });
+    };
+    if (targetGuest) {
+      dispatchToGuest(targetGuest);
+    } else {
+      this.#guests.forEach(dispatchToGuest);
+    }
     const retValue = await Promise.all(guestPromises).then((values) => {
       this.#logger.audit({
         message: "Event feedback received",
@@ -1041,13 +1064,7 @@ class SSFHost {
    * get reference to all guest applications
    * @returns list of guest application references
    */
-  getGuests = () => {
-    const guestList = [];
-    this.#guests.forEach((guest) => {
-      guestList.push(guest);
-    });
-    return guestList;
-  };
+  getGuests = () => Array.from(this.#guests.values());
   /**
    * get the scripting object by id
    * @param objectId - id of the scripting object
@@ -1072,6 +1089,12 @@ class SSFHost {
       options = {}
     } = param;
     if (!guestId) throw new Error("id for guest application is required");
+    if (this.#guests.has(guestId)) {
+      this.#logger.warn(
+        `Guest with id '${guestId}' is already loaded. Unloading existing guest first.`
+      );
+      this.unloadGuest(guestId);
+    }
     const { openMode = import_types.OpenMode.Embed, popupWindowFeatures = {} } = options;
     const srcUrl = this.#getGuestUrl(url, searchParams);
     let guest = null;
@@ -1166,11 +1189,14 @@ class SSFHost {
     const guest = this.#findGuest(guestIdOrWindowOrEle);
     if (guest) {
       guest.dispose();
+      this.#guestsByWindow.delete(guest.window);
+      this.#guestsByUrl.delete(guest.url);
       this.#guests.delete(guest.id);
       this.#logger.audit({
         message: `Guest is removed from host`,
         ...guest.getInfo()
       });
+      this.#stopMonitoringPopupGuests();
     }
   };
   /**

package/dist/cjs/utils.js

@@ -19,10 +19,8 @@ var __toCommonJS = (mod) => __copyProps(__defProp({}, "__esModule", { value: tru
 var utils_exports = {};
 __export(utils_exports, {
   flatten: () => flatten,
-  getObjectId: () => getObjectId,
   getOrigin: () => getOrigin,
-  isFunction: () => isFunction,
-  isScriptingObject: () => isScriptingObject
+  isFunction: () => isFunction
 });
 module.exports = __toCommonJS(utils_exports);
 const getOrigin = (url) => {
@@ -35,22 +33,7 @@ const getOrigin = (url) => {
     return origin;
   }
 };
-const flatten = (source, target = []) => {
-  const retVal = target || [];
-  if (source && source.forEach) {
-    source.forEach((item) => {
-      flatten(item, retVal);
-    });
-  } else if (typeof source !== "undefined") {
-    retVal.push(source);
-  }
-  return retVal;
-};
-const isScriptingObject = (value) => (
-  // eslint-disable-next-line no-underscore-dangle, @typescript-eslint/no-unsafe-member-access
-  typeof value?._toJSON === "function"
-);
+const flatten = (source) => source.flat(Infinity).filter((v) => v !== void 0);
 function isFunction(value) {
   return typeof value === "function";
 }
-const getObjectId = (elementOrId) => elementOrId?.id ?? elementOrId;

package/dist/esm/guest.js

@@ -119,9 +119,22 @@ class Guest {
   handShake = () => new Promise((resolve) => {
     let handShakeCount = 0;
     const handShakeRetries = 5;
-    const handShakeHandle = setInterval(() => {
+    let handShakeHandle;
+    const onAck = () => {
+      clearInterval(handShakeHandle);
+      this.#remoting.unlisten({
+        messageType: MessageType.HandShakeAck,
+        callback: onAck
+      });
+      resolve(true);
+    };
+    handShakeHandle = setInterval(() => {
       if (handShakeCount >= handShakeRetries) {
         clearInterval(handShakeHandle);
+        this.#remoting.unlisten({
+          messageType: MessageType.HandShakeAck,
+          callback: onAck
+        });
         resolve(false);
       } else {
         this.#remoting.send({
@@ -135,10 +148,7 @@ class Guest {
     }, 1e3);
     this.#remoting.listen({
       messageType: MessageType.HandShakeAck,
-      callback: () => {
-        clearInterval(handShakeHandle);
-        resolve(true);
-      }
+      callback: onAck
     });
   });
   /**