Invalid JS identifier if global name of export has a dot in it

[joebandenburg]

I have a particular use case where a library I depend on puts a global inside a namespace. For example, they export a global org.company.lib. If I use a command like:

pure-cjs --input index.js --output dist/foo.js --external '{"lib": {"global": "org.company.lib"}}' --exports foo

This produces invalid output:

(function (factory) {
   ...
}(function (__external_org.company.lib) {
   ...

As you can see, the dots need to be transformed in some way (probably replaced with underscores) to ensure that the identifier is correct. Alternatively, the identifier could be made not to depend on the name of the actual global at all, e.g. __external_1.

I'm happy to work on a pull request.

[joebandenburg]

Note that there is a workaround. Specify an id for the global:

--external '{"lib": {"global": "org.company.lib", "id": "__external_1"}}'