feat: Upgrade Sentry JS SDK to v8

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');