Fix IndexedDB cache comparison to ignore React Query metadata.

The previous hash comparison was including React Query metadata fields that change on every access (like dataUpdatedAt, dataUpdateCount, etc.). Now we extract and compare only the actual query data and errors, ignoring the constantly-changing metadata.

Author: FactorioBlueprints

src/localStorage.ts

@@ -736,6 +736,32 @@ interface Persister {
 export function createIDBPersister(idbValidKey: string = STORAGE_KEYS.QUERY_CACHE): Persister {
 	let lastPersistedHash: string | null = null;
 
+	// Extract only the actual query data, excluding metadata that changes on every access
+	const extractQueryData = (client: any): any => {
+		if (!client || !client.clientState) {
+			return client;
+		}
+
+		// Create a copy with only the actual data from queries
+		const dataOnly = {
+			queries: client.clientState.queries?.map((query: any) => ({
+				queryKey: query.queryKey,
+				queryHash: query.queryHash,
+				data: query.state?.data,
+				// Only include error if it exists
+				...(query.state?.error ? {error: query.state.error} : {}),
+			})),
+			mutations: client.clientState.mutations?.map((mutation: any) => ({
+				mutationKey: mutation.mutationKey,
+				state: mutation.state?.status,
+				data: mutation.state?.data,
+				error: mutation.state?.error,
+			})),
+		};
+
+		return dataOnly;
+	};
+
 	const hashObject = (obj: any): string => {
 		const str = JSON.stringify(obj);
 		let hash = 0;
@@ -750,8 +776,18 @@ export function createIDBPersister(idbValidKey: string = STORAGE_KEYS.QUERY_CACH
 	const debouncedPersist = debounce(
 		async (client: any) => {
 			try {
-				// Calculate hash of current state
-				const currentHash = hashObject(client);
+				// Extract only the data portion for comparison
+				const dataOnly = extractQueryData(client);
+				const currentHash = hashObject(dataOnly);
+
+				// Debug logging
+				console.log('[IndexedDB] Hash comparison:', {
+					current: currentHash,
+					last: lastPersistedHash,
+					equal: lastPersistedHash === currentHash,
+					hasClientState: !!client?.clientState,
+					queryCount: client?.clientState?.queries?.length || 0,
+				});
 
 				// Skip persistence if state hasn't changed
 				if (lastPersistedHash === currentHash) {