don't break ts setups, fix tests

Author: Lms24

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

@@ -1,8 +1,6 @@
 import * as Sentry from '@sentry/browser';
 
-// type cast necessary because TS thinks this file is part of the main
-// thread where self is of type `Window` instead of `Worker`
-Sentry.registerWebWorker({ self: self as unknown as Worker });
+Sentry.registerWebWorker({ self });
 
 // Let the main thread know the worker is ready
 self.postMessage({

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

@@ -1,8 +1,6 @@
 import * as Sentry from '@sentry/browser';
 
-// type cast necessary because TS thinks this file is part of the main
-// thread where self is of type `Window` instead of `Worker`
-Sentry.registerWebWorker({ self: self as unknown as Worker });
+Sentry.registerWebWorker({ self });
 
 // Let the main thread know the worker is ready
 self.postMessage({

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

@@ -1,8 +1,6 @@
 import * as Sentry from '@sentry/browser';
 
-// type cast necessary because TS thinks this file is part of the main
-// thread where self is of type `Window` instead of `Worker`
-Sentry.registerWebWorker({ self: self as unknown as Worker });
+Sentry.registerWebWorker({ self });
 
 // Let the main thread know the worker is ready
 self.postMessage({

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

@@ -3,8 +3,9 @@
     "target": "ES2022",
     "useDefineForClassFields": true,
     "module": "ESNext",
-    "lib": ["ES2022", "DOM", "DOM.Iterable"],
-    "skipLibCheck": true,
+    "lib": ["ES2022", "DOM", "DOM.Iterable", "WebWorker"],
+    "skipLibCheck": false,
+    "skipDefaultLibCheck": true,
 
     /* Bundler mode */
     "moduleResolution": "bundler",

packages/browser/src/integrations/webWorker.ts

@@ -113,8 +113,21 @@ function listenForSentryDebugIdMessages(worker: Worker): void {
   });
 }
 
+/**
+ * Minimal interface for DedicatedWorkerGlobalScope, only requiring the postMessage method.
+ * (which is the only thing we need from the worker's global object)
+ *
+ * @see https://developer.mozilla.org/en-US/docs/Web/API/DedicatedWorkerGlobalScope
+ *
+ * We can't use the actual type because it breaks everyone who doesn't have {"lib": ["WebWorker"]}
+ * but uses {"skipLibCheck": true} in their tsconfig.json.
+ */
+interface MinimalDedicatedWorkerGlobalScope {
+  postMessage: (message: unknown) => void;
+}
+
 interface RegisterWebWorkerOptions {
-  self: DedicatedWorkerGlobalScope & { _sentryDebugIds?: Record<string, string> };
+  self: MinimalDedicatedWorkerGlobalScope & { _sentryDebugIds?: Record<string, string> };
 }
 
 /**