Skip to content

fix: bump litellm lower bound to >=1.77.0#417

Merged
nabinchha merged 3 commits intomainfrom
nm/bump-litellm-lower-bound
Mar 13, 2026
Merged

fix: bump litellm lower bound to >=1.77.0#417
nabinchha merged 3 commits intomainfrom
nm/bump-litellm-lower-bound

Conversation

@nabinchha
Copy link
Contributor

@nabinchha nabinchha commented Mar 13, 2026

Summary

  • Bumps the litellm lower bound from >=1.73.6 to >=1.77.0 in the engine package. ImageURLListItem (imported in litellm_overrides.py) was introduced in litellm 1.77.0 — the previous lower bound allowed resolving versions (1.73.6–1.76.x) that don't have this class, causing an ImportError at runtime.

This version bump should have been included in #385, which added the ImageURLListItem import.

Test plan

  • Confirm from litellm.types.llms.openai import ImageURLListItem works with litellm 1.77.0
  • Run existing test_litellm_overrides.py tests pass

@nabinchha nabinchha requested a review from a team March 13, 2026 22:25
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Mar 13, 2026

Greptile Summary

This PR tightens the litellm lower bound in packages/data-designer-engine/pyproject.toml from >=1.73.6 to >=1.77.0 and updates uv.lock to match, preventing installation of litellm versions (1.73.6–1.76.x) that lack ImageURLListItem — the class imported in litellm_overrides.py and patched at runtime via patch_image_url_list_item().

  • The fix correctly closes an ImportError window introduced in PR fix: patch litellm ImageURLListItem to make index field optional (#384) #385 where from litellm.types.llms.openai import ImageURLListItem would fail on any litellm version below 1.77.0.
  • The upper bound <1.80.12 is unchanged and the currently-resolved version in uv.lock (1.80.11) satisfies the new range.
  • Both pyproject.toml and uv.lock are consistent with each other.
  • No logic, tests, or other source files are modified; the change is purely a dependency constraint correction.

Confidence Score: 5/5

  • This PR is safe to merge — it is a minimal, targeted dependency constraint fix with no code changes.
  • The change is a single lower-bound bump that closes a well-understood ImportError window. The uv.lock is consistent, the resolved version (1.80.11) satisfies the new range, and no callers, contracts, or other packages are affected.
  • No files require special attention.

Important Files Changed

Filename Overview
packages/data-designer-engine/pyproject.toml Bumps litellm lower bound from >=1.73.6 to >=1.77.0 to match the minimum version that provides ImageURLListItem; upper bound <1.80.12 unchanged and resolved lock version (1.80.11) satisfies the new range.
uv.lock requires-dist entry for data-designer-engine updated to reflect the new litellm specifier >=1.77.0,<1.80.12; resolved package version remains 1.80.11, which is valid within the new bounds.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[install data-designer-engine] --> B{litellm version resolved}
    B -->|Before fix 1.73.6 to 1.76.x| C[ImportError - ImageURLListItem missing]
    B -->|After fix 1.77.0 to 1.80.11| D[Import succeeds - ImageURLListItem available]
    D --> E[patch_image_url_list_item runs at startup]
    E --> F[ImageURLListItem.index made optional for OpenRouter compat]
Loading

Last reviewed commit: cac5397

@nabinchha nabinchha merged commit f7fd54e into main Mar 13, 2026
47 checks passed
@nabinchha nabinchha deleted the nm/bump-litellm-lower-bound branch March 13, 2026 22:28
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.

2 participants