@playpilot/tpi 8.21.8 → 8.21.9

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@playpilot/tpi",
-	"version": "8.21.8",
+	"version": "8.21.9",
 	"type": "module",
 	"scripts": {
 		"dev": "vite dev",
@@ -17,7 +17,7 @@
 		"postinstall": "patch-package"
 	},
 	"devDependencies": {
-		"@playpilot/retargeting-tracking": "^1.4.1",
+		"@playpilot/retargeting-tracking": "^1.4.2",
 		"@sveltejs/adapter-auto": "^7.0.1",
 		"@sveltejs/kit": "^2.56.1",
 		"@sveltejs/vite-plugin-svelte": "^4.0.0",

package/src/lib/tracking.ts

@@ -4,7 +4,7 @@ import { isCrawler } from './crawler'
 import { genreSlugsToNames } from './genre'
 import type { TitleData } from './types/title'
 import { getFullUrlPath } from './url'
-import { getUserId, isUserTrackingAllowed } from './user'
+import { getSessionId, getUserId, isUserTrackingAllowed } from './user'
 
 const baseUrl = 'https://insights.playpilot.net'
 
@@ -41,6 +41,7 @@ export async function track(event: string, title: TitleData | null = null, paylo
 
   if (isUserTrackingAllowed()) {
     payload.user_id = getUserId()
+    payload.session_id = getSessionId()
     payload.region = window.PlayPilotLinkInjections.region
   }
 

package/src/lib/user.ts

@@ -2,6 +2,7 @@ import { hasConsentedTo } from './consent'
 import { generateRandomHash } from './hash'
 
 export const localStorageUserKey = 'playpilot_user_id'
+export const sessionStorageSessionKey = 'playpilot_session_id'
 
 export function getUserId(): string | null {
   if (!isUserTrackingAllowed()) return null
@@ -17,6 +18,20 @@ export function getUserId(): string | null {
   }
 }
 
+export function getSessionId(): string | null {
+  if (!isUserTrackingAllowed()) return null
+
+  try {
+    const sessionId = sessionStorage.getItem(sessionStorageSessionKey)
+    if (sessionId) return sessionId
+
+    sessionStorage.setItem(sessionStorageSessionKey, generateRandomHash(16))
+    return sessionStorage.getItem(sessionStorageSessionKey)
+  } catch {
+    return null
+  }
+}
+
 export function isUserTrackingAllowed(): boolean {
   return !!window.PlayPilotLinkInjections?.config?.allow_user_id_tracking && hasConsentedTo('tracking')
 }

package/src/tests/lib/user.test.js

@@ -1,5 +1,5 @@
 import { describe, it, expect, beforeEach, vi } from 'vitest'
-import { getUserId, isUserTrackingAllowed, localStorageUserKey } from '$lib/user'
+import { getSessionId, getUserId, isUserTrackingAllowed, localStorageUserKey, sessionStorageSessionKey } from '$lib/user'
 import { hasConsentedTo } from '$lib/consent'
 
 vi.mock('$lib/consent', () => ({
@@ -45,6 +45,44 @@ describe('user.ts', () => {
     })
   })
 
+  describe('getSessionId', () => {
+    beforeEach(() => {
+    // @ts-ignore
+      window.PlayPilotLinkInjections = {
+        config: {
+          allow_user_id_tracking: true,
+        },
+      }
+
+      vi.mocked(hasConsentedTo).mockImplementation(() => true)
+    })
+
+    it('Should return random hash by default', () => {
+      expect(getSessionId()).toBeTruthy()
+    })
+
+    it('Should return the previously set value', () => {
+      sessionStorage.setItem(sessionStorageSessionKey, 'some_value')
+
+      expect(getSessionId()).toBe('some_value')
+    })
+
+    it('Should return null if allow_user_id_tracking is not set', () => {
+      // @ts-ignore
+      window.PlayPilotLinkInjections = {}
+
+      expect(getSessionId()).toBe(null)
+
+      sessionStorage.setItem(sessionStorageSessionKey, 'some_value')
+
+      expect(getSessionId()).toBe(null)
+    })
+
+    it('Should return the same id when called multiple times', () => {
+      expect(getSessionId()).toBe(getSessionId())
+    })
+  })
+
   describe('isUserTrackingAllowed', () => {
     beforeEach(() => {
     // @ts-ignore

package/src/tests/setup.js

@@ -96,7 +96,9 @@ beforeEach(() => {
   document.cookie.split(';').forEach((cookie) => document.cookie = cookie.replace(/^ +/, '').replace(/=.*/, '=;'))
 
   window.HTMLElement.prototype.scrollIntoView = vi.fn()
+
   localStorage.clear()
+  sessionStorage.clear()
 
   mockAnimations()
 })