Fix study session persistence and add recovery tests#26
Closed
geokoko wants to merge 7 commits intofix/icons-planner-reload-bugsfrom
Closed
Fix study session persistence and add recovery tests#26geokoko wants to merge 7 commits intofix/icons-planner-reload-bugsfrom
geokoko wants to merge 7 commits intofix/icons-planner-reload-bugsfrom
Conversation
8804a1f to
cee0980
Compare
Owner
Author
Manual soak-test plan before merging into the parent PRGoal:Validate that recent persistence/sync fixes hold up under normal usage, suspend/resume, cross-device sync, UI refreshes, and overnight active-session recovery. Scenarios to verify
Record for each test:
Merge only if:
|
This was
linked to
issues
Apr 7, 2026
Owner
Author
Soak-test update as of 2026-04-09Passed:
Still failing:
Not yet tested:
Conclusion:
|
Owner
Author
|
Superseded by #28. Closing this PR in favor of the rewrite branch, which replaces the hot-reload approach with staged Drive download/apply and verified upload guards. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fix study session persistence and add recovery safeguards
Description
A bug was reported where study sessions and other recent changes could appear saved in the UI but were not reliably durable in the local database file. In the reported flow, the app later uploaded or reloaded a stale database snapshot, which caused the missing data to reappear as lost after restart or cross-device sync.
This PR hardens the local persistence and Drive sync flow around study sessions, improves recovery/visibility of active and incomplete session state, and adds regression coverage for the affected paths.
What changed
Why
The goal is to prevent cases where the UI implies a session was saved, but the local database file or uploaded Drive snapshot does not actually contain the latest committed session state.
Testing