ref(tracing): Add transaction source to default router (#5386)

Author: AbhiPrasad

packages/integration-tests/suites/tracing/browsertracing/navigation/test.ts

@@ -13,6 +13,8 @@ sentryTest('should create a navigation transaction on page navigation', async ({
   expect(pageloadRequest.contexts?.trace.op).toBe('pageload');
   expect(navigationRequest.contexts?.trace.op).toBe('navigation');
 
+  expect(navigationRequest.transaction_info?.source).toEqual('url');
+
   const pageloadTraceId = pageloadRequest.contexts?.trace.trace_id;
   const navigationTraceId = navigationRequest.contexts?.trace.trace_id;
 

packages/integration-tests/suites/tracing/browsertracing/pageload/test.ts

@@ -11,4 +11,5 @@ sentryTest('should create a pageload transaction', async ({ getLocalTestPath, pa
 
   expect(eventData.contexts?.trace?.op).toBe('pageload');
   expect(eventData.spans?.length).toBeGreaterThan(0);
+  expect(eventData.transaction_info?.source).toEqual('url');
 });

packages/tracing/src/browser/browsertracing.ts

@@ -208,6 +208,12 @@ export class BrowserTracing implements Integration {
     const expandedContext = {
       ...context,
       ...parentContextFromHeader,
+      ...(parentContextFromHeader && {
+        metadata: {
+          ...context.metadata,
+          ...parentContextFromHeader.metadata,
+        },
+      }),
       trimEnd: true,
     };
     const modifiedContext = typeof beforeNavigate === 'function' ? beforeNavigate(expandedContext) : expandedContext;

packages/tracing/src/browser/router.ts

@@ -20,7 +20,11 @@ export function instrumentRoutingWithDefaults<T extends Transaction>(
 
   let activeTransaction: T | undefined;
   if (startTransactionOnPageLoad) {
-    activeTransaction = customStartTransaction({ name: global.location.pathname, op: 'pageload' });
+    activeTransaction = customStartTransaction({
+      name: global.location.pathname,
+      op: 'pageload',
+      metadata: { source: 'url' },
+    });
   }
 
   if (startTransactionOnLocationChange) {
@@ -46,7 +50,11 @@ export function instrumentRoutingWithDefaults<T extends Transaction>(
           // If there's an open transaction on the scope, we need to finish it before creating an new one.
           activeTransaction.finish();
         }
-        activeTransaction = customStartTransaction({ name: global.location.pathname, op: 'navigation' });
+        activeTransaction = customStartTransaction({
+          name: global.location.pathname,
+          op: 'navigation',
+          metadata: { source: 'url' },
+        });
       }
     });
   }

packages/tracing/test/browser/router.test.ts

@@ -42,7 +42,11 @@ describe('instrumentRoutingWithDefaults', () => {
   it('starts a pageload transaction', () => {
     instrumentRoutingWithDefaults(customStartTransaction);
     expect(customStartTransaction).toHaveBeenCalledTimes(1);
-    expect(customStartTransaction).toHaveBeenLastCalledWith({ name: 'blank', op: 'pageload' });
+    expect(customStartTransaction).toHaveBeenLastCalledWith({
+      name: 'blank',
+      op: 'pageload',
+      metadata: { source: 'url' },
+    });
   });
 
   it('does not start a pageload transaction if startTransactionOnPageLoad is false', () => {
@@ -58,7 +62,11 @@ describe('instrumentRoutingWithDefaults', () => {
 
     it('it is not created automatically', () => {
       instrumentRoutingWithDefaults(customStartTransaction);
-      expect(customStartTransaction).not.toHaveBeenLastCalledWith({ name: 'blank', op: 'navigation' });
+      expect(customStartTransaction).not.toHaveBeenLastCalledWith({
+        name: 'blank',
+        op: 'navigation',
+        metadata: { source: 'url' },
+      });
     });
 
     it('is created on location change', () => {
@@ -67,7 +75,11 @@ describe('instrumentRoutingWithDefaults', () => {
       expect(addInstrumentationHandlerType).toBe('history');
 
       expect(customStartTransaction).toHaveBeenCalledTimes(2);
-      expect(customStartTransaction).toHaveBeenLastCalledWith({ name: 'blank', op: 'navigation' });
+      expect(customStartTransaction).toHaveBeenLastCalledWith({
+        name: 'blank',
+        op: 'navigation',
+        metadata: { source: 'url' },
+      });
     });
 
     it('is not created if startTransactionOnLocationChange is false', () => {