docx-diff-editor 1.0.51 → 1.0.52

package/dist/index.d.mts

@@ -516,8 +516,12 @@ declare function createTrackDeleteMark(author?: TrackChangeAuthor, id?: string):
  *
  * 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.
+ * causing empty values in track change bubbles.
+ *
+ * We use normalizeMarksForTrackFormat which:
+ * 1. Ensures all marks have `attrs` property (required by parseFormatList)
+ * 2. Cleans attrs to remove null/undefined/empty values (prevents "Set X to undefined" bubbles)
+ * 3. Normalizes color values to valid CSS format
  */
 declare function createTrackFormatMark(before: ProseMirrorMark[], after: ProseMirrorMark[], author?: TrackChangeAuthor): ProseMirrorMark;
 

package/dist/index.d.ts

@@ -516,8 +516,12 @@ declare function createTrackDeleteMark(author?: TrackChangeAuthor, id?: string):
  *
  * 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.
+ * causing empty values in track change bubbles.
+ *
+ * We use normalizeMarksForTrackFormat which:
+ * 1. Ensures all marks have `attrs` property (required by parseFormatList)
+ * 2. Cleans attrs to remove null/undefined/empty values (prevents "Set X to undefined" bubbles)
+ * 3. Normalizes color values to valid CSS format
  */
 declare function createTrackFormatMark(before: ProseMirrorMark[], after: ProseMirrorMark[], author?: TrackChangeAuthor): ProseMirrorMark;
 

package/dist/index.js

@@ -541,6 +541,18 @@ function ensureValidCssColor(color) {
 }
 
 // src/services/trackChangeInjector.ts
+function isMeaningfulValue(value) {
+  return value !== null && value !== void 0 && value !== "";
+}
+function cleanAttrs(attrs) {
+  const cleaned = {};
+  for (const [key, value] of Object.entries(attrs)) {
+    if (isMeaningfulValue(value)) {
+      cleaned[key] = value;
+    }
+  }
+  return cleaned;
+}
 function normalizeMark(mark) {
   const attrs = { ...mark.attrs || {} };
   if (attrs.color !== void 0) {
@@ -551,9 +563,23 @@ function normalizeMark(mark) {
     attrs
   };
 }
+function normalizeMarkForTrackFormat(mark) {
+  let attrs = { ...mark.attrs || {} };
+  if (attrs.color !== void 0) {
+    attrs.color = ensureValidCssColor(attrs.color);
+  }
+  attrs = cleanAttrs(attrs);
+  return {
+    type: mark.type,
+    attrs
+  };
+}
 function normalizeMarks(marks) {
   return marks.map(normalizeMark);
 }
+function normalizeMarksForTrackFormat(marks) {
+  return marks.map(normalizeMarkForTrackFormat);
+}
 function normalizeMarksForRendering(marks) {
   return normalizeMarks(marks);
 }
@@ -582,8 +608,8 @@ function createTrackDeleteMark(author = DEFAULT_AUTHOR, id) {
   };
 }
 function createTrackFormatMark(before, after, author = DEFAULT_AUTHOR) {
-  const normalizedBefore = normalizeMarks(before);
-  const normalizedAfter = normalizeMarks(after);
+  const normalizedBefore = normalizeMarksForTrackFormat(before);
+  const normalizedAfter = normalizeMarksForTrackFormat(after);
   return {
     type: "trackFormat",
     attrs: {