Skip to content

Add progress and success output to remote install/update - #5

Merged
imattau merged 1 commit into
masterfrom
add-remote-install-progress-output
Sep 20, 2026
Merged

imattau merged 1 commit into
masterfrom
add-remote-install-progress-output

Conversation

@imattau

@imattau imattau commented Sep 20, 2026

Copy link
Copy Markdown
Owner

Summary

  • Remote install-ref / install <ref> / update previously printed nothing during the whole resolve-download-install pipeline, only a terse install order: ... line at the very end. On a slow relay or Blossom lookup (up to the existing network timeout) this looked hung, with no way to tell it was making progress or that anything actually happened.
  • Adds stderr progress lines -- Connecting to N relay(s)... done in Xs, Resolving <publisher>/<name>..., Downloading <publisher>/<name> <version> (N mirror(s))... done in Xs (N bytes) -- matching the style npack search already uses, so stdout stays script-friendly.
  • Adds a stdout installed <publisher>/<name> <version> line per package on success, matching the wording local .npk installs already use, in addition to the existing install order: ... summary.
  • Verified live against a real relay (wss://relay.damus.io) with a nonexistent package -- connect/resolve progress prints correctly and the error is clear.

Test plan

  • cargo fmt --check
  • cargo clippy --all-targets
  • cargo test (37 passed, no behavior change to covered paths)
  • Manual smoke test against a real relay showing the new progress/error output
  • docs/npack.1 regenerated via npack man (no diff)

🤖 Generated with Claude Code

Remote install-ref/install/update previously printed nothing until a
single terse "install order: ..." line at the very end, making a slow
relay or Blossom fetch look hung with no confirmation that anything
happened. Add stderr progress (connecting, resolving, downloading,
matching npack search's existing style) and a per-package "installed
<publisher>/<name> <version>" stdout line on success, matching the
wording already used by local .npk installs.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@imattau
imattau merged commit e4bb1c6 into master Sep 20, 2026
3 checks passed
@imattau
imattau deleted the add-remote-install-progress-output branch September 20, 2026 23:35
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.

1 participant