refactor: only report moves, not indents & outdents

Author: nperez0111

packages/core/src/api/__snapshots__/blocks-indented-changed.json

@@ -124,6 +124,6 @@
     "source": {
       "type": "local",
     },
-    "type": "indent",
+    "type": "move",
   },
 ]

packages/core/src/api/__snapshots__/blocks-moved-deeper-into-nesting.json

@@ -159,6 +159,6 @@
     "source": {
       "type": "local",
     },
-    "type": "indent",
+    "type": "move",
   },
 ]

packages/core/src/api/__snapshots__/blocks-moved-multiple-in-same-transaction.json

@@ -90,7 +90,7 @@
     "source": {
       "type": "local",
     },
-    "type": "outdent",
+    "type": "move",
   },
   {
     "block": {
@@ -183,6 +183,6 @@
     "source": {
       "type": "local",
     },
-    "type": "outdent",
+    "type": "move",
   },
 ]

packages/core/src/api/__snapshots__/blocks-moved-to-different-parent.json

@@ -73,6 +73,6 @@
     "source": {
       "type": "local",
     },
-    "type": "outdent",
+    "type": "move",
   },
 ]

packages/core/src/api/__snapshots__/blocks-moved-to-root-level.json

@@ -73,6 +73,6 @@
     "source": {
       "type": "local",
     },
-    "type": "outdent",
+    "type": "move",
   },
 ]

packages/core/src/api/__snapshots__/blocks-outdented-changed.json

@@ -124,6 +124,6 @@
     "source": {
       "type": "local",
     },
-    "type": "outdent",
+    "type": "move",
   },
 ]

packages/core/src/api/nodeUtil.ts

@@ -104,14 +104,14 @@ export type BlocksChanged<
         prevBlock: undefined;
       }
     | {
-        type: "update" | "move";
+        type: "update";
         /**
          * The previous block.
          */
         prevBlock: Block<BSchema, ISchema, SSchema>;
       }
     | {
-        type: "indent" | "outdent";
+        type: "move";
         /**
          * The previous block.
          */
@@ -169,30 +169,33 @@ function determineChangeSource(transaction: Transaction): BlockChangeSource {
   return { type: "local" };
 }
 
-type BlockInfo<
-  BSchema extends BlockSchema,
-  ISchema extends InlineContentSchema,
-  SSchema extends StyleSchema,
-> = {
-  block: Block<BSchema, ISchema, SSchema>;
-  parentId: string | undefined;
-  depth: number;
-};
-
 function collectAllBlocks<
   BSchema extends BlockSchema,
   ISchema extends InlineContentSchema,
   SSchema extends StyleSchema,
->(doc: Node): Record<string, BlockInfo<BSchema, ISchema, SSchema>> {
-  const blocks: Record<string, BlockInfo<BSchema, ISchema, SSchema>> = {};
+>(
+  doc: Node,
+): Record<
+  string,
+  {
+    block: Block<BSchema, ISchema, SSchema>;
+    parentId: string | undefined;
+  }
+> {
+  const blocks: Record<
+    string,
+    {
+      block: Block<BSchema, ISchema, SSchema>;
+      parentId: string | undefined;
+    }
+  > = {};
   const pmSchema = getPmSchema(doc);
   doc.descendants((node, pos) => {
     if (isNodeBlock(node)) {
       const parentId = getParentBlockId(doc, pos);
       blocks[node.attrs.id] = {
         block: nodeToBlock(node, pmSchema),
         parentId,
-        depth: doc.resolve(pos).depth,
       };
     }
     return true;
@@ -257,46 +260,20 @@ export function getBlocksChangedByTransaction<
       const prev = prevBlocks[id];
       const next = nextBlocks[id];
       const isParentDifferent = prev.parentId !== next.parentId;
-      const isDepthDifferent = prev.depth !== next.depth;
 
-      if (isParentDifferent || isDepthDifferent) {
-        const isDepthIncrease = next.depth > prev.depth;
-        const isDepthDecrease = next.depth < prev.depth;
-
-        if (isDepthIncrease) {
-          changes.push({
-            type: "indent",
-            block: next.block,
-            prevBlock: prev.block,
-            source,
-            prevParent: prev.parentId
-              ? prevBlocks[prev.parentId]?.block
-              : undefined,
-            currentParent: next.parentId
-              ? nextBlocks[next.parentId]?.block
-              : undefined,
-          });
-        } else if (isDepthDecrease) {
-          changes.push({
-            type: "outdent",
-            block: next.block,
-            prevBlock: prev.block,
-            source,
-            prevParent: prev.parentId
-              ? prevBlocks[prev.parentId]?.block
-              : undefined,
-            currentParent: next.parentId
-              ? nextBlocks[next.parentId]?.block
-              : undefined,
-          });
-        } else {
-          changes.push({
-            type: "move",
-            block: next.block,
-            prevBlock: prev.block,
-            source,
-          });
-        }
+      if (isParentDifferent) {
+        changes.push({
+          type: "move",
+          block: next.block,
+          prevBlock: prev.block,
+          source,
+          prevParent: prev.parentId
+            ? prevBlocks[prev.parentId]?.block
+            : undefined,
+          currentParent: next.parentId
+            ? nextBlocks[next.parentId]?.block
+            : undefined,
+        });
       } else if (areBlocksDifferentExcludingChildren(prev.block, next.block)) {
         changes.push({
           type: "update",