Skip to content

client: polish output of connect and disconnect commands#3529

Merged
juan-malbeclabs merged 5 commits intomainfrom
jo/3528
Apr 14, 2026
Merged

client: polish output of connect and disconnect commands#3529
juan-malbeclabs merged 5 commits intomainfrom
jo/3528

Conversation

@juan-malbeclabs
Copy link
Copy Markdown
Contributor

@juan-malbeclabs juan-malbeclabs commented Apr 13, 2026

Resolves: #3528

Summary of Changes

  • Replace semantically incorrect 🔍 emoji with on action/status lines in both commands
  • Change shared header from "DoubleZero Service Provisioning" to "DoubleZero Network" (accurate for both connect and disconnect)
  • Fix disconnect using println! instead of spinner.println, which left progress bar text visible on screen
  • Normalize device-selection phrasing across IBRL and multicast code paths; remove trailing spaces in format strings
  • Indent tenant lines to match surrounding detail lines

Output

doublezero connect ibrl

DoubleZero Network
⚡  Connecting to testnet...
    DoubleZero ID: DZfHfcCXTLwgZeCRKQ1FL1UuwAwFAZM93g86NMYpfYan
⚡  Provisioning for IP: 67.213.120.9
    Creating account...
    Device selected: nyc-dz001
    Using tenant 'solana' from Access Pass.
    Tunnel Src: 67.213.120.9
    Tunnel Dst: 64.86.248.128
    DoubleZero IP: 67.213.120.9
    Session: Pending BGP Session
✅  User Provisioned

doublezero disconnect

DoubleZero Network
⚡  Disconnecting...
    Client IP: 67.213.120.9
⚡  Removing account: 8F3kvQWf2RK86DkR8zXhLKeFTmQyqXkM1fhEaJi7FfK1
    Account deletion submitted
    Tunnel confirmed removed
✅  Deprovisioning Complete

Diff Breakdown

Category Files Lines (+/-) Net
Scaffolding 3 +15 / -21 -6

Pure string/presentation changes — no logic modified.

Testing Verification

  • cargo test -p doublezero -p doublezero_cli — 290 tests pass

Fix inconsistent emoji usage, normalize message phrasing across IBRL
and multicast code paths, and clear the progress bar correctly on
disconnect.
@elitegreg elitegreg self-requested a review April 13, 2026 18:38
@juan-malbeclabs juan-malbeclabs enabled auto-merge (squash) April 13, 2026 19:27
@juan-malbeclabs juan-malbeclabs merged commit d510e3c into main Apr 14, 2026
33 checks passed
@juan-malbeclabs juan-malbeclabs deleted the jo/3528 branch April 14, 2026 18:19
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.

client: fix output formatting for connect and disconnect commands

2 participants