docx-diff-editor 1.0.46 → 1.0.48

package/dist/index.d.mts

@@ -29,6 +29,10 @@ type DocxContent = File | ProseMirrorJSON | string;
 interface DiffSegment {
     type: 'equal' | 'insert' | 'delete';
     text: string;
+    /** Position in docA where this segment starts (for equal/delete segments) */
+    posA?: number;
+    /** Position in docB where this segment starts (for equal/insert segments) */
+    posB?: number;
 }
 /**
  * A format change on unchanged text
@@ -54,6 +58,8 @@ interface DiffResult {
     textB: string;
     /** Human-readable summary */
     summary: string[];
+    /** Text spans from docB with marks (for mark preservation during merge) */
+    spansB?: TextSpan[];
 }
 /**
  * Type of structural change
@@ -365,6 +371,15 @@ interface DocxDiffEditorRef {
     /** Parse HTML string to ProseMirror JSON (uses hidden SuperDoc instance) */
     parseHtml(html: string): Promise<ProseMirrorJSON>;
 }
+/**
+ * Text span with position and marks (used in diffing)
+ */
+interface TextSpan {
+    text: string;
+    from: number;
+    to: number;
+    marks: ProseMirrorMark[];
+}
 
 /**
  * DocxDiffEditor Component
@@ -454,6 +469,10 @@ declare function parseDocxFile(file: File, SuperDoc: SuperDocConstructor): Promi
 /**
  * Diff two ProseMirror JSON documents at the character level.
  * Detects both text changes and formatting changes.
+ *
+ * Now also tracks positions in both documents for mark preservation:
+ * - posA: position in docA (for equal/delete segments)
+ * - posB: position in docB (for equal/insert segments)
  */
 declare function diffDocuments(docA: ProseMirrorJSON, docB: ProseMirrorJSON): DiffResult;
 
@@ -494,6 +513,11 @@ declare function createTrackInsertMark(author?: TrackChangeAuthor, id?: string):
 declare function createTrackDeleteMark(author?: TrackChangeAuthor, id?: string): ProseMirrorMark;
 /**
  * Create a trackFormat mark.
+ *
+ * Note: SuperDoc's parseFormatList requires all marks in before/after arrays
+ * to have both `type` and `attrs` properties. Marks without `attrs` get filtered out,
+ * causing empty values in track change bubbles. We normalize marks here to ensure
+ * all have at least an empty `attrs` object.
  */
 declare function createTrackFormatMark(before: ProseMirrorMark[], after: ProseMirrorMark[], author?: TrackChangeAuthor): ProseMirrorMark;
 

package/dist/index.d.ts

@@ -29,6 +29,10 @@ type DocxContent = File | ProseMirrorJSON | string;
 interface DiffSegment {
     type: 'equal' | 'insert' | 'delete';
     text: string;
+    /** Position in docA where this segment starts (for equal/delete segments) */
+    posA?: number;
+    /** Position in docB where this segment starts (for equal/insert segments) */
+    posB?: number;
 }
 /**
  * A format change on unchanged text
@@ -54,6 +58,8 @@ interface DiffResult {
     textB: string;
     /** Human-readable summary */
     summary: string[];
+    /** Text spans from docB with marks (for mark preservation during merge) */
+    spansB?: TextSpan[];
 }
 /**
  * Type of structural change
@@ -365,6 +371,15 @@ interface DocxDiffEditorRef {
     /** Parse HTML string to ProseMirror JSON (uses hidden SuperDoc instance) */
     parseHtml(html: string): Promise<ProseMirrorJSON>;
 }
+/**
+ * Text span with position and marks (used in diffing)
+ */
+interface TextSpan {
+    text: string;
+    from: number;
+    to: number;
+    marks: ProseMirrorMark[];
+}
 
 /**
  * DocxDiffEditor Component
@@ -454,6 +469,10 @@ declare function parseDocxFile(file: File, SuperDoc: SuperDocConstructor): Promi
 /**
  * Diff two ProseMirror JSON documents at the character level.
  * Detects both text changes and formatting changes.
+ *
+ * Now also tracks positions in both documents for mark preservation:
+ * - posA: position in docA (for equal/delete segments)
+ * - posB: position in docB (for equal/insert segments)
  */
 declare function diffDocuments(docA: ProseMirrorJSON, docB: ProseMirrorJSON): DiffResult;
 
@@ -494,6 +513,11 @@ declare function createTrackInsertMark(author?: TrackChangeAuthor, id?: string):
 declare function createTrackDeleteMark(author?: TrackChangeAuthor, id?: string): ProseMirrorMark;
 /**
  * Create a trackFormat mark.
+ *
+ * Note: SuperDoc's parseFormatList requires all marks in before/after arrays
+ * to have both `type` and `attrs` properties. Marks without `attrs` get filtered out,
+ * causing empty values in track change bubbles. We normalize marks here to ensure
+ * all have at least an empty `attrs` object.
  */
 declare function createTrackFormatMark(before: ProseMirrorMark[], after: ProseMirrorMark[], author?: TrackChangeAuthor): ProseMirrorMark;
 

package/dist/index.js

@@ -1002,14 +1002,20 @@ function diffDocuments(docA, docB) {
   const segments = [];
   let insertCount = 0;
   let deleteCount = 0;
+  let posA = 0;
+  let posB = 0;
   for (const [op, text] of diffs) {
     if (op === DIFF_EQUAL) {
-      segments.push({ type: "equal", text });
+      segments.push({ type: "equal", text, posA, posB });
+      posA += text.length;
+      posB += text.length;
     } else if (op === DIFF_INSERT) {
-      segments.push({ type: "insert", text });
+      segments.push({ type: "insert", text, posB });
+      posB += text.length;
       insertCount++;
     } else if (op === DIFF_DELETE) {
-      segments.push({ type: "delete", text });
+      segments.push({ type: "delete", text, posA });
+      posA += text.length;
       deleteCount++;
     }
   }
@@ -1034,7 +1040,9 @@ function diffDocuments(docA, docB) {
     formatChanges,
     textA,
     textB,
-    summary
+    summary,
+    spansB
+    // Include docB spans for mark preservation during merge
   };
 }
 
@@ -1295,6 +1303,34 @@ function groupReplacements(changes) {
   }
   return result;
 }
+function ensureValidCssColor(color) {
+  if (typeof color !== "string" || !color) {
+    return void 0;
+  }
+  if (/^[0-9a-fA-F]{6}$/.test(color)) {
+    return `#${color}`;
+  }
+  if (/^[0-9a-fA-F]{3}$/.test(color)) {
+    return `#${color}`;
+  }
+  return color;
+}
+function normalizeMark(mark) {
+  const attrs = { ...mark.attrs || {} };
+  if (attrs.color !== void 0) {
+    attrs.color = ensureValidCssColor(attrs.color);
+  }
+  return {
+    type: mark.type,
+    attrs
+  };
+}
+function normalizeMarks(marks) {
+  return marks.map(normalizeMark);
+}
+function normalizeMarksForRendering(marks) {
+  return normalizeMarks(marks);
+}
 function createTrackInsertMark(author = DEFAULT_AUTHOR, id) {
   return {
     type: "trackInsert",
@@ -1320,6 +1356,8 @@ function createTrackDeleteMark(author = DEFAULT_AUTHOR, id) {
   };
 }
 function createTrackFormatMark(before, after, author = DEFAULT_AUTHOR) {
+  const normalizedBefore = normalizeMarks(before);
+  const normalizedAfter = normalizeMarks(after);
   return {
     type: "trackFormat",
     attrs: {
@@ -1328,8 +1366,8 @@ function createTrackFormatMark(before, after, author = DEFAULT_AUTHOR) {
       authorEmail: author.email,
       authorImage: "",
       date: (/* @__PURE__ */ new Date()).toISOString(),
-      before,
-      after
+      before: normalizedBefore,
+      after: normalizedAfter
     }
   };
 }
@@ -1516,6 +1554,70 @@ function alignListItems(listA, listB, listPathA, listPathB) {
 function cloneNode(node) {
   return JSON.parse(JSON.stringify(node));
 }
+function getMarkSpansForRange(spansB, start, end) {
+  const result = [];
+  for (const span of spansB) {
+    if (span.to > start && span.from < end) {
+      const overlapStart = Math.max(span.from, start);
+      const overlapEnd = Math.min(span.to, end);
+      result.push({
+        relStart: overlapStart - start,
+        relEnd: overlapEnd - start,
+        marks: span.marks || []
+      });
+    }
+  }
+  return result;
+}
+function createInsertedTextNodes(text, posB, spansB, author, replacementId) {
+  const result = [];
+  const trackMark = createTrackInsertMark(author, replacementId);
+  if (posB === void 0 || spansB.length === 0) {
+    return [{
+      type: "text",
+      text,
+      marks: [trackMark]
+    }];
+  }
+  const markSpans = getMarkSpansForRange(spansB, posB, posB + text.length);
+  if (markSpans.length === 0) {
+    return [{
+      type: "text",
+      text,
+      marks: [trackMark]
+    }];
+  }
+  markSpans.sort((a, b) => a.relStart - b.relStart);
+  let processedUpTo = 0;
+  for (const span of markSpans) {
+    if (span.relStart > processedUpTo) {
+      result.push({
+        type: "text",
+        text: text.substring(processedUpTo, span.relStart),
+        marks: [trackMark]
+      });
+    }
+    if (span.relEnd > span.relStart) {
+      const spanText = text.substring(span.relStart, span.relEnd);
+      const normalizedSpanMarks = normalizeMarksForRendering(span.marks);
+      const marks = [...normalizedSpanMarks, trackMark];
+      result.push({
+        type: "text",
+        text: spanText,
+        marks
+      });
+      processedUpTo = span.relEnd;
+    }
+  }
+  if (processedUpTo < text.length) {
+    result.push({
+      type: "text",
+      text: text.substring(processedUpTo),
+      marks: [trackMark]
+    });
+  }
+  return result;
+}
 function mergeDocuments(docA, docB, diffResult, author = DEFAULT_AUTHOR) {
   const merged = cloneNode(docA);
   const charStates = [];
@@ -1550,17 +1652,22 @@ function mergeDocuments(docA, docB, diffResult, author = DEFAULT_AUTHOR) {
         insertions.push({
           afterOffset: docAOffset,
           text: nextSegment.text,
-          replacementId
+          replacementId,
+          posB: nextSegment.posB
+          // Capture docB position for mark lookup
         });
         segIdx++;
       }
     } else if (segment.type === "insert") {
       insertions.push({
         afterOffset: docAOffset,
-        text: segment.text
+        text: segment.text,
+        posB: segment.posB
+        // Capture docB position for mark lookup
       });
     }
   }
+  const spansB = diffResult.spansB || [];
   function transformNode(node, nodeOffset, path) {
     if (node.type === "text" && node.text) {
       const text = node.text;
@@ -1571,11 +1678,14 @@ function mergeDocuments(docA, docB, diffResult, author = DEFAULT_AUTHOR) {
         const charState = charStates[charOffset] || { type: "equal" };
         const insertionsHere = insertions.filter((ins) => ins.afterOffset === charOffset);
         for (const ins of insertionsHere) {
-          result.push({
-            type: "text",
-            text: ins.text,
-            marks: [...node.marks || [], createTrackInsertMark(author, ins.replacementId)]
-          });
+          const insertedNodes = createInsertedTextNodes(
+            ins.text,
+            ins.posB,
+            spansB,
+            author,
+            ins.replacementId
+          );
+          result.push(...insertedNodes);
         }
         const currentFormatChange = getFormatChangeAt(nodeOffset + i);
         let j = i + 1;
@@ -1598,7 +1708,8 @@ function mergeDocuments(docA, docB, diffResult, author = DEFAULT_AUTHOR) {
               currentFormatChange.after,
               author
             );
-            marks = [...currentFormatChange.after, trackFormatMark];
+            const normalizedAfterMarks = normalizeMarksForRendering(currentFormatChange.after);
+            marks = [...normalizedAfterMarks, trackFormatMark];
           }
         }
         result.push({
@@ -1611,11 +1722,14 @@ function mergeDocuments(docA, docB, diffResult, author = DEFAULT_AUTHOR) {
       const endOffset = nodeOffset + text.length;
       const endInsertions = insertions.filter((ins) => ins.afterOffset === endOffset);
       for (const ins of endInsertions) {
-        result.push({
-          type: "text",
-          text: ins.text,
-          marks: [...node.marks || [], createTrackInsertMark(author, ins.replacementId)]
-        });
+        const insertedNodes = createInsertedTextNodes(
+          ins.text,
+          ins.posB,
+          spansB,
+          author,
+          ins.replacementId
+        );
+        result.push(...insertedNodes);
       }
       insertions = insertions.filter(
         (ins) => ins.afterOffset < nodeOffset || ins.afterOffset > endOffset
@@ -1650,18 +1764,19 @@ function mergeDocuments(docA, docB, diffResult, author = DEFAULT_AUTHOR) {
   }
   if (insertions.length > 0) {
     for (const ins of insertions) {
+      const insertedNodes = createInsertedTextNodes(
+        ins.text,
+        ins.posB,
+        spansB,
+        author,
+        ins.replacementId
+      );
       const insertNode = {
         type: "paragraph",
         content: [
           {
             type: "run",
-            content: [
-              {
-                type: "text",
-                text: ins.text,
-                marks: [createTrackInsertMark(author, ins.replacementId)]
-              }
-            ]
+            content: insertedNodes
           }
         ]
       };
@@ -2183,9 +2298,10 @@ function cloneNode2(node) {
 }
 function markAllTextAsInserted(node, sharedId, author) {
   if (node.type === "text") {
+    const existingMarks = normalizeMarksForRendering(node.marks || []);
     return {
       ...node,
-      marks: [...node.marks || [], createTrackInsertMark(author, sharedId)]
+      marks: [...existingMarks, createTrackInsertMark(author, sharedId)]
     };
   }
   if (node.content && Array.isArray(node.content)) {
@@ -2200,9 +2316,10 @@ function markAllTextAsInserted(node, sharedId, author) {
 }
 function markAllTextAsDeleted(node, sharedId, author) {
   if (node.type === "text") {
+    const existingMarks = normalizeMarksForRendering(node.marks || []);
     return {
       ...node,
-      marks: [...node.marks || [], createTrackDeleteMark(author, sharedId)]
+      marks: [...existingMarks, createTrackDeleteMark(author, sharedId)]
     };
   }
   if (node.content && Array.isArray(node.content)) {
@@ -3463,9 +3580,10 @@ var DocxDiffEditor_default = DocxDiffEditor;
 init_nodeFingerprint();
 function markAllTextAsInserted2(node, sharedId, author) {
   if (node.type === "text") {
+    const existingMarks = normalizeMarksForRendering(node.marks || []);
     return {
       ...node,
-      marks: [...node.marks || [], createTrackInsertMark(author, sharedId)]
+      marks: [...existingMarks, createTrackInsertMark(author, sharedId)]
     };
   }
   if (node.content && Array.isArray(node.content)) {
@@ -3480,9 +3598,10 @@ function markAllTextAsInserted2(node, sharedId, author) {
 }
 function markAllTextAsDeleted2(node, sharedId, author) {
   if (node.type === "text") {
+    const existingMarks = normalizeMarksForRendering(node.marks || []);
     return {
       ...node,
-      marks: [...node.marks || [], createTrackDeleteMark(author, sharedId)]
+      marks: [...existingMarks, createTrackDeleteMark(author, sharedId)]
     };
   }
   if (node.content && Array.isArray(node.content)) {