Final polish?

Author: vodkar

backend/app/api/routes/items.py

@@ -1,7 +1,6 @@
 """Item management API endpoints."""
 
 import uuid
-from mailbox import Message
 
 from fastapi import APIRouter, HTTPException
 from sqlmodel import func, select
@@ -10,7 +9,7 @@
 from app.constants import BAD_REQUEST_CODE, NOT_FOUND_CODE
 
 # Removed unused Any import
-from app.models import Item, ItemCreate, ItemPublic, ItemsPublic, ItemUpdate
+from app.models import Item, ItemCreate, ItemPublic, ItemsPublic, ItemUpdate, Message
 
 router = APIRouter(prefix="/items", tags=["items"])
 

backend/app/core/config.py

@@ -78,16 +78,16 @@ class Settings(BaseSettings):  # type: ignore[explicit-any]
     FIRST_SUPERUSER: EmailStr
     FIRST_SUPERUSER_PASSWORD: str
 
+    @computed_field # type: ignore[prop-decorator]
     @property
-    @computed_field
     def all_cors_origins(self) -> list[str]:
         """Get all CORS origins."""
         return [str(origin).rstrip("/") for origin in self.BACKEND_CORS_ORIGINS] + [
             self.FRONTEND_HOST,
         ]
 
+    @computed_field # type: ignore[prop-decorator]
     @property
-    @computed_field
     def SQLALCHEMY_DATABASE_URI(self) -> PostgresDsn:  # noqa: N802
         """Build database URI from configuration."""
         return PostgresDsn.build(
@@ -99,8 +99,8 @@ def SQLALCHEMY_DATABASE_URI(self) -> PostgresDsn:  # noqa: N802
             path=self.POSTGRES_DB,
         )
 
+    @computed_field # type: ignore[prop-decorator]
     @property
-    @computed_field
     def emails_enabled(self) -> bool:
         """Check if email configuration is enabled."""
         return bool(self.SMTP_HOST and self.EMAILS_FROM_EMAIL)

backend/app/models/__init__.py

@@ -31,7 +31,7 @@
     UserUpdateMe,
 )
 
-__all__ = [  # noqa: RUF022
+__all__ = [
     # API models
     "Message",
     "NewPassword",

backend/app/models/db_models.py

@@ -32,3 +32,5 @@ class Item(SQLModel, table=True):
         ondelete="CASCADE",
     )
     owner: User | None = Relationship(back_populates="item_list")
+
+

code_quality_analysis.ipynb

@@ -101,6 +101,7 @@ ignore = [
     "D103", # Ignore missing docstrings in tests
     "D100", # Ignore missing docstrings in public modules
 ]
+"backend/app/models/__init__.py" = ["RUF022"]
 
 [tool.bandit]
 exclude_dirs = ["app/tests"]