fix(browser): Guard auto-register call and update web-vital test assertions

- Guard `client.getIntegrationByName?.()` / `client.addIntegration?.()` in
  `browserTracingIntegration.afterAllSetup` so stripped-down test/mock clients
  don't crash on auto-registering `webVitalsIntegration`.

- Drop `eventData.measurements.{lcp,cls,inp,ttfb,fp,fcp}` assertions from
  browser integration tests; web vitals are no longer recorded as transaction
  measurements. Web-vital-specific tests now assert the new
  `browser.web_vital.X.value` attribute on the corresponding span (the LCP/CLS
  child span, or the pageload span for ttfb/fp/fcp).

- Delete legacy INP test suites (`web-vitals-inp`, `web-vitals-inp-late`,
  `web-vitals-inp-navigate`, `web-vitals-inp-parametrized`,
  `web-vitals-inp-parametrized-late`). They tested the v1 standalone INP
  envelope path which is gone; `web-vitals-inp-streamed-spans` covers the
  supported flow.

Co-Authored-By: Claude Opus 4.7 (1M context) <[email protected]>

Author: logaretm

dev-packages/browser-integration-tests/suites/tracing/metrics/handlers-lcp/test.ts

@@ -26,17 +26,19 @@ sentryTest(
       page.locator('button').click(),
     ]);
 
-    expect(eventData.measurements).toBeDefined();
-    expect(eventData.measurements?.lcp?.value).toBeDefined();
+    const lcpSpan = eventData.spans?.find(({ op }) => op === 'ui.webvital.lcp');
+    expect(lcpSpan).toBeDefined();
+    const lcpAttrs = lcpSpan?.data ?? {};
+    expect(lcpAttrs['browser.web_vital.lcp.value']).toBeGreaterThan(0);
 
     // This should be body > img, but it can be flakey as sometimes it will report
     // the button as LCP.
-    expect(eventData.contexts?.trace?.data?.['lcp.element'].startsWith('body >')).toBe(true);
+    expect(String(lcpAttrs['browser.web_vital.lcp.element']).startsWith('body >')).toBe(true);
 
     // Working around flakiness
     // Only testing this when the LCP element is an image, not a button
-    if (eventData.contexts?.trace?.data?.['lcp.element'] === 'body > img') {
-      expect(eventData.contexts?.trace?.data?.['lcp.size']).toBe(107400);
+    if (lcpAttrs['browser.web_vital.lcp.element'] === 'body > img') {
+      expect(lcpAttrs['browser.web_vital.lcp.size']).toBe(107400);
 
       const lcp = await (await page.waitForFunction('window._LCP')).jsonValue();
       const lcp2 = await (await page.waitForFunction('window._LCP2')).jsonValue();

dev-packages/browser-integration-tests/suites/tracing/metrics/web-vitals-cls/test.ts

@@ -11,43 +11,49 @@ sentryTest.beforeEach(async ({ browserName, page }) => {
   await page.setViewportSize({ width: 800, height: 1200 });
 });
 
+function getClsAttrs(eventData: Event): Record<string, unknown> {
+  const clsSpan = eventData.spans?.find(({ op }) => op === 'ui.webvital.cls');
+  expect(clsSpan).toBeDefined();
+  return clsSpan?.data ?? {};
+}
+
 sentryTest('should capture a "GOOD" CLS vital with its source(s).', async ({ getLocalTestUrl, page }) => {
   const url = await getLocalTestUrl({ testDir: __dirname });
   const eventData = await getFirstSentryEnvelopeRequest<Event>(page, `${url}#0.05`);
 
-  expect(eventData.measurements).toBeDefined();
-  expect(eventData.measurements?.cls?.value).toBeDefined();
+  const clsAttrs = getClsAttrs(eventData);
+  const clsValue = clsAttrs['browser.web_vital.cls.value'] as number;
 
   // Flakey value dependent on timings -> we check for a range
-  expect(eventData.measurements?.cls?.value).toBeGreaterThan(0.03);
-  expect(eventData.measurements?.cls?.value).toBeLessThan(0.07);
+  expect(clsValue).toBeGreaterThan(0.03);
+  expect(clsValue).toBeLessThan(0.07);
 
-  expect(eventData.contexts?.trace?.data?.['cls.source.1']).toBe('body > div#content > p#partial');
+  expect(clsAttrs['browser.web_vital.cls.source.1']).toBe('body > div#content > p#partial');
 });
 
 sentryTest('should capture a "MEH" CLS vital with its source(s).', async ({ getLocalTestUrl, page }) => {
   const url = await getLocalTestUrl({ testDir: __dirname });
   const eventData = await getFirstSentryEnvelopeRequest<Event>(page, `${url}#0.21`);
 
-  expect(eventData.measurements).toBeDefined();
-  expect(eventData.measurements?.cls?.value).toBeDefined();
+  const clsAttrs = getClsAttrs(eventData);
+  const clsValue = clsAttrs['browser.web_vital.cls.value'] as number;
 
   // Flakey value dependent on timings -> we check for a range
-  expect(eventData.measurements?.cls?.value).toBeGreaterThan(0.18);
-  expect(eventData.measurements?.cls?.value).toBeLessThan(0.23);
+  expect(clsValue).toBeGreaterThan(0.18);
+  expect(clsValue).toBeLessThan(0.23);
 
-  expect(eventData.contexts?.trace?.data?.['cls.source.1']).toBe('body > div#content > p');
+  expect(clsAttrs['browser.web_vital.cls.source.1']).toBe('body > div#content > p');
 });
 
 sentryTest('should capture a "POOR" CLS vital with its source(s).', async ({ getLocalTestUrl, page }) => {
   const url = await getLocalTestUrl({ testDir: __dirname });
   const eventData = await getFirstSentryEnvelopeRequest<Event>(page, `${url}#0.35`);
 
-  expect(eventData.measurements).toBeDefined();
-  expect(eventData.measurements?.cls?.value).toBeDefined();
+  const clsAttrs = getClsAttrs(eventData);
+  const clsValue = clsAttrs['browser.web_vital.cls.value'] as number;
 
   // Flakey value dependent on timings -> we check for a range
-  expect(eventData.measurements?.cls?.value).toBeGreaterThan(0.34);
-  expect(eventData.measurements?.cls?.value).toBeLessThan(0.36);
-  expect(eventData.contexts?.trace?.data?.['cls.source.1']).toBe('body > div#content > p');
+  expect(clsValue).toBeGreaterThan(0.34);
+  expect(clsValue).toBeLessThan(0.36);
+  expect(clsAttrs['browser.web_vital.cls.source.1']).toBe('body > div#content > p');
 });

dev-packages/browser-integration-tests/suites/tracing/metrics/web-vitals-fp-fcp/test.ts

@@ -12,8 +12,7 @@ sentryTest('should capture FP vital.', async ({ browserName, getLocalTestUrl, pa
   const url = await getLocalTestUrl({ testDir: __dirname });
   const eventData = await getFirstSentryEnvelopeRequest<Event>(page, url);
 
-  expect(eventData.measurements).toBeDefined();
-  expect(eventData.measurements?.fp?.value).toBeDefined();
+  expect(eventData.contexts?.trace?.data?.['browser.web_vital.fp.value']).toBeGreaterThan(0);
 
   const fpSpan = eventData.spans?.filter(({ description }) => description === 'first-paint')[0];
 
@@ -30,8 +29,7 @@ sentryTest('should capture FCP vital.', async ({ getLocalTestUrl, page }) => {
   const url = await getLocalTestUrl({ testDir: __dirname });
   const eventData = await getFirstSentryEnvelopeRequest<Event>(page, url);
 
-  expect(eventData.measurements).toBeDefined();
-  expect(eventData.measurements?.fcp?.value).toBeDefined();
+  expect(eventData.contexts?.trace?.data?.['browser.web_vital.fcp.value']).toBeGreaterThan(0);
 
   const fcpSpan = eventData.spans?.filter(({ description }) => description === 'first-contentful-paint')[0];