@adobe/spacecat-shared-rum-api-client 2.38.7 → 2.38.9

package/CHANGELOG.md

@@ -1,3 +1,17 @@
+# [@adobe/spacecat-shared-rum-api-client-v2.38.9](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-rum-api-client-v2.38.8...@adobe/spacecat-shared-rum-api-client-v2.38.9) (2025-11-04)
+
+
+### Bug Fixes
+
+* **deps:** update dependency @adobe/helix-universal to v5.3.0 ([#1091](https://github.com/adobe/spacecat-shared/issues/1091)) ([13cbc72](https://github.com/adobe/spacecat-shared/commit/13cbc721f67c066948337faa6c6a4ea5b0c5ec9a))
+
+# [@adobe/spacecat-shared-rum-api-client-v2.38.8](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-rum-api-client-v2.38.7...@adobe/spacecat-shared-rum-api-client-v2.38.8) (2025-10-31)
+
+
+### Bug Fixes
+
+* **traffic-attribution:** limit vendors per source ([#1079](https://github.com/adobe/spacecat-shared/issues/1079)) ([9165913](https://github.com/adobe/spacecat-shared/commit/9165913f379a00df409fe86961ba5df1f5a2a840))
+
 # [@adobe/spacecat-shared-rum-api-client-v2.38.7](https://github.com/adobe/spacecat-shared/compare/@adobe/spacecat-shared-rum-api-client-v2.38.6...@adobe/spacecat-shared-rum-api-client-v2.38.7) (2025-10-29)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adobe/spacecat-shared-rum-api-client",
-  "version": "2.38.7",
+  "version": "2.38.9",
   "description": "Shared modules of the Spacecat Services - Rum API client",
   "type": "module",
   "engines": {
@@ -37,7 +37,7 @@
   "dependencies": {
     "@adobe/fetch": "4.2.3",
     "@adobe/helix-shared-wrap": "2.0.2",
-    "@adobe/helix-universal": "5.2.3",
+    "@adobe/helix-universal": "5.3.0",
     "@adobe/rum-distiller": "1.20.8",
     "@adobe/spacecat-shared-utils": "1.66.0",
     "aws4": "1.13.2",

package/src/common/traffic.js

@@ -146,27 +146,61 @@ const not = (truth) => (text) => {
 
 const notEmpty = (text) => hasText(text);
 
-/*
- * --------- ENFORCEMENTS ----------------
- */
+// overrides
+const OVERRIDES = [
+  { when: (ctx) => (ctx.utmSource || '').toLowerCase() === 'chatgpt.com', set: { type: 'earned', category: 'llm', vendor: 'openai' } },
+];
 
-const ENFORCEMENTS = [
-  // vendors 'openai', 'claude', and 'perplexity' can only be earned/llm
-  {
-    when: (state) => state.vendor === 'openai' || state.vendor === 'claude' || state.vendor === 'perplexity',
-    enforce: (state) => ({ ...state, type: 'earned', category: 'llm' }),
+function applyOverrides(classification, context) {
+  const override = OVERRIDES.find((rule) => rule.when(context));
+  return override ? { ...classification, ...override.set } : classification;
+}
+
+// allowed known vendors per category
+const ALLOWED_VENDORS = {
+  earned: {
+    llm: ['openai', 'claude', 'perplexity', 'microsoft', 'google'],
+    search: ['google', 'bing', 'yahoo', 'yandex', 'baidu', 'duckduckgo', 'brave', 'ecosia', 'aol'],
+    social: null, // any vendor allowed
+    video: ['youtube', 'vimeo', 'twitch', 'tiktok', 'dailymotion'],
+    referral: null, // any vendor allowed
   },
-  // if utm_source=chatgpt.com, force earned/llm and vendor=openai
-  {
-    when: (state, ctx) => (ctx.utmSourceRaw || '').toLowerCase() === 'chatgpt.com',
-    enforce: (state) => ({ ...state, type: 'earned', category: 'llm', vendor: 'openai' }),
+  paid: {
+    search: ['google', 'bing', 'yahoo', 'yandex', 'baidu', 'microsoft'],
+    social: null, // any vendor allowed
+    video: ['youtube', 'vimeo', 'twitch', 'dailymotion'],
+    display: null, // any vendor allowed
+    affiliate: null, // any vendor allowed
+    uncategorized: null, // any vendor allowed
   },
-];
+  owned: {
+    direct: ['direct'],
+    internal: null, // any vendor allowed
+    email: null, // any vendor allowed
+    sms: null, // any vendor allowed
+    qr: null, // any vendor allowed
+    push: null, // any vendor allowed
+    uncategorized: null, // any vendor allowed
+  },
+};
+
+/**
+ * Validates if a vendor is allowed for the given type/category combination.
+ * @param {string} type - Traffic type (earned, paid, owned)
+ * @param {string} category - Traffic category (llm, search, social, etc.)
+ * @param {string} vendor - Vendor name to validate
+ * @returns {string} The vendor if allowed, empty string otherwise
+ */
+function validateVendor(type, category, vendor) {
+  if (!vendor) return '';
 
-function applyEnforcements(initialState, context) {
-  return ENFORCEMENTS
-    .reduce((acc, rule) => (
-      rule.when(acc, context) ? rule.enforce(acc, context) : acc), initialState);
+  const allowedVendors = ALLOWED_VENDORS[type]?.[category];
+
+  // null/undefined means any vendor is allowed
+  if (!allowedVendors) return vendor;
+
+  // Check if vendor is in the allowed list
+  return allowedVendors.includes(vendor) ? vendor : '';
 }
 
 /*
@@ -283,23 +317,25 @@ export function classifyTrafficSource(url, referrer, utmSource, utmMedium, track
     && rule.utmMedium(sanitize(utmMedium))
     && rule.tracking(trackingParams)
   ));
-  const { type, category } = match;
-  const vendor = classifyVendor(referrerDomain, utmSource, utmMedium);
+  let { type, category } = match;
+  let vendor = classifyVendor(referrerDomain, utmSource, utmMedium);
 
-  const enforced = applyEnforcements(
+  // apply overrides
+  const overridden = applyOverrides(
     { type, category, vendor },
-    {
-      utmSourceRaw: utmSource,
-      utmSource: sanitize(utmSource),
-      utmMedium: sanitize(utmMedium),
-      referrerDomain,
-    },
+    { utmSource, utmMedium, referrerDomain },
   );
+  type = overridden.type;
+  category = overridden.category;
+  vendor = overridden.vendor;
+
+  // validate vendor against allowed vendors for this type/category
+  const validatedVendor = validateVendor(type, category, vendor);
 
   return {
-    type: enforced.type,
-    category: enforced.category,
-    vendor: enforced.vendor,
+    type,
+    category,
+    vendor: validatedVendor,
   };
 }