E52F feat: Add uipath_langchain_client, refactor code to use the new client by cosminacho · Pull Request #634 · UiPath/uipath-langchain-python · GitHub
[go: up one dir, main page]
Skip to content

feat: Add uipath_langchain_client, refactor code to use the new client#634

Open
cosminacho wants to merge 1 commit intomainfrom
feat/uipath_llm_client
Open

feat: Add uipath_langchain_client, refactor code to use the new client#634
cosminacho wants to merge 1 commit intomainfrom
feat/uipath_llm_client

Conversation

@cosminacho
Copy link
Contributor
@cosminacho cosminacho commented Feb 26, 2026 •
edited by github-actions bot
Loading

This new client has 2 main components:

  1. uipath-llm-client
  • this is the core package on which uipath-langchain-client depends
  • it handles URL, headers and authentification for both AgentHub and LLMGateway
  • Encapsulates all the auth settings as well as retries, custom exceptions, byo stuff, custom headers for streaming etc into an httx.Client
  • it also has passthrough httpx clients for providers like openai, anthropic, gemini (using their clients)
  • can be added with: uv add "uipath-llm-client[openai, google, all]" etc. Without specifying extra dependencies it installs only the main httpx client that does the url, headers and auth stuff
  1. uipath-langchain-client
  • uses the httpx client from uipath-llm-client
  • the passthrough implementations are minimal, all passthrough clients use a httpx client to make requests, we can just use the httpx client created above and inject it in the langchain-client for that vendor
  • it also has a factory method which does a GET on the discovery endpoint and correctly instantiates the appropiate passthrough client for that model_name and the vendor detected

Summary of the Changes for this package:

  • Retrying moved to the uipath-langchain-client
  • Request Mixin removed because it’s on the uipath-langchain-client now as UiPathBaseLLMClient
  • Streaming fixes added to uipath-langchain-client
  • Auth headers, url creation and refresh token with uipath auth moved to uipath-langchain-client
  • All clients implement UiPathBaseChatModel which has api_config from where we can get vendor_type and api_flavor
  • Retryers are handled in uipath-langchain-client at transport level on the httpx client, so no need to hardcode for each provider
  • Factory is now in uipath-langchain-client (and it uses the discovery endpoint)
  • Gemini streaming fixes moved
  • Comprehensive tests are in the package, we can remove them from here
  • Also the normalized client is in the new client
  • merging generation info with response metadata in the client

Before merging this PR I think it's important to make sure that all the relevant components have been moved to the new client, and also that the new changes aren't breaking stuff that's working

Not tested yet:

  • on Bedrock: Convert file_blocks_to_anthropic_documents

Development Package

  • Use uipath pack --nolock to get the latest dev build from this PR (requires version range).
  • Add this package as a dependency in your pyproject.toml:
[project]
dependencies = [
  # Exact version:
  "uipath-langchain==1.0.0.dev1006343424",

  # Any version from PR
  "uipath-langchain>=1.0.0.dev1006340000,<1.0.0.dev1006350000"
]

[[tool.uv.index]]
name = "testpypi"
url = "https://test.pypi.org/simple/"
publish-url = "https://test.pypi.org/legacy/"
explicit = true

[tool.uv.sources]
uipath-langchain = { index = "testpypi" }

[tool.uv]
override-dependencies = [
    "uipath-langchain>=1.0.0.dev1006340000,<1.0.0.dev1006350000",
]

@cosminacho cosminacho force-pushed the feat/uipath_llm_client branch 20 times, most recently from ba8fc68 to 0455d90 Compare March 2, 2026 21:34
@cosminacho cosminacho force-pushed the feat/uipath_llm_client branch 4 times, most recently from 78278b5 to e92f41c Compare March 10, 2026 12:59
@cosminacho cosminacho force-pushed the feat/uipath_llm_client branch 2 times, most recently from f5ee4b8 to 0a73223 Compare March 12, 2026 12:45
@cosminacho cosminacho force-pushed the feat/uipath_llm_client branch 3 times, most recently from e76f2ab to e8f5c07 Compare March 12, 2026 17:03
@cosminacho cosminacho force-pushed the feat/uipath_llm_client branch 8 times, most recently from d6768e1 to aeab860 Compare March 13, 2026 00:04
@cosminacho cosminacho changed the title [WIP] Add uipath_langchain_client, refactor code to use the new client Add uipath_langchain_client, refactor code to use the new client Mar 13, 2026
@cosminacho cosminacho force-pushed the feat/uipath_llm_client branch 6 times, most recently from 00a647f to face9d8 Compare March 13, 2026 08:49
ionmincu
ionmincu reviewed Mar 13, 2026
View reviewed changes
@cosminacho cosminacho changed the title Add uipath_langchain_client, refactor code to use the new client feat: Add uipath_langchain_client, refactor code to use the new client Mar 13, 2026
@cosminacho cosminacho force-pushed the feat/uipath_llm_client branch from face9d8 to 3e9a1ab Compare March 13, 2026 12:20
radu-mocanu
radu-mocanu requested changes Mar 16, 2026
View reviewed changes
Copy link
Collaborator
@radu-mocanu radu-mocanu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we are trying to decouple uipath-langchain from uipath library. the uipath-llm-client will need to use uipath-platform here https://github.com/UiPath/uipath-llm-client-python/blob/main/pyproject.toml#L12

we are moving the relevant auth methods as part of this PR: UiPath/uipath-python#1439

@cosminacho cosminacho force-pushed the feat/uipath_llm_client branch from 3e9a1ab to 6bcd885 Compare March 16, 2026 14:05
@cosminacho cosminacho force-pushed the feat/uipath_llm_client branch from 6bcd885 to 2124633 Compare March 16, 2026 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ionmincu ionmincu ionmincu left review comments

@radu-mocanu radu-mocanu radu-mocanu requested changes

@andrei-rusu andrei-rusu andrei-rusu approved these changes

Requested changes must be addressed to merge this pull request.

Assignees

No one assigned

Labels

build:dev

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@cosminacho @andrei-rusu @ionmincu @radu-mocanu
0