@dxos/observability 0.8.3 → 0.8.4-main.1068cf700f

package/src/sentry/sentry-log-processor.ts

@@ -1,166 +0,0 @@
-//
-// Copyright 2024 DXOS.org
-//
-
-import type { Breadcrumb, SeverityLevel, Event } from '@sentry/types';
-
-import { InvariantViolation } from '@dxos/invariant';
-import { type LogConfig, type LogEntry, LogLevel, type LogProcessor, shouldLog } from '@dxos/log';
-import { CircularBuffer, getDebugName } from '@dxos/util';
-
-import { withScope, captureException, captureMessage } from './node';
-
-const MAX_LOG_BREADCRUMBS = 150;
-
-export class SentryLogProcessor {
-  private _breadcrumbs = new CircularBuffer<Breadcrumb>(MAX_LOG_BREADCRUMBS);
-
-  public readonly logProcessor: LogProcessor = (config: LogConfig, entry: LogEntry) => {
-    const { level, meta, error } = entry;
-    // Don't forward logs from remote sessions.
-    if (!shouldLog(entry, config.captureFilters) || meta?.S?.remoteSessionId) {
-      if (entry.level > LogLevel.DEBUG) {
-        this._addBreadcrumb(undefined, entry.message, convertLevel(entry.level), undefined);
-      }
-      return;
-    }
-    if (entry.level !== LogLevel.WARN && entry.level !== LogLevel.ERROR) {
-      return;
-    }
-
-    // TODO(nf): add rate limiting to avoid spamming Sentry/consuming excessive quota.
-    withScope((scope) => {
-      const severity = convertLevel(level);
-      scope.setLevel(severity);
-      scope.setContext('dxoslog', entry.context ?? null);
-      if (meta) {
-        scope.setTag('transaction', `${getRelativeFilename(meta.F)}:${meta.L}`);
-
-        if (meta.S?.hostSessionId) {
-          scope.setTags({
-            service_host_issue: true,
-            service_host_session: meta.S?.hostSessionId,
-          });
-        }
-
-        if (!Number.isNaN(meta.S?.uptimeSeconds)) {
-          scope.setExtra('uptime_seconds', meta.S?.uptimeSeconds);
-        }
-      }
-
-      const extendedMessage = formatMessageForSentry(entry);
-      let capturedError = error;
-      if (capturedError == null && entry.level === LogLevel.ERROR) {
-        capturedError = Object.values(entry.context ?? {}).find((v): v is Error => v instanceof Error);
-      }
-      if (capturedError) {
-        if (capturedError instanceof InvariantViolation) {
-          scope.setExtra('invariant_violation', true);
-        }
-        const isMessageDifferentFromStackTrace = error == null;
-        if (isMessageDifferentFromStackTrace) {
-          scope.setExtra('message', extendedMessage);
-        }
-        const eventId = captureException(capturedError);
-        this._addBreadcrumb(eventId, extendedMessage, severity, entry.context);
-        return;
-      }
-
-      scope.setFingerprint([entry.message]);
-      const eventId = captureMessage(extendedMessage);
-      this._addBreadcrumb(eventId, extendedMessage, severity, entry.context);
-    });
-  };
-
-  public addLogBreadcrumbsTo(event: Event): void {
-    event.breadcrumbs ??= [];
-    for (const breadcrumb of this._breadcrumbs) {
-      event.breadcrumbs.push(breadcrumb);
-    }
-    event.breadcrumbs.sort((b1, b2) => {
-      if (b1.timestamp === undefined || b2.timestamp === undefined) {
-        return b1.timestamp === b2.timestamp ? 0 : b1.timestamp === undefined ? -1 : 1;
-      }
-      return b1.timestamp - b2.timestamp;
-    });
-  }
-
-  private _addBreadcrumb(
-    eventId: string | undefined,
-    message: string,
-    severity: SeverityLevel,
-    context: { [key: string]: any } | undefined,
-  ): void {
-    const breadcrumb: Breadcrumb = {
-      type: 'console',
-      level: severity,
-      event_id: eventId,
-      category: 'log',
-      message,
-      data: context,
-      timestamp: Math.floor(Date.now() / 1000),
-    };
-    const lastRecorded = this._breadcrumbs.getLast();
-    if (lastRecorded && lastRecorded.message === breadcrumb.message) {
-      const firstBreadcrumbData = lastRecorded.data?.firstBreadcrumbData ?? lastRecorded.data ?? {};
-      const mergedBreadcrumbCount = Number.isNaN(lastRecorded.data?.mergedBreadcrumbCount)
-        ? 1
-        : Number(lastRecorded.data?.mergedBreadcrumbCount);
-      lastRecorded.data = {
-        mergedBreadcrumbCount: mergedBreadcrumbCount + 1,
-        firstBreadcrumbData,
-        lastBreadcrumbData: breadcrumb.data,
-      };
-      return;
-    }
-    this._breadcrumbs.push(breadcrumb);
-  }
-}
-
-const formatMessageForSentry = (entry: LogEntry) => {
-  let scopePrefix: string | undefined;
-  if (entry.meta?.S) {
-    const scope = entry.meta?.S;
-    scopePrefix = scope.name || getDebugName(scope);
-  }
-  if (scopePrefix == null) {
-    return entry.message;
-  }
-
-  const workerPrefix = entry.meta?.S?.hostSessionId ? '[worker] ' : '';
-  return `${workerPrefix}${scopePrefix} ${entry.message}`;
-};
-
-const convertLevel = (level: LogLevel): SeverityLevel => {
-  switch (level) {
-    case LogLevel.ERROR:
-      return 'error';
-    case LogLevel.VERBOSE:
-      return 'info';
-    case LogLevel.INFO:
-      return 'info';
-    case LogLevel.WARN:
-      return 'warning';
-    case LogLevel.DEBUG:
-    case LogLevel.TRACE:
-      return 'debug';
-    default:
-      throw never(level);
-  }
-};
-
-const never = (_: never) => {
-  return new Error('unhandled value');
-};
-
-const getRelativeFilename = (filename: string) => {
-  // TODO(burdon): Hack uses "packages" as an anchor (pre-parse NX?)
-  // Including `packages/` part of the path so that excluded paths (e.g. from dist) are clickable in vscode.
-  const match = filename.match(/.+\/(packages\/.+\/.+)/);
-  if (match) {
-    const [, filePath] = match;
-    return filePath;
-  }
-
-  return filename;
-};

package/src/sentry/sentry.node.test.ts

@@ -1,34 +0,0 @@
-//
-// Copyright 2022 DXOS.org
-//
-
-import { beforeAll, beforeEach, describe, expect, test } from 'vitest';
-
-import * as Sentry from './node';
-import { sentryTestkit } from '../testing';
-
-const { testkit, sentryTransport } = sentryTestkit();
-
-const MOCK_DESTINATION = 'https://acacaeaccacacacabcaacdacdacadaca@sentry.io/000001';
-
-describe('Error reporting', () => {
-  beforeAll(() => {
-    Sentry.init({
-      destination: MOCK_DESTINATION,
-      release: 'test',
-      transport: sentryTransport,
-    });
-  });
-
-  beforeEach(() => {
-    testkit.reset();
-  });
-
-  test('should capture errors', async () => {
-    const err = new Error('error to look for');
-    Sentry.captureException(err);
-    await expect.poll(() => testkit.reports()).toHaveLength(1);
-    const report = testkit.findReport(err);
-    expect(report).to.exist;
-  });
-});

package/src/sentry/types.ts

@@ -1,22 +0,0 @@
-//
-// Copyright 2022 DXOS.org
-//
-
-import type { Event, Transport } from '@sentry/types';
-
-export type InitOptions = {
-  enable?: boolean;
-  destination?: string;
-  installationId?: string;
-  release?: string;
-  environment?: string;
-  tracing?: boolean;
-  sampleRate?: number;
-  replay?: boolean;
-  replaySampleRate?: number;
-  replaySampleRateOnError?: number;
-  scrubFilenames?: boolean;
-  properties?: object;
-  transport?: () => Transport;
-  onError?: (event: Event) => void;
-};

package/src/testing/index.ts

@@ -1,5 +0,0 @@
-//
-// Copyright 2022 DXOS.org
-//
-
-export * from './testkit';

package/src/testing/testkit/browser.ts

@@ -1,8 +0,0 @@
-//
-// Copyright 2022 DXOS.org
-//
-
-// eslint-disable-next-line no-restricted-imports
-import testkit from 'sentry-testkit/dist/browser';
-
-export const sentryTestkit = testkit;

package/src/testing/testkit/index.ts

@@ -1,7 +0,0 @@
-//
-// Copyright 2022 DXOS.org
-//
-
-import testkit from 'sentry-testkit';
-
-export const sentryTestkit = testkit;

package/src/testing/testkit/shims.d.ts

@@ -1,5 +0,0 @@
-//
-// Copyright 2022 DXOS.org
-//
-
-declare module 'sentry-testkit/dist/browser';