braid-text 0.2.80 → 0.2.81

package/index.js

@@ -997,7 +997,7 @@ function create_braid_text() {
 
             await resource.db_delta(resource.doc.getPatchSince(v_before))
 
-            for (var [client, x] of post_commit_updates) client.my_subscribe(x)
+            await Promise.all(post_commit_updates.map(([client, x]) => client.my_subscribe(x)))
 
             return { change_count }
         })

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "braid-text",
-  "version": "0.2.80",
+  "version": "0.2.81",
   "description": "Library for collaborative text over http using braid.",
   "author": "Braid Working Group",
   "repository": "braid-org/braid-text",

package/test/tests.js

@@ -1144,6 +1144,50 @@ runTest(
     'x'
 )
 
+runTest(
+    "test put awaits subscriber callbacks",
+    async () => {
+        var key = 'test-' + Math.random().toString(36).slice(2)
+
+        var r = await braid_fetch(`/eval`, {
+            method: 'PUT',
+            body: `void (async () => {
+                var order = []
+
+                // Subscribe with an async callback that takes some time
+                braid_text.get('/${key}', {
+                    subscribe: async (update) => {
+                        if (update.version?.[0]?.startsWith('test-v')) {
+                            order.push('subscriber-start')
+                            await new Promise(done => setTimeout(done, 50))
+                            order.push('subscriber-end')
+                        }
+                    }
+                })
+
+                // Wait for subscription to be established
+                await new Promise(done => setTimeout(done, 50))
+
+                // Put should await the subscriber callback
+                await braid_text.put('/${key}', {
+                    version: ['test-v-0'],
+                    parents: [],
+                    body: 'hello'
+                })
+                order.push('put-done')
+
+                // If put properly awaited, order should be: subscriber-start, subscriber-end, put-done
+                // If put didn't await, order would be: subscriber-start, put-done, subscriber-end
+                res.end(order.join(','))
+            })()`
+        })
+        if (!r.ok) return 'eval failed: ' + r.status
+
+        return await r.text()
+    },
+    'subscriber-start,subscriber-end,put-done'
+)
+
 runTest(
     "test out-of-order PUTs",
     async () => {