Changeset: Migrate from opAssembler() to serializeOps()

Author: rhansen

CHANGELOG.md

@@ -35,6 +35,8 @@
     * The `newOp()` function is deprecated; use the new `Op` class instead.
     * The `opIterator()` function is deprecated; use the new `OpIter` class
       instead.
+    * The `opAssembler()` function is deprecated; use the new `serializeOps()`
+      function instead.
 
 ### Notable enhancements
 

src/node/utils/padDiff.js

@@ -209,7 +209,7 @@ PadDiff.prototype._extendChangesetWithAuthor = (changeset, author, apool) => {
   // unpack
   const unpacked = Changeset.unpack(changeset);
 
-  const assem = Changeset.opAssembler();
+  const ops = [];
 
   // create deleted attribs
   const authorAttrib = apool.putAttrib(['author', author || '']);
@@ -224,13 +224,12 @@ PadDiff.prototype._extendChangesetWithAuthor = (changeset, author, apool) => {
       // this is operator changes only attributes, let's mark which author did that
       operator.attribs += `*${Changeset.numToString(authorAttrib)}`;
     }
-
-    // append the new operator to our assembler
-    assem.append(operator);
+    ops.push(operator);
   }
 
   // return the modified changeset
-  return Changeset.pack(unpacked.oldLen, unpacked.newLen, assem.toString(), unpacked.charBank);
+  return Changeset.pack(
+      unpacked.oldLen, unpacked.newLen, Changeset.serializeOps(ops), unpacked.charBank);
 };
 
 // this method is 80% like Changeset.inverse. I just changed so instead of reverting,

src/static/js/Changeset.js

@@ -318,14 +318,16 @@ const copyOp = (op1, op2 = new exports.Op()) => Object.assign(op2, op1);
  * @param {Iterable.<Op>} ops - Iterable of operations to serialize.
  * @returns {string} A string containing the encoded op data (example: '|5=2p=v*4*5+1').
  */
-const serializeOps = (ops) => {
+exports.serializeOps = (ops) => {
   let res = '';
   for (const op of ops) res += op.toString();
   return res;
 };
 
 /**
  * Serializes a sequence of Ops.
+ *
+ * @deprecated Use `serializeOps` instead.
  */
 class OpAssembler {
   constructor() {
@@ -415,7 +417,7 @@ class MergingOpAssembler {
   }
 
   _serialize(finalize) {
-    this._serialized = serializeOps(squashOps(this._ops, finalize));
+    this._serialized = exports.serializeOps(squashOps(this._ops, finalize));
   }
 
   clear() {
@@ -509,7 +511,7 @@ class SmartOpAssembler {
   }
 
   _serialize(finalize) {
-    this._serialized = serializeOps((function* () {
+    this._serialized = exports.serializeOps((function* () {
       this._lengthChange = yield* canonicalizeOps(this._ops, finalize);
     }).call(this));
   }
@@ -622,9 +624,13 @@ exports.smartOpAssembler = () => new SmartOpAssembler();
 exports.mergingOpAssembler = () => new MergingOpAssembler();
 
 /**
+ * @deprecated Use `serializeOps` instead.
  * @returns {OpAssembler}
  */
-exports.opAssembler = () => new OpAssembler();
+exports.opAssembler = () => {
+  warnDeprecated('Changeset.opAssembler() is deprecated; use Changeset.serializeOps() instead');
+  return new OpAssembler();
+};
 
 /**
  * A custom made String Iterator

src/tests/frontend/specs/easysync.js

@@ -31,9 +31,7 @@ const randInt = (maxValue) => Math.floor(Math.random() * maxValue);
 describe('easysync', function () {
   it('throughIterator', async function () {
     const x = '-c*3*4+6|3=az*asdf0*1*2*3+1=1-1+1*0+1=1-1+1|c=c-1';
-    const assem = Changeset.opAssembler();
-    for (const op of new Changeset.OpIter(x)) assem.append(op);
-    expect(assem.toString()).to.equal(x);
+    expect(Changeset.serializeOps(new Changeset.OpIter(x))).to.equal(x);
   });
 
   it('throughSmartAssembler', async function () {