fix: remove redundant start events for multi page mode

Author: zhu-xiaowei

src/tracker/SessionTracker.ts

@@ -15,6 +15,7 @@ import { BaseTracker } from './BaseTracker';
 import { Session } from './Session';
 import { BrowserInfo } from '../browser';
 import { Event } from '../provider';
+import { PageType } from '../types';
 import { StorageUtil } from '../util/StorageUtil';
 
 const logger = new Logger('SessionTracker');
@@ -69,6 +70,8 @@ export class SessionTracker extends BaseTracker {
 			pageViewTracker.setIsEntrances();
 			this.provider.record({ name: Event.PresetEvent.SESSION_START });
 		}
+		if (isFirstTime && this.isMultiPageApp() && this.isFromCurrentHost())
+			return;
 		this.provider.record({
 			name: Event.PresetEvent.APP_START,
 			attributes: {
@@ -77,6 +80,14 @@ export class SessionTracker extends BaseTracker {
 		});
 	}
 
+	isFromCurrentHost() {
+		return window.location.host === this.context.browserInfo.latestReferrerHost;
+	}
+
+	isMultiPageApp() {
+		return this.context.configuration.pageType === PageType.multiPageApp;
+	}
+
 	onPageHide() {
 		logger.debug('page hide');
 		this.storeSession();

test/tracker/SessionTracker.test.ts

@@ -17,17 +17,17 @@ global.TextEncoder = TextEncoder;
 global.TextDecoder = TextDecoder;
 
 import { JSDOM } from 'jsdom';
-import { SendMode } from '../../src';
+import { PageType, SendMode } from '../../src';
 import { BrowserInfo } from '../../src/browser';
 import { NetRequest } from '../../src/network/NetRequest';
-import { AnalyticsEventBuilder } from '../../src/provider';
 import {
+	AnalyticsEventBuilder,
 	ClickstreamContext,
 	ClickstreamProvider,
 	Event,
 	EventRecorder,
 } from '../../src/provider';
-import { SessionTracker, PageViewTracker } from '../../src/tracker';
+import { PageViewTracker, SessionTracker } from '../../src/tracker';
 import { StorageUtil } from '../../src/util/StorageUtil';
 
 describe('SessionTracker test', () => {
@@ -89,6 +89,33 @@ describe('SessionTracker test', () => {
 		expect(session.isNewSession()).toBeTruthy();
 	});
 
+	test('test multi page mode record app start when setUp', () => {
+		Object.assign(provider.configuration, {
+			pageType: PageType.multiPageApp,
+		});
+		sessionTracker.setUp();
+		expect(recordMethodMock).toBeCalledWith({
+			name: Event.PresetEvent.APP_START,
+			attributes: {
+				[Event.ReservedAttribute.IS_FIRST_TIME]: true,
+			},
+		});
+	});
+
+	test('test multi page mode not record app start when come from the same host name', () => {
+		Object.assign(provider.configuration, {
+			pageType: PageType.multiPageApp,
+		});
+		context.browserInfo.latestReferrerHost = 'localhost';
+		sessionTracker.setUp();
+		expect(recordMethodMock).not.toBeCalledWith({
+			name: Event.PresetEvent.APP_START,
+			attributes: {
+				[Event.ReservedAttribute.IS_FIRST_TIME]: true,
+			},
+		});
+	});
+
 	test('test setUp for unsupported env', () => {
 		const addEventListenerMock = jest.spyOn(window, 'addEventListener');
 		const addEventListener = (global as any).document.addEventListener;