I'm somewhat on the fence on this one.  

What do you see as the benefits of having this in a separate file?  Usually the .module files only have the module definition and the routes in them.  Very rarely will they have anything else.  I just wondering if we're over fragmenting here.