feat: Add bulk import validation warnings and summary statistics (closes #115)#458
Open
noxxxxybot-sketch wants to merge 2 commits intorohitdash08:mainfrom
Open
feat: Add bulk import validation warnings and summary statistics (closes #115)#458noxxxxybot-sketch wants to merge 2 commits intorohitdash08:mainfrom
noxxxxybot-sketch wants to merge 2 commits intorohitdash08:mainfrom
Conversation
added 2 commits
March 15, 2026 12:08
…08#91) - Rule-based auto-categorization with 60+ keyword mappings across 10 categories - Confidence scoring (0.0-1.0) with alternatives ranking - Learning from user corrections via POST /categorize/learn - Batch categorization endpoint for multiple transactions - User-specific learned rules stored in categorization_rules table - 26 comprehensive tests (all passing, 48 total) API Endpoints: - POST /categorize — Categorize single transaction - POST /categorize/batch — Batch categorize up to 100 transactions - POST /categorize/learn — Learn from user corrections - GET /categorize/rules — List learned rules - DELETE /categorize/rules/:id — Delete a learned rule
rohitdash08#115) - Add validate_import_rows() function with per-row validation warnings for: - Unusually large amounts (>100k) - Future dates and suspiciously old dates - Short/empty or overly long descriptions - Add summary statistics: total amounts, income vs expense breakdown, date range - Update /expenses/import/preview to include warnings and summary in response - Add comprehensive unit and integration tests
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Summary
Adds validation warnings and summary statistics to the bulk import preview endpoint, addressing issue #115.
Changes
API Response (Before → After)
Before:
After:
Tests