cleanup some hooks

Author: izadoesdev

apps/dashboard/hooks/use-device-sessions.ts

@@ -17,12 +17,10 @@ interface DeviceSessionDetails {
 }
 
 export interface DeviceSessionEntry {
-	// Exporting for use in TopHeader
 	session: DeviceSessionDetails;
 	user?: {
 		email?: string;
 		name?: string;
-		// other user properties from auth
 	};
 	sessionToken: string;
 	isCurrent: boolean;
@@ -73,9 +71,6 @@ export function useDeviceSessions() {
 		} catch (err: any) {
 			const errorMessage = err.message || 'Failed to fetch sessions.';
 			setError(errorMessage);
-			// Toasting errors here might be too aggressive if the hook is used in a non-UI critical way
-			// Consider letting the component using the hook decide on toast notifications
-			// toast.error(errorMessage);
 		} finally {
 			setIsLoading(false);
 		}
@@ -94,7 +89,6 @@ export function useDeviceSessions() {
 			}
 			toast.success('Session switched successfully. Reloading...');
 			window.location.reload();
-			// No need to call fetchSessions() here as the page reloads
 			return { success: true };
 		} catch (err: any) {
 			toast.error(err.message || 'Failed to switch session.');
@@ -115,7 +109,7 @@ export function useDeviceSessions() {
 				throw new Error(result.error.message);
 			}
 			toast.success('Session revoked successfully.');
-			fetchSessions(); // Refresh the list after revoking
+			fetchSessions();
 			return { success: true };
 		} catch (err: any) {
 			toast.error(err.message || 'Failed to revoke session.');
@@ -133,7 +127,7 @@ export function useDeviceSessions() {
 		isLoading,
 		error,
 		operatingSession,
-		fetchSessions, // Expose fetchSessions if manual refresh is needed
+		fetchSessions,
 		setActiveSession,
 		revokeSession,
 	};

apps/dashboard/hooks/use-domain-info.ts

@@ -19,23 +19,21 @@ interface BatchDomainRankResponse {
 	data: Record<string, DomainRankData | null>;
 }
 
-const CACHE_TIME = 60 * 60 * 1000; // 1 hour
-const STALE_TIME = 30 * 60 * 1000; // 30 minutes
+const CACHE_TIME = 60 * 60 * 1000;
+const STALE_TIME = 30 * 60 * 1000;
 
 const queryOptions = {
 	staleTime: STALE_TIME,
 	gcTime: CACHE_TIME + 5 * 60 * 1000,
 	refetchOnWindowFocus: false,
 	refetchOnMount: true,
 	retry: (failureCount: number, error: Error) => {
-		// Don't retry on 4xx errors except 408 (timeout)
 		if (
 			error.message.includes('HTTP 4') &&
 			!error.message.includes('HTTP 408')
 		) {
 			return false;
 		}
-		// Retry up to 3 times for other errors
 		return failureCount < 3;
 	},
 	retryDelay: (attemptIndex: number) =>
@@ -84,32 +82,25 @@ export function useDomainRanks() {
 	};
 
 	return {
-		// Data
 		ranks: query.data?.data ?? {},
 
-		// Loading states
 		isLoading: query.isLoading,
 		isFetching: query.isFetching,
 		isRefetching: query.isRefetching,
 
-		// Error states
 		isError: query.isError,
 		error: query.error,
 
-		// Status
 		status: query.status,
 		fetchStatus: query.fetchStatus,
 
-		// Actions
 		refetch: query.refetch,
 		invalidateAndRefetch,
 		forceRefresh,
 
-		// Timestamps
 		dataUpdatedAt: query.dataUpdatedAt,
 		errorUpdatedAt: query.errorUpdatedAt,
 
-		// Helper functions
 		isStale: query.isStale,
 		isPending: query.isPending,
 	};