Skip to content

Add transaction review workflow tools#19

Closed
yonib05 wants to merge 5 commits intorobcerda:mainfrom
yonib05:feature/transaction-review
Closed

Add transaction review workflow tools#19
yonib05 wants to merge 5 commits intorobcerda:mainfrom
yonib05:feature/transaction-review

Conversation

@yonib05
Copy link
Copy Markdown

@yonib05 yonib05 commented Feb 8, 2026

Summary

This adds tools for the common workflow of reviewing new transactions that need attention.

What's New in This PR

  • get_transactions_needing_review - Fetches transactions that need review with multiple filter options:
    • needs_review flag filtering
    • Recent days filtering
    • Uncategorized transactions
    • Transactions without notes
  • set_transaction_category - Updates a transaction's category with optional auto-mark-as-reviewed
  • update_transaction_notes - Adds or updates notes on transactions (useful for receipt links)
  • mark_transaction_reviewed - Clears the needs_review flag

Also includes the category tools from my previous PR for completeness.

Why

Reviewing transactions is probably the most common task in Monarch. These tools let you quickly find transactions that need attention and process them without having to open the web UI.

Testing

Extended test coverage in tests/test_transactions.py for all new tools.
Manually verified against live account data.

@yonib05 yonib05 force-pushed the feature/transaction-review branch from bc3a3bd to 8099c26 Compare February 8, 2026 00:50
@robcerda
Copy link
Copy Markdown
Owner

Closing in favor of #29 which is a superset of this PR. Please rebase #29 against the current main to resolve conflicts. Thanks for the great work!

@robcerda robcerda closed this Mar 28, 2026
harshitbshah pushed a commit to harshitbshah/monarch-mcp-server that referenced this pull request Apr 14, 2026
Co-authored-by: Istvan Varga-Haszonits <vargahis@Istvans-MacBook-Pro.local>
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
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