Changeset: Move changeset logic to a new Changeset class

Author: rhansen

src/node/handler/PadMessageHandler.js

@@ -571,8 +571,7 @@ const handleUserChanges = async (socket, message) => {
   // create the changeset
   try {
     try {
-      // Verify that the changeset has valid syntax and is in canonical form
-      Changeset.checkRep(changeset);
+      const cs = Changeset.unpack(changeset).validate();
 
       // Verify that the attribute indexes used in the changeset are all
       // defined in the accompanying attribute pool.
@@ -583,7 +582,7 @@ const handleUserChanges = async (socket, message) => {
       });
 
       // Validate all added 'author' attribs to be the same value as the current user
-      for (const op of Changeset.deserializeOps(Changeset.unpack(changeset).ops)) {
+      for (const op of Changeset.deserializeOps(cs.ops)) {
         // + can add text with attribs
         // = can change or add attribs
         // - can have attribs, but they are discarded and don't show up in the attribs -

src/node/utils/padDiff.js

@@ -455,7 +455,7 @@ PadDiff.prototype._createDeletionChangeset = function (cs, startAText, apool) {
     }
   }
 
-  return Changeset.checkRep(builder.toString());
+  return builder.build().validate().toString();
 };
 
 // export the constructor