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

[kapv89]

Checklist

• [x] Are you reporting a bug? Use github issues for bug reports and feature requests. For general questions, please use https://discuss.yjs.dev/
• [x] 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 crdt-backend@1.0.0 /Users/kv/Projects/ref/yjs-scalable-ws-backend ├─┬ y-websocket@1.3.16 │ ├─┬ y-leveldb@0.1.0 │ │ └── yjs@13.5.42 deduped │ └── yjs@13.5.42 deduped └── yjs@13.5.42