js-delta-crdts icon indicating copy to clipboard operation
js-delta-crdts copied to clipboard

migrate internals to immutable types

Open pgte opened this issue 6 years ago • 1 comments

A CRDT type is defined in a stateless manner. All return values are supposed to be immutable, and there's a lot of cloning happening just to make sure we don't mutate past state. Instead of that, I think we could have better performance (and less error-prone) if we used immutable types internally.

pgte avatar Dec 08 '18 16:12 pgte

Looks like immutable is already in use. Strongly suggest the use of immer instead. Immutability is important, but immutable.js was a train wreck. https://immerjs.github.io/immer/docs/introduction

robchristian avatar Nov 26 '19 23:11 robchristian