`ObjectId` already handles the case where `value` is an `ObjectId`. Can we just use that, instead of using our own `isinstance`?