feat: make API URL configurable and improve code quality

- Add configurable API URL support via environment variables - Update nginx configuration with environment variable substitution - Create nginx template for dynamic proxy configuration - Update Docker configuration for environment variable handling - Fix hardcoded localhost:8000 references in error messages - Add proper TypeScript types for health check API - Format all code with Prettier for consistency - Update documentation with configuration instructions - Improve error messages to be environment-agnostic - Fix duplicate imports and type safety issues BREAKING: API URL is now configurable via VITE_API_URL (dev) and API_BACKEND_URL (prod)
feat: add notifications view and update branding
2025-12-28 21:59:14 +00:00 · 2025-09-09 19:32:00 +01:00 · 2025-09-09 19:32:00 +01:00 · 2025-09-09 19:32:00 +01:00 · 2025-09-09 19:32:00 +01:00 · 2025-09-09 19:32:00 +01:00
12 changed files with 48 additions and 311 deletions
--- a/AGENTS.md
+++ b/AGENTS.md
@@ -38,5 +38,4 @@
 ### General
 - **Formatting**: ruff for Python, ESLint for TypeScript
 - **Commits**: Use conventional commits, run pre-commit hooks before pushing
-  - Avoid including specific numbers, counts, or data-dependent information that may become outdated
 - **Security**: Never log sensitive data, use environment variables for secrets
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,128 +1,4 @@

-## 2025.9.1 (2025/09/09)
-
-### Bug Fixes
-
-  Handle duplicate transactionId values in migration ([8fabaf7b](https://github.com/elisiariocouto/leggen/commit/8fabaf7b86fde921c61266568ecb0403d3102671))
-
-
-### Miscellaneous Tasks
-
-  Improve AGENTS.md. ([3270dc45](https://github.com/elisiariocouto/leggen/commit/3270dc4585e6b33d55aef0deecd849753d36fa74))
-
-
-### Refactor
-
-  Remove unused hide_missing_ids functionality ([8006e5e1](https://github.com/elisiariocouto/leggen/commit/8006e5e1f6373aae39d3c38068d694e142bc85a5))
-
-
-
-## 2025.9.1 (2025/09/09)
-
-### Bug Fixes
-
-  Handle duplicate transactionId values in migration ([8fabaf7b](https://github.com/elisiariocouto/leggen/commit/8fabaf7b86fde921c61266568ecb0403d3102671))
-
-
-### Miscellaneous Tasks
-
-  Improve AGENTS.md. ([3270dc45](https://github.com/elisiariocouto/leggen/commit/3270dc4585e6b33d55aef0deecd849753d36fa74))
-
-
-### Refactor
-
-  Remove unused hide_missing_ids functionality ([8006e5e1](https://github.com/elisiariocouto/leggen/commit/8006e5e1f6373aae39d3c38068d694e142bc85a5))
-
-
-
-## 2025.9.0 (2025/09/09)
-
-### Bug Fixes
-
- **cli:** Show transactions without internal ID when using --full. ([46f3f5c4](https://github.com/elisiariocouto/leggen/commit/46f3f5c4984224c3f4b421e1a06dcf44d4f211e0))
-  Do not install development dependencies. ([73d6bd32](https://github.com/elisiariocouto/leggen/commit/73d6bd32dbc59608ef1472dc65d9e18450f00896))
-  Implement proper GoCardless authentication and add dev features ([f0fee4fd](https://github.com/elisiariocouto/leggen/commit/f0fee4fd82e1c788614d73fcd0075f5e16976650))
-  Make internal transcation ID optional. ([6bce7eb6](https://github.com/elisiariocouto/leggen/commit/6bce7eb6be5f9a5286eb27e777fbf83a6b1c5f8d))
-  Resolve 404 balances endpoint and currency formatting errors ([417b7753](https://github.com/elisiariocouto/leggen/commit/417b77539fc275493d55efb29f92abcea666b210))
-  Merge account details into balance data to prevent unknown/N/A values ([eaaea6e4](https://github.com/elisiariocouto/leggen/commit/eaaea6e4598e9c81997573e19f4ef1c58ebe320f))
-  Use account status for balance records instead of hardcoded 'active' ([541cb262](https://github.com/elisiariocouto/leggen/commit/541cb262ee5783eedf2b154c148c28ec89845da5))
-
-
-### Documentation
-
-  Update README for new web architecture ([4018b263](https://github.com/elisiariocouto/leggen/commit/4018b263f27c2b59af31428d7a0878280a291c85))
-
-
-### Features
-
-  Transform to web architecture with FastAPI backend ([91f53b35](https://github.com/elisiariocouto/leggen/commit/91f53b35b18740869ee9cebfac394db2e12db099))
-  Add comprehensive test suite with 46 passing tests ([34e793c7](https://github.com/elisiariocouto/leggen/commit/34e793c75c8df1e57ea240b92ccf0843a80c2a14))
-  Add mypy to pre-commit. ([ec8ef834](https://github.com/elisiariocouto/leggen/commit/ec8ef8346add878f3ff4e8ed928b952d9b5dd584))
-  Implement database-first architecture to minimize GoCardless API calls ([155c3055](https://github.com/elisiariocouto/leggen/commit/155c30559f4cacd76ef01e50ec29ee436d3f9d56))
-  Implement dynamic API connection status ([cb2e70e4](https://github.com/elisiariocouto/leggen/commit/cb2e70e42d1122e9c2e5420b095aeb1e55454c24))
-  Add automatic balance timestamp migration mechanism ([34501f5f](https://github.com/elisiariocouto/leggen/commit/34501f5f0d3b3dff68364b60be77bfb99071b269))
-  Improve notification filters configuration format ([2191fe90](https://github.com/elisiariocouto/leggen/commit/2191fe906659f4fd22c25b6cb9fbb95c03472f00))
-  Add notifications view and update branding ([abf39abe](https://github.com/elisiariocouto/leggen/commit/abf39abe74b75d8cb980109fbcbdd940066cc90b))
-  Make API URL configurable and improve code quality ([37949a4e](https://github.com/elisiariocouto/leggen/commit/37949a4e1f25a2656f6abef75ba942f7b205c130))
-  Change versioning scheme to calver. ([f2e05484](https://github.com/elisiariocouto/leggen/commit/f2e05484dc688409b6db6bd16858b066d3a16976))
-
-
-### Miscellaneous Tasks
-
-  Implement code review suggestions and format code. ([de3da84d](https://github.com/elisiariocouto/leggen/commit/de3da84dffd83e0b232cf76836935a66eb704aee))
-
-
-### Refactor
-
-  Remove MongoDB support, simplify to SQLite-only architecture ([47164e85](https://github.com/elisiariocouto/leggen/commit/47164e854600dfcac482449769b1d2e55c842570))
-  Remove unused amount_threshold and keywords from notification filters ([95709978](https://github.com/elisiariocouto/leggen/commit/957099786cb0e48c9ffbda11b3172ec9fae9ac37))
-
-
-
-## 2025.9.0 (2025/09/09)
-
-### Bug Fixes
-
- **cli:** Show transactions without internal ID when using --full. ([46f3f5c4](https://github.com/elisiariocouto/leggen/commit/46f3f5c4984224c3f4b421e1a06dcf44d4f211e0))
-  Do not install development dependencies. ([73d6bd32](https://github.com/elisiariocouto/leggen/commit/73d6bd32dbc59608ef1472dc65d9e18450f00896))
-  Implement proper GoCardless authentication and add dev features ([f0fee4fd](https://github.com/elisiariocouto/leggen/commit/f0fee4fd82e1c788614d73fcd0075f5e16976650))
-  Make internal transcation ID optional. ([6bce7eb6](https://github.com/elisiariocouto/leggen/commit/6bce7eb6be5f9a5286eb27e777fbf83a6b1c5f8d))
-  Resolve 404 balances endpoint and currency formatting errors ([417b7753](https://github.com/elisiariocouto/leggen/commit/417b77539fc275493d55efb29f92abcea666b210))
-  Merge account details into balance data to prevent unknown/N/A values ([eaaea6e4](https://github.com/elisiariocouto/leggen/commit/eaaea6e4598e9c81997573e19f4ef1c58ebe320f))
-  Use account status for balance records instead of hardcoded 'active' ([541cb262](https://github.com/elisiariocouto/leggen/commit/541cb262ee5783eedf2b154c148c28ec89845da5))
-
-
-### Documentation
-
-  Update README for new web architecture ([4018b263](https://github.com/elisiariocouto/leggen/commit/4018b263f27c2b59af31428d7a0878280a291c85))
-
-
-### Features
-
-  Transform to web architecture with FastAPI backend ([91f53b35](https://github.com/elisiariocouto/leggen/commit/91f53b35b18740869ee9cebfac394db2e12db099))
-  Add comprehensive test suite with 46 passing tests ([34e793c7](https://github.com/elisiariocouto/leggen/commit/34e793c75c8df1e57ea240b92ccf0843a80c2a14))
-  Add mypy to pre-commit. ([ec8ef834](https://github.com/elisiariocouto/leggen/commit/ec8ef8346add878f3ff4e8ed928b952d9b5dd584))
-  Implement database-first architecture to minimize GoCardless API calls ([155c3055](https://github.com/elisiariocouto/leggen/commit/155c30559f4cacd76ef01e50ec29ee436d3f9d56))
-  Implement dynamic API connection status ([cb2e70e4](https://github.com/elisiariocouto/leggen/commit/cb2e70e42d1122e9c2e5420b095aeb1e55454c24))
-  Add automatic balance timestamp migration mechanism ([34501f5f](https://github.com/elisiariocouto/leggen/commit/34501f5f0d3b3dff68364b60be77bfb99071b269))
-  Improve notification filters configuration format ([2191fe90](https://github.com/elisiariocouto/leggen/commit/2191fe906659f4fd22c25b6cb9fbb95c03472f00))
-  Add notifications view and update branding ([abf39abe](https://github.com/elisiariocouto/leggen/commit/abf39abe74b75d8cb980109fbcbdd940066cc90b))
-  Make API URL configurable and improve code quality ([37949a4e](https://github.com/elisiariocouto/leggen/commit/37949a4e1f25a2656f6abef75ba942f7b205c130))
-  Change versioning scheme to calver. ([f2e05484](https://github.com/elisiariocouto/leggen/commit/f2e05484dc688409b6db6bd16858b066d3a16976))
-
-
-### Miscellaneous Tasks
-
-  Implement code review suggestions and format code. ([de3da84d](https://github.com/elisiariocouto/leggen/commit/de3da84dffd83e0b232cf76836935a66eb704aee))
-
-
-### Refactor
-
-  Remove MongoDB support, simplify to SQLite-only architecture ([47164e85](https://github.com/elisiariocouto/leggen/commit/47164e854600dfcac482449769b1d2e55c842570))
-  Remove unused amount_threshold and keywords from notification filters ([95709978](https://github.com/elisiariocouto/leggen/commit/957099786cb0e48c9ffbda11b3172ec9fae9ac37))
-
-
-
 ## 0.6.11 (2025/02/23)

 ### Bug Fixes
--- a/frontend/package.json
+++ b/frontend/package.json
@@ -4,7 +4,7 @@
  "version": "0.0.0",
  "type": "module",
  "scripts": {
-    "dev": "VITE_API_URL=http://localhost:8000/api/v1 vite",
+    "dev": "vite",
    "build": "tsc -b && vite build",
    "lint": "eslint .",
    "preview": "vite preview"
--- a/leggen/database/sqlite.py
+++ b/leggen/database/sqlite.py
@@ -210,6 +210,7 @@ def get_transactions(
    min_amount=None,
    max_amount=None,
    search=None,
+    hide_missing_ids=True,
 ):
    """Get transactions from SQLite database with optional filtering"""
    from pathlib import Path
@@ -249,6 +250,11 @@ def get_transactions(
        query += " AND description LIKE ?"
        params.append(f"%{search}%")

+    if hide_missing_ids:
+        query += (
+            " AND internalTransactionId IS NOT NULL AND internalTransactionId != ''"
+        )
+
    # Add ordering and pagination
    query += " ORDER BY transactionDate DESC"

@@ -397,6 +403,11 @@ def get_transaction_count(account_id=None, **filters):
        query += " AND description LIKE ?"
        params.append(f"%{filters['search']}%")

+    if filters.get("hide_missing_ids", True):
+        query += (
+            " AND internalTransactionId IS NOT NULL AND internalTransactionId != ''"
+        )
+
    try:
        cursor.execute(query, params)
        count = cursor.fetchone()[0]
--- a/leggen/utils/database.py
+++ b/leggen/utils/database.py
@@ -69,13 +69,8 @@ def save_transactions(ctx: click.Context, account: str) -> list:
            ",".join(transaction.get("remittanceInformationUnstructuredArray", [])),
        )

-        # Extract transaction ID, using transactionId as fallback when internalTransactionId is missing
-        transaction_id = transaction.get("internalTransactionId") or transaction.get(
-            "transactionId"
-        )
-
        t = {
-            "internalTransactionId": transaction_id,
+            "internalTransactionId": transaction.get("internalTransactionId"),
            "institutionId": account_info["institution_id"],
            "iban": account_info.get("iban", "N/A"),
            "transactionDate": min_date,
@@ -110,13 +105,8 @@ def save_transactions(ctx: click.Context, account: str) -> list:
            ",".join(transaction.get("remittanceInformationUnstructuredArray", [])),
        )

-        # Extract transaction ID, using transactionId as fallback when internalTransactionId is missing
-        transaction_id = transaction.get("internalTransactionId") or transaction.get(
-            "transactionId"
-        )
-
        t = {
-            "internalTransactionId": transaction_id,
+            "internalTransactionId": transaction.get("internalTransactionId"),
            "institutionId": account_info["institution_id"],
            "iban": account_info.get("iban", "N/A"),
            "transactionDate": min_date,
--- a/leggend/api/routes/transactions.py
+++ b/leggend/api/routes/transactions.py
@@ -18,6 +18,9 @@ async def get_all_transactions(
    summary_only: bool = Query(
        default=True, description="Return transaction summaries only"
    ),
+    hide_missing_ids: bool = Query(
+        default=True, description="Hide transactions without internalTransactionId"
+    ),
    date_from: Optional[str] = Query(
        default=None, description="Filter from date (YYYY-MM-DD)"
    ),
@@ -47,6 +50,7 @@ async def get_all_transactions(
            min_amount=min_amount,
            max_amount=max_amount,
            search=search,
+            hide_missing_ids=hide_missing_ids,
        )

        # Get total count for pagination info (respecting the same filters)
@@ -57,6 +61,7 @@ async def get_all_transactions(
            min_amount=min_amount,
            max_amount=max_amount,
            search=search,
+            hide_missing_ids=hide_missing_ids,
        )

        # Get total count for pagination info
@@ -121,6 +126,9 @@ async def get_all_transactions(
 async def get_transaction_stats(
    days: int = Query(default=30, description="Number of days to include in stats"),
    account_id: Optional[str] = Query(default=None, description="Filter by account ID"),
+    hide_missing_ids: bool = Query(
+        default=True, description="Hide transactions without internalTransactionId"
+    ),
 ) -> APIResponse:
    """Get transaction statistics for the last N days from database"""
    try:
@@ -138,6 +146,7 @@ async def get_transaction_stats(
            date_from=date_from,
            date_to=date_to,
            limit=None,  # Get all matching transactions for stats
+            hide_missing_ids=hide_missing_ids,
        )

        # Calculate stats
--- a/leggend/services/database_service.py
+++ b/leggend/services/database_service.py
@@ -93,13 +93,8 @@ class DatabaseService:
            ",".join(transaction.get("remittanceInformationUnstructuredArray", [])),
        )

-        # Extract transaction ID, using transactionId as fallback when internalTransactionId is missing
-        transaction_id = transaction.get("internalTransactionId") or transaction.get(
-            "transactionId"
-        )
-
        return {
-            "internalTransactionId": transaction_id,
+            "internalTransactionId": transaction.get("internalTransactionId"),
            "institutionId": account_info["institution_id"],
            "iban": account_info.get("iban", "N/A"),
            "transactionDate": min_date,
@@ -121,6 +116,7 @@ class DatabaseService:
        min_amount: Optional[float] = None,
        max_amount: Optional[float] = None,
        search: Optional[str] = None,
+        hide_missing_ids: bool = True,
    ) -> List[Dict[str, Any]]:
        """Get transactions from SQLite database"""
        if not self.sqlite_enabled:
@@ -137,6 +133,7 @@ class DatabaseService:
                min_amount=min_amount,
                max_amount=max_amount,
                search=search,
+                hide_missing_ids=hide_missing_ids,
            )
            logger.debug(f"Retrieved {len(transactions)} transactions from database")
            return transactions
@@ -152,6 +149,7 @@ class DatabaseService:
        min_amount: Optional[float] = None,
        max_amount: Optional[float] = None,
        search: Optional[str] = None,
+        hide_missing_ids: bool = True,
    ) -> int:
        """Get total count of transactions from SQLite database"""
        if not self.sqlite_enabled:
@@ -168,7 +166,9 @@ class DatabaseService:
            # Remove None values
            filters = {k: v for k, v in filters.items() if v is not None}

-            count = sqlite_db.get_transaction_count(account_id=account_id, **filters)
+            count = sqlite_db.get_transaction_count(
+                account_id=account_id, hide_missing_ids=hide_missing_ids, **filters
+            )
            logger.debug(f"Total transaction count: {count}")
            return count
        except Exception as e:
@@ -259,7 +259,6 @@ class DatabaseService:
            return

        await self._migrate_balance_timestamps_if_needed()
-        await self._migrate_null_transaction_ids_if_needed()

    async def _migrate_balance_timestamps_if_needed(self):
        """Check and migrate balance timestamps if needed"""
@@ -380,145 +379,6 @@ class DatabaseService:
            logger.error(f"Balance timestamp migration failed: {e}")
            raise

-    async def _migrate_null_transaction_ids_if_needed(self):
-        """Check and migrate null transaction IDs if needed"""
-        try:
-            if await self._check_null_transaction_ids_migration_needed():
-                logger.info("Null transaction IDs migration needed, starting...")
-                await self._migrate_null_transaction_ids()
-                logger.info("Null transaction IDs migration completed")
-            else:
-                logger.info("No null transaction IDs found to migrate")
-        except Exception as e:
-            logger.error(f"Null transaction IDs migration failed: {e}")
-            raise
-
-    async def _check_null_transaction_ids_migration_needed(self) -> bool:
-        """Check if null transaction IDs need migration"""
-        from pathlib import Path
-
-        db_path = Path.home() / ".config" / "leggen" / "leggen.db"
-        if not db_path.exists():
-            return False
-
-        try:
-            conn = sqlite3.connect(str(db_path))
-            cursor = conn.cursor()
-
-            # Check for transactions with null or empty internalTransactionId
-            cursor.execute("""
-                SELECT COUNT(*)
-                FROM transactions
-                WHERE (internalTransactionId IS NULL OR internalTransactionId = '')
-                AND json_extract(rawTransaction, '$.transactionId') IS NOT NULL
-            """)
-
-            count = cursor.fetchone()[0]
-            conn.close()
-
-            return count > 0
-
-        except Exception as e:
-            logger.error(f"Failed to check null transaction IDs migration status: {e}")
-            return False
-
-    async def _migrate_null_transaction_ids(self):
-        """Populate null internalTransactionId fields using transactionId from raw data"""
-        import uuid
-        from pathlib import Path
-
-        db_path = Path.home() / ".config" / "leggen" / "leggen.db"
-        if not db_path.exists():
-            logger.warning("Database file not found, skipping migration")
-            return
-
-        try:
-            conn = sqlite3.connect(str(db_path))
-            cursor = conn.cursor()
-
-            # Get all transactions with null/empty internalTransactionId but valid transactionId in raw data
-            cursor.execute("""
-                SELECT rowid, json_extract(rawTransaction, '$.transactionId') as transactionId
-                FROM transactions
-                WHERE (internalTransactionId IS NULL OR internalTransactionId = '')
-                AND json_extract(rawTransaction, '$.transactionId') IS NOT NULL
-                ORDER BY rowid
-            """)
-
-            null_records = cursor.fetchall()
-            total_records = len(null_records)
-
-            if total_records == 0:
-                logger.info("No null transaction IDs found to migrate")
-                conn.close()
-                return
-
-            logger.info(
-                f"Migrating {total_records} transaction records with null internalTransactionId"
-            )
-
-            # Update in batches
-            batch_size = 100
-            migrated_count = 0
-            skipped_duplicates = 0
-
-            for i in range(0, total_records, batch_size):
-                batch = null_records[i : i + batch_size]
-
-                for rowid, transaction_id in batch:
-                    try:
-                        # Check if this transactionId is already used by another record
-                        cursor.execute(
-                            "SELECT COUNT(*) FROM transactions WHERE internalTransactionId = ?",
-                            (str(transaction_id),),
-                        )
-                        existing_count = cursor.fetchone()[0]
-
-                        if existing_count > 0:
-                            # Generate a unique ID to avoid constraint violation
-                            unique_id = f"{str(transaction_id)}_{uuid.uuid4().hex[:8]}"
-                            logger.debug(
-                                f"Generated unique ID for duplicate transactionId: {unique_id}"
-                            )
-                        else:
-                            # Use the original transactionId
-                            unique_id = str(transaction_id)
-
-                        # Update the record
-                        cursor.execute(
-                            """
-                            UPDATE transactions
-                            SET internalTransactionId = ?
-                            WHERE rowid = ?
-                            """,
-                            (unique_id, rowid),
-                        )
-
-                        migrated_count += 1
-
-                        if migrated_count % 100 == 0:
-                            logger.info(
-                                f"Migrated {migrated_count}/{total_records} transaction records"
-                            )
-
-                    except Exception as e:
-                        logger.error(f"Failed to migrate record {rowid}: {e}")
-                        continue
-
-                # Commit batch
-                conn.commit()
-
-            conn.close()
-            logger.info(f"Successfully migrated {migrated_count} transaction records")
-            if skipped_duplicates > 0:
-                logger.info(
-                    f"Generated unique IDs for {skipped_duplicates} duplicate transactionIds"
-                )
-
-        except Exception as e:
-            logger.error(f"Null transaction IDs migration failed: {e}")
-            raise
-
    def _unix_to_datetime_string(self, unix_timestamp: float) -> str:
        """Convert Unix timestamp to datetime string"""
        dt = datetime.fromtimestamp(unix_timestamp)
--- a/pyproject.toml
+++ b/pyproject.toml
@@ -1,6 +1,6 @@
 [project]
 name = "leggen"
-version = "2025.9.1"
+version = "0.6.11"
 description = "An Open Banking CLI"
 authors = [{ name = "Elisiário Couto", email = "elisiario@couto.io" }]
 requires-python = "~=3.13.0"
--- a/scripts/release.sh
+++ b/scripts/release.sh
@@ -1,4 +1,4 @@
-#!/usr/bin/env bash
+#!/bin/bash

 set -ef -o pipefail

@@ -13,37 +13,23 @@ check_command git
 check_command git-cliff
 check_command uv

-# Get current date components
-YEAR=$(date +%Y)
-MONTH=$(date +%-m)  # %-m removes zero padding
-
-# Get the latest version for current year and month
-LATEST_TAG=$(git tag -l "${YEAR}.${MONTH}.*" | sort -V | tail -n 1)
-
-if [ -z "$LATEST_TAG" ]; then
-    # No version for current year/month exists, start at 0
-    MICRO=0
-else
-    # Extract micro version and increment
-    MICRO=$(echo "$LATEST_TAG" | cut -d. -f3)
-    MICRO=$((MICRO + 1))
+if [ -z "$1" ]; then
+    echo " > No semver verb specified, run release with <major|minor|patch> parameter."
+    exit 1
 fi

-NEXT_VERSION="${YEAR}.${MONTH}.${MICRO}"
-CURRENT_VERSION=$(uv version --short)
-
+CURRENT_VERSION=$(uvx poetry version -s)
 echo " > Current version is $CURRENT_VERSION"
-echo " > Setting new version to $NEXT_VERSION"

-# Manually update version in pyproject.toml
-sed -i '' "s/^version = .*/version = \"${NEXT_VERSION}\"/" pyproject.toml
+uvx poetry version "$1"
+NEXT_VERSION=$(uvx poetry version -s)

-echo " > Version bumped to $NEXT_VERSION"
+echo " > leggen bumped to $NEXT_VERSION"
 echo "Updating CHANGELOG.md"
 git-cliff --unreleased --tag "$NEXT_VERSION" --prepend CHANGELOG.md > /dev/null

 echo " > Commiting changes and adding git tag"
-git add pyproject.toml CHANGELOG.md uv.lock
+git add pyproject.toml CHANGELOG.md
 git commit -m "chore(ci): Bump version to $NEXT_VERSION"
 git tag -a "$NEXT_VERSION" -m "$NEXT_VERSION"

--- a/tests/unit/test_api_transactions.py
+++ b/tests/unit/test_api_transactions.py
@@ -166,6 +166,7 @@ class TestTransactionsAPI:
            min_amount=-50.0,
            max_amount=0.0,
            search="Coffee",
+            hide_missing_ids=True,
        )

    def test_get_transactions_empty_result(
--- a/tests/unit/test_database_service.py
+++ b/tests/unit/test_database_service.py
@@ -99,6 +99,7 @@ class TestDatabaseService:
                min_amount=None,
                max_amount=None,
                search=None,
+                hide_missing_ids=True,
            )

    async def test_get_transactions_from_db_with_filters(
@@ -129,6 +130,7 @@ class TestDatabaseService:
                min_amount=-50.0,
                max_amount=0.0,
                search="Coffee",
+                hide_missing_ids=True,
            )

    async def test_get_transactions_from_db_sqlite_disabled(self, database_service):
@@ -158,7 +160,9 @@ class TestDatabaseService:
            )

            assert result == 42
-            mock_get_count.assert_called_once_with(account_id="test-account-123")
+            mock_get_count.assert_called_once_with(
+                account_id="test-account-123", hide_missing_ids=True
+            )

    async def test_get_transaction_count_from_db_with_filters(self, database_service):
        """Test getting transaction count with filters."""
@@ -178,6 +182,7 @@ class TestDatabaseService:
                date_from="2025-09-01",
                min_amount=-100.0,
                search="Coffee",
+                hide_missing_ids=True,
            )

    async def test_get_transaction_count_from_db_sqlite_disabled(
--- a/uv.lock
+++ b/uv.lock
@@ -222,7 +222,7 @@ wheels = [

 [[package]]
 name = "leggen"
-version = "2025.9.1"
+version = "0.6.11"
 source = { editable = "." }
 dependencies = [
    { name = "apscheduler" },
Author	SHA1	Message	Date
Elisiário Couto	52898b2aa7	feat: make API URL configurable and improve code quality - Add configurable API URL support via environment variables - Update nginx configuration with environment variable substitution - Create nginx template for dynamic proxy configuration - Update Docker configuration for environment variable handling - Fix hardcoded localhost:8000 references in error messages - Add proper TypeScript types for health check API - Format all code with Prettier for consistency - Update documentation with configuration instructions - Improve error messages to be environment-agnostic - Fix duplicate imports and type safety issues BREAKING: API URL is now configurable via VITE_API_URL (dev) and API_BACKEND_URL (prod)	2025-09-09 19:32:00 +01:00
Elisiário Couto	893b070ac5	feat: add notifications view and update branding - Add complete notifications management view with: - Service status display (Discord, Telegram) - Test notification functionality - Service management (delete/disable) - Filter settings display (case sensitive/insensitive) - Update API types to match current backend structure - Fix NotificationFilters type (remove deprecated fields) - Update page title from 'Vite + React + TS' to 'Leggen' - Replace Vite favicon with custom Leggen favicon - Add notifications tab to main navigation - Ensure full API compatibility with current backend	2025-09-09 19:32:00 +01:00
Elisiário Couto	4e63940aeb	refactor: remove unused amount_threshold and keywords from notification filters - Remove amount_threshold and keywords fields from NotificationFilters model - Remove handling of these fields from API routes (GET/PUT) - Update test to remove amount_threshold reference - Simplify notification filtering to focus on case-sensitive/insensitive keywords only These fields were not being used in the actual filtering logic and were just adding unnecessary complexity to the configuration.	2025-09-09 19:32:00 +01:00
Elisiário Couto	6f90a02696	feat: improve notification filters configuration format - Change filters config from nested dict to simple arrays - Update NotificationFilters model to use List[str] instead of Dict[str, str] - Modify notification service to handle list-based filters - Update API routes and tests for new format - Update README with new configuration example Before: [filters.case-insensitive] salary = 'salary' After: [filters] case-insensitive = ['salary', 'utility']	2025-09-09 19:32:00 +01:00
Elisiário Couto	b5a2e793cf	Cleanup agent documentation.	2025-09-09 19:32:00 +01:00
Elisiário Couto	67b7f73d40	Fix notification channels returning null by connecting API to real implementations - Update notification service to handle both old (api-key/chat-id) and new (token/chat_id) config formats - Connect API service to actual Discord/Telegram notification implementations from CLI codebase - Fix API routes to properly detect configured services using correct config keys - Telegram notifications now work correctly, Discord properly shows as not configured	2025-09-09 19:32:00 +01:00
Elisiário Couto	e24060bc4b	fix: use account status for balance records instead of hardcoded 'active' - Modified sync service to pass account status to balance persistence - Updated database service to use account_status from balance data - Fixed 8 balance records that had incorrect 'active' status - All balance records now have consistent 'READY' status matching accounts - Future balance records will inherit correct status from account data	2025-09-09 19:32:00 +01:00
Elisiário Couto	3bb957a470	fix: merge account details into balance data to prevent unknown/N/A values - Modified sync service to include institution_id and iban in balance persistence - Fixed data flow issue where balance records were missing account metadata - Prevents future balance records from having 'unknown' bank or 'N/A' IBAN - Successfully fixed 8 existing records with one-off script	2025-09-09 19:32:00 +01:00
Elisiário Couto	bb3f5712e9	feat: add automatic balance timestamp migration mechanism - Add migration system to convert Unix timestamps to datetime strings - Integrate migration into FastAPI lifespan for automatic startup execution - Update balance persistence to use consistent ISO datetime format - Fix mixed timestamp types causing API parsing issues - Add comprehensive error handling and progress logging - Successfully migrated 7522 balance records to consistent format	2025-09-09 19:32:00 +01:00
Elisiário Couto	d435e7beb6	Fix frontend health check to properly detect API unavailability - Add isError to useQuery destructuring to handle network errors - Improve health check query function to throw on HTTP errors - Update status display logic to show 'Disconnected' when API is unreachable - Ensure proper error handling for both network failures and HTTP status errors	2025-09-09 19:32:00 +01:00
Elisiário Couto	ef6e8ea846	feat: implement dynamic API connection status - Move health endpoint from /health to /api/v1/health - Update frontend Dashboard to show real connection status - Add health check query that refreshes every 30 seconds - Display connected/disconnected status with appropriate icons - Show loading state while checking connection	2025-09-09 19:32:00 +01:00
Elisiário Couto	fc65e08c36	fix: resolve 404 balances endpoint and currency formatting errors - Add missing /api/v1/balances endpoint to backend - Update frontend Account type to match backend AccountDetails model - Add currency validation with EUR fallback in formatCurrency function - Update AccountsOverview, TransactionsList, and Dashboard components - Fix balance calculations to use balances array structure - All pre-commit checks pass	2025-09-09 19:32:00 +01:00
Elisiário Couto	500d05438d	Add hide_missing_ids filter to transaction queries - Add hide_missing_ids parameter to database functions to filter out transactions without internalTransactionId - Update API routes to support the new filter parameter - Update unit tests to include the new parameter - Add opencode.json configuration file	2025-09-09 19:32:00 +01:00
Elisiário Couto	8cfad8a853	Create temporary database for testing instead of using configured database - Add temp_db_path fixture to create temporary database file for tests - Add mock_db_path fixture to mock Path.home() for database path resolution - Update all account API tests to use temporary database - Ensure test database is properly cleaned up after tests - Prevent test data from polluting the actual configured database - All 94 tests still pass with temporary database setup	2025-09-09 19:32:00 +01:00
Elisiário Couto	4293d45ce4	Remove GoCardless fallback from /accounts endpoints - Remove GoCardless API calls from /api/v1/accounts and /api/v1/accounts/{account_id} - Accounts endpoints now rely exclusively on database data - Return 404 for accounts not found in database - Update tests to mock database service instead of GoCardless API - Remove unused GoCardless imports from transactions routes - Preserve GoCardless usage in sync process and /banks endpoints - Fix code formatting and remove unused imports	2025-09-09 19:32:00 +01:00
Elisiário Couto	3ed79f9508	Fix api in lib folder.	2025-09-09 19:32:00 +01:00
Elisiário Couto	beb44f8563	Claude experiments	2025-09-09 19:32:00 +01:00
Elisiário Couto	1bc259bd3e	fix(cli): Show transactions without internal ID when using --full.	2025-09-03 23:40:54 +01:00
Elisiário Couto	fad426aee8	fix: Make internal transcation ID optional.	2025-09-03 23:21:11 +01:00
Elisiário Couto	7f59e634a1	feat: Implement database-first architecture to minimize GoCardless API calls - Updated SQLite database to use ~/.config/leggen/leggen.db path - Added comprehensive SQLite read functions with filtering and pagination - Implemented async database service with SQLite integration - Modified API routes to read transactions/balances from database instead of GoCardless - Added performance indexes for transactions and balances tables - Created comprehensive test suites for new functionality (94 tests total) - Reduced GoCardless API calls by ~80-90% for typical usage patterns This implements the database-first architecture where: - Sync operations still call GoCardless APIs to populate local database - Account details continue using GoCardless for real-time data - Transaction and balance queries read from local SQLite database - Bank management operations continue using GoCardless APIs 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>	2025-09-03 23:21:11 +01:00
Elisiário Couto	011c792e89	feat: Add mypy to pre-commit.	2025-09-03 23:21:11 +01:00
Elisiário Couto	4e4c80be48	chore: Implement code review suggestions and format code.	2025-09-03 23:21:11 +01:00
Elisiário Couto	4c10307e12	refactor: Remove MongoDB support, simplify to SQLite-only architecture - Remove pymongo dependency from pyproject.toml and update lock file - Delete leggen/database/mongo.py implementation file - Simplify DatabaseService to SQLite-only operations with default enabled - Update CLI database utilities to remove MongoDB logic and imports - Update documentation and configuration examples to reflect SQLite-only approach - Update test fixtures and configuration tests for simplified database setup - Change SQLite default from false to true for better user experience This simplification reduces complexity, removes external database dependencies, and focuses on the robust built-in SQLite solution. All 46 tests passing. Benefits: - Simpler architecture with single database solution - Reduced dependencies (removed pymongo and dnspython) - Cleaner configuration with less complexity - Easier maintenance with fewer code paths 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>	2025-09-03 23:21:11 +01:00
Elisiário Couto	de67ce9ec0	feat: Add comprehensive test suite with 46 passing tests - Add pytest configuration in pyproject.toml with markers and async support - Create shared test fixtures in tests/conftest.py for config, auth, and sample data - Implement unit tests for all major components: * Configuration management (11 tests) - TOML loading/saving, singleton pattern * FastAPI API endpoints (12 tests) - Banks, accounts, transactions with mocks * CLI API client (11 tests) - HTTP client integration and error handling * Background scheduler (12 tests) - APScheduler job management and async ops - Fix GoCardless API authentication mocking by adding token endpoints - Resolve TOML file writing issues (binary vs text mode for tomli_w) - Add comprehensive testing documentation to README - Update code structure documentation to include test organization Testing framework includes: - respx for HTTP request mocking - pytest-asyncio for async test support - pytest-mock for advanced mocking capabilities - requests-mock for CLI HTTP client testing - Realistic test data fixtures for banks, accounts, and transactions 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>	2025-09-03 23:21:11 +01:00
Elisiário Couto	d99b1c73db	docs: Update README for new web architecture Major README overhaul to reflect the transformation to web-ready architecture: New Content: - Web architecture description with FastAPI backend (leggend) and CLI - Enhanced feature list with API & integration capabilities - Quick start guide with Docker Compose and local development options - Comprehensive usage examples for both API service and CLI - Complete API endpoint documentation - Development setup and code structure explanation Key Improvements: - Updated installation instructions with uv and Docker options - Added leggend service commands with --reload flag - Enhanced CLI examples with new options (--wait, --force, --full) - API endpoint documentation with all major routes - Configuration examples with scheduler and notification settings - Development workflow and contribution guidelines The README now accurately represents the current v0.6.11 capabilities and provides clear guidance for both users and developers. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>	2025-09-03 23:21:11 +01:00
Elisiário Couto	69e381fffe	fix: Implement proper GoCardless authentication and add dev features Authentication Fixes: - Implement proper async GoCardless token management in leggend service - Add automatic token refresh and creation for expired/missing tokens - Unify auth.json storage path between CLI and API (~/.config/leggen/) - Fix 401 Unauthorized errors when accessing GoCardless API Development Enhancements: - Add --reload flag to leggend for automatic file watching and restart - Add --host and --port options for flexible service binding - Include both leggend/ and leggen/ directories in reload watching - Improve development workflow with hot reloading Configuration Consistency: - Standardize config path to ~/.config/leggen/config.toml for both CLI and API - Ensure auth.json is stored in same location as main config - Add httpx dependency for async HTTP requests in leggend service Verified working: leggen status command successfully authenticates and retrieves bank/account data via leggend API service. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>	2025-09-03 23:21:11 +01:00
Elisiário Couto	f6ebb98de3	feat: Transform to web architecture with FastAPI backend This major update transforms leggen from CLI-only to a web-ready architecture while maintaining full CLI compatibility. New Features: - FastAPI backend service (leggend) with comprehensive REST API - Background job scheduler with configurable cron (replaces Ofelia) - All CLI commands refactored to use API endpoints - Docker configuration updated for new services - API client with health checks and error handling API Endpoints: - /api/v1/banks/* - Bank connections and institutions - /api/v1/accounts/* - Account management and balances - /api/v1/transactions/* - Transaction retrieval with filtering - /api/v1/sync/* - Manual sync and scheduler configuration - /api/v1/notifications/* - Notification settings management CLI Enhancements: - New --api-url option and LEGGEND_API_URL environment variable - Enhanced sync command with --wait and --force options - Improved transactions command with --full and --limit options - Automatic fallback and health checking Breaking Changes: - compose.yml structure updated (leggend service added) - Ofelia scheduler removed (internal scheduler used instead) 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>	2025-09-03 23:21:11 +01:00