@ai-matrx/records 0.81.0 → 0.81.1

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog — @ai-matrx/records
 
+## 0.81.1
+### lane SORT-REACTS
+
+- A live page no longer patches a changed row in place when the change decides its place: `useRecordPage` / `useRecords` read the page again when a realtime notice or this page's own write changed a value the page is sorted by (the group key included), or any value on a filtered or searched page. A date edited to the top now moves to the top, and a row edited out of the filter leaves the page, without a reload. An edit that places nothing still patches in place and reads no page (`react/pageOrder.ts`).
+
+**Consumer action:** none.
+
 ## 0.81.0
 ### SSR-ROWS: a page's first answers, asked on the server, drawn in the server's HTML
 

package/dist/react/index.cjs

@@ -8175,6 +8175,31 @@ function useHoldKey() {
   return (0, import_react3.useContext)(HoldContext);
 }
 
+// src/react/pageOrder.ts
+function pageOrderOf(sort, narrowed) {
+  return { sortFields: (sort ?? []).map((key) => key.field), narrowed };
+}
+var same = (a, b) => {
+  if (a === b) return true;
+  if ((a === null || a === void 0 || a === "") && (b === null || b === void 0 || b === "")) return true;
+  return JSON.stringify(a) === JSON.stringify(b);
+};
+function patchMovesRows(held, fresh, order) {
+  if (order.sortFields.length === 0 && !order.narrowed) return false;
+  for (const row of held) {
+    const next = fresh.get(row.id);
+    if (!next) continue;
+    const was = row.document ?? {};
+    const now = next.document ?? {};
+    if (order.narrowed) {
+      if (!same(was, now)) return true;
+      continue;
+    }
+    for (const field2 of order.sortFields) if (!same(was[field2], now[field2])) return true;
+  }
+  return false;
+}
+
 // src/react/hooks.ts
 function useAsync(run, key, live) {
   const client = useRecordsClient();
@@ -8358,7 +8383,7 @@ function useReverseLinks(tableId, columns, recordIds, limit = 3, offset = 0) {
     tableId ? `reverse-links:${tableId}:${columnsKey}:${limit}:${offset}:${idsKey}:${writes}` : "reverse-links:none"
   );
 }
-function liveRowsFor(tableId) {
+function liveRowsFor(tableId, order = { sortFields: [], narrowed: false }) {
   return (entry) => {
     const client = entry.client;
     const realtime = client.config.realtime;
@@ -8377,6 +8402,10 @@ function liveRowsFor(tableId) {
         entry.refetch();
         return;
       }
+      if (patchMovesRows(held, fresh, order)) {
+        entry.refetch();
+        return;
+      }
       entry.setData(
         (current) => current === null ? current : {
           ...current,
@@ -8484,7 +8513,7 @@ function useRecords(tableId, options = {}) {
     // on every render must not re-read the page on every render, and a parent
     // that changes the question must re-read on the first one.
     `records:${tableId ?? "none"}:${pageSize}:${page2}:${filterKey}:s${shape}`,
-    liveRowsFor(tableId)
+    liveRowsFor(tableId, pageOrderOf(void 0, filterKey !== "{}"))
   );
   const { live, liveReason } = liveStatus(useRecordsClient(), tableId);
   return { ...useSharedRows(useKeptPage(tableId, base)), live, liveReason };
@@ -8512,7 +8541,7 @@ function useRecordPage(tableId, options = {}) {
       return answered.ok ? { ok: true, data: { rows: answered.data.rows, total: answered.data.total, sortIgnored: answered.data.sort_ignored ?? [] } } : answered;
     },
     `page:${tableId ?? "none"}:${pageSize}:${page2}:${filterKey}:${search ?? ""}:${sortKey}:${viewId ?? ""}:s${shape}`,
-    liveRowsFor(tableId)
+    liveRowsFor(tableId, pageOrderOf(options.sort, filterKey !== "{}" || search !== null))
   );
   const { live, liveReason } = liveStatus(useRecordsClient(), tableId);
   return { ...useSharedRows(useKeptPage(tableId, base)), live, liveReason };