It isn't a method 'findByName' on the CompositeModuleDefinitionService, but just a means to get access to the ModuleDefinitionRepository.  I'm ok having the controller depend directly on ModuleDefinitionRepository and then removing the getter from CompositeModuleDefintionService.