Missing dependency `p-limit`

[vera]

I included @iqss/dataverse-client-javascript as a dependency in a small project, and got the following error message:

Error: Cannot find module 'p-limit'
Require stack:
- /app/node_modules/@iqss/dataverse-client-javascript/dist/files/infra/clients/DirectUploadClient.js
- /app/node_modules/@iqss/dataverse-client-javascript/dist/files/index.js
    at Module._resolveFilename (node:internal/modules/cjs/loader:1569:15)
    at wrapResolveFilename (node:internal/modules/cjs/loader:1123:27)
    at defaultResolveImplForCJSLoading (node:internal/modules/cjs/loader:1147:10)
    at resolveForCJSWithHooks (node:internal/modules/cjs/loader:1174:12)
    at Module._load (node:internal/modules/cjs/loader:1346:5)
    at wrapModuleLoad (node:internal/modules/cjs/loader:261:19)
    at Module.require (node:internal/modules/cjs/loader:1679:12)
    at require (node:internal/modules/helpers:196:16)
    at Object.<anonymous> (/app/node_modules/@iqss/dataverse-client-javascript/dist/files/infra/clients/DirectUploadClient.js:17:35)
    at Module._compile (node:internal/modules/cjs/loader:1934:14) {
  code: 'MODULE_NOT_FOUND',
  requireStack: [
    '/app/node_modules/@iqss/dataverse-client-javascript/dist/files/infra/clients/DirectUploadClient.js',
    '/app/node_modules/@iqss/dataverse-client-javascript/dist/files/index.js'
  ]
}

I had to also add p-limit dependency to my project to resolve the error, although I don't directly use it.

Should the p-limit dependency be added to this project's package.json?

[ChengShi-1]

Thanks for catching this. yes, we should include this since it is imported at runtime