If we're making is_active() a requirement for this function (which it ought to be anyway, right?) I think we want to be explicit about that; it makes reasoning easier and reduces the opportunity for bugs; that's why I suggest that switchup.
If you want to wrap have_pending in an inlined function call to be clear about that and think it's clearer than just a comment, I suppose that works for me.