@sentry/react - npm Package Compare versions

Comparing version 8.47.0 to 8.48.0

build/cjs/reactrouterv6-compat-utils.js

@@ -157,3 +157,3 @@ Object.defineProperty(exports, '__esModule', { value: true });
 
-  const allRoutes = [];
+  const allRoutes = new Set();
 
@@ -183,6 +183,17 @@ const SentryRoutes
         routes.forEach(route => {
-          allRoutes.push(...getChildRoutesRecursively(route));
+          const extractedChildRoutes = getChildRoutesRecursively(route);
+
+          extractedChildRoutes.forEach(r => {
+            allRoutes.add(r);
+          });
         });
 
-        updatePageloadTransaction(getActiveRootSpan(), normalizedLocation, routes, undefined, undefined, allRoutes);
+        updatePageloadTransaction(
+          getActiveRootSpan(),
+          normalizedLocation,
+          routes,
+          undefined,
+          undefined,
+          Array.from(allRoutes),
+        );
         isMountRenderPass.current = false;
@@ -195,3 +206,3 @@ } else {
           version,
-          allRoutes,
+          allRoutes: Array.from(allRoutes),
         });
@@ -315,11 +326,15 @@ }
 
-function getChildRoutesRecursively(route, allRoutes = []) {
-  if (route.children && !route.index) {
-    route.children.forEach(child => {
-      allRoutes.push(...getChildRoutesRecursively(child, allRoutes));
-    });
+function getChildRoutesRecursively(route, allRoutes = new Set()) {
+  if (!allRoutes.has(route)) {
+    allRoutes.add(route);
+
+    if (route.children && !route.index) {
+      route.children.forEach(child => {
+        const childRoutes = getChildRoutesRecursively(child, allRoutes);
+
+        childRoutes.forEach(r => allRoutes.add(r));
+      });
+    }
   }
 
-  allRoutes.push(route);
-
   return allRoutes;
@@ -401,6 +416,6 @@ }
           const newPath = path[0] === '/' || pathBuilder[pathBuilder.length - 1] === '/' ? path : `/${path}`;
-          pathBuilder += newPath;
+          pathBuilder = trimSlash(pathBuilder) + prefixWithSlash(newPath);
 
           // If the path matches the current location, return the path
-          if (location.pathname.endsWith(basename + branch.pathname)) {
+          if (trimSlash(location.pathname) === trimSlash(basename + branch.pathname)) {
             if (
@@ -487,3 +502,3 @@ // If the route defined on the element is something like
 
-  const allRoutes = [];
+  const allRoutes = new Set();
 
@@ -502,6 +517,10 @@ const SentryRoutes = (props) => {
           routes.forEach(route => {
-            allRoutes.push(...getChildRoutesRecursively(route));
+            const extractedChildRoutes = getChildRoutesRecursively(route);
+
+            extractedChildRoutes.forEach(r => {
+              allRoutes.add(r);
+            });
           });
 
-          updatePageloadTransaction(getActiveRootSpan(), location, routes, undefined, undefined, allRoutes);
+          updatePageloadTransaction(getActiveRootSpan(), location, routes, undefined, undefined, Array.from(allRoutes));
           isMountRenderPass.current = false;
@@ -514,3 +533,3 @@ } else {
             version,
-            allRoutes,
+            allRoutes: Array.from(allRoutes),
           });
@@ -517,0 +536,0 @@ }

build/esm/package.json

@@ -1,1 +0,1 @@
-{"type":"module","version":"8.47.0","sideEffects":false}
+{"type":"module","version":"8.48.0","sideEffects":false}

build/esm/reactrouterv6-compat-utils.js

@@ -138,3 +138,3 @@ import { browserTracingIntegration, WINDOW, startBrowserTracingPageLoadSpan, startBrowserTracingNavigationSpan } from '@sentry/browser';
 
-  const allRoutes = [];
+  const allRoutes = new Set();
 
@@ -164,6 +164,17 @@ const SentryRoutes
         routes.forEach(route => {
-          allRoutes.push(...getChildRoutesRecursively(route));
+          const extractedChildRoutes = getChildRoutesRecursively(route);
+
+          extractedChildRoutes.forEach(r => {
+            allRoutes.add(r);
+          });
         });
 
-        updatePageloadTransaction(getActiveRootSpan(), normalizedLocation, routes, undefined, undefined, allRoutes);
+        updatePageloadTransaction(
+          getActiveRootSpan(),
+          normalizedLocation,
+          routes,
+          undefined,
+          undefined,
+          Array.from(allRoutes),
+        );
         isMountRenderPass.current = false;
@@ -176,3 +187,3 @@ } else {
           version,
-          allRoutes,
+          allRoutes: Array.from(allRoutes),
         });
@@ -296,11 +307,15 @@ }
 
-function getChildRoutesRecursively(route, allRoutes = []) {
-  if (route.children && !route.index) {
-    route.children.forEach(child => {
-      allRoutes.push(...getChildRoutesRecursively(child, allRoutes));
-    });
+function getChildRoutesRecursively(route, allRoutes = new Set()) {
+  if (!allRoutes.has(route)) {
+    allRoutes.add(route);
+
+    if (route.children && !route.index) {
+      route.children.forEach(child => {
+        const childRoutes = getChildRoutesRecursively(child, allRoutes);
+
+        childRoutes.forEach(r => allRoutes.add(r));
+      });
+    }
   }
 
-  allRoutes.push(route);
-
   return allRoutes;
@@ -382,6 +397,6 @@ }
           const newPath = path[0] === '/' || pathBuilder[pathBuilder.length - 1] === '/' ? path : `/${path}`;
-          pathBuilder += newPath;
+          pathBuilder = trimSlash(pathBuilder) + prefixWithSlash(newPath);
 
           // If the path matches the current location, return the path
-          if (location.pathname.endsWith(basename + branch.pathname)) {
+          if (trimSlash(location.pathname) === trimSlash(basename + branch.pathname)) {
             if (
@@ -468,3 +483,3 @@ // If the route defined on the element is something like
 
-  const allRoutes = [];
+  const allRoutes = new Set();
 
@@ -483,6 +498,10 @@ const SentryRoutes = (props) => {
           routes.forEach(route => {
-            allRoutes.push(...getChildRoutesRecursively(route));
+            const extractedChildRoutes = getChildRoutesRecursively(route);
+
+            extractedChildRoutes.forEach(r => {
+              allRoutes.add(r);
+            });
           });
 
-          updatePageloadTransaction(getActiveRootSpan(), location, routes, undefined, undefined, allRoutes);
+          updatePageloadTransaction(getActiveRootSpan(), location, routes, undefined, undefined, Array.from(allRoutes));
           isMountRenderPass.current = false;
@@ -495,3 +514,3 @@ } else {
             version,
-            allRoutes,
+            allRoutes: Array.from(allRoutes),
           });
@@ -498,0 +517,0 @@ }

package.json

 {
   "name": "@sentry/react",
-  "version": "8.47.0",
+  "version": "8.48.0",
   "description": "Official Sentry SDK for React.js",
@@ -42,4 +42,4 @@ "repository": "git://github.com/getsentry/sentry-javascript.git",
   "dependencies": {
-    "@sentry/browser": "8.47.0",
-    "@sentry/core": "8.47.0",
+    "@sentry/browser": "8.48.0",
+    "@sentry/core": "8.48.0",
     "hoist-non-react-statics": "^3.3.2"
@@ -46,0 +46,0 @@ },

No alert changes

Improved metrics

Total package byte prevSize

419770

increased by0.77%

Lines of code

4302

increased by0.75%

Worsened metrics

Published versions last week

2

decreased by-33.33%

Dependency changes

• + Added@sentry-internal/browser-utils@8.48.0(transitive)

• + Added@sentry-internal/feedback@8.48.0(transitive)

• + Added@sentry-internal/replay@8.48.0(transitive)

• + Added@sentry-internal/replay-canvas@8.48.0(transitive)

• + Added@sentry/browser@8.48.0(transitive)

• + Added@sentry/core@8.48.0(transitive)

• - Removed@sentry-internal/browser-utils@8.47.0(transitive)

• - Removed@sentry-internal/feedback@8.47.0(transitive)

• - Removed@sentry-internal/replay@8.47.0(transitive)

• - Removed@sentry-internal/replay-canvas@8.47.0(transitive)

• - Removed@sentry/browser@8.47.0(transitive)

• - Removed@sentry/core@8.47.0(transitive)

• Updated@sentry/browser@8.48.0

• Updated@sentry/core@8.48.0