Add storage balloon pre-check with actionable error messages because browsers lie about storage space availability. - #231
Closed
nelsonjchen wants to merge 4 commits into
Closed
nelsonjchen wants to merge 4 commits into
nelsonjchen wants to merge 4 commits into
Conversation
Contributor
Author
Contributor
Author
|
Needs a refactor to unslop. too many lines. |
nelsonjchen
force-pushed
the
codex/storage-balloon-test
branch
from
August 24, 2026 06:07
976a0cf to
a7dba0d
Compare
Contributor
Author
|
Hints for people testing: Ask your agent to make a sparse file to load some weight. It's a lot faster than making a real file to chew space. |
Contributor
Author
Contributor
Author
|
instances where this helped where a user's hdd was full https://discord.com/channels/469524606043160576/1540488202127806494/1540519057323720754 |
nelsonjchen
force-pushed
the
codex/storage-balloon-test
branch
from
September 15, 2026 02:19
a7dba0d to
b6af59a
Compare
Contributor
Author
|
Closing in favor of #233 |
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.


Summary
Browsers can report enough available storage while still failing when a large flash image is written to persistent site storage. This adds a storage reservation check as the first step in the existing flashing flow, before device selection.
If the check fails, the UI gives focused recovery guidance:
chrome://settings/content/siteData, including settings that delete data when the browser closes.The existing skip action remains available for users who know they can proceed. Holding Shift while skipping exposes the failure UI for testing.
Screenshot
Context
Validation
pnpm exec vitest run src/utils/image.test.js src/app/App.test.jsxpnpm run buildImplementation note
The existing image-size estimate is not sufficient on its own because browser storage APIs can overstate what the page can actually reserve.