@magicfeedback/popup-sdk 1.7.2 → 1.8.1

package/dist/index.mjs

@@ -1,4 +1,5 @@
 import {
+  BUSY_JS,
   InMemoryStorage,
   LABELS,
   PRIMARY_COLOR_JS,
@@ -20,7 +21,7 @@ import {
   sdkWarn,
   setLogger,
   style_default
-} from "./chunk-XHR3MHJO.mjs";
+} from "./chunk-AV2TTQXB.mjs";
 
 // src/types/index.ts
 var POPUP_TRIGGER_CONDITION_STATUSES = [
@@ -429,6 +430,7 @@ ${customCss}
   var footer=document.getElementById('dd-footer');
   var spinner=document.getElementById('dd-spinner');
   var formHost=document.getElementById('mf');
+  var formWrapper=document.getElementById('dd-form-wrapper');
   var errorHint=document.getElementById('dd-error');
   var backBtn=document.getElementById('dd-back');
   var startBtn=document.getElementById('dd-start');
@@ -588,6 +590,7 @@ ${customCss}
   }
 
 ${PRIMARY_COLOR_JS}
+${BUSY_JS}
 ${REVEAL_JS}
   // El popup se ense\xF1a cuando el survey est\xE1 pintado; \`ready\` deja al host abrir su propio
   // Modal en ese momento en vez de montar un WebView en blanco.
@@ -598,6 +601,11 @@ ${REVEAL_JS}
 
   function setLoading(isLoading){
     spinner.style.display=isLoading?'flex':'none';
+    // El spinner no tapa el survey (28px en posicion absoluta), asi que sin esto el usuario
+    // puede seguir cambiando de opcion mientras se envia la pagina, y ese cambio ya no viaja
+    // con ella. Se bloquea el wrapper, que es el elemento estable: el div de dentro lo
+    // sustituye @magicfeedback/native por su propio contenedor al generar el survey.
+    ddSetSurveyBusy(formWrapper, isLoading);
     if(!isLoading){
       formHost.style.visibility='visible';
       // Fin de la carga, sea por survey cargado o por error. Entre p\xE1ginas es no-op: la
@@ -860,7 +868,7 @@ function createReactNativeRenderer() {
 }
 
 // src/ui/preload.ts
-var defaultLoader = () => import("./renderPopup-5QXL6EJW.mjs");
+var defaultLoader = () => import("./renderPopup-3CZ6WTXF.mjs");
 var pending = null;
 function preloadRenderPopup(loader = defaultLoader) {
   if (pending) return pending;
@@ -933,7 +941,7 @@ var BrowserPopupRenderer = class {
     if (!this.container) return;
     this.visible = true;
     const container = this.container;
-    void import("./renderPopup-5QXL6EJW.mjs").then(({ renderPopup }) => {
+    void import("./renderPopup-3CZ6WTXF.mjs").then(({ renderPopup }) => {
       renderPopup(container, surveyId, productId, actions, emit, onClose, env, userId, style, sessionId, miniService, analyticsFeedbackSessionId, options);
     });
   }
@@ -2371,6 +2379,9 @@ var DeepdotsPopups = class {
     this.emitEvent("popup_shown", def.surveyId, { popupId: def.id });
   }
   shouldShow(def, pathUrl, skipPathCheck = false) {
+    if (this.isPathExcluded(def, pathUrl)) {
+      return false;
+    }
     if (!skipPathCheck && !this.matchesSegmentsPath(def, pathUrl)) {
       return false;
     }
@@ -2401,40 +2412,85 @@ var DeepdotsPopups = class {
     }
     return matches;
   }
+  /** `segments.path`: rutas donde el popup PUEDE mostrarse. Sin lista, en todas. */
   matchesSegmentsPath(def, pathUrl) {
     const paths = def.segments?.path;
-    if (!paths || paths.length === 0) return true;
-    if (typeof window === "undefined" || !window.location) {
+    if (!Array.isArray(paths) || paths.length === 0) return true;
+    const location = this.resolveCurrentLocation(pathUrl);
+    if (!location) {
       this.debug("No window.location available for path comparison", { popupId: def.id, paths });
       return true;
     }
-    const normalizedHref = this.normalizeUrl(pathUrl || window.location.href || "");
-    const currentUrl = this.safeParseUrl(normalizedHref);
-    const normalizedPath = this.normalizeUrl(currentUrl?.pathname || window.location.pathname || "");
     const matches = paths.some((rawCandidate) => {
-      const candidate = this.normalizeUrl(rawCandidate);
-      let match = false;
-      if (candidate.startsWith("http://") || candidate.startsWith("https://")) {
-        match = normalizedHref === candidate;
-      } else if (candidate.startsWith("/")) {
-        match = normalizedHref.includes(candidate);
-      } else {
-        match = normalizedPath === candidate;
-      }
+      const match = this.matchesPathCandidate(rawCandidate, location);
       this.debug("Path comparison", {
         popupId: def.id,
-        candidate,
-        currentPath: normalizedPath,
-        currentHref: normalizedHref,
+        candidate: rawCandidate,
+        currentPath: location.path,
+        currentHref: location.href,
         match
       });
       return match;
     });
     if (!matches) {
-      this.debug("No path match for popup", { popupId: def.id, paths, currentPath: normalizedPath });
+      this.debug("No path match for popup", { popupId: def.id, paths, currentPath: location.path });
     }
     return matches;
   }
+  /**
+   * `segments.excludedPaths`: rutas donde el popup NO debe mostrarse. Gana sobre
+   * `segments.path` (`path: ['/']` + `excludedPaths: ['/cart']` = en todas menos el
+   * carrito) y se aplica también cuando no hay `path` (= en todas menos las excluidas).
+   */
+  isPathExcluded(def, pathUrl) {
+    const excluded = def.segments?.excludedPaths;
+    if (!Array.isArray(excluded) || excluded.length === 0) return false;
+    const location = this.resolveCurrentLocation(pathUrl);
+    if (!location) {
+      this.debug("No window.location available for excluded path comparison", { popupId: def.id, excluded });
+      return false;
+    }
+    const hit = excluded.find((rawCandidate) => this.matchesPathCandidate(rawCandidate, location));
+    if (hit !== void 0) {
+      this.debug("Popup excluded by path", {
+        popupId: def.id,
+        candidate: hit,
+        currentPath: location.path,
+        currentHref: location.href
+      });
+      return true;
+    }
+    return false;
+  }
+  /**
+   * Compara UN candidato de `segments.path`/`segments.excludedPaths` con la ruta
+   * actual. Mismas reglas para incluir y para excluir:
+   *  - URL absoluta  → href completo exacto
+   *  - empieza por / → subcadena del href (así entran las rutas con hash: `/#/home`)
+   *  - resto         → pathname exacto
+   * Ignora entradas que no sean string: el backend puede colar un `null` en la lista
+   * y no puede tumbar la evaluación del trigger entero (mismo fallo que hubo con
+   * `segments.lang`).
+   */
+  matchesPathCandidate(rawCandidate, location) {
+    if (typeof rawCandidate !== "string") return false;
+    const candidate = this.normalizeUrl(rawCandidate);
+    if (!candidate) return false;
+    if (candidate.startsWith("http://") || candidate.startsWith("https://")) {
+      return location.href === candidate;
+    }
+    if (candidate.startsWith("/")) {
+      return location.href.includes(candidate);
+    }
+    return location.path === candidate;
+  }
+  /** href + pathname normalizados de la ruta a evaluar, o null si no hay `window.location`. */
+  resolveCurrentLocation(pathUrl) {
+    if (typeof window === "undefined" || !window.location) return null;
+    const href = this.normalizeUrl(pathUrl || window.location.href || "");
+    const parsed = this.safeParseUrl(href);
+    return { href, path: this.normalizeUrl(parsed?.pathname || window.location.pathname || "") };
+  }
   evaluateCooldown(def, condition) {
     switch (condition.answered) {
       case "SHOWED":
@@ -2731,7 +2787,7 @@ var DeepdotsPopups = class {
     }
     if (!this.popupContainer) return;
     const container = this.popupContainer;
-    void import("./renderPopup-5QXL6EJW.mjs").then(({ renderPopup }) => {
+    void import("./renderPopup-3CZ6WTXF.mjs").then(({ renderPopup }) => {
       renderPopup(
         container,
         surveyId,