[Regression] default class exports

[manniL]

Having the following test.js file:

export default class Test {}

Test.something = 'ok'

will throw an error with esm@3.2.1

node -r esm test.js
file:///home/..../test.js:1
ReferenceError: Test is not defined
at Object. (file:///home/.../test.js:3:1)
at Generator.next ()

but not with esm@3.2.0 nor esm@3.1.4

Originally posted by @manniL in #722 (comment)

[dnalborczyk]

thank you @manniL for bringing this up!

@jdalton seems to be an introduced bug by acorn? https://github.com/acornjs/acorn/blob/master/acorn/CHANGELOG.md#bug-fixes

fwiw, hoisting still works for exported default function declarations:

export default function Test() {}
Test.something = "ok";

somewhat surprised that there isn't a test262 test for this which could have caught this. gonna check over there as well.

[jdalton]

Thanks for the report @manniL and thanks for digging in @dnalborczyk!

After review it is not an Acorn issue but an esm issue introduced by this commit 9f8af43.

Update:

Patch 55d92f8; Tests a24a4a9;

[manniL]

Thanks for the quick fix @jdalton

Credits to @pimlie who noticed the bug while working on a new Nuxt.js feature. 👍

[jdalton]

v3.2.2 is released 🎉

[adrianheine]

We actually had the same issue, see acornjs/acorn#810.