use Dist::Zilla::Role::FileWatcher

This makes your code simpler, so you don't have to mess around with applying the role directly.

I think your pod_for_source_file code from https://github.com/DarwinAwardWinner/Dist-Zilla-Role-DistFileReader/blob/master/lib/Dist/Zilla/Role/PerlModuleReader.pm#L31 would be best added directly to Dist::Zilla::Role::PPI (where you can cache the PPI document, being even more efficient).


Probably not. It was there before so I left it in.

I think the point of this was to address a concern you'd raised initially - that if a munger changed the content, and then it was later changed back again to the first form, we wouldn't want to re-munge the file in that case as it is just a waste of resources.  But there's no other harm that I can see in re-munging.  Your call.