Fixed docs

Author: mr-fatalyst

docs/advanced/architecture.md

@@ -273,23 +273,37 @@ class RequestData:
 **Extractor Interface:**
 
 ```python
-class BaseAsyncRequestDataExtractor:
-    """Base async extractor"""
+class BaseAsyncRequestDataExtractor(BaseRequestDataExtractor, ABC):
+    """Base async extractor — overrides only body/form/files as async"""
+
+    @classmethod
+    @abstractmethod
+    async def _get_body(cls, request: Any) -> bytes | str | dict: ...
+
+    @classmethod
+    @abstractmethod
+    async def _get_form_data(cls, request: Any) -> dict: ...
+
+    @classmethod
+    @abstractmethod
+    async def _get_files(cls, request: Any) -> dict: ...
 
     @classmethod
     async def extract_request_data(cls, env: RequestEnvelope) -> RequestData:
-        """Extract data from framework request"""
+        request = env.request
         return RequestData(
-            path_params=await cls._get_path_params(env),
-            query_params=await cls._get_query_params(env),
-            headers=await cls._get_headers(env),
-            cookies=await cls._get_cookies(env),
-            body=await cls._get_body(env),
-            form_data=await cls._get_form_data(env),
-            files=await cls._get_files(env),
+            path_params=env.path_params or cls._get_path_params(request),
+            query_params=cls._get_query_params(request),          # sync
+            headers=cls._normalize_headers(cls._get_headers(request)),  # sync
+            cookies=cls._get_cookies(request),                    # sync
+            body=await cls._get_body(request),                    # async
+            form_data=await cls._get_form_data(request),          # async
+            files=await cls._get_files(request),                  # async
         )
 ```
 
+> **Note:** `_get_path_params`, `_get_query_params`, `_get_headers`, `_get_cookies` are **synchronous** methods inherited from `BaseRequestDataExtractor`. Only `_get_body`, `_get_form_data`, and `_get_files` are overridden as async. All `_get_*` methods receive the framework `request` object, not the `RequestEnvelope`.
+
 Each framework provides its own extractor (e.g., `StarletteRequestDataExtractor`, `FlaskRequestDataExtractor`).
 
 ### 2. Parameter Resolver
@@ -374,6 +388,7 @@ class DependencyResolver:
                 self._request_cache[request_data] = {
                     "resolved": {},
                     "resolving": set(),  # For circular detection
+                    "generators": [],  # For yield dependency cleanup
                 }
 
         try:
@@ -389,7 +404,7 @@ class DependencyResolver:
 - **Request-scoped caching** - Same dependency called twice = same instance
 - **Circular dependency detection** - Raises `CircularDependencyError`
 - **Security scopes validation** - Validates OAuth2 scopes
-- **Generator dependencies** - Planned support for `yield` setup/teardown (not yet implemented)
+- **Generator dependencies** - Supports `yield` setup/teardown for both sync and async generators with guaranteed cleanup
 - **Thread-safe** - Double-checked locking pattern
 - **Async support** - Separate async methods that handle both sync and async deps
 

docs/advanced/architecture_quick_reference.md

@@ -95,7 +95,7 @@ To add a new framework, implement:
 
 - [ ] **Extractor class** - Extract data from framework request
   - Inherit from `BaseAsyncRequestDataExtractor` or `BaseRequestDataExtractor`
-  - Implement `_extract_*` methods
+  - Implement `_get_*` methods (`_get_path_params`, `_get_query_params`, `_get_headers`, `_get_cookies`, `_get_body`, `_get_form_data`, `_get_files`)
 
 - [ ] **Router class** - Integrate with framework
   - Inherit from `BaseAdapter`

docs/api_reference/responses.md

@@ -402,25 +402,6 @@ def start_task(task_data: dict):
 
 ## Response Model Options
 
-### Exclude Unset Fields
-
-```python
-from pydantic import BaseModel
-
-class User(BaseModel):
-    id: int
-    username: str
-    email: str | None = None
-    full_name: str | None = None
-
-@router.get("/users/{user_id}", response_model=User)
-def get_user(user_id: int):
-    # Only returns fields that are set
-    return User(id=user_id, username="john")
-    # Returns: {"id": 1, "username": "john"}
-    # email and full_name are excluded
-```
-
 ### Exclude None Fields
 
 ```python

docs/examples/async_tasks.md

@@ -494,7 +494,7 @@ if __name__ == "__main__":
 import asyncio
 from datetime import datetime, timedelta
 from collections import defaultdict
-from fastopenapi import Depends
+from fastopenapi import Depends, Header
 from fastopenapi.errors import AuthorizationError
 
 class AsyncRateLimiter:

docs/examples/file_uploads.md

@@ -108,6 +108,7 @@ curl -X POST http://localhost:5000/upload-multiple \
 
 ```python
 from pydantic import BaseModel
+from fastopenapi import File, FileUpload, Form
 
 class DocumentMetadata(BaseModel):
     title: str
@@ -371,6 +372,7 @@ def upload_csv(file: FileUpload = File(...)):
 ## File Download
 
 ```python
+from fastopenapi import Response
 from fastopenapi.errors import ResourceNotFoundError
 
 @router.get("/download/{filename}", tags=["Files"])
@@ -418,7 +420,7 @@ def download_file(filename: str):
 from flask import Flask
 from pydantic import BaseModel
 from datetime import datetime
-from fastopenapi import File, FileUpload
+from fastopenapi import File, FileUpload, Form, Response
 from fastopenapi.routers import FlaskRouter
 from fastopenapi.errors import ResourceNotFoundError, BadRequestError
 import os

docs/examples/multi_router.md

@@ -107,20 +107,6 @@ async def upload_file(file: FileUpload = File(...)):
 
 ## Sanic-Specific Features
 
-### Using Sanic Request
-
-```python
-from sanic import request
-
-@router.get("/request-info")
-async def get_request_info():
-    return {
-        "ip": request.ip,
-        "host": request.host,
-        "scheme": request.scheme
-    }
-```
-
 ### Using Sanic Response
 
 ```python

docs/frameworks/sanic.md

@@ -108,20 +108,6 @@ async def upload_file(file: FileUpload = File(...)):
 
 ## Starlette-Specific Features
 
-### Using Starlette Request
-
-```python
-from starlette.requests import Request
-
-@router.get("/request-info")
-async def get_request_info(request: Request):
-    return {
-        "method": request.method,
-        "url": str(request.url),
-        "client": request.client.host
-    }
-```
-
 ### Using Starlette Response
 
 ```python
@@ -513,26 +499,6 @@ app.add_middleware(
 )
 ```
 
-## Session Management
-
-```python
-from starlette.middleware.sessions import SessionMiddleware
-
-app.add_middleware(SessionMiddleware, secret_key="your-secret-key")
-
-@router.post("/login")
-async def login(request: Request, username: str = Form(...)):
-    request.session["user"] = username
-    return {"message": "Logged in"}
-
-@router.get("/profile")
-async def profile(request: Request):
-    user = request.session.get("user")
-    if not user:
-        raise AuthenticationError("Not logged in")
-    return {"user": user}
-```
-
 ## Next Steps
 
 - [Core Concepts](../getting_started/core_concepts.md)

docs/frameworks/starlette.md

@@ -101,16 +101,10 @@ To contribute to FastOpenAPI or run tests, clone the repository and install in d
 ```bash
 git clone https://github.com/mr-fatalyst/fastopenapi.git
 cd fastopenapi
-pip install -e ".[dev]"
+poetry install --all-extras
 ```
 
-This installs FastOpenAPI in editable mode with all development dependencies including:
-
-- pytest - Testing framework
-- black - Code formatter
-- flake8 - Linter
-- mypy - Type checker
-- All supported frameworks for testing
+This installs FastOpenAPI with all development dependencies and framework extras.
 
 ## Verifying Installation
 

docs/getting_started/installation.md

@@ -415,26 +415,6 @@ def list_items(user_id: int | None = Depends(get_optional_token)):
         return get_public_items()
 ```
 
-### Conditional Dependencies
-
-```python
-def require_auth_if_private(is_private: bool = Query(False)):
-    if is_private:
-        return Depends(get_current_user)
-    return None
-
-@router.get("/content/{content_id}")
-def get_content(
-    content_id: int,
-    is_private: bool = Query(False),
-    user = Depends(require_auth_if_private(is_private))
-):
-    if is_private and not user:
-        raise AuthenticationError("Authentication required for private content")
-    
-    return get_content_by_id(content_id)
-```
-
 ### Dependency Composition
 
 ```python