Skip to content

Fix: remove top-level LiteLLM import (optional dep)#60

Merged
cosminacho merged 2 commits intomainfrom
fix/remove-litellm-optional-import
Apr 13, 2026
Merged

Fix: remove top-level LiteLLM import (optional dep)#60
cosminacho merged 2 commits intomainfrom
fix/remove-litellm-optional-import

Conversation

@cosminacho
Copy link
Copy Markdown
Collaborator

Summary

  • Removed the top-level UiPathLiteLLM import from __init__.py — it caused an ImportError when the optional litellm extra was not installed
  • Bumped core version from 1.8.01.8.1 to sync with langchain 1.8.1
  • Updated langchain pyproject.toml dependency to uipath-llm-client>=1.8.1

Packages affected

  • core (uipath-llm-client): version bump + import fix
  • langchain (uipath-langchain-client): dependency constraint update

Test plan

  • ruff check passes
  • ruff format --check passes
  • pyright passes (0 errors)
  • pytest tests — 1502 passed, 736 skipped

🤖 Generated with Claude Code

UiPathLiteLLM was imported at the top level of __init__.py, causing an
ImportError for users who don't have the litellm extra installed.
Removed the eager import so the core package works without litellm.

Bumps core version to 1.8.1 to sync with langchain 1.8.1.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@cosminacho cosminacho deployed to LLMGW_SETTINGS April 13, 2026 11:41 — with GitHub Actions Active
@cosminacho cosminacho merged commit 3c747c5 into main Apr 13, 2026
7 checks passed
@cosminacho cosminacho deleted the fix/remove-litellm-optional-import branch April 13, 2026 11:45
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.

1 participant