The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
litellm	Ready	Preview, Comment	Mar 28, 2026 6:25pm

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

Merging this PR will not alter performance

✅ 16 untouched benchmarks

_{Comparing litellm_gha_p2 (7851567) with main (428d837)}

Greptile Summary

This PR fixes two categories of flaky CI failures: leaked MagicMock state causing TypeError in the TestAgentHealthCheck tests under pytest-xdist, and CWD-dependent path resolution breaking documentation validation tests when run from a non-standard working directory. The fixes are targeted and well-explained.

• test_endpoints.py: monkeypatch.setattr(\"litellm.proxy.proxy_server.prisma_client\", None) cleanly isolates the health-check tests from cross-scope mock leakage — correct approach that does not weaken coverage.
• test_router_settings.py: Path resolution replaced with __file__-based _repo_root — correct fix, but the old variable name repo_base is still referenced in the except block (line 64), which will raise a NameError and hide the original error if the docs file cannot be opened.
• config_settings.md: ~25 new general_settings / router_settings entries backfill documentation that was causing the validation test to fail.
• test-unit-documentation.yml: Restores the documentation CI workflow now that the path issue is resolved; uses pinned action commit SHAs throughout.

Confidence Score: 4/5

Safe to merge after fixing the stale repo_base reference in the except block of test_router_settings.py.

One P1 bug was introduced: repo_base was removed but is still referenced in the except block, meaning any future failure to open the docs file will surface as a confusing NameError rather than the intended diagnostic message. All other changes are clean and do not weaken any existing assertions or coverage.

tests/documentation_tests/test_router_settings.py — stale repo_base reference on line 64

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[pytest-xdist worker starts TestAgentHealthCheck] --> B{prisma_client leaked\nfrom another scope?}
    B -- Before fix: Yes --> C["prisma_client = MagicMock()"]
    C --> D["GET /v1/agents calls find_many()"]
    D --> E["TypeError: MagicMock can't\nbe used in await → 500"]
    B -- After fix: monkeypatch sets None --> F["prisma_client = None"]
    F --> G["GET /v1/agents skips DB query"]
    G --> H["200 OK — uses mock_registry only"]

    subgraph doc_tests["Documentation Test Path Resolution"]
        I["poetry run python ./tests/documentation_tests/test_router_settings.py"]
        I --> J["_test_dir = os.path.dirname(os.path.abspath(__file__))"]
        J --> K["_repo_root = os.path.abspath(join(_test_dir, '..', '..'))"]
        K --> L["docs_path = _repo_root/docs/my-website/..."]
        L --> M{File exists?}
        M -- Yes --> N["Parse & validate docs"]
        M -- No --> O["except block raises Exception\n⚠ repo_base NameError here"]
    end

_{Reviews (2): Last reviewed commit: "[Fix] Scope documentation workflow to ma..." | Re-trigger Greptile}