@magic-marker/prosemirror-suggest-changes 0.3.3-wrap-unwrap.26 → 0.3.3-wrap-unwrap.28

package/dist/__tests__/playwrightPage.d.ts

@@ -56,4 +56,5 @@ export declare class EditorPage {
     pressKeyMultiple(key: string, count: number, opts?: {
         waitForSelectionChange?: boolean;
     }): Promise<void>;
+    setNextNodeId(nextNodeId: number): Promise<void>;
 }

package/dist/features/startToStartTextblockDeletion/index.d.ts

@@ -0,0 +1,6 @@
+import { type Selection } from "prosemirror-state";
+import { type ReplaceStep, type Step } from "prosemirror-transform";
+export declare function adjustForStartToStartTextblockDeletion(selection: Selection, step: ReplaceStep, prevSteps: Step[]): {
+    from: number;
+    to: number;
+};

package/dist/features/startToStartTextblockDeletion/index.js

@@ -0,0 +1,54 @@
+import { TextSelection } from "prosemirror-state";
+export function adjustForStartToStartTextblockDeletion(selection, step, prevSteps) {
+    if (prevSteps.length > 0) return {
+        from: step.from,
+        to: step.to
+    };
+    if (!(selection instanceof TextSelection)) {
+        return {
+            from: step.from,
+            to: step.to
+        };
+    }
+    if (selection.empty || step.slice.size !== 0) {
+        return {
+            from: step.from,
+            to: step.to
+        };
+    }
+    const { $from, $to } = selection;
+    if (!$from.parent.isTextblock || !$to.parent.isTextblock) {
+        return {
+            from: step.from,
+            to: step.to
+        };
+    }
+    if ($from.parentOffset !== 0 || $to.parentOffset !== 0) {
+        return {
+            from: step.from,
+            to: step.to
+        };
+    }
+    if ($from.start() === $to.start()) return {
+        from: step.from,
+        to: step.to
+    };
+    if (step.from !== $from.before($from.depth)) {
+        return {
+            from: step.from,
+            to: step.to
+        };
+    }
+    if (step.to !== $to.before($to.depth)) {
+        return {
+            from: step.from,
+            to: step.to
+        };
+    }
+    // The step boundary is moved before the right textblock, but the selection
+    // boundary is the user-visible end of the deleted text range.
+    return {
+        from: step.from,
+        to: selection.to
+    };
+}

package/dist/features/transactionShaping/tipTapParagraphIntoListJoin/handleTipTapParagraphIntoListJoin.js

@@ -1,9 +1,6 @@
 import { EditorState } from "prosemirror-state";
 import { preserveTransactionData, transformToSuggestionTransaction } from "../../../transformToSuggestionTransaction.js";
-import { getSuggestionMarks } from "../../../utils.js";
-import { getNodeId } from "../../wrapUnwrap/getNodeId.js";
 import { suggestStructureChanges } from "../../wrapUnwrap/structureChangesPlugin.js";
-import { guardStructureMarkAttrs } from "../../wrapUnwrap/types.js";
 import { detectSpecialTransactionShape } from "../detectSpecialTransactionShape.js";
 // handle the specific TipTap pattern when backspacing from a paragraph into a list above
 // causes a transaction with 3 steps
@@ -16,8 +13,6 @@ export function handleTipTapParagraphIntoListJoin(args) {
     if (!args.structuralContextPaths || !args.ensureUniqueNodeIds) return null;
     const docBefore = args.transaction.docs[0];
     if (!docBefore) return null;
-    const movedNodeId = getNodeId(shape.movedNode);
-    if (!movedNodeId) return null;
     const trackedTransaction = args.state.tr;
     try {
         trackedTransaction.step(shape.deleteStep);
@@ -32,7 +27,7 @@ export function handleTipTapParagraphIntoListJoin(args) {
         trackedTransaction.step(step);
     });
     const structureChangesResult = suggestStructureChanges(docBefore, uniqueNodeIdsTransform.doc, args.structuralContextPaths, args.generateId);
-    if (!structureChangesResult.handled || !hasMoveStructureMark(structureChangesResult, movedNodeId)) {
+    if (!structureChangesResult.handled) {
         return null;
     }
     structureChangesResult.transform.steps.forEach((step)=>{
@@ -64,17 +59,3 @@ export function handleTipTapParagraphIntoListJoin(args) {
 function isTipTapParagraphIntoListJoinShape(shape) {
     return shape?.type === "tipTapParagraphIntoListJoin";
 }
-function hasMoveStructureMark(structureChangesResult, movedNodeId) {
-    const { structure } = getSuggestionMarks(structureChangesResult.transform.doc.type.schema);
-    let found = false;
-    structureChangesResult.transform.doc.descendants((node)=>{
-        if (getNodeId(node) !== movedNodeId) return true;
-        found = node.marks.some((mark)=>{
-            if (mark.type !== structure) return false;
-            if (!guardStructureMarkAttrs(mark.attrs)) return false;
-            return mark.attrs.data.op.op === "move";
-        });
-        return !found;
-    });
-    return found;
-}

package/dist/replaceStep.js

@@ -4,6 +4,7 @@ import { rebasePos } from "./rebasePos.js";
 import { getSuggestionMarks } from "./utils.js";
 import { joinBlocks } from "./features/joinBlocks/index.js";
 import { collapseZWSPNodes, findJoinMark, joinNodesAndMarkJoinPoints, removeZWSPDeletions } from "./features/joinOnDelete/index.js";
+import { adjustForStartToStartTextblockDeletion } from "./features/startToStartTextblockDeletion/index.js";
 /**
  * Transform a replace step into its equivalent tracked steps.
  *
@@ -35,17 +36,18 @@ import { collapseZWSPNodes, findJoinMark, joinNodesAndMarkJoinPoints, removeZWSP
  * zero-width spaces will be removed.
  */ export function suggestReplaceStep(trackedTransaction, state, doc, step, prevSteps, suggestionId) {
     const { deletion, insertion } = getSuggestionMarks(state.schema);
+    const semanticStep = adjustForStartToStartTextblockDeletion(state.selection, step, prevSteps);
     // Check for insertion and deletion marks directly
     // adjacent to this step's boundaries. If they exist,
     // we'll use their ids, rather than producing a new one
-    const nodeBefore = doc.resolve(step.from).nodeBefore;
+    const nodeBefore = doc.resolve(semanticStep.from).nodeBefore;
     const markBefore = nodeBefore?.marks.find((mark)=>mark.type === deletion || mark.type === insertion) ?? null;
-    const nodeAfter = doc.resolve(step.to).nodeAfter;
+    const nodeAfter = doc.resolve(semanticStep.to).nodeAfter;
     const markAfter = nodeAfter?.marks.find((mark)=>mark.type === deletion || mark.type === insertion) ?? null;
     let markId = markBefore?.attrs["id"] ?? markAfter?.attrs["id"] ?? suggestionId;
     // Rebase this step's boundaries onto the newest doc
-    let stepFrom = rebasePos(step.from, prevSteps, trackedTransaction.steps);
-    let stepTo = rebasePos(step.to, prevSteps, trackedTransaction.steps);
+    let stepFrom = rebasePos(semanticStep.from, prevSteps, trackedTransaction.steps);
+    let stepTo = rebasePos(semanticStep.to, prevSteps, trackedTransaction.steps);
     if (state.selection.empty && stepFrom !== stepTo) {
         trackedTransaction.setSelection(TextSelection.near(trackedTransaction.doc.resolve(stepFrom)));
     }
@@ -58,8 +60,8 @@ import { collapseZWSPNodes, findJoinMark, joinNodesAndMarkJoinPoints, removeZWSP
     }
     // Update the step boundaries, since we may have just changed
     // the document
-    stepFrom = rebasePos(step.from, prevSteps, trackedTransaction.steps);
-    stepTo = rebasePos(step.to, prevSteps, trackedTransaction.steps);
+    stepFrom = rebasePos(semanticStep.from, prevSteps, trackedTransaction.steps);
+    stepTo = rebasePos(semanticStep.to, prevSteps, trackedTransaction.steps);
     // Re-resolve nodeAfter and markAfter if we did a block join
     // The original values are stale after joinBlocks modifies the document
     let nodeAfterResolved = nodeAfter;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@magic-marker/prosemirror-suggest-changes",
-  "version": "0.3.3-wrap-unwrap.26",
+  "version": "0.3.3-wrap-unwrap.28",
   "license": "MIT",
   "type": "module",
   "module": "dist/index.js",
@@ -55,17 +55,17 @@
     "markdown-toc": "^1.2.0",
     "playwright": "^1.56.0",
     "prettier": "^3.5.3",
-    "prosemirror-commands": "^1.7.0",
-    "prosemirror-history": "^1.5.0",
-    "prosemirror-inputrules": "^1.4.0",
-    "prosemirror-keymap": "^1.2.2",
-    "prosemirror-model": "^1.24.1",
-    "prosemirror-schema-basic": "^1.2.3",
-    "prosemirror-schema-list": "^1.5.0",
-    "prosemirror-state": "^1.4.3",
-    "prosemirror-test-builder": "^1.1.1",
-    "prosemirror-transform": "^1.10.2",
-    "prosemirror-view": "^1.38.0",
+    "prosemirror-commands": "1.7.1",
+    "prosemirror-history": "1.5.0",
+    "prosemirror-inputrules": "1.5.1",
+    "prosemirror-keymap": "1.2.3",
+    "prosemirror-model": "1.25.4",
+    "prosemirror-schema-basic": "1.2.4",
+    "prosemirror-schema-list": "1.5.1",
+    "prosemirror-state": "1.4.4",
+    "prosemirror-test-builder": "1.1.1",
+    "prosemirror-transform": "1.12.0",
+    "prosemirror-view": "1.41.9",
     "remark-parse": "^11.0.0",
     "rimraf": "^6.0.1",
     "typescript": "^5.8.2",
@@ -79,5 +79,17 @@
     "prosemirror-state": "^1.0.0",
     "prosemirror-transform": "^1.0.0",
     "prosemirror-view": "^1.0.0"
+  },
+  "resolutions": {
+    "prosemirror-model": "1.25.4",
+    "prosemirror-state": "1.4.4",
+    "prosemirror-transform": "1.12.0",
+    "prosemirror-view": "1.41.9",
+    "prosemirror-keymap": "1.2.3",
+    "prosemirror-commands": "1.7.1",
+    "prosemirror-schema-basic": "1.2.4",
+    "prosemirror-schema-list": "1.5.1",
+    "prosemirror-history": "1.5.0",
+    "prosemirror-inputrules": "1.5.1"
   }
 }