jeffemmett
/
canvas-website
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix: TypeScript errors in sync version state 

- Fixed variable scope issue with totalMerged counter
- Added syncVersion to return type declaration

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
Jeff Emmett 2026-01-09 18:58:24 +01:00
parent edb386ec3c
commit 08bea8490d
1 changed files with 7 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/automerge/useAutomergeSyncRepo.ts
View File

@ -117,6 +117,7 @@ export function useAutomergeSync(config: AutomergeSyncConfig): TLStoreWithStatus
presence: ReturnType<typeof useAutomergePresence>; presence: ReturnType<typeof useAutomergePresence>;
connectionState: ConnectionState; connectionState: ConnectionState;
isNetworkOnline: boolean; isNetworkOnline: boolean;
syncVersion: number;
} { } {
const { uri, user } = config const { uri, user } = config
@ -486,6 +487,9 @@ export function useAutomergeSync(config: AutomergeSyncConfig): TLStoreWithStatus
// 3. Otherwise, only add server records that don't exist locally // 3. Otherwise, only add server records that don't exist locally
// (preserve offline changes, let Automerge CRDT sync handle conflicts) // (preserve offline changes, let Automerge CRDT sync handle conflicts)
if (serverDoc.store && serverRecordCount > 0) { if (serverDoc.store && serverRecordCount > 0) {
// Track if we merged any data (needed outside the change callback)
let totalMerged = 0
handle.change((doc: any) => { handle.change((doc: any) => {
// Initialize store if it doesn't exist // Initialize store if it doesn't exist
if (!doc.store) { if (!doc.store) {
@ -534,6 +538,7 @@ export function useAutomergeSync(config: AutomergeSyncConfig): TLStoreWithStatus
} }
}) })
totalMerged = addedFromServer + replacedFromServer
console.log(`🔄 Server sync: added=${addedFromServer}, replaced=${replacedFromServer}, skipped=${skippedExisting}, shouldPreferServer=${shouldPreferServer}`) console.log(`🔄 Server sync: added=${addedFromServer}, replaced=${replacedFromServer}, skipped=${skippedExisting}, shouldPreferServer=${shouldPreferServer}`)
}) })
@ -542,8 +547,8 @@ export function useAutomergeSync(config: AutomergeSyncConfig): TLStoreWithStatus
// CRITICAL: Force React to re-render after merging server data // CRITICAL: Force React to re-render after merging server data
// The handle object reference doesn't change, so we increment syncVersion // The handle object reference doesn't change, so we increment syncVersion
if ((addedFromServer > 0 || replacedFromServer > 0) && mounted) { if (totalMerged > 0 && mounted) {
console.log(`🔄 Forcing UI update after server sync (${addedFromServer + replacedFromServer} records merged)`) console.log(`🔄 Forcing UI update after server sync (${totalMerged} records merged)`)
// Increment sync version to trigger React re-render // Increment sync version to trigger React re-render
setSyncVersion(v => v + 1) setSyncVersion(v => v + 1)
} }