@betarena/ad-engine 0.3.1 → 0.3.3

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@betarena/ad-engine",
-	"version": "0.3.1",
+	"version": "0.3.3",
 	"private": false,
 	"description": "Betarena ad-engine widget",
 	"keywords": [
@@ -88,7 +88,7 @@
 		"vite": "5.1.6"
 	},
 	"dependencies": {
-		"@betarena/scores-lib": "4.1.0",
+		"@betarena/scores-lib": "4.14.11",
 		"@fontsource/roboto": "5.0.12",
 		"chalk": "5.3.0",
 		"colorthief": "2.6.0",

package/src/lib/Advert-Engine-Widget.svelte

@@ -106,7 +106,14 @@
      * @description
      * 📝 Condition for testing
      */
-    isStandalone = true
+    isStandalone = true,
+    /**
+     * @description
+     * 📝 Signal from the host that the current page is an article page.
+     * Zone 1 body-mounted ads (slider/popup) must only inject when this is `true`.
+     * Defaults to `false` so the widget fails closed on non-article pages.
+     */
+    isArticlePage: boolean = false
   ;
 
   // Retained as a supported public prop for API compatibility.
@@ -587,7 +594,7 @@
     // │ |:   (not from DOM-filtered targetZoneIds) so global placements inject correctly
     // │ |:   even when no zone-1 element exists in the DOM.
     // ╰─────
-    if (includesGlobalZone && (authorId || authorArticleTagIds.length > 0))
+    if (includesGlobalZone && isArticlePage && (authorId || authorArticleTagIds.length > 0))
     {
       // ╭─────
       // │ NOTE:
@@ -764,7 +771,7 @@
     // │ |: Inject STANDARD SLIDER adverts in 'document.body'
     // │ |: Guard with `includesGlobalZone` to avoid body injections on scoped refreshes.
     // ╰─────
-    else if (includesGlobalZone)
+    else if (includesGlobalZone && isArticlePage)
     {
       // [🐞]
       logger
@@ -777,11 +784,25 @@
 
       // ╭─────
       // │ NOTE:
-      // │ |: Loop over creative data and inject adverts
+      // │ |: Loop over creative data and inject adverts.
+      // │ |: Build a Set of creative IDs that belong to zone 1 so we only
+      // │ |:   inject zone-1-specific creatives rather than all slider-type creatives.
       // ╰─────
-      for (const [, adData] of mapCreative)
+      const
+        zone1CampaignIds
+          = mapZoneIdToCampaignId.get(1) ?? [],
+        zone1CreativeIds
+          = new Set
+          (
+            zone1CampaignIds
+              .flatMap(campaignId => mapCampaignIdToCreativeId.get(campaignId) ?? [])
+          )
+      ;
+
+      for (const [creativeId, adData] of mapCreative)
       {
         if (adData.type != 1) continue;
+        if (!zone1CreativeIds.has(creativeId)) continue;
 
         listAdWidgetElements.push
         (
@@ -886,6 +907,25 @@
       targetZoneIds = targetZoneIds.filter(id => opts.zoneIds!.includes(id));
     ;
 
+    // ╭─────
+    // │ NOTE:
+    // │ |: Zone 1 (global slider/popup) has no DOM element and will never appear
+    // │ |:   in the DOM scan. On article pages it must still be requested so the
+    // │ |:   backend returns zone-1 campaigns/creatives.
+    // │ |:   Zone 1 placement has empty authors/tags — it is globally available on
+    // │ |:   article pages; creative-level targeting handles device/country/author filtering.
+    // │ |:   Respect scoped refreshes: only add zone 1 if the caller's scope includes it.
+    // ╰─────
+    const
+      shouldRequestZone1
+        = isArticlePage
+        && (!opts?.zoneIds || opts.zoneIds.length === 0 || opts.zoneIds.includes(1))
+    ;
+
+    if (shouldRequestZone1 && !targetZoneIds.includes(1))
+      targetZoneIds.push(1);
+    ;
+
     // ╭─────
     // │ NOTE:
     // │ |: Only tear down previously injected global/body components when the