Format and lint web directory

.pre-commit-config.yaml

@@ -61,18 +61,18 @@ repos:
           - --profile=black
           - --line-length=80
 
-#  - repo: https://github.com/astral-sh/ruff-pre-commit
-#    rev: v0.9.7
-#    hooks:
-#      - id: ruff
-#        types_or: [python,pyi]
-#        args:
-#          - --fix
-#          - --select=B,C,E,F,W,B9
-#          - --line-length=80
-#          - --ignore=E203,E402,E501,E261
-#      - id: ruff-format
-#        types_or: [ python,pyi]
-#        args:
-#          - --target-version=py310
-#          - --line-length=80
+  - repo: https://github.com/astral-sh/ruff-pre-commit
+    rev: v0.9.7
+    hooks:
+      - id: ruff
+        types_or: [python,pyi]
+        args:
+          - --fix
+          - --select=B,C,E,F,W,B9
+          - --line-length=80
+          - --ignore=E203,E402,E501,E261
+      - id: ruff-format
+        types_or: [ python,pyi]
+        args:
+          - --target-version=py310
+          - --line-length=80

app/migrations/env.py

@@ -14,9 +14,7 @@
 # This line sets up loggers basically.
 if config.config_file_name is not None:
     # disable_existing_loggers prevents loguru disabling
-    fileConfig(
-        config.config_file_name, disable_existing_loggers=False
-    )
+    fileConfig(config.config_file_name, disable_existing_loggers=False)
 
 # add your model's MetaData object here
 # for 'autogenerate' support

app/shared/db/models.py

@@ -20,7 +20,7 @@ def generate_uuid():
     return str(uuid.uuid4())
 
 
-# many to many association tables
+# many-to-many association tables
 association_table_archive_tags = Table(
     "mtm_archives_tags",
     Base.metadata,

app/tests/shared/db/test_worker_crud.py

@@ -14,19 +14,19 @@ def test_update_sheet_last_url_archived_at(db_session):
     assert isinstance(test_sheet.last_url_archived_at, datetime)
     before = test_sheet.last_url_archived_at
     assert (
-            worker_crud.update_sheet_last_url_archived_at(db_session, "sheet-123")
-            is True
+        worker_crud.update_sheet_last_url_archived_at(db_session, "sheet-123")
+        is True
     )
     db_session.refresh(test_sheet)
     assert isinstance(test_sheet.last_url_archived_at, datetime)
     assert test_sheet.last_url_archived_at > before
 
     # Test non-existent sheet
     assert (
-            worker_crud.update_sheet_last_url_archived_at(
-                db_session, "non-existent-sheet"
-            )
-            is False
+        worker_crud.update_sheet_last_url_archived_at(
+            db_session, "non-existent-sheet"
+        )
+        is False
     )
 
 
@@ -42,14 +42,14 @@ def test_create_or_get_user(test_data, db_session):
 
     # already exists
     assert (
-               u1 := worker_crud.create_or_get_user(db_session, "rick@example.com")
-           ) is not None
+        u1 := worker_crud.create_or_get_user(db_session, "rick@example.com")
+    ) is not None
     assert u1.email == "rick@example.com"
 
     # new user
     assert (
-               u2 := worker_crud.create_or_get_user(db_session, "beth@example.com")
-           ) is not None
+        u2 := worker_crud.create_or_get_user(db_session, "beth@example.com")
+    ) is not None
     assert u2.email == "beth@example.com"
 
     assert db_session.query(models.User).count() == 4
@@ -64,8 +64,8 @@ def test_create_tag(db_session):
     assert create_tag.id == "tag-101"
     assert db_session.query(models.Tag).count() == 1
     assert (
-            db_session.query(models.Tag).filter(models.Tag.id == "tag-101").first()
-            == create_tag
+        db_session.query(models.Tag).filter(models.Tag.id == "tag-101").first()
+        == create_tag
     )
 
     # same id does not add new db entry

app/tests/web/db/test_crud.py

@@ -712,7 +712,11 @@ async def test_find_by_store_until(async_db_session):
 
 @pytest.mark.asyncio
 async def test_get_sheets_by_id_hash(async_db_session):
-    author_emails = ["rick@example.com", "morty@example.com", "jerry@example.com"]
+    author_emails = [
+        "rick@example.com",
+        "morty@example.com",
+        "jerry@example.com",
+    ]
 
     # Add test data
     sheets = [

app/tests/web/test_main.py

@@ -7,6 +7,7 @@
 import pytest
 from fastapi.testclient import TestClient
 
+from app.web.main import app_factory
 from app.web.utils.metrics import EXCEPTION_COUNTER
 
 
@@ -59,7 +60,6 @@ def test_serve_local_archive_logic(get_settings):
     try:
         # modify the settings
         get_settings.SERVE_LOCAL_ARCHIVE = "/app/local_archive_test"
-        from app.web.main import app_factory
 
         app = app_factory(get_settings)
 

app/web/config.py

@@ -8,7 +8,10 @@
 - You can use this API to archive single URLs or entire Google Sheets.
 - Once you submit a URL or Sheet for archiving, the API will return a task_id that you can use to check the status of the archiving process. It works asynchronously.
 """
-BREAKING_CHANGES = {"minVersion": "0.4.0", "message": "The latest update has breaking changes, please update the extension to the most recent version."}
+BREAKING_CHANGES = {
+    "minVersion": "0.4.0",
+    "message": "The latest update has breaking changes, please update the extension to the most recent version.",
+}
 
 # changing this will corrupt the database logic
 ALLOW_ANY_EMAIL = "*"