Uhm, consider that in some modules installation from upstream source might be possible as option (see the elastic search stdmod module) so install there makes more sense (and actually it's for cases like that that I think it makes sense to split classes in package|install / service / config subclasses ).
But I'm ready to adapt to any common sense or existing practices here.