ref: Use previous source when logging name changes

Author: AbhiPrasad

packages/tracing/src/transaction.ts

@@ -78,9 +78,10 @@ export class Transaction extends SpanClass implements TransactionInterface {
   public setName(name: string, source: TransactionMetadata['source'] = 'custom'): void {
     // `source` could change without the name changing if we discover that an unparameterized route is actually
     // parameterized by virtue of having no parameters in its path
-    if (name !== this.name || source !== this.metadata.source) {
+    if (this.metadata.source && (name !== this.name || source !== this.metadata.source)) {
       this.metadata.changes.push({
-        source,
+        // log previous source
+        source: this.metadata.source,
         timestamp: timestampInSeconds(),
         propagations: this.metadata.propagations,
       });

packages/types/src/transaction.ts

@@ -186,7 +186,7 @@ export interface TransactionNameChange {
    */
   timestamp: number;
 
-  /** New source applied for transaction name change */
+  /** Previous source before transaction name change */
   source: TransactionSource;
 
   /** Number of propagations since start of transaction */