fastapi-code-review

These are idiomatic FastAPI patterns that may appear problematic but are correct: Only flag issues when the context warrants it: FastAPI + Pydantic handle many concerns automatically: Before flagging "missing" functionality, verify FastAPI isn't handling it. Load and follow review-verification-protocol before reporting any issue. - Pydantic validates request body automatically - No manual validation needed when using typed Pydantic models as parameters - Dependency injection for database sessions - Sessions come from Depends() , not passed as function arguments - HTTPException for all HTTP errors - FastAPI handles conversion to proper HTTP responses - Async def endpoint without await - May be using sync dependencies or simple operations; FastAPI handles this - Type annotation on Depends() - This is documentation/IDE support, not a type assertion - Query/Path/Body defaults - FastAPI processes these at runtime, not traditional Python defaults - Returning dict from endpoint - Pydantic converts automatically if response_model is set - Flag missing validation ONLY IF the field isn't already in a Pydantic model with validators - Flag missing auth ONLY IF the endpoint isn't using Depends() with an auth dependency - Flag missing error handling ONLY IF HTTPException isn't raised appropriately for error cases - Flag sync in async ONLY IF the operation is actually blocking (file I/O,...