deep require is no longer supported for uuid library

Question

deep require is no longer supported for uuid library

hannaol opened this issue 4 years ago · comments

First of all, thank you for your very helpful library !

I ran into the following problem:
In the file node-pptx/lib/factories/ppt/index.js line 5 a deep requires is used for the uuid library :
const uuidv4 = require('uuid/v4');
According to their page this is deprecated and no longer supported: https://github.com/uuidjs/uuid#deep-requires-no-longer-supported

Therefore the following error is thrown in my project: Error [ERR_PACKAGE_PATH_NOT_EXPORTED]: Package subpath './v4' is not defined by "exports" in .../node_modules/uuid/package.json

Mark Daggett · Answer 1 · Wed Jan 13 2021 01:59:22 GMT+0800 (China Standard Time)

@gregdolley Would you mind checking this out?

gregdolley · Answer 2 · Fri Jan 15 2021 23:58:36 GMT+0800 (China Standard Time)

Will do!

…

On Tue, Jan 12, 2021, 09:59 Mark Daggett ***@***.***> wrote: @gregdolley <https://github.com/gregdolley> Would you mind checking this out? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#70 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABN3O6L7EO5BRN64RSHLFQDSZSEYVANCNFSM4UFDGIVA> .

gregdolley · Answer 3 · Mon Feb 08 2021 01:42:21 GMT+0800 (China Standard Time)

@heavysixer: this is popping up because we use "require" the uuid library like this in many places:

    const uuidv4 = require('uuid/v4');

...where there is a "uuid/" inside the require param. The warning is telling us, that in uuid lib v7.x (we are on 3.x), we can't import it like this anymore, but rather should use the "import" syntax or something like: "const { v4: uuidv4 } = require('uuid');" for commonJS. But since we're still on v3.x, we shouldn't have to worry about this. I'll try and find why @hannaol was still getting this error.

gregdolley · Answer 4 · Sat Feb 13 2021 13:06:29 GMT+0800 (China Standard Time)

@hannaol - what version of node are you running your code under? Also, could you please post the part of your code that is causing this error?

Thanks!

Able1991 · Answer 5 · Mon Mar 01 2021 22:38:39 GMT+0800 (China Standard Time)

An error occurs when using require("node js-pptx");
Maybe the problem is that the version is not explicitly specified in the dependencies?

Stas Volianskyi · Answer 6 · Mon Oct 18 2021 18:01:31 GMT+0800 (China Standard Time)

The issue is still present and makes the library unusable
Any updates about the fix?

hannaol · Answer 7 · Mon Oct 18 2021 19:02:59 GMT+0800 (China Standard Time)

Sorry, i totally forgot about replying to this!

So for me the issue was caused because another library in my project was also using uuid, but version 8. As far as i can remember i installed that dependency before adding nodejs-pptx.
I was able to fix this by including "uuid": "~7" in the dependencies in the package.json of my project and doing a clean install.

(node version 12, and the code that caused this error to arise was the code sample to create a new powerpoint presentation, as far as i can remember)