Skip to content

feat(cmd): Add progress spinner during analysis#87

Merged
Joseph94m merged 1 commit intogetplumber:mainfrom
stephrobert:feat/progress-spinner
Mar 3, 2026
Merged

feat(cmd): Add progress spinner during analysis#87
Joseph94m merged 1 commit intogetplumber:mainfrom
stephrobert:feat/progress-spinner

Conversation

@stephrobert
Copy link
Contributor

Closes #64

What

Adds a visual progress indicator during plumber analyze, showing which step is running with an animated spinner and progress bar.

Changes

  • cmd/spinner.go: braille-animated spinner with progress bar (█/░)
  • configuration/configuration.go: ProgressFunc callback to decouple UI from logic
  • control/task.go: reportProgress/clearProgressLine helpers, 12 progress steps
  • cmd/analyze.go: wires spinner (only when not verbose)
  • Logrus hook prevents log messages from interleaving with spinner

Demo

Capture d'écran 2026-03-03 151144

Add a visual progress indicator that shows analysis steps while running.
The spinner displays a braille animation with a progress bar and step
description, providing user feedback during potentially long API calls.

- Add progressSpinner in cmd/spinner.go with animated braille frames
- Add ProgressFunc callback in Configuration to decouple UI from logic
- Add reportProgress/clearProgressLine helpers in control/task.go
- Wire spinner in cmd/analyze.go (only when not verbose)
- Install logrus hook to prevent log interleaving with spinner

Closes getplumber#64
@Joseph94m Joseph94m merged commit 80b1bad into getplumber:main Mar 3, 2026
7 checks passed
@Joseph94m
Copy link
Collaborator

Thanks @stephrobert

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.

[FEAT] Add something to let know something is happening during analysis

2 participants