-
Notifications
You must be signed in to change notification settings - Fork 1
feat: Implement Dave debt tracking CLI with full feature set #1
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
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Add complete debt tracking system with SQLite storage - Implement snowball, avalanche, and manual sorting modes - Add payment tracking with interest/principal breakdown - Support optional payment dates (yyyy-mm-dd format) - Auto-increment snowball when debts are paid off - Support order number or name for all commands - Add beautiful terminal UI with Lipgloss tables - Hide paid debts from UI while preserving in database - Add reset command to clear all data with confirmation - Include GitHub Actions for PR testing and releases - Add comprehensive test suite (20 test functions) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Sonnet 4.5 <[email protected]>
CRITICAL SECURITY FIXES: - Fix SQL injection vulnerability by replacing string concatenation with explicit query selection - Restrict database directory permissions from 0755 to 0700 (owner-only) HIGH PRIORITY FIXES: - Add database connection cleanup on application exit - Add input validation to trim whitespace from all user inputs - Fix duplicate GetSettings call in pay command - Add database indexes on current_balance and apr columns for improved query performance CODE QUALITY IMPROVEMENTS: - Define constants for magic numbers (MaxProjectionMonths, FloatingPointTolerance, MonthsPerYear) - Update Go version to 1.25 in GitHub Actions workflows - Add comprehensive TODO.md documenting remaining improvements All tests passing (20/20). 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Sonnet 4.5 <[email protected]>
✅ Test Results: PASSEDTest Output |
- Add stylish DAVE ASCII art banner to table display - Include screenshot in README with centered layout - Add demo data setup scripts (setup-demo.bat/sh) for screenshots - ASCII art uses cyan color (lipgloss color 86) for visual appeal 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Sonnet 4.5 <[email protected]>
✅ Test Results: PASSEDTest Output |
✅ Test Results: PASSEDTest Output |
Owner
Author
|
Vibe Coded, YOLO, YMMV, NO Warranty, I Just wanted to replace an excel spreadsheet with a cli LOL |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🤖 Generated with Claude Code