From 3f9da22b58d7d5e487bbe7b11bc146314ddcf000 Mon Sep 17 00:00:00 2001
From: Tom Butcher <thomasbutcher15@gmail.com>
Date: Fri, 13 Mar 2026 19:47:26 +0000
Subject: [PATCH] Minor performance improvements.

---
 src/components/Dashboard/common/FilePreview.jsx  | 2 +-
 src/components/Dashboard/common/FileUpload.jsx   | 2 +-
 src/components/Dashboard/context/AuthContext.jsx | 3 ++-
 3 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/src/components/Dashboard/common/FilePreview.jsx b/src/components/Dashboard/common/FilePreview.jsx
index 14fcdac..9a3a0b6 100644
--- a/src/components/Dashboard/common/FilePreview.jsx
+++ b/src/components/Dashboard/common/FilePreview.jsx
@@ -29,7 +29,7 @@ const FilePreview = ({ file, style = {} }) => {
     }
     setFileObjectUrl(objectUrl)
     setLoading(false)
-  }, [file, fetchFileContent])
+  }, [file, fetchFileContent, error])
 
   useEffect(() => {
     if (file?.type && token != null) {
diff --git a/src/components/Dashboard/common/FileUpload.jsx b/src/components/Dashboard/common/FileUpload.jsx
index be83fe1..de8521a 100644
--- a/src/components/Dashboard/common/FileUpload.jsx
+++ b/src/components/Dashboard/common/FileUpload.jsx
@@ -57,7 +57,7 @@ const FileUpload = ({
       : !currentFiles
     setHasNoItems(noItems)
     onChange(currentFiles)
-  }, [currentFiles, multiple])
+  }, [currentFiles, multiple, onChange])
 
   const handleFileUpload = async (file) => {
     try {
diff --git a/src/components/Dashboard/context/AuthContext.jsx b/src/components/Dashboard/context/AuthContext.jsx
index c4c796d..9f1fbd3 100644
--- a/src/components/Dashboard/context/AuthContext.jsx
+++ b/src/components/Dashboard/context/AuthContext.jsx
@@ -231,7 +231,8 @@ const AuthProvider = ({ children }) => {
     return () => {
       cancelled = true
     }
-  }, [isElectron, getAuthSession, clearPersistedSession, getUserInfo])
+    // eslint-disable-next-line react-hooks/exhaustive-deps -- run only on mount to load persisted session; deps are stable in behavior
+  }, [])
 
   // Set up cookie synchronization between tabs
   useEffect(() => {