ga4-export-fixer 0.9.0-dev.12 → 0.9.0-dev.14

package/helpers/ga4Transforms.js

@@ -235,9 +235,11 @@ const itemListAttributionExpr = (lookbackType, timestampColumn, lookbackTimeMs)
     frameBounds = `range between ${lookbackMicros} preceding and current row`;
   }
 
-  // Refund events occur outside the selection-driven journey window — propagating
-  // item_list_* attribution into them would misattribute. Suppress attribution on refund rows.
-  return `if(event_name = 'refund', null, last_value(
+  // Suppress attribution for:
+  //   - refund events (outside the selection-driven journey window)
+  //   - unconsented events (user_pseudo_id is NULL) — attribution requires a visitor
+  //     identity to stitch select_* events to later receivers within the same visitor.
+  return `if(event_name = 'refund' or user_pseudo_id is null, null, last_value(
       if(${selectEvents}, ${structExpr}, null) ignore nulls
     ) over(
       partition by ${partitionBy}
@@ -260,6 +262,10 @@ const itemListAttributionExpr = (lookbackType, timestampColumn, lookbackTimeMs)
  * the rows are interchangeable, so arbitrary row number assignment between them
  * produces the same result.
  *
+ * Unconsented events (user_pseudo_id is NULL) use an empty-string sentinel inside
+ * concat — without it, CONCAT NULL-propagates and the row_id becomes NULL, which
+ * would prevent enrichments from applying to such events.
+ *
  * @param {string} ecommerceEventsFilter - Comma-separated, quoted list of event names
  *        (e.g., "'purchase', 'add_to_cart'").
  * @returns {string} SQL expression that evaluates to the row id or NULL.
@@ -268,7 +274,7 @@ const itemRowId = (ecommerceEventsFilter) => {
   return `if(
       event_name in (${ecommerceEventsFilter}),
       farm_fingerprint(concat(
-        user_pseudo_id,
+        ifnull(user_pseudo_id, ''),
         cast(event_timestamp as string),
         event_name,
         to_json_string(items),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ga4-export-fixer",
-  "version": "0.9.0-dev.12",
+  "version": "0.9.0-dev.14",
   "description": "",
   "main": "index.js",
   "files": [

package/tables/ga4EventsEnhanced/index.js

@@ -357,14 +357,17 @@ ${excludedEventsSQL}`,
         // items_unnested. When itemListAttribution is configured, override the three
         // attribution entries with their package-generated coalesce-with-passthrough
         // expressions. Item-level enrichment columns layer on top via the spread below.
+        // References are qualified with `items_unnested.` so that overlapping item-level
+        // enrichments (which JOIN against enrich_<name> CTEs that may share column names)
+        // do not produce ambiguous bare-column references.
         const preItemExpressions = {};
         for (const f of helpers.ga4ItemStructFields) {
-            preItemExpressions[f] = f;
+            preItemExpressions[f] = `items_unnested.${f}`;
         }
         if (itemListAttribution) {
-            preItemExpressions.item_list_name = `coalesce(if(${passthroughEvents}, item_list_name, _item_list_attr.item_list_name), '(not set)')`;
-            preItemExpressions.item_list_id = `coalesce(if(${passthroughEvents}, item_list_id, _item_list_attr.item_list_id), '(not set)')`;
-            preItemExpressions.item_list_index = `coalesce(if(${passthroughEvents}, item_list_index, _item_list_attr.item_list_index))`;
+            preItemExpressions.item_list_name = `coalesce(if(${passthroughEvents}, items_unnested.item_list_name, _item_list_attr.item_list_name), '(not set)')`;
+            preItemExpressions.item_list_id = `coalesce(if(${passthroughEvents}, items_unnested.item_list_id, _item_list_attr.item_list_id), '(not set)')`;
+            preItemExpressions.item_list_index = `coalesce(if(${passthroughEvents}, items_unnested.item_list_index, _item_list_attr.item_list_index))`;
         }
 
         // Wrap overlapping item-level enrichment columns in coalesce(<enrichExpr>, <originalExpr>)