Skip to content

core: skip preload notice on first run - #303

Merged
BenWestgate merged 1 commit into
masterfrom
claude/project-thread-nk99yf
Oct 3, 2026
Merged

BenWestgate merged 1 commit into
masterfrom
claude/project-thread-nk99yf

Conversation

@BenWestgate

Copy link
Copy Markdown
Owner

Requested by Ben · project thread

Before: on a fresh install, "Chainstate preloaded to RAM" pops up right after Bitcoin Core first starts, because the empty chainstate "loads" instantly.

After: the notice is skipped when the chainstate was under 100 MiB when chainstate-preload started. Later runs with a real chainstate still show it.

How: du -sm chainstate is taken once before the preload loop; the success branch exits quietly below the threshold.

Overlaps: #301 changes the icon on the same zenity line (one-line conflict, whichever lands second rebases). #283 removes this notice entirely; if that merges, this PR can be closed.

🤖 Generated with Claude Code

https://claude.ai/code/session_01YWFmpKa9cgSLp61StJJax5


Generated by Claude Code

On a fresh install the chainstate folder is empty or nearly so when
Bitcoin Core first starts, so the background cat finishes at once and
the "Chainstate preloaded to RAM" notice pops up over the Welcome
dialog without telling the user anything useful.

Measure the chainstate before preloading and exit quietly when it is
under 100 MiB. Later runs, with a real chainstate to load, still show
the notice.
@chatgpt-codex-connector

This comment has been minimized.

@BenWestgate BenWestgate self-assigned this Oct 2, 2026

@BenWestgate BenWestgate left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@BenWestgate
BenWestgate merged commit e019b74 into master Oct 3, 2026
6 checks passed
@BenWestgate
BenWestgate deleted the claude/project-thread-nk99yf branch October 3, 2026 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants