Skip to content

link-dotfiles: wait 3 min before no-peer prompt - #304

Merged
BenWestgate merged 1 commit into
masterfrom
claude/project-thread-r1ge5y
Oct 2, 2026
Merged

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

Conversation

@BenWestgate

Copy link
Copy Markdown
Owner

Requested by Ben · project thread

Before: "Bitcoin Core has no peers" popped up after one check 30 seconds after Core finished starting. Over Tor the first peer often takes longer, so users were told to open Tor Connection while everything was fine.

After: the prompt only appears if Core has had zero peers for 3 full minutes after startup. If any peer connects (or Core stops answering RPC) during that time, nothing is shown.

How: the --core-startup background check in bails/.local/bin/link-dotfiles now polls bitcoin-cli getnetworkinfo every 15 seconds for up to 180 seconds and exits early on the first nonzero connection count. Dialog text and the Open Tor Connection action are unchanged.

Tested with bash -n, shellcheck, and a stubbed bitcoin-cli/zenity run: with no peers the dialog appears after the full window (6 polls); with a peer on the 3rd poll it exits without a dialog. Not tested on real Tails.

No overlap with #293 or #301 (neither touches link-dotfiles).

🤖 Generated with Claude Code

https://claude.ai/code/session_012s4QWDRFD7xomyKNn1Zd8N


Generated by Claude Code

The "Bitcoin Core has no peers" prompt fired after a single check 30
seconds after Core finished starting. Over Tor the first peer often
takes longer than that, so users were told to open Tor Connection
while everything was working normally.

Poll getnetworkinfo every 15 seconds instead and only show the prompt
if Core has had no peers for the full 3 minutes. Stop watching as
soon as a peer connects or Core stops answering RPC.
@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 94309c5 into master Oct 2, 2026
6 checks passed
@BenWestgate
BenWestgate deleted the claude/project-thread-r1ge5y branch October 2, 2026 06:37
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