fix: add litellm[proxy] dependency to fix missing fastapi on fresh install#1737
Conversation
…stall Fixes agent0ai#1728. On a fresh install, litellm raises APIConnectionError because fastapi is not installed. Adding the [proxy] extra to the litellm dependency ensures fastapi is installed automatically.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 4384e506f4
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| langchain-unstructured==0.1.6 | ||
| liteparse==2.0.3 | ||
| litellm==1.88.1 # CVE-2026-42271 fix: patched floor is 1.83.7 | ||
| litellm[proxy]==1.88.1 # CVE-2026-42271 fix: patched floor is 1.83.7 |
There was a problem hiding this comment.
Avoid installing proprietary LiteLLM enterprise code
In fresh installs (including Docker builds that run uv pip install -r /git/agent-zero/requirements.txt), the proxy extra does much more than add FastAPI: LiteLLM 1.88.1 declares litellm-enterprise==0.1.42; extra == "proxy", and that package is published as LicenseRef-Proprietary. This pulls proprietary enterprise code into every MIT Agent Zero installation even though the repo only imports the LiteLLM SDK; adding the missing FastAPI/Starlette dependencies directly would avoid changing the redistribution/licensing footprint.
Useful? React with 👍 / 👎.
Issue
Fixes #1728
After installing A0 2.0/2.1 from a fresh environment, users encounter:
Root Cause
The
litellmpackage listsfastapias an optional dependency under the[proxy]extra. Withoutfastapiinstalled, litellm cannot make HTTP connections and raisesAPIConnectionError.Fix
Change the
litellmdependency inrequirements.txtfrom:Adding the
[proxy]extra ensuresfastapi(and other proxy-related dependencies) are installed automatically alongside litellm.Verification
requirements.txtis modified==1.88.1) and CVE comment are preserved