cleanup

lib/models/node.js

@@ -765,7 +765,6 @@ const Node = {
    */
 
   getPath(key) {
-    debugger
     key = Normalize.key(key)
 
     if (key == this.key) return []

lib/models/transform.js

@@ -76,6 +76,20 @@ class Transform {
     return 'transform'
   }
 
+  /**
+   * Add an `operation` to the transform that resulted in `state`.
+   *
+   * @param {State} state
+   * @param {Object} operation
+   * @return {Transform}
+   */
+
+  add(state, operation) {
+    this.state = state
+    this.operations.push(operation)
+    return this
+  }
+
   /**
    * Apply the transform and return the new state.
    *

lib/transforms/by-key.js

@@ -25,16 +25,13 @@ export function addMarkByKey(transform, key, offset, length, mark) {
   document = document.updateDescendant(node)
   state = state.merge({ document })
 
-  transform.state = state
+  return transform.add(state, {
-  transform.operations.push({
     type: 'add-mark',
-    offset,
     length,
     mark,
+    offset,
     path,
   })
-
-  return transform
 }
 
 /**
@@ -60,15 +57,12 @@ export function insertNodeByKey(transform, key, index, node) {
   document = document.normalize()
   state = state.merge({ document })
 
-  transform.state = state
+  return transform.add(state, {
-  transform.operations.push({
     type: 'insert-node',
     index,
     node,
     path,
   })
-
-  return transform
 }
 
 /**
@@ -92,16 +86,13 @@ export function insertTextByKey(transform, key, offset, text, marks) {
   document = document.updateDescendant(node)
   state = state.merge({ document })
 
-  transform.state = state
+  return transform.add(state, {
-  transform.operations.push({
     type: 'insert-text',
     offset,
     marks,
     path,
     text,
   })
-
-  return transform
 }
 
 /**
@@ -135,15 +126,12 @@ export function moveNodeByKey(transform, key, newKey, newIndex) {
   document = document.normalize()
   state = state.merge({ document })
 
-  transform.state = state
+  return transform.add(state, {
-  transform.operations.push({
     type: 'move-node',
     path,
     newPath,
     newIndex,
   })
-
-  return transform
 }
 
 /**
@@ -169,16 +157,13 @@ export function removeMarkByKey(transform, key, offset, length, mark) {
   document = document.updateDescendant(node)
   state = state.merge({ document })
 
-  transform.state = state
+  return transform.add(state, {
-  transform.operations.push({
     type: 'remove-mark',
     offset,
     length,
     mark,
     path,
   })
-
-  return transform
 }
 
 /**
@@ -203,13 +188,10 @@ export function removeNodeByKey(transform, key) {
   document = document.normalize()
   state = state.merge({ document })
 
-  transform.state = state
+  return transform.add(state, {
-  transform.operations.push({
     type: 'remove-node',
     path,
   })
-
-  return transform
 }
 
 /**
@@ -233,15 +215,12 @@ export function removeTextByKey(transform, key, offset, length) {
   document = document.normalize()
   state = state.merge({ document })
 
-  transform.state = state
+  return transform.add(state, {
-  transform.operations.push({
     type: 'remove-text',
     offset,
     length,
     path,
   })
-
-  return transform
 }
 
 /**
@@ -268,8 +247,7 @@ export function setMarkByKey(transform, key, offset, length, mark, properties) {
   document = document.updateDescendant(node)
   state = state.merge({ document })
 
-  transform.state = state
+  return transform.add(state, {
-  transform.operations.push({
     type: 'set-mark',
     offset,
     length,
@@ -277,8 +255,6 @@ export function setMarkByKey(transform, key, offset, length, mark, properties) {
     path,
     properties,
   })
-
-  return transform
 }
 
 /**
@@ -303,14 +279,11 @@ export function setNodeByKey(transform, key, properties) {
   document = document.normalize()
   state = state.merge({ document })
 
-  transform.state = state
+  return transform.add(state, {
-  transform.operations.push({
     type: 'set-node',
     path,
     properties,
   })
-
-  return transform
 }
 
 /**
@@ -366,13 +339,10 @@ export function splitNodeByKey(transform, key, offset) {
   document = isParent ? parent : document.updateDescendant(parent)
   state = state.merge({ document })
 
-  transform.state = state
+  return transform.add(state, {
-  transform.operations.push({
     type: 'split-node',
     offset,
     path,
   })
-
-  return transform
 }