@namiml/sdk-core 3.4.12-dev.202610062108 → 3.4.12-dev.202610081956

package/dist/index.cjs

@@ -106,7 +106,7 @@ const {
 // version — stamped by scripts/version.sh
 NAMI_SDK_VERSION: exports.NAMI_SDK_VERSION = "3.4.12", 
 // full package version including dev suffix — stamped by scripts/version.sh
-NAMI_SDK_PACKAGE_VERSION: exports.NAMI_SDK_PACKAGE_VERSION = "3.4.12-dev.202610062108", 
+NAMI_SDK_PACKAGE_VERSION: exports.NAMI_SDK_PACKAGE_VERSION = "3.4.12-dev.202610081956", 
 // environments
 PRODUCTION: exports.PRODUCTION = "production", DEVELOPMENT: exports.DEVELOPMENT = "development", 
 // error messages
@@ -65607,8 +65607,32 @@ class NamiFlow extends BasicNamiFlow {
             this.purchasedSku = normalizePurchasedSku(data.sku);
         }
     }
-    triggerActions(actionId, component, data) {
+    /**
+     * True when a tap belongs to a screen the flow has already left.
+     *
+     * A button is dispatched by the screen that rendered it, but {@link lifecycles} resolves
+     * against whatever step is current when the tap is PROCESSED. A screen that advances itself
+     * — `__appear__` authoring `after: flowNav` — moves the step out from under an in-flight tap,
+     * so the outgoing screen's button would run the INCOMING step's actions for the same button
+     * id. Observed live on NFL Vizio: "skip intro" on the Welcome screen ran the next screen's
+     * `flowExit` instead of its own `flowNav`, and the analytics it emitted carried the old
+     * button's `{{ buttonText }}` against the new screen's `{{ pageId }}` (NAM-4395).
+     *
+     * Only drops when BOTH ids are known and differ: a caller that supplies no screen id, or a
+     * step with no screen of its own (entry/branch/exit), keeps the previous behaviour.
+     */
+    isStaleTap(actionId, screenId) {
+        const currentScreen = this.currentFlowStep?.screen;
+        if (!screenId || !currentScreen || screenId === currentScreen)
+            return false;
+        this.flowLog(`dropping stale tap ${actionId} from screen ${screenId} — the flow is on ${currentScreen} (NAM-4395)`);
+        logger.info(`Ignoring "${actionId}" from a screen the flow has already left (${screenId} -> ${currentScreen})`);
+        return true;
+    }
+    triggerActions(actionId, component, data, screenId) {
         this.captureBuySkuData(actionId, data);
+        if (this.isStaleTap(actionId, screenId))
+            return;
         const lifecycles = this.lifecycles(actionId);
         if (!lifecycles)
             return;
@@ -65639,8 +65663,10 @@ class NamiFlow extends BasicNamiFlow {
             }));
         }
     }
-    triggerBeforeActions(actionId, component, data) {
+    triggerBeforeActions(actionId, component, data, screenId) {
         this.captureBuySkuData(actionId, data);
+        if (this.isStaleTap(actionId, screenId))
+            return;
         const lifecycles = this.lifecycles(actionId);
         if (!lifecycles)
             return;
@@ -65649,8 +65675,10 @@ class NamiFlow extends BasicNamiFlow {
             cycle.before?.forEach((action) => this.shouldRun(action) && this.performAction(action, data));
         }
     }
-    triggerAfterActions(actionId, component, data) {
+    triggerAfterActions(actionId, component, data, screenId) {
         this.captureBuySkuData(actionId, data);
+        if (this.isStaleTap(actionId, screenId))
+            return;
         const lifecycles = this.lifecycles(actionId);
         if (!lifecycles)
             return;

package/dist/index.d.ts

@@ -2452,10 +2452,25 @@ declare class NamiFlow extends BasicNamiFlow {
      * has no `__buy_sku__` lifecycles of its own.
      */
     private captureBuySkuData;
-    triggerActions(actionId: string, component?: any, data?: Record<string, any>): void;
+    /**
+     * True when a tap belongs to a screen the flow has already left.
+     *
+     * A button is dispatched by the screen that rendered it, but {@link lifecycles} resolves
+     * against whatever step is current when the tap is PROCESSED. A screen that advances itself
+     * — `__appear__` authoring `after: flowNav` — moves the step out from under an in-flight tap,
+     * so the outgoing screen's button would run the INCOMING step's actions for the same button
+     * id. Observed live on NFL Vizio: "skip intro" on the Welcome screen ran the next screen's
+     * `flowExit` instead of its own `flowNav`, and the analytics it emitted carried the old
+     * button's `{{ buttonText }}` against the new screen's `{{ pageId }}` (NAM-4395).
+     *
+     * Only drops when BOTH ids are known and differ: a caller that supplies no screen id, or a
+     * step with no screen of its own (entry/branch/exit), keeps the previous behaviour.
+     */
+    isStaleTap(actionId: string, screenId?: string): boolean;
+    triggerActions(actionId: string, component?: any, data?: Record<string, any>, screenId?: string): void;
     executeFullLifecycles(lifecycles: NamiFlowOn[], data?: Record<string, any>): void;
-    triggerBeforeActions(actionId: string, component?: any, data?: Record<string, any>): void;
-    triggerAfterActions(actionId: string, component?: any, data?: Record<string, any>): void;
+    triggerBeforeActions(actionId: string, component?: any, data?: Record<string, any>, screenId?: string): void;
+    triggerAfterActions(actionId: string, component?: any, data?: Record<string, any>, screenId?: string): void;
     currentStepHasHoistedPrimaryActions(actionId: string): boolean;
     private shouldRun;
     private performAction;

package/dist/index.mjs

@@ -104,7 +104,7 @@ const {
 // version — stamped by scripts/version.sh
 NAMI_SDK_VERSION = "3.4.12", 
 // full package version including dev suffix — stamped by scripts/version.sh
-NAMI_SDK_PACKAGE_VERSION = "3.4.12-dev.202610062108", 
+NAMI_SDK_PACKAGE_VERSION = "3.4.12-dev.202610081956", 
 // environments
 PRODUCTION = "production", DEVELOPMENT = "development", 
 // error messages
@@ -65605,8 +65605,32 @@ class NamiFlow extends BasicNamiFlow {
             this.purchasedSku = normalizePurchasedSku(data.sku);
         }
     }
-    triggerActions(actionId, component, data) {
+    /**
+     * True when a tap belongs to a screen the flow has already left.
+     *
+     * A button is dispatched by the screen that rendered it, but {@link lifecycles} resolves
+     * against whatever step is current when the tap is PROCESSED. A screen that advances itself
+     * — `__appear__` authoring `after: flowNav` — moves the step out from under an in-flight tap,
+     * so the outgoing screen's button would run the INCOMING step's actions for the same button
+     * id. Observed live on NFL Vizio: "skip intro" on the Welcome screen ran the next screen's
+     * `flowExit` instead of its own `flowNav`, and the analytics it emitted carried the old
+     * button's `{{ buttonText }}` against the new screen's `{{ pageId }}` (NAM-4395).
+     *
+     * Only drops when BOTH ids are known and differ: a caller that supplies no screen id, or a
+     * step with no screen of its own (entry/branch/exit), keeps the previous behaviour.
+     */
+    isStaleTap(actionId, screenId) {
+        const currentScreen = this.currentFlowStep?.screen;
+        if (!screenId || !currentScreen || screenId === currentScreen)
+            return false;
+        this.flowLog(`dropping stale tap ${actionId} from screen ${screenId} — the flow is on ${currentScreen} (NAM-4395)`);
+        logger.info(`Ignoring "${actionId}" from a screen the flow has already left (${screenId} -> ${currentScreen})`);
+        return true;
+    }
+    triggerActions(actionId, component, data, screenId) {
         this.captureBuySkuData(actionId, data);
+        if (this.isStaleTap(actionId, screenId))
+            return;
         const lifecycles = this.lifecycles(actionId);
         if (!lifecycles)
             return;
@@ -65637,8 +65661,10 @@ class NamiFlow extends BasicNamiFlow {
             }));
         }
     }
-    triggerBeforeActions(actionId, component, data) {
+    triggerBeforeActions(actionId, component, data, screenId) {
         this.captureBuySkuData(actionId, data);
+        if (this.isStaleTap(actionId, screenId))
+            return;
         const lifecycles = this.lifecycles(actionId);
         if (!lifecycles)
             return;
@@ -65647,8 +65673,10 @@ class NamiFlow extends BasicNamiFlow {
             cycle.before?.forEach((action) => this.shouldRun(action) && this.performAction(action, data));
         }
     }
-    triggerAfterActions(actionId, component, data) {
+    triggerAfterActions(actionId, component, data, screenId) {
         this.captureBuySkuData(actionId, data);
+        if (this.isStaleTap(actionId, screenId))
+            return;
         const lifecycles = this.lifecycles(actionId);
         if (!lifecycles)
             return;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@namiml/sdk-core",
-  "version": "3.4.12-dev.202610062108",
+  "version": "3.4.12-dev.202610081956",
   "description": "Platform-agnostic core for the Nami SDK — business logic, API, types, and state management",
   "type": "module",
   "main": "./dist/index.cjs",