Missing dependency in 6.0.0 version about html-minifier-terser HOT 6 CLOSED

Hi @97carmine,

it seems you use the new plug and play feature of yarn which reports this:

terser tried to access acorn, but it isn't declared in its dependencies; this makes the require call ambiguous and unsound.

Please report this error in the repo of the affected package which is terser itself so they can fix this.

https://github.com/terser/terser/issues

I can see acorn at https://github.com/terser/terser/blob/master/package.json#L52 so maybe it has to be moved to the normal dependencies.

Alternatively you can disable the pnp feature of yarn which will apply the legacy behavior or you can use npm.

from html-minifier-terser.

I couldn't seem to find a submitted issue on terser so I submitted one: terser/terser#1076

Should this issue remain open in the meantime? Even though the source of the issue is in terser, once it's fixed there, the dependency would have to be bumped in this package. A package breaking Yarn PnP builds is a fairly rare occurrence nowadays. I'll track both issues and hopefully, this can be resolved soon.

from html-minifier-terser.

For anyone who needs a quick fix in the meantime, v5.1.1 of html-minifier-terser is not affected by this issue.

from html-minifier-terser.

This issue here can stay closed since the consumers of html-minifier-terser will receive minor and patch releases of terser.

See also https://github.com/terser/html-minifier-terser/blob/master/package.json#L73

^5.8.0 will pick up any 5.x release.

from html-minifier-terser.

Terser doesn't depend on Acorn unless it is instructed to parse with Acorn - https://github.com/terser/terser#use-acorn-for-parsing - and so it shouldn't be a prod dependency.

Is Yarn trying to aggressively follow dependencies wherever it sees a require()? If there's a code comment or something that can be placed next to the require() that tells it not to do that, it would be neighborly of Terser to add that. But AFAICT Terser is doing nothing wrong.

from html-minifier-terser.

We should keep the discussion in one place.

Since this is a problem with terser, I have replied at terser/terser#1076 (comment)

Locking this issue since any further replies here should belong to the issue at terser/terser#1076

from html-minifier-terser.