don't use esm or support esm for typescript modules that still use require

[balupton]

also update valid-module to detect this, if it isn't already, or use valid-module for this detection

this can also be detected by actually testing the generated esm editions

package in question is https://github.com/bevry/pluginloader

the other alternative approach would be for pluginloader to use dynamic imports, however I am unsure how that converts to CJS - for plugin loader, this won't work unless there are breaking changes to the API, as dynamic imports require await, however they are occurring inside the constructor, however that is not a big deal and we can adapt

so a bit of exploration here will go far