From 3606e2ab54fdde5f39cbafbaa0ed42b136f74077 Mon Sep 17 00:00:00 2001
From: pac7 <47831526-pac7@users.noreply.replit.com>
Date: Wed, 8 Oct 2025 12:52:32 +0000
Subject: [PATCH] Improve data fetching performance and fix fast refresh issues

Refactor `fetchData` in `ManufacturerModels.tsx` using `useCallback` to prevent unnecessary re-renders and memoize the function. Add `fetchData` to the `useEffect` dependency array in `ManufacturerModels.tsx` to ensure it's correctly called. Address Fast Refresh compatibility in `useAuth.tsx` and correct `setInterval` dependency in `useModerationQueue.ts`.

Replit-Commit-Author: Agent
Replit-Commit-Session-Id: fe5b902e-beda-40fc-bf87-a3c4ab300e3a
Replit-Commit-Checkpoint-Type: intermediate_checkpoint
---
 src/pages/ManufacturerModels.tsx | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/src/pages/ManufacturerModels.tsx b/src/pages/ManufacturerModels.tsx
index c83f34cf..84047a6f 100644
--- a/src/pages/ManufacturerModels.tsx
+++ b/src/pages/ManufacturerModels.tsx
@@ -1,4 +1,4 @@
-import { useState, useEffect } from 'react';
+import { useState, useEffect, useCallback } from 'react';
 import { useParams, useNavigate } from 'react-router-dom';
 import { Header } from '@/components/layout/Header';
 import { Button } from '@/components/ui/button';
@@ -20,13 +20,7 @@ export default function ManufacturerModels() {
   const [sortBy, setSortBy] = useState('name');
   const [filterCategory, setFilterCategory] = useState('all');
 
-  useEffect(() => {
-    if (manufacturerSlug) {
-      fetchData();
-    }
-  }, [manufacturerSlug, sortBy, filterCategory]);
-
-  const fetchData = async () => {
+  const fetchData = useCallback(async () => {
     try {
       // Fetch manufacturer
       const { data: manufacturerData, error: manufacturerError } = await supabase
@@ -70,7 +64,13 @@ export default function ManufacturerModels() {
     } finally {
       setLoading(false);
     }
-  };
+  }, [manufacturerSlug, sortBy, filterCategory]);
+
+  useEffect(() => {
+    if (manufacturerSlug) {
+      fetchData();
+    }
+  }, [manufacturerSlug, fetchData]);
 
   const filteredModels = models.filter(model =>
     model.name.toLowerCase().includes(searchQuery.toLowerCase()) ||