feat(astro): Update @sentry/astro to use OpenTelemetry

Author: mydea

packages/astro/package.json

@@ -51,7 +51,7 @@
   "dependencies": {
     "@sentry/browser": "8.0.0-alpha.2",
     "@sentry/core": "8.0.0-alpha.2",
-    "@sentry/node-experimental": "8.0.0-alpha.2",
+    "@sentry/node": "8.0.0-alpha.2",
     "@sentry/types": "8.0.0-alpha.2",
     "@sentry/utils": "8.0.0-alpha.2",
     "@sentry/vite-plugin": "^2.14.2"

packages/astro/src/index.server.ts

@@ -1,5 +1,5 @@
 // Node SDK exports
-// Unfortunately, we cannot `export * from '@sentry/node-experimental'` because in prod builds,
+// Unfortunately, we cannot `export * from '@sentry/node'` because in prod builds,
 // Vite puts these exports into a `default` property (Sentry.default) rather than
 // on the top - level namespace.
 
@@ -37,7 +37,6 @@ export {
   setHttpStatus,
   withScope,
   withIsolationScope,
-  autoDiscoverNodePerformanceMonitoringIntegrations,
   makeNodeTransport,
   getDefaultIntegrations,
   defaultStackParser,
@@ -47,7 +46,6 @@ export {
   addRequestDataToEvent,
   DEFAULT_USER_INCLUDES,
   extractRequestData,
-  Integrations,
   consoleIntegration,
   onUncaughtExceptionIntegration,
   onUnhandledRejectionIntegration,
@@ -59,7 +57,6 @@ export {
   functionToStringIntegration,
   inboundFiltersIntegration,
   linkedErrorsIntegration,
-  Handlers,
   setMeasurement,
   getActiveSpan,
   startSpan,
@@ -74,10 +71,24 @@ export {
   SEMANTIC_ATTRIBUTE_SENTRY_ORIGIN,
   SEMANTIC_ATTRIBUTE_SENTRY_SOURCE,
   SEMANTIC_ATTRIBUTE_SENTRY_SAMPLE_RATE,
-} from '@sentry/node-experimental';
+  expressIntegration,
+  expressErrorHandler,
+  setupExpressErrorHandler,
+  fastifyIntegration,
+  graphqlIntegration,
+  mongoIntegration,
+  mongooseIntegration,
+  mysqlIntegration,
+  mysql2Integration,
+  nestIntegration,
+  postgresIntegration,
+  prismaIntegration,
+  hapiIntegration,
+  setupHapiErrorHandler,
+} from '@sentry/node';
 
 // We can still leave this for the carrier init and type exports
-export * from '@sentry/node-experimental';
+export * from '@sentry/node';
 
 export { init } from './server/sdk';
 

packages/astro/src/index.types.ts

@@ -13,10 +13,6 @@ import sentryAstro from './index.server';
 /** Initializes Sentry Astro SDK */
 export declare function init(options: Options | clientSdk.BrowserOptions | serverSdk.NodeOptions): void;
 
-// We only export server Integrations for now, until the exports are removed from Svelte and Node SDKs.
-// Necessary to avoid type collision.
-export declare const Integrations: typeof serverSdk.Integrations;
-
 export declare const linkedErrorsIntegration: typeof clientSdk.linkedErrorsIntegration;
 export declare const contextLinesIntegration: typeof clientSdk.contextLinesIntegration;
 
@@ -26,5 +22,17 @@ export declare const defaultStackParser: StackParser;
 export declare function close(timeout?: number | undefined): PromiseLike<boolean>;
 export declare function flush(timeout?: number | undefined): PromiseLike<boolean>;
 
+// eslint-disable-next-line deprecation/deprecation
+export declare const makeMain: typeof clientSdk.makeMain;
+export declare const getActiveSpan: typeof clientSdk.getActiveSpan;
+// eslint-disable-next-line deprecation/deprecation
+export declare const getCurrentHub: typeof clientSdk.getCurrentHub;
+export declare const getClient: typeof clientSdk.getClient;
+export declare const startSpan: typeof clientSdk.startSpan;
+export declare const startInactiveSpan: typeof clientSdk.startInactiveSpan;
+export declare const startSpanManual: typeof clientSdk.startSpanManual;
+export declare const withActiveSpan: typeof clientSdk.withActiveSpan;
+export declare const Span: clientSdk.Span;
+
 export declare const metrics: typeof clientSdk.metrics & typeof serverSdk.metrics;
 export default sentryAstro;

packages/astro/src/integration/index.ts

@@ -91,7 +91,7 @@ export const sentryAstro = (options: SentryOptions = {}): AstroIntegration => {
                   // @sentry/node is required in case we have 2 different @sentry/node
                   // packages installed in the same project.
                   // Ref: https://github.com/getsentry/sentry-javascript/issues/10121
-                  noExternal: ['@sentry/astro', '@sentry/node-experimental', '@sentry/node'],
+                  noExternal: ['@sentry/astro', '@sentry/node'],
                 },
               },
             });

packages/astro/src/server/middleware.ts

@@ -1,13 +1,15 @@
-import { SEMANTIC_ATTRIBUTE_SENTRY_ORIGIN, SEMANTIC_ATTRIBUTE_SENTRY_SOURCE, setHttpStatus } from '@sentry/core';
 import {
+  SEMANTIC_ATTRIBUTE_SENTRY_ORIGIN,
+  SEMANTIC_ATTRIBUTE_SENTRY_SOURCE,
   captureException,
   continueTrace,
   getActiveSpan,
   getClient,
   getCurrentScope,
+  setHttpStatus,
   startSpan,
   withIsolationScope,
-} from '@sentry/node-experimental';
+} from '@sentry/node';
 import type { Client, Scope, Span, SpanAttributes } from '@sentry/types';
 import { addNonEnumerableProperty, objectify, stripUrlQueryAndFragment } from '@sentry/utils';
 import type { APIContext, MiddlewareResponseHandler } from 'astro';

packages/astro/src/server/sdk.ts

@@ -1,6 +1,6 @@
 import { applySdkMetadata } from '@sentry/core';
-import type { NodeOptions } from '@sentry/node-experimental';
-import { init as initNodeSdk, setTag } from '@sentry/node-experimental';
+import type { NodeOptions } from '@sentry/node';
+import { init as initNodeSdk, setTag } from '@sentry/node';
 
 /**
  *

packages/astro/test/client/sdk.test.ts

@@ -1,11 +1,15 @@
 import type { BrowserClient } from '@sentry/browser';
-import { getActiveSpan } from '@sentry/browser';
-import { browserTracingIntegration } from '@sentry/browser';
+import {
+  browserTracingIntegration,
+  getActiveSpan,
+  getCurrentScope,
+  getGlobalScope,
+  getIsolationScope,
+} from '@sentry/browser';
 import * as SentryBrowser from '@sentry/browser';
 import { SDK_VERSION, getClient } from '@sentry/browser';
 import { vi } from 'vitest';
 
-import { getCurrentScope, getGlobalScope, getIsolationScope } from '@sentry/core';
 import { init } from '../../../astro/src/client/sdk';
 
 const browserInit = vi.spyOn(SentryBrowser, 'init');

packages/astro/test/server/meta.test.ts

@@ -5,7 +5,7 @@ import { vi } from 'vitest';
 
 import { getTracingMetaTags, isValidBaggageString } from '../../src/server/meta';
 
-const TRACE_FLAG_SAMPLED = 0x1;
+const TRACE_FLAG_SAMPLED = 1;
 
 const mockedSpan = new SentrySpan({
   traceId: '12345678901234567890123456789012',

packages/astro/test/server/middleware.test.ts

@@ -1,5 +1,5 @@
 import { SEMANTIC_ATTRIBUTE_SENTRY_SOURCE } from '@sentry/core';
-import * as SentryNode from '@sentry/node-experimental';
+import * as SentryNode from '@sentry/node';
 import type { Client, Span } from '@sentry/types';
 import { vi } from 'vitest';
 

packages/astro/test/server/sdk.test.ts

@@ -1,5 +1,5 @@
-import * as SentryNode from '@sentry/node-experimental';
-import { SDK_VERSION } from '@sentry/node-experimental';
+import * as SentryNode from '@sentry/node';
+import { SDK_VERSION } from '@sentry/node';
 import { vi } from 'vitest';
 
 import { init } from '../../src/server/sdk';