fixed the CI issue

Author: rohitdash08

packages/backend/app/__init__.py

@@ -26,8 +26,10 @@ def create_app(settings: Settings | None = None) -> Flask:
 
     # Logging
     log_level = os.getenv("LOG_LEVEL", "INFO").upper()
-    logging.basicConfig(level=getattr(logging, log_level, logging.INFO),
-                        format="%(asctime)s %(levelname)s [%(name)s] %(message)s")
+    logging.basicConfig(
+        level=getattr(logging, log_level, logging.INFO),
+        format="%(asctime)s %(levelname)s [%(name)s] %(message)s",
+    )
     logger = logging.getLogger("finmind")
     logger.info("Starting FinMind backend with log level %s", log_level)
 

packages/backend/app/models.py

@@ -92,7 +92,9 @@ class UserSubscription(db.Model):
     __tablename__ = "user_subscriptions"
     id = db.Column(db.Integer, primary_key=True)
     user_id = db.Column(db.Integer, db.ForeignKey("users.id"), nullable=False)
-    plan_id = db.Column(db.Integer, db.ForeignKey("subscription_plans.id"), nullable=False)
+    plan_id = db.Column(
+        db.Integer, db.ForeignKey("subscription_plans.id"), nullable=False
+    )
     active = db.Column(db.Boolean, default=False, nullable=False)
     started_at = db.Column(db.DateTime, default=datetime.utcnow, nullable=False)
 

packages/backend/app/routes/auth.py

@@ -1,6 +1,11 @@
 from flask import Blueprint, request, jsonify
 from werkzeug.security import generate_password_hash, check_password_hash
-from flask_jwt_extended import create_access_token, create_refresh_token, jwt_required, get_jwt_identity
+from flask_jwt_extended import (
+    create_access_token,
+    create_refresh_token,
+    jwt_required,
+    get_jwt_identity,
+)
 from ..extensions import db
 from ..models import User
 import logging

packages/backend/app/routes/bills.py

@@ -14,21 +14,28 @@
 @jwt_required()
 def list_bills():
     uid = int(get_jwt_identity())
-    items = db.session.query(Bill).filter_by(user_id=uid, active=True).order_by(Bill.next_due_date).all()
+    items = (
+        db.session.query(Bill)
+        .filter_by(user_id=uid, active=True)
+        .order_by(Bill.next_due_date)
+        .all()
+    )
     logger.info("List bills user=%s count=%s", uid, len(items))
-    return jsonify([
-        {
-            "id": b.id,
-            "name": b.name,
-            "amount": float(b.amount),
-            "currency": b.currency,
-            "next_due_date": b.next_due_date.isoformat(),
-            "cadence": b.cadence.value,
-            "channel_whatsapp": b.channel_whatsapp,
-            "channel_email": b.channel_email,
-        }
-        for b in items
-    ])
+    return jsonify(
+        [
+            {
+                "id": b.id,
+                "name": b.name,
+                "amount": float(b.amount),
+                "currency": b.currency,
+                "next_due_date": b.next_due_date.isoformat(),
+                "cadence": b.cadence.value,
+                "channel_whatsapp": b.channel_whatsapp,
+                "channel_email": b.channel_email,
+            }
+            for b in items
+        ]
+    )
 
 
 @bp.post("")
@@ -71,5 +78,7 @@ def mark_paid(bill_id: int):
         b.active = False
     db.session.commit()
     cache_delete_patterns([f"user:{uid}:upcoming_bills*"])
-    logger.info("Marked bill paid id=%s user=%s next_due_date=%s", b.id, uid, b.next_due_date)
+    logger.info(
+        "Marked bill paid id=%s user=%s next_due_date=%s", b.id, uid, b.next_due_date
+    )
     return jsonify(message="updated")

packages/backend/app/routes/categories.py

@@ -12,7 +12,9 @@
 @jwt_required()
 def list_categories():
     uid = int(get_jwt_identity())
-    items = db.session.query(Category).filter_by(user_id=uid).order_by(Category.name).all()
+    items = (
+        db.session.query(Category).filter_by(user_id=uid).order_by(Category.name).all()
+    )
     logger.info("List categories for user=%s count=%s", uid, len(items))
     return jsonify([{"id": c.id, "name": c.name} for c in items])
 

packages/backend/app/routes/expenses.py

@@ -47,14 +47,20 @@ def create_expense():
         currency=data.get("currency", "USD"),
         category_id=data.get("category_id"),
         notes=data.get("notes"),
-        spent_at=date.fromisoformat(data.get("spent_at")) if data.get("spent_at") else date.today(),
+        spent_at=(
+            date.fromisoformat(data.get("spent_at"))
+            if data.get("spent_at")
+            else date.today()
+        ),
     )
     db.session.add(e)
     db.session.commit()
     logger.info("Created expense id=%s user=%s amount=%s", e.id, uid, e.amount)
     # Invalidate caches
-    cache_delete_patterns([
-        monthly_summary_key(uid, e.spent_at.strftime("%Y-%m")),
-        f"insights:{uid}:*",
-    ])
+    cache_delete_patterns(
+        [
+            monthly_summary_key(uid, e.spent_at.strftime("%Y-%m")),
+            f"insights:{uid}:*",
+        ]
+    )
     return jsonify(id=e.id), 201

packages/backend/app/routes/reminders.py

@@ -14,18 +14,25 @@
 @jwt_required()
 def list_reminders():
     uid = int(get_jwt_identity())
-    items = db.session.query(Reminder).filter_by(user_id=uid).order_by(Reminder.send_at).all()
+    items = (
+        db.session.query(Reminder)
+        .filter_by(user_id=uid)
+        .order_by(Reminder.send_at)
+        .all()
+    )
     logger.info("List reminders user=%s count=%s", uid, len(items))
-    return jsonify([
-        {
-            "id": r.id,
-            "message": r.message,
-            "send_at": r.send_at.isoformat(),
-            "sent": r.sent,
-            "channel": r.channel,
-        }
-        for r in items
-    ])
+    return jsonify(
+        [
+            {
+                "id": r.id,
+                "message": r.message,
+                "send_at": r.send_at.isoformat(),
+                "sent": r.sent,
+                "channel": r.channel,
+            }
+            for r in items
+        ]
+    )
 
 
 @bp.post("")
@@ -52,7 +59,9 @@ def run_due():
     now = datetime.utcnow() + timedelta(minutes=1)
     items = (
         db.session.query(Reminder)
-        .filter(Reminder.user_id == uid, Reminder.sent == False, Reminder.send_at <= now)
+        .filter(
+            Reminder.user_id == uid, Reminder.sent == False, Reminder.send_at <= now
+        )
         .all()
     )
     for r in items:

packages/backend/app/services/ai.py

@@ -18,7 +18,11 @@ def _heuristic_budget(uid: int, ym: str):
     year, month = map(int, ym.split("-"))
     total = (
         db.session.query(func.coalesce(func.sum(Expense.amount), 0))
-        .filter(Expense.user_id == uid, extract("year", Expense.spent_at) == year, extract("month", Expense.spent_at) == month)
+        .filter(
+            Expense.user_id == uid,
+            extract("year", Expense.spent_at) == year,
+            extract("month", Expense.spent_at) == month,
+        )
         .scalar()
     )
     # Suggest 90% of last spend with 50/30/20 breakdown
@@ -42,7 +46,11 @@ def monthly_budget_suggestion(uid: int, ym: str):
             year, month = map(int, ym.split("-"))
             rows = (
                 db.session.query(Expense.category_id, func.sum(Expense.amount))
-                .filter(Expense.user_id == uid, extract("year", Expense.spent_at) == year, extract("month", Expense.spent_at) == month)
+                .filter(
+                    Expense.user_id == uid,
+                    extract("year", Expense.spent_at) == year,
+                    extract("month", Expense.spent_at) == month,
+                )
                 .group_by(Expense.category_id)
                 .all()
             )

packages/backend/app/services/reminders.py

@@ -37,7 +37,12 @@ def send_email(to_email: str, subject: str, body: str):
 
 
 def send_whatsapp(to_number: str, body: str):
-    if not (_settings.twilio_account_sid and _settings.twilio_auth_token and _settings.twilio_whatsapp_from and TwilioClient):
+    if not (
+        _settings.twilio_account_sid
+        and _settings.twilio_auth_token
+        and _settings.twilio_whatsapp_from
+        and TwilioClient
+    ):
         return False
     try:
         client = TwilioClient(_settings.twilio_account_sid, _settings.twilio_auth_token)

packages/backend/requirements.txt

@@ -13,3 +13,7 @@ requests==2.32.3
 openai==1.37.1
 twilio==9.3.2
 pytest==8.2.2
+black==24.8.0
+flake8==7.0.0
+bandit==1.7.9
+