Use MiniDot spinner style with ACCESSIBLE environment variable support

[Copilot]

The spinner implementation was already using Bubbles but only supported the Dot style and lacked accessibility controls. This changes the spinner to use MiniDot style and adds ACCESSIBLE environment variable support.

Changes

Spinner Style:

• Changed from Dot style to MiniDot style (⣾ ⣽ ⣻ ⢿ ⡿ ⣟ ⣯ ⣷)
• NewSpinner() creates MiniDot spinners - backward compatible API

Accessibility:

• Spinner disabled when ACCESSIBLE environment variable is set
• Combined with existing TTY detection for complete accessibility support

Testing:

• Accessibility mode test
• Concurrent access and multiple start/stop cycle tests
• All tests passing

Documentation:

• Package-level docs with usage examples
• MiniDot spinner documentation in console README
• Accessibility and TTY detection behavior

Usage

// Creates MiniDot spinner (backward compatible API)
spinner := console.NewSpinner("Loading...")
spinner.Start()
// operation
spinner.Stop()

# Accessibility mode
ACCESSIBLE=1 gh aw compile workflow.md  # Spinners disabled

Files Modified

• pkg/console/spinner.go - Simplified implementation with MiniDot and ACCESSIBLE support
• pkg/console/spinner_test.go - Test coverage for core functionality
• pkg/console/README.md - Spinner documentation with MiniDot style

Original prompt

---

This section details on the original issue you should resolve

<issue_title>[plan] Migrate spinner to Bubbles spinner models</issue_title>
<issue_description>## Objective

Replace the current spinner implementation in pkg/console/spinner.go with github.com/charmbracelet/bubbles/spinner for smoother animations and more spinner style options.

Context

The Terminal Stylist analysis (#8747) identified that while our current spinner works, Bubbles provides more sophisticated animations (dots, line, miniDots, etc.) that enhance user experience during long-running operations.

Approach

1. Import github.com/charmbracelet/bubbles/spinner in pkg/console/spinner.go
2. Create a Bubble Tea model that wraps the Bubbles spinner
3. Maintain the existing Spinner API for backward compatibility
4. Add support for multiple spinner styles (dots, line, miniDots, etc.)
5. Ensure TTY detection works properly (no animations in non-TTY environments)
6. Preserve accessibility support (ACCESSIBLE, NO_COLOR environment variables)

Files to Modify

• Update: pkg/console/spinner.go - Replace spinner implementation
• Update: pkg/console/spinner_test.go - Update tests for new implementation
• Update: go.mod - Add Bubbles dependency if not already present

Acceptance Criteria

• Spinner uses Bubbles spinner models internally
• Existing Spinner API continues to work (backward compatible)
• Multiple spinner styles are available (dots, line, miniDots)
• Spinner animations are smoother than before
• TTY detection prevents animations in pipes/redirects
• Accessibility mode disables animations (ACCESSIBLE env var)
• Tests cover TTY and non-TTY environments
• No breaking changes to existing spinner usage

Example Usage

// After implementation
s := console.NewSpinner("Loading...")
s.Start()
// Long-running operation
s.Stop()

Related to #8748

AI generated by Plan Command for discussion #8747

Comments on the Issue (you are @copilot in this section)

• Fixes [plan] Migrate spinner to Bubbles spinner models #8749

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[pelikhan]

@copilot Only support spinnerminidot

[Copilot]

Simplified to only support MiniDot spinner style in commit 5a3f93f. Removed all other spinner styles (Dot, Line, Points, Globe, Moon, Jump, Pulse, Ellipsis) and the NewSpinnerWithStyle() function. The NewSpinner() function now creates MiniDot spinners by default.

[pelikhan]

@copilot Lint

[Copilot]

@copilot Lint

Linting complete - all checks pass:

• make lint: ✓ 0 issues
• make golint: ✓ 0 issues
• make fmt-check: ✓ All files properly formatted

No code changes needed.