literal exports not supported about cjs-module-lexer

[guybedford]

Yeah that’s the sort of case we have to draw a line on in not supporting unfortunately. Maybe if it were an explicit iteration and copy it could be considered, but the implicit ‘use’ function is definitely outside this.

…

On Fri, Oct 2, 2020 at 21:57 Fred K. Schott ***@***.***> wrote: @guybedford <https://github.com/guybedford> are you looking for popular packages not working? Just ran into chai not working, which I know is a popular project. But they use some very strange export language that even a true parser would have trouble with: ***@***.***/lib/chai.js — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#6 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAESFSQE7XKMLY4LMHKF4BTSI2VKZANCNFSM4R6VFTEA> .

[FredKSchott]

@guybedford are you looking for popular packages not working? Just ran into chai not working, which I know is a popular project. But they use some very strange export language that even a true parser would have trouble with: https://unpkg.com/[email protected]/lib/chai.js

[guybedford]

Hi @FredKSchott, thanks for the feedback here.

Do you have an exact example of the sort of code pattern you'd like to see supported? Arbitrary expressions can't be supported as this is not a parser. But specific expression types might be possible.

More info on this in https://github.com/guybedford/cjs-module-lexer#exports-object-assignment.

[FredKSchott]

Ah, maybe this is a regression then. In the docs you have:

// DETECTS EXPORTS: a, b, c
module.exports = {
  a,
  b: 'c',
  c: c
};

But when I run this with the lexer, I get:

{exports: ["a"], reexports: []}

That seems to contradict the comment in the code snippet.

[guybedford]

Thanks, that was actually a readme typo - fixed in 4d56d33.

Would exactly supporting strings on the RHS specifically help in your use case? I didn't think this was such an important feature but they are a good example of a case that is easy to add as an extension. Again an example would help to understand what would be useful.

[FredKSchott]

no real-world use-case on my end, this came out of a simple test file that I'd written to test that it was working correctly. If I see one, I'll definitely let you know.

We'll be moving forward with this in Snowpack as our default-on CJS handling, so hopefully our community surfaces some good feedback as well! Moving this from config opt-in to default-on is very exciting for us, especially for React developers who seem to run into this the most.

[guybedford]

Thanks @fks, the faster you can run a feedback cycle here the better as we only have a short window before it locks down on Nodejs 12. If there’s something to fix now we can still get that in but over the next week or so that changes fast. Appreciated for the investigations!

…

On Wed, Sep 30, 2020 at 17:04 Fred K. Schott ***@***.***> wrote: no real-world use-case on my end, this came out of a simple test file that I'd written to test that it was working correctly. If I see one, I'll definitely let you know. We'll be moving forward with this in Snowpack as our default-on CJS handling, so hopefully our community surfaces some good feedback as well! Moving this from config opt-in to default-on is very exciting for us, especially for React developers who seem to run into this the most. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#6 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAESFSSOQOJ4GCORTXH6CHLSIPBSLANCNFSM4R6VFTEA> .