But from the user's perspective that module-already-exists check is simply part of a save operation, so arguably still viewed as simply a repository. I think moving it into ModuleDefinitionRepository seems right. It would also simplify things considering this impl currently has a reference to the ModuleDefinitionRepository.