do_aggressive_sentient logical issue

[mattgodbolt]

    if (IS_SET(ch->act, ACT_AGGRESSIVE) && !IS_SET(ch->in_room->room_flags, ROOM_SAFE) && !IS_AFFECTED(ch, AFF_CALM)
        && (ch->fighting == nullptr) // Changed by Moog
        && !IS_AFFECTED(ch, AFF_CHARM) && IS_AWAKE(ch) && !(IS_SET(ch->act, ACT_WIMPY) && IS_AWAKE(wch))
        && can_see(ch, wch) && !number_bits(1) == 0) {

the !number_bits(1) == 0 doesn't do what we'd expect. clang-tidy says: "Logical not is only applied to the left hand side of this comparison" and suggests changing it to !(number_bits(1) == 0) which is to say number_bits(1) == 1. But it's not clear what we were trying to do here at all

[mattgodbolt]

Came across this again...anyone have a guess at the original intention?