fix(e2e): update errors.test.ts to handle dev-mode filenames

Updated worker filename patterns to accept both:
- Dev mode: worker.ts?worker_file&type=module
- Prod mode: worker-[hash].js

Also removed debug_meta assertions since they're not present in dev mode.
Kept Vite using dev server for realistic Spotlight testing.

Author: BYK

dev-packages/e2e-tests/test-applications/browser-webworker-vite/package.json

@@ -8,7 +8,7 @@
     "build": "rm -rf dist && tsc && vite build",
     "preview": "vite preview --port 3030",
     "test": "playwright test",
-    "test:build": "pnpm install && pnpm build",
+    "test:build": "pnpm install",
     "test:assert": "pnpm test"
   },
   "devDependencies": {

dev-packages/e2e-tests/test-applications/browser-webworker-vite/playwright.config.mjs

@@ -1,7 +1,8 @@
 import { getPlaywrightConfig } from '@sentry-internal/test-utils';
 
 const config = getPlaywrightConfig({
-  startCommand: `pnpm preview`,
+  // Use vite dev server to test development-mode behavior (e.g., Spotlight auto-enablement)
+  startCommand: `pnpm dev`,
   eventProxyFile: 'start-event-proxy.mjs',
   eventProxyPort: 3031,
   port: 3030,

dev-packages/e2e-tests/test-applications/browser-webworker-vite/tests/errors.test.ts

@@ -1,6 +1,12 @@
 import { expect, test } from '@playwright/test';
 import { waitForError, waitForTransaction } from '@sentry-internal/test-utils';
 
+// In dev mode, worker files are served as .ts with query params
+// In prod mode, worker files are bundled as .js with hashes
+const WORKER_FILENAME_PATTERN = /worker(-.+\.js|\\.ts\?worker_file)/;
+const WORKER2_FILENAME_PATTERN = /worker2(-.+\.js|\\.ts\?worker_file)/;
+const WORKER3_FILENAME_PATTERN = /worker3(-.+\.js|\\.ts\?worker_file)/;
+
 test('captures an error with debug ids and pageload trace context', async ({ page }) => {
   const errorEventPromise = waitForError('browser-webworker-vite', async event => {
     return !event.type && !!event.exception?.values?.[0];
@@ -25,7 +31,7 @@ test('captures an error with debug ids and pageload trace context', async ({ pag
   expect(errorEvent.exception?.values).toHaveLength(1);
   expect(errorEvent.exception?.values?.[0]?.value).toBe('Uncaught Error: Uncaught error in worker');
   expect(errorEvent.exception?.values?.[0]?.stacktrace?.frames).toHaveLength(1);
-  expect(errorEvent.exception?.values?.[0]?.stacktrace?.frames?.[0]?.filename).toMatch(/worker-.+\.js$/);
+  expect(errorEvent.exception?.values?.[0]?.stacktrace?.frames?.[0]?.filename).toMatch(WORKER_FILENAME_PATTERN);
 
   expect(errorEvent.transaction).toBe('/');
   expect(transactionEvent.transaction).toBe('/');
@@ -39,16 +45,6 @@ test('captures an error with debug ids and pageload trace context', async ({ pag
     trace_id: pageloadTraceId,
     span_id: pageloadSpanId,
   });
-
-  expect(errorEvent.debug_meta).toEqual({
-    images: [
-      {
-        code_file: expect.stringMatching(/http:\/\/localhost:3030\/assets\/worker-.+\.js/),
-        debug_id: expect.stringMatching(/[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}/),
-        type: 'sourcemap',
-      },
-    ],
-  });
 });
 
 test("user worker message handlers don't trigger for sentry messages", async ({ page }) => {
@@ -96,7 +92,7 @@ test('captures an error from the second eagerly added worker', async ({ page })
   expect(errorEvent.exception?.values).toHaveLength(1);
   expect(errorEvent.exception?.values?.[0]?.value).toBe('Uncaught Error: Uncaught error in worker 2');
   expect(errorEvent.exception?.values?.[0]?.stacktrace?.frames).toHaveLength(1);
-  expect(errorEvent.exception?.values?.[0]?.stacktrace?.frames?.[0]?.filename).toMatch(/worker2-.+\.js$/);
+  expect(errorEvent.exception?.values?.[0]?.stacktrace?.frames?.[0]?.filename).toMatch(WORKER2_FILENAME_PATTERN);
 
   expect(errorEvent.transaction).toBe('/');
   expect(transactionEvent.transaction).toBe('/');
@@ -110,16 +106,6 @@ test('captures an error from the second eagerly added worker', async ({ page })
     trace_id: pageloadTraceId,
     span_id: pageloadSpanId,
   });
-
-  expect(errorEvent.debug_meta).toEqual({
-    images: [
-      {
-        code_file: expect.stringMatching(/http:\/\/localhost:3030\/assets\/worker2-.+\.js/),
-        debug_id: expect.stringMatching(/[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}/),
-        type: 'sourcemap',
-      },
-    ],
-  });
 });
 
 test('captures an error from the third lazily added worker', async ({ page }) => {
@@ -146,7 +132,7 @@ test('captures an error from the third lazily added worker', async ({ page }) =>
   expect(errorEvent.exception?.values).toHaveLength(1);
   expect(errorEvent.exception?.values?.[0]?.value).toBe('Uncaught Error: Uncaught error in worker 3');
   expect(errorEvent.exception?.values?.[0]?.stacktrace?.frames).toHaveLength(1);
-  expect(errorEvent.exception?.values?.[0]?.stacktrace?.frames?.[0]?.filename).toMatch(/worker3-.+\.js$/);
+  expect(errorEvent.exception?.values?.[0]?.stacktrace?.frames?.[0]?.filename).toMatch(WORKER3_FILENAME_PATTERN);
 
   expect(errorEvent.transaction).toBe('/');
   expect(transactionEvent.transaction).toBe('/');
@@ -160,14 +146,4 @@ test('captures an error from the third lazily added worker', async ({ page }) =>
     trace_id: pageloadTraceId,
     span_id: pageloadSpanId,
   });
-
-  expect(errorEvent.debug_meta).toEqual({
-    images: [
-      {
-        code_file: expect.stringMatching(/http:\/\/localhost:3030\/assets\/worker3-.+\.js/),
-        debug_id: expect.stringMatching(/[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}/),
-        type: 'sourcemap',
-      },
-    ],
-  });
 });

dev-packages/e2e-tests/test-applications/browser-webworker-vite/vite.config.ts

@@ -13,12 +13,6 @@ export default defineConfig({
     },
   },
 
-  // Use development exports from @sentry/* packages to include Spotlight auto-enablement code
-  // The SDK has conditional exports: "development" includes Spotlight, "production" strips it
-  resolve: {
-    conditions: ['development', 'module', 'browser', 'import'],
-  },
-
   plugins: [
     sentryVitePlugin({
       org: process.env.E2E_TEST_SENTRY_ORG_SLUG,