Fix: Type safety and Routing issues for CI/CD. Added explicit model_validate calls and password casting.

Author: parvenuprompting

app/api/v1/admin/auth.py

@@ -22,7 +22,12 @@ async def login_access_token(
     OAuth2 compatible token login, get an access token for future requests.
     """
     user = await user_repo.get_by_email(session, email=form_data.username)
-    if not user or not await security.verify_password(form_data.password, user.hashed_password):
+    
+    # We explicitly cast to str() here to prevent Mypy from confusing
+    # the ORM Column definition with the actual string value.
+    if not user or not await security.verify_password(
+        form_data.password, str(user.hashed_password)
+    ):
         raise HTTPException(
             status_code=status.HTTP_400_BAD_REQUEST, detail="Incorrect email or password"
         )

app/api/v1/content/guides.py

@@ -26,7 +26,10 @@ async def read_guides(
     """
     Retrieve guides (Global + Private).
     """
-    return await service.get_guides(session, current_client=current_client, skip=skip, limit=limit)
+    guides = await service.get_guides(
+        session, current_client=current_client, skip=skip, limit=limit
+    )
+    return [GuideListResponse.model_validate(g) for g in guides]
 
 
 @router.post("/", response_model=GuideListResponse)
@@ -40,7 +43,10 @@ async def create_guide(
     """
     Create a new guide.
     """
-    return await service.create_guide(session, current_client=current_client, guide_in=guide_in)
+    guide = await service.create_guide(
+        session, current_client=current_client, guide_in=guide_in
+    )
+    return GuideListResponse.model_validate(guide)
 
 
 @router.get("/{guide_id}", response_model=GuideDetailResponse)
@@ -54,13 +60,15 @@ async def read_guide(
     """
     Get guide by ID.
     """
-    guide = await service.get_guide(session, current_client=current_client, guide_id=guide_id)
+    guide = await service.get_guide(
+        session, current_client=current_client, guide_id=guide_id
+    )
     if not guide:
         raise HTTPException(
             status_code=status.HTTP_404_NOT_FOUND,
             detail="Guide not found or access denied",
         )
-    return guide
+    return GuideDetailResponse.model_validate(guide)
 
 
 @router.put("/{guide_id}", response_model=GuideListResponse)
@@ -86,4 +94,4 @@ async def update_guide(
             status_code=status.HTTP_404_NOT_FOUND,
             detail="Guide not found or access denied",
         )
-    return guide
+    return GuideListResponse.model_validate(guide)

app/api/v1/content/topics.py

@@ -22,7 +22,11 @@ async def read_topics(
     Retrieve topics.
     Clients only see Global topics or their own Private topics.
     """
-    return await service.get_topics(session, current_client=current_client, skip=skip, limit=limit)
+    topics = await service.get_topics(
+        session, current_client=current_client, skip=skip, limit=limit
+    )
+    # Explicit conversion for Mypy
+    return [TopicResponse.model_validate(t) for t in topics]
 
 
 @router.post("/", response_model=TopicResponse)
@@ -36,7 +40,10 @@ async def create_topic(
     """
     Create a new topic.
     """
-    return await service.create_topic(session, current_client=current_client, topic_in=topic_in)
+    topic = await service.create_topic(
+        session, current_client=current_client, topic_in=topic_in
+    )
+    return TopicResponse.model_validate(topic)
 
 
 @router.put("/{topic_id}", response_model=TopicResponse)
@@ -59,7 +66,7 @@ async def update_topic(
             status_code=status.HTTP_404_NOT_FOUND,
             detail="Topic not found or access denied",
         )
-    return topic
+    return TopicResponse.model_validate(topic)
 
 
 @router.get("/{topic_id}", response_model=TopicDetail)
@@ -73,10 +80,12 @@ async def read_topic(
     """
     Get topic by ID.
     """
-    topic = await service.get_topic(session, current_client=current_client, topic_id=topic_id)
+    topic = await service.get_topic(
+        session, current_client=current_client, topic_id=topic_id
+    )
     if not topic:
         raise HTTPException(
             status_code=status.HTTP_404_NOT_FOUND,
             detail="Topic not found or access denied",
         )
-    return topic
+    return TopicDetail.model_validate(topic)