@guardian/ophan-tracker-js 2.4.1 → 2.4.3

package/NPM-dist/contribution.js

@@ -12,5 +12,5 @@ if (window.addEventListener != null) {
 }
 export default {
     record: transmit.sendMore,
-    viewId: transmit.viewId,
+    get viewId() { return transmit.viewId; },
 };

package/NPM-dist/core.js

@@ -11,6 +11,7 @@ init = function (servingPlatform, httpStatus) {
     platform = servingPlatform;
     window.addEventListener('pageshow', function (event) {
         if (event.persisted) {
+            transmit.bumpViewId();
             sendInitialEvent(httpStatus, location.href, window.document.referrer, 'bfcache');
         }
     }, false);
@@ -221,7 +222,7 @@ export default {
     servingPlatform: function () {
         return platform;
     },
-    viewId: transmit.viewId,
+    get viewId() { return transmit.viewId; },
     sendInitialEvent: sendInitialEvent,
 };
 /**

package/NPM-dist/embed.js

@@ -4,5 +4,5 @@ import transmit from './transmit.js';
 core.init('embed');
 export default {
     record: transmit.sendMore,
-    viewId: transmit.viewId,
+    get viewId() { return transmit.viewId; },
 };

package/NPM-dist/fb-instant.js

@@ -3,5 +3,5 @@ import core from './core.js';
 import transmit from './transmit.js';
 core.init('facebook-instant-article');
 export default {
-    viewId: transmit.viewId,
+    get viewId() { return transmit.viewId; },
 };

package/NPM-dist/holidays.js

@@ -4,5 +4,5 @@ import transmit from './transmit.js';
 core.init('holidays');
 export default {
     record: transmit.sendMore,
-    viewId: transmit.viewId,
+    get viewId() { return transmit.viewId; },
 };

package/NPM-dist/interactive.js

@@ -12,6 +12,6 @@ else {
         setEventEmitter: attention.setEventEmitter,
         trackComponentAttention: attention.initComponent,
         record: transmit.sendMore,
-        viewId: transmit.viewId,
+        get viewId() { return transmit.viewId; },
     };
 }

package/NPM-dist/jobs-courses.js

@@ -4,5 +4,5 @@ import transmit from './transmit.js';
 core.init('jobs-courses');
 export default {
     record: transmit.sendMore,
-    viewId: transmit.viewId,
+    get viewId() { return transmit.viewId; },
 };

package/NPM-dist/jobs.js

@@ -4,5 +4,5 @@ import transmit from './transmit.js';
 core.init('jobs');
 export default {
     record: transmit.sendMore,
-    viewId: transmit.viewId,
+    get viewId() { return transmit.viewId; },
 };

package/NPM-dist/manage-my-account.js

@@ -22,7 +22,7 @@ const sendInitialEvent = (url = location.href, referrer = document.referrer) =>
 const exports = {
     sendInitialEvent,
     record: transmit.sendMore,
-    viewId: transmit.viewId,
+    get viewId() { return transmit.viewId; },
 };
 window.guardian = window.guardian || {};
 window.guardian.ophan = exports;

package/NPM-dist/membership.js

@@ -12,5 +12,5 @@ if (window.addEventListener != null) {
 }
 export default {
     record: transmit.sendMore,
-    viewId: transmit.viewId,
+    get viewId() { return transmit.viewId; },
 };

package/NPM-dist/ng.js

@@ -39,8 +39,8 @@ else {
         setEventEmitter: attention.setEventEmitter,
         trackComponentAttention: attention.initComponent,
         record: transmit.sendMore,
-        viewId: transmit.viewId,
-        pageViewId: transmit.viewId,
+        get viewId() { return transmit.viewId; },
+        get pageViewId() { return transmit.viewId; },
         trackClickComponentEvent: trackClickComponentEvent,
     };
     ng = window.guardian.ophan;

package/NPM-dist/r2.js

@@ -11,5 +11,5 @@ if (window.addEventListener != null) {
     attention.init(visibility);
 }
 export default {
-    viewId: transmit.viewId,
+    get viewId() { return transmit.viewId; },
 };

package/NPM-dist/smart-news.js

@@ -9,5 +9,5 @@ if (window.addEventListener != null) {
 }
 export default {
     record: transmit.sendMore,
-    viewId: transmit.viewId,
+    get viewId() { return transmit.viewId; },
 };

package/NPM-dist/support.js

@@ -24,5 +24,5 @@ export default {
     init,
     sendInitialEvent,
     record: transmit.sendMore,
-    viewId: transmit.viewId,
+    get viewId() { return transmit.viewId; },
 };

package/NPM-dist/tests/click-path-capture.test.js

@@ -126,7 +126,7 @@ describe('click-path-capture.js', () => {
                 .spyOn(ophan, 'storeDataToSendOnNextEvent')
                 .mockImplementation(() => { });
             jest.spyOn(ophan, 'servingPlatform').mockReturnValue('web');
-            ophan.viewId = 'test-view-id';
+            jest.spyOn(ophan, 'viewId', 'get').mockReturnValue('test-view-id');
         });
         afterEach(() => {
             jest.clearAllMocks();

package/NPM-dist/tests/core.test.js

@@ -194,6 +194,14 @@ describe('loadType pageshow handling', () => {
             loadType: 'bfcache'
         }));
     });
+    it('should call bumpViewId when pageshow event has persisted=true', () => {
+        expect(pageShowHandler).toBeTruthy();
+        transmit.bumpViewId.mockClear();
+        // Simulate a bfcache restore event
+        const mockEvent = { persisted: true };
+        pageShowHandler(mockEvent);
+        expect(transmit.bumpViewId).toHaveBeenCalledTimes(1);
+    });
     it('should not call sendInitialEvent when pageshow event has persisted=false', () => {
         expect(pageShowHandler).toBeTruthy();
         transmit.sendInitial.mockClear();

package/NPM-dist/tests/transmit.test.js

@@ -74,6 +74,15 @@ describe('buildQueryString', () => {
         expect(queryString).not.toContain(`nullValue`);
     });
 });
+describe('bumpViewId', () => {
+    it('should update viewId when called', () => {
+        const originalViewId = transmit.viewId;
+        transmit.bumpViewId();
+        expect(transmit.viewId).not.toBe(originalViewId);
+        expect(typeof transmit.viewId).toBe('string');
+        expect(transmit.viewId.length).toBe(20);
+    });
+});
 describe('sendMore', () => {
     it('should call send with correct URL and callback', async () => {
         const obj = {

package/NPM-dist/transmit.js

@@ -117,7 +117,7 @@ const send = (url, f) => {
 export default {
     sendInitial,
     sendMore,
-    viewId,
+    get viewId() { return viewId; },
     bumpViewId,
 };
 /**

package/NPM-dist/types/contribution.d.ts

@@ -1,5 +1,5 @@
 declare namespace _default {
     let record: (obj: import("./ng.js").EventPayload, f?: (() => void) | undefined) => void;
-    let viewId: string;
+    const viewId: string;
 }
 export default _default;

package/NPM-dist/types/core.d.ts

@@ -3,7 +3,7 @@ declare namespace _default {
     export { storeDataToSendOnNextEvent };
     export { onLoadCapture };
     export function servingPlatform(): any;
-    export let viewId: string;
+    export const viewId: string;
     export { sendInitialEvent };
 }
 export default _default;

package/NPM-dist/types/embed.d.ts

@@ -1,5 +1,5 @@
 declare namespace _default {
     let record: (obj: import("./ng.js").EventPayload, f?: (() => void) | undefined) => void;
-    let viewId: string;
+    const viewId: string;
 }
 export default _default;

package/NPM-dist/types/fb-instant.d.ts

@@ -1,4 +1,4 @@
 declare namespace _default {
-    let viewId: string;
+    const viewId: string;
 }
 export default _default;

package/NPM-dist/types/holidays.d.ts

@@ -1,5 +1,5 @@
 declare namespace _default {
     let record: (obj: import("./ng.js").EventPayload, f?: (() => void) | undefined) => void;
-    let viewId: string;
+    const viewId: string;
 }
 export default _default;

package/NPM-dist/types/jobs-courses.d.ts

@@ -1,5 +1,5 @@
 declare namespace _default {
     let record: (obj: import("./ng.js").EventPayload, f?: (() => void) | undefined) => void;
-    let viewId: string;
+    const viewId: string;
 }
 export default _default;

package/NPM-dist/types/jobs.d.ts

@@ -1,5 +1,5 @@
 declare namespace _default {
     let record: (obj: import("./ng.js").EventPayload, f?: (() => void) | undefined) => void;
-    let viewId: string;
+    const viewId: string;
 }
 export default _default;

package/NPM-dist/types/manage-my-account.d.ts

@@ -2,6 +2,6 @@ export default exports;
 declare namespace exports {
     export { sendInitialEvent };
     export let record: (obj: import("./ng.js").EventPayload, f?: (() => void) | undefined) => void;
-    export let viewId: string;
+    export const viewId: string;
 }
 declare function sendInitialEvent(url?: string, referrer?: string): void;

package/NPM-dist/types/membership.d.ts

@@ -1,5 +1,5 @@
 declare namespace _default {
     let record: (obj: import("./ng.js").EventPayload, f?: (() => void) | undefined) => void;
-    let viewId: string;
+    const viewId: string;
 }
 export default _default;

package/NPM-dist/types/r2.d.ts

@@ -1,4 +1,4 @@
 declare namespace _default {
-    let viewId: string;
+    const viewId: string;
 }
 export default _default;

package/NPM-dist/types/smart-news.d.ts

@@ -1,5 +1,5 @@
 declare namespace _default {
     let record: (obj: import("./ng.js").EventPayload, f?: (() => void) | undefined) => void;
-    let viewId: string;
+    const viewId: string;
 }
 export default _default;

package/NPM-dist/types/support.d.ts

@@ -2,7 +2,7 @@ declare namespace _default {
     export { init };
     export { sendInitialEvent };
     export let record: (obj: import("./ng.js").EventPayload, f?: (() => void) | undefined) => void;
-    export let viewId: string;
+    export const viewId: string;
 }
 export default _default;
 declare function init(): void;

package/NPM-dist/types/transmit.d.ts

@@ -2,7 +2,7 @@ import { EventPayload } from './types/event.js';
 declare const _default: {
     sendInitial: (obj: Record<string, any>) => void;
     sendMore: (obj: EventPayload, f?: (() => void) | undefined) => void;
-    viewId: string;
+    readonly viewId: string;
     bumpViewId: () => void;
 };
 export default _default;

package/NPM-dist/types/witness.d.ts

@@ -1,5 +1,5 @@
 declare namespace _default {
     let record: (obj: import("./ng.js").EventPayload, f?: (() => void) | undefined) => void;
-    let viewId: string;
+    const viewId: string;
 }
 export default _default;

package/NPM-dist/witness.js

@@ -4,5 +4,5 @@ import transmit from './transmit.js';
 core.init('witness');
 export default {
     record: transmit.sendMore,
-    viewId: transmit.viewId,
+    get viewId() { return transmit.viewId; },
 };

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@guardian/ophan-tracker-js",
-	"version": "2.4.1",
+	"version": "2.4.3",
 	"type": "module",
 	"engines": {
 		"node": ">=16"