provide path to js in main entry, fixes #494

[gugu]

No description provided.

[cjbarth]

I'm OK with this change, and I actually expected this to be this way from the beginning. I guess I'm curious why everything has been working for so long with an abstract main (I checked the history) and now it is broken.

[markstos]

@gugu did you test the fix with Babel? I don't think is this the issue. If you look at the JavaScript we generate in ../lib/passport-saml/index.js you can see it contains __esModule: true. If you look the code in #494, you can see references to cjs, or Common JS modules. So it seems like we have perhaps there's a mismatch between an expectation about whether the module is a Common JS module or an ES 6 module. There may be specific issues related to this topic and Babel:

https://2ality.com/2017/01/babel-esm-spec-mode.html

It seems Babel may be looking for __esModule in particular (it's mentioned at the URL above).

I'm not an expert in TypeScript, Babel and how these interact with Common JS and ES 6 modules, but that seems to the be area where the issue lies.

[markstos]

The actual fix as posted here: #494 (comment)