🔇 postpone/cleanup cookie and telemetry monitor-until dates (#4840)

Author: thomas-lebeau

packages/browser-core/src/browser/cookieAccess.ts

@@ -18,13 +18,17 @@ export interface CookieAccess {
 
 export type CookieAccessFactory = (cookieName: string, cookieOptions: CookieOptions) => CookieAccess
 
+// Used to identify capability-probe cookies so their write failures aren't reported as telemetry:
+// failing to write them is an expected outcome (it triggers the document.cookie fallback), not a bug.
+export const TEST_COOKIE_NAME_PREFIX = 'dd_cookie_test_'
+
 export async function areCookiesAuthorized(
   createAccess: CookieAccessFactory,
   cookieOptions: CookieOptions
 ): Promise<boolean> {
   // Use a unique cookie name to avoid issues when the SDK is initialized multiple times during
   // the test cookie lifetime
-  const testCookieName = `dd_cookie_test_${generateUUID()}`
+  const testCookieName = `${TEST_COOKIE_NAME_PREFIX}${generateUUID()}`
   const testCookieValue = 'test'
   const access = createAccess(testCookieName, cookieOptions)
   try {
@@ -83,20 +87,22 @@ export function createCookieStoreAccess(cookieName: string, cookieOptions: Cooki
           partitioned: cookieOptions.partitioned,
         })
       } catch (error) {
-        const documentCookies = getCookies(cookieName)
-        // monitor-until: 2026-07-01
-        addTelemetryDebug('Failed to set cookie using Cookie Store API', {
-          'error.message': (error as Error).message,
-          newValue: value,
-          cookieOptions: {
-            ...cookieOptions,
-          },
-          cookies: items.map((item) => ({
-            ...item,
-          })),
-          cookieCount: items.length,
-          documentCookies,
-        })
+        if (!cookieName.startsWith(TEST_COOKIE_NAME_PREFIX)) {
+          const documentCookies = getCookies(cookieName)
+          // monitor-until: 2026-10-01
+          addTelemetryDebug('Failed to set cookie using Cookie Store API', {
+            'error.message': (error as Error).message,
+            newValue: value,
+            cookieOptions: {
+              ...cookieOptions,
+            },
+            cookies: items.map((item) => ({
+              ...item,
+            })),
+            cookieCount: items.length,
+            documentCookies,
+          })
+        }
       }
     },
 

packages/browser-core/src/domain/telemetry/telemetry.ts

@@ -66,7 +66,6 @@ export const enum TelemetryMetrics {
   REMOTE_CONFIGURATION_METRIC_NAME = 'remote configuration metrics',
   RECORDER_INIT_METRICS_TELEMETRY_NAME = 'Recorder init metrics',
   SEGMENT_METRICS_TELEMETRY_NAME = 'Segment network request metrics',
-  INITIAL_VIEW_METRICS_TELEMETRY_NAME = 'Initial view metrics',
   SESSION_MANAGER_INIT_METRICS_TELEMETRY_NAME = 'Session manager init metrics',
 }
 

packages/browser-rum-core/src/boot/startRum.ts

@@ -50,7 +50,6 @@ import type { SdkName } from '../domain/contexts/defaultContext'
 import { startDefaultContext } from '../domain/contexts/defaultContext'
 import type { Hooks } from '../domain/hooks'
 import { startEventCollection } from '../domain/event/eventCollection'
-import { startInitialViewMetricsTelemetry } from '../domain/view/viewMetrics/startInitialViewMetricsTelemetry'
 import { startSourceCodeContext } from '../domain/contexts/sourceCodeContext'
 import type { RecorderApi, ProfilerApi } from './rumPublicApi'
 
@@ -102,9 +101,6 @@ export function startRum(
     cleanupTasks.push(() => pageMayExitSubscription.unsubscribe())
   }
 
-  const { stop: stopInitialViewMetricsTelemetry } = startInitialViewMetricsTelemetry(lifeCycle, telemetry)
-  cleanupTasks.push(stopInitialViewMetricsTelemetry)
-
   const { stop: stopRumEventCollection, ...startRumEventCollectionResult } = startRumEventCollection(
     lifeCycle,
     hooks,

packages/browser-rum-core/src/domain/view/viewMetrics/startInitialViewMetricsTelemetry.spec.ts

@@ -45,7 +45,7 @@ export function startSegmentTelemetry(
       (requestEvent.type === 'success' && requestEvent.payload.isFullSnapshot)
     ) {
       const metrics = createSegmentMetrics(requestEvent.type, requestEvent.bandwidth, requestEvent.payload)
-      // monitor-until: 2026-07-01
+      // monitor-until: forever
       addTelemetryMetrics(TelemetryMetrics.SEGMENT_METRICS_TELEMETRY_NAME, { metrics })
     }
   })

packages/browser-rum-core/src/domain/view/viewMetrics/startInitialViewMetricsTelemetry.ts

@@ -57,7 +57,7 @@ export function startRecorderInitTelemetry(telemetry: Telemetry, observable: Obs
         unsubscribe()
 
         if (startContext) {
-          // monitor-until: 2026-07-01
+          // monitor-until: forever
           addTelemetryMetrics(TelemetryMetrics.RECORDER_INIT_METRICS_TELEMETRY_NAME, {
             metrics: createRecorderInitMetrics(
               startContext.forced,