Skip to content

fix: align optional-extras pins so uv sync resolves - #2794

Merged
Lancetnik merged 2 commits into
ag2ai:mainfrom
SarthakB11:fix/issue-2770
May 10, 2026
Merged

Lancetnik merged 2 commits into
ag2ai:mainfrom
SarthakB11:fix/issue-2770

Conversation

@SarthakB11

Copy link
Copy Markdown
Contributor

Why are these changes needed?

uv sync fails on a fresh clone because several optional extras in pyproject.toml carry pins that conflict with each other or with their own transitive dependencies. pip install resolves greedily for the active interpreter and silently picks one path, so the conflicts stay invisible to it; uv cross-resolves the full extras matrix and surfaces them. Issue #2770 enumerates the specific cases.

pyproject.toml only:

  • [rag]: widen llama-index-llms-openai and llama-index-embeddings-openai upper bounds to <0.8. llama-index>=0.14.13 transitively requires the newer sub-package versions, so the previous <0.6 pins made [rag] internally inconsistent.
  • [neo4j]: align llama-index / llama-index-core to >=0.14.13,<0.15 (matching [rag]) so the two extras can co-install. Bump llama-index-graph-stores-neo4j and llama-index-readers-web upper bounds to <0.7 accordingly.
  • [autobuild]: align chromadb to >=1.4.1,<1.5 (matching [retrievechat]).
  • [test]: replace exact pins with ranges. fastapi>=0.115.0,<1 (was ==0.116.1, blocked [nlip] whose nlip-server requires fastapi<0.116.0) and pandas>=2.2.0,<3 (was ==2.3.3, conflicted with llama-index-readers-file<0.5.6).

Validated locally: uv lock --no-cache resolves successfully on Python 3.13 with uv 0.11.7 (802 packages); without these changes resolution fails with the conflicts listed in #2770.

The graphrag_sdk==0.8.2 / pypdf / pandas chain (issue conflict #5) is not pinned here; it resolves transitively through llama-index-readers-file. Adding uv sync to CI (issue suggestion #5) is intentionally out of scope.

Related issue number

Fixes #2770

Checks

AI assistance

  • I understand the changes in this PR and can explain them in my own words.
  • I have verified that the PR description accurately reflects the actual diff.
  • If AI assistance was used, I reviewed, tested, and validated the generated code/text before submitting.

@vvlrff vvlrff self-assigned this May 9, 2026
@vvlrff

vvlrff commented May 9, 2026

Copy link
Copy Markdown
Collaborator

@SarthakB11 Thank you for the pull request! @Lancetnik I've verified that these changes solve the uv sync issue.

@codecov

codecov Bot commented May 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
see 52 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@Lancetnik
Lancetnik added this pull request to the merge queue May 10, 2026
Merged via the queue into ag2ai:main with commit edb5942 May 10, 2026
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Dependency conflicts prevent uv sync from resolving

3 participants