ref(node): Add sentry.parent_span_already_sent attribute (#16870)

This adds a new attribute to spans that we send from the Node SDK
because their parent span was already sent. This can be used by us or
customers to debug why certain things show up as transactions/root spans
in product. We could possibly also use this in the trace view to give
users helpful pointers.

Author: mydea

packages/opentelemetry/src/spanExporter.ts

@@ -191,6 +191,14 @@ export class SentrySpanExporter {
       sentSpans.add(span);
       const transactionEvent = createTransactionForOtelSpan(span);
 
+      // Add an attribute to the transaction event to indicate that this transaction is an orphaned transaction
+      if (root.parentNode && this._sentSpans.has(root.parentNode.id)) {
+        const traceData = transactionEvent.contexts?.trace?.data;
+        if (traceData) {
+          traceData['sentry.parent_span_already_sent'] = true;
+        }
+      }
+
       // We'll recursively add all the child spans to this array
       const spans = transactionEvent.spans || [];
 

packages/opentelemetry/test/integration/transactions.test.ts

@@ -612,6 +612,20 @@ describe('Integration | Transactions', () => {
     expect(transactions).toHaveLength(2);
     expect(transactions[0]?.spans).toHaveLength(1);
 
+    expect(transactions[0]?.transaction).toBe('test name');
+    expect(transactions[0]?.contexts?.trace?.data).toEqual({
+      'sentry.origin': 'manual',
+      'sentry.sample_rate': 1,
+      'sentry.source': 'custom',
+    });
+
+    expect(transactions[1]?.transaction).toBe('inner span 2');
+    expect(transactions[1]?.contexts?.trace?.data).toEqual({
+      'sentry.parent_span_already_sent': true,
+      'sentry.origin': 'manual',
+      'sentry.source': 'custom',
+    });
+
     const finishedSpans: any = exporter['_finishedSpanBuckets'].flatMap(bucket =>
       bucket ? Array.from(bucket.spans) : [],
     );