Avoid importing types from node inbuilts on vercel-edge, vendor them in

Author: andreiborza

packages/sveltekit/src/client/sdk.ts

@@ -64,7 +64,7 @@ function getDefaultIntegrations(options: BrowserOptions): Integration[] | undefi
  * @returns the function that was previously on `window.fetch`.
  */
 function switchToFetchProxy(): typeof fetch | undefined {
-  const globalWithSentryFetchProxy: WindowWithSentryFetchProxy = WINDOW as WindowWithSentryFetchProxy;
+  const globalWithSentryFetchProxy: WindowWithSentryFetchProxy = WINDOW;
 
   // eslint-disable-next-line @typescript-eslint/unbound-method
   const actualFetch = globalWithSentryFetchProxy.fetch;
@@ -81,7 +81,7 @@ function switchToFetchProxy(): typeof fetch | undefined {
  * and puts our fetch proxy back onto `window._sentryFetchProxy`.
  */
 function restoreFetch(actualFetch: typeof fetch): void {
-  const globalWithSentryFetchProxy: WindowWithSentryFetchProxy = WINDOW as WindowWithSentryFetchProxy;
+  const globalWithSentryFetchProxy: WindowWithSentryFetchProxy = WINDOW;
 
   // eslint-disable-next-line @typescript-eslint/unbound-method
   globalWithSentryFetchProxy._sentryFetchProxy = globalWithSentryFetchProxy.fetch;

packages/vercel-edge/src/vendored/abstract-async-hooks-context-manager.ts

@@ -32,10 +32,22 @@
 /* eslint-disable @typescript-eslint/no-this-alias */
 
 import type { Context, ContextManager } from '@opentelemetry/api';
-import type { EventEmitter } from 'events';
 
 type Func<T> = (...args: unknown[]) => T;
 
+// Inline EventEmitter interface to avoid Node.js module dependency
+// This prevents Node.js type leaks in edge runtime environments
+interface EventEmitter {
+  addListener?(event: string, listener: Func<void>): this;
+  on?(event: string, listener: Func<void>): this;
+  once?(event: string, listener: Func<void>): this;
+  prependListener?(event: string, listener: Func<void>): this;
+  prependOnceListener?(event: string, listener: Func<void>): this;
+  removeListener?(event: string, listener: Func<void>): this;
+  off?(event: string, listener: Func<void>): this;
+  removeAllListeners?(event?: string): this;
+}
+
 /**
  * Store a map for each event of all original listeners and their "patched"
  * version. So when a listener is removed by the user, the corresponding

packages/vercel-edge/src/vendored/async-local-storage-context-manager.ts

@@ -28,10 +28,17 @@
 import type { Context } from '@opentelemetry/api';
 import { ROOT_CONTEXT } from '@opentelemetry/api';
 import { debug, GLOBAL_OBJ } from '@sentry/core';
-import type { AsyncLocalStorage } from 'async_hooks';
 import { DEBUG_BUILD } from '../debug-build';
 import { AbstractAsyncHooksContextManager } from './abstract-async-hooks-context-manager';
 
+// Inline AsyncLocalStorage interface to avoid Node.js module dependency
+// This prevents Node.js type leaks in edge runtime environments
+interface AsyncLocalStorage<T> {
+  getStore(): T | undefined;
+  run<R>(store: T, callback: (...args: any[]) => R, ...args: any[]): R;
+  disable(): void;
+}
+
 export class AsyncLocalStorageContextManager extends AbstractAsyncHooksContextManager {
   private _asyncLocalStorage: AsyncLocalStorage<Context>;
 
@@ -46,12 +53,11 @@ export class AsyncLocalStorageContextManager extends AbstractAsyncHooksContextMa
           "Tried to register AsyncLocalStorage async context strategy in a runtime that doesn't support AsyncLocalStorage.",
         );
 
-      // @ts-expect-error Vendored type shenanigans
       this._asyncLocalStorage = {
         getStore() {
           return undefined;
         },
-        run(_store: unknown, callback: () => Context, ...args: unknown[]) {
+        run<R>(_store: Context, callback: (...args: any[]) => R, ...args: any[]): R {
           return callback.apply(this, args);
         },
         disable() {