@magic-marker/prosemirror-suggest-changes 0.3.3-wrap-unwrap.22 → 0.3.3-wrap-unwrap.24

package/dist/ensureSelectionPlugin.js

@@ -1,7 +1,7 @@
 import { Plugin, PluginKey, TextSelection } from "prosemirror-state";
 import { getSuggestionMarks } from "./utils.js";
 import { ZWSP } from "./constants.js";
-const TRACE_ENABLED = true;
+const TRACE_ENABLED = false;
 function trace(...args) {
     // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition
     if (!TRACE_ENABLED) return;

package/dist/features/wrapUnwrap/__tests__/listStructureTextEdits.playwright.test.d.ts

@@ -0,0 +1 @@
+export {};

package/dist/features/wrapUnwrap/buildMaterializedPaths.js

@@ -1,6 +1,6 @@
 import { getNodeId } from "./getNodeId.js";
 import { DOC_NODE_ID } from "./constants.js";
-const TRACE_ENABLED = true;
+const TRACE_ENABLED = false;
 function trace(...args) {
     // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition
     if (!TRACE_ENABLED) return;

package/dist/features/wrapUnwrap/revert/deleteNodeUpwards.js

@@ -1,5 +1,5 @@
 import { getNodeId } from "../getNodeId.js";
-const TRACE_ENABLED = true;
+const TRACE_ENABLED = false;
 function trace(...args) {
     // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition
     if (!TRACE_ENABLED) return;

package/dist/features/wrapUnwrap/revert/revertStructureSuggestions.js

@@ -6,7 +6,7 @@ import { sameParentChain } from "../sameParentChain.js";
 import { buildMaterializedPaths } from "../buildMaterializedPaths.js";
 import { revertAddOp } from "./revertAddOp.js";
 import { revertMoveOp } from "./revertMoveOp.js";
-const TRACE_ENABLED = true;
+const TRACE_ENABLED = false;
 function trace(...args) {
     // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition
     if (!TRACE_ENABLED) return;

package/dist/features/wrapUnwrap/structureChangesPlugin.js

@@ -8,7 +8,7 @@ import { Transform } from "prosemirror-transform";
 import { isSuggestChangesEnabled, suggestChangesKey } from "../../plugin.js";
 import { buildMaterializedPaths } from "./buildMaterializedPaths.js";
 import { sameParentChain } from "./sameParentChain.js";
-const TRACE_ENABLED = true;
+const TRACE_ENABLED = false;
 function trace(...args) {
     // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition
     if (!TRACE_ENABLED) return;

package/dist/features/wrapUnwrap/types.d.ts

@@ -9,12 +9,19 @@ interface NodeParent {
     childSiblingIds: [string | null, string | null];
     childIndex: number;
 }
-export interface StructureMarkAttrs {
+interface StructureMoveMarkAttrs {
     id: SuggestionId;
     data: {
-        op: Op;
+        op: MoveOp;
     };
 }
+interface StructureAddMarkAttrs {
+    id: SuggestionId;
+    data: {
+        op: AddOp;
+    };
+}
+export type StructureMarkAttrs = StructureMoveMarkAttrs | StructureAddMarkAttrs;
 export type StructuralContextPath = readonly [string, ...string[]];
 export interface DocParent extends Omit<NodeParent, "nodeId"> {
     nodeId: typeof DOC_NODE_ID;
@@ -37,9 +44,11 @@ export type MaterializedPaths = Map<string, {
 }>;
 export declare function guardDocParent(parent: Parent | undefined): parent is DocParent;
 export declare function guardStructureMarkAttrs(attrs: Attrs): attrs is StructureMarkAttrs;
+export declare function guardStructureMoveMarkAttrs(attrs: Attrs): attrs is StructureMoveMarkAttrs;
+export declare function guardStructureAddMarkAttrs(attrs: Attrs): attrs is StructureAddMarkAttrs;
 export interface StructureMarkObject {
     type: "structure";
     attrs: StructureMarkAttrs;
 }
-export declare function isStructureMarkObject(mark: unknown): mark is StructureMarkObject;
+export declare function guardStructureMarkObject(mark: unknown): mark is StructureMarkObject;
 export {};

package/dist/features/wrapUnwrap/types.js

@@ -9,7 +9,13 @@ export function guardStructureMarkAttrs(attrs) {
     if (!("op" in data)) return false;
     return true;
 }
-export function isStructureMarkObject(mark) {
+export function guardStructureMoveMarkAttrs(attrs) {
+    return guardStructureMarkAttrs(attrs) && attrs.data.op.op === "move";
+}
+export function guardStructureAddMarkAttrs(attrs) {
+    return guardStructureMarkAttrs(attrs) && attrs.data.op.op === "add";
+}
+export function guardStructureMarkObject(mark) {
     if (mark === null || typeof mark !== "object") return false;
     if (!("type" in mark) || mark.type !== "structure") return false;
     if (!("attrs" in mark)) return false;

package/dist/features/wrapUnwrap/uniqueNodeIdsPlugin.js

@@ -6,22 +6,27 @@ import { Transform } from "prosemirror-transform";
 // (also checks and fix duplicates that inevitably appear)
 export const uniqueNodeIdsPluginKey = new PluginKey("@handlewithcare/prosemirror-suggest-changes-unique-node-ids");
 export const UNIQUE_NODE_IDS_PLUGIN_META = "unique-node-ids-plugin";
+const TRACE_ENABLED = false;
+function trace(...args) {
+    // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition
+    if (!TRACE_ENABLED) return;
+    console.log("[uniqueNodeIdsPlugin]", ...args);
+}
 export function uniqueNodeIdsPlugin({ attributeName, generateID }) {
     return new Plugin({
         key: uniqueNodeIdsPluginKey,
         appendTransaction (transactions, oldState, newState) {
-            console.log("uniqueNodeIdsPlugin.appendTransaction");
+            trace("appendTransaction");
             const pluginState = uniqueNodeIdsPluginKey.getState(newState);
             // do nothing if doc hasn't changed (but make sure it runs initially)
             const docChanged = transactions.some((transaction)=>transaction.docChanged);
             if (!docChanged && pluginState?.completedInitialRun) {
-                console.warn("uniqueNodeIdsPlugin", "doc not changed, skipping", [
+                trace("doc not changed, skipping", [
                     ...transactions
                 ]);
                 return;
             }
-            console.groupCollapsed("uniqueNodeIdsPlugin", "appendTransaction");
-            console.log("uniqueNodeIdsPlugin", "appendTransaction", [
+            trace("appendTransaction", [
                 ...transactions
             ]);
             const tr = newState.tr;
@@ -32,8 +37,7 @@ export function uniqueNodeIdsPlugin({ attributeName, generateID }) {
             transform.steps.forEach((step)=>{
                 tr.step(step);
             });
-            console.log("ensureUniqueNodeIdsPlugin", "tr steps", tr.steps);
-            console.groupEnd();
+            trace("tr steps", tr.steps);
             if (!tr.steps.length) return;
             tr.setMeta(uniqueNodeIdsPluginKey, UNIQUE_NODE_IDS_PLUGIN_META);
             return tr;
@@ -59,6 +63,8 @@ export function uniqueNodeIdsPlugin({ attributeName, generateID }) {
 export function ensureUniqueNodeIds(_transactions, _oldDoc, newDoc, options) {
     const tr = new Transform(newDoc);
     const nodeIds = new Set();
+    // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition
+    if (TRACE_ENABLED) console.groupCollapsed("ensureUniqueNodeIds");
     tr.doc.descendants((node, pos)=>{
         if (node.isText) return false;
         const nodeId = getNodeId(node);
@@ -75,7 +81,7 @@ export function ensureUniqueNodeIds(_transactions, _oldDoc, newDoc, options) {
                 ...node.attrs,
                 [options.attributeName]: id
             }, node.marks);
-            console.log("ensureUniqueNodeIds", "fixed duplicate id", id, "for node", node.type.name, "at pos", pos, {
+            trace("fixed duplicate id", id, "for node", node.type.name, "at pos", pos, {
                 was: nodeId,
                 is: id
             });
@@ -89,10 +95,12 @@ export function ensureUniqueNodeIds(_transactions, _oldDoc, newDoc, options) {
                 ...node.attrs,
                 [options.attributeName]: id
             }, node.marks);
-            console.log("ensureUniqueNodeIds", "set unique id", id, "for node", node.type.name, "at pos", pos);
+            trace("set unique id", id, "for node", node.type.name, "at pos", pos);
             return true;
         }
         return true;
     });
+    // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition
+    if (TRACE_ENABLED) console.groupEnd();
     return tr;
 }

package/dist/withSuggestChanges.js

@@ -4,7 +4,7 @@ import { getRequiredStructuralContextPaths, suggestStructureChanges } from "./fe
 import { handleSpecialTransactionShape } from "./features/transactionShaping/index.js";
 import { transformToSuggestionTransaction } from "./transformToSuggestionTransaction.js";
 export { transformToSuggestionTransaction } from "./transformToSuggestionTransaction.js";
-const TRACE_ENABLED = true;
+const TRACE_ENABLED = false;
 function trace(...args) {
     // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition
     if (!TRACE_ENABLED) return;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@magic-marker/prosemirror-suggest-changes",
-  "version": "0.3.3-wrap-unwrap.22",
+  "version": "0.3.3-wrap-unwrap.24",
   "license": "MIT",
   "type": "module",
   "module": "dist/index.js",