Skip to content

Conversation

@bircni
Copy link
Owner

@bircni bircni commented Jul 16, 2025

No description provided.

@bircni bircni requested a review from Copilot July 16, 2025 16:35
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Adds a fully interactive terminal UI for repository scanning and Git operations, updates status coloring to support both table and TUI modes, and adjusts CLI flags and dependencies to enable the new feature.

  • Introduces a --interactive flag and the InteractiveMode module built with crossterm and ratatui.
  • Refactors status coloring by renaming color to comfy_color, adding ratatui_color, and updating all consumers.
  • Adjusts CLI short options (-s, -n) and updates snapshots; adds new dependencies (crossterm, ratatui, and castaway).

Reviewed Changes

Copilot reviewed 11 out of 12 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
typos.toml Added “ratatui” to the typo correction list.
src/cli.rs Added interactive flag, rebased short options for summary, subdir, and non_clean.
src/main.rs Imported and invoked InteractiveMode when --interactive is set.
src/gitinfo/status.rs Renamed color to comfy_color, introduced ratatui_color.
src/printer.rs Updated status cells to use comfy_color.
src/tests/snapshots/.../git-statuses.snap Updated snapshots for new short flags and interactive option.
src/interactive/* New interactive mode implementation and UI rendering code.
Cargo.toml Added dependencies for crossterm, ratatui, and castaway.
Comments suppressed due to low confidence (3)

src/cli.rs:55

  • No tests validate the new interactive mode flag. Consider adding an integration or unit test to ensure --interactive correctly enters interactive mode and that fallback behavior remains unchanged.
    pub interactive: bool,

src/gitinfo/status.rs:123

  • [nitpick] The newly added ratatui_color method lacks a doc comment. Adding a brief description of its mapping from status variants to TUI colors would improve code clarity.
    pub const fn ratatui_color(&self) -> ratatui::style::Color {

Cargo.toml:29

  • The castaway dependency is not referenced anywhere in the codebase. If it's not required for interactive mode or other features, consider removing it to reduce dependency bloat.
castaway = "0.2.4"

@bircni bircni force-pushed the feat/interactive-mode branch from 5f544bf to 12d48e4 Compare July 16, 2025 16:59
@bircni bircni merged commit 4713543 into main Jul 18, 2025
4 of 5 checks passed
@bircni bircni deleted the feat/interactive-mode branch July 18, 2025 15:05
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.

2 participants