Why not add "type": "module" in package.json of y-websocket?

[kapv89]

Checklist

• Are you reporting a bug? Use github issues for bug reports and feature requests. For general questions, please use https://discuss.yjs.dev/
• Try to report your issue in the correct repository. Yjs consists of many modules. When in doubt, report it to https://github.com/yjs/yjs/issues/

Describe the bug
As seen in this repo - https://github.com/kapv89/yjs-scalable-ws-backend/tree/external_api_persistence ... adding "type": "module" to package.json of y-websocket allows y-websocket to be used in node-environment for unit tests. Not sure why this is not done yet by default on this package.

To Reproduce
Steps to reproduce the behavior:

1. Clone https://github.com/kapv89/yjs-scalable-ws-backend/tree/external_api_persistence (the specific branch)
2. Follow steps in Usage. See tests working fine
3. In one run, don't follow step#5 in Usage, complete the steps and see the error caused by y-websocket lacking "type": "module"

Expected behavior
It should be trivial to use y-websocket in tests of your y-crdt websocket-backend. It's not right now.

Environment Information

• Node.js
  [email protected] /Users/kv/Projects/ref/yjs-scalable-ws-backend
  ├─┬ [email protected]
  │ ├─┬ [email protected]
  │ │ └── [email protected] deduped
  │ └── [email protected] deduped
  └── [email protected]