feat: Integrate UiPath-LangChain-Client and cleanup old clients#496
Open
cosminacho wants to merge 1 commit intomainfrom
Open
feat: Integrate UiPath-LangChain-Client and cleanup old clients#496cosminacho wants to merge 1 commit intomainfrom
cosminacho wants to merge 1 commit intomainfrom
Conversation
cristipufu
reviewed
Jan 29, 2026
cristipufu
reviewed
Jan 29, 2026
44addbc to
163ce88
Compare
cristipufu
reviewed
Jan 30, 2026
b9fc17d to
e64365a
Compare
e6c3ebf to
304f333
Compare
Contributor
|
The discovery API returns the API Flavors that a model supports. Some clients with BYO do support only one API flavor, for example just bedrock converse or just bedrock invoke. I don't see where the API Flavor returned by the LLMGw on the discovery API is taking into account when instantiating the LLM client. I see that for bedrock api_config.api_flavor is always set to "invoke" (in uipath_langchain_client), which I think is wrong. |
304f333 to
6b6bdb6
Compare
d325c0a to
8b2ce8d
Compare
8b2ce8d to
212e3d1
Compare
e4f96c5 to
7c87d75
Compare
dd6638f to
e7a130c
Compare
e7a130c to
a7279c0
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This new client package has 2 main components:
Summary of the Changes for this package:
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
Still WIP, but maybe not necessary for this merge:
Some personal recommendations:
Development Package