ClassroomCopilot/app
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(canvas): move ensureStoreIsUsable() to AFTER snapshot load — loadSnapshot() overwrites TLINSTANCE before it could be pre-initialized
Some checks failed
app-ci-deploy / test-build-deploy (push) Has been cancelled
Details

Browse Source
This commit is contained in:
CC Worker 2026-06-01 06:13:49 +00:00
parent ba829be8e9
commit 6cba932146
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/pages/tldraw/singlePlayerPage.tsx
View File

@ -98,9 +98,6 @@ export default function SinglePlayerPage() {
shapeUtils: allShapeUtils, shapeUtils: allShapeUtils,
bindingUtils: allBindingUtils bindingUtils: allBindingUtils
}); });
// Pre-initialize tldraw's required records (TLINSTANCE, page, cameras)
// so computed signals that read currentPageId don't crash before Editor construction completes
(newStore as unknown as { ensureStoreIsUsable(): void }).ensureStoreIsUsable();
const snapSvc = new NavigationSnapshotService(newStore, editorRef.current || undefined); const snapSvc = new NavigationSnapshotService(newStore, editorRef.current || undefined);
if (accessToken) snapSvc.setAccessToken(accessToken); if (accessToken) snapSvc.setAccessToken(accessToken);
@ -127,6 +124,10 @@ export default function SinglePlayerPage() {
return; return;
} }
// After snapshot loading (which may overwrite or clear store records),
// ensure TLINSTANCE + page + cameras exist so tldraw never crashes on empty/corrupt store
(newStore as unknown as { ensureStoreIsUsable(): void }).ensureStoreIsUsable();
let debounce: ReturnType<typeof setTimeout> | null = null; let debounce: ReturnType<typeof setTimeout> | null = null;
newStore.listen(() => { newStore.listen(() => {
if (!snapshotServiceRef.current?.getCurrentNodePath()) return; if (!snapshotServiceRef.current?.getCurrentNodePath()) return;