@playpilot/tpi 8.23.1 → 8.23.4

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@playpilot/tpi",
-	"version": "8.23.1",
+	"version": "8.23.4",
 	"type": "module",
 	"scripts": {
 		"dev": "vite dev",

package/src/lib/splitTest.ts

@@ -1,4 +1,3 @@
-import { hasConsentedTo } from './consent'
 import { TrackingEvent } from './enums/TrackingEvent'
 import { track } from './tracking'
 
@@ -15,8 +14,6 @@ export const sessionStoragePrefix = 'playpilot_split_test_'
  * The identifier is saved on the window object so that it is consistent across this session.
  */
 export function getSplitTestIdentifier({ key }: SplitTest): number {
-  if (!hasConsentedTo('tracking')) return 0
-
   const windowIdentifier = window.PlayPilotLinkInjections?.split_test_identifiers?.[key]
   if (windowIdentifier !== undefined) return windowIdentifier
 
@@ -25,7 +22,7 @@ export function getSplitTestIdentifier({ key }: SplitTest): number {
   let storedIdentifier = sessionStorage.getItem(sessionStoragePrefix + key)
 
   if (!storedIdentifier) {
-    const randomIdentifier = Math.random().toString()
+    const randomIdentifier = (Math.floor(Math.random() * 100) / 100).toString()
     sessionStorage.setItem(sessionStoragePrefix + key, randomIdentifier)
     storedIdentifier = randomIdentifier
   }
@@ -53,8 +50,6 @@ export function getSplitTestVariantName(test: SplitTest): string {
 }
 
 export function isInSplitTestVariant(test: SplitTest, variant = 1): boolean {
-  if (!hasConsentedTo('tracking')) return false
-
   return getSplitTestVariantIndex(test) === variant
 }
 

package/src/lib/tracking.ts

@@ -3,7 +3,7 @@ import { mobileBreakpoint } from './constants'
 import { isCrawler } from './crawler'
 import { genreSlugsToNames } from './genre'
 import type { TitleData } from './types/title'
-import { getFullUrlPath } from './url'
+import { getFullUrlPath, getUrlParams } from './url'
 import { getSessionId, getUserId, isUserTrackingAllowed } from './user'
 
 const baseUrl = 'https://insights.playpilot.net'
@@ -48,6 +48,7 @@ export async function track(event: string, title: TitleData | null = null, paylo
   payload.version = __SCRIPT_VERSION__
   payload.time_since_initialize = window.PlayPilotLinkInjections?.time_at_initialize ? Date.now() - window.PlayPilotLinkInjections.time_at_initialize : 0
   payload.url = getFullUrlPath()
+  payload.url_params = getUrlParams()
   payload.organization_sid = window.PlayPilotLinkInjections?.organization_sid || 'undefined'
   payload.domain_sid = window.PlayPilotLinkInjections?.domain_sid || 'undefined'
   payload.device = {

package/src/lib/url.ts

@@ -10,6 +10,15 @@ export function getFullUrlPath(): string {
   return location.protocol + '//' + location.host + cleanedPathname
 }
 
+export function getUrlParams(): string {
+  try {
+    const url = new URL(location.href)
+    return Object.fromEntries(url.searchParams.entries())
+  } catch {
+    return {}
+  }
+}
+
 export function paramsToString(params: Record<string, any>): string {
   const filteredParams = Object.entries(params).filter(([_, value]) => value !== undefined && value !== null && value !== '')
   return filteredParams.map(([key, value]) => `${key}=${value}`).join('&')

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

@@ -2,11 +2,6 @@ import { describe, it, expect, vi, afterEach } from 'vitest'
 import { getSplitTestIdentifier, getSplitTestVariantIndex, getSplitTestVariantName, isInSplitTestVariant, sessionStoragePrefix, trackSplitTestAction, trackSplitTestView } from '$lib/splitTest'
 import { track } from '$lib/tracking'
 import { TrackingEvent } from '$lib/enums/TrackingEvent'
-import { hasConsentedTo } from '$lib/consent'
-
-vi.mock('$lib/consent', () => ({
-  hasConsentedTo: vi.fn(() => true),
-}))
 
 vi.mock('$lib/tracking', () => ({
   track: vi.fn(),
@@ -23,8 +18,6 @@ describe('$lib/splitTest', () => {
     window.PlayPilotLinkInjections = {}
 
     vi.resetAllMocks()
-
-    vi.mocked(hasConsentedTo).mockImplementation(() => true)
   })
 
   describe('getSplitTestIdentifier', () => {
@@ -60,12 +53,6 @@ describe('$lib/splitTest', () => {
       })
     })
 
-    it('Should return 0 if user did not consent', () => {
-      vi.mocked(hasConsentedTo).mockImplementation(() => false)
-
-      expect(getSplitTestIdentifier(splitTest)).toBe(0)
-    })
-
     it('Should store value in sessionStorage', () => {
       getSplitTestIdentifier(splitTest)
 
@@ -118,16 +105,6 @@ describe('$lib/splitTest', () => {
       expect(isInSplitTestVariant(multiple, 1)).toBe(false)
       expect(isInSplitTestVariant(multiple, 2)).toBe(true)
     })
-
-    it('Should return false if user did not consent', () => {
-      vi.mocked(hasConsentedTo).mockImplementation(() => false)
-
-      // @ts-ignore
-      window.PlayPilotLinkInjections.split_test_identifiers = {}
-      window.PlayPilotLinkInjections.split_test_identifiers[splitTest.key] = 0.75
-
-      expect(isInSplitTestVariant(splitTest)).toBe(false)
-    })
   })
 
   describe('trackSplitTestView', () => {

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

@@ -1,5 +1,5 @@
 import { describe, it, expect, beforeEach, afterEach } from 'vitest'
-import { getFullUrlPath, isUrlExcludedViaConfig, paramsToString } from '$lib/url'
+import { getFullUrlPath, getUrlParams, isUrlExcludedViaConfig, paramsToString } from '$lib/url'
 
 describe('url.ts', () => {
   describe('getFullUrlPath', () => {
@@ -53,6 +53,22 @@ describe('url.ts', () => {
     })
   })
 
+  describe('getUrlParams', () => {
+    it('Should return query params as object', () => {
+      window.location.pathname = '/some-path'
+      window.location.search = '?some=param&other=thing'
+
+      expect(getUrlParams()).toEqual({ some: 'param', other: 'thing' })
+    })
+
+    it('Should return empty object if no params are given', () => {
+      window.location.pathname = '/some-path'
+      window.location.search = ''
+
+      expect(getUrlParams()).toEqual({ })
+    })
+  })
+
   describe('paramsToString', () => {
     it('Should convert a flat object to a query string', () => {
       expect(paramsToString({ some: 'value', param: 123 })).toBe('some=value&param=123')