Rollup 1.11 changed AMD output

[surma]

How Do We Reproduce?

• 1.10 REPL
• 1.11 REPL

Expected Behavior

require() calls should use .js extension.

Actual Behavior

They don’t.

This might be related to #2831

[lukastaegert]

Is this actually causing issues for you? This was changed intentionally as requirejs will only use the proper relative resolution algorithm respecting base urls etc. if the extension is skipped. I.e. to my knowledge to make AMD work correctly, extensions need to be skipped.

[surma]

Hm, I was relying on the file extension being present for the AMD loader I wrote for loadz0r. I just thought this would be considered a breaking change.

Either way, happy to adjust loadz0r just wanted to make sure it was an intentional change.

[lukastaegert]

Yes, sorry about that! I was thinking long if it would be a breaking change but on the other, the old behaviour was just not working when not all chunks were in the same directory so I thought it was too important to keep it for the next major.

[surma]

Alright, that seems reasonable. I’ll fix loadz0r :)