Dynamic import() error

[ltheinrich]

main.js

import("./test.js").then((module) => { console.log(module.message)});

test.js

export const message = "This should print!";

It should work, but instead I get an error (since v2.9.16):

ERROR: cannot minify main.js: expected String, Identifier, *, or { instead of ( in import statement on line 1 and column 7
    1: import("./test.js").then((module) => { console.log(module...
             ^

[tdewolff]

Thanks for the report. However, looking at the specification at https://tc39.es/ecma262/#sec-imports I don't see any definition of this syntax. Is this from a syntax proposal?

[ltheinrich]

I did not notice, it is actually from a proposal. But the feature is supported by most modern browsers.
This should be the proposal: https://github.com/tc39/proposal-dynamic-import
My project uses dynamic imports, so I would appreciate if you could implement it.

[ltheinrich]

Now I have found it in the specification (Import Calls): https://tc39.es/ecma262/#sec-import-calls
So it looks like it was already approved.

[tdewolff]

Yes, this looks actually like a bug. I'm on it!

[tdewolff]

Let me know if it is working for you now!

[ltheinrich]

Hi, I compiled the master branch using Go 1.15.8 and it shows a different error (used the main.js code from above):

[lennart@nexus minify]$ go version
go version go1.15.8 linux/amd64
[lennart@nexus minify]$ go build -o minify-8d45d96
[lennart@nexus minify]$ minify-8d45d96 main.js
/usr/local/bin/minify-8d45d96: line 1: syntax error near unexpected token `newline'
/usr/local/bin/minify-8d45d96: line 1: `!<arch>'
[lennart@nexus minify]$

[tdewolff]

I'm not sure what that is, it seems to work fine here and Travis builds fine. Perhaps a problem at your end?

[ltheinrich]

Yes, it was a problem on my end.
I built it using make install now and it works. Thanks! :)