Skip to content

Conversation

@lidel
Copy link
Member

@lidel lidel commented Dec 9, 2025

This PR is a follow-up for discussions triggered by #10987.

Changes

  • add TTY auto-detection for progress display (matching dag export)
  • use single-line progress with carriage return instead of flooding
  • show human-readable sizes alongside raw bytes in summary
  • update --progress flag to be auto-detected by default

Demo

progress format: Fetched/Processed N blocks, M bytes (X MB)
summary format: Total Size: 99 (99 B)

- add TTY auto-detection for progress display (matching `dag export`)
- use single-line progress with carriage return instead of flooding
- show human-readable sizes alongside raw bytes in summary
- update --progress flag to be auto-detected by default

progress format: `Fetched/Processed N blocks, M bytes (X MB)`
summary format: `Total Size: 99 (99 B)`
@lidel lidel requested a review from hsanjuan December 9, 2025 21:43
@lidel lidel mentioned this pull request Dec 9, 2025
56 tasks
@lidel lidel marked this pull request as ready for review December 10, 2025 00:13
@lidel lidel requested a review from a team as a code owner December 10, 2025 00:13
@lidel lidel added the status/ready Ready to be worked label Dec 17, 2025
Copy link
Contributor

@gammazero gammazero left a comment

Choose a reason for hiding this comment

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

LGTM

…provements

# Conflicts:
#	docs/changelogs/v0.40.md
@lidel lidel self-assigned this Jan 9, 2026
@lidel lidel merged commit 5288946 into master Jan 9, 2026
18 checks passed
@lidel lidel deleted the feat/dag-stat-ux-improvements branch January 9, 2026 17:41
@lidel lidel mentioned this pull request Jan 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status/ready Ready to be worked

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants