svc-infra 0.1.589__py3-none-any.whl → 0.1.706__py3-none-any.whl

svc_infra/api/fastapi/db/nosql/mongo/add.py

@@ -24,12 +24,15 @@ from .health import make_mongo_health_router
 def add_mongo_db_with_url(app: FastAPI, url: str, db_name: str) -> None:
     @asynccontextmanager
     async def lifespan(_app: FastAPI):
-        await init_mongo(MongoSettings(url=url, db_name=db_name))
+        # MongoSettings expects url as AnyUrl, which can be constructed from str via Pydantic
+        await init_mongo(MongoSettings(url=url, db_name=db_name))  # type: ignore[arg-type]  # Pydantic coerces str to AnyUrl
         try:
             expected = get_mongo_dbname_from_env(required=False)
             db = await acquire_db()
             if expected and db.name != expected:
-                raise RuntimeError(f"Connected to Mongo DB '{db.name}', expected '{expected}'.")
+                raise RuntimeError(
+                    f"Connected to Mongo DB '{db.name}', expected '{expected}'."
+                )
             yield
         finally:
             await close_mongo()
@@ -38,19 +41,23 @@ def add_mongo_db_with_url(app: FastAPI, url: str, db_name: str) -> None:
 
 
 def add_mongo_db(app: FastAPI, *, dsn_env: str = "MONGO_URL") -> None:
-    @app.on_event("startup")
-    async def _startup() -> None:
+    @asynccontextmanager
+    async def lifespan(_app: FastAPI):
         if not os.getenv(dsn_env):
             raise RuntimeError(f"Missing environment variable {dsn_env} for Mongo URL")
         await init_mongo()
         expected = get_mongo_dbname_from_env(required=False)
         db = await acquire_db()
         if expected and db.name != expected:
-            raise RuntimeError(f"Connected to Mongo DB '{db.name}', expected '{expected}'.")
+            raise RuntimeError(
+                f"Connected to Mongo DB '{db.name}', expected '{expected}'."
+            )
+        try:
+            yield
+        finally:
+            await close_mongo()
 
-    @app.on_event("shutdown")
-    async def _shutdown() -> None:
-        await close_mongo()
+    app.router.lifespan_context = lifespan
 
 
 def add_mongo_health(
@@ -58,50 +65,58 @@ def add_mongo_health(
 ) -> None:
     if include_in_schema is None:
         include_in_schema = CURRENT_ENVIRONMENT == LOCAL_ENV
-    app.include_router(make_mongo_health_router(prefix=prefix, include_in_schema=include_in_schema))
+    app.include_router(
+        make_mongo_health_router(prefix=prefix, include_in_schema=include_in_schema)
+    )
 
 
 def add_mongo_resources(app: FastAPI, resources: Sequence[NoSqlResource]) -> None:
-    for r in resources:
+    for resource in resources:
         repo = NoSqlRepository(
-            collection_name=r.resolved_collection(),
-            id_field=r.id_field,
-            soft_delete=r.soft_delete,
-            soft_delete_field=r.soft_delete_field,
-            soft_delete_flag_field=r.soft_delete_flag_field,
+            collection_name=resource.resolved_collection(),
+            id_field=resource.id_field,
+            soft_delete=resource.soft_delete,
+            soft_delete_field=resource.soft_delete_field,
+            soft_delete_flag_field=resource.soft_delete_flag_field,
+        )
+        svc = (
+            resource.service_factory(repo)
+            if resource.service_factory
+            else NoSqlService(repo)
         )
-        svc = r.service_factory(repo) if r.service_factory else NoSqlService(repo)
 
-        if r.read_schema and r.create_schema and r.update_schema:
-            Read, Create, Update = r.read_schema, r.create_schema, r.update_schema
-        elif r.document_model is not None:
+        if resource.read_schema and resource.create_schema and resource.update_schema:
+            Read, Create, Update = (
+                resource.read_schema,
+                resource.create_schema,
+                resource.update_schema,
+            )
+        elif resource.document_model is not None:
             # CRITICAL: teach Pydantic to dump ObjectId/PyObjectId
             Read, Create, Update = make_document_crud_schemas(
-                r.document_model,
-                create_exclude=r.create_exclude,
-                read_name=r.read_name,
-                create_name=r.create_name,
-                update_name=r.update_name,
-                read_exclude=r.read_exclude,
-                update_exclude=r.update_exclude,
+                resource.document_model,
+                create_exclude=resource.create_exclude,
+                read_name=resource.read_name,
+                create_name=resource.create_name,
+                update_name=resource.update_name,
+                read_exclude=resource.read_exclude,
+                update_exclude=resource.update_exclude,
                 json_encoders={ObjectId: str, PyObjectId: str},
             )
         else:
             raise RuntimeError(
-                f"Resource for collection '{r.collection}' requires either explicit schemas "
+                f"Resource for collection '{resource.collection}' requires either explicit schemas "
                 f"(read/create/update) or a 'document_model' to derive them."
             )
 
         router = make_crud_router_plus_mongo(
-            collection=r.resolved_collection(),
-            repo=repo,
             service=svc,
             read_schema=Read,
             create_schema=Create,
             update_schema=Update,
-            prefix=r.prefix,
-            tags=r.tags,
-            search_fields=r.search_fields,
+            prefix=resource.prefix,
+            tags=resource.tags,
+            search_fields=resource.search_fields,
             default_ordering=None,
             allowed_order_fields=None,
         )

svc_infra/api/fastapi/db/nosql/mongo/crud_router.py

@@ -1,7 +1,14 @@
 from typing import Annotated, Any, Optional, Sequence, Type, cast
 
 from fastapi import APIRouter, Body, Depends, HTTPException
-from motor.motor_asyncio import AsyncIOMotorDatabase
+
+try:
+    from motor.motor_asyncio import AsyncIOMotorDatabase
+
+    HAS_MOTOR = True
+except ImportError:  # pragma: no cover
+    HAS_MOTOR = False
+    AsyncIOMotorDatabase = Any  # type: ignore[assignment, misc]
 
 from svc_infra.api.fastapi.db.http import (
     LimitOffsetParams,
@@ -46,6 +53,9 @@ def make_crud_router_plus_mongo(
     allowed_order_fields: Optional[list[str]] = None,
     mount_under_db_prefix: bool = True,
 ) -> APIRouter:
+    read_model = cast(Any, read_schema)
+    page_model = cast(Any, Page[read_schema])  # type: ignore[valid-type]
+
     router_prefix = ("/_mongo" + prefix) if mount_under_db_prefix else prefix
     router = public_router(
         prefix=router_prefix,
@@ -56,7 +66,7 @@ def make_crud_router_plus_mongo(
     # LIST
     @router.get(
         "",
-        response_model=cast(Any, Page[read_schema]),
+        response_model=page_model,
         description=f"List items in {prefix} collection",
     )
     async def list_items(
@@ -68,20 +78,25 @@ def make_crud_router_plus_mongo(
         sort = _parse_sort(op.order_by or default_ordering, allowed_order_fields)
         if sp.q and search_fields:
             items = await service.search(
-                db, q=sp.q, fields=search_fields, limit=lp.limit, offset=lp.offset, sort=sort
+                db,
+                q=sp.q,
+                fields=search_fields,
+                limit=lp.limit,
+                offset=lp.offset,
+                sort=sort,
             )
             total = await service.count_filtered(db, q=sp.q, fields=search_fields)
         else:
             items = await service.list(db, limit=lp.limit, offset=lp.offset, sort=sort)
             total = await service.count(db)
-        return Page[read_schema].from_items(
+        return Page[Any].from_items(
             total=total, items=items, limit=lp.limit, offset=lp.offset
         )
 
     # GET by id
     @router.get(
         "/{item_id}",
-        response_model=cast(Any, read_schema),
+        response_model=read_model,
         description=f"Get item from {prefix} collection",
     )
     async def get_item(db: DBDep, item_id: Any):
@@ -93,22 +108,26 @@ def make_crud_router_plus_mongo(
     # CREATE
     @router.post(
         "",
-        response_model=cast(Any, read_schema),
+        response_model=read_model,
         status_code=201,
         description=f"Create item in {prefix} collection",
     )
-    async def create_item(db: DBDep, payload: create_schema = Body(...)):
-        data = payload.model_dump(exclude_unset=True)
+    async def create_item(db: DBDep, payload: create_schema = Body(...)):  # type: ignore[valid-type]
+        data = cast(Any, payload).model_dump(exclude_unset=True)
         return await service.create(db, data)
 
     # UPDATE
     @router.patch(
         "/{item_id}",
-        response_model=cast(Any, read_schema),
+        response_model=read_model,
         description=f"Update item in {prefix} collection",
     )
-    async def update_item(db: DBDep, item_id: Any, payload: update_schema = Body(...)):
-        data = payload.model_dump(exclude_unset=True)
+    async def update_item(
+        db: DBDep,
+        item_id: Any,
+        payload: update_schema = Body(...),  # type: ignore[valid-type]
+    ):
+        data = cast(Any, payload).model_dump(exclude_unset=True)
         row = await service.update(db, item_id, data)
         if not row:
             raise HTTPException(404, "Not found")

svc_infra/api/fastapi/db/sql/__init__.py

@@ -1,4 +1,8 @@
-from svc_infra.api.fastapi.db.sql.add import add_sql_db, add_sql_health, add_sql_resources
+from svc_infra.api.fastapi.db.sql.add import (
+    add_sql_db,
+    add_sql_health,
+    add_sql_resources,
+)
 from svc_infra.api.fastapi.db.sql.session import SqlSessionDep
 
 __all__ = [

svc_infra/api/fastapi/db/sql/add.py

@@ -10,14 +10,16 @@ from svc_infra.db.sql.management import make_crud_schemas
 from svc_infra.db.sql.repository import SqlRepository
 from svc_infra.db.sql.resource import SqlResource
 
-from .crud_router import make_crud_router_plus_sql
+from .crud_router import make_crud_router_plus_sql, make_tenant_crud_router_plus_sql
 from .health import _make_db_health_router
 from .session import dispose_session, initialize_session
 
 
 def add_sql_resources(app: FastAPI, resources: Sequence[SqlResource]) -> None:
     for r in resources:
-        repo = SqlRepository(model=r.model, id_attr=r.id_attr, soft_delete=r.soft_delete)
+        repo = SqlRepository(
+            model=r.model, id_attr=r.id_attr, soft_delete=r.soft_delete
+        )
 
         if r.service_factory:
             svc = r.service_factory(repo)
@@ -37,52 +39,95 @@ def add_sql_resources(app: FastAPI, resources: Sequence[SqlResource]) -> None:
                 update_name=r.update_name,
             )
 
-        router = make_crud_router_plus_sql(
-            model=r.model,
-            service=svc,
-            read_schema=Read,
-            create_schema=Create,
-            update_schema=Update,
-            prefix=r.prefix,
-            tags=r.tags,
-            search_fields=r.search_fields,
-            default_ordering=r.ordering_default,
-            allowed_order_fields=r.allowed_order_fields,
-        )
+        if r.tenant_field:
+            # wrap service factory/instance through tenant router
+            def _factory():
+                return svc
+
+            router = make_tenant_crud_router_plus_sql(
+                model=r.model,
+                service_factory=_factory,
+                read_schema=Read,
+                create_schema=Create,
+                update_schema=Update,
+                prefix=r.prefix,
+                tenant_field=r.tenant_field,
+                tags=r.tags,
+                search_fields=r.search_fields,
+                default_ordering=r.ordering_default,
+                allowed_order_fields=r.allowed_order_fields,
+            )
+        else:
+            router = make_crud_router_plus_sql(
+                model=r.model,
+                service=svc,
+                read_schema=Read,
+                create_schema=Create,
+                update_schema=Update,
+                prefix=r.prefix,
+                tags=r.tags,
+                search_fields=r.search_fields,
+                default_ordering=r.ordering_default,
+                allowed_order_fields=r.allowed_order_fields,
+            )
         app.include_router(router)
 
 
-def add_sql_db(app: FastAPI, *, url: Optional[str] = None, dsn_env: str = "SQL_URL") -> None:
-    """Configure DB lifecycle for the app (either explicit URL or from env)."""
+def add_sql_db(
+    app: FastAPI, *, url: Optional[str] = None, dsn_env: str = "SQL_URL"
+) -> None:
+    """Configure DB lifecycle for the app (either explicit URL or from env).
+
+    This preserves any existing lifespan context (like user-defined lifespans)
+    and wraps it with the database session initialization/cleanup.
+    """
+    # Preserve existing lifespan to wrap it
+    existing_lifespan = getattr(app.router, "lifespan_context", None)
+
     if url:
 
         @asynccontextmanager
-        async def lifespan(_app: FastAPI):
+        async def lifespan_with_url(_app: FastAPI):
             initialize_session(url)
             try:
-                yield
+                if existing_lifespan is not None:
+                    async with existing_lifespan(_app):
+                        yield
+                else:
+                    yield
             finally:
                 await dispose_session()
 
-        app.router.lifespan_context = lifespan
+        app.router.lifespan_context = lifespan_with_url
         return
 
-    @app.on_event("startup")
-    async def _startup() -> None:  # noqa: ANN202
+    # Use lifespan context manager instead of deprecated on_event
+    @asynccontextmanager
+    async def lifespan_from_env(_app: FastAPI):
         env_url = os.getenv(dsn_env)
         if not env_url:
-            raise RuntimeError(f"Missing environment variable {dsn_env} for database URL")
+            raise RuntimeError(
+                f"Missing environment variable {dsn_env} for database URL"
+            )
         initialize_session(env_url)
+        try:
+            if existing_lifespan is not None:
+                async with existing_lifespan(_app):
+                    yield
+            else:
+                yield
+        finally:
+            await dispose_session()
 
-    @app.on_event("shutdown")
-    async def _shutdown() -> None:  # noqa: ANN202
-        await dispose_session()
+    app.router.lifespan_context = lifespan_from_env
 
 
 def add_sql_health(
     app: FastAPI, *, prefix: str = "/_sql/health", include_in_schema: bool = False
 ) -> None:
-    app.include_router(_make_db_health_router(prefix=prefix, include_in_schema=include_in_schema))
+    app.include_router(
+        _make_db_health_router(prefix=prefix, include_in_schema=include_in_schema)
+    )
 
 
 def setup_sql(