Skip to content

Add pluggable bank connector architecture#1065

Open
Entr0zy wants to merge 1 commit into
rohitdash08:mainfrom
Entr0zy:bank-sync-connectors
Open

Add pluggable bank connector architecture#1065
Entr0zy wants to merge 1 commit into
rohitdash08:mainfrom
Entr0zy:bank-sync-connectors

Conversation

@Entr0zy
Copy link
Copy Markdown

@Entr0zy Entr0zy commented May 22, 2026

Closes #75

Implements a pluggable backend bank-sync connector foundation:

  • Adds a BankConnector interface for provider integrations.
  • Adds ConnectorRegistry for connector lookup and registration.
  • Adds MockBankConnector as the included mock connector.
  • Adds import_bank_transactions(...) and refresh_bank_transactions(...) helpers that normalize provider rows into FinMind's existing import transaction shape.
  • Covers registry lookup, import date windows, refresh support, normalization, and validation errors with unit tests.

Validation:

  • python -m pytest tests/test_bank_connectors.py -q -> 5 passed
  • python -m py_compile app/services/bank_connectors.py tests/test_bank_connectors.py -> passed
  • git diff --check -> passed

Additional note:

  • python -m pytest tests/test_expenses.py tests/test_bank_connectors.py -q runs the new connector tests but the existing expense tests error in this local environment while attempting to connect to redis:6379 during auth setup. The failure happens before the bank connector code is exercised.

@Entr0zy Entr0zy requested a review from rohitdash08 as a code owner May 22, 2026 13:01
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.

Bank Sync Connector Architecture

1 participant