fix: migrate CLIs to alpha version of clack#43
Closed
ascorbic wants to merge 10 commits into
Closed
Conversation
- Upgrade @clack/prompts from 0.11.0 to 1.0.0-alpha.9 in both packages
- Replace custom progress bar implementation with clack's progress() API
- Simplifies blob transfer progress tracking in migration command
- Progress bar now uses native clack rendering with advance() method
The alpha version provides a cleaner API for progress tracking:
- progress({ max }) initialization
- start(message) to begin
- advance(count) to update progress
- stop(message) to complete
This removes ~10 lines of custom progress bar rendering code.
Replace custom info box hacks with alpha's native box() prompt: - Remove brightNote() ANSI escape workarounds - Use box() with titleAlign for centered titles - Consistent formatting across all CLIs Add taskLog for transient setup steps: - Secret generation in `pds init` (clears on success) - Project setup in `create-pds` (template copy, git init) - Reduces visual noise while maintaining reassurance Use spinner.error() for failure states: - Replaces spinner.stop() on errors across all commands - Clearer semantic distinction between success/failure - Better visual feedback with error icons Goals: reassurance during operations, delight with polished UX
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ✅ Deployment successful! View logs |
atproto-pds | a6d5707 | Jan 01 2026, 07:55 PM |
commit: |
Add vibrant colors to the PDS setup completion box: - Cyan labels for configuration items - Bold handle and auth token for emphasis - Dimmed DIDs and keys for less critical info - Green checkmark for success states - Yellow highlight for important auth token - Blue link for migration docs Makes the setup completion more delightful and easier to scan! ✨
The progress bar was showing a spinner without progress info. Now displays '123/456 images transferred' as it works, with failure count when errors occur. Fixes the 'just spinning' issue during blob migration.
Set progress bar to use 'block' style (█) instead of line style. Also set width to 30 characters for better visibility. Available styles: 'light' (─), 'heavy' (━), 'block' (█)
Enhance visual hierarchy and user experience by adding picocolors styling: - Cyan for domains and technical identifiers (targetDomain, sourceDomain) - Green for success states (authenticated, imported, activated) - Yellow for warnings and important counts - Bold for handles and important identifiers - Consistent use of spinner.error() for failure states This complements the previous UX improvements (box, taskLog, progress bar) and adds visual delight throughout the migration, activation, and deactivation workflows.
The 'heavy' style (━) is more visible than 'block' (█) with the default magenta color used by @clack/prompts progress bar.
Block style (█) is more visible than heavy style (━) in most terminals. Also includes Prettier formatting fixes (tabs instead of spaces).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.