fix(rollup): allow default require for non standard export names in .json files

[alexzhang1030]

closes #217

Why?

The @rollup/plugin-json plugin transforms a json module into an ESM.

For example:

{ "foo": "bar" }

becomes:

export default { foo: "bar" }

However, CJS require statement can natively import a json file using the following syntax:

const json = require("./foo.json")
json.foo // directly access fields.

But when a json module is transformed to an ESM and exported using export default, we need to change the require statement to:

require("./foo.json")['default']

Resolution

For every require('*.json') statement, add the ['default'] suffix.

[pi0]

Hi! Thanks for your pull request and updated fixture. I think this is a valid fix as Node.js native behavior with .cjs files is similar to this. However running a transform on ALL entries like current changes to detect require() syntax seems a risky change and also potentially slowing down builds that's why still kept PR in pending state. If you have any alternative ideas please mention 👍🏼

[pi0]

I'm an not sure if this behavior would be aligned with ESM standards. But still open to discussion if you think a core change worth to do it.