feat: move error reporting functionality to the core module

packages/analytics-js-common/__mocks__/ErrorHandler.ts

@@ -1,14 +1,11 @@
-import type { IErrorHandler, PreLoadErrorData } from '../src/types/ErrorHandler';
-import { BufferQueue } from './BufferQueue';
+import type { IErrorHandler } from '../src/types/ErrorHandler';
+import { defaultHttpClient } from './HttpClient';
 
 // Mock all the methods of the ErrorHandler class
 class ErrorHandler implements IErrorHandler {
   onError = jest.fn();
   leaveBreadcrumb = jest.fn();
-  notifyError = jest.fn();
-  init = jest.fn();
-  attachErrorListeners = jest.fn();
-  errorBuffer = new BufferQueue<PreLoadErrorData>();
+  httpClient = defaultHttpClient;
 }
 
 const defaultErrorHandler = new ErrorHandler();

packages/analytics-js-plugins/__mocks__/HttpClient.ts → packages/analytics-js-common/__mocks__/HttpClient.ts

@@ -10,6 +10,7 @@ class HttpClient implements IHttpClient {
   getAsyncData = jest.fn();
   setAuthHeader = jest.fn();
   resetAuthHeader = jest.fn();
+  init = jest.fn();
 }
 
 const defaultHttpClient = new HttpClient();

packages/analytics-js-common/src/services/ExternalSrcLoader/types.ts

@@ -1,4 +1,4 @@
-import type { ErrorState } from '../../types/ErrorHandler';
+import type { ErrorState, IErrorHandler } from '../../types/ErrorHandler';
 import type { ILogger } from '../../types/Logger';
 
 export interface IExternalSourceLoadConfig {
@@ -11,16 +11,7 @@ export interface IExternalSourceLoadConfig {
 }
 
 export interface IExternalSrcLoader {
-  errorHandler?: {
-    onError(
-      error: unknown,
-      context?: string,
-      customMessage?: string,
-      shouldAlwaysThrow?: boolean,
-    ): void;
-    leaveBreadcrumb(breadcrumb: string): void;
-    notifyError(error: Error, errorState: ErrorState): void;
-  };
+  errorHandler?: IErrorHandler;
   logger?: ILogger;
   timeout: number;
   loadJSFile(config: IExternalSourceLoadConfig): void;

packages/analytics-js-common/src/types/ErrorHandler.ts

@@ -1,26 +1,13 @@
-import type { IPluginEngine } from './PluginEngine';
 import type { ILogger } from './Logger';
-import type { BufferQueue } from '../services/BufferQueue/BufferQueue';
 import type { IHttpClient } from './HttpClient';
-import type { IExternalSrcLoader } from '../services/ExternalSrcLoader/types';
 
 export type SDKError = unknown | Error | ErrorEvent | Event | PromiseRejectionEvent;
 
 export interface IErrorHandler {
+  httpClient: IHttpClient;
   logger?: ILogger;
-  pluginEngine?: IPluginEngine;
-  errorBuffer: BufferQueue<PreLoadErrorData>;
-  init(httpClient: IHttpClient, externalSrcLoader: IExternalSrcLoader): void;
-  onError(
-    error: SDKError,
-    context?: string,
-    customMessage?: string,
-    shouldAlwaysThrow?: boolean,
-    errorType?: string,
-  ): void;
+  onError(error: SDKError, context?: string, customMessage?: string, errorType?: string): void;
   leaveBreadcrumb(breadcrumb: string): void;
-  notifyError(error: Error, errorState: ErrorState): void;
-  attachErrorListeners(): void;
 }
 
 export type ErrorState = {

packages/analytics-js-common/src/types/HttpClient.ts

@@ -65,4 +65,5 @@ export interface IHttpClient {
   getAsyncData<T = any>(config: IAsyncRequestConfig<T>): void;
   setAuthHeader(value: string, noBto?: boolean): void;
   resetAuthHeader(): void;
+  init(errorHandler: IErrorHandler): void;
 }

packages/analytics-js-common/src/types/Metrics.ts

@@ -13,23 +13,24 @@ export type MetricServicePayload = {
 };
 
 export type ErrorEventPayload = {
+  payloadVersion: string;
   notifier: {
     name: string;
     version: string;
     url: string;
   };
-  events: ErrorEventType[];
+  events: ErrorEvent[];
 };
 
-export type ErrorEventType = {
-  payloadVersion: string;
+export type ErrorEvent = {
   exceptions: Exception[];
   severity: string;
   unhandled: boolean;
   severityReason: { type: string };
   app: {
     version: string;
     releaseStage: string;
+    type: string;
   };
   device: {
     locale?: string;
@@ -41,25 +42,19 @@ export type ErrorEventType = {
     clientIp: string;
   };
   breadcrumbs: Breadcrumb[] | [];
-  context: string;
   metaData: {
     [index: string]: any;
   };
   user: {
     id: string;
+    name: string;
   };
 };
 
 export type GeneratedEventType = {
   errors: Exception[];
 };
 
-export interface Exception {
-  message: string;
-  errorClass: string;
-  type: string;
-  stacktrace: Stackframe[];
-}
 export interface Stackframe {
   file: string;
   method?: string;
@@ -68,3 +63,10 @@ export interface Stackframe {
   code?: Record<string, string>;
   inProject?: boolean;
 }
+
+export interface Exception {
+  message: string;
+  errorClass: string;
+  type: string;
+  stacktrace: Stackframe[];
+}

packages/analytics-js-common/src/types/PluginsManager.ts

@@ -13,11 +13,9 @@ export interface IPluginsManager {
 
 export type PluginName =
   | 'BeaconQueue'
-  | 'Bugsnag'
   | 'CustomConsentManager'
   | 'DeviceModeDestinations'
   | 'DeviceModeTransformation'
-  | 'ErrorReporting'
   | 'ExternalAnonymousId'
   | 'GoogleLinker'
   | 'IubendaConsentManager'

packages/analytics-js-common/src/types/Source.ts

@@ -15,5 +15,6 @@ export type SourceConfig = {
 export type Source = {
   id: string;
   config?: SourceConfig;
+  name: string;
   workspaceId: string;
 };

packages/analytics-js-plugins/__mocks__/state.ts

@@ -138,7 +138,6 @@ const defaultStateValues: ApplicationState = {
   reporting: {
     isErrorReportingEnabled: signal(false),
     isMetricsReportingEnabled: signal(false),
-    isErrorReportingPluginLoaded: signal(false),
     breadcrumbs: signal([]),
   },
   session: {

packages/analytics-js-plugins/__tests__/deviceModeTransformation/index.test.ts

@@ -5,6 +5,7 @@ import { defaultErrorHandler } from '@rudderstack/analytics-js-common/__mocks__/
 import { defaultLogger } from '@rudderstack/analytics-js-common/__mocks__/Logger';
 import { defaultStoreManager } from '@rudderstack/analytics-js-common/__mocks__/StoreManager';
 import type { ExtensionPoint } from '@rudderstack/analytics-js-common/types/PluginEngine';
+import { defaultHttpClient } from '@rudderstack/analytics-js-common/__mocks__/HttpClient';
 import * as utils from '../../src/deviceModeTransformation/utilities';
 import { DeviceModeTransformation } from '../../src/deviceModeTransformation';
 import {
@@ -15,7 +16,6 @@ import {
 } from '../../__fixtures__/fixtures';
 import { server } from '../../__fixtures__/msw.server';
 import { resetState, state } from '../../__mocks__/state';
-import { defaultHttpClient } from '../../__mocks__/HttpClient';
 import { defaultPluginsManager } from '../../__mocks__/PluginsManager';
 import type { RetryQueue } from '../../src/utilities/retryQueue/RetryQueue';
 import type { QueueItem, QueueItemData } from '../../src/types/plugins';

packages/analytics-js-plugins/__tests__/xhrQueue/index.test.ts

@@ -6,12 +6,12 @@ import type { RudderEvent } from '@rudderstack/analytics-js-common/types/Event';
 import { defaultLogger } from '@rudderstack/analytics-js-common/__mocks__/Logger';
 import { defaultErrorHandler } from '@rudderstack/analytics-js-common/__mocks__/ErrorHandler';
 import type { ExtensionPoint } from '@rudderstack/analytics-js-common/types/PluginEngine';
+import { defaultHttpClient } from '@rudderstack/analytics-js-common/__mocks__/HttpClient';
 import type { RetryQueue } from '../../src/utilities/retryQueue/RetryQueue';
 import type { QueueItem, QueueItemData } from '../../src/types/plugins';
 import { resetState, state } from '../../__mocks__/state';
 import { XhrQueue } from '../../src/xhrQueue';
 import { Schedule } from '../../src/utilities/retryQueue/Schedule';
-import { defaultHttpClient } from '../../__mocks__/HttpClient';
 
 jest.mock('@rudderstack/analytics-js-common/utilities/timestamp', () => ({
   ...jest.requireActual('@rudderstack/analytics-js-common/utilities/timestamp'),

packages/analytics-js-plugins/rollup.config.mjs

@@ -37,11 +37,9 @@ const isNpmPackageBuild = moduleType === 'npm';
 const isCDNPackageBuild = moduleType === 'cdn';
 const pluginsMap = {
   './BeaconQueue': './src/beaconQueue/index.ts',
-  './Bugsnag': './src/bugsnag/index.ts',
   './CustomConsentManager': './src/customConsentManager/index.ts',
   './DeviceModeDestinations': './src/deviceModeDestinations/index.ts',
   './DeviceModeTransformation': './src/deviceModeTransformation/index.ts',
-  './ErrorReporting': './src/errorReporting/index.ts',
   './ExternalAnonymousId': './src/externalAnonymousId/index.ts',
   './GoogleLinker': './src/googleLinker/index.ts',
   './IubendaConsentManager': './src/iubendaConsentManager/index.ts',