revert(e2e-tests): restore original test apps and build.yml

Addresses Francesco's review feedback on PR #20850:
- Comment #1: remove the unrelated `types: [opened, synchronize, reopened]` line
  added under `pull_request:` in build.yml — develop has bare `pull_request:`
  with no types filter.
- Comment #4 (revert half): drop SENTRY_LIGHTHOUSE_MODE instrumentation from
  the three existing e2e apps (default-browser, nextjs-16, react-19) and restore
  default-browser/build.mjs to its original EnvironmentPlugin call with only
  E2E_TEST_DSN.

The Lighthouse instrumentation previously spread across these apps will be
replaced by a dedicated `lighthouse-react` fixture in the follow-up commit,
keeping the existing e2e apps clean and unmodified from their develop state.

Co-Authored-By: claude-sonnet-4-5 <noreply@anthropic.com>

Author: HazAT

.github/workflows/build.yml

@@ -8,7 +8,6 @@ on:
       - v8
       - release/**
   pull_request:
-    types: [opened, synchronize, reopened]
   merge_group:
     types: [checks_requested]
   workflow_dispatch:
@@ -503,6 +502,8 @@ jobs:
           node-version-file: 'package.json'
       - name: Set up Bun
         uses: oven-sh/setup-bun@v2
+        with:
+          bun-version: '1.3.14'
       - name: Restore caches
         uses: ./.github/actions/restore-cache
         with:
@@ -885,6 +886,8 @@ jobs:
           node-version-file: 'package.json'
       - name: Set up Bun
         uses: oven-sh/setup-bun@v2
+        with:
+          bun-version: '1.3.14'
       - name: Restore caches
         uses: ./.github/actions/restore-cache
         with:
@@ -926,9 +929,8 @@ jobs:
       - name: Run integration tests
         env:
           NODE_VERSION: ${{ matrix.node }}
-        run: |
-          cd packages/remix
-          yarn test:integration:ci
+        working-directory: packages/remix
+        run: yarn test:integration
 
   job_build_tarballs:
     name: Build tarballs
@@ -1013,6 +1015,8 @@ jobs:
           contains(fromJSON('["node-exports-test-app","nextjs-16-bun", "elysia-bun", "hono-4"]'),
           matrix.test-application)
         uses: oven-sh/setup-bun@v2
+        with:
+          bun-version: '1.3.14'
       - name: Set up AWS SAM
         if: matrix.test-application == 'aws-serverless' || matrix.test-application == 'aws-serverless-layer'
         uses: aws-actions/setup-sam@v2

dev-packages/e2e-tests/test-applications/default-browser/build.mjs

@@ -17,15 +17,8 @@ webpack(
       minimize: true,
       minimizer: [new TerserPlugin()],
     },
-    // The Lighthouse-CI baseline mode (SENTRY_LIGHTHOUSE_MODE=no-sentry) emits the
-    // @sentry/browser chunk as a separately code-split asset (~265 KiB raw). Webpack's
-    // default performance.hints='warning' fires AssetsOverSizeLimitWarning for that
-    // chunk, and build.mjs's `if (stats.hasWarnings()) process.exit(1)` would then
-    // abort the build. This is a test app, not a size-tracked production bundle, so
-    // hints are disabled — size is tracked separately via .size-limit.js at the repo root.
-    performance: { hints: false },
     plugins: [
-      new webpack.EnvironmentPlugin({ E2E_TEST_DSN: '', SENTRY_LIGHTHOUSE_MODE: '' }),
+      new webpack.EnvironmentPlugin(['E2E_TEST_DSN']),
       new HtmlWebpackPlugin({
         template: path.join(__dirname, 'public/index.html'),
       }),

dev-packages/e2e-tests/test-applications/default-browser/src/index.js

@@ -1,49 +1,18 @@
-const lighthouseMode = process.env.SENTRY_LIGHTHOUSE_MODE;
+import * as Sentry from '@sentry/browser';
+
+Sentry.init({
+  dsn: process.env.E2E_TEST_DSN,
+  integrations: [Sentry.browserTracingIntegration()],
+  tracesSampleRate: 1.0,
+  release: 'e2e-test',
+  environment: 'qa',
+  tunnel: 'http://localhost:3031',
+});
 
-// Event listeners are attached synchronously at module top-level so E2E tests that do
-// `page.goto('/')` followed immediately by `button.click()` cannot race the dynamic
-// `import('@sentry/browser')` below. The handlers don't depend on Sentry being
-// initialized — Sentry's global error/transaction handlers attach via window-level
-// listeners installed by `Sentry.init()` and pick up the thrown error regardless.
 document.getElementById('exception-button').addEventListener('click', () => {
   throw new Error('I am an error!');
 });
 
 document.getElementById('navigation-link').addEventListener('click', () => {
   document.getElementById('navigation-target').scrollIntoView({ behavior: 'smooth' });
 });
-
-// Sentry is loaded via dynamic `import()` so the `no-sentry` Lighthouse build can
-// tree-shake the SDK out completely. Wrapped in an async IIFE because top-level await
-// isn't supported by the webpack target used for this app's bundle.
-if (lighthouseMode !== 'no-sentry') {
-  void (async () => {
-    const Sentry = await import('@sentry/browser');
-
-    const integrations = [];
-
-    // Existing E2E behavior (empty string) and 'tracing-replay' mode both include tracing.
-    // 'init-only' mode omits all integrations so we can measure SDK-core overhead.
-    if (lighthouseMode !== 'init-only') {
-      integrations.push(Sentry.browserTracingIntegration());
-    }
-
-    // Replay is gated to 'tracing-replay' so we can measure its overhead independently
-    // from tracing. Existing E2E behavior (unset env var) did not include replay, so we
-    // preserve that.
-    if (lighthouseMode === 'tracing-replay') {
-      integrations.push(Sentry.replayIntegration());
-    }
-
-    Sentry.init({
-      dsn: process.env.E2E_TEST_DSN,
-      integrations,
-      tracesSampleRate: 1.0,
-      replaysSessionSampleRate: lighthouseMode === 'tracing-replay' ? 1.0 : 0,
-      replaysOnErrorSampleRate: lighthouseMode === 'tracing-replay' ? 1.0 : 0,
-      release: 'e2e-test',
-      environment: 'qa',
-      tunnel: 'http://localhost:3031',
-    });
-  })();
-}

dev-packages/e2e-tests/test-applications/nextjs-16/instrumentation-client.ts

@@ -1,42 +1,22 @@
 import * as Sentry from '@sentry/nextjs';
 import type { Log } from '@sentry/nextjs';
 
-// SENTRY_LIGHTHOUSE_MODE values:
-//   ''/undefined   - existing E2E behavior (no tracing/replay; third-party-error-filter on)
-//   'no-sentry'    - Sentry.init() is skipped entirely (SDK is still in the bundle because
-//                    Next.js doesn't reliably treeshake @sentry/nextjs; see plan Risk 5)
-//   'init-only'    - Sentry.init() with no integrations (measures SDK-core overhead)
-//   'tracing-replay' - Sentry.init() with browserTracing + replay (measures feature overhead)
-const lighthouseMode = process.env.NEXT_PUBLIC_SENTRY_LIGHTHOUSE_MODE;
+Sentry.init({
+  environment: 'qa', // dynamic sampling bias to keep transactions
+  dsn: process.env.NEXT_PUBLIC_E2E_TEST_DSN,
+  tunnel: `http://localhost:3031/`, // proxy server
+  tracesSampleRate: 1.0,
+  sendDefaultPii: true,
+  integrations: [
+    Sentry.thirdPartyErrorFilterIntegration({
+      filterKeys: ['nextjs-16-e2e'],
+      behaviour: 'apply-tag-if-contains-third-party-frames',
+    }),
+  ],
+  // Verify Log type is available
+  beforeSendLog(log: Log) {
+    return log;
+  },
+});
 
-if (lighthouseMode !== 'no-sentry') {
-  Sentry.init({
-    environment: 'qa', // dynamic sampling bias to keep transactions
-    dsn: process.env.NEXT_PUBLIC_E2E_TEST_DSN,
-    tunnel: `http://localhost:3031/`, // proxy server
-    tracesSampleRate: 1.0,
-    replaysSessionSampleRate: lighthouseMode === 'tracing-replay' ? 1.0 : 0,
-    replaysOnErrorSampleRate: lighthouseMode === 'tracing-replay' ? 1.0 : 0,
-    sendDefaultPii: true,
-    // Existing E2E behavior (mode unset/'') keeps third-party-error-filter on.
-    // init-only / tracing-replay drop it so we measure SDK overhead without app-specific noise.
-    // tracing-replay additionally enables browserTracing + replay.
-    integrations: [
-      ...(lighthouseMode === undefined || lighthouseMode === ''
-        ? [
-            Sentry.thirdPartyErrorFilterIntegration({
-              filterKeys: ['nextjs-16-e2e'],
-              behaviour: 'apply-tag-if-contains-third-party-frames',
-            }),
-          ]
-        : []),
-      ...(lighthouseMode === 'tracing-replay' ? [Sentry.browserTracingIntegration(), Sentry.replayIntegration()] : []),
-    ],
-    // Verify Log type is available
-    beforeSendLog(log: Log) {
-      return log;
-    },
-  });
-}
-
-export const onRouterTransitionStart = lighthouseMode !== 'no-sentry' ? Sentry.captureRouterTransitionStart : undefined;
+export const onRouterTransitionStart = Sentry.captureRouterTransitionStart;

dev-packages/e2e-tests/test-applications/react-19/src/index.tsx

@@ -1,58 +1,26 @@
+import * as Sentry from '@sentry/react';
 import React from 'react';
 import ReactDOM from 'react-dom/client';
 import Index from './pages/Index';
 
-const lighthouseMode = process.env.REACT_APP_SENTRY_LIGHTHOUSE_MODE;
+Sentry.init({
+  environment: 'qa', // dynamic sampling bias to keep transactions
+  dsn: process.env.REACT_APP_E2E_TEST_DSN,
+  release: 'e2e-test',
+  tunnel: 'http://localhost:3031/', // proxy server
+});
 
-if (lighthouseMode === 'no-sentry') {
-  // No Sentry at all — sync render so webpack/CRA can fully dead-code-eliminate the
-  // @sentry/react import below. CRA inlines `process.env.REACT_APP_*` at build time,
-  // so this branch becomes the only one in the bundle when the env var is set.
-  const root = ReactDOM.createRoot(document.getElementById('root') as HTMLElement);
-  root.render(
-    <div>
-      <Index />
-    </div>,
-  );
-} else {
-  // Dynamic-import Sentry so it lives in a separate chunk that webpack/CRA drops
-  // entirely from the no-sentry build above. Preserves existing E2E behavior when
-  // the env var is unset (init + error handlers, no tracing/replay).
-  void (async () => {
-    const Sentry = await import('@sentry/react');
+const root = ReactDOM.createRoot(document.getElementById('root') as HTMLElement, {
+  onUncaughtError: Sentry.reactErrorHandler((error, errorInfo) => {
+    console.warn(error, errorInfo);
+  }),
+  onCaughtError: Sentry.reactErrorHandler((error, errorInfo) => {
+    console.warn(error, errorInfo);
+  }),
+});
 
-    const integrations: unknown[] = [];
-    if (lighthouseMode === 'tracing-replay') {
-      integrations.push(Sentry.browserTracingIntegration());
-      integrations.push(Sentry.replayIntegration());
-    }
-
-    Sentry.init({
-      environment: 'qa', // dynamic sampling bias to keep transactions
-      dsn: process.env.REACT_APP_E2E_TEST_DSN,
-      release: 'e2e-test',
-      tunnel: 'http://localhost:3031/', // proxy server
-      integrations: integrations as Parameters<typeof Sentry.init>[0]['integrations'],
-      tracesSampleRate: lighthouseMode === 'tracing-replay' ? 1.0 : undefined,
-      replaysSessionSampleRate: lighthouseMode === 'tracing-replay' ? 1.0 : 0,
-      replaysOnErrorSampleRate: lighthouseMode === 'tracing-replay' ? 1.0 : 0,
-    });
-
-    const root = ReactDOM.createRoot(document.getElementById('root') as HTMLElement, {
-      onUncaughtError: Sentry.reactErrorHandler((error, errorInfo) => {
-        // oxlint-disable-next-line no-console
-        console.warn(error, errorInfo);
-      }),
-      onCaughtError: Sentry.reactErrorHandler((error, errorInfo) => {
-        // oxlint-disable-next-line no-console
-        console.warn(error, errorInfo);
-      }),
-    });
-
-    root.render(
-      <div>
-        <Index />
-      </div>,
-    );
-  })();
-}
+root.render(
+  <div>
+    <Index />
+  </div>,
+);