Imports are incorrectly reordered

[bdwain]

• Rollup Version: 0.66.2 (but this seems to have been introduced in 0.55)
• Operating System (or Browser): OS X High Sierra
• Node Version: 9.11.1

How Do We Reproduce?

repl reproduction

Expected Behavior

The original import order should be maintained

Actual Behavior

The imports that pull in content from other files get pushed to the top, above imports that are not resolved (because they are external packages)

[jamiewinder]

Fixed link

[bdwain]

oops thanks @jamiewinder . fixed in the original

[bdwain]

I was able to work around this temporarily by moving the imports that need to be first to a separate file

repl

[lukastaegert]

Ok, this issue was a really enlightening one. Previously, the only guarantees our bundling had were:

• Modules inside a bundle/chunk have the correct execution order
• An internal or external import is executed before the importing module

There was, however, no guarantee that external imports were executed in a particular order between themselves. I created a PR that solves your problem and many similar problems at the same time while improving execution order, please check it out: #2508

[bdwain]

Awesome! Thank you!