fix: PR feedback

Author: e-maad

libraries/core/src/conversation/conversationService/conversationService.ts

@@ -999,7 +999,7 @@ export class ConversationService extends TypedEventEmitter<Events> {
     } catch (error: unknown) {
       // For unmapped or unrecoverable errors, avoid surfacing exceptions from event handling
       // and instead log and return null so the event processing queue can continue safely.
-      this.logger.error('Failed to handle MLS message-add event after recovery; returning null', {error});
+      this.logger.error('Failed to handle MLS message-add event after recovery; returning null', {error, event});
       return null;
     }
   }
@@ -1040,7 +1040,7 @@ export class ConversationService extends TypedEventEmitter<Events> {
    * Always resolves to null; the effects are applied to local state.
    */
   private async handleMLSWelcomeMessageEvent(event: ConversationMLSWelcomeEvent): Promise<HandledEventPayload | null> {
-    this.logger.info('Handling MLS welcome message event (orchestrated)');
+    this.logger.info('Handling MLS welcome message event (orchestrated)', {event});
     await this.MLSRecoveryOrchestrator.execute({
       context: {
         operationName: OperationName.handleWelcome,

libraries/core/src/notification/notificationService.ts

@@ -212,7 +212,7 @@ export class NotificationService extends TypedEventEmitter<Events> {
     source: NotificationSource,
   ): AsyncGenerator<HandledEventPayload> {
     for (const event of notification.payload) {
-      this.logger.debug(`Handling event of type "${event.type}"`);
+      this.logger.debug(`Handling event of type "${event.type}"`, event);
       let lastEventDate: Date | undefined = undefined;
       try {
         lastEventDate = await this.database.getLastEventDate();