feat(tracing): Change resource span op name and add tags

Author: AbhiPrasad

packages/eslint-config-sdk/src/index.js

@@ -39,6 +39,10 @@ module.exports = {
         // Enforce type annotations to maintain consistency. This is especially important as
         // we have a public API, so we want changes to be very explicit.
         '@typescript-eslint/typedef': ['error', { arrowParameter: false }],
+
+        // Although for most codebases inferencing the return type is fine, we explicitly ask to annotate
+        // all functions with a return type. This is so that intent is as clear as possible. We are guarding against
+        // cases where you accidently refactor a function's return type to be the wrong type.
         '@typescript-eslint/explicit-function-return-type': ['error', { allowExpressions: true }],
 
         // Consistent ordering of fields, methods and constructors for classes should be enforced

packages/tracing/src/browser/metrics.ts

@@ -1,3 +1,4 @@
+/* eslint-disable max-lines */
 /* eslint-disable @typescript-eslint/no-explicit-any */
 import { SpanContext } from '@sentry/types';
 import { getGlobalObject, logger } from '@sentry/utils';
@@ -214,7 +215,7 @@ function addMeasureSpans(
 /** Create resource related spans */
 function addResourceSpans(
   transaction: Transaction,
-  entry: Record<string, any>,
+  entry: Record<string, unknown>,
   resourceName: string,
   startTime: number,
   duration: number,
@@ -226,14 +227,26 @@ function addResourceSpans(
     return undefined;
   }
 
+  const tags: Record<string, string> = {};
+  if (entry.transferSize) {
+    tags.transferSize = (entry.transferSize as number).toString();
+  }
+  if (entry.encodedBodySize) {
+    tags.encodedBodySize = (entry.encodedBodySize as number).toString();
+  }
+  if (entry.decodedBodySize) {
+    tags.decodedBodySize = (entry.decodedBodySize as number).toString();
+  }
+
   const startTimestamp = timeOrigin + startTime;
   const endTimestamp = startTimestamp + duration;
 
   _startChild(transaction, {
-    description: `${entry.initiatorType} ${resourceName}`,
+    description: resourceName,
     endTimestamp,
-    op: 'resource',
+    op: entry.initiatorType && entry.initiatorType !== '' ? `resource.${entry.initiatorType}` : 'resource',
     startTimestamp,
+    tags,
   });
 
   return endTimestamp;