Renovate/major sentry javascript monorepo

.changeset/@envelop_immediate-introspection-2235-dependencies.md

@@ -0,0 +1,8 @@
+---
+'@envelop/immediate-introspection': major
+---
+
+dependencies updates:
+
+- Updated dependency [`@sentry/node@^8.0.0` ↗︎](https://www.npmjs.com/package/@sentry/node/v/8.0.0)
+  (from `^6 || ^7`, in `peerDependencies`)

.changeset/@envelop_sentry-2235-dependencies.md

@@ -0,0 +1,8 @@
+---
+'@envelop/sentry': major
+---
+
+dependencies updates:
+
+- Updated dependency [`@sentry/node@^8.0.0` ↗︎](https://www.npmjs.com/package/@sentry/node/v/8.0.0)
+  (from `^6 || ^7`, in `peerDependencies`)

packages/plugins/immediate-introspection/package.json

@@ -48,7 +48,7 @@
   "typings": "dist/typings/index.d.ts",
   "peerDependencies": {
     "@envelop/core": "^5.0.0",
-    "@sentry/node": "^6 || ^7",
+    "@sentry/node": "^8.0.0",
     "graphql": "^14.0.0 || ^15.0.0 || ^16.0.0"
   },
   "dependencies": {},

packages/plugins/sentry/package.json

@@ -48,7 +48,7 @@
   "typings": "dist/typings/index.d.ts",
   "peerDependencies": {
     "@envelop/core": "^5.0.2",
-    "@sentry/node": "^6 || ^7",
+    "@sentry/node": "^8.0.0",
     "graphql": "^14.0.0 || ^15.0.0 || ^16.0.0"
   },
   "dependencies": {
@@ -57,9 +57,9 @@
   "devDependencies": {
     "@envelop/core": "workspace:^",
     "@graphql-tools/schema": "10.0.6",
-    "@sentry/node": "7.108.0",
-    "@sentry/tracing": "7.108.0",
-    "@sentry/types": "7.108.0",
+    "@sentry/node": "^8.22.0",
+    "@sentry/tracing": "^7.114.0",
+    "@sentry/types": "^8.22.0",
     "graphql": "16.8.1",
     "sentry-testkit": "5.0.9",
     "typescript": "5.1.3"

packages/plugins/sentry/src/index.ts

@@ -131,15 +131,20 @@ export const useSentry = <PluginContext extends Record<string, any> = {}>(
         ...addedTags,
       };
 
-      let rootSpan: Span;
+      let rootSpan: Span | undefined;
 
       if (startTransaction) {
-        rootSpan = Sentry.startTransaction({
-          name: transactionName,
-          op,
-          tags,
-          ...traceparentData,
-        });
+        Sentry.startSpan(
+          {
+            name: transactionName,
+            op,
+            attributes: tags,
+            ...traceparentData,
+          },
+          span => {
+            rootSpan = span;
+          },
+        );
 
         if (!rootSpan) {
           const error = [
@@ -149,15 +154,33 @@ export const useSentry = <PluginContext extends Record<string, any> = {}>(
           throw new Error(error.join('\n'));
         }
       } else {
-        const scope = Sentry.getCurrentHub().getScope();
-        const parentSpan = scope?.getSpan();
-        const span = parentSpan?.startChild({
-          description: transactionName,
-          op,
-          tags,
+        let childSpan: Span | undefined;
+        const scope = Sentry.getCurrentScope();
+        const parentSpan = scope?.getScopeData().span;
+        if (!parentSpan) {
+          // eslint-disable-next-line no-console
+          console.warn(
+            [
+              `Flag "startTransaction" is disabled but Sentry failed to find a transaction.`,
+              `Try to create a transaction before GraphQL execution phase is started.`,
+            ].join('\n'),
+          );
+          return {};
+        }
+        Sentry.withActiveSpan(parentSpan, () => {
+          Sentry.startSpan(
+            {
+              name: transactionName,
+              op,
+              attributes: tags,
+            },
+            span => {
+              childSpan = span;
+            },
+          );
         });
 
-        if (!span) {
+        if (!childSpan) {
           // eslint-disable-next-line no-console
           console.warn(
             [
@@ -168,26 +191,25 @@ export const useSentry = <PluginContext extends Record<string, any> = {}>(
           return {};
         }
 
-        rootSpan = span;
+        rootSpan = childSpan;
 
         if (renameTransaction) {
           scope!.setTransactionName(transactionName);
         }
       }
 
-      Sentry.configureScope(scope => scope.setSpan(rootSpan));
-
-      rootSpan.setData('document', document);
+      rootSpan.setAttribute('document', document);
 
       if (options.configureScope) {
-        Sentry.configureScope(scope => options.configureScope!(args, scope));
+        options.configureScope(args, Sentry.getCurrentScope());
       }
 
       return {
         onExecuteDone(payload) {
           const handleResult: OnExecuteDoneHookResultOnNextHook<{}> = ({ result, setResult }) => {
             if (includeRawResult) {
-              rootSpan.setData('result', result);
+              // @ts-expect-error TODO: not sure if this is correct
+              rootSpan?.setAttribute('result', result);
             }
 
             if (result.errors && result.errors.length > 0) {
@@ -245,7 +267,7 @@ export const useSentry = <PluginContext extends Record<string, any> = {}>(
               });
             }
 
-            rootSpan.finish();
+            rootSpan?.end();
           };
           return handleStreamOrSingleExecutionResult(payload, handleResult);
         },