@ai-matrx/office 0.8.30 → 0.8.32

package/CHANGELOG.md

@@ -1,10 +1,21 @@
 # Changelog
 
-## 0.8.30
+## 0.8.32
 
 UNVERIFIED: this release has no Unreleased entry and the source comparison could not be made (the caller supplied no source comparison). Whether source changed is unknown; consumer action is unknown.
 Check the package's gitHead on npm against this version's gitHead: `git diff <previous gitHead> <this gitHead> -- apps/shared/office/src`.
 
+## 0.8.30
+
+SOURCE CHANGED WITH NO RELEASE NOTE. This release shipped source changes that nobody described in an Unreleased entry,
+so they may include breaking API changes. Consumer action is UNKNOWN: read the diff before upgrading.
+Diff: `git diff acce3dd470 149d6b9267 -- apps/shared/office/src`
+
+Changed shipped source files (3):
+- `apps/shared/office/src/docs/document-model.ts`
+- `apps/shared/office/src/sheets/comment-anchors.ts`
+- `apps/shared/office/src/univer/collab/sheets-ot/univer-port.ts`
+
 ## 0.8.29
 
 SOURCE CHANGED WITH NO RELEASE NOTE. This release shipped source changes that nobody described in an Unreleased entry,
@@ -101,9 +112,9 @@ Changed shipped source files (2):
 - **Docs state hash: plugin resources by content, zoom never.** The browser editor (full docs preset) writes one `resources` entry per plugin on every save (`"[]"`, `""`, `{"links":[]}`, `{"data":{},"order":[]}`) and `settings.zoomRatio`; a stored row and a headless load have neither, so live seats still refused a restore after 0.8.18. The canonical form now parses each resource, drops empty ones, sorts by name, and drops the zoom. Forcing case: the live row and what the browser held after loading it (red on 0.8.18, green). Consumer action: none.
 
 ## 0.8.18
-### C7a restore hash, stale checkpoints, collaborator colours
+### C7a restore hash, stale checkpoints, collaborator colors
 
-- **Docs co-editing: one canonical state hash, before and after Univer loads it (C7a follow-up).** `canonicalDocsSnapshot` now first brings a snapshot to the form every seat holds after loading it (`asLoadedDocsSnapshot`: deterministic ids, `sanitizeUniverDocSnapshot`, Univer's own `DocumentDataModel` defaults) and drops `sectionId` like `paragraphId`. A History restore of a document not open in this tab hashed the raw row while every open seat hashed what Univer held, so both seats refused it (`checkpoint_hash at rev 10`) and kept the old text; now the two agree (corpus + real History versions, through a real Univer load and save). Docs rooms: a keystroke the session refused while reloading puts the seat back on the room's document instead of staying on that seat alone; collaborator colours by join order (`claimCollaboratorColor` from data). `sanitizeUniverDocSnapshot` takes `{ quiet }`. Consumer action: none (no live room predates this hash; every existing room is on a trashed test document).
+- **Docs co-editing: one canonical state hash, before and after Univer loads it (C7a follow-up).** `canonicalDocsSnapshot` now first brings a snapshot to the form every seat holds after loading it (`asLoadedDocsSnapshot`: deterministic ids, `sanitizeUniverDocSnapshot`, Univer's own `DocumentDataModel` defaults) and drops `sectionId` like `paragraphId`. A History restore of a document not open in this tab hashed the raw row while every open seat hashed what Univer held, so both seats refused it (`checkpoint_hash at rev 10`) and kept the old text; now the two agree (corpus + real History versions, through a real Univer load and save). Docs rooms: a keystroke the session refused while reloading puts the seat back on the room's document instead of staying on that seat alone; collaborator colors by join order (`claimCollaboratorColor` from data). `sanitizeUniverDocSnapshot` takes `{ quiet }`. Consumer action: none (no live room predates this hash; every existing room is on a trashed test document).
 
 ## 0.8.17
 
@@ -152,6 +163,7 @@ Changed shipped source files (8):
 
 ### Unreleased
 
+- **US spelling: the British spelling of "color" no longer exists in this package (Arman, 2026-10-21).** Every label, template string, comment, identifier, CSS class/variable and test name that spelled it the British way now says color. Guard: `design-system/src/us-spelling.test.ts` scans all of `apps/` for the word. Consumer action: none for imports; if you matched on a British-spelled identifier, class or CSS variable from this package, use the `color` spelling; update the package.
 - **Sheets co-editing on the ordered log (C6).** `@ai-matrx/office/univer/collab` adds the Sheets room: `openSheetsOtRoom` (the C2 session over the C5a/C5b group transform, saves at a quiet point with a matching hash, restore through `collab_replace`, undo items rebased over every remote op), `univerSheetsOtPort` (capture: onlyLocal/fromCollab filtered, grouped by outermost command with a try/finally depth guard on both execute paths, params allowlist, completeCellContent, exact merge removals, style objects never ids, whole-style overrides restated as merges, annotateFormulas, inverses from the applied params against a pre-state copy; a barrier or unknown id is `not_shared` and rolls the seat back), `sheetsStateHash` / `canonicalSheet` (merges as a set, defined names by id; the transform fuzz now hashes through it), and `startWorkbookOtRoom` / `workbookUsesOrderedRoom` (the `udt_workbook_snapshots` store, the C1 doors, presence). A remote row/column insert, remove or move carries this seat's selection along and never leaves it past the sheet's edge (the port's `selections` option; a stale whole-row selection after a peer's column delete crashed the editor in the live run). Remote applies run with `{ fromCollab: true }` only (not `onlyLocal`): Univer's formula engine skips onlyLocal value writes, so a peer's formula showed blank on every other seat in the live run. Flip is per room exactly like Docs. Consumer action: matrx-frontend's WorkbookEditor opens it when the workbook's room exists or `collab.ot_rooms` says so (shipped with this release); every other consumer: none.
 
 - **`Workbook` and `DocumentRow` carry `test_expires_at` / `test_run_id`** — the `workbench.udt_workbooks` / `udt_documents` rows gained these test-fixture columns; the hand-written types now match the generated row again (matrx-frontend `features/data-tables/office-types-agree.ts`). Consumer action: none (a literal building one of these rows must add the two fields, `null` for real records).

package/dist/collab-sheets-transform.cjs

@@ -632,7 +632,8 @@ registerRangedResource({
   }),
   removeOf: (sheet, key) => ({ id: "sheet.mutation.delete-conditional-rule", params: { subUnitId: sheet, cfId: key, derived: true } }),
   hasFormula: (m) => CF_FORMULA(cfRule(m)),
-  orderedAdds: true
+  orderedAdds: true,
+  setsOverwrite: true
 });
 var DV = { add: "data-validation.mutation.addRule", update: "data-validation.mutation.updateRule", remove: "data-validation.mutation.removeRule" };
 var DV_RANGE = 1;
@@ -1864,6 +1865,24 @@ function transformMutation(x, y, side) {
   return ok2([x]);
 }
 var blocked = (p) => !Array.isArray(p);
+function compactDerivedSets(ops) {
+  if (ops.length < 2) return ops;
+  const later = /* @__PURE__ */ new Set();
+  const out = [];
+  for (let i = ops.length - 1; i >= 0; i--) {
+    const m = ops[i];
+    const spec = rangedSpecOf(m.id);
+    if (spec?.setsOverwrite && spec.ids[m.id] === "set") {
+      const at = (k, facet2) => `${spec.family}|${sheetOf(m)}|${k}|${facet2}`;
+      const facet = spec.facet?.(m) ?? "all";
+      const keys = spec.keys(m);
+      if (isDerived(m) && keys.every((k) => later.has(at(k, facet)) || later.has(at(k, "all")))) continue;
+      for (const k of keys) later.add(at(k, facet));
+    }
+    out.push(m);
+  }
+  return out.length === ops.length ? ops : out.reverse();
+}
 function tSeq(xs, ys, side, acc) {
   if (xs.length === 0 || ys.length === 0) return [xs, ys];
   if (xs.length === 1 && ys.length === 1) {
@@ -1887,33 +1906,17 @@ function tSeq(xs, ys, side, acc) {
       }
       const r = tSeq(curX, [y], side, acc);
       if (blocked(r)) return r;
-      curX = r[0];
+      curX = r[0].length > 1 ? compactDerivedSets(r[0]) : r[0];
       for (const m of r[1]) nextY.push(m);
     }
     for (const m of curX) outX.push(m);
-    curY = nextY;
+    curY = compactDerivedSets(nextY);
   }
-  return [outX, curY];
-}
-function tSeqOld(xs, ys, side, acc) {
-  if (xs.length === 0 || ys.length === 0) return [xs, ys];
-  if (xs.length === 1 && ys.length === 1) return tSeq(xs, ys, side, acc);
-  if (xs.length > 1) {
-    const first2 = tSeqOld([xs[0]], ys, side, acc);
-    if (blocked(first2)) return first2;
-    const rest2 = tSeqOld(xs.slice(1), first2[1], side, acc);
-    if (blocked(rest2)) return rest2;
-    return [[...first2[0], ...rest2[0]], rest2[1]];
-  }
-  const first = tSeqOld(xs, [ys[0]], side, acc);
-  if (blocked(first)) return first;
-  const rest = tSeqOld(first[0], ys.slice(1), side, acc);
-  if (blocked(rest)) return rest;
-  return [rest[0], [...first[1], ...rest[1]]];
+  return [compactDerivedSets(outX), curY];
 }
 function transformGroup(x, y, side) {
   const acc = { lost: [], dropped: [] };
-  const r = (process.env.OLD_TSEQ ? tSeqOld : tSeq)([...x], [...y], side, acc);
+  const r = tSeq([...x], [...y], side, acc);
   if (blocked(r)) return r;
   return { op: r[0], lost: [...new Set(acc.lost)], dropped: [...new Set(acc.dropped)] };
 }