I find it very odd that we use a *Composite*ModuleDefinitionService to retrieve a module definition (xml file) when composite modules are the exact case where this does not make sense.

(commenting via github web, haven't tested). But this is either broken, or at the very least counter intuitive