loader respect the main field in package.json

[bobzhang]

relevant to #2950
@ccapndave would you mind contribute a minimal example and its expected behavior? thanks

[bobzhang]

relevant discussions microsoft/TypeScript#21423.
A typical main field:

"main" : "dist/src.js"

or

"main" : "dist/src"

[ccapndave]

Sure! https://github.com/ccapndave/bucklescript-issue-2986 is a minimal example to reproduce (details in the README).

[bobzhang]

fixed in #2997 , not there is a bug in bs-css package.json giraud/bs-css#72

[ccapndave]

I'm not certain that this has fixed it - I replaced loader.js with the version in this pull request, but now the page tries to load http://localhost:8080/node_modules/bs-css/src/Css.js/src/glamor.re.js

[bobzhang]

@ccapndave it works on your provided repo, can you check if it works on your provided repo but not on your production code?

[bobzhang]

@ccapndave I think it is already fixed, can you confirm? btw, the bs-css main field is wrong. If not fixed, please provide a minimal example, thanks

[cristianoc]

This breaks https://github.com/cristianoc/genFlow/tree/master/sample-project.
I was going to sync up loader.js to this version, but won't.

[ccapndave]

I can confirm that the latest version of loader.js works in both my test and my real project (without any changes to the bs-css main field). Thanks very much!