fixup! feat(effect): Add metrics to Sentry.effectLayer

Author: JPeer264

packages/effect/src/metrics.ts

@@ -20,18 +20,18 @@ function sendMetricToSentry(pair: MetricPair.MetricPair.Untyped): void {
   const attributes = labelsToAttributes(metricKey.tags);
 
   if (MetricState.isCounterState(metricState)) {
-    const value = typeof metricState.count === 'bigint' ? Number(metricState.count) : metricState.count;
+    const value = Number(metricState.count);
     sentryMetrics.count(name, value, { attributes });
   } else if (MetricState.isGaugeState(metricState)) {
-    const value = typeof metricState.value === 'bigint' ? Number(metricState.value) : metricState.value;
+    const value = Number(metricState.value);
     sentryMetrics.gauge(name, value, { attributes });
   } else if (MetricState.isHistogramState(metricState)) {
-    sentryMetrics.distribution(`${name}.sum`, metricState.sum, { attributes });
+    sentryMetrics.gauge(`${name}.sum`, metricState.sum, { attributes });
     sentryMetrics.gauge(`${name}.count`, metricState.count, { attributes });
     sentryMetrics.gauge(`${name}.min`, metricState.min, { attributes });
     sentryMetrics.gauge(`${name}.max`, metricState.max, { attributes });
   } else if (MetricState.isSummaryState(metricState)) {
-    sentryMetrics.distribution(`${name}.sum`, metricState.sum, { attributes });
+    sentryMetrics.gauge(`${name}.sum`, metricState.sum, { attributes });
     sentryMetrics.gauge(`${name}.count`, metricState.count, { attributes });
     sentryMetrics.gauge(`${name}.min`, metricState.min, { attributes });
     sentryMetrics.gauge(`${name}.max`, metricState.max, { attributes });
@@ -59,7 +59,7 @@ function sendDeltaMetricToSentry(
   const metricId = getMetricId(pair);
 
   if (MetricState.isCounterState(metricState)) {
-    const currentValue = typeof metricState.count === 'bigint' ? Number(metricState.count) : metricState.count;
+    const currentValue = Number(metricState.count);
 
     const previousValue = previousCounterValues.get(metricId) ?? 0;
     const delta = currentValue - previousValue;

packages/effect/src/server/index.ts

@@ -16,7 +16,7 @@ export type EffectServerLayerOptions = NodeOptions;
  * This layer provides Effect applications with full Sentry instrumentation including:
  * - Effect spans traced as Sentry spans
  * - Effect logs forwarded to Sentry (when `enableLogs` is set)
-
+ * - Effect metrics sent to Sentry (when `enableMetrics` is set)
  *
  * @example
  * ```typescript

packages/effect/test/metrics.test.ts

@@ -196,7 +196,7 @@ describe('createMetricsFlusher', () => {
 
       flusher.flush();
 
-      expect(mockDistribution).toHaveBeenCalledWith('flush_test_histogram.sum', expect.any(Number), { attributes: {} });
+      expect(mockGauge).toHaveBeenCalledWith('flush_test_histogram.sum', expect.any(Number), { attributes: {} });
       expect(mockGauge).toHaveBeenCalledWith('flush_test_histogram.count', expect.any(Number), { attributes: {} });
       expect(mockGauge).toHaveBeenCalledWith('flush_test_histogram.min', expect.any(Number), { attributes: {} });
       expect(mockGauge).toHaveBeenCalledWith('flush_test_histogram.max', expect.any(Number), { attributes: {} });
@@ -220,7 +220,7 @@ describe('createMetricsFlusher', () => {
 
       flusher.flush();
 
-      expect(mockDistribution).toHaveBeenCalledWith('flush_test_summary.sum', 60, { attributes: {} });
+      expect(mockGauge).toHaveBeenCalledWith('flush_test_summary.sum', 60, { attributes: {} });
       expect(mockGauge).toHaveBeenCalledWith('flush_test_summary.count', 3, { attributes: {} });
       expect(mockGauge).toHaveBeenCalledWith('flush_test_summary.min', 10, { attributes: {} });
       expect(mockGauge).toHaveBeenCalledWith('flush_test_summary.max', 30, { attributes: {} });