feat(core): Add span.updateName() and deprecate setName()

You can call `span.setMetadata()` if you need to update the source.

Author: mydea

packages/angular/src/tracing.ts

@@ -118,7 +118,8 @@ export class TraceService implements OnDestroy {
       const transaction = getActiveTransaction();
       // TODO (v8 / #5416): revisit the source condition. Do we want to make the parameterized route the default?
       if (transaction && transaction.metadata.source === 'url') {
-        transaction.setName(route, 'route');
+        transaction.updateName(route);
+        transaction.setMetadata({ source: 'route' });
       }
     }),
   );

packages/angular/test/tracing.test.ts

@@ -10,7 +10,8 @@ let transaction: any;
 const defaultStartTransaction = (ctx: any) => {
   transaction = {
     ...ctx,
-    setName: jest.fn(name => (transaction.name = name)),
+    updateName: jest.fn(name => (transaction.name = name)),
+    setMetadata: jest.fn(),
   };
 
   return transaction;
@@ -110,7 +111,7 @@ describe('Angular Tracing', () => {
             ...ctx.metadata,
             source: 'custom',
           },
-          setName: jest.fn(name => (transaction.name = name)),
+          updateName: jest.fn(name => (transaction.name = name)),
         };
 
         return transaction;
@@ -137,7 +138,7 @@ describe('Angular Tracing', () => {
         metadata: { source: 'url' },
       });
 
-      expect(transaction.setName).toHaveBeenCalledTimes(0);
+      expect(transaction.updateName).toHaveBeenCalledTimes(0);
       expect(transaction.name).toEqual(url);
       expect(transaction.metadata.source).toBe('custom');
 
@@ -327,7 +328,8 @@ describe('Angular Tracing', () => {
           origin: 'auto.navigation.angular',
           metadata: { source: 'url' },
         });
-        expect(transaction.setName).toHaveBeenCalledWith(result, 'route');
+        expect(transaction.updateName).toHaveBeenCalledWith(result);
+        expect(transaction.setMetadata).toHaveBeenCalledWith({ source: 'route' });
 
         env.destroy();
       });

packages/core/src/tracing/span.ts

@@ -168,9 +168,11 @@ export class Span implements SpanInterface {
   public get name(): string {
     return this.description || '';
   }
-  /** Update the name of the span. */
+  /**
+   * Update the name of the span.
+   */
   public set name(name: string) {
-    this.setName(name);
+    this.updateName(name);
   }
 
   /**
@@ -244,10 +246,15 @@ export class Span implements SpanInterface {
     return this;
   }
 
+  /** @inheritdoc */
+  public setName(name: string): void {
+    this.updateName(name);
+  }
+
   /**
    * @inheritDoc
    */
-  public setName(name: string): void {
+  public updateName(name: string): void {
     this.description = name;
   }
 

packages/core/src/tracing/transaction.ts

@@ -82,19 +82,29 @@ export class Transaction extends SpanClass implements TransactionInterface {
     return this._name;
   }
 
-  /** Setter for `name` property, which also sets `source` as custom */
+  /**
+   * Setter for `name` property, which also sets `source` as custom.
+   */
   public set name(newName: string) {
+    // eslint-disable-next-line deprecation/deprecation
     this.setName(newName);
   }
 
   /**
-   * JSDoc
+   * Setter for `name` property, which also sets `source` on the metadata.
+   *
+   * @deprecated Use `updateName()` and `setMetadata()` instead.
    */
   public setName(name: string, source: TransactionMetadata['source'] = 'custom'): void {
     this._name = name;
     this.metadata.source = source;
   }
 
+  /** @inheritdoc */
+  public updateName(name: string): void {
+    this._name = name;
+  }
+
   /**
    * Attaches SpanRecorder to the span itself
    * @param maxlen maximum number of spans that can be recorded

packages/core/test/lib/tracing/span.test.ts

@@ -19,7 +19,7 @@ describe('span', () => {
     expect(span.description).toEqual(undefined);
   });
 
-  it('allows to update the name', () => {
+  it('allows to update the name via setter', () => {
     const span = new Span({ name: 'span name' });
     expect(span.name).toEqual('span name');
     expect(span.description).toEqual('span name');
@@ -29,4 +29,27 @@ describe('span', () => {
     expect(span.name).toEqual('new name');
     expect(span.description).toEqual('new name');
   });
+
+  it('allows to update the name via setName', () => {
+    const span = new Span({ name: 'span name' });
+    expect(span.name).toEqual('span name');
+    expect(span.description).toEqual('span name');
+
+    // eslint-disable-next-line deprecation/deprecation
+    span.setName('new name');
+
+    expect(span.name).toEqual('new name');
+    expect(span.description).toEqual('new name');
+  });
+
+  it('allows to update the name via updateName', () => {
+    const span = new Span({ name: 'span name' });
+    expect(span.name).toEqual('span name');
+    expect(span.description).toEqual('span name');
+
+    span.updateName('new name');
+
+    expect(span.name).toEqual('new name');
+    expect(span.description).toEqual('new name');
+  });
 });

packages/core/test/lib/tracing/transaction.test.ts

@@ -6,12 +6,39 @@ describe('transaction', () => {
     expect(transaction.name).toEqual('span name');
   });
 
-  it('allows to update the name', () => {
+  it('allows to update the name via setter', () => {
     const transaction = new Transaction({ name: 'span name' });
+    transaction.setMetadata({ source: 'route' });
     expect(transaction.name).toEqual('span name');
 
     transaction.name = 'new name';
 
     expect(transaction.name).toEqual('new name');
+    expect(transaction.metadata.source).toEqual('custom');
+  });
+
+  it('allows to update the name via setName', () => {
+    const transaction = new Transaction({ name: 'span name' });
+    transaction.setMetadata({ source: 'route' });
+    expect(transaction.name).toEqual('span name');
+
+    transaction.setMetadata({ source: 'route' });
+
+    // eslint-disable-next-line deprecation/deprecation
+    transaction.setName('new name');
+
+    expect(transaction.name).toEqual('new name');
+    expect(transaction.metadata.source).toEqual('custom');
+  });
+
+  it('allows to update the name via updateName', () => {
+    const transaction = new Transaction({ name: 'span name' });
+    transaction.setMetadata({ source: 'route' });
+    expect(transaction.name).toEqual('span name');
+
+    transaction.updateName('new name');
+
+    expect(transaction.name).toEqual('new name');
+    expect(transaction.metadata.source).toEqual('route');
   });
 });

packages/node/src/handlers.ts

@@ -329,7 +329,8 @@ export function trpcMiddleware(options: SentryTrpcMiddlewareOptions = {}) {
     const sentryTransaction = getCurrentScope().getTransaction();
 
     if (sentryTransaction) {
-      sentryTransaction.setName(`trpc/${path}`, 'route');
+      sentryTransaction.updateName(`trpc/${path}`);
+      sentryTransaction.setMetadata({ source: 'route' });
       sentryTransaction.op = 'rpc.server';
 
       const trpcContext: Record<string, unknown> = {

packages/opentelemetry-node/src/spanprocessor.ts

@@ -216,7 +216,8 @@ function updateTransactionWithOtelData(transaction: Transaction, otelSpan: OtelS
   transaction.setStatus(mapOtelStatus(otelSpan));
 
   transaction.op = op;
-  transaction.setName(description, source);
+  transaction.updateName(description);
+  transaction.setMetadata({ source });
 }
 
 function convertOtelTimeToSeconds([seconds, nano]: [number, number]): number {

packages/react/src/reactrouter.tsx

@@ -165,7 +165,8 @@ export function withSentryRouting<P extends Record<string, any>, R extends React
 
   const WrappedRoute: React.FC<P> = (props: P) => {
     if (activeTransaction && props && props.computedMatch && props.computedMatch.isExact) {
-      activeTransaction.setName(props.computedMatch.path, 'route');
+      activeTransaction.updateName(props.computedMatch.path);
+      activeTransaction.setMetadata({ source: 'route' });
     }
 
     // @ts-expect-error Setting more specific React Component typing for `R` generic above

packages/react/src/reactrouterv6.tsx

@@ -134,7 +134,9 @@ function updatePageloadTransaction(
     : (_matchRoutes(routes, location, basename) as unknown as RouteMatch[]);
 
   if (activeTransaction && branches) {
-    activeTransaction.setName(...getNormalizedName(routes, location, branches, basename));
+    const [name, source] = getNormalizedName(routes, location, branches, basename);
+    activeTransaction.updateName(name);
+    activeTransaction.setMetadata({ source });
   }
 }