we want to s/shown/show, so:
user_ids_to_show.include?(user.id)

Also, so long as we are tweaking, curious about the TODO line above. If it is significantly more efficient to call uniq! rather than check if message has already been read by the user, let's do that, and otherwise, just we can remove the TODO line.