-
Notifications
You must be signed in to change notification settings - Fork 0
Simple interactive terminal user interface (without brick) #8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
b52e1c5 to
bd68f40
Compare
TODO
|
c0db88b to
d0d96de
Compare
|
TODO
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR introduces a simple interactive terminal user interface (TUI) for the time tracking tool, replacing the previous brick-based approach with haskeline for a lighter-weight solution.
Key Changes:
- Replaced the brick UI library with haskeline for terminal interactions
- Added a new TUI executable (
t5) with interactive prompts for clocking in/out and viewing reports - Enhanced the completion system with haskeline integration and case-insensitive matching
Reviewed Changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| terminal-time-tracking-tool.cabal | Replaced brick dependency with haskeline and added new t5 executable |
| t4-tui/TUI.hs | New interactive TUI implementation using haskeline for user input and completion |
| lib/Completion.hs | Added haskeline integration functions and made completion matching case-insensitive |
| lib/T4/Report.hs | Extracted showDurMap function to format duration maps as strings |
| t4-commands/Commands.hs | Simplified report command logic and moved formatting to T4.Report module |
| lib/Util.hs | Added lastMaybe helper function for safe list operations |
| test/UtilSpec.hs | Added tests for the new lastMaybe function |
| test/T4/ReportSpec.hs | Added tests for showDurMap text report formatting |
| test/CompletionSpec.hs | Updated tests for case-insensitive matching and added haskeline completion tests |
| README.md | Added documentation for running the new t5 TUI |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Replaces #2