Repository navigation
Restore the last opened file and editor mode on refresh - #22
Conversation
DebelToni
left a comment
There was a problem hiding this comment.
Reviewed the complete four-file diff at bbc81bb against current main 3c0da06. No blocking findings.
Source preference stores only file ID/mode under an unambiguous account/project key. Restore selects an ID from the current authorized file list before fetching it, preserving rename behavior and falling back to the existing default/first-file choice after deletion. Persistence follows committed selection/mode state after initial loading; failed navigation does not replace the previous preference. Reviewed the surrounding navigation guards and session/route lifecycle. New helpers handle malformed or unavailable storage without making storage authoritative for file access.
Regression tests cover both modes across remount, stable IDs after rename, deletion fallback and its persistence, failed selection, account/project key isolation and malformed/blocked storage at the helper boundary. No server/API/permission, credential/content storage, dependency/workflow, deployment or host-automation changes. Exact-head hosted test/analyze and CodeQL passed. No contributor code, tests, installers or builds executed on DGX; no independent browser refresh test is claimed.
Summary
Verification
pnpm checkpassed (lint, typecheck, full tests, production build).pnpm test:compilerpassed with real Docker.NODE_OPTIONS=--no-experimental-webstorageso jsdom owns test localStorage.No server, production data, deployment configuration, or host automation changes.