feat(node): Make @sentry/node powered by OpenTelemetry (#10762)

This PR makes the former node-experimental package `@sentry/node`, and
the former node package `@sentry/node-experimental`.

All meta-SDKs (nextjs, sveltekit, remix, astro) as well as bun,
serverless, depend on `@sentry/node-experimental` for now. We'll need to
change this over one by one to instead depend on `@sentry/node`, and fix
any things we find along the way.

I've moved the test as much as possible, but didn't specifically invest
a lot of time to fix tests that failed - I left them to point to
node-experimental for now. This mainly affects express tests (we also
have some new express tests, but may need to port some over), and some
select others - we'll also need to investigate these one by one and
replace/update them until none are left.

Author: mydea

dev-packages/e2e-tests/test-applications/node-experimental-fastify-app/package.json

@@ -10,13 +10,11 @@
     "test:assert": "pnpm test"
   },
   "dependencies": {
-    "@sentry/node-experimental": "latest || *",
+    "@sentry/node": "latest || *",
     "@sentry/types": "latest || *",
     "@sentry/core": "latest || *",
     "@sentry/utils": "latest || *",
-    "@sentry/node": "latest || *",
-    "@sentry/opentelemetry-node": "latest || *",
-    "@sentry-internal/tracing": "latest || *",
+    "@sentry/opentelemetry": "latest || *",
     "@types/node": "18.15.1",
     "fastify": "4.23.2",
     "fastify-plugin": "4.5.1",

dev-packages/e2e-tests/test-applications/node-experimental-fastify-app/src/app.js

@@ -1,6 +1,6 @@
 require('./tracing');
 
-const Sentry = require('@sentry/node-experimental');
+const Sentry = require('@sentry/node');
 const { fastify } = require('fastify');
 const fastifyPlugin = require('fastify-plugin');
 const http = require('http');

dev-packages/e2e-tests/test-applications/node-experimental-fastify-app/src/tracing.js

@@ -1,10 +1,9 @@
-const Sentry = require('@sentry/node-experimental');
+const Sentry = require('@sentry/node');
 
 Sentry.init({
   environment: 'qa', // dynamic sampling bias to keep transactions
   dsn: process.env.E2E_TEST_DSN,
   integrations: [],
-  debug: true,
   tracesSampleRate: 1,
   tunnel: 'http://localhost:3031/', // proxy server
 });

dev-packages/e2e-tests/test-applications/node-exports-test-app/package.json

@@ -12,7 +12,7 @@
     "test:assert": "pnpm test"
   },
   "dependencies": {
-    "@sentry/node": "latest || *",
+    "@sentry/node-experimental": "latest || *",
     "@sentry/sveltekit": "latest || *",
     "@sentry/remix": "latest || *",
     "@sentry/astro": "latest || *",

dev-packages/e2e-tests/test-applications/node-exports-test-app/scripts/consistentExports.ts

@@ -1,7 +1,7 @@
 import * as SentryAstro from '@sentry/astro';
 import * as SentryBun from '@sentry/bun';
 import * as SentryNextJs from '@sentry/nextjs';
-import * as SentryNode from '@sentry/node';
+import * as SentryNode from '@sentry/node-experimental';
 import * as SentryRemix from '@sentry/remix';
 import * as SentryServerless from '@sentry/serverless';
 import * as SentrySvelteKit from '@sentry/sveltekit';

dev-packages/e2e-tests/test-applications/node-express-app/src/app.ts

@@ -22,9 +22,6 @@ Sentry.init({
 const app = express();
 const port = 3030;
 
-app.use(Sentry.Handlers.requestHandler());
-app.use(Sentry.Handlers.tracingHandler());
-
 app.get('/test-success', function (req, res) {
   res.send({ version: 'v1' });
 });
@@ -73,7 +70,7 @@ app.get('/test-local-variables-caught', function (req, res) {
   res.send({ exceptionId, randomVariableToRecord });
 });
 
-app.use(Sentry.Handlers.errorHandler());
+Sentry.setupExpressErrorHandler(app);
 
 // @ts-ignore
 app.use(function onError(err, req, res, next) {

dev-packages/e2e-tests/test-applications/node-hapi-app/src/app.js

@@ -10,7 +10,6 @@ Sentry.init({
   environment: 'qa', // dynamic sampling bias to keep transactions
   dsn: process.env.E2E_TEST_DSN,
   includeLocalVariables: true,
-  integrations: [Sentry.hapiIntegration({ server })],
   debug: true,
   tunnel: `http://localhost:3031/`, // proxy server
   tracesSampleRate: 1,
@@ -56,6 +55,7 @@ const init = async () => {
 
 (async () => {
   init();
+  await Sentry.setupHapiErrorHandler(server);
   await server.start();
   console.log('Server running on %s', server.info.uri);
 })();

dev-packages/node-integration-tests/package.json

@@ -16,12 +16,9 @@
     "build:types": "tsc -p tsconfig.types.json",
     "clean": "rimraf -g **/node_modules && run-p clean:script",
     "clean:script": "node scripts/clean.js",
-    "prisma:init": "(cd suites/tracing/prisma-orm && ts-node ./setup.ts)",
-    "prisma:init:new": "(cd suites/tracing-new/prisma-orm && ts-node ./setup.ts)",
     "lint": "eslint . --format stylish",
     "fix": "eslint . --format stylish --fix",
     "type-check": "tsc",
-    "pretest": "run-s --silent prisma:init prisma:init:new",
     "test": "ts-node ./utils/run-tests.ts",
     "jest": "jest --config ./jest.config.js",
     "test:watch": "yarn test --watch"

dev-packages/node-integration-tests/suites/anr/basic-session.js

@@ -12,6 +12,7 @@ Sentry.init({
   release: '1.0',
   debug: true,
   integrations: [Sentry.anrIntegration({ captureStackTrace: true, anrThreshold: 100 })],
+  autoSessionTracking: true,
 });
 
 function longWork() {

dev-packages/node-integration-tests/suites/express/handle-error/server.ts

@@ -1,5 +1,5 @@
 import { loggingTransport, startExpressServerAndSendPortToRunner } from '@sentry-internal/node-integration-tests';
-import * as Sentry from '@sentry/node';
+import * as Sentry from '@sentry/node-experimental';
 import express from 'express';
 
 const app = express();