Sync method

[brettz9]

Hi,

I'm working with eslint-plugin-jsdoc and we have a need for parsing import statements (for use within @import tags). However, ESLint does not allow asynchronous rules.

I know that es-module-lexer mentions using async to be safe, but when I tested it within eslint, it seems enough time has passed for Wasm to load such that I could successfully get results using its parse method synchronously.

I am wondering if you could expose a sync version of parseImports (that presupposes it is called with enough time for Wasm to load)?

[TomerAberbach]

What you observed does make sense given that the WASM apparently takes only 5ms to boot!

I think a parseImportsSync method could make sense. Maybe I would also expose an wasmBootPromise that people can await if they want (even if you personally wouldn't be depending on that). Will probably be able to get to this soon. Thanks for filing :)

[TomerAberbach]

One thing I will mention is that jsdoc isn't valid JavaScript if you consider only the content within the comment and exclude the comment delimiters. I do wonder whether es-module-lexer always parses jsdoc imports correctly (given that there's an @ before import). I'll assume you've already looked into that

[brettz9]

That's right, I'd be supplying my own well-formed import statements based on the comments content, but not including the @. (TS suggests these comments are otherwise just well-formed imports.)

[TomerAberbach]

Got it. Makes sense!

[TomerAberbach]

@brettz9 are you able to confirm whether #11 does what you need?

Will need to be breaking change since I'm swapping the async method from a default import to a named import

[brettz9]

LGTM, thanks!

[TomerAberbach]

Published in v2.0.0!

[brettz9]

Awesome, appears to be working great too, thanks!

[TomerAberbach]

No problem! Glad it's working 😁