In `rescindAndUpdateBadgeCounts` we were previously skipping `updateBadgeCount` for `deviceToken`s "handled" by `rescindPushNotifs`. I haven't dug too deeply into the git history yet, but my hunch is that this was because `rescindPushNotifs` previously handled updating the badge count by including the `badge` entry in the APS dictionary.
However, the `badge` entry in the APS dictionary of the rescind (`background`) notif **isn't** updating the badge count (at least anymore). We need a separate `alert`-type notification in order to update the badge count.
This `alert` notification is already being sent for the "mark as unread" and "mark as read" actions, but isn't being sent when a thread is marked as read after being focused (because that results in a rescind as well that excludes the `deviceToken` from being handled by `updateBadgeCount`).
Because rescinding and updating badge count must always happen separately, there's no reason for a rescind to affect whether a badge count update notif is sent. This the exclusion behavior from `rescindAndUpdateBadgeCounts` and updates `updateBadgeCount` to remove the no longer relevant `excludeDeviceTokens` argument.