Skip to content

Conversation

@pyramation
Copy link
Contributor

Summary

Adds animated spinner feedback to the pgpm install command for a better user experience. The spinner shows progress during:

  1. Checking for missing modules (when called without arguments)
  2. Installing modules (both explicit packages and auto-detected missing modules)

Uses the createSpinner function from inquirerer 4.2.0 (already available from the previous upgrade PR).

Review & Testing Checklist for Human

  • Test spinner visually: Run pgpm install @pgpm/base32 in a real terminal and verify the spinner animates correctly
  • Test auto-install flow: In a module with missing dependencies in .control, run pgpm install without arguments and verify both spinners appear (checking → installing)
  • Verify non-TTY behavior: Run in a non-interactive environment (e.g., pipe output) to ensure spinner degrades gracefully

Recommended test plan:

cd /path/to/pgpm/module
pgpm install @pgpm/base32  # Should show spinner during install
pgpm install               # Should show spinner for checking + installing

Notes

- Add spinner while checking for missing modules
- Add spinner while installing modules
- Show success message with count of installed modules
@devin-ai-integration
Copy link
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@pyramation pyramation merged commit 500884b into main Dec 27, 2025
35 checks passed
@pyramation pyramation deleted the devin/1766840097-install-spinner branch December 27, 2025 13:17
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