@ai-matrx/records 0.58.27 → 0.58.28

package/dist/react/index.js

@@ -36166,7 +36166,8 @@ import {
   createContext,
   createElement,
   useContext,
-  useMemo
+  useMemo,
+  useRef
 } from "react";
 
 // src/archive.ts
@@ -41798,8 +41799,15 @@ function asWriteConflict(error, recordId) {
 // src/react/context.tsx
 var RecordsContext = createContext(null);
 function RecordsProvider({ config, children }) {
+  const latestOnError = useRef(config.onError);
+  latestOnError.current = config.onError;
+  const hasOnError = config.onError !== void 0;
+  const stableOnError = useMemo(
+    () => hasOnError ? (error) => latestOnError.current?.(error) : void 0,
+    [hasOnError]
+  );
   const client = 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.
     [
@@ -41810,7 +41818,7 @@ function RecordsProvider({ config, children }) {
       config.actor.on_behalf_of,
       config.realtime,
       config.resolver,
-      config.onError,
+      stableOnError,
       config.schema
     ]
   );
@@ -41833,7 +41841,7 @@ function useRecordsActor() {
 }
 
 // src/react/hooks.ts
-import { useCallback, useEffect, useMemo as useMemo2, useRef, useState, useSyncExternalStore } from "react";
+import { useCallback, useEffect, useMemo as useMemo2, useRef as useRef2, useState, useSyncExternalStore } from "react";
 var SHARED_ANSWER_MS = 2e3;
 var sharedAnswers = /* @__PURE__ */ new WeakMap();
 function askShared(client, key, fresh, run) {
@@ -41862,7 +41870,7 @@ function useAsync(run, deps, share) {
     { data: null, loading: true, error: null }
   );
   const [nonce, setNonce] = useState(0);
-  const alive = useRef(true);
+  const alive = useRef2(true);
   useEffect(() => {
     alive.current = true;
     return () => {
@@ -41971,6 +41979,8 @@ function useLiveRows(tableId, base) {
   const client = useRecordsClient();
   const realtime = client.config.realtime;
   const { reload, setData } = base;
+  const heldRows = useRef2([]);
+  heldRows.current = base.data?.rows ?? [];
   useEffect(() => {
     if (!realtime || !tableId) return;
     return realtime.subscribeRecords(
@@ -41989,7 +41999,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();
@@ -42012,19 +42022,51 @@ 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 };
 }
 function useKeptPage(tableId, base) {
-  const lastGood = useRef(null);
+  const lastGood = useRef2(null);
   if (base.data !== null && !base.error) lastGood.current = { tableId, data: base.data };
   if (base.error && base.data === null && lastGood.current && lastGood.current.tableId === tableId) {
     return { ...base, data: lastGood.current.data };
   }
   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 = useRef2({
+    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;
@@ -42045,7 +42087,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;
@@ -42071,7 +42113,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] = useState(0);
@@ -42153,7 +42195,7 @@ function useTableArchive(tableId, options) {
   const [interrupted, setInterrupted] = useState(false);
   const [chunkInUse, setChunkInUse] = useState(chunk);
   const [slowedDown, setSlowedDown] = useState(false);
-  const cut = useRef(false);
+  const cut = useRef2(false);
   const preview = useCallback(async () => {
     setError(null);
     const answered = await client.tableArchive({ table_id: tableId, chunk: 0, includeTable });