It would be possible to take this further and absorb the contents of AbstractModuleNoTraits into the Module.php itself, but I quite like the separation of the more generic Module.php stuff into the Abstract class.  It also does maintain a bit of parity between TccAbstractModule and Payum's AbstractModule so that we can share any improvements.

Again, though, it's your call and I just thought I'd mention that I was aware that I could take it further.