Replace 'tracemetric' in public facing api with 'metric' (eg. enableMetrics) since trace metric is an internal dataset detail

Author: k-fish

dev-packages/browser-integration-tests/suites/public-api/metrics/init.js

@@ -5,7 +5,7 @@ window.Sentry = Sentry;
 Sentry.init({
   dsn: 'https://[email protected]/1337',
   _experiments: {
-    enableTraceMetrics: true,
+    enableMetrics: true,
   },
   release: '1.0.0',
   environment: 'test',

dev-packages/node-core-integration-tests/suites/public-api/metrics/scenario.ts

@@ -7,7 +7,7 @@ const client = Sentry.init({
   release: '1.0.0',
   environment: 'test',
   _experiments: {
-    enableTraceMetrics: true,
+    enableMetrics: true,
   },
   transport: loggingTransport,
 });

dev-packages/node-integration-tests/suites/public-api/metrics/scenario.ts

@@ -6,7 +6,7 @@ Sentry.init({
   release: '1.0.0',
   environment: 'test',
   _experiments: {
-    enableTraceMetrics: true,
+    enableMetrics: true,
   },
   transport: loggingTransport,
 });

packages/browser/src/client.ts

@@ -110,7 +110,7 @@ export class BrowserClient extends Client<BrowserClientOptions> {
 
     const { sendDefaultPii, sendClientReports, enableLogs, _experiments } = this._options;
 
-    if (WINDOW.document && (sendClientReports || enableLogs || _experiments?.enableTraceMetrics)) {
+    if (WINDOW.document && (sendClientReports || enableLogs || _experiments?.enableMetrics)) {
       WINDOW.document.addEventListener('visibilitychange', () => {
         if (WINDOW.document.visibilityState === 'hidden') {
           if (sendClientReports) {
@@ -119,7 +119,7 @@ export class BrowserClient extends Client<BrowserClientOptions> {
           if (enableLogs) {
             _INTERNAL_flushLogsBuffer(this);
           }
-          if (_experiments?.enableTraceMetrics) {
+          if (_experiments?.enableMetrics) {
             _INTERNAL_flushMetricsBuffer(this);
           }
         }
@@ -142,7 +142,7 @@ export class BrowserClient extends Client<BrowserClientOptions> {
       });
     }
 
-    if (_experiments?.enableTraceMetrics) {
+    if (_experiments?.enableMetrics) {
       this.on('flush', () => {
         _INTERNAL_flushMetricsBuffer(this);
       });

packages/core/src/metrics/internal.ts

@@ -134,8 +134,8 @@ export function _INTERNAL_captureMetric(beforeMetric: Metric, options?: Internal
   }
 
   const { release, environment, _experiments } = client.getOptions();
-  if (!_experiments?.enableTraceMetrics) {
-    DEBUG_BUILD && debug.warn('trace metrics option not enabled, metric will not be captured.');
+  if (!_experiments?.enableMetrics) {
+    DEBUG_BUILD && debug.warn('metrics option not enabled, metric will not be captured.');
     return;
   }
 

packages/core/src/metrics/public-api.ts

@@ -38,7 +38,7 @@ function captureMetric(type: MetricType, name: string, value: number | string, o
 }
 
 /**
- * @summary Increment a counter metric. Requires the `_experiments.enableTraceMetrics` option to be enabled.
+ * @summary Increment a counter metric. Requires the `_experiments.enableMetrics` option to be enabled.
  *
  * @param name - The name of the counter metric.
  * @param value - The value to increment by (defaults to 1).
@@ -72,7 +72,7 @@ export function count(name: string, value: number = 1, options?: MetricOptions):
 }
 
 /**
- * @summary Set a gauge metric to a specific value. Requires the `_experiments.enableTraceMetrics` option to be enabled.
+ * @summary Set a gauge metric to a specific value. Requires the `_experiments.enableMetrics` option to be enabled.
  *
  * @param name - The name of the gauge metric.
  * @param value - The current value of the gauge.
@@ -106,7 +106,7 @@ export function gauge(name: string, value: number, options?: MetricOptions): voi
 }
 
 /**
- * @summary Record a value in a distribution metric. Requires the `_experiments.enableTraceMetrics` option to be enabled.
+ * @summary Record a value in a distribution metric. Requires the `_experiments.enableMetrics` option to be enabled.
  *
  * @param name - The name of the distribution metric.
  * @param value - The value to record in the distribution.

packages/core/src/server-runtime-client.ts

@@ -84,7 +84,7 @@ export class ServerRuntimeClient<
       });
     }
 
-    if (this._options._experiments?.enableTraceMetrics) {
+    if (this._options._experiments?.enableMetrics) {
       // eslint-disable-next-line @typescript-eslint/no-this-alias
       const client = this;
 

packages/core/src/types-hoist/options.ts

@@ -285,12 +285,12 @@ export interface ClientOptions<TO extends BaseTransportOptions = BaseTransportOp
     [key: string]: any;
 
     /**
-     * If trace metrics support should be enabled.
+     * If metrics support should be enabled.
      *
      * @default false
      * @experimental
      */
-    enableTraceMetrics?: boolean;
+    enableMetrics?: boolean;
 
     /**
      * An event-processing callback for metrics, guaranteed to be invoked after all other metric