Skip to content

Add resume capability and enhanced logging - #9

Open
mdinic wants to merge 1 commit into
Fallenstedt:masterfrom
mdinic:add-resume-capability-and-logging
Open

Add resume capability and enhanced logging#9
mdinic wants to merge 1 commit into
Fallenstedt:masterfrom
mdinic:add-resume-capability-and-logging

Conversation

@mdinic

@mdinic mdinic commented Jan 5, 2026

Copy link
Copy Markdown

Summary

This PR adds the ability to resume interrupted downloads by checking if files already exist locally and comparing file sizes. It also adds detailed logging to track download progress.

Changes

  • Check if files exist before downloading
  • Compare local and remote file sizes to skip already-downloaded files
  • Re-download files with size mismatches (handles incomplete downloads)
  • Add detailed logging for each file showing:
    • Processing status
    • File existence and size comparison
    • Skip/download/re-download decisions
    • Download start and completion

Benefits

  • Saves bandwidth by not re-downloading existing files
  • Allows resuming interrupted download sessions
  • Handles incomplete downloads by detecting size mismatches
  • Enhanced visibility into download progress with detailed logging
  • No changes to CLI interface - works transparently

Testing

Tested by running downloads, interrupting them, and resuming - verified that already-downloaded files are skipped and incomplete files are re-downloaded.

- Add file existence check before downloading
- Compare local and remote file sizes to skip already-downloaded files
- Re-download files with size mismatches (incomplete downloads)
- Add detailed logging for each file:
  - Processing status
  - File existence and size comparison
  - Skip/download/re-download decisions
  - Download start and completion

This allows resuming interrupted downloads and provides better visibility
into download progress. Logs are written to the existing info log.
@francoisbruneau

francoisbruneau commented Feb 22, 2026

Copy link
Copy Markdown

Super useful! Looking forward for this to be merged :)

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