fix: tear down also event handlers in addon service

src/lib/features/events/event-service.ts

@@ -110,6 +110,13 @@ export default class EventService {
         return this.eventStore.on(eventName, listener);
     }
 
+    off(
+        eventName: string | symbol,
+        listener: (...args: any[]) => void,
+    ): EventEmitter {
+        return this.eventStore.off(eventName, listener);
+    }
+
     private async enhanceEventsWithTags(
         events: IBaseEvent[],
     ): Promise<IBaseEvent[]> {

src/lib/services/addon-service.ts

@@ -59,6 +59,8 @@ export default class AddonService {
     fetchAddonConfigs: (() => Promise<IAddon[]>) &
         memoizee.Memoized<() => Promise<IAddon[]>>;
 
+    private eventHandlers: Map<string, (event: IEvent) => void>;
+
     constructor(
         {
             addonStore,
@@ -83,6 +85,7 @@ export default class AddonService {
         this.logger = getLogger('services/addon-service.js');
         this.tagTypeService = tagTypeService;
         this.eventService = eventService;
+        this.eventHandlers = new Map();
 
         this.addonProviders =
             addons ||
@@ -124,15 +127,18 @@ export default class AddonService {
     }
 
     registerEventHandler(): void {
-        SUPPORTED_EVENTS.forEach((eventName) =>
+        SUPPORTED_EVENTS.forEach((eventName) => {
-            this.eventService.onEvent(eventName, this.handleEvent(eventName)),
+            const handler = this.handleEvent(eventName);
-        );
+            this.eventHandlers.set(eventName, handler);
+            this.eventService.onEvent(eventName, handler);
+        });
     }
 
     handleEvent(eventName: string): (event: IEvent) => void {
         const { addonProviders } = this;
         return (event) => {
-            this.fetchAddonConfigs().then((addonInstances) => {
+            this.fetchAddonConfigs()
+                .then((addonInstances) => {
                     addonInstances
                         .filter((addon) => addon.events.includes(eventName))
                         .filter(
@@ -159,6 +165,18 @@ export default class AddonService {
                                 addon.id,
                             ),
                         );
+                })
+                .catch((error) => {
+                    if (error.message === 'aborted') {
+                        this.logger.debug(
+                            `Addon event handling aborted during shutdown for event ${eventName}`,
+                        );
+                    } else {
+                        this.logger.warn(
+                            `Failed to handle addon event ${eventName}:`,
+                            error,
+                        );
+                    }
                 });
         };
     }
@@ -349,6 +367,20 @@ export default class AddonService {
     }
 
     destroy(): void {
+        // Clean up event handlers first to prevent race conditions during teardown
+        this.eventHandlers.forEach((handler, eventName) => {
+            try {
+                this.eventService.off(eventName, handler);
+            } catch (error) {
+                this.logger.debug(
+                    `Failed to remove event handler for ${eventName}:`,
+                    error,
+                );
+            }
+        });
+        this.eventHandlers.clear();
+
+        // Then destroy addon providers
         Object.values(this.addonProviders).forEach((addon) =>
             addon.destroy?.(),
         );