feat(vue/v8): Remove all deprecated exports from vue

Author: AbhiPrasad

packages/vue/src/index.ts

@@ -1,13 +1,7 @@
 export * from '@sentry/browser';
 
 export { init } from './sdk';
-// eslint-disable-next-line deprecation/deprecation
-export { vueRouterInstrumentation } from './router';
 export { browserTracingIntegration } from './browserTracingIntegration';
 export { attachErrorHandler } from './errorhandler';
 export { createTracingMixins } from './tracing';
-export {
-  vueIntegration,
-  // eslint-disable-next-line deprecation/deprecation
-  VueIntegration,
-} from './integration';
+export { vueIntegration } from './integration';

packages/vue/src/integration.ts

@@ -1,8 +1,9 @@
-import { convertIntegrationFnToClass, defineIntegration, hasTracingEnabled } from '@sentry/core';
-import type { Client, Integration, IntegrationClass, IntegrationFn } from '@sentry/types';
+import { defineIntegration, hasTracingEnabled } from '@sentry/core';
+import type { Client, IntegrationFn } from '@sentry/types';
 import { GLOBAL_OBJ, arrayify, consoleSandbox } from '@sentry/utils';
 
 import { DEFAULT_HOOKS } from './constants';
+import { DEBUG_BUILD } from './debug-build';
 import { attachErrorHandler } from './errorhandler';
 import { createTracingMixins } from './tracing';
 import type { Options, Vue, VueOptions } from './types';
@@ -33,17 +34,6 @@ const _vueIntegration = ((integrationOptions: Partial<VueOptions> = {}) => {
 
 export const vueIntegration = defineIntegration(_vueIntegration);
 
-/**
- * Initialize Vue error & performance tracking.
- *
- * @deprecated Use `vueIntegration()` instead.
- */
-// eslint-disable-next-line deprecation/deprecation
-export const VueIntegration = convertIntegrationFnToClass(
-  INTEGRATION_NAME,
-  vueIntegration,
-) as IntegrationClass<Integration>;
-
 function _setupIntegration(client: Client, integrationOptions: Partial<VueOptions>): void {
   const options: Options = { ...DEFAULT_CONFIG, ...client.getOptions(), ...integrationOptions };
   if (!options.Vue && !options.app) {
@@ -67,23 +57,25 @@ Update your \`Sentry.init\` call with an appropriate config option:
 }
 
 const vueInit = (app: Vue, options: Options): void => {
-  // Check app is not mounted yet - should be mounted _after_ init()!
-  // This is _somewhat_ private, but in the case that this doesn't exist we simply ignore it
-  // See: https://github.com/vuejs/core/blob/eb2a83283caa9de0a45881d860a3cbd9d0bdd279/packages/runtime-core/src/component.ts#L394
-  const appWithInstance = app as Vue & {
-    _instance?: {
-      isMounted?: boolean;
+  if (DEBUG_BUILD) {
+    // Check app is not mounted yet - should be mounted _after_ init()!
+    // This is _somewhat_ private, but in the case that this doesn't exist we simply ignore it
+    // See: https://github.com/vuejs/core/blob/eb2a83283caa9de0a45881d860a3cbd9d0bdd279/packages/runtime-core/src/component.ts#L394
+    const appWithInstance = app as Vue & {
+      _instance?: {
+        isMounted?: boolean;
+      };
     };
-  };
 
-  const isMounted = appWithInstance._instance && appWithInstance._instance.isMounted;
-  if (isMounted === true) {
-    consoleSandbox(() => {
-      // eslint-disable-next-line no-console
-      console.warn(
-        '[@sentry/vue]: Misconfigured SDK. Vue app is already mounted. Make sure to call `app.mount()` after `Sentry.init()`.',
-      );
-    });
+    const isMounted = appWithInstance._instance && appWithInstance._instance.isMounted;
+    if (isMounted === true) {
+      consoleSandbox(() => {
+        // eslint-disable-next-line no-console
+        console.warn(
+          '[@sentry/vue]: Misconfigured SDK. Vue app is already mounted. Make sure to call `app.mount()` after `Sentry.init()`.',
+        );
+      });
+    }
   }
 
   attachErrorHandler(app, options);

packages/vue/src/router.ts

@@ -1,9 +1,13 @@
 import { WINDOW, captureException } from '@sentry/browser';
-import { SEMANTIC_ATTRIBUTE_SENTRY_ORIGIN, SEMANTIC_ATTRIBUTE_SENTRY_SOURCE, spanToJSON } from '@sentry/core';
+import {
+  SEMANTIC_ATTRIBUTE_SENTRY_ORIGIN,
+  SEMANTIC_ATTRIBUTE_SENTRY_SOURCE,
+  getActiveSpan,
+  getRootSpan,
+  spanToJSON,
+} from '@sentry/core';
 import type { SpanAttributes, Transaction, TransactionContext, TransactionSource } from '@sentry/types';
 
-import { getActiveTransaction } from './tracing';
-
 interface VueRouterInstrumationOptions {
   /**
    * What to use for route labels.
@@ -139,17 +143,17 @@ export function instrumentVueRouter(
     }
 
     if (options.instrumentPageLoad && isPageLoadNavigation) {
-      // eslint-disable-next-line deprecation/deprecation
-      const pageloadTransaction = getActiveTransaction();
-      if (pageloadTransaction) {
-        const existingAttributes = spanToJSON(pageloadTransaction).data || {};
+      const activeSpan = getActiveSpan();
+      const rootSpan = activeSpan && getRootSpan(activeSpan);
+      if (rootSpan) {
+        const existingAttributes = spanToJSON(rootSpan).data || {};
         if (existingAttributes[SEMANTIC_ATTRIBUTE_SENTRY_SOURCE] !== 'custom') {
-          pageloadTransaction.updateName(transactionName);
-          pageloadTransaction.setAttribute(SEMANTIC_ATTRIBUTE_SENTRY_SOURCE, transactionSource);
+          rootSpan.updateName(transactionName);
+          rootSpan.setAttribute(SEMANTIC_ATTRIBUTE_SENTRY_SOURCE, transactionSource);
         }
         // Set router attributes on the existing pageload transaction
-        // This will  the origin, and add params & query attributes
-        pageloadTransaction.setAttributes({
+        // This will override the origin, and add params & query attributes
+        rootSpan.setAttributes({
           ...attributes,
           [SEMANTIC_ATTRIBUTE_SENTRY_ORIGIN]: 'auto.pageload.vue',
         });
@@ -158,6 +162,7 @@ export function instrumentVueRouter(
 
     if (options.instrumentNavigation && !isPageLoadNavigation) {
       attributes[SEMANTIC_ATTRIBUTE_SENTRY_SOURCE] = transactionSource;
+      attributes[SEMANTIC_ATTRIBUTE_SENTRY_ORIGIN] = 'auto.navigation.vue';
       startNavigationSpanFn({
         name: transactionName,
         op: 'navigation',

packages/vue/src/tracing.ts

@@ -1,5 +1,5 @@
-import { getActiveSpan, getCurrentScope, startInactiveSpan } from '@sentry/browser';
-import type { Span, Transaction } from '@sentry/types';
+import { getActiveSpan, startInactiveSpan } from '@sentry/browser';
+import type { Span } from '@sentry/types';
 import { logger, timestampInSeconds } from '@sentry/utils';
 
 import { DEFAULT_HOOKS } from './constants';
@@ -32,16 +32,6 @@ const HOOKS: { [key in Operation]: Hook[] } = {
   update: ['beforeUpdate', 'updated'],
 };
 
-/**
- * Grabs active transaction off scope.
- *
- * @deprecated You should not rely on the transaction, but just use `startSpan()` APIs instead.
- */
-export function getActiveTransaction(): Transaction | undefined {
-  // eslint-disable-next-line deprecation/deprecation
-  return getCurrentScope().getTransaction();
-}
-
 /** Finish top-level span and activity with a debounce configured using `timeout` option */
 function finishRootSpan(vm: VueSentry, timestamp: number, timeout: number): void {
   if (vm.$_sentryRootSpanTimer) {