@jmruthers/pace-core 0.6.2 → 0.6.3

package/dist/{chunk-SFZUDBL5.js → chunk-2T2IG7T7.js}

@@ -1,9 +1,9 @@
 import {
   useEvents
-} from "./chunk-MMZ7JXPU.js";
+} from "./chunk-6Z7LTB3D.js";
 import {
   isSuperAdmin
-} from "./chunk-24UVZUZG.js";
+} from "./chunk-RWEBCB47.js";
 import {
   assertAppId
 } from "./chunk-QXHPKYJV.js";
@@ -1210,7 +1210,7 @@ function getPublicFileDisplayCacheStats() {
 }
 
 // src/hooks/useFileDisplay.ts
-import { useState as useState4, useEffect as useEffect5, useCallback as useCallback4 } from "react";
+import { useState as useState4, useEffect as useEffect5, useCallback as useCallback4, useRef as useRef3 } from "react";
 
 // src/utils/file-reference/index.ts
 var log3 = createLogger("FileReferenceService");
@@ -1691,14 +1691,20 @@ function useFileDisplay(table_name, record_id, organisation_id, category, option
   const [fileCount, setFileCount] = useState4(0);
   const [isLoading, setIsLoading] = useState4(false);
   const [error, setError] = useState4(null);
+  const isMountedRef = useRef3(true);
+  const safeSetState = (setter, value) => {
+    if (isMountedRef.current) {
+      setter(value);
+    }
+  };
   const fetchFiles = useCallback4(async () => {
     if (!table_name || !record_id || !supabase) {
-      setFileUrl(null);
-      setFileReference(null);
-      setFileReferences([]);
-      setFileUrls(/* @__PURE__ */ new Map());
-      setFileCount(0);
-      setIsLoading(false);
+      safeSetState(setFileUrl, null);
+      safeSetState(setFileReference, null);
+      safeSetState(setFileReferences, []);
+      safeSetState(setFileUrls, /* @__PURE__ */ new Map());
+      safeSetState(setFileCount, 0);
+      safeSetState(setIsLoading, false);
       return;
     }
     if (organisation_id) {
@@ -1721,31 +1727,31 @@ function useFileDisplay(table_name, record_id, organisation_id, category, option
               expiresIn: 3600
             });
             const regeneratedUrl = signedUrlResult?.url || null;
-            setFileUrl(regeneratedUrl);
-            setFileReference(cachedData.fileReference);
-            setFileReferences(cachedData.fileReferences || []);
-            setFileUrls(cachedData.fileUrls || /* @__PURE__ */ new Map());
-            setFileCount(cachedData.fileCount || 0);
-            setIsLoading(false);
-            setError(null);
+            safeSetState(setFileUrl, regeneratedUrl);
+            safeSetState(setFileReference, cachedData.fileReference);
+            safeSetState(setFileReferences, cachedData.fileReferences || []);
+            safeSetState(setFileUrls, cachedData.fileUrls || /* @__PURE__ */ new Map());
+            safeSetState(setFileCount, cachedData.fileCount || 0);
+            safeSetState(setIsLoading, false);
+            safeSetState(setError, null);
             return;
           } catch (err) {
             logger.warn("useFileDisplay", "Failed to regenerate signed URL from cache, falling back to fetch:", err);
           }
         }
-        setFileUrl(cachedData.fileUrl || null);
-        setFileReference(cachedData.fileReference || null);
-        setFileReferences(cachedData.fileReferences || []);
-        setFileUrls(cachedData.fileUrls || /* @__PURE__ */ new Map());
-        setFileCount(cachedData.fileCount || 0);
-        setIsLoading(false);
-        setError(null);
+        safeSetState(setFileUrl, cachedData.fileUrl || null);
+        safeSetState(setFileReference, cachedData.fileReference || null);
+        safeSetState(setFileReferences, cachedData.fileReferences || []);
+        safeSetState(setFileUrls, cachedData.fileUrls || /* @__PURE__ */ new Map());
+        safeSetState(setFileCount, cachedData.fileCount || 0);
+        safeSetState(setIsLoading, false);
+        safeSetState(setError, null);
         return;
       }
     }
     try {
-      setIsLoading(true);
-      setError(null);
+      safeSetState(setIsLoading, true);
+      safeSetState(setError, null);
       const service = createFileReferenceService(supabase);
       let files = [];
       const shouldSearchBothScopes = organisation_id === void 0;
@@ -1943,11 +1949,11 @@ function useFileDisplay(table_name, record_id, organisation_id, category, option
         }
       }
       if (files.length === 0) {
-        setFileUrl(null);
-        setFileReference(null);
-        setFileReferences([]);
-        setFileUrls(/* @__PURE__ */ new Map());
-        setFileCount(0);
+        safeSetState(setFileUrl, null);
+        safeSetState(setFileReference, null);
+        safeSetState(setFileReferences, []);
+        safeSetState(setFileUrls, /* @__PURE__ */ new Map());
+        safeSetState(setFileCount, 0);
         if (enableCache) {
           authenticatedFileCache.set(cacheKey, {
             data: { fileUrl: null, fileReference: null, fileReferences: [], fileUrls: /* @__PURE__ */ new Map(), fileCount: 0 },
@@ -1958,11 +1964,11 @@ function useFileDisplay(table_name, record_id, organisation_id, category, option
         }
         return;
       }
-      setFileReferences(files);
-      setFileCount(files.length);
+      safeSetState(setFileReferences, files);
+      safeSetState(setFileCount, files.length);
       if (category && files.length > 0) {
         const firstFile = files[0];
-        setFileReference(firstFile);
+        safeSetState(setFileReference, firstFile);
         let url = null;
         if (firstFile.is_public) {
           url = getPublicUrl(supabase, firstFile.file_path, true);
@@ -1982,7 +1988,7 @@ function useFileDisplay(table_name, record_id, organisation_id, category, option
             });
           }
         }
-        setFileUrl(url);
+        safeSetState(setFileUrl, url);
       } else {
         const urlMap = await generateFileUrlsBatch(supabase, files, {
           appName: "pace-core",
@@ -1990,9 +1996,9 @@ function useFileDisplay(table_name, record_id, organisation_id, category, option
           userId: organisation_id ? void 0 : record_id,
           expiresIn: 3600
         });
-        setFileUrls(urlMap);
-        setFileReference(null);
-        setFileUrl(null);
+        safeSetState(setFileUrls, urlMap);
+        safeSetState(setFileReference, null);
+        safeSetState(setFileUrl, null);
       }
       if (enableCache) {
         let cacheData = {
@@ -2030,28 +2036,36 @@ function useFileDisplay(table_name, record_id, organisation_id, category, option
     } catch (err) {
       logger.error("useFileDisplay", "Error fetching files:", err);
       const error2 = err instanceof Error ? err : new Error("Unknown error occurred");
-      setError(error2);
-      setFileUrl(null);
-      setFileReference(null);
-      setFileReferences([]);
-      setFileUrls(/* @__PURE__ */ new Map());
-      setFileCount(0);
+      safeSetState(setError, error2);
+      safeSetState(setFileUrl, null);
+      safeSetState(setFileReference, null);
+      safeSetState(setFileReferences, []);
+      safeSetState(setFileUrls, /* @__PURE__ */ new Map());
+      safeSetState(setFileCount, 0);
     } finally {
-      setIsLoading(false);
+      safeSetState(setIsLoading, false);
     }
   }, [table_name, record_id, organisation_id, category, supabase, cacheTtl, enableCache]);
   useEffect5(() => {
+    isMountedRef.current = true;
     if (table_name && record_id && supabase) {
-      fetchFiles();
+      fetchFiles().catch((err) => {
+        if (isMountedRef.current) {
+          safeSetState(setError, err instanceof Error ? err : new Error("Unknown error"));
+        }
+      });
     } else {
-      setFileUrl(null);
-      setFileReference(null);
-      setFileReferences([]);
-      setFileUrls(/* @__PURE__ */ new Map());
-      setFileCount(0);
-      setIsLoading(false);
-      setError(null);
+      safeSetState(setFileUrl, null);
+      safeSetState(setFileReference, null);
+      safeSetState(setFileReferences, []);
+      safeSetState(setFileUrls, /* @__PURE__ */ new Map());
+      safeSetState(setFileCount, 0);
+      safeSetState(setIsLoading, false);
+      safeSetState(setError, null);
     }
+    return () => {
+      isMountedRef.current = false;
+    };
   }, [table_name, record_id, organisation_id, supabase]);
   const refetch = useCallback4(async () => {
     if (!table_name || !record_id || !supabase) return;
@@ -2143,11 +2157,11 @@ function useEventTheme(event) {
 }
 
 // src/hooks/usePreventTabReload.ts
-import { useEffect as useEffect7, useRef as useRef3 } from "react";
+import { useEffect as useEffect7, useRef as useRef4 } from "react";
 function usePreventTabReload(options = {}) {
   const { enabled = true, gracePeriodMs = 2e3 } = options;
-  const isRestoringFromCacheRef = useRef3(false);
-  const gracePeriodTimeoutRef = useRef3(null);
+  const isRestoringFromCacheRef = useRef4(false);
+  const gracePeriodTimeoutRef = useRef4(null);
   useEffect7(() => {
     if (!enabled || typeof window === "undefined") return;
     const handlePageShow = (event) => {
@@ -2220,4 +2234,4 @@ export {
   useEventTheme,
   usePreventTabReload
 };
-//# sourceMappingURL=chunk-SFZUDBL5.js.map
+//# sourceMappingURL=chunk-2T2IG7T7.js.map