require() of ES Module

[christian-bromann]

It seems like the recent update to the package, bumped the version of got to 12.6.0 which is a ESM only package. This breaks the whole package because it runs in CJS (see #153).

Raised by webdriverio/webdriverio#11077 (comment)

[diemol]

That was an oversight on my part.

I will move got to 11.8.6 and make a new release.

[diemol]

7.3.1 is out.

[redbugz]

Seems we need a 7.3.2 for query-string now:
a781410#commitcomment-127284025

[itssumitrai]

Same, seeing ES module issue with query-string

[diemol]

See #207

I will have time again to check this next week.

[snyamathi]

@diemol Has this change been released, or do you have time to create a release? I still see query-string v8.1.0 in the latest version

$ curl -s https://registry.npmjs.org/saucelabs/latest | jq '.dependencies' -
{
  "@xhmikosr/downloader": "^13.0.1",
  "change-case": "^4.1.2",
  "form-data": "^4.0.0",
  "got": "^11.8.6",
  "hash.js": "^1.1.7",
  "query-string": "^8.1.0",
  "tunnel": "^0.0.6",
  "yargs": "^17.2.1"
}

[diemol]

Work still needs to be done to fix this.

[lotuswood]

@diemol
Based upon what I see in the 7.3.1 tag, I wonder if you can update your saucelabs/build/index.js to have something like
const queryString = await import('./query-string');
instead of the current
import queryString from 'query-string;'

to fix this ERR_REQUIRE_ESM error, given that cjs modules generally need to load esm modules using the dynamic import syntax as per suggested by the original error message?

It may not be a good long term solution to downgrade your query-string dependency to its older cjs module, if #207 is what you intend to release as a fix to this prob.