Avoid `Event` type in utils `worldwide`

[Vignatus]

Is there an existing issue for this?

• I have checked for existing issues https://github.com/getsentry/sentry-javascript/issues
• I have reviewed the documentation https://docs.sentry.io/
• I am using the latest SDK release https://github.com/getsentry/sentry-javascript/releases

How do you use Sentry?

Sentry Saas (sentry.io)

Which SDK are you using?

@sentry/node

SDK Version

7.88.0

Framework Version

No response

Link to Sentry event

No response

SDK Setup

No response

Steps to Reproduce

• Installed the latest sentry packages
• Simple http server fails to build
• Shows an error in @sentry/utils/types/worldwide.d.ts

My dependencies are

"dependencies": {
    "@sentry/node": "^7.88.0",
    "@sentry/profiling-node": "^1.3.2",
    "express": "^4.17.3",
    "reflect-metadata": "^0.1.13",
    "rimraf": "^3.0.2"
  },
  "devDependencies": {
    "@sentry/cli": "^2.23.0",
    "@sentry/wizard": "^3.19.0",
    "@types/express": "^4.17.13",
    "@types/node": "^17.0.22",
    "ts-node": "^10.7.0",
    "tsc": "^2.0.4",
    "typescript": "^4.6.3"
  }

Expected Result

A successful build

Actual Result

Failed build with an error from sentry

node_modules/@sentry/utils/types/worldwide.d.ts:24:17 - error TS2304: Cannot find name 'Event'. 
24     (event: Event | string, source?: string, lineno?: number, colno?: number, error?: Error): any;

[mydea]

Hey, thanks for writing in.

This seems to be pulling in some browser-specific types, for whatever reason 🤔 do you happen to have a reproduction for us? That would help a lot!

[Vignatus]

I will try to provide a reproduction when I get the time. However in worldwide.ts, importing Event solves the problem.

Change

import type { Integration } from '@sentry/types';

to

import type { Integration, Event } from '@sentry/types';

Although, I admit, I don't know if its the correct type. But what can be clearly seen is a missing Event import.

[mydea]

Yeah, in this case this is not the Sentry Event but actually the browser Event type - which does not exist in node, but this should be tree shaken out (~~not be used) when using @sentry/node 🤔