From 28c1b5640fd3ad6c442bd0ec3f8fd7577843f937 Mon Sep 17 00:00:00 2001
From: Jake <jake.kennedy@guardian.co.uk>
Date: Tue, 6 Aug 2024 12:22:05 +0100
Subject: [PATCH] Record interscroller progress with logs and metrics (#1497)

---
 .changeset/silly-cobras-grab.md              |  5 ++
 src/core/lib/video-interscroller-progress.ts | 54 ++++++++++++++++++++
 src/core/messenger/background.ts             | 25 ++++-----
 3 files changed, 72 insertions(+), 12 deletions(-)
 create mode 100644 .changeset/silly-cobras-grab.md
 create mode 100644 src/core/lib/video-interscroller-progress.ts

diff --git a/.changeset/silly-cobras-grab.md b/.changeset/silly-cobras-grab.md
new file mode 100644
index 000000000..a2f2470f8
--- /dev/null
+++ b/.changeset/silly-cobras-grab.md
@@ -0,0 +1,5 @@
+---
+'@guardian/commercial': minor
+---
+
+Log interscroller progress with logs and metrics
diff --git a/src/core/lib/video-interscroller-progress.ts b/src/core/lib/video-interscroller-progress.ts
new file mode 100644
index 000000000..5644151d4
--- /dev/null
+++ b/src/core/lib/video-interscroller-progress.ts
@@ -0,0 +1,54 @@
+import { EventTimer } from 'core/event-timer';
+import { bypassMetricsSampling } from 'experiments/utils';
+
+const endpoint = window.guardian.config.page.isDev
+	? '//logs.code.dev-guardianapis.com/log'
+	: '//logs.guardianapis.com/log';
+
+let creativeId: number | undefined;
+let progress: number = 0;
+
+const sendProgress = () => {
+	if (!creativeId || !progress) {
+		return;
+	}
+	void fetch(endpoint, {
+		method: 'POST',
+		body: JSON.stringify({
+			label: 'commercial.interscroller.videoProgress',
+			properties: [
+				{ name: 'id', value: creativeId },
+				{ name: 'progress', value: progress },
+			],
+		}),
+		keepalive: true,
+		cache: 'no-store',
+		mode: 'no-cors',
+	});
+};
+
+const sendProgressOnUnloadViaLogs = () => {
+	window.addEventListener('visibilitychange', sendProgress, { once: true });
+	// Safari does not reliably fire the `visibilitychange` on page unload.
+	window.addEventListener('pagehide', sendProgress, { once: true });
+};
+
+const initVideoProgressReporting = (gamCreativeId: number) => {
+	creativeId = gamCreativeId;
+
+	EventTimer.get().setProperty('videoInterscrollerCreativeId', creativeId);
+
+	bypassMetricsSampling();
+
+	sendProgressOnUnloadViaLogs();
+};
+
+const updateVideoProgress = (updatedProgress: number) => {
+	progress = updatedProgress;
+	EventTimer.get().setProperty(
+		'videoInterscrollerPercentageProgress',
+		updatedProgress,
+	);
+};
+
+export { initVideoProgressReporting, updateVideoProgress };
diff --git a/src/core/messenger/background.ts b/src/core/messenger/background.ts
index 4ab7c4136..cc80c5626 100644
--- a/src/core/messenger/background.ts
+++ b/src/core/messenger/background.ts
@@ -1,7 +1,9 @@
 import { isObject } from '@guardian/libs';
-import { EventTimer } from 'core/event-timer';
+import {
+	initVideoProgressReporting,
+	updateVideoProgress,
+} from 'core/lib/video-interscroller-progress';
 import type { RegisterListener } from 'core/messenger';
-import { bypassCommercialMetricsSampling } from 'core/send-commercial-metrics';
 import fastdom from 'utils/fastdom-promise';
 import {
 	renderAdvertLabel,
@@ -239,7 +241,10 @@ const setupBackground = async (
 						const creativeTemplateId =
 							slot.getResponseInformation()?.creativeTemplateId;
 						if (creativeTemplateId === 11885667) {
-							return slot.getResponseInformation()?.creativeId;
+							return (
+								slot.getResponseInformation()?.creativeId ??
+								undefined
+							);
 						}
 					}
 					return undefined;
@@ -263,21 +268,17 @@ const setupBackground = async (
 
 				observer.observe(backgroundParent);
 
-				EventTimer.get().setProperty(
-					'videoInterscrollerCreativeId',
-					getCreativeId(),
-				);
+				const creativeId = getCreativeId();
 
-				void bypassCommercialMetricsSampling();
+				if (creativeId) {
+					initVideoProgressReporting(creativeId);
+				}
 
 				video.ontimeupdate = function () {
 					const percent = Math.round(
 						100 * (video.currentTime / video.duration),
 					);
-					EventTimer.get().setProperty(
-						'videoInterscrollerPercentageProgress',
-						percent,
-					);
+					updateVideoProgress(percent);
 				};
 			}
 		} else {