boardgameio / boardgame.io / 30728433977

100%

master: 100%

push

github

chore(deps): bump nanoid to 6 and p-queue to 9 (#1312)

* build(jest): transform the ESM-only dependencies

nanoid and p-queue publish ESM-only builds from their next majors, and
p-queue pulls in p-timeout, which is ESM-only too. Jest leaves node_modules
untransformed by default, so every suite that reaches them fails to parse.
Adding them to the allowlist lets babel convert them like the other ESM
packages already listed here.

* chore(deps): bump nanoid to 6 and p-queue to 9

Supersedes #1301 and #1306. Both are ESM-only from these majors, which is
why they land together with the jest transform change rather than on their
own.

* refactor(server): drop the p-queue default-export unwrap

p-queue 6 shipped CommonJS with `exports.default`, so the ESM server build
bound the default import to the exports object and `new PQueue()` threw
"PQueue is not a constructor". p-queue 9 is native ESM with a real default
export and both builds get the class directly.

Verified by constructing a queue from the built dist/cjs/server.js and
dist/esm/server.js.

* test(integration): construct a match queue in the node smoke tests

The smoke tests only loaded boardgame.io/server, so nothing checked that
the p-queue default import survives bundling — the unwrap removed in the
previous commit could have been wrong in either direction without CI
noticing. Pairing these assertions with p-queue 6 turns them red with
"PQueue is not a constructor".

* build: raise the node floor to 22.12

dist/cjs/*.js requires nanoid and p-queue, which are now ESM-only.
require() of an ES module was only unflagged in Node 22.12, so CommonJS
consumers on 22.0-22.11 would get ERR_REQUIRE_ESM against a package that
claims to support them.

---------

Co-authored-by: Rupesh Pandey <pandeyrupesh00@gmail.com>

2038 of 2038 branches covered (100.0%)

Branch coverage included in aggregate %.

2 of 2 new or added lines in 1 file covered. (100.0%)

2918 of 2918 relevant lines covered (100.0%)

26350.97 hits per line