feat: Upgrade Sentry JS SDK to v8 (#154)

package.json

@@ -20,8 +20,8 @@
     "README"
   ],
   "dependencies": {
-    "@sentry/node": "7.114.0",
-    "@sentry/profiling-node": "7.114.0",
+    "@sentry/node": "^8.3.0",
+    "@sentry/profiling-node": "^8.3.0",
     "canvas": "^2.11.2",
     "dotenv": "^8.2.0",
     "echarts": "5.4.0",

src/index.ts

@@ -1,6 +1,7 @@
 #!/usr/bin/env node
+// Make sure to import the Sentry SDK before importing any other modules
+import './instrument';
 
-import * as Sentry from '@sentry/node';
 import dotenv from 'dotenv';
 import yargs from 'yargs';
 
@@ -11,7 +12,6 @@ import {renderStream} from './renderStream';
 import {PollingConfig} from './types';
 
 dotenv.config();
-Sentry.init({dsn: process.env.SENTRY_DSN});
 
 const defaultPollingConfig: PollingConfig = {
   /**

src/instrument.ts

@@ -0,0 +1,9 @@
+import * as Sentry from '@sentry/node';
+import {nodeProfilingIntegration} from '@sentry/profiling-node';
+
+Sentry.init({
+  dsn: process.env.SENTRY_DSN,
+  integrations: [nodeProfilingIntegration()],
+  profilesSampleRate: 1,
+  tracesSampleRate: 1,
+});

src/renderServer.ts

@@ -2,7 +2,6 @@
 import {performance} from 'node:perf_hooks';
 
 import * as Sentry from '@sentry/node';
-import {ProfilingIntegration} from '@sentry/profiling-node';
 import express from 'express';
 
 import {ConfigService} from './config';
@@ -17,23 +16,8 @@ export function renderServer(config: ConfigService) {
   const app = express();
   const renderRoutes = express.Router();
 
-  Sentry.init({
-    dsn: process.env.SENTRY_DSN,
-    integrations: [
-      new Sentry.Integrations.Http({tracing: true}),
-      new Sentry.Integrations.Express({router: renderRoutes}),
-      new ProfilingIntegration(),
-    ],
-    profilesSampleRate: 1,
-    tracesSampleRate: 1,
-    _experiments: {
-      metricsAggregator: true,
-    },
-  });
-
   renderRoutes.use(express.json({limit: '20mb'}));
-  renderRoutes.use(Sentry.Handlers.requestHandler());
-  renderRoutes.use(Sentry.Handlers.tracingHandler());
+
   renderRoutes.use((req, resp) => {
     if (!config.isLoaded) {
       resp.status(503).send();
@@ -91,7 +75,6 @@ export function renderServer(config: ConfigService) {
     Sentry.metrics.increment('render.count');
     Sentry.metrics.distribution('render.time', time);
   });
-  renderRoutes.use(Sentry.Handlers.errorHandler());
 
   app.post('/render', renderRoutes);
 
@@ -105,5 +88,7 @@ export function renderServer(config: ConfigService) {
       : resp.status(503).send('NOT CONFIGURED')
   );
 
+  Sentry.setupExpressErrorHandler(app);
+
   return app;
 }

tests/renderStream.spec.ts

@@ -1,12 +1,19 @@
 import type {EChartsOption} from 'echarts';
-import fs from 'node:fs';
+import fs, {readFile} from 'node:fs';
 
 import {ConfigService} from 'app/config';
 import {renderStream} from 'app/renderStream';
 import {RenderData} from 'app/types';
 
 jest.mock('node:fs');
 
+jest.mock('node:fs', () => {
+  return {
+    ...jest.requireActual('node:fs'),
+    readFile: jest.fn(),
+  };
+});
+
 describe('renderStream', () => {
   it('can render graphs given a valid config and render data', async () => {
     const config = new ConfigService('./example');