As explained in this comment, I merged #2394 when I shouldn't have.
Solution
The isTargetChatroomCurrentlyActive check should either be done inside of messageReceivePostEffect or passed in as a boolean to messageReceivePostEffect, so that 'gi.actions/identity/kv/addChatRoomUnreadMessage' always gets called.
Problem
As explained in this comment, I merged #2394 when I shouldn't have.
Solution
The
isTargetChatroomCurrentlyActivecheck should either be done inside ofmessageReceivePostEffector passed in as a boolean tomessageReceivePostEffect, so that'gi.actions/identity/kv/addChatRoomUnreadMessage'always gets called.