export name changed with noExternal

[ianzone]

Reproduction link or steps

the exports of emitter.js don't match the exports of emitter.d.ts

What is expected?

exports name should be unchanged

What is actually happening?

they are changed

Any additional comments?

related #285

[IWANABETHATGUY]

It seems that rolldown has an issue when handling interop CJS with unbundle mode. A temporary workaround would be to change the platform to browser, and then the resolver would prefer ESM first.

[IWANABETHATGUY]

Oh, I realized that is not a bug because the import_index_cjs is referenced here https://github.com/ianzone/tsdown-bug/blob/fc68112bcfd556382c5b8f7243ca7302014475bf/dist/index.js?plain=1#L6, this extra variable is used to cjs interop, so you could either choose to platform: browser or change inputOptions.resolve.mainFields order to make sure resolve module mainFields first.

[ianzone]

Oh, I realized that is not a bug because the import_index_cjs is referenced here https://github.com/ianzone/tsdown-bug/blob/fc68112bcfd556382c5b8f7243ca7302014475bf/dist/index.js?plain=1#L6, this extra variable is used to cjs interop, so you could either choose to platform: browser or change inputOptions.resolve.mainFields order to make sure resolve module mainFields first.

I consider it's a bug cause it failed the test here https://github.com/NervJS/taro/actions/runs/15467838954/job/43544516076?pr=17746#step:15:106

[ianzone]

Hi, is it an upstream issue? Would you please drop the link?