Fix/neotraverse

[fratzinger]

see this: https://github.com/puruvj/neotraverse

if you missed it, this is the drama behind the update of [email protected] -> [email protected]: https://x.com/passle_/status/1810805530706792930

see current graph of feathers-hooks-common:

source: https://npmgraph.js.org/?q=feathers-hooks-common#zoom=h

[fratzinger]

released as v8.2.0

[marshallswain]

Much better. 😁

[PuruVJ]

Thanks a lot for considering neotraver 😄! That graph looks svelte and lean!

[fratzinger]

That was fast. I didn't think you'll see this. Thanks for your work! 💚

[eXigentCoder]

Hey there, thanks for the great work!
I think there might be an issue with this though (but perhaps I just need to change something my side), it looks like it's importing the ESM version of traverse and the minified output contains the export keyword:

... minifiedCode;export{m as Traverse,v as default};

Should the import statements in feathers-hooks-common not be changed to this instead?:

import traverse from 'neotraverse/legacy';

If not, I'm getting an error in my project when I run my Jest tests
SyntaxError: Unexpected token 'export'
Coming from node_modules/neotraverse/dist/min/index.js which is in my project due to [email protected]

[PuruVJ]

Ah that’s correct, it should be legacy if you’re still shipping commonJS builds

…

On Sun, 21 Jul 2024 at 9:59 AM, Ryan Kotzen ***@***.***> wrote: Hey there, thanks for the great work! I think there might be an issue with this though (but perhaps I just need to change something my side), it looks like it's importing the ESM version of traverse and the minified output contains the export keyword: ... minifiedCode;export{m as Traverse,v as default}; Should the import statements in feathers-hooks-common not be changed to this instead?: import traverse from 'neotraverse/legacy'; If not, I'm getting an error in my project when I run my Jest tests SyntaxError: Unexpected token 'export' Coming from node_modules/neotraverse/dist/min/index.js which is in my project due to ***@***.*** — Reply to this email directly, view it on GitHub <#753 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ALMH4FYY557ZIRILJBPREOLZNM2MLAVCNFSM6AAAAABLBDXTDGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENBRGQ3DAMZXHA> . You are receiving this because you commented.Message ID: ***@***.*** com>

[fratzinger]

released as v8.2.1 🎉 @eXigentCoder please try it out and let me know.