@sentry/ember 8.0.0-beta.4 → 8.0.0-beta.6

package/addon/index.ts

@@ -5,7 +5,7 @@ import { getOwnConfig, isDevelopingApp, macroCondition } from '@embroider/macros
 import { startSpan } from '@sentry/browser';
 import type { BrowserOptions } from '@sentry/browser';
 import * as Sentry from '@sentry/browser';
-import { SEMANTIC_ATTRIBUTE_SENTRY_SOURCE, applySdkMetadata } from '@sentry/core';
+import { SEMANTIC_ATTRIBUTE_SENTRY_ORIGIN, SEMANTIC_ATTRIBUTE_SENTRY_SOURCE, applySdkMetadata } from '@sentry/core';
 import { GLOBAL_OBJ } from '@sentry/utils';
 import Ember from 'ember';
 
@@ -75,9 +75,11 @@ export const instrumentRoutePerformance = <T extends RouteConstructor>(BaseRoute
       {
         attributes: {
           [SEMANTIC_ATTRIBUTE_SENTRY_SOURCE]: source,
+          [SEMANTIC_ATTRIBUTE_SENTRY_ORIGIN]: 'auto.ui.ember',
         },
         op,
         name,
+        onlyIfParent: true,
       },
       () => {
         return fn(...args);

package/addon/instance-initializers/sentry-performance.ts

@@ -145,6 +145,12 @@ export function _instrumentEmberRouter(
 
   routerService.on('routeWillChange', (transition: Transition) => {
     const { fromRoute, toRoute } = getTransitionInformation(transition, routerService);
+
+    // We want to ignore loading && error routes
+    if (transitionIsIntermediate(transition)) {
+      return;
+    }
+
     activeRootSpan?.end();
 
     activeRootSpan = startBrowserTracingNavigationSpan(client, {
@@ -167,8 +173,8 @@ export function _instrumentEmberRouter(
     });
   });
 
-  routerService.on('routeDidChange', () => {
-    if (!transitionSpan || !activeRootSpan) {
+  routerService.on('routeDidChange', transition => {
+    if (!transitionSpan || !activeRootSpan || transitionIsIntermediate(transition)) {
       return;
     }
     transitionSpan.end();
@@ -492,3 +498,18 @@ function _instrumentNavigation(
 export default {
   initialize,
 };
+
+function transitionIsIntermediate(transition: Transition): boolean {
+  //  We want to use ignore, as this may actually be defined on new versions
+  // eslint-disable-next-line @typescript-eslint/ban-ts-comment
+  // @ts-ignore This actually exists on newer versions
+  const isIntermediate: boolean | undefined = transition.isIntermediate;
+
+  if (typeof isIntermediate === 'boolean') {
+    return isIntermediate;
+  }
+
+  // For versions without this, we look if the route is a `.loading` or `.error` route
+  // This is not perfect and may false-positive in some cases, but it's the best we can do
+  return transition.to?.localName === 'loading' || transition.to?.localName === 'error';
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sentry/ember",
-  "version": "8.0.0-beta.4",
+  "version": "8.0.0-beta.6",
   "description": "Official Sentry SDK for Ember.js",
   "repository": "git://github.com/getsentry/sentry-javascript.git",
   "homepage": "https://github.com/getsentry/sentry-javascript/tree/master/packages/ember",
@@ -25,26 +25,27 @@
     "lint:ts": "tsc",
     "fix": "eslint . --format stylish --fix",
     "start": "ember serve",
-    "test": "ember b --prod && ember test",
+    "test": "ember b --prod && yarn ember test",
     "test:all": "ember try:each",
     "prepack": "ember ts:precompile",
     "postpack": "ember ts:clean"
   },
   "dependencies": {
-    "@embroider/macros": "^1.9.0",
-    "@sentry/browser": "8.0.0-beta.4",
-    "@sentry/core": "8.0.0-beta.4",
-    "@sentry/types": "8.0.0-beta.4",
-    "@sentry/utils": "8.0.0-beta.4",
-    "ember-auto-import": "^1.12.1 || ^2.4.3",
-    "ember-cli-babel": "^7.26.11",
+    "@babel/core": "^7.24.4",
+    "@embroider/macros": "^1.16.0",
+    "@sentry/browser": "8.0.0-beta.6",
+    "@sentry/core": "8.0.0-beta.6",
+    "@sentry/types": "8.0.0-beta.6",
+    "@sentry/utils": "8.0.0-beta.6",
+    "ember-auto-import": "^2.7.2",
+    "ember-cli-babel": "^8.2.0",
     "ember-cli-htmlbars": "^6.1.1",
-    "ember-cli-typescript": "^5.1.1"
+    "ember-cli-typescript": "^5.3.0"
   },
   "devDependencies": {
     "@ember/optional-features": "~1.3.0",
     "@ember/test-helpers": "2.9.4",
-    "@embroider/test-setup": "~1.8.3",
+    "@embroider/test-setup": "~4.0.0",
     "@glimmer/component": "~1.1.2",
     "@glimmer/tracking": "~1.1.2",
     "@types/ember": "~3.16.5",
@@ -63,14 +64,13 @@
     "ember-cli-typescript-blueprints": "~3.0.0",
     "ember-disable-prototype-extensions": "~1.1.3",
     "ember-load-initializers": "~2.1.1",
-    "ember-maybe-import-regenerator": "~1.0.0",
+    "ember-maybe-import-regenerator": "1.0.0",
     "ember-qunit": "~6.0.0",
     "ember-resolver": "11.0.0",
     "ember-sinon-qunit": "7.1.4",
     "ember-source": "~4.8.0",
     "ember-source-channel-url": "~2.0.1",
     "ember-template-lint": "~4.16.1",
-    "ember-test-selectors": "~6.0.0",
     "ember-try": "~2.0.0",
     "ember-window-mock": "~0.8.1",
     "eslint-plugin-ember": "11.9.0",