nhcarrigan/elysium
9
0
Fork 0
generated from nhcarrigan/template
Code Issues 5 Pull Requests 1 Actions Releases 10 Activity

fix: prevent auto-save race condition during force sync

Browse Source 
Skip auto-save when isSyncingRef is true and reset the auto-save timer
at the start of forceSync to prevent concurrent saves from sharing the
same HMAC signature and causing alternating save failures.
This commit is contained in:
Hikari
2026-03-06 19:30:38 -08:00
committed by Naomi Carrigan
parent 5aae3eb389
commit 4e32709e07
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
apps/web/src/context/GameContext.tsx
+3 -2
View File
@@ -154,10 +154,10 @@ export const GameProvider = ({ children }: { children: React.ReactNode }): React
newlyUnlockedRef.current = []; newlyUnlockedRef.current = [];
} }
// Auto-save every 30 seconds // Auto-save every 30 seconds (skip if a force sync is in-flight to avoid signature collisions)
if (Date.now() - lastSaveRef.current >= AUTO_SAVE_INTERVAL_MS) { if (Date.now() - lastSaveRef.current >= AUTO_SAVE_INTERVAL_MS) {
lastSaveRef.current = Date.now(); lastSaveRef.current = Date.now();
if (stateRef.current) { if (stateRef.current && !isSyncingRef.current) {
void saveGame({ void saveGame({
state: stateRef.current, state: stateRef.current,
signature: signatureRef.current ?? undefined, signature: signatureRef.current ?? undefined,
@@ -203,6 +203,7 @@ export const GameProvider = ({ children }: { children: React.ReactNode }): React
const forceSync = useCallback(async () => { const forceSync = useCallback(async () => {
if (!stateRef.current || isSyncingRef.current) return; if (!stateRef.current || isSyncingRef.current) return;
isSyncingRef.current = true; isSyncingRef.current = true;
lastSaveRef.current = Date.now(); // push auto-save timer back so it doesn't fire concurrently
setIsSyncing(true); setIsSyncing(true);
try { try {
const response = await saveGame({ const response = await saveGame({