🎨 Auto format and update with pre-commit

Author: pre-commit-ci-lite[bot]

backend/app/api/deps.py

@@ -56,6 +56,7 @@ def get_current_active_superuser(current_user: CurrentUser) -> User:
         )
     return current_user
 
+
 def require_role(*allowed_roles: UserRole):
     def role_checker(current_user: CurrentUser) -> User:
         if current_user.role not in allowed_roles:

backend/app/api/main.py

@@ -1,6 +1,6 @@
 from fastapi import APIRouter
 
-from app.api.routes import items, login, private, users, utils, metrics
+from app.api.routes import items, login, metrics, private, users, utils
 from app.core.config import settings
 
 api_router = APIRouter()

backend/app/api/routes/metrics.py

@@ -26,4 +26,4 @@ def read_metrics(session: SessionDep) -> MetricsPublic:
         total_users=total_users,
         active_users=active_users,
         total_items=total_items,
-    )
+    )

backend/app/api/routes/users.py

@@ -20,10 +20,10 @@
     UserCreate,
     UserPublic,
     UserRegister,
+    UserRole,
     UsersPublic,
     UserUpdate,
     UserUpdateMe,
-    UserRole,
 )
 from app.utils import generate_new_account_email, send_email
 
@@ -136,9 +136,7 @@ def delete_user_me(session: SessionDep, current_user: CurrentUser) -> Any:
     Delete own user. Accessible manager and member.
     """
     if current_user.role == UserRole.ADMIN:
-        raise HTTPException(
-            status_code=403, detail="Admins cannot delete themselves"
-        )
+        raise HTTPException(status_code=403, detail="Admins cannot delete themselves")
     session.delete(current_user)
     session.commit()
     return Message(message="User deleted successfully")
@@ -223,12 +221,9 @@ def delete_user(
     if not user:
         raise HTTPException(status_code=404, detail="User not found")
     if user == current_user:
-        raise HTTPException(
-            status_code=403, detail="Admins cannot delete themselves"
-        )
+        raise HTTPException(status_code=403, detail="Admins cannot delete themselves")
     statement = delete(Item).where(col(Item.owner_id) == user_id)
     session.exec(statement)
     session.delete(user)
     session.commit()
     return Message(message="User deleted successfully")
-

backend/app/core/db.py

@@ -65,4 +65,4 @@ def _ensure_user(
         role=role,
         is_superuser=is_superuser,
     )
-    return crud.create_user(session=session, user_create=user_in)
+    return crud.create_user(session=session, user_create=user_in)

backend/app/models.py

@@ -1,15 +1,16 @@
 import uuid
 from datetime import datetime, timezone
-
 from enum import Enum
+
 from pydantic import EmailStr
-from sqlalchemy import DateTime,  String
+from sqlalchemy import DateTime, String
 from sqlmodel import Field, Relationship, SQLModel
 
 
 def get_datetime_utc() -> datetime:
     return datetime.now(timezone.utc)
 
+
 # User roles for RBAC.
 class UserRole(str, Enum):
     ADMIN = "admin"
@@ -136,7 +137,8 @@ class NewPassword(SQLModel):
     token: str
     new_password: str = Field(min_length=8, max_length=128)
 
+
 class MetricsPublic(SQLModel):
     total_users: int
     active_users: int
-    total_items: int
+    total_items: int

backend/tests/api/routes/test_rbac.py

@@ -6,28 +6,22 @@
 def test_admin_can_list_users(
     client: TestClient, superuser_token_headers: dict[str, str]
 ) -> None:
-    r = client.get(
-        f"{settings.API_V1_STR}/users/", headers=superuser_token_headers
-    )
+    r = client.get(f"{settings.API_V1_STR}/users/", headers=superuser_token_headers)
     assert 200 <= r.status_code < 300
     assert "data" in r.json()
 
 
 def test_manager_can_list_users(
     client: TestClient, manager_token_headers: dict[str, str]
 ) -> None:
-    r = client.get(
-        f"{settings.API_V1_STR}/users/", headers=manager_token_headers
-    )
+    r = client.get(f"{settings.API_V1_STR}/users/", headers=manager_token_headers)
     assert 200 <= r.status_code < 300
 
 
 def test_member_cannot_list_users(
     client: TestClient, normal_user_token_headers: dict[str, str]
 ) -> None:
-    r = client.get(
-        f"{settings.API_V1_STR}/users/", headers=normal_user_token_headers
-    )
+    r = client.get(f"{settings.API_V1_STR}/users/", headers=normal_user_token_headers)
     assert r.status_code == 403
 
 
@@ -45,9 +39,7 @@ def test_manager_cannot_create_user(
 def test_member_can_read_own_profile(
     client: TestClient, normal_user_token_headers: dict[str, str]
 ) -> None:
-    r = client.get(
-        f"{settings.API_V1_STR}/users/me", headers=normal_user_token_headers
-    )
+    r = client.get(f"{settings.API_V1_STR}/users/me", headers=normal_user_token_headers)
     current_user = r.json()
     assert 200 <= r.status_code < 300
     assert "role" in current_user
@@ -56,21 +48,17 @@ def test_member_can_read_own_profile(
 def test_member_cannot_view_metrics(
     client: TestClient, normal_user_token_headers: dict[str, str]
 ) -> None:
-    r = client.get(
-        f"{settings.API_V1_STR}/metrics/", headers=normal_user_token_headers
-    )
+    r = client.get(f"{settings.API_V1_STR}/metrics/", headers=normal_user_token_headers)
     assert r.status_code == 403
 
 
 def test_manager_can_view_metrics(
     client: TestClient, manager_token_headers: dict[str, str]
 ) -> None:
-    r = client.get(
-        f"{settings.API_V1_STR}/metrics/", headers=manager_token_headers
-    )
+    r = client.get(f"{settings.API_V1_STR}/metrics/", headers=manager_token_headers)
     assert 200 <= r.status_code < 300
 
 
 def test_unauthenticated_cannot_list_users(client: TestClient) -> None:
     r = client.get(f"{settings.API_V1_STR}/users/")
-    assert r.status_code == 401
+    assert r.status_code == 401

backend/tests/conftest.py

@@ -41,6 +41,7 @@ def normal_user_token_headers(client: TestClient, db: Session) -> dict[str, str]
         client=client, email=settings.EMAIL_TEST_USER, db=db
     )
 
+
 @pytest.fixture(scope="module")
 def manager_token_headers(client: TestClient, db: Session) -> dict[str, str]:
     return authentication_token_from_email(

backend/tests/utils/user.py

@@ -3,7 +3,7 @@
 
 from app import crud
 from app.core.config import settings
-from app.models import User, UserCreate, UserUpdate, UserRole
+from app.models import User, UserCreate, UserRole, UserUpdate
 from tests.utils.utils import random_email, random_lower_string
 
 

frontend/src/components/Admin/columns.tsx

@@ -1,21 +1,21 @@
-import type { ColumnDef } from "@tanstack/react-table";
+import type { ColumnDef } from "@tanstack/react-table"
 
-import type { UserPublic } from "@/client";
-import { Badge } from "@/components/ui/badge";
-import { can } from "@/lib/auth/permissions";
-import { cn } from "@/lib/utils";
-import { UserActionsMenu } from "./UserActionsMenu";
+import type { UserPublic } from "@/client"
+import { Badge } from "@/components/ui/badge"
+import { can } from "@/lib/auth/permissions"
+import { cn } from "@/lib/utils"
+import { UserActionsMenu } from "./UserActionsMenu"
 
 export type UserTableData = UserPublic & {
-  isCurrentUser: boolean;
-};
+  isCurrentUser: boolean
+}
 
 const baseColumns: ColumnDef<UserTableData>[] = [
   {
     accessorKey: "full_name",
     header: "Full Name",
     cell: ({ row }) => {
-      const fullName = row.original.full_name;
+      const fullName = row.original.full_name
       return (
         <div className="flex items-center gap-2">
           <span
@@ -29,7 +29,7 @@ const baseColumns: ColumnDef<UserTableData>[] = [
             </Badge>
           )}
         </div>
-      );
+      )
     },
   },
   {
@@ -43,12 +43,12 @@ const baseColumns: ColumnDef<UserTableData>[] = [
     accessorKey: "role",
     header: "Role",
     cell: ({ row }) => {
-      const role = row.original.role ?? "member";
+      const role = row.original.role ?? "member"
       return (
         <Badge variant={role === "admin" ? "default" : "secondary"}>
           {role.charAt(0).toUpperCase() + role.slice(1)}
         </Badge>
-      );
+      )
     },
   },
   {
@@ -68,7 +68,7 @@ const baseColumns: ColumnDef<UserTableData>[] = [
       </div>
     ),
   },
-];
+]
 
 const actionsColumn: ColumnDef<UserTableData> = {
   id: "actions",
@@ -78,13 +78,13 @@ const actionsColumn: ColumnDef<UserTableData> = {
       <UserActionsMenu user={row.original} />
     </div>
   ),
-};
+}
 
 export function getColumns(
   currentUser: UserPublic | null | undefined,
 ): ColumnDef<UserTableData>[] {
   const canManageUsers =
-    can(currentUser, "updateAnyUser") || can(currentUser, "deleteAnyUser");
+    can(currentUser, "updateAnyUser") || can(currentUser, "deleteAnyUser")
 
-  return canManageUsers ? [...baseColumns, actionsColumn] : baseColumns;
+  return canManageUsers ? [...baseColumns, actionsColumn] : baseColumns
 }