@ai-matrx/records 0.58.27 → 0.58.28

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog — @ai-matrx/records
 
+## 0.58.28
+### lane RENDER-AUDIT (2026-09-26)
+
+- **A re-read page keeps every row that did not change.** `useRecords` / `useRecordPage` hand back the SAME object for a row the store answered exactly as before (same id, same everything it carries), so a page read after one write — the write door's own re-read, a reconnect, a colleague's new row — no longer gives every record a new identity and redraws every row of a grid. What is on screen is still exactly what the door answered.
+- **The realtime subscription is no longer torn down on every page change.** `useLiveRows` listed the page's rows in its effect, so every patch, read or optimistic write left the channel and joined it again (and a notice in the gap landed on nobody). It now subscribes per (port, table, client) and reads the page it judges a notice against at the moment the notice lands.
+- **An inline `onError` no longer rebuilds the client.** `RecordsProvider` holds one stable error sink that calls whatever the host passed last; a host writing `onError={(e) => …}` inline no longer re-reads every hook on every render. Whether a sink is bound at all is still part of the client's identity.
+
 ## 0.58.27
 
 Automatic changed-only republish (docs/metadata drift since the last tag — see

package/dist/react/index.cjs

@@ -41954,8 +41954,15 @@ function asWriteConflict(error, recordId) {
 // src/react/context.tsx
 var RecordsContext = (0, import_react.createContext)(null);
 function RecordsProvider({ config, children }) {
+  const latestOnError = (0, import_react.useRef)(config.onError);
+  latestOnError.current = config.onError;
+  const hasOnError = config.onError !== void 0;
+  const stableOnError = (0, import_react.useMemo)(
+    () => hasOnError ? (error) => latestOnError.current?.(error) : void 0,
+    [hasOnError]
+  );
   const client = (0, import_react.useMemo)(
-    () => createRecordsClient(config),
+    () => createRecordsClient(stableOnError === void 0 ? config : { ...config, onError: stableOnError }),
     // The identity of the organization and the actor is what makes a client a
     // different client. Rebuilding on those is the point, not a cost.
     [
@@ -41966,7 +41973,7 @@ function RecordsProvider({ config, children }) {
       config.actor.on_behalf_of,
       config.realtime,
       config.resolver,
-      config.onError,
+      stableOnError,
       config.schema
     ]
   );
@@ -42127,6 +42134,8 @@ function useLiveRows(tableId, base) {
   const client = useRecordsClient();
   const realtime = client.config.realtime;
   const { reload, setData } = base;
+  const heldRows = (0, import_react2.useRef)([]);
+  heldRows.current = base.data?.rows ?? [];
   (0, import_react2.useEffect)(() => {
     if (!realtime || !tableId) return;
     return realtime.subscribeRecords(
@@ -42145,7 +42154,7 @@ function useLiveRows(tableId, base) {
               return;
             }
             const fresh = new Map(answered.data.map((row) => [row.id, row]));
-            const onThisPage = (id) => (base.data?.rows ?? []).some((row) => row.id === id);
+            const onThisPage = (id) => heldRows.current.some((row) => row.id === id);
             const needsThePage = ids.some((id) => !onThisPage(id) && fresh.has(id));
             if (needsThePage) {
               void reload();
@@ -42168,7 +42177,7 @@ function useLiveRows(tableId, base) {
         }
       }
     );
-  }, [realtime, tableId, client, reload, setData, base.data]);
+  }, [realtime, tableId, client, reload, setData]);
   const live = Boolean(realtime && tableId);
   const liveReason = realtime ? tableId ? "Live: changes to this table re-read through the read door." : "Not live: no table selected." : "Not live: this host bound no realtime port, so this list updates when you reload it.";
   return { live, liveReason };
@@ -42181,6 +42190,38 @@ function useKeptPage(tableId, base) {
   }
   return base;
 }
+function sameRowAnswer(a, b) {
+  if (a === b) return true;
+  if (a.id !== b.id) return false;
+  try {
+    return JSON.stringify(a) === JSON.stringify(b);
+  } catch {
+    return false;
+  }
+}
+function useSharedRows(state) {
+  const held = (0, import_react2.useRef)({
+    source: null,
+    shared: null,
+    byId: /* @__PURE__ */ new Map()
+  });
+  const data = state.data;
+  if (data === null) return state;
+  const memo = held.current;
+  if (memo.source === data && memo.shared !== null) return { ...state, data: memo.shared };
+  let reused = 0;
+  const rows = data.rows.map((row) => {
+    const before = memo.byId.get(row.id);
+    if (before && sameRowAnswer(before, row)) {
+      reused += 1;
+      return before;
+    }
+    return row;
+  });
+  const shared = reused === 0 ? data : { ...data, rows };
+  held.current = { source: data, shared, byId: new Map(rows.map((row) => [row.id, row])) };
+  return { ...state, data: shared };
+}
 function useRecords(tableId, options = {}) {
   const pageSize = options.pageSize ?? 50;
   const page = options.page ?? 0;
@@ -42201,7 +42242,7 @@ function useRecords(tableId, options = {}) {
     [tableId, pageSize, page, filterKey]
   );
   const { live, liveReason } = useLiveRows(tableId, base);
-  return { ...useKeptPage(tableId, base), live, liveReason };
+  return { ...useSharedRows(useKeptPage(tableId, base)), live, liveReason };
 }
 function useRecordPage(tableId, options = {}) {
   const pageSize = options.pageSize ?? 50;
@@ -42227,7 +42268,7 @@ function useRecordPage(tableId, options = {}) {
     [tableId, pageSize, page, filterKey, search, sortKey, viewId]
   );
   const { live, liveReason } = useLiveRows(tableId, base);
-  return { ...useKeptPage(tableId, base), live, liveReason };
+  return { ...useSharedRows(useKeptPage(tableId, base)), live, liveReason };
 }
 function useRecordChangeRevision(recordId) {
   const [revision, setRevision] = (0, import_react2.useState)(0);