fix: propagate message handler from MessagingManager to providers

- Add setMessageHandler to MessagingProvider interface
- MessagingManager.setMessageHandler now propagates to all registered providers
- MessagingManager.registerProvider sets handler on new providers if already configured

Author: Brennmeister1

src/messaging/manager.ts

@@ -20,10 +20,13 @@ export class MessagingManager {
   }
 
   /**
-   * Set the message handler callback
+   * Set the message handler callback and propagate to all registered providers
    */
   setMessageHandler(handler: MessageHandlerCallback): void {
     this.messageHandler = handler;
+    for (const provider of this.providers.values()) {
+      provider.setMessageHandler(handler);
+    }
   }
 
   /**
@@ -34,6 +37,10 @@ export class MessagingManager {
       this.logger.warn({ platform: provider.platform }, 'Provider already registered, replacing');
     }
     this.providers.set(provider.platform, provider);
+    // Propagate existing handler to newly registered provider
+    if (this.messageHandler) {
+      provider.setMessageHandler(this.messageHandler);
+    }
     this.logger.info({ platform: provider.platform }, 'Provider registered');
   }
 

src/messaging/types.ts

@@ -68,6 +68,11 @@ export interface MessagingProvider {
   readonly platform: Platform;
   readonly isEnabled: boolean;
 
+  /**
+   * Set the message handler callback for incoming messages
+   */
+  setMessageHandler(handler: MessageHandlerCallback): void;
+
   /**
    * Start the provider (connect to platform, start webhook server, etc.)
    */