Skip to content

ui: request persistent storage on startup - #31299

Merged
ggerganov merged 1 commit into
masterfrom
allozaur/feat/ui-request-persistent-storage
Oct 29, 2026
Merged

ggerganov merged 1 commit into
masterfrom
allozaur/feat/ui-request-persistent-storage

Conversation

@allozaur

Copy link
Copy Markdown
Contributor

Overview

Conversations and settings live in IndexedDB and localStorage, which browsers treat as best-effort storage and may evict under storage pressure (Safari also clears it for sites not visited recently). The UI now calls navigator.storage.persist() once on startup, skipping it when storage is already persistent, so the browser keeps this data unless the user clears it.

Chromium grants or denies silently based on site engagement and PWA install, while Firefox shows a one-time permission prompt.

Requirements

Ask the browser to keep IndexedDB and localStorage data under storage
pressure, so conversations and settings are less likely to be evicted.

Assisted-by: Claude (Opus 5.5 Medium)
@allozaur
allozaur marked this pull request as ready for review October 28, 2026 07:52
Copilot AI balanced review requested due to automatic review settings October 28, 2026 07:52
@allozaur
allozaur requested a review from a team as a code owner October 28, 2026 07:52
@allozaur
allozaur requested review from ServeurpersoCom and ggerganov and removed request for Copilot October 28, 2026 07:52
@allozaur allozaur added the merge ready A maintainer can use this label to indicate that they consider the changes final and ready to merge. label Oct 28, 2026
@ggerganov
ggerganov merged commit b7cde58 into master Oct 29, 2026
7 checks passed
edwardyoon pushed a commit to edwardyoon/focus-llama that referenced this pull request Oct 30, 2026
Ask the browser to keep IndexedDB and localStorage data under storage
pressure, so conversations and settings are less likely to be evicted.

Assisted-by: Claude (Opus 5.5 Medium)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merge ready A maintainer can use this label to indicate that they consider the changes final and ready to merge. server/ui

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants