@mparticle/web-sdk 3.29.0 → 3.29.1

package/src/buttonTracking/buttonTracker.ts

@@ -162,8 +162,9 @@ export class ButtonTracker {
     // Bumped on reset so classification queued before it is dropped.
     private generation = 0;
     private lastCommitClick = -Infinity;
-    // The latest trusted click: whether the page excluded it, and the intent it
-    // already carries to the kit, with its source, if any.
+    // The latest trusted click: whether the page excluded it or it was withheld
+    // from the kit, and the intent it already carries to the kit, with its
+    // source, if any.
     private lastClick: {
         t: number;
         excluded: boolean;
@@ -408,15 +409,26 @@ export class ButtonTracker {
                     ? undefined
                     : findControl(path, true, PATH_LIMIT) ||
                       findRoleControl(path, PATH_LIMIT));
+            // A click that link tracking logs as Auto Link Click gives the kit no
+            // intent when the Rokt connection does not receive those events.
+            let withheld = false;
             if (logged) {
                 // With links alone, only an anchor can be logged.
-                click =
+                const logsThis =
                     logsEvents &&
-                    (this.logButtons || logged.matches('a[href]'))
+                    (this.logButtons || logged.matches('a[href]'));
+                const linksWithheld =
+                    signalsKit && this.logLinks && !this.linkClicksReachKit();
+                const described =
+                    logsThis || linksWithheld
                         ? describeButtonClick(logged, path)
                         : undefined;
+                const linkClick =
+                    this.logLinks && !!described && isLinkClick(described);
+                withheld = linksWithheld && linkClick;
+                click = logsThis ? described : undefined;
                 if (click) {
-                    asLink = this.logLinks && isLinkClick(click);
+                    asLink = linkClick;
                     // Without link tracking, a link that leaves the page stays a
                     // button click, except a plain link to another page of the
                     // site, which button tracking never logged.
@@ -435,7 +447,7 @@ export class ButtonTracker {
                     box = logged as HTMLInputElement;
                 }
             }
-            if (control) {
+            if (control && !withheld) {
                 // Checkout intent is only worked out when the kit can use it.
                 const intent = signalsKit
                     ? classifyIntent(control, pathname)
@@ -460,7 +472,14 @@ export class ButtonTracker {
                     checkedNames = checked ? checked.names : undefined;
                 }
             }
-            this.lastClick = { t, excluded, kind, source };
+            // A withheld click cannot start an Apple Pay session for the kit
+            // either.
+            this.lastClick = {
+                t,
+                excluded: excluded || withheld,
+                kind,
+                source,
+            };
         } else if (layer === 'submit') {
             const submitter = (event as Event & { submitter?: Element | null })
                 .submitter;
@@ -643,6 +662,11 @@ export class ButtonTracker {
         );
     }
 
+    private linkClicksReachKit(): boolean {
+        const roktManager = this.mpInstance._RoktManager;
+        return !!roktManager && roktManager.areAutoLinkClicksForwarded();
+    }
+
     private log(message: string): void {
         this.mpInstance.Logger.verbose('mParticle button tracking: ' + message);
     }

package/src/mp-instance.ts

@@ -1390,6 +1390,7 @@ function completeSDKInitialization(apiKey, config, mpInstance) {
         // Button tracking never runs as a native passthrough.
         resetButtonTracking();
         mpInstance._RoktManager.setAutoButtonClicksForwarded(false);
+        mpInstance._RoktManager.setAutoLinkClicksForwarded(false);
     } else {
         // Main SDK initialization flow
 
@@ -1459,8 +1460,8 @@ function completeSDKInitialization(apiKey, config, mpInstance) {
         // init or from a returning shopper's landing page view.
         const logButtons = !!getFeatureFlag(AutoButtonTracking);
         const logLinks = !!getFeatureFlag(AutoLinkTracking);
-        // Links feed intent too, so with link tracking on the kit must also
-        // receive Auto Link Click events.
+        // With link tracking on, a click logged as Auto Link Click feeds intent
+        // only when the kit also receives those events; button clicks still do.
         const buttonClickBlocker = roktConfig
             ? getButtonClickBlocker(roktConfig, kitBlocker)
             : undefined;
@@ -1472,11 +1473,14 @@ function completeSDKInitialization(apiKey, config, mpInstance) {
             mpInstance.Logger.warning(
                 buttonClickBlocker
                     ? `Checkout intent is off: the ${buttonClickBlocker} keeps some or all Auto Button Click events from the Rokt kit.`
-                    : `Checkout intent is off: the ${linkClickBlocker} keeps some or all Auto Link Click events from the Rokt kit.`
+                    : `Checkout intent skips link clicks: the ${linkClickBlocker} keeps some or all Auto Link Click events from the Rokt kit.`
             );
         }
         mpInstance._RoktManager.setAutoButtonClicksForwarded(
-            !!roktConfig && !buttonClickBlocker && !linkClickBlocker
+            !!roktConfig && !buttonClickBlocker
+        );
+        mpInstance._RoktManager.setAutoLinkClicksForwarded(
+            !!roktConfig && !linkClickBlocker
         );
 
         mpInstance._Forwarders.processForwarders(

package/src/roktManager.ts

@@ -175,6 +175,9 @@ export default class RoktManager {
     private pendingCheckoutIntents: Array<{ signal: ICheckoutIntentSignal; pathname: string }> = [];
     // Off until SDK init finds the Rokt connection receives auto button clicks.
     private autoButtonClicksForwarded = false;
+    // Off until SDK init finds the Rokt connection receives auto link clicks, or
+    // link tracking is off. Only link clicks lose their intent without it.
+    private autoLinkClicksForwarded = false;
 
     // Keys this manager's route subscription, so a re-executed bundle's manager replaces
     // it while a second named instance keeps its own.
@@ -331,6 +334,16 @@ export default class RoktManager {
         this.autoButtonClicksForwarded = forwarded;
     }
 
+    public setAutoLinkClicksForwarded(forwarded: boolean): void {
+        this.autoLinkClicksForwarded = forwarded;
+    }
+
+    // The Rokt kit reads this too: while false, its page views keep
+    // dispatchDelayMs, since a checkout link could never release the wait.
+    public areAutoLinkClicksForwarded(): boolean {
+        return this.autoLinkClicksForwarded;
+    }
+
     // noTargeting (which noDeviceId implies), doNotShareOrSell or Global Privacy
     // Control. Unlike noFunctional, these also stop auto button and link clicks
     // from being logged.