@manuscripts/track-changes-plugin 0.4.4-LEAN-1839 → 0.4.4-LEAN-1839-v3

package/dist/index.cjs

@@ -1618,7 +1618,7 @@ function diffChangeSteps(deleted, inserted) {
  * @returns
  */
 const getSelectionStaticConstructor = (sel) => Object.getPrototypeOf(sel).constructor;
-const isHighlightMarkerNode = (node) => node.type === node.type.schema.nodes.highlight_marker;
+const isHighlightMarkerNode = (node) => node && node.type === node.type.schema.nodes.highlight_marker;
 /**
  * Inverts transactions to wrap their contents/operations with track data instead
  *
@@ -1648,7 +1648,7 @@ function trackTransaction(tr, oldState, newTr, authorID) {
     let iters = 0;
     log.info('ORIGINAL transaction', tr);
     tr.steps.forEach((step) => {
-        var _a, _b;
+        var _a, _b, _c, _d;
         log.info('transaction step', step);
         iters += 1;
         if (iters > 20) {
@@ -1662,13 +1662,20 @@ function trackTransaction(tr, oldState, newTr, authorID) {
             return;
         }
         else if (step instanceof prosemirrorTransform.ReplaceStep) {
-            let [steps, startPos] = trackReplaceStep(step, oldState, newTr, emptyAttrs);
             const { slice } = step;
             if (((_b = (_a = slice === null || slice === void 0 ? void 0 : slice.content) === null || _a === void 0 ? void 0 : _a.content) === null || _b === void 0 ? void 0 : _b.length) === 1 &&
                 isHighlightMarkerNode(slice.content.content[0])) {
                 // don't track highlight marker nodes
                 return;
             }
+            let [steps, startPos] = trackReplaceStep(step, oldState, newTr, emptyAttrs);
+            if (steps.length === 1) {
+                const step = steps[0]; // eslint-disable-line @typescript-eslint/no-explicit-any
+                if (isHighlightMarkerNode((step === null || step === void 0 ? void 0 : step.node) || ((_d = (_c = step === null || step === void 0 ? void 0 : step.slice) === null || _c === void 0 ? void 0 : _c.content) === null || _d === void 0 ? void 0 : _d.content[0]))) {
+                    // don't track deleted highlight marker nodes
+                    return;
+                }
+            }
             log.info('CHANGES: ', steps);
             // deleted and merged really...
             const deleted = steps.filter((s) => s.type !== 'insert-slice');

package/dist/index.js

@@ -1610,7 +1610,7 @@ function diffChangeSteps(deleted, inserted) {
  * @returns
  */
 const getSelectionStaticConstructor = (sel) => Object.getPrototypeOf(sel).constructor;
-const isHighlightMarkerNode = (node) => node.type === node.type.schema.nodes.highlight_marker;
+const isHighlightMarkerNode = (node) => node && node.type === node.type.schema.nodes.highlight_marker;
 /**
  * Inverts transactions to wrap their contents/operations with track data instead
  *
@@ -1640,7 +1640,7 @@ function trackTransaction(tr, oldState, newTr, authorID) {
     let iters = 0;
     log.info('ORIGINAL transaction', tr);
     tr.steps.forEach((step) => {
-        var _a, _b;
+        var _a, _b, _c, _d;
         log.info('transaction step', step);
         iters += 1;
         if (iters > 20) {
@@ -1654,13 +1654,20 @@ function trackTransaction(tr, oldState, newTr, authorID) {
             return;
         }
         else if (step instanceof ReplaceStep) {
-            let [steps, startPos] = trackReplaceStep(step, oldState, newTr, emptyAttrs);
             const { slice } = step;
             if (((_b = (_a = slice === null || slice === void 0 ? void 0 : slice.content) === null || _a === void 0 ? void 0 : _a.content) === null || _b === void 0 ? void 0 : _b.length) === 1 &&
                 isHighlightMarkerNode(slice.content.content[0])) {
                 // don't track highlight marker nodes
                 return;
             }
+            let [steps, startPos] = trackReplaceStep(step, oldState, newTr, emptyAttrs);
+            if (steps.length === 1) {
+                const step = steps[0]; // eslint-disable-line @typescript-eslint/no-explicit-any
+                if (isHighlightMarkerNode((step === null || step === void 0 ? void 0 : step.node) || ((_d = (_c = step === null || step === void 0 ? void 0 : step.slice) === null || _c === void 0 ? void 0 : _c.content) === null || _d === void 0 ? void 0 : _d.content[0]))) {
+                    // don't track deleted highlight marker nodes
+                    return;
+                }
+            }
             log.info('CHANGES: ', steps);
             // deleted and merged really...
             const deleted = steps.filter((s) => s.type !== 'insert-slice');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@manuscripts/track-changes-plugin",
-  "version": "0.4.4-LEAN-1839",
+  "version": "0.4.4-LEAN-1839-v3",
   "author": "Atypon Systems LLC",
   "license": "Apache-2.0",
   "homepage": "https://github.com/Atypon-OpenSource/manuscripts-quarterback/tree/main/quarterback-packages/track-changes-plugin",