Repository navigation
Conversation
…fix/session-persistence-r05 # Conflicts: # .project/PROGRESS.md # src/components/App/App.vue
hvab
marked this pull request as ready for review
October 18, 2026 19:00
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.
Preserve pending edits after Reset and on page leave, without allowing an untouched older tab to overwrite another tab's newer saved theme.
The original R05 shared flush remains debounced during editing and synchronous on pagehide, hidden visibility and before unmount. Each App now compares its current session with its restored or last successfully written snapshot. Idle flushes do not write; comparison happens synchronously so an edit before the async watcher is still saved. Explicit Reset invalidates the snapshot before writing defaults; valid URL imports retain save intent. saveSession reports success/failure, so a failed write is retried rather than marked saved. Session format, key and existing last-writer behavior for actual concurrent edits are retained.
R02 954f71d and main baseline 8662ce5 are already incorporated. Current shell, radio groups, Reka overlays, Share feedback, separate locale preference and concrete light/dark session policy stay intact. No old UI/system mode returns.
Ubuntu/Node 22 CI passed on exact head e785be0, including clean dependency installation and all existing quality commands. The branch is mergeable against main. Local quality commands and production build pass. The temporary two-tab reproduction failed before the correction and passes now; it also verifies immediate exit before the watcher, repeated idle flush, repeated default Reset, quota retry and explicit default-state URL import. No repository test infrastructure was added.
The actual resolved R03/R05/hvab-blocks 0.3 integration is published separately as PR #6, with its own successful exact-head CI. That PR retains measured panel sizing, container/Field adaptation, body font and a single AppLayers owner. This focused R05 branch retains the original 0.2 library pin.
Limits: local npm Git-package restrictions were not changed; local checks used matching installed dependencies and the exact pinned library source, while CI confirmed a fresh install. No new Safari/BFCache, Windows/forced-colors, screen-reader or OS-crash durability coverage is claimed. No theme/export/preview contract, main/develop push, PR merge or deployment change is included.