fix: use ref for refresh state checks to prevent race conditions

The previous commit switched to using isPullRefreshing state in event
handlers, which caused stale closures and race conditions. Event handlers
would see outdated state values, allowing multiple refreshes to start
simultaneously.
This fixes the issue by:
- Re-adding isRefreshingRef for synchronous state checks
- Setting the ref immediately in handleRefresh() before state updates
- Updating all handlers to check isRefreshingRef.current instead of state
- Keeping the 5-second cooldown as an additional safeguard
The ref updates synchronously, blocking rapid-fire events instantly rather
than waiting for React's state update cycle.
Co-authored-by: Anthony <AnthonyRonning@users.noreply.github.com>

Author: github-actions[bot]

frontend/src/components/ChatHistoryList.tsx

@@ -91,6 +91,7 @@ export function ChatHistoryList({
 constpullDistanceRef=useRef(0);
 constwheelDeltaAccumulator=useRef(0);
 constlastRefreshTime=useRef(0);
+constisRefreshingRef=useRef(false);
 
 // Fetch initial conversations from API using the OpenSecret SDK
 const{ isPending, error }=useQuery({
@@ -185,6 +186,7 @@ export function ChatHistoryList({
 
 // Pull-to-refresh handler
 consthandleRefresh=useCallback(async()=>{
+isRefreshingRef.current=true;
 setIsPullRefreshing(true);
 try{
 awaitpollForUpdates();
@@ -195,6 +197,7 @@ export function ChatHistoryList({
 setTimeout(()=>{
 setIsPullRefreshing(false);
 setPullDistance(0);
+isRefreshingRef.current=false;
 // Record the time when refresh completes for cooldown
 lastRefreshTime.current=Date.now();
 },300);
@@ -211,14 +214,14 @@ export function ChatHistoryList({
 
 consthandleTouchStart=(e: TouchEvent)=>{
 // Only start pull if we're at the top of the scroll
-if(container.scrollTop===0&&!isPullRefreshing){
+if(container.scrollTop===0&&!isRefreshingRef.current){
 pullStartY.current=e.touches[0].clientY;
 isPulling.current=true;
 }
 };
 
 consthandleTouchMove=(e: TouchEvent)=>{
-if(!isPulling.current||isPullRefreshing)return;
+if(!isPulling.current||isRefreshingRef.current)return;
 
 constcurrentY=e.touches[0].clientY;
 constdistance=currentY-pullStartY.current;
@@ -258,14 +261,14 @@ export function ChatHistoryList({
 if(isDesktopPlatform)return;
 
 // Only start pull if we're at the top of the scroll
-if(container.scrollTop===0&&!isPullRefreshing){
+if(container.scrollTop===0&&!isRefreshingRef.current){
 pullStartY.current=e.clientY;
 isPulling.current=true;
 }
 };
 
 consthandleMouseMove=(e: MouseEvent)=>{
-if(!isPulling.current||isPullRefreshing||isDesktopPlatform)return;
+if(!isPulling.current||isRefreshingRef.current||isDesktopPlatform)return;
 
 constcurrentY=e.clientY;
 constdistance=currentY-pullStartY.current;
@@ -302,7 +305,7 @@ export function ChatHistoryList({
 constWHEEL_THRESHOLD=-50;// Require accumulated scroll before triggering
 
 consthandleWheel=(e: WheelEvent)=>{
-if(!isDesktopPlatform||isPullRefreshing)return;
+if(!isDesktopPlatform||isRefreshingRef.current)return;
 
 // Only handle if the event target is within our container
 if(!container.contains(e.targetasNode))return;