lib/stream.js

@@ -208,7 +208,8 @@ class SyncStream extends Pipeable {
     this.#remoteHave.set(id, remoteHaveRange)
     this.#remoteWant.set(id, remoteWantRange)
     const goal = this.#goals.get(id)
-    const haveRange = this.#localHave.get(id) ?? [-1, -1]
+    const haveRange = this.#localHave.get(id)
+    if (!haveRange) throw new Error(`Local have-range not set for ${id}`)
     const localWant = await this.#algo.wantRange(haveRange, remoteHaveRange, goal)
     this.#localWant.set(id, localWant)
     const localBloom0 = await this.#algo.bloomFor(id, 0, localWant)

package.json

@@ -1,6 +1,6 @@
 {
   "name": "pzp-sync",
-  "version": "1.0.4",
+  "version": "1.0.3",
   "description": "PZP replication using Kleppmann's hash graph sync",
   "author": "Andre Staltz <contact@staltz.com>",
   "license": "MIT",

test/realtime.test.js

@@ -306,76 +306,6 @@ test('create 100 messages in parallel that still manage to sync realtime (withou
   }
   assert.equal(arr.length, n, `alice has ${arr.length} posts from bob`)
 
-  await p(remoteAlice.close)(true)
-  await p(alice.close)(true)
-  await p(bob.close)(true)
-})
-
-test('create 200 messages that manage to replicate with low "newest" goals', async (t) => {
-  const n = 200
-
-  const alice = createPeer({ name: 'alice' })
-  const bob = createPeer({ name: 'bob' })
-
-  await alice.db.loaded()
-  await bob.db.loaded()
-
-  const bobID = await p(bob.db.account.create)({
-    subdomain: 'account',
-    _nonce: 'bob',
-  })
-
-  const bobPostsID = bob.db.feed.getID(bobID, 'post')
-
-  {
-    const arr = (await flatten(alice.db.msgs()))
-      .filter((msg) => msg.metadata.account === bobID && msg.data)
-      .map((msg) => msg.data.text)
-    assert.deepEqual(arr, [], 'alice has no posts from bob')
-  }
-
-  const confirmed = []
-  // for keeping track of which msgs have arrived
-  for (let i = 0; i < n; i++) {
-    confirmed.push(false)
-  }
-
-  alice.db.onRecordAdded(rec => {
-    if (rec.msg.data?.text) {
-      const num = Number.parseInt(rec.msg.data.text)
-      confirmed[num] = true
-    }
-  })
-
-  bob.goals.set(bobPostsID, 'newest-50')
-  alice.goals.set(bobPostsID, 'newest-50')
-
-  const remoteAlice = await p(bob.connect)(alice.getAddress())
-  assert('alice and bob connected')
-
-  bob.sync.start()
-  await p(setTimeout)(1000)
-  assert('sync!')
-
-  const hundred = []
-  for (let i = 0; i < n; i++) {
-      hundred.push(i)
-  }
-  Promise.all(hundred.map(i => p(bob.db.feed.publish)({
-    account: bobID,
-    domain: 'post',
-    data: { text: `${i}` },
-  })))
-  assert(`bob published ${n} posts in parallel`)
-
-  let tries = 30
-  // just waiting for them to arrive
-  do {
-    await p(setTimeout)(100)
-  } while (!confirmed.every(v => v === true) && tries-- > 0)
-
-  assert.equal(confirmed.filter(v => v === true).length, n, `alice has all of bob's posts`)
-
   await p(remoteAlice.close)(true)
   await p(alice.close)(true)
   await p(bob.close)(true)