automerge / hypermerge

Build p2p collaborative applications without any server infrastructure in Node.js
MIT License
1.28k stars 66 forks source link

Use RepoId for ClockStore #39

Closed mjtognetti closed 4 years ago

mjtognetti commented 4 years ago

Update ClockStore to use RepoId. There's some strangeness with exposing the repo id as a buffer to be used by discover-swarm/discovery-cloud. I believe we'll be changing those requirements, so for now we expose a swarmId. I expect to remove swarmId in a followup patch.

NOTE: This patch doesn't migrate the sql schema, it assumes we start from scratch. This assumes we haven't updated any downstream apps to use the latest hypermerge since the original ClockStore was introduced (pushpin master is yarn locked to a version of hypermerge before SqlDatabase/ClockStore).

Happy to introduce a migration if anyone thinks it's important or useful.