@ai-matrx/office 0.8.36 → 0.8.38

package/dist/univer-collab.cjs

@@ -1225,29 +1225,38 @@ function undoStacksOf(service) {
   if (!s || !(s._undoStacks instanceof Map) || !(s._redoStacks instanceof Map)) return null;
   return { undo: s._undoStacks, redo: s._redoStacks };
 }
-function rebaseUndoStack(items, remote) {
+function threadThrough(list, remote) {
+  if (!list?.length || isNoopDocsOp(remote)) return { next: null, after: remote };
+  let R = remote;
+  let changed = false;
+  const next = list.map((m) => {
+    if (m.id !== RICH_TEXT_EDITING || isNoopDocsOp(R)) return m;
+    const own = m.params.actions;
+    if (touchesPageSetup(own) && touchesPageSetup(R)) throw new Error("page setup");
+    const x = transformDocsOp(own, R, "left");
+    if (isNoopDocsOp(x.op)) throw new Error("emptied");
+    R = transformDocsOp(R, own, "right").op;
+    changed = true;
+    const params = { ...m.params, actions: x.op };
+    delete params.textRanges;
+    delete params.layoutState;
+    return { ...m, params };
+  });
+  return { next: changed ? next : null, after: R };
+}
+function rebaseUndoStack(items, remote, stack) {
+  if (isNoopDocsOp(remote)) return 0;
+  const first = stack === "undo" ? "undoMutations" : "redoMutations";
+  const second = stack === "undo" ? "redoMutations" : "undoMutations";
   let dropped = 0;
   for (let i = items.length - 1; i >= 0; i--) {
     const item = items[i];
-    let ok3 = true;
-    for (const list of [item.undoMutations, item.redoMutations]) {
-      for (const m of list ?? []) {
-        if (m.id !== RICH_TEXT_EDITING) continue;
-        try {
-          if (touchesPageSetup(m.params.actions) && touchesPageSetup(remote)) throw new Error("page setup");
-          const r = transformDocsOp(m.params.actions, remote, "left");
-          if (isNoopDocsOp(r.op)) ok3 = false;
-          else {
-            m.params = { ...m.params, actions: r.op };
-            delete m.params.textRanges;
-            delete m.params.layoutState;
-          }
-        } catch {
-          ok3 = false;
-        }
-      }
-    }
-    if (!ok3) {
+    try {
+      const a = threadThrough(item[first], remote);
+      const b = threadThrough(item[second], a.after);
+      if (a.next) item[first] = a.next;
+      if (b.next) item[second] = b.next;
+    } catch {
       items.splice(i, 1);
       dropped++;
     }
@@ -1268,6 +1277,7 @@ function openDocsOtRoom(o) {
     }
   };
   const applyNow = (op) => {
+    for (const t of port.undoTargets()) t.flushHistory?.();
     for (const b of branches(op)) {
       const params = { segmentId: segmentOf(b), actions: b };
       port.applyRemote(RICH_TEXT_EDITING, params);
@@ -1275,10 +1285,10 @@ function openDocsOtRoom(o) {
     for (const t of port.undoTargets()) {
       const stacks = undoStacksOf(t.service);
       if (!stacks) continue;
-      for (const m of [stacks.undo, stacks.redo]) {
-        const items = m.get(t.unitId);
-        if (items?.length) rebaseUndoStack(items, op);
-      }
+      const undo = stacks.undo.get(t.unitId);
+      if (undo?.length) rebaseUndoStack(undo, op, "undo");
+      const redo = stacks.redo.get(t.unitId);
+      if (redo?.length) rebaseUndoStack(redo, op, "redo");
     }
     port.markDirty();
   };
@@ -1898,10 +1908,13 @@ var SHEET_MUTATIONS = {
   "data-validation.mutation.updateRule": covered("dv", "ruleId", "payload", "source", "derived"),
   "data-validation.mutation.removeRule": covered("dv", "ruleId", "source", "derived"),
   "sheet.mutation.set-filter-range": covered("filter", "range", "derived"),
-  // C5f: Univer's cell notes, keyed by note id, anchored to one cell (their transform: comment-anchor-transform.ts)
-  "sheet.mutation.update-note": covered("note", "sheetId", "row", "col", "note", "silent", "derived"),
-  "sheet.mutation.remove-note": covered("note", "sheetId", "noteId", "row", "col", "silent", "derived"),
-  "sheet.mutation.update-note-position": covered("note", "sheetId", "noteId", "row", "col", "newPosition", "silent", "derived"),
+  // C5f: Univer's cell notes. Their transform is built (comment-anchor-transform.ts `transformCellNote`, pairwise and
+  // chained fuzz green) but the session fuzz still diverges on note cells (seed 1015, 6 seats: a note at column 5 on
+  // one seat, 8 on another), so they stay barriers until that seed is green. Covered form, for the next lane:
+  //   covered("note", "sheetId", "row", "col", "note" | "noteId", "newPosition", "silent", "derived")
+  "sheet.mutation.update-note": barrier("C5c", "cell note: the session fuzz diverges on note cells (seed 1015)"),
+  "sheet.mutation.remove-note": barrier("C5c", "cell note: the session fuzz diverges on note cells (seed 1015)"),
+  "sheet.mutation.update-note-position": barrier("C5c", "cell note: the session fuzz diverges on note cells (seed 1015)"),
   "sheet.mutation.set-filter-criteria": covered("filter", "col", "criteria", "reCalc", "derived"),
   "sheet.mutation.remove-filter": covered("filter", "derived"),
   "sheet.mutation.add-table": barrier("C5c", RESOURCE),