@adobe/helix-rum-enhancer 2.30.2 → 2.30.3

package/CHANGELOG.md

@@ -1,3 +1,14 @@
+## [2.30.3](https://github.com/adobe/helix-rum-enhancer/compare/v2.30.2...v2.30.3) (2025-02-03)
+
+
+### Bug Fixes
+
+* collect repeating clicks upto max configuration ([6daba19](https://github.com/adobe/helix-rum-enhancer/commit/6daba19b6dcf3717e8330a923a0d87e61e3080ee))
+* comment for max events tracking ([b4b1c6d](https://github.com/adobe/helix-rum-enhancer/commit/b4b1c6da79a31485bb2165382f9fb9ece611a716))
+* implement flood control with 1024 max events ([e06e1e5](https://github.com/adobe/helix-rum-enhancer/commit/e06e1e51b6dd3db0a015f4dcccb3ab8dcfb11b9f))
+* Implement flood control with 1024 max events ([ddfdb1e](https://github.com/adobe/helix-rum-enhancer/commit/ddfdb1e9dd58ff41cbd761774af9ad616290aae0))
+* review feedback ([f4e6dd1](https://github.com/adobe/helix-rum-enhancer/commit/f4e6dd1b3fae8e37024045156c85923e776f40c3))
+
 ## [2.30.2](https://github.com/adobe/helix-rum-enhancer/compare/v2.30.1...v2.30.2) (2025-01-17)
 
 

package/modules/index.js

@@ -34,10 +34,16 @@ const blocksMO = window.MutationObserver ? new MutationObserver(blocksMCB)
 // eslint-disable-next-line no-use-before-define, max-len
 const mediaMO = window.MutationObserver ? new MutationObserver(mediaMCB)
   /* c8 ignore next */ : {};
+/**
+ * Maximum number of events. The first call will be made by rum-js,
+ * leaving 1023 events for the enhancer to track
+ */
+let maxEvents = 1023;
 
 function trackCheckpoint(checkpoint, data, t) {
   const { weight, id } = window.hlx.rum;
-  if (isSelected) {
+  if (isSelected && maxEvents) {
+    maxEvents -= 1;
     const sendPing = (pdata = data) => {
       // eslint-disable-next-line object-curly-newline, max-len
       const body = JSON.stringify({ weight, id, referer: urlSanitizers[window.hlx.RUM_MASK_URL || 'path'](), checkpoint, t, ...data }, KNOWN_PROPERTIES);
@@ -301,17 +307,10 @@ function mediaMCB(mutations) {
 }
 
 function addTrackingFromConfig() {
-  let lastSource;
-  let lastTarget;
   document.addEventListener('click', (event) => {
-    const source = sourceSelector(event.target);
-    const target = targetSelector(event.target);
-    if (source !== lastSource || target !== lastTarget) {
-      sampleRUM('click', { target, source });
-      lastSource = source;
-      lastTarget = target;
-    }
+    sampleRUM('click', { target: targetSelector(event.target), source: sourceSelector(event.target) });
   });
+
   addCWVTracking();
   addFormTracking(window.document.body);
   addNavigationTracking();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adobe/helix-rum-enhancer",
-  "version": "2.30.2",
+  "version": "2.30.3",
   "description": "Helix RUM Enhancer",
   "main": "src/index.js",
   "type": "module",
@@ -52,10 +52,10 @@
     "husky": "9.1.7",
     "jsdoc-to-markdown": "9.1.1",
     "junit-report-builder": "5.1.1",
-    "lint-staged": "15.3.0",
-    "mocha": "11.0.1",
+    "lint-staged": "15.4.3",
+    "mocha": "11.1.0",
     "mocha-multi-reporters": "1.5.1",
-    "rollup": "4.30.1",
+    "rollup": "4.32.0",
     "rollup-plugin-checksum": "1.0.1",
     "rollup-plugin-cleanup": "3.2.1",
     "rollup-plugin-eslint-bundle": "9.0.0",

package/src/index.js

@@ -199,9 +199,11 @@
      : {};
   const mediaMO = window.MutationObserver ? new MutationObserver(mediaMCB)
      : {};
+  let maxEvents = 1023;
   function trackCheckpoint(checkpoint, data, t) {
     const { weight, id } = window.hlx.rum;
-    if (isSelected) {
+    if (isSelected && maxEvents) {
+      maxEvents -= 1;
       const sendPing = (pdata = data) => {
         const body = JSON.stringify({ weight, id, referer: urlSanitizers[window.hlx.RUM_MASK_URL || 'path'](), checkpoint, t, ...data }, KNOWN_PROPERTIES);
         const urlParams = window.RUM_PARAMS ? `?${new URLSearchParams(window.RUM_PARAMS).toString()}` : '';
@@ -422,16 +424,8 @@
       });
   }
   function addTrackingFromConfig() {
-    let lastSource;
-    let lastTarget;
     document.addEventListener('click', (event) => {
-      const source = sourceSelector(event.target);
-      const target = targetSelector(event.target);
-      if (source !== lastSource || target !== lastTarget) {
-        sampleRUM('click', { target, source });
-        lastSource = source;
-        lastTarget = target;
-      }
+      sampleRUM('click', { target: targetSelector(event.target), source: sourceSelector(event.target) });
     });
     addCWVTracking();
     addFormTracking(window.document.body);

package/src/index.md5

@@ -1 +1 @@
-389f38e9847798177fc9bd3f8f9b763b
+6a3cb05a18fc7e179bd98f2c310e8e21