braid-text 0.2.46 → 0.2.47

package/index.js

@@ -293,14 +293,20 @@ braid_text.get = async (key, options) => {
         } else {
 
             if (options.accept_encoding?.match(/updates\s*\((.*)\)/)?.[1].split(',').map(x=>x.trim()).includes('dt')) {
-                var bytes = resource.doc.toBytes()
-                if (options.parents) {
-                    var doc = Doc.fromBytes(bytes)
-                    bytes = doc.getPatchSince(
-                        dt_get_local_version(bytes, options.parents))
-                    doc.free()
+                // optimization: if client wants past current version,
+                //               send empty dt
+                if (options.parents && v_eq(options.parents, version)) {
+                    options.subscribe({ encoding: 'dt', body: new Doc().toBytes() })
+                } else {
+                    var bytes = resource.doc.toBytes()
+                    if (options.parents) {
+                        var doc = Doc.fromBytes(bytes)
+                        bytes = doc.getPatchSince(
+                            dt_get_local_version(bytes, options.parents))
+                        doc.free()
+                    }
+                    options.subscribe({ encoding: 'dt', body: bytes })
                 }
-                options.subscribe({ encoding: 'dt', body: bytes })
             } else {
                 var updates = null
                 if (!options.parents && !options.version) {

package/package.json

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

package/test/test.html

@@ -674,6 +674,45 @@ runTest(
     'xy'
 )
 
+runTest(
+    "test accept-encoding updates(dt) (with parents which are current version)",
+    async () => {
+        await dt_p
+        let key = 'test-' + Math.random().toString(36).slice(2)
+        var doc = new Doc('hi')
+        doc.ins(0, 'xy')
+
+        let r = await braid_fetch(`/${key}`, {
+            method: 'PUT',
+            version: ['hi-1'],
+            parents: [],
+            body: 'xy'
+        })
+        if (!r.ok) throw 'got: ' + r.statusCode
+
+        var a = new AbortController()
+        let r2 = await braid_fetch(`/${key}`, {
+            signal: a.signal,
+            parents: ['hi-1'],
+            subscribe: true,
+            headers: {
+                'merge-type': 'dt',
+                'X-Accept-Encoding': 'updates(dt)'
+            }
+        })
+
+        return await new Promise(done => {
+            r2.subscribe(u => {
+                doc.mergeBytes(u.body)
+                done(doc.get())
+                doc.free()
+                a.abort()
+            })
+        })
+    },
+    'xy'
+)
+
 runTest(
     "test accept-encoding updates(dt)",
     async () => {