Added ReferenceIdPlugin to default set of plugins, removed client.config.useReferenceIds()

Author: niemyjski

packages/core/src/configuration/Configuration.ts

@@ -481,13 +481,6 @@ export class Configuration {
     }
   }
 
-  /**
-   * Automatically set a reference id for error events.
-   */
-  public useReferenceIds(): void {
-    this.addPlugin(new ReferenceIdPlugin());
-  }
-
   /**
    * Use localStorage for persisting things like server configuration cache and persisted queue entries (depends on usePersistedQueueStorage).
    */

packages/core/src/plugins/EventPluginManager.ts

@@ -1,11 +1,12 @@
 import { Configuration } from "../configuration/Configuration.js";
 import { ConfigurationDefaultsPlugin } from "./default/ConfigurationDefaultsPlugin.js";
 import { DuplicateCheckerPlugin } from "./default/DuplicateCheckerPlugin.js";
-import { SimpleErrorPlugin } from "./default/SimpleErrorPlugin.js";
-import { EventExclusionPlugin } from "./default/EventExclusionPlugin.js";
-import { SubmissionMethodPlugin } from "./default/SubmissionMethodPlugin.js";
 import { EventPluginContext } from "./EventPluginContext.js";
+import { EventExclusionPlugin } from "./default/EventExclusionPlugin.js";
 import { PluginContext } from "./PluginContext.js";
+import { ReferenceIdPlugin } from "./default/ReferenceIdPlugin.js";
+import { SimpleErrorPlugin } from "./default/SimpleErrorPlugin.js";
+import { SubmissionMethodPlugin } from "./default/SubmissionMethodPlugin.js";
 
 export class EventPluginManager {
   public static async startup(context: PluginContext): Promise<void> {
@@ -58,6 +59,7 @@ export class EventPluginManager {
   public static addDefaultPlugins(config: Configuration): void {
     config.addPlugin(new ConfigurationDefaultsPlugin());
     config.addPlugin(new SimpleErrorPlugin());
+    config.addPlugin(new ReferenceIdPlugin());
     config.addPlugin(new DuplicateCheckerPlugin());
     config.addPlugin(new EventExclusionPlugin());
     config.addPlugin(new SubmissionMethodPlugin());

packages/core/test/ExceptionlessClient.test.ts

@@ -1,6 +1,7 @@
 
 import { ExceptionlessClient } from "../src/ExceptionlessClient.js";
 import { KnownEventDataKeys } from "../src/models/Event.js";
+import { ReferenceIdPlugin } from "../src/plugins/default/ReferenceIdPlugin.js";
 
 describe("ExceptionlessClient", () => {
   test("should use event reference ids", async () => {
@@ -17,7 +18,7 @@ describe("ExceptionlessClient", () => {
     expect(lastReferenceIdManager.getLast()).toBeNull();
 
     const numberOfPlugins = client.config.plugins.length;
-    client.config.useReferenceIds();
+    client.config.addPlugin(new ReferenceIdPlugin());
     expect(client.config.plugins.length).toBe(numberOfPlugins + 1);
 
     context = await client.submitException(error);