Resolve personas/list 422 errors

Author: NeonDaniel

chat_server/blueprints/personas.py

@@ -26,7 +26,7 @@
 # NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS
 # SOFTWARE,  EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 
-from fastapi import APIRouter
+from fastapi import APIRouter, Query, Depends
 from starlette.responses import JSONResponse
 
 from chat_server.utils.enums import RequestModelType, UserRoles
@@ -48,6 +48,7 @@
     PersonaData,
 )
 from chat_server.utils.api_dependencies.validators import permitted_access
+from chat_server.utils.api_dependencies.direct_query import list_personas_query
 from klatchat_utils.database_utils.mongo_utils import MongoFilter, MongoLogicalOperators
 from klatchat_utils.database_utils.mongo_utils.queries.wrapper import MongoDocumentsAPI
 
@@ -60,7 +61,7 @@
 @router.get("/list")
 async def list_personas(
     current_user: CurrentUserData,
-    request_model: ListPersonasQueryModel = permitted_access(ListPersonasQueryModel),
+    request_model: ListPersonasQueryModel = Depends(list_personas_query),
 ) -> JSONResponse:
     """Lists personas matching query params"""
     filters = []

chat_server/utils/api_dependencies/direct_query.py

@@ -0,0 +1,21 @@
+"""
+Direct query parameter handling for FastAPI endpoints.
+"""
+
+from fastapi import Query
+from typing import Optional, Type, List
+from pydantic import BaseModel
+
+from chat_server.utils.api_dependencies.models import ListPersonasQueryModel
+
+
+def list_personas_query(
+    llms: Optional[List[str]] = Query(None),
+    user_id: Optional[str] = Query(None),
+    only_enabled: bool = Query(False),
+) -> ListPersonasQueryModel:
+    """
+    Creates a ListPersonasQueryModel from direct query parameters.
+    This avoids the issue with FastAPI's dependency injection system.
+    """
+    return ListPersonasQueryModel(llms=llms, user_id=user_id, only_enabled=only_enabled)

chat_server/utils/api_dependencies/models/personas.py

@@ -28,6 +28,7 @@
 
 from fastapi import Query
 from pydantic import Field, computed_field, BaseModel
+from typing import Optional
 
 
 class PersonaModel(BaseModel):
@@ -84,6 +85,6 @@ class TogglePersonaStatusModel(PersonaModel):
 
 
 class ListPersonasQueryModel(BaseModel):
-    llms: list[str] | None = Field(Query(default=None), examples=[["doctor"]])
-    user_id: str | None = Field(Query(default=None), examples=["test_user_id"])
-    only_enabled: bool = Field(Query(default=False), examples=[True, False])
+    llms: list[str] | None = Field(default=None, examples=[["doctor"]])
+    user_id: str | None = Field(default=None, examples=["test_user_id"])
+    only_enabled: bool = Field(default=False, examples=[True, False])