@adobe/spacecat-shared-rum-api-client 2.40.11 → 2.40.12

package/CHANGELOG.md

@@ -1,3 +1,9 @@
+## [@adobe/spacecat-shared-rum-api-client-v2.40.12](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-rum-api-client-v2.40.11...@adobe/spacecat-shared-rum-api-client-v2.40.12) (2026-04-09)
+
+### Bug Fixes
+
+* **deps:** update adobe fixes ([#1503](https://github.com/adobe/spacecat-shared/issues/1503)) ([21d4d4b](https://github.com/adobe/spacecat-shared/commit/21d4d4b3cfa95ca7748ef4b65b04cace24768dfc))
+
 ## [@adobe/spacecat-shared-rum-api-client-v2.40.11](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-rum-api-client-v2.40.10...@adobe/spacecat-shared-rum-api-client-v2.40.11) (2026-04-06)
 
 ### Bug Fixes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adobe/spacecat-shared-rum-api-client",
-  "version": "2.40.11",
+  "version": "2.40.12",
   "description": "Shared modules of the Spacecat Services - Rum API client",
   "type": "module",
   "engines": {
@@ -35,9 +35,9 @@
     "access": "public"
   },
   "dependencies": {
-    "@adobe/fetch": "4.2.3",
+    "@adobe/fetch": "4.3.0",
     "@adobe/helix-shared-wrap": "2.0.2",
-    "@adobe/helix-universal": "5.4.0",
+    "@adobe/helix-universal": "5.4.1",
     "@adobe/rum-distiller": "1.23.0",
     "@adobe/spacecat-shared-utils": "1.81.1",
     "aws4": "1.13.2",

package/src/common/rum-bundler-client.js

@@ -121,7 +121,9 @@ function generateUrlsForDateRange(startDate, endDate, domain, granularity, domai
  * then data for the `/a1/` and `/a2` are counted towards `/`'s data
  */
 async function mergeBundlesWithSameId(bundles) {
-  if (!bundles[0]?.url?.includes('bamboohr.com')) return bundles;
+  if (!bundles[0]?.url?.includes('bamboohr.com')) {
+    return bundles;
+  }
   const prodBaseUrl = 'https://www.bamboohr.com/experiments/';
   const previewBaseUrl = 'https://main--bamboohr-website--bamboohr.hlx.page/experiments/archive/';
   const manifestUrls = [
@@ -178,14 +180,18 @@ async function mergeBundlesWithSameId(bundles) {
 
   const getControlPath = (url) => {
     const path = new URL(url).pathname;
-    if (variantPaths.includes(path)) return mapping[path];
+    if (variantPaths.includes(path)) {
+      return mapping[path];
+    }
     return path;
   };
 
   const byIdAndPath = bundles.reduce((acc, cur) => {
     const controlPath = getControlPath(cur.url);
     const key = `${cur.id}-${controlPath}`;
-    if (!acc[key]) acc[key] = [];
+    if (!acc[key]) {
+      acc[key] = [];
+    }
     if (variantPaths.includes(new URL(cur.url).pathname)) {
       // eslint-disable-next-line no-param-reassign
       cur.url = new URL(controlPath, cur.url).href;

package/src/common/traffic.js

@@ -25,8 +25,12 @@ import URI from 'urijs';
  *                    URL if it does not contain any text.
  */
 export function getSecondLevelDomain(url) {
-  if (!hasText(url)) return url;
-  if (url === '(direct)') return '';
+  if (!hasText(url)) {
+    return url;
+  }
+  if (url === '(direct)') {
+    return '';
+  }
 
   try {
     const uri = new URI(prependSchema(url));
@@ -132,8 +136,12 @@ const emailTrackingParams = ['email'];
 const any = () => true;
 
 const anyOf = (truth) => (text) => {
-  if (Array.isArray(truth)) return truth.includes(text);
-  if (truth instanceof RegExp) return truth.test(text);
+  if (Array.isArray(truth)) {
+    return truth.includes(text);
+  }
+  if (truth instanceof RegExp) {
+    return truth.test(text);
+  }
   return truth === text;
 };
 
@@ -141,9 +149,15 @@ const anyOf = (truth) => (text) => {
 const none = (input) => (Array.isArray(input) ? input.length === 0 : !hasText(input));
 
 const not = (truth) => (text) => {
-  if (!hasText(text)) return false;
-  if (Array.isArray(truth)) return !truth.includes(text);
-  if (truth instanceof RegExp) return !truth.test(text);
+  if (!hasText(text)) {
+    return false;
+  }
+  if (Array.isArray(truth)) {
+    return !truth.includes(text);
+  }
+  if (truth instanceof RegExp) {
+    return !truth.test(text);
+  }
   return truth !== text;
 };
 
@@ -199,12 +213,16 @@ const ALLOWED_VENDORS = {
  * @returns {string} The vendor if allowed, empty string otherwise
  */
 function validateVendor(type, category, vendor) {
-  if (!vendor) return '';
+  if (!vendor) {
+    return '';
+  }
 
   const allowedVendors = ALLOWED_VENDORS[type]?.[category];
 
   // null/undefined means any vendor is allowed
-  if (!allowedVendors) return vendor;
+  if (!allowedVendors) {
+    return vendor;
+  }
 
   // Check if vendor is in the allowed list
   return allowedVendors.includes(vendor) ? vendor : '';
@@ -308,9 +326,15 @@ export function extractTrafficHints(bundle) {
  */
 export function classifyVendor(referrer, utmSource, utmMedium) {
   const result = vendorClassifications.find(({ regex }) => {
-    if (regex.test(referrer)) return true;
-    if (regex.test(utmSource)) return true;
-    if (regex.test(utmMedium)) return true;
+    if (regex.test(referrer)) {
+      return true;
+    }
+    if (regex.test(utmSource)) {
+      return true;
+    }
+    if (regex.test(utmMedium)) {
+      return true;
+    }
     return false;
   });
   return result ? result.result : '';

package/src/functions/experiment.js

@@ -50,7 +50,9 @@ function handler(bundles) {
   const result = {};
   experiments.forEach((uev) => {
     const [url, experiment] = uev.value.split(DELIMITER);
-    if (!result[url]) result[url] = [];
+    if (!result[url]) {
+      result[url] = [];
+    }
     result[url].push({
       experiment,
       variants: [],

package/src/functions/opportunities/high-organic-low-ctr.js

@@ -112,7 +112,9 @@ function convertToOpportunity(traffic) {
  * @returns {Array} List of pages sorted by joint strength
  */
 function sortPagesByEarnedAndOverallTraffic(pages) {
-  if (!Array.isArray(pages) || pages.length === 0) return [];
+  if (!Array.isArray(pages) || pages.length === 0) {
+    return [];
+  }
 
   const sortedOverall = [...pages].sort((a, b) => a.total - b.total);
   const sortedEarned = [...pages].sort((a, b) => {

package/src/functions/traffic-analysis.js

@@ -57,16 +57,24 @@ function getClicked(bundle) {
   const latestClickEvent = bundle.events
     .filter((e) => e.checkpoint === 'click')
     .reduce((latest, current) => {
-      if (!latest || !latest.timeDelta) return current;
-      if (current?.timeDelta > latest.timeDelta) return current;
+      if (!latest || !latest.timeDelta) {
+        return current;
+      }
+      if (current?.timeDelta > latest.timeDelta) {
+        return current;
+      }
       return latest;
     }, null);
 
-  if (!latestClickEvent) return 0;
+  if (!latestClickEvent) {
+    return 0;
+  }
 
   const isConsentClick = !!utils.reclassifyConsent(latestClickEvent).vendor;
 
-  if (isConsentClick) return 0;
+  if (isConsentClick) {
+    return 0;
+  }
 
   return 1;
 }
@@ -78,14 +86,18 @@ function getConsent(bundle) {
 
   const consentClick = bundle.events.find((e) => e.checkpoint === 'click' && utils.reclassifyConsent(e).vendor);
 
-  if (!consentClick) return consentBannerStatus || null;
+  if (!consentClick) {
+    return consentBannerStatus || null;
+  }
 
   return utils.reclassifyConsent(consentClick).target;
 }
 
 function trafficType(bundle, memo) {
   const key = `${bundle.id}${bundle.url}${bundle.time}`;
-  if (memo[key]) return memo[key];
+  if (memo[key]) {
+    return memo[key];
+  }
 
   const type = classifyTraffic(bundle);
   // eslint-disable-next-line no-param-reassign

package/src/index.js

@@ -65,7 +65,9 @@ export default class RUMAPIClient {
     const { env, log = console } = context;
     const { RUM_ADMIN_KEY: rumAdminKey } = env;
 
-    if (context.rumApiClient) return context.rumApiClient;
+    if (context.rumApiClient) {
+      return context.rumApiClient;
+    }
 
     const client = new RUMAPIClient({ rumAdminKey }, log);
     context.rumApiClient = client;
@@ -126,7 +128,9 @@ export default class RUMAPIClient {
   // eslint-disable-next-line class-methods-use-this
   async query(query, opts) {
     const { handler, checkpoints } = HANDLERS[query] || {};
-    if (!handler) throw new Error(`Unknown query ${query}`);
+    if (!handler) {
+      throw new Error(`Unknown query ${query}`);
+    }
 
     try {
       const domainkey = await this._getDomainkey(opts);
@@ -186,7 +190,9 @@ export default class RUMAPIClient {
 
   async queryStream(query, opts) {
     const { handler, checkpoints } = HANDLERS[query] || {};
-    if (!handler) throw new Error(`Unknown query ${query}`);
+    if (!handler) {
+      throw new Error(`Unknown query ${query}`);
+    }
 
     try {
       const domainkey = await this._getDomainkey(opts);