Nuxt.js: [edge] html minify issue

[yoyo837]

Version

edge

Reproduction link

https://github.com/yoyo837/nuxt-3392

Steps to reproduce

See https://github.com/nuxt/nuxt.js/pull/3392#issuecomment-393863232

What is expected ?

HTML minify work fine.

What is actually happening?

It report error.

_{This bug report is available on Nuxt.js community (#c7197)}

[yoyo837]

@pi0 I have new discoveries:

Using yarn install dependencies does not work, but npm can.

Yarn: 1.7.0

Node: 8.11.1

[aldarund]

Strange things. For me it works fine on my real project with latest nuxt-edge. But the example above just give empty error message. All with yarn
https://i.imgur.com/TDeLt96.png

[yoyo837]

@aldarund Please take a look at what is wrong with your page, not just console.

[aldarund]

@yoyo837 ye i see it now. Although still all works fine with my real project with same version of nuxt :))

[yoyo837]

@aldarund Are you sure all with yarn？And what’s the version of yarn？

[aldarund]

@yoyo837 yes, all with yarn. Im sure because i did a lot of full delete & reinstall yesterday
1.7.0

[cristijora]

Encountered the same issue today

[rdunk]

I encountered this and it seems to have been related to an older version of the html-minifier package. Forcing an update to the latest version seems to resolve it. In my case 3.5.10 was specified in my package-lock.json and now I'm using 3.5.16.

[manniL]

Cannot reproduce this with [email protected]. If you can, please ping me @tim-yao

[yoyo837]

No,This problem still exists until now. I'm very sure.

[manniL]

@yoyo837 reopened.

[manniL]

@yoyo837 Can't reproduce it with your repo linked in the issues and after updating :thinking:

Could you give a details reproduction instruction?

[yoyo837]

@manniL That is a repo small enough to reproduce the problem. This is caused by the #3392 merged, but that PR is trying to fix another issue. Now I'm not sure whether it should be revert.

[yoyo837]

I have been having this problem since that PR was merged, every time.

[manniL]

@yoyo837 I cloned the repo, updated the dependencies and was able to build nuxt without problems.

[yoyo837]

See comment: https://github.com/nuxt/nuxt.js/issues/3411#issuecomment-394224616

Which pkg manager do you use? npm or yarn ? Use npm is work fine.

[manniL]

Missed the comment. However, if NPM works but yarn doesn't, are you sure it's a Nuxt bug then?

[aldarund]

@mannil well, even if it's yarn bug it still should work for people who are using yarn via some workaround. It also might be a bug in npm that make it work fine, but not as it should :)
It must be something with resolving nested dependency difference on yarn vs npm

[Atinux]

Will look into it

[yoyo837]

yarn upgrade

This command can make it work fine.

[cmty[bot]]

This bug-report has been fixed by @Atinux, @aldarund, @yoyo837, @manniL.

[lock[bot]]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.