Overview

This tracking issue covers refactoring the spinner implementation in pkg/console/spinner.go to follow idiomatic Bubble Tea patterns instead of the current anti-pattern of manual goroutine management.

Source: Discussion #9188 (Go Fan Bubble Tea Report)

Problem

The current spinner implementation uses a custom goroutine with a manual update loop, bypassing Bubble Tea's rendering pipeline. This leads to:

• Manual mutex management requirements
• Potential race conditions
• Inability to leverage framerate optimization
• Difficulty in testing
• Non-standard architecture

Planned Tasks

This work is broken down into focused, sequential tasks:

1. Research and design - Evaluate idiomatic Bubble Tea patterns for inline spinners
2. Implement new spinner - Create new implementation using proper Bubble Tea patterns
3. Add comprehensive tests - Ensure spinner behavior is correct in all scenarios
4. Update documentation - Document TTY detection and usage patterns
5. Migrate and remove old code - Switch all usage to new implementation and clean up

Success Criteria

• Spinner uses tea.NewProgram() or direct component usage (no custom goroutines)
• No manual mutex management required
• TTY detection still works correctly
• Accessibility support maintained
• All existing spinner usage continues to work
• Comprehensive test coverage added
• ~30% reduction in code complexity

Related

• Discussion: [go-fan] Go Module Review: charmbracelet/bubbletea #9188
• Files: pkg/console/spinner.go

AI generated by Plan Command for discussion #9188