lean-poker / boardgame.io / 25944781738

100%

push

github

refactor(client): adopt svelte-5 mount/unmount API

Svelte 5 removed the class-based component constructor; \`new Debug({target,
props})\` is now \`mount(Debug, {target, props})\` and \`$destroy()\` is now
\`unmount(handle)\`. ClientManager.debugPanel changes from a Debug instance
to whatever mount() returns (an exports proxy), and DebugOpt.impl from
\`typeof Debug\` to \`Component\` so callers that supply a custom impl pass
the new component shape rather than the old class.

Two test-only fixes ride along because the rendered code path now goes
through svelte's client runtime in jsdom:

- moduleNameMapper routes \`svelte\` (and its client/server-split
  subpaths) to the index-client builds; jest's customExportConditions
  is "node", which would otherwise pull index-server and trip svelte's
  \`lifecycle_function_unavailable\` guard. Adding the browser
  condition globally is a non-starter because it would also force
  nanoid onto its ESM-only browser bundle.
- jest.setup.js stubs Element.prototype.animate. jsdom doesn't ship
  Web Animations and svelte 5's transitions call animate(); the stub
  fires onfinish synchronously so outro transitions complete and
  Svelte removes the element in the same tick.

1786 of 1789 branches covered (99.83%)

Branch coverage included in aggregate %.

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

2669 of 2673 relevant lines covered (99.85%)

28548.08 hits per line