Multiple exports issue

[PaulMaly]

$$props is not defined

REPL

[Conduitry]

It looks like the export { name }; isn't marking the earlier let name = 'world'; as non-hoistable, which it should be. We're then left with let { name = 'world' } = $$props; outside of the instance function which accepts $$props as an argument.

[Conduitry]

The current hoisting logic - last significantly changed in 8cae8f2 - contains a check for bailing on hoisting when if (v.export_name && v.export_name !== v.name) return false. I'm thinking we want to bail whenever we have export_name, not just when it's a renamed export. This doesn't break any other unit tests. Does this make sense to you @jches?

[jches]

Yeah that sounds right to me - seems obvious in retrospect but it looks like this case didn't make it into the tests. I added a test and the fix here #2351