added ruff linter and remove few old test cases

.gitignore

@@ -3,5 +3,6 @@
 __pycache__/
 .venv
 .vscode
+.ruff_cache/
 
 

api-service/app.py

@@ -1,7 +1,6 @@
 from dotenv import load_dotenv
-from flask import *
+from flask import Flask
 import logging
-from langchain_classic.prompts import PromptTemplate
 from routes.NewsRoutes import routes
 
 # Load environment variables

api-service/pytest.ini

@@ -0,0 +1,2 @@
+[pytest]
+pythonpath = .

api-service/requirements.txt

@@ -19,3 +19,5 @@ sentence-transformers==5.6.0
 gunicorn==26.0.0
 transformers==5.12.1
 torch==2.12.1
+pytest==8.3.5
+pytest-mock==3.14.1

api-service/routes/NewsRoutes.py

@@ -1,16 +1,16 @@
-from flask import *
-import logging
-from controllers.NewsController import NewsController
-from models.SubscriberModel import SubscriberDB
-from models.SourceModel import SourceDB
-from models.NewsModel import CybernewsDB
-from agents.notification import KNOWN_INTEREST_TAGS
 import json
 import os
+
 import redis
+from flask import Blueprint, g, jsonify, render_template, request
+
+from agents.notification import KNOWN_INTEREST_TAGS
+from controllers.NewsController import NewsController
+from models.NewsModel import CybernewsDB
+from models.SourceModel import SourceDB
+from models.SubscriberModel import SubscriberDB
 
 routes = Blueprint("routes", __name__)
-logger = logging.getLogger(__name__)
 
 REDIS_URL = os.getenv("REDIS_URL", "redis://localhost:6379/0")
 SESSION_TTL = int(os.getenv("SESSION_TTL", "3600"))

api-service/services/NewsService.py

@@ -1,4 +1,3 @@
-import os
 import json
 from dotenv import dotenv_values
 from flask import jsonify

api-service/tests/test_notification.py

@@ -93,6 +93,7 @@ def test_default_frequency_is_daily(self, mocker):
         state = make_state(user_input="subscribe vishak@example.com to malware")
         result = notification_agent(state)
         mock_db.create_subscriber.assert_called_once_with(email="vishak@example.com", frequency="daily", interests=["malware"])
+        assert result["notification_triggered"] is True
 
     def test_newly_added_interest_tags_matched(self, mocker):
         from agents import notification as notification_module

ingestion-service/embeddings.py

@@ -5,7 +5,7 @@
 """
 import logging
 
-from config import EMBEDDING_DIM, EMBEDDING_MODEL, MAX_EMBEDDING_INPUT_CHARS
+from config import EMBEDDING_MODEL, MAX_EMBEDDING_INPUT_CHARS
 
 logger = logging.getLogger(__name__)
 

requirements-dev.txt

@@ -0,0 +1,2 @@
+# Local/CI quality tools. Runtime deps stay in each service's requirements.txt.
+ruff==0.12.11

ruff.toml

@@ -0,0 +1,26 @@
+target-version = "py311"
+line-length = 100
+
+exclude = [
+    ".git",
+    ".venv",
+    ".pytest_cache",
+    ".ruff_cache",
+    "__pycache__",
+    "api-service/.venv",
+    "ingestion-service/.venv",
+    "notification-service/.venv",
+    "api-service/cybernews",
+    "api-service/db_update",
+]
+
+[lint]
+select = ["E", "F"]
+ignore = [
+    "E501",  # existing lines are already long; don't restyle the whole repo
+]
+
+[lint.per-file-ignores]
+# Tests and the summarizer job import after sys.path / load_dotenv on purpose.
+"**/tests/**" = ["E402", "F401"]
+"api-service/jobs/summarize_articles.py" = ["E402"]