Apply rustfmt to fix formatting issues:
- Reformat nested use statement for std::sync imports
- Reformat thread::spawn closure to use inline loop syntax
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
**CRITICAL BUG FIX**: When using -l (loop), the outer loop in main.rs
was restarting the animation even after user pressed exit keys.
Changes:
- render() now returns Result<bool> instead of Result<()>
- Returns true when user presses exit key (q/ESC/Ctrl+C)
- Returns false when animation completes naturally
- main.rs checks return value and breaks loop on user exit
This fixes the infinite loop issue where pressing q/ESC/Ctrl+C
had no effect when using the -l flag.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
The issue was that sleep() was blocking for 16-33ms without checking
the exit flag. Now:
- Check should_exit at 5 points in the render loop
- Break sleep into 5ms chunks, checking between each chunk
- This gives <5ms response time to exit commands
Exit responds within 5ms: Press 'q', ESC, or Ctrl+C
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Previous attempts to poll keyboard events in the render loop were
failing. Now using a dedicated background thread that continuously
monitors for exit keys and communicates with the render loop via
an atomic boolean flag.
This ensures keyboard events are never missed, even during heavy
rendering operations.
Exit: Press 'q', ESC, or Ctrl+C
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Removed complex signal handling that wasn't working reliably.
Now using simpler event polling with 10ms timeout instead of 0ms,
which allows keyboard events to be properly detected.
Exit methods:
- Press 'q' (most reliable)
- Press ESC
- Ctrl+C (with 10ms poll window)
- Ctrl+D (alternative)
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Implemented both keyboard event and signal detection for Ctrl+C:
1. Check for Ctrl+C as keyboard event (KeyModifiers::CONTROL + 'c')
2. Check for SIGINT signal via tokio::signal::unix
This ensures Ctrl+C works reliably in both raw terminal mode and
through signal handling.
Exit options: Ctrl+C, 'q', or ESC
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Previous keyboard event polling didn't work for Ctrl+C in raw mode.
Now using tokio::signal::ctrl_c() with a background task that signals
the render loop through a channel when Ctrl+C is pressed.
Changes:
- Added "signal" and "sync" features to tokio dependency
- Spawn background task to listen for Ctrl+C signal
- Use mpsc channel to communicate signal to render loop
- Keep 'q' and ESC keyboard shortcuts for convenience
Ctrl+C now properly exits looping animations.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Fixed issue where Ctrl+C did not work to exit looping animations.
In raw terminal mode, signals are not automatically handled, so we
need to manually poll for keyboard events.
Changes:
- Added crossterm event polling in render loop
- Check for Ctrl+C, 'q', or ESC key to exit animation
- Polls with 0ms timeout to avoid blocking animation frames
Users can now exit with:
- Ctrl+C
- Press 'q'
- Press ESC
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Previously all binaries were uploaded with the same name 'piglet',
causing them to overwrite each other. Now each binary is renamed
to include the target platform (e.g., piglet-x86_64-unknown-linux-gnu)
before uploading.
This ensures all 4 platform binaries are available in the release.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Added 'contents: write' permission to allow GITHUB_TOKEN to create
releases. This fixes the 403 error when the workflow tries to create
a GitHub release.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Rustfmt requires long chained method calls to be broken across
multiple lines when they exceed line length limits.
Changes:
- Break visual_width() iterator chains in renderer.rs
- Break visual_width() iterator chains in terminal.rs
- Fix comment alignment in ansi.rs
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Windows support is removed from CI workflows:
- Removed windows-latest from test matrix
- Removed x86_64-pc-windows-msvc from build targets
- Simplified artifact upload path (no .exe conditional)
CI now only runs on Ubuntu and macOS.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
The motion effects were appearing with incorrect row offsets because
the text width calculation included ANSI color escape sequences in
the byte length, causing misaligned centering.
Changes:
- Added utils/ansi.rs module with strip_ansi() and visual_width()
- Updated renderer to use visual_width() for offset positioning
- Updated TerminalManager::print_centered() to use visual_width()
- All text positioning now correctly ignores ANSI escape sequences
This fixes the "shifted in the rows" issue where colored text was
not properly centered due to escape sequence byte counts.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Previously, the animation loop would exit before rendering the final
frame at progress=1.0. The loop checked is_complete() before rendering,
so the last visible frame was at (total_frames-1)/total_frames progress
(~96-97%).
Changed the loop to:
1. Render the current frame
2. Check if complete and break
3. Advance to next frame
This ensures the final frame at progress=1.0 is rendered before exiting,
completing the animation properly.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Windows figlet installation was unreliable in CI. Tests will
gracefully skip when figlet is not available.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Chocolatey installation was failing in CI. Winget is built into
Windows and provides a more reliable installation method.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
- Update actions/upload-artifact from v3 to v4
- Update actions/dependency-review-action from v3 to v4
- Fix deprecated action warnings
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
- Implement complete animation system with 20+ motion effects
- Add 18+ easing functions (quad, cubic, elastic, back, bounce)
- Implement color system with palette and gradient support
- Add parser for durations, colors, and CSS gradients
- Create comprehensive test suite (14 tests passing)
- Add linting and formatting with clippy/rustfmt
- Support for figlet integration with custom fonts
- Terminal rendering with crossterm
- Fix all clippy warnings and lint issues
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
